Repository navigation
Replace custom extensions with published Pi packages - #1
Conversation
Swap the local manage-history and register-mcp-tools extensions for npm:@8monkey/pi-context-history and npm:pi-mcp-adapter, installed project-locally and declared in .pi/settings.json so Pi restores them automatically on startup. - rename GATO_* env flags to the packages' PI_* equivalents - move the summary prompt to .pi/prompts/compact.md (package override path) - MCP servers now configured in .pi/mcp.json (adapter's project override) - drop tsconfig, oxlint, and the typecheck/lint scripts — no local TS left - unwrap hard-wrapped README paragraphs to one line each Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
📝 WalkthroughWalkthroughThis PR migrates the Gato project from local ChangesPi package migration
Estimated code review effort: 3 (Moderate) | ~25 minutes 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Compresses closed session files on shutdown; /resume-compressed restores and reopens them. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
There was a problem hiding this comment.
🧹 Nitpick comments (1)
.pi/settings.json (1)
7-8: 📐 Maintainability & Code Quality | 🔵 Trivial | ⚡ Quick winPin the Pi package specs in
.pi/settings.json
The current barenpm:entries track the latest release on each fresh install; add explicit versions or semver ranges if you want reproducible restores.🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In @.pi/settings.json around lines 7 - 8, The package specs in the settings configuration are currently unpinned, so fresh installs can pull different releases over time. Update the `packages` entries in `.pi/settings.json` to use explicit versions or stable semver ranges for the Pi dependencies (`@8monkey/pi-context-history` and `pi-mcp-adapter`) while keeping the existing `steeringMode` setting unchanged.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Nitpick comments:
In @.pi/settings.json:
- Around line 7-8: The package specs in the settings configuration are currently
unpinned, so fresh installs can pull different releases over time. Update the
`packages` entries in `.pi/settings.json` to use explicit versions or stable
semver ranges for the Pi dependencies (`@8monkey/pi-context-history` and
`pi-mcp-adapter`) while keeping the existing `steeringMode` setting unchanged.
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: 17a98588-c4d5-4c57-a482-3b4be9ea84cd
⛔ Files ignored due to path filters (1)
bun.lockis excluded by!**/*.lock
📒 Files selected for processing (15)
.claude/CLAUDE.md.env.template.gitignore.oxlintrc.json.pi/extensions/filter-history.ts.pi/extensions/generate-summary.ts.pi/extensions/inject-system-prompt.ts.pi/extensions/lib/helpers.ts.pi/extensions/register-mcp-tools.ts.pi/prompts/compact.md.pi/settings.jsonREADME.mdlefthook.ymlpackage.jsontsconfig.json
💤 Files with no reviewable changes (8)
- lefthook.yml
- .pi/extensions/register-mcp-tools.ts
- .pi/extensions/inject-system-prompt.ts
- .oxlintrc.json
- tsconfig.json
- .pi/extensions/lib/helpers.ts
- .pi/extensions/generate-summary.ts
- .pi/extensions/filter-history.ts
Summary
Replaces the two local extensions with their published package equivalents, installed project-locally and declared in
.pi/settings.json(Pi restores missing packages automatically on startup):manage-history.ts→@8monkey/pi-context-history— same four context-management features, plus new/compact-session,/add-user-message, and/add-assistant-messagecommandsregister-mcp-tools.ts→pi-mcp-adapter— full MCP client with lazy lifecycles, OAuth, and a proxy tool;directTools: truekeeps the current per-tool registration behaviorAlso adds
@8monkey/pi-session-gzip, which compresses closed session files on shutdown and provides/resume-compressedto reopen them.🤖 Generated with Claude Code