feat(provider): add Qoder CN PAT provider - #3350
feat(provider): add Qoder CN PAT provider#3350Flowershangfromthebranches wants to merge 5 commits into
Conversation
|
Important Draft PR not reviewedDraft PRs are not automatically reviewed by default.
To automatically review draft PRs, update your CodeRabbit configuration: reviews:
auto_review:
drafts: trueThanks 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 |
|
✅ Deterministic PR hygiene checks passed. |
⏳ DRAFT
What to do
Review readiness checklist
0/4 boxes ticked. This PR stays in draft until every box above is ticked. |
리뷰 · 우선순위 47 / 80이 PR은 Qoder CN을 Global과 분리된 공식 PAT + 공식 CLI 프로바이더로 넣는 작업이다. 프로바이더 id는 CN 전용 커밋 자체는 작다. 새 어댑터 파일을 만들지 않고 선행 작업 라이브 증거는 Global보다 조금 더 있다. 설치된 CN CLI는 정책·릴리즈 위치는 경로 메인테이너의 판단이 필요한 지점
너의 추천 이 댓글은 grok-bot이 작성했습니다 |
Summary
Adds Qoder CN as a separate official PAT-backed CLI provider.
This is PR 3 of 3. It is intentionally stacked on #3349, which is stacked on #3340, until the preceding shared-runtime PRs merge. The CN-only commit is
a4e805084. The branch will be rebased after its predecessors merge so this PR's final diff remains CN-only.Contract and isolation
qoder-cnhttps://qoder.cnQODERCN_PERSONAL_ACCESS_TOKENqodercnwith legacy vendor binary nameqoderclicnacceptednpm install -g @qodercn-ai/qoderclicn--list-modelsdiscovery is account-authoritative; static CN roster is degraded fallbackNo desktop session, browser cookie, OpenCodex OAuth token, ambient
~/.qoder-cncredential, private endpoint, or embedded signing binary is read or used.Prior work: #3010
Credit to Liang Xu (
Liang-Psych) for the Qoder CN exploration in #3010, especially identifying the official CLI, headless stream-json direction, and tools-disabled execution.This PR does not copy #3010's implementation. It deliberately replaces its OAuth/private-protocol and ambient-session designs with the now-documented
QODERCN_PERSONAL_ACCESS_TOKENcontract and the shared coding-agent CLI adapter. It also avoids model slug guessing, embedded WASM, prompt-in-argv, inherited full process environments, fabricated usage, and success-on-truncated-stream behavior called out during #3010 review.Primary evidence reviewed (2026-09-03)
Verification
bun run typecheck: passedbun run privacy:scan: passedgit diff --check: passedLive evidence and boundary
The installed CN CLI is
qoderclicn 1.1.34. Its authenticated local session returned this roster on 2026-09-03:A tools/settings-disabled headless turn reached the CN service and returned vendor code 118 with zero available credits. This is evidence for the local CLI transport/model route, not for the new PAT injection path.
LIVE_PAT_E2E_NOT_VERIFIED: no separate Qoder CN PAT was available in the test environment, and no successful inference is claimed.
Maintainer decisions requested
Review readiness checklist
This PR stays in draft until every box below is ticked. Tick all four boxes once the requirements are met:
All CI tests are green on my local testing.
I pushed my PR to the latest dev commit.
I resolved all correct Codex and CodeRabbit findings.
My PR is ready for review.