Repository navigation
bump pi deps to 1.3.0 - #57
Merged
Merged
Conversation
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
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.
Bumps
@earendil-works/pi-agent-core,pi-ai, andpi-coding-agentfrom 0.84.4 → 1.3.0.What actually had to change
Nothing in
src/. Typecheck and the full test suite pass unchanged — worth stating explicitly, because the 0.86 → 1.0 breaking-change list is long:Context→TranscriptContext(0.86.0) only affects custom providers. The harness registers a custom tool set, not a provider.shouldStopAfterTurnremoved (0.87.0) — the harness never used it.SessionManagercanonical for provider context (0.87.0) — the harness never assignssession.agent.state.messages; it goes throughSessionManager.openat construction andsession.prompt()thereafter, which is the supported path.ContextEditEntryadded toSessionEntry,AgentBeforeSettleEventadded toExtensionEvent— nothing here switches exhaustively over either union.settings.json,models.json, orauth.json.So the harness lands on 1.x cleanly. The risk in an upgrade like this isn't the type errors it surfaces, it's the behaviour changes it doesn't, which is why the runtime checks below exist.
Also in this branch
README.md,docs/operating.md,docs/persona.md,src/config.ts,src/bin/setup.ts,agent-template/.env.example) fromv0.74.0tov1.3.0. Verified both tags and paths resolve onearendil-works/pi.bun.lockbregenerated;bun install --frozen-lockfilewas verified against it in a clean directory (this is what CI runs).Verification
Compile + unit level:
bun run typecheck(tsc 5.9.3) clean,bun test102/102 pass.Runtime level — the compiler can't prove any of this, so it was checked against 1.3.0 directly by building a session with the same call sequence
createAgentPooluses, against the real agent home but with a throwaway session file:createAgentSessionServices→SessionManager.open→createAgentSessionFromServices→prompt()runs end to end ondeepseek/deepseek-v4.1-flash.noTools: "builtin"still excludes the built-ins while keeping the harness's reconstructedbashand the extension-providedremembertool — extension loading is intact.agent_start,turn_start,message_update(+text_delta/text_end),message_end,tool_execution_start/_end,turn_end(assistantstopReasonpresent),agent_end,compaction_start/_end.prompt(..., { streamingBehavior: "steer" })gated onsession.agent.state.isStreaming) still works, probed from inside a long bash tool call — the case where a user types while the agent is mid-tool.terminate: truestill halts the run at the tool result —thread,react, andrestart_selfall depend on this and would silently stop ending turns if it regressed.isCompactingis true atcompaction_startand cleared bycompaction_end, andContextUsage.tokensisnullafter compaction then a number again after the next response.whenNotCompactingand the typing indicator rely on exactly this.Compaction/Retry/SteeringModekeys in pollux'ssettings.jsonand the provider entries inmodels.jsonwere checked against the schemas 1.3.0 ships — all still valid.Notes for review
pi-client/pi-protocolpackages. A stale 0.84.4 copy of each was left behind innode_modulesby the previous install; they are not inbun.lockb, and a clean install does not produce them. Nothing to do — flagging it so the stale directories aren't mistaken for a resolution problem.mcp.jsonis deferred to a follow-up (per frolic).