Skip to content

docs(agents): record the reviewer quota fallback for pull requests - #433

Merged
ScriptedAlchemy merged 1 commit into
mainfrom
docs/agents-review-quota-fallback
Sep 3, 2026
Merged

ScriptedAlchemy merged 1 commit into
mainfrom
docs/agents-review-quota-fallback

Conversation

@ScriptedAlchemy

@ScriptedAlchemy ScriptedAlchemy commented Sep 3, 2026 •

Copy link
Copy Markdown
Owner

Summary

Final-sweep verification of the Changesets/PR process infrastructure on main found one documentation gap: AGENTS.md "Pull requests" carries the @codex review re-request and head-SHA confirmation rule but not the quota fallback every lane applied tonight (connector answers usage limits reached → record last-reviewed / unreviewed SHAs in the PR body and merge on green CI). This PR adds that bullet.

Verified as already complete on main (no change needed): .changeset/config.json (private packages ignored via ignore + privatePackages, baseBranch: main, tests/** exempt), .github/workflows/changeset.yml (Changeset present check with the skip-changeset label path and the release-branch exemption scoped to this repository), .github/workflows/release.yml (changesets/action@v2, publish-script empty unless AGENT_BUNDLE_NPM_PUBLISH=true and NPM_TOKEN set — publishing gated off by default; release gates run on the Version Packages merge), .changeset/README.md conventions, and the skip-changeset label.

Two gaps that no PR can close (repository settings; reported to the maintainer in the sweep report): main has no branch protection or ruleset, so Changeset present is not a required check; and CHANGESETS_GITHUB_TOKEN is not set, so the Version Packages PR (#411) gets no PR CI (documented fallback: close/reopen).

Evidence

  • gh api repos/ScriptedAlchemy/agent-bundle/branches/main/protection → 404 "Branch not protected"; gh api repos/.../rules/branches/main → [].
  • gh pr view 411 --json statusCheckRollup → [].

Test plan

  • Docs-only change; no gates apply beyond the Changeset present check passing without a changeset.

Review status

  • @codex review requested at 16:51 UTC for head 8a90b3e03dca1853de742d82f128f708fc7b1bf7; the connector answered usage limits reached (twice). Unreviewed SHA: 8a90b3e03dca1853de742d82f128f708fc7b1bf7. Merged on green CI under the quota fallback this PR documents.

@changeset-bot

changeset-bot Bot commented Sep 3, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 8a90b3e

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@ScriptedAlchemy

Copy link
Copy Markdown
Owner Author

@codex review

@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.
To continue using code reviews, add credits to your account and enable them for code reviews in your settings.

1 similar comment
@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.
To continue using code reviews, add credits to your account and enable them for code reviews in your settings.

@pkg-pr-new

pkg-pr-new Bot commented Sep 3, 2026

Copy link
Copy Markdown
npm i https://pkg.pr.new/ScriptedAlchemy/agent-bundle@433
npm i https://pkg.pr.new/ScriptedAlchemy/agent-bundle/create-agent-bundle@433
npm i https://pkg.pr.new/ScriptedAlchemy/agent-bundle/@agent-bundle/runtime@433

commit: 8a90b3e

@ScriptedAlchemy
ScriptedAlchemy merged commit b90ba71 into main Sep 3, 2026
13 checks passed
@ScriptedAlchemy
ScriptedAlchemy deleted the docs/agents-review-quota-fallback branch September 3, 2026 22:36
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