Skip to content

feat(ci): implement automated code review via GitHub Actions#105

Merged
pentaxis93 merged 2 commits intomainfrom
issue-101/feat-ci-implement-automated-code-review
Mar 9, 2026
Merged

feat(ci): implement automated code review via GitHub Actions#105
pentaxis93 merged 2 commits intomainfrom
issue-101/feat-ci-implement-automated-code-review

Conversation

@pentaxis93
Copy link
Owner

@pentaxis93 pentaxis93 commented Mar 9, 2026

Summary

  • add a GitHub Actions workflow that runs codex exec review --base <branch> --json --output-last-message on pull request events
  • post line-anchored inline PR review comments from Codex findings, with a fallback summary comment for unpostable findings
  • codify repository-specific review criteria in AGENTS.md so Codex behavior is governed in-repo

Changes

  • new .github/workflows/code-review.yml:
    • triggers on opened, synchronize, reopened, and ready_for_review
    • uses openai/codex-action@v1 to install/configure Codex and proxy with read-only sandboxing
    • executes codex exec review directly (built-in review mode)
    • parses generated review output and posts inline comments via actions/github-script
  • updated AGENTS.md with a Codex Review Guidelines section focused on correctness, regression, security, and actionable findings

Issue(s)

Closes #101

Test plan

  • local parse check: workflow YAML parses successfully
  • local syntax check: embedded github-script JavaScript parses successfully
  • CI smoke (in PR context):
    • verify workflow triggers automatically on PR events
    • verify Codex review runs in read-only sandbox
    • verify inline comments are posted with path + line anchors

@pentaxis93
feat(ci): add codex review workflow for pull requests
fd08c87 
Add GitHub Actions automation that runs codex exec review on PR events and posts findings as inline comments.

Document project-specific review guidance in AGENTS.md for Codex to follow.

Refs #101
@pentaxis93
fix(ci): remove unsupported codex exec approval flag
6800c20 
Drop --ask-for-approval from codex exec review invocation because current CLI rejects the argument in this command path.

Refs #101
@pentaxis93 pentaxis93 merged commit 83d252b into main Mar 9, 2026
1 check passed
@pentaxis93 pentaxis93 deleted the issue-101/feat-ci-implement-automated-code-review branch March 9, 2026 09:57
@pentaxis93 pentaxis93 mentioned this pull request Mar 9, 2026
Closed
11 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

feat(ci): implement automated code review via GitHub Actions

1 participant

@pentaxis93