Repository navigation
fix: 对齐 iOS 工程发现的深度与跳过规则 - #110
Merged
Merged
Conversation
- 移除工程发现遍历中与深度无关的 visited 去重,避免浅路径漏检 - bundle 校验的 visited 记录最浅到达深度,保持循环保护同时覆盖浅路径 - 普通目录与目录链接统一应用 .git/Pods/.build/DerivedData 跳过规则 - 增加深路径先到、共享目录多路径和 DerivedData 链接回归测试
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.
来源
修复
2026-11-13/MCP第五轮复审/复审报告.md中的 2 项问题。修复内容
P1 visited 未考虑到达深度
visited去重visited,但改为记录“最浅到达深度”;只有本次深度不更浅时才跳过,同时保留循环保护P2 目录链接未应用跳过规则
.git、Pods、.build、DerivedData的跳过规则统一在普通目录和目录链接分流前应用DerivedData链接不再被误报越界验证
cargo fmt --all --check-D warningscargo test -p shellsmith-mcp:3 个单元测试 + 37 个集成测试通过cargo test -p shield-cli:22 passedcargo test -p shield-core:263 passed, 8 ignoredpython3 -m unittest discover -s scripts/tests:48 passedproject.pbxprojDerivedData -> 外部缓存目录时直接 CLI 和 MCP 均成功返回正常工程信息未实机验证