Thanks to visit codestin.com
Credit goes to github.com

Skip to content

docs: refresh the lib knowledge-base tree for the extraction era - #589

Merged
ndycode merged 2 commits into
mainfrom
claude/audit-70-lib-docs-refresh
Jun 11, 2026
Merged

ndycode merged 2 commits into
mainfrom
claude/audit-70-lib-docs-refresh

Conversation

@ndycode

@ndycode ndycode commented Jun 11, 2026

Copy link
Copy Markdown
Owner

Summary

What Changed

Validation

  • npm run typecheck (pre-commit hook)
  • npm test -- test/documentation.test.ts — passes
  • Every added path verified to exist on current main
  • npm test / npm run build — not run for this markdown-only change

Docs and Governance Checklist

  • No user-visible behavior changed; contributor-facing knowledge base only

Risk and Rollback

  • Risk level: minimal — markdown only.
  • Rollback plan: revert the single commit.

https://claude.ai/code/session_01XNtnkLbBiXZxfQQYLMpucB


Generated by Claude Code

note: greptile review for oc-chatgpt-multi-auth. cite files like lib/foo.ts:123. confirm regression tests + windows concurrency/token redaction coverage.

Greptile Summary

refreshes lib/AGENTS.md to reflect the module layout created during the §4.1 extraction era. all added paths have been verified to exist on main.

  • structure tree gains codex-manager/formatters/, policy/, request/ sub-entries, and runtime/rotation-*.ts (all five helpers including token-refresh), plus four new top-level entries: fs-retry.ts, temp-path.ts, budget-guard.ts, local-bridge.ts.
  • WHERE TO LOOK gains "retry policies" → fs-retry.ts and "temp/staging paths" → temp-path.ts; errors.ts description is updated to name the CodexError hierarchy.

Confidence Score: 5/5

markdown-only change; every added path verified to exist on main; no runtime behavior touched.

all documented paths exist, rotation-*.ts now covers all five helpers including token-refresh, and withRetry/withRetrySync exports match the WHERE TO LOOK entry. no code changed.

no files require special attention.

Important Files Changed

Filename Overview
lib/AGENTS.md docs-only update; all added paths verified to exist on main; rotation-*.ts description now covers all five helpers including token-refresh

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    LIB[lib/] --> CM[codex-manager/]
    LIB --> POL[policy/]
    LIB --> REQ[request/]
    LIB --> RT[runtime/]
    LIB --> TOP[top-level modules]

    CM --> FMT[formatters/ ← NEW\ntext/account/model/quota]
    CM --> SCMD[commands/]

    POL --> RPT[runtime-policy.ts ← NEW]

    REQ --> HLP[helpers/ ← NEW\nmodel-map, input/tool utils]
    REQ --> RLD[rate-limit-decision.ts ← NEW]
    REQ --> SFR[stream-failover-runtime.ts ← NEW]

    RT --> ROT[rotation-*.ts ← NEW\nselection/state/storage-meta\nserver-type/token-refresh]

    TOP --> FSR[fs-retry.ts ← NEW\nwithRetry / withRetrySync]
    TOP --> TMP[temp-path.ts ← NEW\ncrypto-random staging paths]
    TOP --> BGD[budget-guard.ts ← NEW]
    TOP --> LBR[local-bridge.ts ← NEW]
    TOP --> ERR[errors.ts\nCodexError hierarchy]
Loading

Reviews (2): Last reviewed commit: "docs: include token-refresh in the rotat..." | Re-trigger Greptile

Context used:

  • Context used - lib/AGENTS.md (source)

The §4.1 extractions and companion fixes added module clusters the
curated STRUCTURE tree never picked up: codex-manager/formatters/,
request/helpers/ + rate-limit-decision + stream-failover-runtime, the
runtime rotation-* helpers, policy/, fs-retry, temp-path, budget-guard,
and local-bridge. Adds them in tree style, points WHERE TO LOOK at the
shared retry and temp-path helpers, and names the CodexError hierarchy
in the errors.ts row.

https://claude.ai/code/session_01XNtnkLbBiXZxfQQYLMpucB
@chatgpt-codex-connector

Copy link
Copy Markdown

Codex usage limits have been reached for code reviews. Please check with the admins of this repo to increase the limits by adding credits.
Credits must be used to enable repository wide code reviews.

@coderabbitai

coderabbitai Bot commented Jun 11, 2026

Copy link
Copy Markdown
Contributor

Warning

Review limit reached

@ndycode, we couldn't start this review because you've reached your PR review rate limit.

More reviews will be available in 4 minutes and 7 seconds. Learn how PR review limits work.

Your organization has run out of usage credits. Purchase more credits in the billing tab to continue.

⌛ How to resolve this issue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans include higher PR review limits than trial, open-source, and free plans. In all cases, reviews become available again over time. During sustained high-volume PR review activity, CodeRabbit may temporarily slow when the next review becomes available.

Please see our Fair Usage Limits Policy for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: cc56af3b-a99c-4b52-8612-cdd0d3b9f7b5

📥 Commits

Reviewing files that changed from the base of the PR and between 1d6c3e0 and d455944.

📒 Files selected for processing (1)
  • lib/AGENTS.md
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch claude/audit-70-lib-docs-refresh
✨ Simplify code
  • Create PR with simplified code
  • Commit simplified code in branch claude/audit-70-lib-docs-refresh

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Comment thread lib/AGENTS.md Outdated
ndycode added a commit that referenced this pull request Jun 11, 2026
docs: list login-menu-accounts beside its login-menu siblings
@ndycode
ndycode merged commit 07f9adc into main Jun 11, 2026
2 checks passed
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.

2 participants