Repository navigation
chore(idd): add chatgpt-codex-connector to advisoryBotLogins - #378
Conversation
During the IDD loop for #373, chatgpt-codex-connector[bot] posted a Codex usage-limit notice on PR #374 that had to be classified and dispositioned by hand, since disposition-non-review-notices only auto-handles logins already in advisoryBotLogins. Register it, and record docs/idd-policy.md's confirming evidence alongside the two existing logins. Refs #376 Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01NNdkbvQS3Tc8jAD1Vr6Ymz
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
This comment has been minimized.
This comment has been minimized.
There was a problem hiding this comment.
🟢 Approval recommended
The change is a small, clearly-scoped additive update to documented policy/config with consistent citations and no behavioral risk beyond recognizing an additional bot login.
Pull request overview
Adds chatgpt-codex-connector[bot] as an advisory bot identity in the IDD policy/config so the existing non-review-notice disposition tooling can automatically recognize and handle its comments.
Changes:
- Extend
.github/idd/config.jsonadvisoryBotLoginswith"chatgpt-codex-connector[bot]". - Update
docs/idd-policy.mdto document the new login and cite PR #374 as the confirming evidence.
File summaries
| File | Description |
|---|---|
| docs/idd-policy.md | Documents the expanded advisoryBotLogins list and adds PR #374 as evidence for the Codex connector login. |
| .github/idd/config.json | Adds chatgpt-codex-connector[bot] to advisoryBotLogins (JSON remains valid; no other keys changed). |
Review details
- Files reviewed: 2/2 changed files
- Comments generated: 0
- Review effort level: Lite
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
|
Rejected — chatgpt-codex-connector[bot] did not review HEAD 90f7bf5 (reached its Codex usage limits for code reviews); this is not a completed review (source: #issuecomment-5538710211) |
|
Rejected — coderabbitai[bot] did not review HEAD 90f7bf5 (hit its review-capacity rate limit for this organization); this is not a completed review (source: #issuecomment-5538711455) |
This comment has been minimized.
This comment has been minimized.
|
F4 Cleanup Evidence
|
Summary
"chatgpt-codex-connector[bot]"to.github/idd/config.json'sadvisoryBotLoginsarray, alongside the existingcopilot-pull-request-reviewer[bot]andcoderabbitai[bot]entries.docs/idd-policy.md's "Advisory Bot Logins" section to record the new login and cite PR fix(mise): pin @playwright/cli to 0.1.18 pending upstream OIDC provenance fix #374 as its confirming evidence, matching the section's existing citation style for the other two logins (confirmed against PR feat(pwsh): declarative winget package dirs in user path #193).Closes #376
Background
During the IDD loop for #373,
chatgpt-codex-connector[bot]posted a Codex usage-limit notice on PR #374 that had to be classified and dispositioned by hand, sincedisposition-non-review-noticesonly auto-handles logins already present inadvisoryBotLogins.idd-pre-merge.instructions.mdalready names "a Codex connector" as a typical example of a non-CopilotadvisoryBotLoginsentry, so this closes that gap for future PRs.This is purely additive/config-scoped:
advisoryWait.primaryBotLogin/secondaryBotLoginremain unset, so the Copilot-only settle/wait window is unaffected — the new login is only dispositioned on arrival (E6's non-review-notice rule), never gates the advisory-wait clock.Verification
.github/idd/config.jsonremains valid JSON with onlyadvisoryBotLoginschanged; no other keys touched.git diff master...HEADtouches only.github/idd/config.jsonanddocs/idd-policy.md.npx markdownlint-cli2 --fix && npx markdownlint-cli2 && npx cspell lint "**" --no-progress(fix-validate, now including the cspell check added by chore(idd): add cspell to fix-validate so pre-push catches what CI's lint.yml catches #375) exits0from a clean worktree.tests/bash/helpers/bats-core/bin/bats tests/bash/andpwsh -c "Invoke-Pester tests/powershell/ -Output Detailed -CI"(pre-push-validate) both pass.chatgpt-codex-connector[bot], a Bot-typed account, posted a usage-limit comment on PR fix(mise): pin @playwright/cli to 0.1.18 pending upstream OIDC provenance fix #374 at2026-09-04T07:35:09Z) was independently re-verified against live GitHub state before implementation.🤖 Generated with Claude Code
https://claude.ai/code/session_01NNdkbvQS3Tc8jAD1Vr6Ymz