Skip to content

docs: refresh sprint QA checklist with evidence gates - #2038

Closed
groupthinking with Copilot wants to merge 2 commits into
mainfrom
copilot/add-evidence-oriented-qa-checklist
Closed

groupthinking with Copilot wants to merge 2 commits into
mainfrom
copilot/add-evidence-oriented-qa-checklist

Conversation

Copilot AI commented Sep 18, 2026

Copy link
Copy Markdown
Contributor

Canonical issue

Linked by PR metadata.

Outcome

Replaces the stale sprint-specific QA note with a reusable internal checklist that captures exact evidence, distinguishes release blockers from known limitations, and keeps QA anchored to the current YouTube URL → Video Pack → studio flow.

Scope

  • Included:
    • Rewrite docs/SPRINT_QA_CHECKLIST.md around evidence capture, blocker classification, and current repo commands
    • Remove stale PR / Notion references and sprint-specific status calls
    • Add manual verification guidance for the canonical studio flow and deploy-claim guardrails
    • Add a compact evidence log template for QA handoff
  • Explicitly excluded:
    • Product code, CI config, and test behavior changes
    • New docs tooling or automation
    • Production rollout or environment changes

Risk

  • Risk level: low
  • Failure mode:
    • QA operators follow outdated commands or misclassify findings if the document drifts from repo reality
  • Rollback:
    • Revert docs/SPRINT_QA_CHECKLIST.md to the previous version

Verification

Head SHA: 09e344014

git diff --check HEAD^ HEAD
python - <<'PY'
import json
from pathlib import Path
root = Path('.')
text = (root/'docs/SPRINT_QA_CHECKLIST.md').read_text()
root_pkg = json.loads((root/'package.json').read_text())
web_pkg = json.loads((root/'apps/web/package.json').read_text())
checks = {
    'contains blocker phrase': 'Release-blocking checks' in text,
    'contains known limitations phrase': 'Known limitations' in text,
    'contains evidence log template': 'QA evidence log template' in text,
    'root script lint exists': 'lint' in root_pkg['scripts'],
    'root script test exists': 'test' in root_pkg['scripts'],
    'root script build exists': 'build' in root_pkg['scripts'],
    'root script dev:web exists': 'dev:web' in root_pkg['scripts'],
    'web script type-check exists': 'type-check' in web_pkg['scripts'],
    'AGENTS ref exists': (root/'AGENTS.md').exists(),
    'LAUNCH ref exists': (root/'docs/LAUNCH_CHECKLIST.md').exists(),
    'NEXT-PHASE ref exists': (root/'docs/NEXT-PHASE.md').exists(),
    'stale notion removed': 'app.notion.com' not in text,
    'stale draft pr refs removed': 'Draft PR #' not in text and 'PR #1978' not in text and 'PR #1980' not in text,
}
failed = [name for name, ok in checks.items() if not ok]
for name, ok in checks.items():
    print(f'{name}: {ok}')
print('failed_checks=', failed)
raise SystemExit(1 if failed else 0)
PY
  • Focused tests
  • Required CI
  • Review threads resolved

Production evidence

Not applicable. Docs-only change; no runtime or deployment surface changed.

Agent handoff

  • One canonical issue is linked
  • No competing PR implements the same issue
  • Acceptance criteria are satisfied
  • Required checks pass on the current head
  • Human decision is requested only for product, security, irreversible infrastructure, or production approval

@coderabbitai

coderabbitai Bot commented Sep 18, 2026

Copy link
Copy Markdown
Contributor

Important

Review skipped

Ignore keyword(s) in the title.

⛔ Ignored keywords (3)
  • WIP
  • DO NOT MERGE
  • [skip ci]

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository: groupthinking/EventRelay/.coderabbit.yaml

Review profile: ASSERTIVE

Plan: Advanced

Run ID: 506210e3-961d-463f-958c-f5ea4d62e002

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

Co-authored-by: groupthinking <154503486+groupthinking@users.noreply.github.com>
Copilot AI changed the title [WIP] Add evidence-oriented sprint QA checklist document docs: refresh sprint QA checklist with evidence gates Sep 18, 2026
Copilot AI requested a review from groupthinking September 18, 2026 21:10
@vercel

vercel Bot commented Sep 18, 2026

Copy link
Copy Markdown
Contributor

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

Project Deployment Actions Updated
v0-uvai Ready Ready Preview, v0 Sep 18, 2026 10:13pm UTC

@kk-agent

Copy link
Copy Markdown
Collaborator

AXIOM keep-moving: closing as duplicate of docs checklist/agenda PRs (#1993/#1992).

@kk-agent kk-agent closed this Sep 18, 2026
@groupthinking
groupthinking deleted the copilot/add-evidence-oriented-qa-checklist branch September 19, 2026 03:31
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.

Add evidence-oriented sprint QA checklist document

3 participants