Structural verification that .bg-shell/ is included in the
BASELINE_PATTERNS array in gitignore.ts.
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Structural verification that selectConfirm handler and onSetupAuth
callback exist in provider-manager.ts (#3579).
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Structural verification that MAX_UAT_ATTEMPTS constant exists and
incrementUatCount is called before dispatch (#3624).
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Structural verification that createRequire import and _require.resolve
usage exist in mcp-server.ts (#3603).
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Structural verification that classification === "note" filter and
markCaptureExecuted call exist in executeTriageResolutions (#3578).
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Structural verification that process.cwd() is wrapped in try/catch (#3598)
and nativeBranchExists validates prefs.main_branch (#3589).
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Long messages now word-wrap onto continuation lines aligned with the
message start instead of being truncated with ellipsis. Overlay box
sizes to content height instead of padding to fill the viewport.
migrateHierarchyToDb imported milestones with all-done roadmap slices
as "active" when SUMMARY.md was missing. This let plan-milestone
overwrite already-completed work.
Now checks parsed roadmap slices — if all are done, imports as
"complete" even without SUMMARY.md.
Closes#3390Closes#3379
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
The completing-milestone dispatch guard blocked completion when
operational verification was planned but the validation was
intentionally skipped by a budget profile preference. The guard
now detects skip-by-preference markers in the validation content
and allows completion to proceed.
Closes#3399Closes#3344
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
All slices got sequence=0 because the three code paths that insert
slices never set the sequence column. This made positional ordering
degenerate to alphabetical-by-ID, causing deadlocks when dependency-
free slices were ordered after blocked ones.
Now passes sequence at:
- plan-milestone: array index from agent-ordered params.slices
- reassess-roadmap: existingCount + index for newly added slices
- md-importer: parse order from roadmap content
Closes#3356
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1. complete-milestone: move gsd_requirement_update before
gsd_complete_milestone so agents don't exit before updating
requirement status (#3155)
2. complete-slice: use gsd_requirement_update instead of
gsd_save_decision for requirement status transitions (#3154)
3. uncaughtException handler: log instead of re-throwing to prevent
fatal double-fault that kills the session (#3163)
4. session_before_compact: only cancel during active auto-mode, not
when paused — allows compaction during interactive work (#3165)
Closes#3155Closes#3154Closes#3163Closes#3165
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Hallucinated tool calls could auto-create phantom milestones as
"active", hijacking the state machine. Observed: complete-slice issued
a stray gsd_task_complete for M000, which was auto-created as active
and promoted over the real M028.
Changing the default from "active" to "queued" means phantom milestones
from stray tool calls won't be promoted as active. Combined with the
ghost detection fix (#3645), they'll be cleaned up automatically.
Legitimate callers (plan-milestone, state reconciliation, md-importer)
already pass status explicitly.
Closes#3380
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
logWarning fired on every preferences load when YAML was malformed,
flooding the TUI. Now warns at most once per session with a suppression
notice.
Closes#3376
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Two fixes:
1. gsd_complete_task now normalizes keyFiles/keyDecisions from strings
(newline-delimited bullet lists) into arrays at the tool boundary,
preventing type mismatch rejections on first call (#3361)
2. Legacy roadmap table parser now detects the dependency column from
the header and only parses deps from that column or cells with
explicit depends/deps keywords — prevents false deps from slice
titles that mention other S-IDs (#3383, #3336)
Closes#3361Closes#3383Closes#3336
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Two fixes:
1. handleStatus now calls ensureDbOpen() before deriveState so cold
sessions use DB-backed state instead of filesystem fallback (#3385)
2. /gsd quick now checks getIsolationMode() and skips branch creation
when isolation is "none", consistent with milestone behavior (#3337)
Closes#3385Closes#3337
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
The bg-shell process manifest directory is ephemeral runtime state that
should never be committed, but ensureGitignore() was not including it.
Closes#3389Closes#3388
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
When "None of the above" was selected and the notes field was already
visible but empty, pressing Enter re-opened the notes field endlessly
because the guard only checked !notes (empty string is falsy).
Add !notesVisible guard so auto-open only triggers on the initial
selection, not when the user is confirming from an already-open notes
field.
Closes#3450Closes#3449
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
The provider manager let users navigate with arrow keys but pressing
Enter did nothing. Users had no way to set up authentication from within
the /provider command.
Adds selectConfirm (Enter) handler that routes to showLoginDialog for
the selected provider, with a hint in the status bar.
Closes#3579Closes#3567
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
When verification commands fail before writing an ASSESSMENT verdict,
checkNeedsRunUat keeps returning the same slice, causing infinite
re-dispatch until the provider throttles the session.
Adds a per-slice disk-persisted counter (survives crash/restart) that
caps run-uat at 3 attempts. After the cap, stops auto-mode with an
actionable message instead of looping.
Closes#3624
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Node ESM can't resolve wildcard export patterns without .js extension.
The MCP SDK's server/stdio and types subpaths use wildcard exports that
fail at runtime with ERR_MODULE_NOT_FOUND.
Use createRequire (CJS resolver) to resolve physical file paths before
passing to dynamic import(), since CJS auto-appends .js.
Closes#3603
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Note captures (informational-only, no action needed) were never marked
as Executed because loadActionableCaptures excluded them and there was
no dedicated handler. They stayed in "resolved but not executed" limbo.
Now marks note captures as executed immediately since the triage
classification is their complete lifecycle.
Closes#3578
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
mergeMilestoneToMain used prefs.main_branch without checking if the
branch exists in the repo. A stale preference (e.g. "master" when repo
uses "main") caused merge failure.
Now validates with nativeBranchExists before using the preference,
falling through to nativeDetectMainBranch if invalid.
Closes#3589
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
process.cwd() throws ENOENT when the worktree directory was deleted
during a failed merge. Every /gsd command calls projectRoot(), causing
the entire extension to crash.
Now catches the ENOENT and falls back to HOME directory.
Closes#3598
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
syncWorktreeStateBack copied all milestone directories including the one
being merged. This caused dirty-tree conflicts when mergeMilestoneToMain
ran because the files already existed from the copy-back.
Now skips the current milestoneId in the sync loop since its files are
already in the milestone branch being merged.
Closes#3641
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
The slice discuss template hardcoded ask_user_questions references with
no fallback for providers that don't have the tool. The milestone
discuss template was already fixed with a conditional.
Now buildDiscussSlicePrompt accepts and passes structuredQuestionsAvailable,
and the prompt template uses the same conditional pattern as the milestone
discuss template.
Closes#3604
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
dispatchWorkflow scoped tools down for discuss-* flows but never
restored them. The narrowed set persisted into subsequent dispatches,
making GSD execution tools permanently unavailable for the session.
Now saves the full tool list before scoping and restores it immediately
after sendMessage queues the turn. The LLM turn has already captured
the scoped set so it's unaffected.
Closes#3628
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
