docs: strip grill changelogs + cast refs from docs/plans (#Phase2 WI-3)#485
Merged
Conversation
Implements WI-3 of the Phase 2 docs-alignment plan (ref: .squad/repos/dev-setup/plans/phase2-docs-alignment.md, Section 4 WI-3). What was removed: - 441-profile-path.md: deleted 4 grill changelog blocks (## v4 / v5 / v5.1 / v5.2 Changes) at top of file; stripped cast-name attribution *(updated v4 -- Doc/Chip)* from one body heading. - 451-pwsh-parity-gaps.md: deleted ## Revision History section (end of file); stripped inline cast-name refs from 6 body locations (CI Integration heading, T7 detail heading, Risk Assessment parenthetical, chip/history.md dead path, Open Questions attribution, Out-of-Scope heading). - 468-customizable-install.md: deleted all 10 grill changelog blocks (## v5 through ## v14 Changelog, ~170 lines); stripped inline cast-name parenthetical attributions from 3 body locations (Real-Defaults Drift Test heading, Blocker Resolution Map Donald-1 and Duck-2 row labels); added one-line NOTE banner near squad-cli sections flagging staleness. squad-cli content in 468 deliberately preserved -- feature-correctness deferred to #468 implementation (per Phase 2 D4). The pre-WI-3 file had 21 squad-cli/ Install-SquadCli references: 10 were in the changelog blocks (correctly deleted with the changelogs), 11 body references remain untouched (DEFAULT_TOOLS arrays, ToolRegistry, Fixture Provenance, Graceful Degradation). This matches the authoritative plan appendix which explicitly states v10/v11/v12 changelog squad-cli refs are DELETED by WI-3. ASCII-clean: zero bytes > 0x7F in all 3 files post-scrub. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.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
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.
WI-3: Strip grill changelogs + cast-name refs from docs/plans/
Implements WI-3 of the approved Phase 2 docs-alignment plan
(
/.squad/repos/dev-setup/plans/phase2-docs-alignment.md, Section 4 WI-3, Sections 2 D3/D4).Per-doc summary of changes
441-profile-path.md
## v5 Changes,## v5.2 Changes,## v5.1 Changes,## v4 Changes) including all reviewer-finding tables with Pluto/Chip/Doc labels. These sat between the YAML front-matter and the first spec section (## 1. Problem).*(updated v4 -- Doc/Chip)*from body headingv3-D4; substantive content of that decision preserved verbatim.451-pwsh-parity-gaps.md
## Revision Historysection (end of file); front-matterupdatedfield + git log provide sufficient history.## CI Integrationheading: removed(v2 addition -- resolves Mickey BLOCKING-1/BLOCKING-2, Goofy Finding-1)T7 detailheading: removed(v2 -- replaces vague "assert shebang valid", resolves Mickey MAJOR-2)(v2 decision -- resolves Mickey MAJOR-1 / Goofy T_C/T_D note)from one sentenceUse established ASCII safety pattern from chip/history.md->Use the established ASCII safety pattern(dead squad artifact path removed)RESOLVED (by Goofy).->RESOLVED.(v3 addition -- resolves Goofy Round 2 caveat)468-customizable-install.md
## v5 Changelogthrough## v14 Changelog, ~170 lines).Real-Defaults Drift Testheading: removed(v5 fix #3 -- Duck-2)Donald-1 (baseline format mismatch)->baseline format mismatchDuck-2 (real defaults drift unprotected)->real defaults drift unprotected> NOTE: squad-cli entries below are stale pending #468 implementation.(per plan recommendation R5/WI-3 notes)squad-cli content: deliberately preserved (deferred to #468 implementation)
Per Phase 2 D4, squad-cli feature content in 468 is left entirely untouched:
DEFAULT_TOOLSbash array (Linux)$DefaultToolsPS array (Windows)$ToolRegistryentry ('squad-cli' = { Install-SquadCli })The pre-WI-3 file had 21
squad-cli/Install-SquadClireferences total. Of these, 10 were in the changelog blocks (correctly deleted with the changelogs, as noted in the authoritative plan appendix: "v10/v11/v12 changelogs: squad-cli.sh/squad-cli.ps1 refs -- DELETED by WI-3"). The remaining 11 body references are all preserved. The NOTE banner adds one additional reference (meta-comment only). Total post-scrub: 12 matching lines.Verification
git grep -niE "mickey|goofy|jiminy|pluto|donald|grill" -- docs/plans/: ZERO matchesgit grep -niE "\bduck\b|\bchip\b|\bdale\b|\bdoc\b" -- docs/plans/: 4 hits, all ordinary English:451:90: inline doc comment-- doc = code comment (ordinary)README.md:11: Every plan doc-- doc = document (ordinary)README.md:35: doc kept for reference-- doc = document (ordinary)README.md:36: plan doc (note supersedes)-- doc = document (ordinary)