-
Notifications
You must be signed in to change notification settings - Fork 2
docs: finalize feat_home_first_run_demo_nudge post-PR-188 merge #189
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Large diffs are not rendered by default.
| Original file line number | Diff line number | Diff line change | ||||
|---|---|---|---|---|---|---|
|
|
@@ -20,4 +20,15 @@ | |||||
| - Phases covered: Phase 1 (banner + badges + CI guard). Phase 2 (reseed endpoint + UI) remains deferred per [`phase2_idea.md`](phase2_idea.md). | ||||||
|
|
||||||
| ## Implementation | ||||||
| - Status: Not started | ||||||
| - Status: Complete | ||||||
| - Date: 2026-05-22 | ||||||
| - PR: #188 (squash `21325432`) | ||||||
| - CI: green (5/5 jobs: backend lint+typecheck+pytest+coverage, backend unit fast lane, frontend lint+typecheck+vitest+build, docker buildx, operator-path smoke) | ||||||
| - Stories completed: 12 (1 in Epic 1, 4 in Epic 2, 4 in Epic 3, 3 in Epic 4) | ||||||
| - Cross-model review: Gemini Code Assist 3 Medium (2 accepted in `cb0bdc4` — `useClusters({ enabled })` + parity-guard regex broadening; 1 rejected with counter-evidence — `useMemo` would save nothing given React's render model + early-return). Final GPT-5.5: 0 High / 0 Medium / 2 Low (1 fixed in `2cdc44a` — banner-comment accuracy; 1 deferred to this finalization step). | ||||||
| - Phase 2 (reseed endpoint + UI): split out at finalization to a new planned folder [`feat_home_demo_reseed_endpoint`](../feat_home_demo_reseed_endpoint/idea.md) so it surfaces in `/pipeline --status`. | ||||||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. The relative path to the new planned folder is incorrect. It should point to
Suggested change
|
||||||
|
|
||||||
| ## Done | ||||||
| - Date: 2026-05-22 | ||||||
| - Tag: none (feature ships as part of MVP1; no separate release) | ||||||
| - Implemented-features path: `docs/00_overview/implemented_features/2026_05_22_feat_home_first_run_demo_nudge/` | ||||||
Large diffs are not rendered by default.
Large diffs are not rendered by default.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The relative path to the deferred idea is incorrect.
../../from this directory resolves todocs/00_overview/, but theplanned_featuresfolder is located indocs/02_product/.