Repository navigation
fix(coding-agent): share Pi packages by default - #30
Merged
Kushalkhemka merged 8 commits intoOct 15, 2026
Merged
Conversation
boonlai
requested review from
Kushalkhemka and
opensec-intelligence
as code owners
October 15, 2026 09:35
…install/remove effective - Convert a piBolt.packages list copied from packages by Pi-Bolt before 0.9.1 once per settings file (global, trusted project): exclude shared packages missing from the copy, drop entries identical to their shared entry, keep replacements and Pi-Bolt-only entries. Every writer marks the list with piBolt.packagesOverlay so it is never converted again. - Plain pi-bolt install/remove also drop Pi-Bolt's entry for that package in the same file (replacement, addition or exclusion), so remove cannot leave Pi-Bolt loading it and install cannot be hidden by a stale replacement or an exclusion. A plain remove no longer keeps installed files for a Pi-Bolt entry it drops. - Package CLI flushes queued settings writes before main() exits. - SDK getPackages adds a project's Pi-Bolt entries to the global packages when the project has no packages list of its own.
Member
|
Pushed 47209a9 on top, with pi-bolt merged in, for 0.9.1:
Tests: settings-pibolt-packages (73), package-manager-cli-preflight, package-manager, settings-manager suites (434 across 8 files), tests/installer/extensions.py; |
…-defaults # Conflicts: # site/docs/search.json
Kushalkhemka
pushed a commit
that referenced
this pull request
Oct 15, 2026
Shared package defaults (#30); site/docs/search.json built again.
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
3 participants
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.
What and why
Closes #28.
Pi-Bolt now inherits Pi’s shared
packageslist instead of copying it on the first package mutation.--pi-boltexplicitly modifies additivepiBolt.packagesoverrides, supporting additions, replacements and exclusions.Existing copied lists remain readable without loading duplicate packages. Packages omitted from those lists are now inherited; explicit exclusions disable them.
How it was tested
559 focused tests passed across package management, settings, resource loading and CLI parsing.
Five installer tests and
npm run checkpassed.Regression tests cover filter preservation, concurrent settings updates, queued writes, unreadable settings, selector precedence and argument parsing.
Earlier compiled macOS ARM64 smoke checks passed.
Full-suite verification previously encountered a TUI cancellation reproduced on the unchanged baseline.
Latest upstream
pi-boltchanges merge cleanly. Windows execution remains unverified.cipassesEngine or build change:
tests/aot/run.shandtests/pi/run.shon a build with it — not applicable