Skip to content

docs: add evidence-gated sprint demo agenda - #1994

Merged
groupthinking merged 3 commits into
mainfrom
docs/confirm-sprint-demo-agenda
Sep 18, 2026
Merged

groupthinking merged 3 commits into
mainfrom
docs/confirm-sprint-demo-agenda

Conversation

@groupthinking

Copy link
Copy Markdown
Owner

Canonical issue

Closes #

Outcome

Adds a time-boxed internal agenda with owners, observable results, evidence commands, and fallback language for every demo segment.

Scope

  • Included: Verified Home-to-Studio, sign-in, QA, and production-boundary references.
  • Explicitly excluded: Application behavior, deployment, payment, OAuth, secrets, CI settings, and merges.

Risk

  • Risk level: low
  • Failure mode: A presenter could mistake a local check for remote proof.
  • Rollback: Revert commit 6332be0.

Verification

  • git diff --check
  • Validated required agenda markers and referenced local paths
  • Required CI
  • Review threads resolved

Production evidence

Not applicable. This PR explicitly distinguishes local demonstration evidence from preview and production results.

Agent handoff

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

@vercel

vercel Bot commented Sep 17, 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:07pm UTC

@coderabbitai

coderabbitai Bot commented Sep 17, 2026

Copy link
Copy Markdown
Contributor

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

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

Review profile: ASSERTIVE

Plan: Advanced

Run ID: 2d6f7b70-47d3-4136-9284-28dc725c2cc4


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.

@github-actions github-actions Bot added the documentation Improvements or additions to documentation label Sep 17, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Snapshot Warnings

⚠️: No snapshots were found for the head SHA 6332be0.
Ensure that dependencies are being submitted on PR branches. Re-running this action after a short time may resolve the issue. See the documentation for more information and troubleshooting advice.

Scanned Files

None

@groupthinking
groupthinking marked this pull request as ready for review September 18, 2026 20:51
Copilot AI balanced review requested due to automatic review settings September 18, 2026 20:51
@groupthinking

Copy link
Copy Markdown
Owner Author

@copilot resolve the merge conflicts in this pull request

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟡 Changes recommended

Several pull-request statuses are stale, and the task references do not render visibly.

Get a fresh assessment by requesting another Copilot review.

Pull request overview

Adds an internal, evidence-gated sprint demo agenda for the UVAI Home-to-Studio workflow.

Changes:

  • Defines a time-boxed demo with owners and fallbacks.
  • Documents local verification commands and production boundaries.
  • Adds closing records and task references.
File summaries
File Description
docs/SPRINT_DEMO_AGENDA.md Adds the sprint demonstration agenda and evidence requirements.
Review details

Suppressed comments (2)

docs/SPRINT_DEMO_AGENDA.md:47

  • This required script still calls PR #1984 a draft even though GitHub currently marks it open and non-draft. Make the fallback conditional on live PR state so the presenter does not repeat stale evidence.
**Google sign-in change not yet reviewed:** “The retry behavior is a tested candidate in draft PR #1984. It does not change OAuth credentials, callback policy, or the server authentication gate, and it is not presented as a merged result.”

docs/SPRINT_DEMO_AGENDA.md:58

  • These facts are already stale: PR #1981 merged and issue #1975 closed on 2026-09-18, while PR #1984 is open and non-draft. Since the agenda's purpose is accurate evidence reporting, record durable outcomes and require live lookup for statuses that can still change.
1. The landing-page Notion task remains **In progress**. [PR #1976](https://github.com/groupthinking/EventRelay/pull/1976) is closed as a duplicate; [PR #1981](https://github.com/groupthinking/EventRelay/pull/1981) is the canonical landing-page implementation and remains blocked on preview-dependent evidence.
2. Google sign-in recovery is represented by draft [PR #1984](https://github.com/groupthinking/EventRelay/pull/1984). The local test coverage is relevant evidence, but open remote Vercel and E2E failures prevent a readiness claim.
  • Files reviewed: 1/1 changed files
  • Comments generated: 2
  • Review effort level: Balanced

💡 Configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment on lines +16 to +17
| 4:30–6:00 | **Engineering owner: _assign_** | Explain the Google sign-in recovery behavior: rejected or non-navigating initiation returns the button to a retryable state with generic error copy. | The audience sees the code or test narrative, not a real OAuth completion. | Draft [PR #1984](https://github.com/groupthinking/EventRelay/pull/1984) contains the implementation and its component test. Only demonstrate it as a reviewed outcome after review and its remote check failures are resolved. Until then, explain it as a tested candidate. |
| 6:00–7:30 | **QA owner: _assign_** | Summarize the local quality gates for the two sprint changes. | The team sees exact commands and the distinction between local tests and remote checks. | Reference [draft QA checklist PR #1993](https://github.com/groupthinking/EventRelay/pull/1993). If that document is still pending review, present the commands directly and do not claim full release readiness. |
Comment on lines +64 to +67
[1]: https://app.notion.com/p/3dd3c2339c0481439d43eafdeacd4136?pvs=204 "Design sprint landing page task"
[2]: https://app.notion.com/p/3dd3c2339c0481a2a35dce563f281ab1?pvs=204 "Fix login bug task"
[3]: https://app.notion.com/p/3dd3c2339c04810ca45be4d289f3e240?pvs=204 "Confirm sprint demo agenda task"
[4]: https://app.notion.com/p/3dd3c2339c0481019c8fdf22053d0a4d?pvs=204 "Prepare QA checklist task"
…emo-agenda

# Conflicts:
#	docs/SPRINT_DEMO_AGENDA.md

Co-authored-by: groupthinking <154503486+groupthinking@users.noreply.github.com>

Copilot AI commented Sep 18, 2026

Copy link
Copy Markdown
Contributor

@copilot resolve the merge conflicts in this pull request

Resolved by merging the latest main into this branch and fixing the conflict in docs/SPRINT_DEMO_AGENDA.md in commit 296bd3d.

@groupthinking
groupthinking merged commit ad22cc6 into main Sep 18, 2026
10 of 11 checks passed
@groupthinking
groupthinking deleted the docs/confirm-sprint-demo-agenda branch September 18, 2026 21:05
@linear-code

linear-code Bot commented Sep 18, 2026

Copy link
Copy Markdown
Contributor

GRV-608

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants