fix(nap): heading-aware archival#650
Closed
diberry wants to merge 1 commit intobradygaster:devfrom
Closed
Conversation
This was referenced Mar 27, 2026
larsontim12
pushed a commit
to larsontim12/squad
that referenced
this pull request
Mar 27, 2026
…er#650) * feat: REPL UX polish — figlet banner, proper-case names, keyword emoji, 80-col wrap, activity hints - Add figlet ASCII art SQUAD banner in header (App.tsx) - Fix agent name casing: SessionRegistry uses case-insensitive keys, preserves display names (sessions.ts) - Add keyword-based role emoji fallback matching in getRoleEmoji (lifecycle.ts) - Add extractAgentHint() to parse coordinator text for specific task descriptions (index.ts) - Cap content width at 80 columns across App.tsx, MessageStream.tsx, AgentPanel.tsx - Remove [system] prefix from system messages in MessageStream - Add slash command passthrough while processing (InputPrompt.tsx) - Pin header to top via Ink Static block, simplified header content - Register agents with proper case names, use display names in activity hints - Update tests: 192 passing Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * feat: wire REPL telemetry — EventBus bridge, StreamingPipeline, agent metrics, model display Wire all defined OTel instrumentation into the REPL code path: - Pass EventBus to initSquadTelemetry() so EventBus→OTel bridge activates (bradygaster#645) - Instantiate StreamingPipeline and feed message_delta/usage events for token and response latency metrics (TTFT, duration, tokens/sec) (bradygaster#646) - Call recordAgentSpawn/Duration/Error/Destroy from dispatch handlers (bradygaster#647) - Add model field to AgentSession, display model name in AgentPanel (bradygaster#648) - Enable shell metrics when OTel endpoint is configured (no longer requires SQUAD_TELEMETRY=1) (bradygaster#649) - Export RuntimeEventBus from SDK barrel for REPL consumption - Listen for usage events to capture model name and feed to StreamingPipeline - Update tests: replace stale [WORK]/[STREAM] assertions, update banner/header checks for figlet art, update source-code parsing tests for simplified header Closes bradygaster#644, bradygaster#645, bradygaster#646, bradygaster#647, bradygaster#648, bradygaster#649 Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> --------- Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
diberry
force-pushed
the
squad/22-heading-aware-archival
branch
from
419079f to
5312bc2
Compare
Collaborator
Author
|
Closing -- will re-open via fork-first pipeline when fully polished. |
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.
Extends archiveDecisions() parser to recognize ## section headings as boundaries. Preserves section structure during archival — ## headings stay if any ### child is kept.
44 tests pass (42 existing + 2 new). Code-only fix in nap.ts.
Closes diberry#22
@bradygaster — postmortem action item A2. Fixes orphaned section headers during archival.
Co-authored-by: Copilot 223556219+Copilot@users.noreply.github.com