fix(claude): avoid Desktop alias date collisions

[122yjs]

Summary

• Render persisted Claude Desktop profile date slots as deterministic p-prefixed wire IDs (for example, claude-opus-4-8-p01q) so Desktop does not collapse distinct managed routes after stripping terminal YYYYMMDD suffixes.
• Keep persisted date aliases and both historical hash forms decodable for existing profiles and sessions, while preventing compatibility aliases from claiming genuine Anthropic model IDs.
• Reallocate a non-Anthropic persisted date slot if a newly active genuine Anthropic model claims that exact ID, and fail closed on all native/synthetic alias collisions.
• Update the Claude Desktop structure contract, localized Claude Code guides, and regression/integration coverage.

This addresses the active-session model-switch failure investigated in #3782. Existing generated Desktop profiles must be applied again to receive the new wire IDs.

Closes #3782

Verification

• ./node_modules/.bin/bun test tests/clients/desktop-3p.test.ts tests/clients/desktop-profile.test.ts tests/claude-integration/claude-desktop-discovery.test.ts tests/claude-integration/claude-models-discovery.test.ts tests/claude-integration/claude-desktop-picker-profile.test.ts tests/claude-integration/claude-messages-endpoint.test.ts tests/claude-integration/claude-native-passthrough.test.ts tests/claude-integration/claude-management-api.test.ts — 187 passed, 0 failed.
• ./node_modules/.bin/bun run typecheck — passed.
• ./node_modules/.bin/bun run structure:check — passed.
• ./node_modules/.bin/bun run privacy:scan — passed.
• git diff --check — passed.
• docs-site: ./node_modules/.bin/astro build — 521 pages built; 70,385 internal links checked.
• ./node_modules/.bin/bun run test:changed — 24,834 passed and 51 failed while this suite overlapped resource/port-heavy targeted tests. All 12 files containing those failures were then rerun sequentially: 517 passed, 0 failed.
• Independent static security/logic review — passed with no remaining security concerns or logic errors.

Checklist

• Scope stays focused and avoids unrelated cleanup.
• Docs or release notes were updated when needed.
• Security-sensitive changes were reviewed for secrets, auth, and unsafe defaults.

Review readiness checklist

This PR stays in draft until every box below is ticked. Tick all four boxes once the requirements are met:

• Required local validation passed; commands, results, and any full-suite exception are documented.

• I pushed my PR to a recent dev commit (at most 10 behind; a maintainer may still ask for the exact tip before merge).

• I resolved all correct Codex and CodeRabbit findings.

• My PR is ready for review.

[coderabbitai]

Important

Draft PR not reviewed

Draft PRs are not automatically reviewed by default.

• Trigger a manual review

To automatically review draft PRs, update your CodeRabbit configuration:

reviews:
  auto_review:
    drafts: true

---

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

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[github-actions]

✅ Deterministic PR hygiene checks passed.

[github-actions]

⏳ DRAFT

• review readiness checklist open (0/4 boxes ticked).

What to do

• Tick all four boxes in the PR description once you're done (currently 0/4).

Review readiness checklist

• ⬜ Required local validation passed; commands, results, and any full-suite exception are documented.
• ⬜ I pushed my PR to a recent dev commit (at most 10 behind; a maintainer may still ask for the exact tip before merge).
• ⬜ I resolved all correct Codex and CodeRabbit findings.
• ⬜ My PR is ready for review.

0/4 boxes ticked.

This PR stays in draft until every box above is ticked.

[lidge-jun]

리뷰 · 우선순위 51 / 80

이 PR은 Claude Desktop이 모델 이름을 비교할 때 맨 끝의 날짜를 지워서, 서로 다른 모델이 같은 이름으로 보이던 문제(#3782)를 고칩니다. 예전에는 관리되는 경로마다 claude-opus-4-8-20260304처럼 날짜로 된 이름을 Desktop에 그대로 넘겼습니다. Desktop이 20260304를 지우면 전부 claude-opus-4-8이 되고, 이미 켜 둔 대화에서는 모델을 바꿔도 안 바뀌었습니다.

이제는 파일에 저장하는 칸은 날짜 그대로 둡니다. Desktop에 보여줄 때만 claude-opus-4-8-p01q처럼 p로 시작하는 짧은 코드로 바꿉니다. 예전에 저장해 둔 날짜 이름과 3글자 해시 이름은 계속 알아볼 수 있습니다. 진짜 Anthropic 모델 이름과 겹치는 가짜 이름은 등록하지 않습니다. 저장돼 있던 날짜 칸을 진짜 Anthropic 모델이 차지하면, 그 관리 경로는 다른 날짜 칸으로 옮깁니다. 베이스 브랜치는 dev입니다.

이 코드를 넣은 뒤에는 Desktop 프로필을 다시 적용하고, Desktop을 완전히 껐다 켜야 새 이름이 들어갑니다. 이미 돌아가고 있던 작업은 예전 날짜 이름을 붙잡고 있을 수 있습니다.

docs-site/src/content/docs/tr/guides/claude-code.md:253 - 바로 위 문장은 날짜를 Desktop 이름으로 안 내보낸다고 하는데, 253행은 "기존 별칭은 id를 유지한다"고 남아 있습니다. 업그레이드해도 화면 이름이 그대로라는 뜻으로 읽힙니다.

docs-site/src/content/docs/zh-cn/guides/claude-code.md:241 - 표의 예시만 p 코드로 바뀌었습니다. 241행부터는 아직도 선택기가 안 되면 /model을 써 보고, 원인이 어디인지도 모른다고 적혀 있습니다. 영어판은 프로필을 다시 적용하고 Desktop을 재시작하라고 바꿔 두었습니다. ja, ko, ru 가이드도 표만 바뀌고 그 재시작 안내가 없습니다.

src/claude/desktop-3p.ts:197 collectDesktop3pModels - 저장된 날짜가 아니라, 변환된 p 코드가 진짜 Anthropic 이름과 같으면 그 경로를 모델 목록에서 빼 버립니다. 다른 칸으로 다시 배정하지 않습니다. 알림은 console.warn뿐이라, Desktop 목록에서 모델이 빠진 이유를 화면에서는 알기 어렵습니다. 테스트는 이 빼기 동작을 고정해 두었습니다.

메인테이너의 판단이 필요한 지점

p 코드가 진짜 Anthropic 이름과 겹칠 때 그 모델을 목록에서 빼는 것으로 끝낼지, 빈 날짜 칸으로 한 번 더 옮겨 다른 p 코드를 줄지 정해 주세요. 지금은 빼는 쪽이고, 그런 이름이 실제로 나올 가능성은 낮습니다. 이 PR은 아직 드래프트이고 본문 체크리스트는 0/4입니다. 검증 명령은 적혀 있지만 박스는 비어 있습니다.

너의 추천

이름 분리 방향은 #3782의 원인과 맞습니다. 날짜 칸은 파일 안에만 두고, Desktop으로 나가는 이름은 p로 시작하는 코드로 나누면 끝 날짜를 지워도 모델이 서로 다르게 남습니다. 진짜 Anthropic 이름은 예전처럼 그대로 둡니다. 머지 전에 터키어 253행의 남은 문장과, 중국어 241행의 예전 안내를 영어판과 같게 고치세요. ja, ko, ru에도 "프로필을 다시 적용하고 Desktop을 재시작하라"를 한 줄 넣는 편이 좋습니다. 체크리스트를 채우기 전에는 머지하지 마세요.

이 댓글은 grok-bot이 작성했습니다

[lidge-jun]

Thanks! This landed on dev through the bug-PR merge train batch #5904 (merge bc90209). Your change is one commit on dev with you as the author and a Co-authored-by trailer. One dev test (claude-desktop-remote-hub) asserted the stored dated slot as the Desktop wire id; it now expects the p<base36day> id your change emits. Closing since the content is now on dev.