docs: 2-tier architecture, canonical ownership, fact sync#17
Conversation
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
|
@codex review |
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 5c3da00030
ℹ️ 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".
| - After a new push, a human must post the native trigger comment | ||
| (`@codex review`, `/gemini review`, or `@claude review once`), or run | ||
| `pnpm run review:switch -- --to <agent>`. Bot-posted triggers are rejected |
There was a problem hiding this comment.
This wording implies manual triggers are only needed after subsequent pushes, but with codex as the selected reviewer the first PR review is not automatic: .github/workflows/ai-review.yml sets trigger_mode="skip" on pull_request events, and in skip mode scripts/ai-review-gate.mjs polls for an existing Codex review instead of posting @codex review. On a newly opened PR, that can leave AI Review waiting until timeout unless a human posts the trigger immediately, so the workflow doc should explicitly include the initial PR review case.
Useful? React with 👍 / 👎.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
|
@codex review |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 5f216d120a
ℹ️ 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".
|
|
||
| - implementation: `claude` | ||
| - review: `gemini` | ||
| - review: `codex` (switched from `gemini` on 2026-04-17) |
There was a problem hiding this comment.
This line now declares codex as the repository default reviewer, but the automation still falls back to gemini when AI_REVIEW_AGENT is unset (.github/workflows/ai-review.yml sets selected="gemini" when the variable is empty, and .github/workflows/ai-command-policy.yml uses defaults.review = "gemini"). In environments where the variable is missing (for example a fresh fork or accidental variable deletion), operators following this doc will trigger @codex review while the AI Review gate is actually polling Gemini, which can lead to avoidable timeout/failure behavior.
Useful? React with 👍 / 👎.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
|
@codex review |
|
Codex Review: Didn't find any major issues. Keep them coming! ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
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". |
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
|
@codex review |
|
Codex Review: Didn't find any major issues. Swish! ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
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". |
1 participant
Summary
CLAUDE.md+AGENTS.md);constitution.mdanddocs_pallete_maker/README.mdbecome content/index files, not entry points.project-idea.md; grid columns →frontend-docs.md; review policy →ai-orchestration-protocol.md; Standard Feature Loop →constitution.md; bot-trigger rejection →review-trigger-automation.md; OMC modes →CLAUDE.md.up to 15 (12 chromatic + 3 achromatic)fromharmony.mjs:72-73; grid11 cols ≥1280pxfromindex.html:221; default review backendcodex(switched 2026-04-17); Claude review marked non-operational (dead-code pending cleanup).macos-local-runners.md(AGENTS.md rule 4, constitution.md).> Audience: ... | Prereq: ... | Next: ...) to all 6 devops docs.docs_pallete_maker/adr/0001-review-trigger-design.md; shrinksreview-trigger-automation.mdto active Tier 1 only.Net
-70lines; duplicated blocks consolidated to single canonical homes with cross-references.Test plan
pnpm run preflight— feature-memory gate + baseline + html-validate + build + format + 59/59 tests greenbaseline-checks,guard,AI Reviewgreen on PR@codex review)🤖 Generated with Claude Code