perf: skip the per-skill realpath in skill discovery when no workspace snapshot exists
维护者通常 1 天内回复
还没有人认领这个 Issue。
评估
- 难度
- 3/5
- 预计耗时
- 1-2 天
- 新手友好度
- 78/100
- Issue 类型
- 重构
- 描述清晰度
- 描述清楚
- 活跃度
- 活跃
- 技术栈
- typescript
- 领域
- cli, performance
调研方向
Read packages/opencode/src/skill/index.ts, focusing on withoutForeignSnapshots, then run the workspace snapshot attribution tests in test/skill/skill.test.ts. The change is done when discovery avoids unnecessary per-skill realpath work without an existing workspace snapshot, while the covered symlink and fail-closed cases continue to pass.
由索引模型根据 Issue 内容生成。
描述
Found during the v0.12.4 release review. Deferred because it is a small, correctness-preserving optimisation rather than a bug.
withoutForeignSnapshots in packages/opencode/src/skill/index.ts (added in #1377) calls fs.realpath on every skill file discovery finds, on every discovery pass, for every user — including users who have never enabled workspaces and have no .altimate-code/skill/_workspace directory anywhere. For them the check can never withhold anything, so the cost buys nothing.
Suggested fix: skip the realpath pass when no match's path contains a _workspace snapshot segment and no snapshot directory exists under the scanned roots, keeping the current fail-closed behaviour whenever a snapshot is present. The symlink cases covered by the "workspace snapshot attribution" tests in test/skill/skill.test.ts must keep passing.
- 主要语言
- TypeScript
- 星标
- 810
- 派生
- 121
- 平均合并
- 1 天 10 小时
- 30 天内合并 PR
- 89
环境准备
在浏览器里用你自己的 GitHub 账号启动这个项目的开发容器。
- 没有 Dockerfile 或 Docker Compose 文件
- 有 Pull Request 模板
- 阅读贡献指南
从这里开始
- 先读完整个 Issue，再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库，在一个分支上完成修改。
- 提交 Pull Request，并在描述里引用这个 Issue 编号。
AltimateAI/altimate-code 的其他 Issue
-
难度 2/5 1-3 小时 新手友好度 68/100
AltimateAI/altimate-code#1454 ·
维护者通常 1 天内回复
-
test: MCP tests fail when the developer's ~/.claude.json has MCP servers (HOME is not sandboxed)未关闭
难度 2/5 1-3 小时 新手友好度 78/100
AltimateAI/altimate-code#1386 ·
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 84/100
AltimateAI/altimate-code#1384 ·
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 84/100
AltimateAI/altimate-code#1323 ·
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 86/100
AltimateAI/altimate-code#1288 ·
维护者通常 1 天内回复
查看 AltimateAI/altimate-code 的全部 Issue
相似的 Issue
-
The inspector never shows the instances field, so scaling out is unreachable可能已有人在做 @Mrkewl 今天认领。 未关闭
难度 2/5 1-3 小时 新手友好度 76/100
xevrion/breakscale#96 ·
维护者通常 4 天内回复
-
Object parameters lose their structure when resources is enabled可能已有人在做 @asts-top 今天认领。 未关闭
难度 2/5 1-3 小时 新手友好度 74/100
维护者通常 2 天内回复
-
难度 2/5 1-3 小时 新手友好度 65/100
rossant/awesome-math#199 ·
-
bug
难度 2/5 1-3 小时 新手友好度 88/100
jaegertracing/jaeger-ui#4618 ·
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 70/100
mifi/lossless-cut#3125 ·
维护者通常 1 天内回复