Skip to content

Add communication guidelines and PR review skill#373

Merged
ivan-digital merged 3 commits into
mainfrom
codex/agent-communication-review
Jul 4, 2026
Merged

Add communication guidelines and PR review skill#373
ivan-digital merged 3 commits into
mainfrom
codex/agent-communication-review

Conversation

@ivan-digital

@ivan-digital ivan-digital commented Jul 4, 2026

Copy link
Copy Markdown
Member

Summary

  • Add communication guidance to lead with conceptual status, risk, tests, and recommendation.
  • Add a /review-pr project skill for architecture fit, adversarial review, security review, docs impact, regression risk, test coverage, and merge readiness.
  • Require review summaries to report E2E tests run, or explain why E2E does not apply.
  • Add repo-local .codex/skills relative symlinks so agents that scan that path can use the same project skills from .claude/skills.
  • Add the new review command to the project skills table.

Validation

  • git diff --check
  • Parsed the new skill frontmatter locally
  • Verified .codex/skills symlinks resolve locally

Runtime tests were not run because this is guidance-only.

@ivan-digital ivan-digital marked this pull request as ready for review July 4, 2026 17:57
@ivan-digital ivan-digital merged commit 6a4b8f0 into main Jul 4, 2026
1 check passed
@ivan-digital ivan-digital deleted the codex/agent-communication-review branch July 6, 2026 18:53
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