Repository navigation
feat!: enable shell tools for writable Pi nodes - #41
Merged
Merged
Conversation
Epsirom
added a commit
that referenced
this pull request
Oct 4, 2026
## Change Prepare matching `@chrok/braid` and `@chrok/pi-braid` 0.3.0 releases from #41 and #42. This minor version accounts for writable Pi shell tools and Git-aware checkpoint semantics; it also includes Pi graph diagnostics, focused status/usage exports, cumulative merge-source baselines, and reminders between model steps. Update both manifests, Pi's exact core dependency, and the workspace lockfile. Finalize the changelog with implementing PR links and author credits, document 0.2 → 0.3 migration, and align both READMEs and the roadmap. Both included PRs are by @Epsirom, whose earlier contributions were verified; there are no first-time human contributors. ## Validation - `npm ci` and `npm run verify`: 227 core tests, 90 Pi tests, both type checks, offline examples, and isolated tarball checks passed. The package smoke verifies public imports, declarations, transitive core installation, and a Pi background job outside the checkout. - `npm pack --dry-run` for both packages: inspected clean file lists, licenses, and 0.3.0 versions; core has 39 files and Pi has 11. - `RELEASE_TAG=v0.3.0 node scripts/check-release.mjs` and `git diff --check` passed. - Hosted CI and CodeQL must pass before merge/tag. Publication will use the existing trusted-publishing workflow, followed by exact-version, latest-tag, provenance, and clean registry installation checks. - [x] Relevant tests and type checks pass. - [x] User-facing docs and changelog are updated where needed. - [x] Breaking changes include migration guidance. - [x] Release notes link feature/fix PRs, credit authors, and identify first-time contributors per the release guide.
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
1 participant
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.
Change
Writable Pi nodes can edit code but currently cannot build or test it, pushing the verification and repair loop back to the parent agent. This change gives writable nodes Pi's built-in
bashtool and, on Windows,powershell, while read-only nodes keep their existing capabilities.git add --allcheckpoint semantics so ignored dependencies, caches, and build outputs are excluded. Tracked and deliberately force-added files remain tracked. Add regressions for command repair, cancellation, child cleanup, read-only tool rejection, and checkpoint contents.This intentionally changes writable-node capabilities and ignored-output recovery, and is marked for the next minor release in
CHANGELOG.md. Migration guidance covers callers relying on shell-free workers or ignored artifacts being preserved.Validation
npm run verifypassed locally on macOS: 223 core tests, 72 Pi tests, both type checks, offline examples, and package installation/build smoke checks.git diff --checkpassed.No live model calls were used. Windows process cleanup still needs CI coverage; detached daemons and external services are outside the process-group lifecycle.
Relevant tests and type checks pass.
User-facing docs and changelog are updated where needed.
Breaking changes include migration guidance.
Release checklist: not applicable; this PR does not publish or bump package versions.