Sacboyz NFL Next Action Queue
Decision-aware queue that keeps fixture-mode work separate from provider, live-data, and repository setup dependencies.
Reviewer Handoff Checklist
Reviewer Stacked PR Handoff Instructions
To inspect and validate the stacked report PRs before merge:
- Run the local quality gate:
python run_quality_gate.py - Verify git status:
git status --short
Note: All live-provider adapter calls remain disabled by design. These PRs represent stacked work and will remain stacked until Chris and Claude return for a final review.
Reviewer "What Changed" Summary
Changed Artifacts: Rebuilt static report HTMLs, sitemaps, checklists, and JSON definitions to reflect polished reviewer copy and print query formatting.
Validation State: PASS (All 77 handoff artifacts validated successfully against offline schemas; sitemaps fully populated).
No-Live-Call Safety: ENFORCED (Confirmed zero live REST or WebSocket queries to providers; runtime defaults to live_no_call).
PR Review Order Print Snapshot
Follow this sequential order to inspect and approve the stacked PRs:
- PR #24 (codex/provider-samples-backlog): Review baseline provider decisions and JSON structures.
- PR #25 (codex/live-v1-provider-foundation): Verify initial provider live connection hooks.
- PR #26 (antigravity/report-docs-polish): Inspect report landing layouts and CSS variables.
- PR #27 (antigravity/report-qa-polish): Verify conservative empty states and quality gate scripts.
- PR #28 (antigravity/report-accessibility-polish): Review table caption elements and aria labels.
- PR #29 (antigravity/report-mobile-print-polish): Inspect print query stylesheets and narrow screen layouts.
- PR #30 (antigravity/report-reviewer-flow-polish): Review relative link paths and sitemap integrity.
- PR #31 (antigravity/reviewer-artifact-map-polish): Verify artifact maps and decision guidelines.
- PR #32 (antigravity/report-reviewer-queue-polish): Inspect action item backlogs and priority tags.
- PR #33 (antigravity/report-release-review-polish): Review handoff summaries and release bundle paths.
- PR #34 (antigravity/report-decision-summary-polish): Verify final decision badges and mock slates.
- PR #35 (antigravity/report-review-checklist-polish): Inspect handoff checklist checkbox elements.
- PR #36 (antigravity/report-final-review-polish): Review skip-link accessibility anchors.
- PR #37 (antigravity/report-final-qa-polish): Verify conservative empty-state fallback messages.
- PR #38 (antigravity/report-review-index-polish): Review glossary boxes and mockup index page headers.
- PR #39 (antigravity/report-reviewer-print-final): Verify print-safe panel colors and link audit checks.
- PR #40 (antigravity/report-reviewer-final-polish): Verify consolidated final QA checklists, glossaries, and print layouts.
- PR #41 (antigravity/report-reviewer-final-qa): Verify landing page QA metrics, sitemaps, link warnings, and copy consistency.
- PR #42 (antigravity/report-reviewer-review-guide): Current PR. Verify landing page stack review guides and evidence table styling.
Reviewer Final QA
Review Order: PR #24 → PR #42 (Chronological stacked branch path verification).
Current Stacked PR Lane: Antigravity report-reviewer-review-guide (Final reviewer stack flow documentation guide and evidence table styling).
Quality Gate Expectation: Local gate checks (pytest test suites, secret scans, provider decisions, artifact schemas) must pass 100% cleanly.
No Live Provider Calls: ENFORCED (All live provider integration calls are disabled by default).
Generated Artifact Changed: YES (Recompiled static HTML reports, sitemaps, checklists, and packet markdown files).
How to Review This Stack
Follow these steps to systematically verify the entire branch sequence:
- Step 1: Check baseline provider fixtures (Inspect PR #24 provider-samples-backlog and verify offline intake data models).
- Step 2: Verify adapter connections (Inspect PR #25 live-v1-provider-foundation and confirm zero live connection leaks).
- Step 3: Review UI report presentation (Verify navigation bars, progression lines, print-friendly CSS styles, and accessibility headings from PR #26 to PR #37).
- Step 4: Audit index links & glossary cards (Verify relative sitemap links, local hashes, and key terminology glossaries from PR #38 to PR #42).
- Step 5: Run quality gate validation (Execute
python run_quality_gate.pyto confirm sitemaps, secret scanning, and pytest suites pass cleanly).
Reviewer Glossary & Key Terms
No-Play: Candidate does not meet the necessary threshold requirements or is filtered out due to active risk gates. Systematic abstention is enforced to manage variance.
Review: Candidate is in a cautionary state requiring human verification (e.g., pending key player status or final inactives confirmations).
Risk Flag: An alert indicating potential elevated risk, such as extreme weather, rapid line movement, or defensive matchup anomalies.
Best Line: The optimal wagering line selected dynamically after shopping multiple books for the best available price.
Market Confirmation: Verification that a candidate is supported by consensus across multiple independent bookmakers.
Offline Fixture: Standard operating state using pre-packaged, offline fixture data for testing, simulation, and schema validation.
Live Disabled: Safe default status indicating that live REST or WebSocket network connections to third-party data providers are completely shut off.
Queue
| Priority | Action | Owner / Decision | Lane | Status | Reason | Evidence |
|---|---|---|---|---|---|---|
| HIGH | Continue Fixture-Mode QA | Codex Owned | offline_safe | Ready | Fixture mode is available and required scenarios are covered. | 20/20 fixture scenarios covered |
| HIGH | Review Provider Selection Rubric | Provider Review | decision_needed | Review | Provider defaults are ready for no-call adapter planning, but live provider calls still need separate approval. | approval_state=adapter_planning_only; adapter_planning_approved=7; live_approval_required=7 |
| MED | Review Sanitized Provider Schema Samples | Codex Owned | offline_safe | Ready | Provider-neutral schema samples are present; provider approval is still separate. | 7/7 provider sample families ready |
| HIGH | Review Live V1 Readiness Artifacts | Codex Owned | offline_safe | Ready | Provider-specific request plan, sample bundle, and manual injury overrides are no-call safe. | odds_plan_safe=True; sample_bundle_complete=True; manual_injuries_ready=True |
| HIGH | Review Live Switch Audit | Codex Owned | offline_safe | Ready | Live switches are declared, default-off, and safe for calls-only staging. | declared=True; default_false=True; calls_only=True; publish_allowed=False |
| P3 | Implement Live Provider Adapters | Codex Owned | live_blocked | Hold | Live mode remains disabled by design. | season phase is PRESEASON; runtime mode is not live |
| P3 | Wire GitHub And Cloudflare Pages | Codex Owned | publishing | Ready | Standalone GitHub repo is ready; Cloudflare Pages settings remain external. | 23 issue seeds ready/imported for remote planning |
| MED | Review Release Readiness Bundle | Ryan/Sacboyz Decision | offline_safe | Ready | Release readiness artifacts validate locally without live data. | release ready=True; live providers disabled |