Record Stage 2 extension design as DL-006 through DL-011 - #35
Closed
jimutt wants to merge 2 commits into
Closed
Conversation
Six decisions for the Pi extension, grounded in source review of the existing ecosystem rather than inference: - DL-006 dld-kit becomes a Pi package; TypeScript extension, no build step, bun test alongside the existing bats suite - DL-007 the extension reads state but delegates every mutation to the skill scripts, removing the dual-implementation drift DL-005 flagged - DL-008 in-session continuation on agent_end behind idle, token and bounds gates - DL-009 child-session rotation is re-entrant, driven by a typed tool and verified against disk state - DL-010 compaction during a run is assembled deterministically from disk so decision rationale is never paraphrased - DL-011 a single status line rather than a live dashboard widget DL-009 and DL-011 correct assumptions made before reading the sources: ralphi drives iterations re-entrantly from a retained command context (loop-controller.ts, loop-finalizer.ts), not from a long-lived handler, and pi-goal-x's widget history is the reason for the status-line limit. Co-Authored-By: Claude Opus 5 <noreply@pi.dev> Generated-By: pi 0.84.2
The original decision generalised pi-goal-x's scrollback incident into 'no rich UI'. The actual failure was an unbounded, expandable, always redrawing widget - not rich output as such. Pi has four surfaces with different costs, so match content to surface: always-on status line, a widget whose height is fixed regardless of run size, rich per-item cards as transcript entries (TUI-only, outside LLM context, free to redraw because they scroll), and a board overlay on demand for the full picture. Adjusted in place rather than superseded: DL-011 is proposed and has no implementation yet. Co-Authored-By: Claude Opus 5 <noreply@pi.dev> Generated-By: pi 0.84.2
Owner
Author
|
Superseded by feat/pi-harness-extension — all commits from this branch are merged there. See #41 for the findings log on top. |
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
1 participant
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.
What
Six decision records for the Pi extension design, grounded in source review of the existing ecosystem (ralphi's loop-controller and loop-finalizer, pi-goal-pro, pi-goal-x) rather than inference.
DL-011 was rewritten after recording: the original banned rich UI based on pi-goal-x's scrollback incident; the rewrite matches content to surface cost instead, with the widget's fixed height as the unit-tested safety property.
Stacks on #33. v1 scope is DL-006, 007, 008, 011; DL-009 and DL-010 are v2.