Phase 3 polish + 13MSG E2E test handoff to DCT - #26
Merged
Conversation
Per DCT's 12MSG response: DCT does not have JSON merge functionality and won't add it. The README documentation pattern is the canonical answer for VS Code workspace settings in templates. readme-structure.md: new "VS Code settings pattern" section - Rule: templates must not ship .vscode/settings.json or .vscode/extensions.json files (would risk overwriting user config) - Pattern: document the setting in the README under a "VS Code tip" section, with the literal one-line addition the user pastes into their workspace settings - Four rules: symptom sentence, fix sentence, copy-pasteable JSON, "don't replace the file" reminder - Universally-needed settings belong in DCT's base devcontainer image, not template files Investigation: marked 11MSG and 12MSG as resolved (done by DCT and TMP). Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Phase 3 (cosmetic) — done:
- A1: Created postgresql-demo-logo.svg (PostgreSQL blue + "PG")
- A5: Already done in Phase 2 — JSON example uses Xa7mP9... placeholder
- B10: Created python-basic-webserver-database-logo.svg (Python blue
with DB badge); template-info.yaml updated to reference it
- C3: Tag scheme documented in naming-conventions.md with categories,
examples, and common mistakes. Fixed three inconsistencies:
- golang-basic-webserver: dropped redundant `go`, kept `golang`
- typescript-basic-webserver: dropped `nodejs` (runtime is implied
by language)
- python-basic-webserver-database: dropped misleading `demo` tag
(it's a real-use template, not a demo)
- C4: Created webserver-database-logo.svg category logo (navy + WS+DB);
templates/template-categories.yaml updated to reference it
Phase 1 + Phase 3 of the investigation are now both shipped. Phase 2
remains deferred (quality-of-life improvements awaiting real feedback).
Investigation: 13MSG added — TMP → DCT handoff for the E2E test that
DCT had as their last Phase 1 work item (Phase 3 of PLAN-p1-dct-shim.md,
blocked on TMP's README rewrites). The rewritten README is now the test
plan — DCT just follows the canonical 7-step workflow literally and
confirms /tasks returns the seeded rows.
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Ships Phase 3 (cosmetic polish) of the template docs investigation, plus a handoff message (13MSG) telling DCT they're unblocked for their E2E test.
Phase 3 — Polish
postgresql-demo-logo.svg(PostgreSQL blue + "PG")Xa7mP9...placeholder textpython-basic-webserver-database-logo.svg(Python blue with DB badge); template-info.yaml updatednaming-conventions.mdwith categories, examples, and common mistakes. Fixed three inconsistencies:golang-basic-webserver: dropped redundantgo, keptgolangtypescript-basic-webserver: droppednodejs(runtime is implied by language)python-basic-webserver-database: dropped misleadingdemotag (it's a real-use template, not a demo)webserver-database-logo.svgcategory logo (navy + WS+DB); template-categories.yaml updated13MSG — TMP → DCT handoff
DCT's Phase 3 of PLAN-p1-dct-shim.md was an E2E test in a fresh devcontainer with TMP's rewritten templates. Blocked on TMP's README rewrites — which are now shipped. 13MSG explicitly hands off:
python-basic-webserver-databaseREADME/tasksreturns the 3 seeded rowspostgresql-demo14MSG: DCT → TMP: ...)Phase status after this PR
Test plan
validate-metadata.shpassesvalidate-docs.shpasses (0 errors)npm run buildpasses in devcontainer🤖 Generated with Claude Code