Skip to content

fix(code-review): protect review artifact directories#10

Open
davidalee wants to merge 1 commit intomainfrom
fix/code-review-artifact-permissions
Open

fix(code-review): protect review artifact directories#10
davidalee wants to merge 1 commit intomainfrom
fix/code-review-artifact-permissions

Conversation

@davidalee
Copy link
Copy Markdown
Owner

Summary

  • chmod ce-code-review run artifact directories to owner-only permissions after creation
  • document why umask is insufficient for per-reviewer findings/evidence artifacts
  • replace stale .context wording with the actual OS temp artifact path in the subagent template

Verification

  • bun test tests/review-skill-contract.test.ts --grep "ce-code-review contract"

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant