Skip to content

[Impl] Surface incident and tool authority cues in text overview#214

Merged
luoyuctl merged 1 commit into
masterfrom
impl/213-text-overview-evidence
May 17, 2026
Merged

[Impl] Surface incident and tool authority cues in text overview#214
luoyuctl merged 1 commit into
masterfrom
impl/213-text-overview-evidence

Conversation

@luoyuctl
Copy link
Copy Markdown
Owner

Closes #213

Summary

  • Add compact Incident timeline cues to text overview output when timeline evidence exists.
  • Add compact Tool authority cues to text overview output when authority data exists.
  • Add regression coverage so demo text output cannot silently drop these cues.

User value

Terminal-first users can see the same local debugging signals from the fastest readable overview path without switching to JSON, Markdown, or HTML.

Scope

  • Text overview rendering only.
  • Existing JSON, Markdown, and HTML report contracts remain compatible.
  • Existing authority classification and incident timeline generation are unchanged.

Changed files

  • internal/engine/report.go
  • internal/engine/engine_test.go
  • scripts/ci/check-report-semantics.sh

Test plan

  • go test ./internal/engine
  • go test ./...
  • go build -o /tmp/agenttrace-parser-check ./cmd/agenttrace
  • /tmp/agenttrace-parser-check --doctor || true
  • /tmp/agenttrace-parser-check --demo --overview -f text >/tmp/agenttrace-parser-overview-213.txt
  • /tmp/agenttrace-parser-check --demo --overview -f json >/tmp/agenttrace-parser-overview-213.json
  • /tmp/agenttrace-parser-check --demo --overview -f markdown -o /tmp/agenttrace-parser-overview-213.md >/tmp/agenttrace-parser-overview-213.md.stdout
  • /tmp/agenttrace-parser-check --demo --overview -f html -o /tmp/agenttrace-parser-overview-213.html >/tmp/agenttrace-parser-overview-213.html.stdout
  • AGENTTRACE_BIN=/tmp/agenttrace-parser-check AGENTTRACE_CI_OUT=/tmp/agenttrace-parser-ci-output-213 scripts/ci/check-output-contract.sh
  • AGENTTRACE_BIN=/tmp/agenttrace-parser-check AGENTTRACE_CI_OUT=/tmp/agenttrace-parser-ci-deterministic-213 scripts/ci/check-deterministic-output.sh
  • AGENTTRACE_BIN=/tmp/agenttrace-parser-check AGENTTRACE_CI_OUT=/tmp/agenttrace-parser-ci-semantics-213 scripts/ci/check-report-semantics.sh
  • AGENTTRACE_BIN=/tmp/agenttrace-parser-check AGENTTRACE_CI_OUT=/tmp/agenttrace-parser-ci-docs-213 scripts/ci/check-docs-commands.sh
  • scripts/ci/check-release-surfaces.sh
  • scripts/ci/check-pages-artifact.sh site

Fixture/privacy note

Uses demo and synthetic unit-test sessions only. No real user prompts, logs, tokens, or secrets were added.

Risk

Medium-low. This changes terminal text output, but keeps the new sections bounded and reuses existing timeline and authority summary data.

Non-goals

  • Do not make text output as dense as Markdown or HTML tables.
  • Do not change JSON field names or authority classification rules.
  • Do not add hosted telemetry, upload, or external sharing behavior.
  • Do not change release, package, npm, Homebrew, or public site surfaces.

Blackboard events

Release impact

patch: text overview visible output changes. No parser/source compatibility change, no CLI flag change, no JSON/Markdown/HTML contract change, no install/package docs change.

Handoff suggestion

Handoff to: Growth after merge if release notes are being collected.
Suggested release note: text overview now surfaces compact incident timeline and tool authority cues in the terminal.

@luoyuctl luoyuctl added lane/product Product experience and user journey work status/ready-for-review Ready for review source/product Created or updated by product manager routing labels May 17, 2026
@luoyuctl
Copy link
Copy Markdown
Owner Author

Event: Validation
Actor: Product
Scope: PR #214 / Issue #213
Result: PASS

Evidence:

  • GitHub CI: Test and build SUCCESS on head 750a562.
  • Local clean clone at 750a562:
    • go test ./...
    • go build -o /tmp/agenttrace-product-214-check ./cmd/agenttrace
    • AGENTTRACE_BIN=/tmp/agenttrace-product-214-check AGENTTRACE_CI_OUT=/tmp/agenttrace-product-214-ci-semantics scripts/ci/check-report-semantics.sh
    • AGENTTRACE_BIN=/tmp/agenttrace-product-214-check AGENTTRACE_CI_OUT=/tmp/agenttrace-product-214-ci-output scripts/ci/check-output-contract.sh
    • AGENTTRACE_BIN=/tmp/agenttrace-product-214-check AGENTTRACE_CI_OUT=/tmp/agenttrace-product-214-ci-determinism scripts/ci/check-deterministic-output.sh
    • AGENTTRACE_BIN=/tmp/agenttrace-product-214-check AGENTTRACE_CI_OUT=/tmp/agenttrace-product-214-ci-docs scripts/ci/check-docs-commands.sh
    • scripts/ci/check-release-surfaces.sh
    • scripts/ci/check-pages-artifact.sh site
    • /tmp/agenttrace-product-214-check --doctor || true

Manual acceptance check:

  • demo text overview now includes Incident timeline.
  • demo text overview now includes Tool authority.
  • demo text overview includes Highest category: test_or_build, Authority category counts, and High-authority tools: terminal.
  • Markdown overview still exposes Tool authority and Incident timeline.

Decision: Product validation passes. This is ready for maintainer/merge-owner review under the existing PR flow.

@luoyuctl
Copy link
Copy Markdown
Owner Author

Quality review PASS for #214.\n\nValidated locally in a temporary review worktree:\n- go test ./internal/engine\n- go test ./...\n- go build -o /tmp/agenttrace-review-pr-214 ./cmd/agenttrace\n- git diff --check origin/master...HEAD\n- /tmp/agenttrace-review-pr-214 --doctor || true\n- /tmp/agenttrace-review-pr-214 --demo --overview -f text/json\n- scripts/ci/check-output-contract.sh\n- scripts/ci/check-deterministic-output.sh\n- scripts/ci/check-report-semantics.sh\n- scripts/ci/check-docs-commands.sh\n- scripts/ci/check-release-surfaces.sh\n- scripts/ci/check-pages-artifact.sh site\n\nNotes:\n- Scope stays within text overview rendering and report semantics coverage.\n- No new external IO, telemetry, package, release, or public site behavior.\n- Demo text output now includes compact incident timeline and tool authority cues as intended.

@luoyuctl luoyuctl merged commit 5a48c9d into master May 17, 2026
1 check passed
@luoyuctl luoyuctl deleted the impl/213-text-overview-evidence branch May 17, 2026 10:46
@luoyuctl
Copy link
Copy Markdown
Owner Author

Event: Post-merge validation
Actor: Product
Scope: PR #214
Result: PASS

Evidence: merged as 5a48c9d, issue #213 is closed, labels moved to status/auto-merged. Clean origin/master validation passed across tests, build, report semantics, output contract, deterministic output, docs command smoke, release-surface checks, Pages artifact checks, doctor, and manual demo overview checks.

User value confirmed: terminal text overview now exposes compact incident timeline and tool authority cues without requiring JSON/Markdown/HTML export.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lane/product Product experience and user journey work source/product Created or updated by product manager routing status/auto-merged

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Surface incident and tool authority cues in text overview

1 participant