Skip to content

docs: add terminal screenshot and Charm library overview - #216

Merged
Desperado merged 1 commit into
mainfrom
Desperado/qmax-readme-terminal-demo
Oct 16, 2026
Merged

Desperado merged 1 commit into
mainfrom
Desperado/qmax-readme-terminal-demo

Conversation

@Desperado

Copy link
Copy Markdown
Contributor

The README currently has architecture diagrams but no screenshot of the terminal. Add a capture of the real v1.38.0 standalone startup banner and a short overview of the Charm libraries used by the UI. This also meets the screenshot and library-overview prerequisites for submission to charm-in-the-wild.

The image was captured from tui.Terminal.PrintBanner with a local-only session context, using Freeze and the Menlo font. It contains no account details, credentials, or simulated test results. No runtime behavior changes.

Validation: visually checked the PNG, checked image links and library usage against the source, git diff --check, go test ./..., go vet ./..., and go run github.com/golangci/golangci-lint/cmd/golangci-lint@v1.64.8 run all passed. Ponytail review found no issues in this documentation-only change.

@qualitymaxapp

qualitymaxapp Bot commented Oct 16, 2026 •

Copy link
Copy Markdown

QualityMax Review

Verdict: COMMENT · Confidence: unavailable — no completed model review

Re-review: Open the QualityMax check and select Re-review after it completes.

Files eligible: 2 · Files reviewed: 2 · Files with findings: 0 · Findings: 0 · Inline cards: 0

Priority findings

priority location finding
— — No blocking findings

Review gates

gate status
AI diff review skipped · eligible 0, reviewed 0 · LLM · All changed files were excluded as non-code, generated, or binary content.
SAST completed · eligible 2, reviewed 2 · hybrid
Overall review evidence unavailable — insufficient model review evidence
Inline evidence not needed

Important files

file risk note next step
— — No findings —

Review lifecycle

Use the inline cards to inspect evidence and suggested remediation. Re-run the QualityMax review after pushing a fix; unchanged cards are identified by their stable finding marker. Dismiss with a reason through the existing QualityMax/GitHub review feedback flow. 0 prior card(s) are stale/resolved on this head. @qmax Q&A is tracked separately.

Proof legend: GROUNDED deterministic scanner evidence · MODEL-ONLY advisory model judgment · MODEL-REVIEWED patch model judgment, not reproduction.

QualityMax project results are available in the configured project.

Receipt · commit cd14f4c72ec45094f1eea1310b9c6eaba9fc6c9b · run 2026-10-16T14:28:31+00:00 · model served not applicable · model requested not applicable · model review unavailable — no completed model review · model source not recorded · re-review 2 · proof counts {}

@qualitymaxapp qualitymaxapp Bot 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.

QualityMax Review — canonical overview updated; no inline findings were created for this review.

@Desperado
Desperado merged commit af9fa99 into main Oct 16, 2026
6 checks passed
@qualitymaxapp

qualitymaxapp Bot commented Oct 16, 2026

Copy link
Copy Markdown

⚠️ QualityMax Pipeline

Gate Result
🔍 AI diff review ⏭️ Skipped — All changed files were excluded as non-code, generated, or binary content. · skipped · 0 eligible / 0 reviewed · All changed files were excluded as non-code, generated, or binary content.
🔍 SAST completed · 2 eligible / 2 reviewed
🔍 Canonical PR review delivery completed · 0 eligible / 0 reviewed · exact-head review #5549153415 and overview #6217674063 confirmed
🧪 Repo Tests ✅ 844/844 passed (go)
🤖 AI Tests ⚠️ 51/56 passed

Powered by QualityMax — AI-Powered Test Automation

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant