Skip to content

chore: remove embedded squad (infra + config)#475

Merged
primetimetank21 merged 7 commits into
developfrom
chore/remove-embedded-squad
Jun 11, 2026
Merged

chore: remove embedded squad (infra + config)#475
primetimetank21 merged 7 commits into
developfrom
chore/remove-embedded-squad

Conversation

@primetimetank21

Copy link
Copy Markdown
Owner

Summary

Removed dev-setup's own embedded squad infrastructure and squad-cli installation. This is Phase 1 of the de-squad initiative.

What was removed

Squad coordination directory:

  • Deleted .squad/ (entire embedded squad coordination structure)

GitHub automation:

  • Deleted .github/agents/squad.agent.md
  • Deleted 6 squad workflow files: squad-heartbeat.yml, squad-history-check.yml, squad-issue-assign.yml, squad-label-enforce.yml, squad-triage.yml, sync-squad-labels.yml

Scripts and tools:

  • Deleted squad-spawn.ps1/.sh and lint-spawn-prompt.ps1/.sh scripts
  • Deleted squad-cli.ps1/.sh tool installers
  • Deleted est_squad_spawn.ps1/.sh and est_spawn_prompt_lint.ps1/.sh tests

Copilot skills:

  • Deleted 8 skill directories: squad-conventions, cross-squad, personal-squad, worktree-base-refresh, �gent-collaboration, �gent-conduct, init-mode, cli-wiring

Config and test updates

  • Removed squad-cli from .tool-versions (no longer installed)
  • Removed squad-cli row from README.md tools table
  • Removed squad runtime .gitignore entries (logs, inbox, sessions, .squad-workstream)
  • Removed squad .gitattributes merge=union block
  • Removed SQUAD_WORKTREES env var from .devcontainer/devcontainer.json
  • Updated pre-commit hook: removed rogue .squad/ path check, staged inbox guard, and history.md size gate (reduced from 7 checks to 4)
  • Updated est_precommit_hygiene.sh: removed tests for deleted squad checks
  • Updated �alidate.yml: removed est_squad_spawn and est_spawn_prompt_lint CI steps
  • Updated �2e-install.yml: removed squad-cli installation checks
  • Updated CONTRIBUTING.md: replaced squad-cli with gh in tool version pin examples
  • Updated README.md: removed squad-cli from repo structure diagram and updated pre-commit hook description

What's NOT in this PR

  • Docs narrative rewrite (README/ARCHITECTURE/CONTRIBUTING broad squad sections) — deferred to separate PR (Tech)
  • Repo label deletion — handled separately via GitHub UI

Relates to

De-squad initiative (Phase 1/PR 1)

Co-authored-by: Copilot 223556219+Copilot@users.noreply.github.com

primetimetank21 and others added 7 commits June 11, 2026 00:24
Removed dev-setup's own .squad/ directory and all squad-specific:
- Deleted .squad/ (entire embedded squad coordination structure)
- Deleted .github/agents/squad.agent.md
- Deleted 6 squad workflow files (heartbeat, history-check, issue-assign, label-enforce, triage, sync-squad-labels)
- Deleted squad-spawn.ps1/.sh and lint-spawn-prompt.ps1/.sh scripts
- Deleted squad-cli.ps1/.sh tool installers
- Deleted test_squad_spawn.ps1/.sh and test_spawn_prompt_lint.ps1/.sh tests
- Deleted 8 copilot skill directories (squad-conventions, cross-squad, personal-squad, worktree-base-refresh, agent-collaboration, agent-conduct, init-mode, cli-wiring)

Config and test updates:
- Removed squad-cli from .tool-versions (no longer installed)
- Removed squad-cli row from README.md tools table
- Removed squad runtime .gitignore entries (logs, inbox, sessions, .squad-workstream)
- Removed squad .gitattributes merge=union block
- Removed SQUAD_WORKTREES env var from .devcontainer/devcontainer.json
- Updated pre-commit hook: removed rogue .squad/ path check, staged inbox guard, and history.md size gate (reduced from 7 checks to 4)
- Updated test_precommit_hygiene.sh: removed tests for deleted squad checks
- Updated validate.yml: removed test_squad_spawn and test_spawn_prompt_lint CI steps
- Updated e2e-install.yml: removed squad-cli installation checks
- Updated CONTRIBUTING.md: replaced squad-cli with gh in tool version pin examples
- Updated README.md: removed squad-cli.sh/.ps1 from repo structure diagram and updated pre-commit hook description

Docs narrative rewrite (README/ARCHITECTURE/CONTRIBUTING broad squad sections) deferred to separate PR.
Repo label deletion handled separately - not part of this PR.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
ROOT CAUSES FIXED (CI-breaking):
- e2e-install.yml: restored Node version gate to proper multi-line bash
  (backtick-n sequences triggered command substitution)
- scripts/linux/setup.sh: removed squad-cli run_tool call
- scripts/windows/setup.ps1: removed squad-cli.ps1 dot-source and
  Install-SquadCli call
- tests/test_nvm_bootstrap.sh: removed all squad-cli test cases (T5-T13)
- tests/test_windows_setup.ps1: removed Groups G, U, and squad-cli cases
  from Groups X, DD

NON-BREAKING RESIDUE REMOVED:
- scripts/windows/tools/profile.ps1: removed Invoke-CopilotSquad and
  gosquad alias
- tests/test_alias_parity.sh: removed Invoke-CopilotSquad from internal
  function exclusion list
- tests/test_precommit_hygiene.sh: updated squad/* branch references to
  feature/* (tests now generic)
- scripts/lib/ascii-sweep.py: changed comment from "Squad status emoji"
  to "Status emoji"
- scripts/sprint-end-labels.sh/.ps1: changed "squad/priority" label
  comment to "owner/priority" (scripts are GENERAL release tooling, KEPT)
- .github/workflows/e2e-install.yml: removed squad CLI assertions
- .github/workflows/sprint-end-labels.yml: updated label comment

DECISION: sprint-end-labels scripts KEPT. They are general release-label
automation tools (remove release:backlog, add release:shipped-X.Y.Z).
The "squad" references were only in label-type documentation
(type/area/squad/priority) describing general GitHub label categories.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@primetimetank21 primetimetank21 merged commit baee5d9 into develop Jun 11, 2026
10 checks passed
@primetimetank21 primetimetank21 deleted the chore/remove-embedded-squad branch June 11, 2026 05:23
primetimetank21 added a commit that referenced this pull request Jun 13, 2026
* chore: de-squad mechanical scrub (skills, plans, config, headers)

Removed 8 squad-product skill dirs:
- distributed-mesh, economy-mode, model-selection, nap
- reskill, humanizer, external-comms, session-recovery

Reframed error-recovery skill to neutral agent terminology.

Scrubbed 16 keeper skills for branding:
- git-workflow: updated to feat|fix|chore|docs|refactor/{issue}-{slug} pattern (aligns #476)
- Neutralized cast-name references in git-workflow, changelog-fold-completeness,
  windows-compatibility, gh-auth-isolation (e.g., "Mickey" -> "the release process",
  "Squad runs on" -> "This project runs on", removed squad repo examples)

Deleted 18 grill-round plan artifacts (441-grill-*.md variants).

Scrubbed 3 keeper plan docs (metadata only):
- 441-profile-path.md, 451-pwsh-parity-gaps.md, 468-customizable-install.md
- Replaced cast-name author/reviewer attributions with "Contributor"/"revision N"

Neutralized branding in config files:
- .gitconfig.template: "whole squad" -> "convenient git shortcuts"
- .mailmap: "Squad agent fake-emails" -> "Local automation identity mappings"

Removed Owner: headers from ~20 script files (comment-only scrub).

Fixed pre-existing non-ASCII in auth.sh and uv.sh (em-dash -> double-dash).

All edits ASCII-only (pre-commit Check 2 enforced).

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* docs: de-squad narrative (README, ARCHITECTURE, CONTRIBUTING, CHANGELOG, dotfiles)

Removes embedded-squad narrative from documentation while preserving all
legitimate dev-setup facts:

README.md:
- Removed squad roster section, .squad/ directory tree, gosquad alias
- Removed squad-cli from tool list and .tool-versions examples
- Updated branch naming from squad/{issue}-{slug} to feat|fix|chore|docs|refactor/{issue}-{slug}
- Neutralized squad automation references to "automation"
- Removed pre-commit .squad/ path allow-list references
- Removed squad label references from sprint-end-labels documentation

ARCHITECTURE.md:
- Removed Owner attributions (Mickey, Donald, Goofy, Pluto, Chip, etc.)
- Removed .squad/ directory tree from file structure
- Removed squad-cli from dependency chains and tool tables
- Removed squad automation workflows from file tree
- Removed Squad Roster and Team Ownership Map sections
- Removed squad-cli from tool version pinning list
- Neutralized squad references in hooks/workflows documentation

CONTRIBUTING.md:
- Removed "Disney Classic Squad" intro
- Updated branch naming from squad/{issue}-{slug} to feat|fix|chore|docs|refactor/{issue}-{slug}
- Changed "Mickey approval required" to "Approval from a maintainer required"
- Neutralized cast-name references to role titles
- Removed .squad/decisions/inbox/ references from hotfix audit trail
- Removed Squad Operational Gates section

CHANGELOG.md:
- Added new Unreleased entry documenting embedded-squad removal (PR #475, #477)
- Removed author attributions from recent entries
- Removed .squad/ path references from recent entries
- Removed squad-cli from tool names in recent entries
- Left older historical entries mostly intact per Option C hybrid approach

config/dotfiles/README.md:
- Removed "Managed by Pluto" attribution
- Removed gosquad alias from shell aliases table

All files remain ASCII-only (pre-commit Check 2 compliant).

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* fix: restore gosquad dev alias (over-scrubbed in de-squad)

gosquad (copilot --agent squad --yolo) is a personal dev launcher, not
embedded-squad infra. Restore it with bash/zsh + Windows parity and its
doc entries; correct the CHANGELOG to not claim its removal.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* fix(tests): exclude Invoke-CopilotSquad from alias parity check

gosquad alias restored in #478 but its PS wrapper function leaked through
the parity test's function extractor. Add it to the exclusion regex.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

---------

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
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