Added Dependencies section describing third-party and pi package dependencies.
EditClarified normal dependencies versus built-in pi packages.
EditRan npm check via commit hook.
VerifyCommitted and pushed documentation changes.
Cleanup@packages/coding-agent/docs/packages.md should also describe how to include 3rd party dependencies (simply via package.json) that they automatically get installed along with shit, but that for pi dependencies, you need to make them peer dependencies with * (i think we already say that?)
Summary:
Tests not run (documentation change).
dude, did you read all of @packages/coding-agent/docs/packages.md i was talking about the "built-in" pi packages, pi-ai, pi-agent-core, pi-coding-agent, pi-tui, typebox
Summary:
Tests not run (documentation change).
+132 Other pi packages should also be peerDependencies with a "*" range so users install them
separately via settings or pi install, which keeps a single copy. Use bundledDependencies only if you
want to ship a pi package inside your tarball.
no, pi packages depending on other pi packages must bundle, otherwise we get into devil's kitchen. the modules are loaded separately from each other and can't collide
Summary:
Tests not run (documentation change).
do we state the other pi packages shit twice now? read the ful lfile
Yes. It is covered in two places: the “Bundling Other Pi Packages” section under Package Structure and the Dependencies section. Do you want me to consolidate into one section and remove the other?
consolidate
Summary:
Tests not run (documentation change).
commit and push
Summary:
Tests: npm run check (triggered by commit hook).
what about plain old dependencies?
Plain third party deps go in dependencies. Pi runs npm install on install, so they are installed automatically. The special cases are the built-in pi packages as peerDependencies with "*" and other pi packages bundled. If you want this called out more explicitly in the doc, say where.
in the same section you just consolidated. "normal dependencies that themselves don't register extensions, skills, prompt tempaltes, or themes go into the regular "dependencies""
Summary:
Tests not run (documentation change).
commit and push
Summary:
Tests: npm run check (commit hook).