Skip to content

docs: finalize feat_website_walkthrough_guides (PR #448 merged)#449

Merged
SoundMindsAI merged 1 commit into
mainfrom
docs/finalize-website-walkthrough-guides
Jun 4, 2026
Merged

docs: finalize feat_website_walkthrough_guides (PR #448 merged)#449
SoundMindsAI merged 1 commit into
mainfrom
docs/finalize-website-walkthrough-guides

Conversation

@SoundMindsAI

Copy link
Copy Markdown
Owner

Summary

Post-merge finalization for #448 (feat_website_walkthrough_guides, squash-merged 36932256).

Docs-only — no code changes.

Test plan

  • Pre-commit hooks pass (dashboard regen lockstep, two-shot commit)
  • No code touched

🤖 Generated with Claude Code

- pipeline_status.md → Implementation Complete + Release: mvp2 marker
- implementation_plan.md status → Complete (PR #448, squash 3693225)
- state.md: prepend PR #448 merge one-liner, refresh branch/focus context
- move planned_features/02_mvp2/feat_website_walkthrough_guides →
  implemented_features/2026_06_04_website_walkthrough_guides/ (flat, date-prefixed)
- dashboards + public roadmap regenerated by the pre-commit hook

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Signed-off-by: SoundMindsAI <eric.starr@soundminds.ai>

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Code Review

This pull request updates the project's dashboards, roadmap, and state documentation to mark the feat_website_walkthrough_guides feature as completed. However, a critical issue was identified where the feature's folder name is missing the required feat_ prefix. This omission causes the automated dashboard generator to fail to recognize the feature as completed, resulting in incorrect metrics and omissions across the generated documentation files. Renaming the folder to include the prefix and regenerating the dashboards is required to resolve this issue.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

Comment on lines +33 to +35
- Status: Complete
- Date: 2026-06-04
- PR: #448 (squash-merged `36932256`)

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

high

⚠️ Critical Dashboard/Roadmap Generation Issue: Missing feat_ Prefix in Folder Name

The feature folder was renamed to implemented_features/2026_06_04_website_walkthrough_guides/ instead of implemented_features/2026_06_04_feat_website_walkthrough_guides/ (missing the feat_ prefix).

Because of this missing prefix, the dashboard generator script (scripts/build_public_roadmap.py / make dashboard) does not recognize it as a completed feature. This has caused several silent failures in the generated files in this PR:

  1. Roadmap Omission: The feature is completely missing from the "Done" list in website/docs/roadmap.md.
  2. Dashboard Omission: It is missing from the "Done" list in MVP2_DASHBOARD.md and mvp2_dashboard.html.
  3. Incorrect Metrics: The completed features count remains at 18 instead of incrementing to 19, and the total specced features count incorrectly decreased from 28 to 27 instead of remaining at 28.

To resolve this and restore correct metrics and roadmap generation, please rename the folder to:
docs/00_overview/implemented_features/2026_06_04_feat_website_walkthrough_guides/

And then re-run the dashboard regeneration hook/script.

@SoundMindsAI SoundMindsAI merged commit 3924175 into main Jun 4, 2026
8 checks passed
@SoundMindsAI SoundMindsAI deleted the docs/finalize-website-walkthrough-guides branch June 4, 2026 17:07
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