Skip to content

Route monthly optimization tasks to Codex bridge#49

Merged
Pigbibi merged 2 commits into
mainfrom
monthly-codex-remediation-bridge
May 10, 2026
Merged

Route monthly optimization tasks to Codex bridge#49
Pigbibi merged 2 commits into
mainfrom
monthly-codex-remediation-bridge

Conversation

@Pigbibi

@Pigbibi Pigbibi commented May 10, 2026

Copy link
Copy Markdown
Collaborator

Summary

This PR routes the normal CryptoSnapshotPipelines monthly code-remediation path through the self-hosted VPS ccbot/Codex bridge instead of the GitHub-hosted Claude Action path.

  • label low-risk non-experiment [auto-pr-safe] Crypto monthly optimization task issues with codex-bridge and auto-merge-ok
  • add an explicit Codex bridge contract to generated task issues, including branch, PR marker, draft-first, test, and guarded-file requirements
  • keep the existing Claude Action workflow as a manual-dispatch/non-Codex fallback by skipping issues already queued for codex-bridge
  • extend the post-CI merge gate to recognize codex/monthly-optimization-issue-* branches and require the PR-level auto-merge-ok label for Codex PRs
  • add Codex PR Feedback so failed CI or changes-requested reviews are posted back to the source codex-bridge issue for another ccbot/Codex pass
  • document the ccbot/Codex remediation flow in README and the operator runbook

Validation

  • python3 -m pytest tests/test_fanout_monthly_optimization_tasks.py tests/test_auto_optimization_pr_workflow_config.py tests/test_prepare_auto_optimization_pr.py -q
  • git diff --check
  • GitHub Actions CI test: passing on ed88f8d

Refs QuantStrategyLab/AuditOrchestrator#1

@Pigbibi Pigbibi force-pushed the monthly-codex-remediation-bridge branch from cd7c7fd to ed88f8d Compare May 10, 2026 02:01
@Pigbibi Pigbibi marked this pull request as ready for review May 10, 2026 02:04
@Pigbibi Pigbibi merged commit 23d73aa into main May 10, 2026
1 check passed
@Pigbibi Pigbibi deleted the monthly-codex-remediation-bridge branch May 10, 2026 02:06

@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: ed88f8da6e

ℹ️ 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".

Comment on lines +61 to +64
and task_level_allowed
and not pr.get("isDraft")
and guard["allowed"]
and (not codex_branch or has_auto_merge_label)

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P1 Badge Remove undocumented task-level gate for Codex PR merges

The Codex branch path is now routed through this gate, but should_merge still requires task_level_allowed from a literal PR-body string ("Task-level auto-merge eligible: yes"). The new Codex contract text generated in scripts/fanout_monthly_optimization_tasks.py does not instruct Codex to include that marker, so a Codex PR can satisfy the documented branch/marker/label rules and still be permanently skipped with task_level_guard. This effectively blocks the new Codex auto-merge flow unless out-of-band behavior adds that exact line.

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