Skip to content

add compact test output with step descriptions#198

Open
theodore-s-beers wants to merge 1 commit into
mainfrom
tsb_compact_output
Open

add compact test output with step descriptions#198
theodore-s-beers wants to merge 1 commit into
mainfrom
tsb_compact_output

Conversation

@theodore-s-beers

Copy link
Copy Markdown
Member
  • Add optional plain-language descriptions for CLI test steps
  • Show compact pass/fail output by default for submissions and local tests
  • Add --verbose to display full details for every step
  • Expand failed steps while hiding later steps
  • Apply visual truncation to HTTP response bodies
  • Preserve command/request fallbacks for older manifests

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR enhances the CLI test renderer to support plain-language step descriptions and a more compact default output mode, with a --verbose flag to show full per-step detail. It also adds visual truncation for large command/HTTP outputs and wires step descriptions through the manifest → runner → renderer pipeline.

Changes:

  • Add description support for CLI steps (manifest + message plumbing) and render it alongside a technical detail string.
  • Default to compact pass/fail step lines, expanding failed steps (and hiding later steps) unless --verbose is used.
  • Truncate large stdout/HTTP response bodies for readability, and add renderer-focused tests.

Reviewed changes

Copilot reviewed 12 out of 12 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
render/view.go Implements compact/verbose rendering logic, failure expansion, and output truncation.
render/view_test.go Adds tests for compact vs verbose output, description fallback, and truncation behavior.
render/render.go Adds support for step description + detail fields and threads verbose into the model.
render/models.go Renames step display fields and adds verbose to the root model/init.
render/http.go Applies visual truncation to HTTP response bodies (including pretty JSON).
messages/messages.go Extends StartStepMsg with an optional plain-language Description.
cmd/submit.go Adds --verbose/-v flag and passes it into the renderer.
cmd/run.go Adds --verbose/-v flag for non-submit runs.
cmd/localtest.go Adds --verbose/-v flag and passes it into the renderer.
cmd/localtest_test.go Updates local CLI data parsing test to include/verify step description in YAML.
client/lessons.go Adds CLIStep.Description YAML field to support descriptions in manifests.
checks/runner.go Passes step description into StartStepMsg for both CLI and HTTP steps.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread render/view.go Outdated
Comment thread render/view.go Outdated
Comment thread render/view.go Outdated

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 12 out of 12 changed files in this pull request and generated 1 comment.

Comment thread render/view.go
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.

2 participants