Skip to content

feat: QA Bot — automated visual QA & video evidence system#189

Draft
snomiao wants to merge 1 commit intomainfrom
sno-qabot
Draft

feat: QA Bot — automated visual QA & video evidence system#189
snomiao wants to merge 1 commit intomainfrom
sno-qabot

Conversation

@snomiao
Copy link
Copy Markdown
Member

@snomiao snomiao commented Apr 4, 2026

Summary

Adds a centralized QA Bot sub-agent to ComfyPR-Bot that automatically reproduces bugs, verifies PRs, and records HD video evidence — then posts results back to GitHub issues/PRs and Slack.

Unlike the prototype in ComfyUI_frontend PR #9430 (single-repo CI), this lives in Comfy-PR as a centralized service targeting any frontend repo.

What's Included

Implementation (bot/qa/ — 14 files)

Module Purpose
types.ts, config.ts Shared types, default config (resolution, timeouts, agent model)
orchestrator.ts Full pipeline: clone → detect → boot → agent → report
bootstrap/ (5 files) Auto-detect & start ComfyUI-frontend, Vite, Next.js apps
browser/ (2 files) Playwright launch, cursor overlay injection, video recording, screenshots
qa-agent.ts AI-driven browser automation loop (Claude via AI SDK)
report/ (2 files) Markdown report generation, GitHub comment posting, label management
cli.ts CLI handlers wired into bot/cli.ts

CLI Commands

prbot qa reproduce -i Comfy-Org/ComfyUI_frontend#10688
prbot qa verify    -p Comfy-Org/ComfyUI_frontend#9500
prbot qa smoke     -r Comfy-Org/ComfyUI_frontend
prbot qa demo      -r Comfy-Org/ComfyUI_frontend --prompt 'Demo the template browser'
prbot qa run       -r Comfy-Org/ComfyUI_frontend --prompt 'Check drag-and-drop on canvas'

Design Docs (docs/qabot/ — 9 files)

README, Architecture, Triggers, Video Recording, Browser Automation, Multi-Repo Support, Reporting, CLI Spec, Roadmap

Roadmap (in TODO.md)

  • Phase 0 ✅ Foundation (this PR)
  • Phase 1 Video delivery (ffmpeg post-processing, GCS upload, Slack posting)
  • Phase 2 PR before/after verification
  • Phase 3 Multi-repo triggers (GitHub webhooks, Slack mentions, /qa comments)
  • Phase 4–7 Smoke scheduling, high-quality video production, Electron support, advanced features

Dependencies

  • Adds playwright (browser automation + video recording)
  • Uses existing ai, @ai-sdk/anthropic (already in deps)
  • Leverages existing lib/slack/, lib/video/, src/cli/ infra

…ording, CLI

- bot/qa/: 14 source files implementing the full Phase 0 pipeline
- docs/qabot/: 9 design docs (architecture, triggers, video, etc.)
- TODO.md: phased implementation tracker
- Adds playwright dependency

Amp-Thread-ID: https://ampcode.com/threads/T-019d5249-c1e2-7274-b789-86547ec8c826
Co-authored-by: Amp <amp@ampcode.com>
@vercel
Copy link
Copy Markdown
Contributor

vercel bot commented Apr 4, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
comfy-pr Ready Ready Preview, Comment Apr 4, 2026 4:06am

Request Review

@socket-security
Copy link
Copy Markdown

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Added@​pulumi/​pulumi@​3.229.073100889780
Addedcross-env@​10.1.010010010082100
Addedoxfmt@​0.26.0911008896100
Added@​pulumi/​gcp@​9.18.01001009298100
Addedplaywright@​1.59.110010010099100

View full report

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