Skip to content

feat(pi-extension): add lifecycle client without native tools - #5

Open
ZeR020 wants to merge 3 commits into
stack/5-cli-ingest-injectfrom
stack/6-pi-extension-client
Open

feat(pi-extension): add lifecycle client without native tools#5
ZeR020 wants to merge 3 commits into
stack/5-cli-ingest-injectfrom
stack/6-pi-extension-client

Conversation

@ZeR020

@ZeR020 ZeR020 commented Aug 17, 2026

Copy link
Copy Markdown
Owner

Sixth stacked PR for pi support (kunickiaj#1430). Depends on layer 5 (stack/5-cli-ingest-inject, #4).

Opened on the fork because GitHub cannot use kunickiaj:stack/5-cli-ingest-inject as a base.

Why

Maintainer blockers in this layer: project was cwd basename (not Git root); injection framing sniffed ## codemem memories in untrusted pack text; boundary flush CLI used the short HTTP timeout.

What

  • New @codemem/pi-extension package without tools.ts (native tools are the next layer).
  • projectFromCwd walks to .git / worktree gitdir; tests match core resolveProject.
  • fetchPackText / file-context return { text, preformatted }. No substring sniff.
  • Boundary ingest CLI timeout 30s. File-context uses the 8s pack budget, not 3s.
  • Workspace: tsconfig reference + lockfile importer.

Version stays 0.40.1 until layer 8 (0.41.0).

Test plan

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

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

Copilot AI lite review requested due to automatic review settings August 17, 2026 15:31

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.

@ZeR020
ZeR020 force-pushed the stack/5-cli-ingest-inject branch from 8c06664 to fefd593 Compare August 17, 2026 20:31
@ZeR020
ZeR020 force-pushed the stack/6-pi-extension-client branch 2 times, most recently from 1158ab6 to f2b91fc Compare August 19, 2026 07:49
@ZeR020
ZeR020 force-pushed the stack/5-cli-ingest-inject branch 2 times, most recently from ed06c83 to 19bd665 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
@ZeR020
ZeR020 force-pushed the stack/5-cli-ingest-inject branch from 19bd665 to 7abd173 Compare September 12, 2026 10:51
@ZeR020
ZeR020 force-pushed the stack/6-pi-extension-client branch 2 times, most recently from b91b584 to e756ea4 Compare September 12, 2026 20:06
@ZeR020
ZeR020 force-pushed the stack/5-cli-ingest-inject branch from 7abd173 to 983c167 Compare September 12, 2026 20:06
Git-root project walk (parity with core resolveProject). Pack fetch
returns { text, preformatted } so callers never sniff memory text.
Boundary CLI timeout is 30s, independent of HTTP ingest.
Core (merged kunickiaj#1546) hashes pi event ids as pi_evt_<24hex>; the layer's
tests still asserted the legacy pi:<session>:<part> format after the
rebase onto the updated stack.
@ZeR020
ZeR020 force-pushed the stack/5-cli-ingest-inject branch from 983c167 to cf78e04 Compare September 12, 2026 20:19
@ZeR020
ZeR020 force-pushed the stack/6-pi-extension-client branch from e756ea4 to 3ed2011 Compare September 12, 2026 20:19
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