Skip to content

Show measured pnpm prune recipe and clarify watched storage - #36

Merged
wxy merged 6 commits into
mainfrom
codex/pnpm-owner-recipe
Sep 27, 2026
Merged

wxy merged 6 commits into
mainfrom
codex/pnpm-owner-recipe

Conversation

@wxy

@wxy wxy commented Sep 27, 2026 •

Copy link
Copy Markdown
Owner

本轮界面与配方

  • pnpm store 作为按占用出现的配方:探测到有效 store 且实际已分配空间达到 10 MB 才显示。条目展示当前占用,详情给出路径、官方命令和明确确认;执行 pnpm store prune,不按目录直接删除。实际可释放量在执行前未知,store 总占用不冒充可释放量。
  • 在菜单栏面板中把「只观察」移至「手动清理」之后,使用与「不可清理」相同的色块,并说明两者数字有包含关系。观察项详情给出原因和 Finder 路径,不再提供「不再清理」。Xcode Archives 与模拟器设备有各自的解释。
  • GUI 最先找到的 Corepack shim 不可用时继续尝试本机其他 pnpm;多个有效 pnpm 指向不同 store 时拒绝操作。扫描后及执行前重新核验目标,探测异常时隐藏操作入口并显示原因。

清理边界

  • 命令只在用户从配方详情明确确认后执行;自动清理、CLI 和其他配方不得把 pnpm store 当成原始目录删除。
  • 所有已观察的 pnpm store 路径独立持久化。store 从 A 改到 B 后,旧 A 仍从其他配方的统计与清理中排除;记录读取或迁移失败、历史容量上限溢出时原始路径清理失败关闭。
  • 首次升级只能从仍保留的旧快照迁移历史目标;升级前已被快照保留策略移除且当前位置也不可探测的旧自定义目标无法追溯。

验证

  • 核心 pnpm 端到端测试 10/10 通过,覆盖坏 shim 回退、目标歧义、确认前后目标变化、A→B 历史路径、聚合占用和删除守卫、历史溢出。先失败后修复的记录和可重复命令见 docs/pnpm-store-recipe-verification.md 及 docs/verification-artifacts/。

  • 在本机最小 GUI PATH 下只读探测成功,使用 NVM pnpm 找到真实 store;扫描测得 426,921,984 已分配字节(约 407 MiB),高于显示阈值。没有对真实 store 执行 prune。

  • 最新 Xcode build-for-testing 通过,应用及 AppService 测试目标可编译。此环境的 macOS 测试运行器在握手前退出,AppService 断言未实际运行;记录见 docs/verification-artifacts/pnpm-store-child-xcode-test.txt。

  • 原有 pnpm 命令的隔离空 store 验证及先前的完整 Swift Package 检查保留在原提交与验证文档中。本轮新增功能的针对性端到端检查和 Xcode 编译已通过。

  • GitHub CI 的 swift test --skip cliScan 在本轮测试前置条件修正后通过,实际运行 213 项 Swift Testing 测试;CI 运行记录。

待界面验收

请在当前仓库的 codex/pnpm-owner-recipe 分支重新构建、扫描并打开菜单栏面板:确认 pnpm 配方随占用显示、详情不把 store 大小写成可释放量、「只观察」位置与解释符合预期。若要验证真实释放效果,应由用户阅读确认后在应用中执行官方命令。本 PR 保持草稿,不自动合并。

@wxy
wxy changed the base branch from codex/cache-child-clarity to main September 27, 2026 12:15
@wxy wxy changed the title Add manual pnpm store prune recipe Show measured pnpm prune recipe and clarify watched storage Sep 27, 2026
@wxy
wxy marked this pull request as ready for review September 27, 2026 12:46
@wxy
wxy merged commit 9ebc7dd into main Sep 27, 2026
3 checks passed
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Sep 27, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-09-27T12:47:33.004537Z a31ea7d Draft marked ready
🔒 Security Review ✅ Completed 2026-09-27T12:48:38.868745Z a31ea7d Draft marked ready
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant