fix(templates): install to .squad/templates/ not .squad-templates/#185
Merged
bradygaster merged 1 commit intobradygaster:devfrom Mar 4, 2026
Merged
Conversation
squad.agent.md references .squad/templates/ throughout, but index.js was installing template files to .squad-templates/ at the repo root. This mismatch meant the coordinator prompt could never find its templates on a fresh install. Root cause: PR bradygaster#113 migrated all path references in squad.agent.md to .squad/templates/ but PR bradygaster#111's code change (which renamed the dir to .squad-templates/) was never reconciled with it. Both landed in v0.5.0. Fix: - Change install destination from .squad-templates to .squad/templates - Update migrate-directory rename to match the new path - Update help text and replace-regex to reference .squad/templates - Add a v0.5.5 migration entry that moves .squad-templates/ to .squad/templates/ for existing users upgrading from v0.5.4 or earlier Bumps version to 0.5.5 so the migration runs on upgrade.
bradygaster
added a commit
that referenced
this pull request
Mar 4, 2026
…tes (#185, #188, #191, #192, #195, #196, #199, #201, #203, #206, #207) Documentation Epic #182 — complete: Docs Content (McManus): - Architecture overview: SDK ↔ CLI ↔ SquadUI system design - Migration guide: Beta → v1 with 10-step checklist - Global CLI install guide: npm, npx, GitHub native - VS Code integration guide: client compatibility, extension patterns - SDK API reference: 574 lines, all 30+ exports documented Docs Site Engine (Keaton): - Static site generator: node docs/build.js → docs/dist/ - GitHub Pages ready, responsive design, sidebar nav - Index landing page linking all guides Mechanical Updates (Fenster): - .ai-team/ → .squad/ across 25 doc files (#191) - CLI invocation references verified current (#192) - Beta repo URLs updated to squad-pr (#195) Docs Tests (Hockney): - 17 docs validation tests: headings, code blocks, links, build - Fixed link checker for parent-dir refs, Windows rmSync Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Owner
|
Hey @williamhallatt — thank you so much for this contribution! 🙏 This is a great catch. The .squad-templates/ → .squad/templates/ path mismatch has been a silent bug since v0.5.0 and you tracked it down beautifully. The migration entry for existing users upgrading from ≤v0.5.4 is a really thoughtful touch — that's the kind of detail that saves people real headaches. Welcome to the Squad! �� |
bradygaster
added a commit
that referenced
this pull request
Mar 4, 2026
# Conflicts: # package.json
Closed
6 tasks
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
2 participants
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.
What
index.jsinstalls template files to.squad-templates/at the repo root, butsquad.agent.mdreferences.squad/templates/throughout. On a fresh install the coordinator prompt can never find its templates.Why
PR #113 migrated all path references in
squad.agent.mdto.squad/templates/but PR #111's code change (which renamed the dir to.squad-templates/) was never reconciled with it. Both landed in v0.5.0 and the mismatch has been live since.Changes
.squad-templates→.squad/templatesmigrate-directoryrename path to match.squad/templates.squad-templates/→.squad/templates/for existing users upgrading from ≤v0.5.4 (uses the existingsafeRenamehelper)0.5.5so the migration runs on upgradeTesting
All 95 existing tests pass. The 6 assertions in
test/migrate-directory.test.jswere updated to match the corrected path.