Repository navigation
fix: Windows EPERM fallback + --version shows installed version - #149
Merged
Merged
Conversation
Session: 20260227T010300Z-repo-cleanup Requested by: Brady Changes: - Logged repository cleanup work by Keaton and Kobayashi - Merged 2 decisions from inbox (fenster-init-command, keaton-repo-cleanup) - Deleted merged inbox decision files - Consolidation: Branch divergence analysis, cherry-pick plan, cleanup execution Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
- Add safeRename() helper that catches EPERM/EACCES on fs.renameSync and falls back to recursive copy + delete (fixes Windows file-lock issue during migrate-directory when VS Code holds handles on .ai-team/ files). - --version now shows both Package and Installed versions, reading the installed version from squad.agent.md HTML comment. - Update test assertion for new --version output format. Fixes #135, fixes #137 Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
This was referenced Feb 27, 2026
bradygaster
added a commit
that referenced
this pull request
Feb 27, 2026
Session: 20260227T012300Z-bug-fixes-135-137 Requested by: Brady Changes: - Logged orchestration: Fenster fixed bugs #135 (Windows EPERM) and #137 (--version) - Created session log: CLI bug fixes with safeRename() and dual version display - Decision merged: safeRename pattern and version display rationale Status: PR #149 open, 95/95 tests passing
bradygaster
added a commit
that referenced
this pull request
Feb 27, 2026
Squash-merged PR #149 from squad/135-137-cli-fixes. Fixes #135: safeRename() fallback — catches EPERM/EACCES on Windows, falls back to fs.cpSync + fs.rmSync for both directory migrations. Fixes #137: --version now shows Package (npx) and Installed (from squad.agent.md) versions so users see the difference. 95/95 tests passing. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
joniba
pushed a commit
to joniba/squad
that referenced
this pull request
Mar 26, 2026
…ils (github#148) (github#149) Changes: - notify-blocked.ps1: add -Issues parameter for multi-issue mode Each issue gets its own card row with clickable #N link, title, and reason - notify.ps1 Build-UrgentCard: render per-issue rows with FactSet for Agent/Severity (fixes same-line formatting bug), action buttons per issue - notify-squad-event.ps1: accept -Issues JSON string, parse and forward as structured array; backward compatible with single-issue -Why/-Link - routing.md Rule 16: document multi-issue invocation syntax Backward compatible: single-issue mode (legacy -Reason/-BlockerUrl) unchanged. Co-authored-by: Jonathan Ben Ami <elrond@squad.local> 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.
Fixes two CLI bugs:
Bug 135: Windows EPERM on migrate-directory - Added safeRename() helper that catches EPERM/EACCES and falls back to fs.cpSync + fs.rmSync. Applied to both directory renames.
Bug 137: --version shows package version, not installed version - Now displays both Package and Installed versions. Reads installed version from squad.agent.md HTML comment. Shows 'not installed' if file doesn't exist.
All 95 tests pass. One test assertion updated for new --version output format.
Fixes #135, fixes #137