fix(workbench): keep repository inventory and snapshots within the target - #472
fix(workbench): keep repository inventory and snapshots within the target#472mldangelo-oai wants to merge 16 commits into
Conversation
|
@codex review |
|
Codex Review: Didn't find any major issues. Already looking forward to the next diff. Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
|
Security review completed. No security issues were found in this pull request. Reviewed commit: Only the user who started this review can view the report in Codex. ℹ️ About Codex security reviews in GitHubThis is an experimental Codex feature. Security reviews are triggered when:
Once complete, Codex will leave suggestions, or a comment if no findings are found. |
|
@codex review Please review the current head, |
|
Codex Review: Didn't find any major issues. More of your lovely PRs please. Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
|
Security review completed. No security issues were found in this pull request. Reviewed commit: Only the user who started this review can view the report in Codex. ℹ️ About Codex security reviews in GitHubThis is an experimental Codex feature. Security reviews are triggered when:
Once complete, Codex will leave suggestions, or a comment if no findings are found. |
|
@codex review Please review commit |
|
Security review completed. No security issues were found in this pull request. Reviewed commit: Only the user who started this review can view the report in Codex. ℹ️ About Codex security reviews in GitHubThis is an experimental Codex feature. Security reviews are triggered when:
Once complete, Codex will leave suggestions, or a comment if no findings are found. |
|
Codex Review: Didn't find any major issues. What shall we delve into next? Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
Summary
Keep generated repository inventories and Git snapshots inside the selected repository target.
Changes
Testing
c9c12142:pnpm run types,pnpm run format, andpnpm run build— passed.pnpm pack, the static package contract, and bounded installed public SDK/CLI import, version, and help checks — passed. All packed bundled-plugin files match the committed source.648c1525: focused inventory/ranking/compact-diff checks passed (11 passed, 2 expected platform skips); the full randomized SDK suite passed (1,294 passed, 11 skipped, 0 failed, 9,150 assertions).Risk and rollout
Ordinary files, selected target roots, deleted history, existing missing-file handling, and supported diff modes retain their behavior. Both bundled-plugin version declarations are
0.1.21in this standalone branch. If another plugin-changing PR lands first, refresh against main and increment both declarations together to the next unused version so already-staged plugins refresh correctly. Fresh pushed-head CI remains required.Public disclosure review
Newly authored material was reviewed for public disclosure. Existing automated review comments contain access-restricted report references; maintainer cleanup remains outstanding.