FE-744: Document Pi command containment evidence#150
Conversation
new pseudo skill: minimal notation typology for human-agent design (tree, chain, graph, matrix, state-machine, data-shape, lanes) with shared overlay grammar (#anchor, @owner, +/-/~/!/?, ->/~>/x>) and a routing chain expressed in its own chain notation. cross-link directives in ln-{spec,design,scope,plan,refactor,review, sync,build} using a grep-able **Notation aid.** lead-in. ln-sync directive explicitly preserves existing pseudo artifacts and consolidates qualifying prose into them, so future sync runs do not collapse pseudo back to prose. PLAN.md updates as living examples: - Dependencies block: hand-drawn tree -> pseudo graph with cross-edges (m5 -[optional]-> subagents, graph-plane -[on promotion]-> oracle) visible and horizon items grouped under unconnected - agent-graph-integration acceptance: paragraph -> pseudo tree (obligation decomposition) with each leaf individually testable
…de-judo moves Amp-Thread-ID: https://ampcode.com/threads/T-019e77c1-cc1c-73db-a4ab-d1e9bec3be5e Co-authored-by: Amp <amp@ampcode.com>
🤖 Augment PR SummarySummary: Documents Pi command-containment evidence and tightens the Brunch↔Pi UI extension boundary, while adding an explicit workspace/spec/session selection flow for TUI startup and in-session switching. Changes:
Technical Notes: Workspace inventory surfaces unavailable sessions with explicit reasons and activation updates 🤖 Was this summary useful? React with 👍 or 👎 |
| type InspectedSessionFile = WorkspaceLaunchSession | WorkspaceUnavailableSession | ||
|
|
||
| async function inspectSessionFile(file: string): Promise<InspectedSessionFile> { | ||
| const entries = await readJsonl(file) |
There was a problem hiding this comment.
inspectSessionFile() calls readJsonl() which JSON.parses every line; a single corrupted/partial session file will throw and abort inventory/activation, which can crash the CLI instead of just treating that session as unavailable. Consider catching parse errors and returning an unavailable entry/reason (also applies to the readJsonl() usage in verifyWorkspaceSessionStores()).
Severity: medium
Other Locations
src/workspace-session-coordinator.ts:631
🤖 Was this useful? React with 👍 or 👎, or 🚀 if it prevented an incident/outage.
FE-744: Document Pi command containment evidence
FE-744: Prove dynamic Brunch chrome wrapper
restore provisional plan
FE-744: Refresh provisional Pi UI handoff
Tighten ln-build artifact cleanup guardrails
capture brunch ANSI logo exploration and decision
FE-744: Add workspace launch inventory
FE-744: Activate workspace switch decisions
FE-744: Add workspace switcher decision UI
FE-744: Gate TUI startup on workspace switch
FE-744: Rename implicit coordinator operations
FE-744: Split coordinator caller interfaces
FE-744: Remove source-string boundary tests
FE-744: Require activated chrome session state
FE-744: Narrow coordinator test doubles
FE-744: Route fixture capture through RPC handlers
FE-744: Extract Brunch Pi extension entrypoint
FE-744: Split workspace switcher modules
FE-744: Replace shell source test with helpers
FE-744: Fix offline default env typing
FE-744: Retire exhausted refactor queue
FE-744: Split Brunch extension surfaces
FE-744: Allocate Brunch chrome surfaces
FE-744: Add in-session workspace switch command
FE-744: Add startup no-resume oracle
FE-744: Reconcile Pi UI extension memo