fix(cli,core): address dogfood pain points from ANGA-553 session#80
Merged
Conversation
Three fixes derived from live dogfood run: 1. fix(core): add --verbose flag to cc stream-json subprocess (ANGA-995) --output-format stream-json with --print requires --verbose or the subprocess exits with exit status 1. Resolves --stream regression. 2. fix(cli): memory recent accepts session name in addition to UUID (ANGA-996) `anvil memory recent <name>` now routes to recent_by_name() when the argument is not a valid UUID, matching the --session <name> convention users already know. UUIDs still work for backward compatibility. 3. fix(cli): show full session UUID in footer, not truncated 8-char prefix Makes `anvil memory recent <uuid>` copy-paste friendly from the session footer line. Co-Authored-By: Paperclip <noreply@paperclip.ing>
Co-Authored-By: Paperclip <noreply@paperclip.ing>
Owner
Author
|
@user Agent — please validate this PR. CI is green (all 5 checks pass), all review comments addressed. What changed:
|
anhermon
commented
Apr 26, 2026
anhermon
left a comment
Owner
Author
There was a problem hiding this comment.
LGTM
Validated ANGA-996 implementation on PR #80.
CI: All 5 checks green (Check & Format, Test, Security Audit, Deny, MSRV 1.75) ✓
Spec coverage checked:
anvil memory recent <name>— argument changed fromUuidtoString; routes torecent_by_name()when UUID parse fails. Backward-compatible: UUIDs still work.session_namecolumn already in main (landed via PR #66):insert_named()is called inagent.rs:288withopts.session_name, so named sessions populate the column on every episode insert.- Full UUID in footer:
&session.id.to_string()[..8]→session.id✓ - Empty-state message added:
No episodes found for session: {session}✓ - ANGA-995 stream fix:
--verboseadded to cc provider stream-json subprocess ✓
No unaddressed automated review comments.
Ready to merge.
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.
Summary
Three fixes from live dogfood run on Anvil (ANGA-553):
--stream --provider ccwas failing withstream-json requires --verbose. Added--verboseto the cc subprocess stream invocation.anvil memory recent <name>now works with human-readable session names from--session, not just UUIDs. Routes torecent_by_name()when argument is not a valid UUID.memory recent <uuid>copy-paste friendly.Test plan
anvil run --stream --provider cc --goal "..."completes without erroranvil memory recent dogfood-testreturns episodes (named session)anvil memory recent <full-uuid>still works (backward compat)🤖 Generated with Claude Code