Skip to content

chore(idd): add chatgpt-codex-connector to advisoryBotLogins - #378

Merged
kurone-kito merged 1 commit into
masterfrom
issue/376-chore-idd-add-chatgpt-codex-connector
Sep 4, 2026
Merged

kurone-kito merged 1 commit into
masterfrom
issue/376-chore-idd-add-chatgpt-codex-connector

Conversation

@kurone-kito

Copy link
Copy Markdown
Owner

Summary

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, since disposition-non-review-notices only auto-handles logins already present in advisoryBotLogins. idd-pre-merge.instructions.md already names "a Codex connector" as a typical example of a non-Copilot advisoryBotLogins entry, so this closes that gap for future PRs.

This is purely additive/config-scoped: advisoryWait.primaryBotLogin/secondaryBotLogin remain 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.json remains valid JSON with only advisoryBotLogins changed; no other keys touched.
  • git diff master...HEAD touches only .github/idd/config.json and docs/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) exits 0 from a clean worktree.
  • tests/bash/helpers/bats-core/bin/bats tests/bash/ and pwsh -c "Invoke-Pester tests/powershell/ -Output Detailed -CI" (pre-push-validate) both pass.
  • The premise fact cited in the issue (that 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 at 2026-09-04T07:35:09Z) was independently re-verified against live GitHub state before implementation.

🤖 Generated with Claude Code

https://claude.ai/code/session_01NNdkbvQS3Tc8jAD1Vr6Ymz

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
Copilot AI lite review requested due to automatic review settings September 4, 2026 09:47
@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@coderabbitai

This comment has been minimized.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟢 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.json advisoryBotLogins with "chatgpt-codex-connector[bot]".
  • Update docs/idd-policy.md to 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.

@kurone-kito

This comment has been minimized.

@kurone-kito

This comment has been minimized.

@kurone-kito

Copy link
Copy Markdown
Owner Author

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)

@kurone-kito

Copy link
Copy Markdown
Owner Author

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)

@kurone-kito

This comment has been minimized.

@kurone-kito
kurone-kito merged commit e66bd94 into master Sep 4, 2026
16 checks passed
@kurone-kito
kurone-kito deleted the issue/376-chore-idd-add-chatgpt-codex-connector branch September 4, 2026 09:53
@kurone-kito

Copy link
Copy Markdown
Owner Author

F4 Cleanup Evidence

Field Value
Status applied
Applied 3
Failed 0
Skipped 3
Permission-blocked 0
Retry attempts (bound-exhausted) 1 (false)
Notes Skipped items: chatgpt-codex-connector[bot]'s non-review notice (no completed-review signal to minimize as RESOLVED), Copilot's review body (no associated review threads), and the already-minimized first watermark comment.

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.

chore(idd): add chatgpt-codex-connector to advisoryBotLogins

2 participants