chore(mobile): 审计收口——37 死 i18n 键/WS 死类型/mock-hub 旧路由清理 + hub URL 去重 - #2160
Merged
Conversation
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Team Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
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 |
- strings.ts 删 37 个无消费者键(保留 4 个 categoryToStringKey 消费的 generic* 键) - HubWsLegacyEventType 裁掉无生产者的 thread/approval/presence.updated,保留 mock-hub 真实生产的 snapshot/run.updated - mock-hub 删 /v1/threads REST 与 /v1/events WS 旧路径(无消费者,#1422) - app.config issuer 占位默认改空串,构建缺失时显式失败 - App.tsx 复用 app.config resolveDefaultHubBaseUrl,删重复解析 - 顺带清 2 个存量 lint error(未用 catch 参数 / 死 import) Co-authored-by: Cursor <cursor@vectorcontrol.tech>
DeliciousBuding
force-pushed
the
chore/cleanup-round-17
branch
from
August 31, 2026 21:32
a79a3d5 to
c3836b9
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
范围(Dirac P3 mobile 清理,round-17)
i18n/strings.ts:删除 37 个无消费者键(en/zh 同步,key 面 parity 测试保持 325=325)。保留genericNetworkError/genericServerError/genericAuthError/genericUnknownError——被categoryToStringKey()实时消费;messages/thinking确认无t()/strings.消费者。HubWsLegacyEventType只保留 mock-hub 实际生产的snapshot.updated/run.updated,删除无生产者thread.updated/approval.updated/presence.updated(hub/web: WS 死协议面 — sync.request 无服务端实现、agent.regenerate/message.edited 无生产者、plan_* 无 Hub 传输 #1362/chore(ws): 删除无生产者死协议面与坏死 useHubWebSocket #1422 死面禁令),App.tsx 监听同步收敛。scripts/mock-hub.mjs:删除无消费者的/v1/threadsREST 与/v1/eventsWS 升级路径,只留/client/sessions//client/contacts//client/ws。app.config.ts:oidcIssuer占位默认https://id.example.com→ 空串,缺失时登录显式不可用而不是假装成功;导出resolveDefaultHubBaseUrl供 App 复用。验证(本地已跑)
pnpm typecheck✅pnpm test✅ 32 files / 397 testspnpm lint✅ 0 error(余 1 个存量 warning,master 同源)pnpm mock:hub:check✅(health/snapshot/CORS 预检/client 路由//client/ws帧)git diff --check✅