Skip to content

feat(skills): support bidirectional PR reviews - #949

Open
seoseo-ai wants to merge 1 commit into
mainfrom
agent/gh-pr-flow-review-profiles
Open

feat(skills): support bidirectional PR reviews#949
seoseo-ai wants to merge 1 commit into
mainfrom
agent/gh-pr-flow-review-profiles

Conversation

@seoseo-ai

Copy link
Copy Markdown
Collaborator

Summary

  • consolidate Seoseo-held cross-account approval into an allowlisted seoseo-ai|jinon86 review-profile helper
  • retain the existing seoseo-ai entry point and add a fixed jinon86 compatibility wrapper
  • enforce exact-head, opposite-author, requested-reviewer, green-check, and owner-only credential gates in both directions
  • add reverse-direction security and failure-path coverage to the harness

Validation

  • bash scripts/gh-pr-flow-seoseo-ai.test.sh (PASS=10 FAIL=0)
  • bash scripts/gh-pr-flow-jinon86.test.sh (PASS=12 FAIL=0)
  • bash scripts/validate-harness.test.sh (PASS=5 FAIL=0)
  • PYTHONPATH=scripts python scripts/ccc_codex_skills_test.py (11 tests passed)
  • isolated managed-skill apply verified all three helper scripts are installed
  • skill quick validation: Skill is valid!
  • bash -n and git diff --check

Local full-harness note: the new review-profile tests pass; the Termux run also reports unrelated existing doctor verdict and observability spool/archive failures. CI is left to validate the repository-wide Linux environment.

@seoseo-ai
seoseo-ai requested a review from jinon86 as a code owner August 5, 2026 04:20
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.

2 participants