Remove the DW_VAULT model: per-demo artifact downloads from the named distribution repos (4.0.0)#42
Merged
Merged
Conversation
…base Drop the shared machine-wide vault (:DW_VAULT, five slots, INDEX.md) from the foundation skill. Add a versions prompt (DW10 + Swift version, recorded in CUSTOMISATIONS.md) and a per-demo artifact-download model: baselines, themes, and feature packs download into <demo-root>\baselines\ from the named ecosystem repos. setup-checks now probes gh CLI + writable baselines\ + versions instead of the DW_VAULT env var and slot inventory. Repoint every dw10source citation to a local clone of the DW10 source (location per machine, ask/discover). DB fast-restore becomes a per-machine local-artifact note.
Repoint the Swift style assets at the DemoThemes release (downloaded per-demo into baselines\themes\), default DW_BASELINE_REPO / DW_PACKS_REPO to the named ecosystem repos (justdynamics/Truvio.Commerce.Serializer.Baselines and .FeaturePacks), and name the DemoThemes repo directly. Drop the $env:DW_VAULT path-resolution rows, 'vault resolution' / 'vault resolved' wording, and the no-resolved-DW_VAULT sibling clauses from the swift/pim/erp/bc skills. Repoint dw10source citations to a local clone of the DW10 source.
Both encode the shared-vault model that no longer exists: compare-vault.md detected cross-machine vault drift and INDEX.md.template was the five-slot vault index. All links and cross-refs to both were already removed in the preceding commits.
The truvio scrub is lifted for the ecosystem repo URLs (operator policy), so the validator's check_no_truvio purge check and its now-unused TEXT_SUFFIXES constant are removed; the rest of the validator is intact. Bump marketplace.json to 4.0.0 (breaking distribution-model change; avoids the 3.9.0 collision held by an open PR) and record the DW_VAULT removal + per-demo download model in the changelog. Update the vault-model wording in README and CLAUDE.
Justinvolved
force-pushed
the
fold/vault-removal
branch
from
0e50076 to
a61051b
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
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.
Completes the distribution migration #39 started: the shared machine-wide vault (
$env:DW_VAULT+ five slots) is removed from the corpus entirely and replaced by the per-demo download model.<demo-root>\baselines\: serialized baselines from https://github.com/justdynamics/Truvio.Commerce.Serializer.Baselines, themes/style assets from https://github.com/justdynamics/Truvio.Commerce.DemoThemes, feature packs from https://github.com/justdynamics/Truvio.Commerce.FeaturePacks (gh release download;$env:DW_BASELINE_REPO/$env:DW_PACKS_REPOremain as per-machine overrides). Vault-layout section removed; setup-checks probes gh CLI + baselines-folder writability instead of vault slots.references/compare-vault.md,assets/INDEX.md.template.check_no_truviopurge check removed — the ecosystem repo URLs are now deliberately named (maintainer policy change). All other sanitization rules unchanged.git grep -i dw_vault -- skills/→ 0 hits. Validator 0 errors / 0 warnings; manifest up to date. Breaking distribution-model change → 4.0.0.STACKING NOTE: stacks on #41 (
fold/pr39-review-fixes). Merge #41 first, then this PR's diff against v2 is clean.