Skip to content

feat(gui): close conversion-progress & visual-QA gaps (#105)#137

Merged
PAMulligan merged 2 commits into
mainfrom
105-conversion-qa-view
Jun 21, 2026
Merged

feat(gui): close conversion-progress & visual-QA gaps (#105)#137
PAMulligan merged 2 commits into
mainfrom
105-conversion-qa-view

Conversation

@PAMulligan

Copy link
Copy Markdown
Collaborator

Closes #105. Part of #100. Milestone v3.0.0.

Live streaming, running/success/failure states, theme-location surfacing, and the
visual-diff + Lighthouse rendering shipped in #129. This PR adds the rest.

  • Direct Activate action: the conversion result has an Activate-theme button
    (runs activate-theme, shows activating/activated/failed inline) — not just a
    pointer to the WordPress tab.
  • Design vs result: discoverQaArtifacts pairs
    .claude/visual-qa/screenshots/figma/<page>.png with the matching
    wordpress/chromium render (pure, unit-tested pairDesignResults); the Visual
    QA view shows them side-by-side when available.
  • Troubleshooting links on conversion failure → docs/COMMON-FAILURES-FIXES.md
    and docs/TROUBLESHOOTING.md.

Criteria

Stacked on #103#104#102#133 → main.

🤖 Generated with Claude Code

…105)

Closes the remaining conversion-progress / visual-QA-review gaps.

- Conversion result now surfaces a direct **Activate theme** action (runs
  activate-theme and shows activating/activated/failed inline) instead of only
  pointing at the WordPress tab.
- On conversion failure, links to docs/COMMON-FAILURES-FIXES.md and
  docs/TROUBLESHOOTING.md (via the sandboxed openPath bridge).
- Visual QA renders a **Design vs result** section: discoverQaArtifacts pairs
  .claude/visual-qa/screenshots/figma/<page>.png with the matching
  wordpress/chromium render (pure pairDesignResults helper, unit-tested), shown
  side-by-side when available.

Live streaming, success/failure states, theme location, and the visual-diff +
Lighthouse rendering shipped in #129; this adds the activate action,
design-vs-result comparison, and troubleshooting links the issue asks for.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@PAMulligan PAMulligan added this to the v3.0.0 milestone Jun 21, 2026
Base automatically changed from 103-pipeline-selection-ui to main June 21, 2026 05:06
@PAMulligan PAMulligan self-assigned this Jun 21, 2026
@PAMulligan PAMulligan merged commit dd95ace into main Jun 21, 2026
@PAMulligan PAMulligan deleted the 105-conversion-qa-view branch June 21, 2026 05:09
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.

Conversion progress & visual-QA review view

1 participant