Fold brand-build field gotchas into the demo corpus (4.1.0)#45
Merged
Conversation
Second batch of live-verified findings from the autonomous partner-simulation build (fresh DW 10.27.4, skills followed verbatim), now through brand re-content, MCP catalog authoring, feature-pack install, and a headless storefront on a second backend. Nine folds across four skills: - Autonomous JSON-RPC MCP transport fallback (dw-demo-base mcp-setup.md) - Root / binding via Area.AreaDomain+AreaFrontpage, no AreaDns on 10.27.x (dw-demo-swift) - Area/style/item-type restart semantics + nav-label-is-data (cache-invalidation.md) - Index-instance Warning benign (dw-demo-pim canonical-setup-order Step 16) - Isolated pack-fragment staging (dw-demo-swift pack-activation.md) - MCP recipe gotchas batch (pim-modelling, search-indexing, extend-mcp-tools, content-modelling) - Product-completeness checklist (dw-demo-pim canonical-setup-order) - Headless drift notes: version-dependent status, dual image keys, env-configurable repo/query names, empty area ecom bindings (dw-demo-headless) Minor bump (substantial new guidance across 4 skills). Stacks on PR #44 (4.0.2). Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01BpTYGpMUm8MkKqYSpZVzd5
4ad55fb to
b632f2c
Compare
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
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.
Provenance
Same autonomous partner-simulation build as #44 — a fresh DW 10.27.4 host with the skills followed verbatim — now carried through full brand re-content, catalog authoring via MCP recipes, feature-pack install, and a headless storefront on a second backend. Every finding below was verified live during that run.
Stacking
This PR STACKS on #44 (
fold/fresh-install-and-deserialize-gotchas). It is branched off #44's head, notv2, because both touch the same files (mcp-setup.md,deserialize-flow.md, the foundational candidates). Base is set tov2per instruction; until #44 merges, this PR's diff will also show #44's commit. Merge #44 first, then this.The 9 folds — where each landed
dw-demo-base/references/mcp-setup.md(new section). The Claude-client approval is interactive-only; an unattended agent hangs on "Pending approval". The DW/admin/mcpendpoint is plain JSON-RPC 2.0 — with the API-Key bearer the full surface (~393 tools on 10.27.x) is directly callable (initialize→tools/list→tools/call), with the caution that it bypasses the client's approval layer so guarded-writes discipline still applies./binding on 10.27.x →dw-demo-swift/references/deserialize-flow.md§7. Root 404s post-deserialize; binding isArea.AreaDomain+Area.AreaFrontpage; noAreaDnstable on 10.27.x; restart required.dw-demo-base/references/foundational/cache-invalidation.md. Three restart-only rows (Area/style/item-type — startup-materialised,CacheInformationRefreshinsufficient); bulkGetServiceCachesflush can500; nav/menu labels are group-tree data, not a "nav cache".dw-demo-pim/references/canonical-setup-order.mdStep 16 (both variants). A Warning from an unbuilt secondary balancer instance is a false alarm — judge by the primary instance's build result + doc count.dw-demo-swift/references/pack-activation.md§8. Staging a fragment into aSerializeRootthat still holds base trees re-deserializes the base seed (resurrects the purged sample catalog on a re-contented demo). Now parks/clears base trees, stages isolated, restores.dw-demo-base/references/surface-priority.md. Made explicit that BULK REMOVAL of sample data during brand re-content is exactly the last-resort clause (a) cleanup/teardown — not create-shaped, not recipe-gated.foundational/pim-modelling.md(§2.5 variant combos NULL Active/Price; §2.10import_product_images_from_urlssets no default → Swift card NRE degrades the whole PLP),foundational/search-indexing.md(custom fields index asCustomField_<SystemName>, other patterns fail silently;synchronous:truedoes not block — poll),foundational/extend-mcp-tools.md§5 +foundational/content-modelling.md(save_pagesignoresurlName— slug derives frommenuText).dw-demo-pim/references/canonical-setup-order.md(new closing section). Per-product + per-variant gate: Active, priced, stocked-or-NeverOutOfStock, default image, texts in every language layer — each with its frontend symptom, run as a SQL sweep.dw-demo-headlessdrift notes →headless-backend.md+headless-frontend.md: two-token trap status is version-dependent (404 on 10.26.x, 400 on 10.27.x) → "a non-401 error", don't pin a code (§3); images live underassetCategoriesORimagePatternImages— read both (frontend §2); repository/query names must be env (query name without.query) so a second-backend swap is pure env (frontend §3 + backend §5); areas can ship empty ecom bindings (ecomShopId="") → passLanguageId/ShopIdexplicitly on every call (backend §4).Version decision — minor (4.1.0)
Bumped 4.0.2 → 4.1.0 (minor), not patch. Corpus precedent reserves patch (4.0.1, 4.0.2) for tight single-theme fixes to a broken documented flow; minor (3.7.0/3.8.0 substantial multi-file guidance reworks; 3.9.0 new surface) is used for substantial additive guidance. This PR adds substantial new guidance — a new transport section, a new checklist, a recipe-gotcha batch — across four skills (base, swift, pim, headless), overwhelmingly "Added". That exceeds the patch bar and matches the minor bar. Not breaking, so not major.
Validation
python scripts/validate-skills.py→ 0 errors, 0 warningsnode scripts/build-manifest.mjs --check→ manifest up to date (32 skills)justdynamics/Truvio.Commerce.*URLs untouched)🤖 Generated with Claude Code
https://claude.ai/code/session_01BpTYGpMUm8MkKqYSpZVzd5