Skip to content

fix(macos): 拆分超限的 AppModel+CodeNavigation 扩展以恢复 verify-core 门禁 - #1760

Merged
1lck merged 1 commit into
previewfrom
fix/appmodel-code-navigation-size
Nov 5, 2026
Merged

1lck merged 1 commit into
previewfrom
fix/appmodel-code-navigation-size

Conversation

@1lck

@1lck 1lck commented Nov 5, 2026

Copy link
Copy Markdown
Owner

原因

#1730 合入后 AppModel+CodeNavigation.swift 达到 627 行,超过 verify-service-boundaries.sh 对单个 AppModel 扩展 600 行的上限,导致所有 PR 的 CI 失败。

拆分方式

把前进/后退历史(canNavigateBack/Forward、navigateBack/Forward)以及编辑器位置跳转(navigateToEditorLocation、navigate(to:recordsHistory:replaysHistory:)、私有的 currentEditorNavigationLocation)原样移到同目录新文件 AppModel+NavigationHistory.swift。原文件 502 行,新文件 134 行。

行为

纯代码移动,未改行为、签名和访问级别(私有函数与其唯一调用者同在新文件内)。未改动门禁脚本,也没有加豁免。

验证

  • verify-core.sh、verify-service-boundaries.sh 通过
  • node scripts/i18n/verify-app-language-localization.mjs 通过
  • swift build(macos/)通过
  • 未运行 test-macos.sh 子集

🤖 Generated with Claude Code

@ghfind-review ghfind-review Bot added the review: high ghfind author score; see https://ghfind.com label Nov 5, 2026
将前进/后退历史与编辑器位置跳转部分原样移到 AppModel+NavigationHistory.swift,
不改行为、签名与可见性。

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@1lck
1lck force-pushed the fix/appmodel-code-navigation-size branch from f7c0c4d to a38c9b0 Compare November 5, 2026 09:15
@1lck

1lck commented Nov 5, 2026

Copy link
Copy Markdown
Owner Author

本地验证(代替 GitHub macOS / Windows CI)

本 PR 与 #1760、#1755、#1753、#1759、#1756、#1758、#1757 一起,在本地集成分支 integration/issue-1662-round1 上按合并顺序集成后,按 ci-macos.yml / ci-windows.yml 的步骤在维护者的 Mac 和 Windows 机器上完整验证,未使用 GitHub 的 macOS / Windows runner。

macOS(本机,macOS 26.6 arm64)

Windows(lithe-dev,Windows Server)

  • 通过:依赖安装、verify-windows-boundaries.ps1、共享编辑器测试、PHP 插件构建与隔离验证、ci-windows.yml 中各组 bun 测试、dist 验证、Rust 测试(shared、shared-hosts、MCP stdio、agent-host、git-host、ide-host、project、terminal、windows)。
  • 发现并已修复的回归(均为测试替身未跟上新的 Core 契约,生产代码无回归):bun run typecheck 3 个 TS 错误与 3 个 agent 测试文件(feat(core): #1662 车道 F Agent/AI 下沉(AGT-03/04/07~09/11~13/18) #1757)、git-workspace-status-panel.test.tsx(feat(core): #1662 车道 A Git 流程下沉(GIT-24/25/28/29) #1753)。
  • lithe-windows memory::platform::tests::current_process_private_working_set_is_available 失败,干净 preview 基线同样失败,属于该 Windows Server 环境的既有问题。
  • 跳过:real-jdt(机器无 JDK 21/25 与 Maven)、需要 GPU 或交互桌面的 GUI 验证、GitHub 专属的缓存/上传/gate 步骤。

合并前每个 PR 已 rebase 到最新 preview,并重新执行编译、i18n、verify-core、verify-shared-contracts 等快速检查。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

review: high ghfind author score; see https://ghfind.com

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant