Skip to content

docs(deck): match the shipped wizard, kiosk, and steam-art behavior#279

Merged
mabry1985 merged 1 commit into
mainfrom
docs/deck-setup-refresh
Jun 21, 2026
Merged

docs(deck): match the shipped wizard, kiosk, and steam-art behavior#279
mabry1985 merged 1 commit into
mainfrom
docs/deck-setup-refresh

Conversation

@mabry1985

@mabry1985 mabry1985 commented Jun 21, 2026

Copy link
Copy Markdown
Contributor

Catches the Deck setup docs up to what actually shipped:

  • Setup wizard is now Welcome → Identity → Model → Finish (dropped Persona/Tools/Workspace); the API key is required on first run (Finish disabled until set).
  • Kiosk opens the console directly — the file:// "starting…" splash was removed (it couldn't fetch http and stranded the kiosk).
  • steam-art.sh renames the shortcut to pwnDeck and installs the grid art (dependency-free), not just "brand the art".

npm run docs:build clean — no dead links.

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Documentation
    • Updated Steam Deck setup instructions with improved guidance for configuring pwnDeck shortcut and installing library art
    • Enhanced troubleshooting documentation with clearer procedures for diagnosing connection errors and verifying backend connectivity
    • Streamlined setup wizard workflow and removed redundant configuration steps

Catch the Deck docs up to what shipped after the container-first rewrite:
- setup wizard is now Welcome→Identity→Model→Finish (dropped Persona/Tools/
  Workspace); the API key is required on first run.
- kiosk opens the console directly (the file:// "starting…" splash was removed —
  it couldn't fetch http and stranded the kiosk).
- steam-art.sh renames the shortcut to pwnDeck AND installs the grid art
  (dependency-free), not just "brand the art".

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@protoquinn

protoquinn Bot commented Jun 21, 2026

Copy link
Copy Markdown

👀 Quinn is reviewing — verdict (PASS / WARN / FAIL) + findings to follow.

@coderabbitai

coderabbitai Bot commented Jun 21, 2026

Copy link
Copy Markdown

Review Change Stack

Caution

Review failed

Pull request was closed or merged during review

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: 6cc16a3d-4657-4ddb-bb02-283a64b53ab1

📥 Commits

Reviewing files that changed from the base of the PR and between bf49add and efda107.

📒 Files selected for processing (2)
  • docs/steam-deck-setup.md
  • docs/tutorials/steam-deck-setup.md

Walkthrough

Two Steam Deck setup documentation files are updated to reflect that deck/steam-art.sh now renames the Steam shortcut to pwnDeck and installs library art. The tutorial's first-run wizard section is expanded with restart instructions and revised wizard steps, and a troubleshooting table entry's symptom wording is corrected.

Changes

Steam Deck Setup Documentation

Layer / File(s) Summary
Checklist and first-run wizard updates
docs/steam-deck-setup.md, docs/tutorials/steam-deck-setup.md
Updates the TL;DR checklist description for steam-art.sh to mention shortcut renaming to pwnDeck and art installation; expands the tutorial's Game Mode/first-run wizard section with steam-art.sh steps (rename, install art, restart Steam) and removes the previously listed "Persona / Tools / Workspace" wizard step.
Troubleshooting symptom wording fix
docs/tutorials/steam-deck-setup.md
Changes the troubleshooting table symptom from "Kiosk stays on the 'starting…' splash" to "Kiosk shows a connection error," keeping the same backend reachability check (curl localhost:7870/app/).

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

Suggested reviewers

  • protoquinn
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title directly summarizes the main change: updating documentation to match shipped wizard, kiosk, and steam-art behavior across multiple files.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch docs/deck-setup-refresh

Comment @coderabbitai help to get the list of available commands and usage tips.

@protoquinn protoquinn 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.

QA Audit — PR #279 | docs(deck): match the shipped wizard, kiosk, and steam-art behavior

VERDICT: WARN (non-terminal CI — review_comment placeholder; expect PASS once CI settles)


CI Status

  • Verify workspace config: queued
  • Lint: queued

Diff Review

  • Docs-only changes across 2 files, ~40 lines — zero code risk.
  • steam-deck-setup.md: tightened steam-art.sh description to include shortcut rename + grid art install.
  • tutorials/steam-deck-setup.md: rewired setup wizard flow (Identity → Model → Finish, dropped Persona/Tools/Workspace), documented API-key-required gate on Finish, removed stale file:// splash reference for kiosk, updated troubleshooting row to match.

Observations

  • LOW: CI still queued — both checks are lightweight (workspace config + lint) and should go green on a docs diff. No reason to expect failure.
  • No unresolved CodeRabbit threads, no dead links (npm run docs:build clean per PR body), no security or logic concerns.

— Quinn, QA Engineer

@protoquinn

protoquinn Bot commented Jun 21, 2026

Copy link
Copy Markdown

Submitted COMMENT review on #279.

@protoquinn protoquinn 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.

CI terminal-green, no blockers on prior review — auto-approving on green (#748).

@mabry1985 mabry1985 merged commit d2bfce0 into main Jun 21, 2026
5 of 6 checks passed
@mabry1985 mabry1985 deleted the docs/deck-setup-refresh branch June 21, 2026 23:59
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