Skip to content

feat(pi-extension): register native memory tools - #6

Open
ZeR020 wants to merge 1 commit into
stack/6-pi-extension-clientfrom
stack/7-native-tools
Open

feat(pi-extension): register native memory tools#6
ZeR020 wants to merge 1 commit into
stack/6-pi-extension-clientfrom
stack/7-native-tools

Conversation

@ZeR020

@ZeR020 ZeR020 commented Aug 17, 2026

Copy link
Copy Markdown
Owner

Seventh stacked PR for pi support (kunickiaj#1430). Depends on layer 6 (stack/6-pi-extension-client, #5).

Opened on the fork because GitHub cannot use kunickiaj:stack/6-pi-extension-client as a base.

Why

Maintainer blocker in this layer: file-context replaced string tool_result content (original read text disappeared).

What

  • Native memory_* tools from 5f84fe3b + aee5090a (CLI JSON parse + distill all_projects).
  • String tool_result content is normalized before file-context append; original text remains.
  • Default tool project is the Git-root identity. all_projects still drops project.
  • pi.tools_mode=mcp-adapter skips native registration.

Version stays 0.40.1 until layer 8 (0.41.0).

Test plan

  • vitest run packages/pi-extension — 52/52
  • tsc -p packages/pi-extension --noEmit

Stacked on: #5 / #4 / #3 / #2 / #1 / kunickiaj#1473
Related: kunickiaj#1430, kunickiaj#1429

Copilot AI lite review requested due to automatic review settings August 17, 2026 16:25

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

Normalize string tool_result content before file-context
append so the original read text remains. Default tool
project is the Git-root identity; all_projects drops project.
@ZeR020
ZeR020 force-pushed the stack/7-native-tools branch from 152c5c4 to 59e9172 Compare August 23, 2026 15:33
@ZeR020
ZeR020 force-pushed the stack/6-pi-extension-client branch from f2b91fc to 1ce0cf1 Compare August 23, 2026 15:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants