Skip to content

feat: automate codex feedback loop and localize eligibility - #7

Merged
markusleben merged 1 commit into
mainfrom
chore/codex-feedback-loop
Feb 16, 2026
Merged

markusleben merged 1 commit into
mainfrom
chore/codex-feedback-loop

Conversation

@markusleben

@markusleben markusleben commented Feb 16, 2026 •

Copy link
Copy Markdown
Owner

Summary

  • add OpenUsage-style PR review playbook command at .cursor/commands/pr-review.md
  • add codex-feedback-loop workflow to automatically request codex follow-up on codex review findings, with /codex-fix and manual dispatch support
  • fix codex review finding: pass selected app language to get_apple_intelligence_eligibility and localize backend eligibility reasons
  • update contributing docs and decision/spec breadcrumbs

Verification

  • cd src-tauri && cargo fmt && cargo test --quiet
  • npm test
  • npm run build
  • pre-push gate (clippy + coverage + tsc) passed

@markusleben
markusleben merged commit deecb0a into main Feb 16, 2026
3 checks passed
@markusleben
markusleben deleted the chore/codex-feedback-loop branch February 16, 2026 13:59

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 8315180271

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

PR_NUMBER: ${{ steps.ctx.outputs.pr_number }}
REVIEW_ID: ${{ steps.ctx.outputs.review_id }}
run: |
MARKER="<!-- codex-autofix:pr-${PR_NUMBER}:review-${REVIEW_ID:-latest} -->"

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Deduplicate manual reruns by concrete review id

When /codex-fix (or workflow_dispatch without review_id) runs, REVIEW_ID is empty, so this marker falls back to review-latest; after the first manual autofix comment is posted, every later manual trigger on the same PR is suppressed, even if new Codex findings were added. This breaks the documented manual retry path because dedupe is keyed to a static token instead of the current review context.

Useful? React with 👍 / 👎.

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