Conversation
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars. ⚙️ Run configurationConfiguration used: Repository: luvs01/opencodex/.coderabbit.yaml Review profile: ASSERTIVE Plan: Advanced Run ID: 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. Comment |
![devin-ai-integration[bot]](https://avatars.githubusercontent.com/in/811515?s=60&v=4){kind=small}
This comment was marked as resolved.
This comment was marked as resolved.
Sorry, something went wrong.
|
✅ Deterministic PR hygiene checks passed. |
A Codex account with the id "auto" wins exact-id precedence, leaving
"ocx account use openai auto" unable to clear the pin. The new verb
sends {accountId: null} without resolving a selector, and the CLI
reference gains the precedence note in all locales.
Co-Authored-By: Epinephrine <luvs01@hanmail.net>
This comment was marked as resolved.
This comment was marked as resolved.
Sorry, something went wrong.
cli-account.test.ts sits exactly at its file-size ratchet cap (2313); the new test pushed it to 2336 and failed 'file-size ratchet: repository'. cli-account-alias-target.test.ts is the sibling kept out of the #180 matrix file for this reason, so the case moves there byte-for-byte semantics, ported to that file's harness. Co-Authored-By: Devin AI <devin-ai-integration[bot]@users.noreply.github.com>
This comment was marked as resolved.
This comment was marked as resolved.
Sorry, something went wrong.
…bservable state Co-Authored-By: Epinephrine <luvs01@hanmail.net>
|
이관됨: lidge-jun#6050 |
|
동일 수정이 상류 저장소에 제출되어 이 포크 PR의 목적은 달성됐습니다. |
|
Devin is currently unreachable - the session may have died. |
1 participant
Motivation
autowas being interpreted as the CLI control word and cleared the active selection instead of selecting that account.auto.Description
src/cli/account-target.tsto resolve exact stored account ids before treating CLI control words as reserved, and add anunavailableerror kind for safe failure when the account list cannot be fetched.auto→nullmapping and instead mapautotonullonly when resolution indicates it is the reserved selector, preserving literalautoids when present.tests/cli/cli-account-alias-target.test.tsto assert that an account with idautois selectable and manageable and that a missing account list causes a fail-closed error for reserved selectors.structure/providers/openai-accounts.mdto document the exact-ID precedence and the CLI'sautoresolution semantics.Testing
node_modules/.bin/bun test tests/cli/cli-account-alias-target.test.tsand the suite passed (21 tests passed).node_modules/.bin/bun test tests/cli/cli-account-pool-verbs.test.tsand the suite passed (30 tests passed).node_modules/.bin/bun run typecheck,node_modules/.bin/bun run structure:check, andnode_modules/.bin/bun run privacy:scan, all of which succeeded.bun testwith the globally installed Bun 1.2.14 produced a tooling error unrelated to the change, while the repository-pinnednode_modules/.bin/bun(1.4.0) ran the focused tests successfully.Codex Task