Skip to content

Add show and logs for named Commands - #782

Merged
JacobStephens2 merged 2 commits into
issue-711from
issue-772
Oct 18, 2026
Merged

JacobStephens2 merged 2 commits into
issue-711from
issue-772

Conversation

@JacobStephens2

@JacobStephens2 JacobStephens2 commented Oct 18, 2026 •

Copy link
Copy Markdown
Owner

Summary

Command name (repo#issue / Issue URL / kind-stamp)
  resolve across this machine's retained Commands
  choose live, otherwise latest
  show -> Run tree, stage history, log paths, latest Session event
  logs -> Command log or latest Session kind -> optional follow

Adds show and logs on the existing machine-wide status reader, including Ticket Runs inside Spec runs. Documents the version 1 show --json shape in the README.

Closes #772

Evidence

  • Before: cargo test --test show_logs show_names_one_command_and_logs_prints_its_exact_output failed with not a GitHub issue URL: show.
    After: all 11 show_logs end-to-end tests pass. They cover human names and ambiguity, live/latest selection, Architect stamps, Ticket lookup, Claude/Codex events, Resume and Repair selection, history fallback, and streamed Command/Session output through the final bytes. The 19 existing status tests also pass.
  • cargo clippy --all-targets -- -D warnings passes.
  • Full cargo test: 1,913 passed, 0 failed, 2 ignored across 47 test binaries. cargo fmt --check passes.
  • Both Spec regression commands failed as reported before their fixes and passed afterward; both tests are retained:
    • cargo test --test show_logs show_lists_the_original_architect_session_and_its_resume -- --exact
    • cargo test --test show_logs show_activity_history_lists_its_session_logs -- --exact

Test seams

  • The thirdshift binary, end to end with the existing fake GitHub and Harnesses: Command naming, detailed state and latest events, raw log selection and following.

Review coverage

Independent Standards and Spec reviews against issue-711 at 6f2cf77fe400f3945d890f179532bdf9aa70b1c1.

  • Standards: one P3 repeated-switch heuristic, resolved by a shared Pass-prefix mapping; no hard violations.
  • Spec: one P2 missing-Session-path finding, resolved for Architect Resumes and Activity-history JSON, with retained regressions.
  • Both axes' final files-read lists cover all ten changed files. Changed files left unread: none. Generated-output exclusions: none.

Unaddressed findings

Standards

None.

Spec

None.

Merge Danger

Door: two-way

Adds read commands; retained Command records keep their existing format.

Blast Radius: CLI

Shares record discovery with status; its existing end-to-end tests pass.

Built with codex · gpt-6.1-sol · high

@JacobStephens2
JacobStephens2 merged commit 276f02d into issue-711 Oct 18, 2026
18 checks passed
@JacobStephens2
JacobStephens2 deleted the issue-772 branch October 18, 2026 19:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant