@opengsd/gsd-pi 1.4.0-dev.c18009cd → 1.5.0-dev.4fb2d460
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/README.md +10 -10
- package/dist/resources/.managed-resources-content-hash +1 -1
- package/dist/resources/extensions/ask-user-questions.js +1 -1
- package/dist/resources/extensions/gsd/auto/closeout.js +97 -5
- package/dist/resources/extensions/gsd/auto/orchestrator.js +4 -3
- package/dist/resources/extensions/gsd/auto/pre-dispatch.js +6 -3
- package/dist/resources/extensions/gsd/auto/unit-phase.js +5 -9
- package/dist/resources/extensions/gsd/auto-artifact-paths.js +12 -3
- package/dist/resources/extensions/gsd/auto-direct-dispatch.js +11 -7
- package/dist/resources/extensions/gsd/auto-dispatch.js +16 -2
- package/dist/resources/extensions/gsd/auto-recovery.js +50 -6
- package/dist/resources/extensions/gsd/auto-start.js +4 -3
- package/dist/resources/extensions/gsd/auto-worktree-branch-lifecycle.js +189 -0
- package/dist/resources/extensions/gsd/auto-worktree-cleanup.js +133 -0
- package/dist/resources/extensions/gsd/auto-worktree-conflict-auto-resolve.js +35 -0
- package/dist/resources/extensions/gsd/auto-worktree-creation.js +130 -0
- package/dist/resources/extensions/gsd/auto-worktree-entry.js +118 -0
- package/dist/resources/extensions/gsd/auto-worktree-merge-already-merged.js +98 -0
- package/dist/resources/extensions/gsd/auto-worktree-merge-branch-head.js +52 -0
- package/dist/resources/extensions/gsd/auto-worktree-merge-cleanup.js +87 -0
- package/dist/resources/extensions/gsd/auto-worktree-merge-code-changes.js +46 -0
- package/dist/resources/extensions/gsd/auto-worktree-merge-db-ready.js +68 -0
- package/dist/resources/extensions/gsd/auto-worktree-merge-integration-branch.js +51 -0
- package/dist/resources/extensions/gsd/auto-worktree-merge-message.js +106 -0
- package/dist/resources/extensions/gsd/auto-worktree-merge-pre-teardown.js +64 -0
- package/dist/resources/extensions/gsd/auto-worktree-merge-stash.js +168 -0
- package/dist/resources/extensions/gsd/auto-worktree-merge.js +302 -0
- package/dist/resources/extensions/gsd/auto-worktree-milestone-shelter.js +119 -0
- package/dist/resources/extensions/gsd/auto-worktree-path-resolution.js +5 -0
- package/dist/resources/extensions/gsd/auto-worktree-path-utils.js +5 -0
- package/dist/resources/extensions/gsd/auto-worktree-resource-version.js +41 -0
- package/dist/resources/extensions/gsd/auto-worktree-runtime-cleanup.js +99 -0
- package/dist/resources/extensions/gsd/auto-worktree-session-registry.js +57 -0
- package/dist/resources/extensions/gsd/auto-worktree-sync.js +249 -0
- package/dist/resources/extensions/gsd/auto-worktree-teardown.js +120 -0
- package/dist/resources/extensions/gsd/auto-worktree.js +20 -2073
- package/dist/resources/extensions/gsd/auto.js +9 -3
- package/dist/resources/extensions/gsd/bootstrap/dynamic-tools.js +1 -1
- package/dist/resources/extensions/gsd/bootstrap/register-hooks.js +20 -2
- package/dist/resources/extensions/gsd/bootstrap/system-context.js +1 -1
- package/dist/resources/extensions/gsd/bootstrap/tool-call-loop-guard.js +185 -27
- package/dist/resources/extensions/gsd/codebase-generator.js +7 -1
- package/dist/resources/extensions/gsd/commands-handlers.js +1 -1
- package/dist/resources/extensions/gsd/commands-workflow-templates.js +3 -3
- package/dist/resources/extensions/gsd/db/queries.js +9 -9
- package/dist/resources/extensions/gsd/db/writers/cascades.js +20 -2
- package/dist/resources/extensions/gsd/doctor-engine-checks.js +93 -11
- package/dist/resources/extensions/gsd/flat-phase-migration.js +32 -3
- package/dist/resources/extensions/gsd/forensics.js +1 -1
- package/dist/resources/extensions/gsd/git-conflict-resolve.js +2 -2
- package/dist/resources/extensions/gsd/git-service.js +45 -0
- package/dist/resources/extensions/gsd/gsd-db.js +23 -11
- package/dist/resources/extensions/gsd/guided-flow.js +22 -17
- package/dist/resources/extensions/gsd/markdown-renderer.js +15 -3
- package/dist/resources/extensions/gsd/md-importer.js +89 -23
- package/dist/resources/extensions/gsd/migration-auto-check.js +30 -3
- package/dist/resources/extensions/gsd/milestone-merge-transaction.js +11 -0
- package/dist/resources/extensions/gsd/milestone-settlement.js +1 -1
- package/dist/resources/extensions/gsd/parallel-merge.js +1 -1
- package/dist/resources/extensions/gsd/parallel-orchestrator.js +3 -1
- package/dist/resources/extensions/gsd/paths.js +17 -3
- package/dist/resources/extensions/gsd/preferences-skills.js +10 -16
- package/dist/resources/extensions/gsd/preferences-types.js +1 -0
- package/dist/resources/extensions/gsd/preferences-validation.js +82 -0
- package/dist/resources/extensions/gsd/preferences.js +20 -0
- package/dist/resources/extensions/gsd/prompts/system.md +6 -6
- package/dist/resources/extensions/gsd/quick.js +12 -3
- package/dist/resources/extensions/gsd/skill-discovery.js +36 -23
- package/dist/resources/extensions/gsd/slice-parallel-orchestrator.js +2 -1
- package/dist/resources/extensions/gsd/state/derive/from-db.js +88 -153
- package/dist/resources/extensions/gsd/tool-contract.js +14 -0
- package/dist/resources/extensions/gsd/tools/reopen-slice.js +5 -0
- package/dist/resources/extensions/gsd/tools/reopen-task.js +13 -5
- package/dist/resources/extensions/gsd/unit-registry.js +12 -2
- package/dist/resources/extensions/gsd/unmerged-milestone-guard.js +1 -1
- package/dist/resources/extensions/gsd/validation-block-guard.js +2 -1
- package/dist/resources/extensions/gsd/workspace-git-preflight.js +2 -2
- package/dist/resources/extensions/gsd/worktree-command.js +1 -1
- package/dist/resources/extensions/gsd/worktree-lifecycle.js +14 -9
- package/dist/resources/extensions/gsd/worktree-reentry.js +2 -1
- package/dist/resources/extensions/shared/interview-ui.js +62 -9
- package/dist/tsconfig.extensions.tsbuildinfo +1 -1
- package/dist/web/standalone/.next/BUILD_ID +1 -1
- package/dist/web/standalone/.next/app-path-routes-manifest.json +8 -8
- package/dist/web/standalone/.next/build-manifest.json +2 -2
- package/dist/web/standalone/.next/prerender-manifest.json +3 -3
- package/dist/web/standalone/.next/server/app/_global-error.html +1 -1
- package/dist/web/standalone/.next/server/app/_global-error.rsc +1 -1
- package/dist/web/standalone/.next/server/app/_global-error.segments/_full.segment.rsc +1 -1
- package/dist/web/standalone/.next/server/app/_global-error.segments/_global-error/__PAGE__.segment.rsc +1 -1
- package/dist/web/standalone/.next/server/app/_global-error.segments/_global-error.segment.rsc +1 -1
- package/dist/web/standalone/.next/server/app/_global-error.segments/_head.segment.rsc +1 -1
- package/dist/web/standalone/.next/server/app/_global-error.segments/_index.segment.rsc +1 -1
- package/dist/web/standalone/.next/server/app/_global-error.segments/_tree.segment.rsc +1 -1
- package/dist/web/standalone/.next/server/app/_not-found.html +1 -1
- package/dist/web/standalone/.next/server/app/_not-found.rsc +1 -1
- package/dist/web/standalone/.next/server/app/_not-found.segments/_full.segment.rsc +1 -1
- package/dist/web/standalone/.next/server/app/_not-found.segments/_head.segment.rsc +1 -1
- package/dist/web/standalone/.next/server/app/_not-found.segments/_index.segment.rsc +1 -1
- package/dist/web/standalone/.next/server/app/_not-found.segments/_not-found/__PAGE__.segment.rsc +1 -1
- package/dist/web/standalone/.next/server/app/_not-found.segments/_not-found.segment.rsc +1 -1
- package/dist/web/standalone/.next/server/app/_not-found.segments/_tree.segment.rsc +1 -1
- package/dist/web/standalone/.next/server/app/index.html +1 -1
- package/dist/web/standalone/.next/server/app/index.rsc +1 -1
- package/dist/web/standalone/.next/server/app/index.segments/__PAGE__.segment.rsc +1 -1
- package/dist/web/standalone/.next/server/app/index.segments/_full.segment.rsc +1 -1
- package/dist/web/standalone/.next/server/app/index.segments/_head.segment.rsc +1 -1
- package/dist/web/standalone/.next/server/app/index.segments/_index.segment.rsc +1 -1
- package/dist/web/standalone/.next/server/app/index.segments/_tree.segment.rsc +1 -1
- package/dist/web/standalone/.next/server/app-paths-manifest.json +8 -8
- package/dist/web/standalone/.next/server/middleware-build-manifest.js +1 -1
- package/dist/web/standalone/.next/server/pages/404.html +1 -1
- package/dist/web/standalone/.next/server/pages/500.html +1 -1
- package/dist/web/standalone/.next/server/server-reference-manifest.json +1 -1
- package/dist/web/standalone/node_modules/@gsd/native/package.json +1 -1
- package/dist/web/standalone/node_modules/node-pty/build/Makefile +1 -1
- package/integrations/hermes/open_gsd_hermes/gsd_client.py +24 -35
- package/integrations/hermes/tests/test_gsd_client_timeout.py +4 -4
- package/package.json +6 -6
- package/packages/cloud-mcp-gateway/package.json +2 -2
- package/packages/contracts/package.json +1 -1
- package/packages/daemon/package.json +4 -4
- package/packages/gsd-agent-core/dist/export-html/template.js +53 -4
- package/packages/gsd-agent-core/package.json +5 -5
- package/packages/gsd-agent-modes/dist/modes/interactive/controllers/chat-controller.d.ts.map +1 -1
- package/packages/gsd-agent-modes/dist/modes/interactive/controllers/chat-controller.js +3 -2
- package/packages/gsd-agent-modes/dist/modes/interactive/controllers/chat-controller.js.map +1 -1
- package/packages/gsd-agent-modes/package.json +7 -7
- package/packages/mcp-server/package.json +4 -4
- package/packages/native/package.json +1 -1
- package/packages/pi-agent-core/dist/agent-loop.d.ts.map +1 -1
- package/packages/pi-agent-core/dist/agent-loop.js +14 -9
- package/packages/pi-agent-core/dist/agent-loop.js.map +1 -1
- package/packages/pi-agent-core/package.json +1 -1
- package/packages/pi-ai/dist/index.d.ts +1 -0
- package/packages/pi-ai/dist/index.d.ts.map +1 -1
- package/packages/pi-ai/dist/index.js +1 -0
- package/packages/pi-ai/dist/index.js.map +1 -1
- package/packages/pi-ai/dist/providers/transform-messages.d.ts.map +1 -1
- package/packages/pi-ai/dist/providers/transform-messages.js +7 -3
- package/packages/pi-ai/dist/providers/transform-messages.js.map +1 -1
- package/packages/pi-ai/dist/tool-result-content.d.ts +3 -0
- package/packages/pi-ai/dist/tool-result-content.d.ts.map +1 -0
- package/packages/pi-ai/dist/tool-result-content.js +62 -0
- package/packages/pi-ai/dist/tool-result-content.js.map +1 -0
- package/packages/pi-ai/dist/types.d.ts +8 -20
- package/packages/pi-ai/dist/types.d.ts.map +1 -1
- package/packages/pi-ai/dist/types.js.map +1 -1
- package/packages/pi-ai/package.json +1 -1
- package/packages/pi-coding-agent/dist/core/package-manager.d.ts +9 -0
- package/packages/pi-coding-agent/dist/core/package-manager.d.ts.map +1 -1
- package/packages/pi-coding-agent/dist/core/package-manager.js +74 -53
- package/packages/pi-coding-agent/dist/core/package-manager.js.map +1 -1
- package/packages/pi-coding-agent/dist/core/resource-loader.d.ts +6 -0
- package/packages/pi-coding-agent/dist/core/resource-loader.d.ts.map +1 -1
- package/packages/pi-coding-agent/dist/core/resource-loader.js +30 -2
- package/packages/pi-coding-agent/dist/core/resource-loader.js.map +1 -1
- package/packages/pi-coding-agent/dist/core/skill-directories.d.ts +66 -0
- package/packages/pi-coding-agent/dist/core/skill-directories.d.ts.map +1 -0
- package/packages/pi-coding-agent/dist/core/skill-directories.js +138 -0
- package/packages/pi-coding-agent/dist/core/skill-directories.js.map +1 -0
- package/packages/pi-coding-agent/dist/core/skills.d.ts +4 -1
- package/packages/pi-coding-agent/dist/core/skills.d.ts.map +1 -1
- package/packages/pi-coding-agent/dist/core/skills.js +4 -1
- package/packages/pi-coding-agent/dist/core/skills.js.map +1 -1
- package/packages/pi-coding-agent/dist/index.d.ts +1 -0
- package/packages/pi-coding-agent/dist/index.d.ts.map +1 -1
- package/packages/pi-coding-agent/dist/index.js +1 -0
- package/packages/pi-coding-agent/dist/index.js.map +1 -1
- package/packages/pi-coding-agent/package.json +7 -7
- package/packages/pi-tui/package.json +2 -2
- package/packages/rpc-client/package.json +2 -2
- package/pkg/dist/core/export-html/template.js +53 -4
- package/pkg/package.json +1 -1
- package/src/resources/extensions/ask-user-questions.ts +1 -1
- package/src/resources/extensions/gsd/auto/closeout.ts +114 -7
- package/src/resources/extensions/gsd/auto/orchestrator.ts +4 -3
- package/src/resources/extensions/gsd/auto/pre-dispatch.ts +6 -3
- package/src/resources/extensions/gsd/auto/unit-phase.ts +5 -9
- package/src/resources/extensions/gsd/auto-artifact-paths.ts +12 -3
- package/src/resources/extensions/gsd/auto-direct-dispatch.ts +13 -7
- package/src/resources/extensions/gsd/auto-dispatch.ts +18 -2
- package/src/resources/extensions/gsd/auto-recovery.ts +50 -6
- package/src/resources/extensions/gsd/auto-start.ts +5 -3
- package/src/resources/extensions/gsd/auto-worktree-branch-lifecycle.ts +237 -0
- package/src/resources/extensions/gsd/auto-worktree-cleanup.ts +166 -0
- package/src/resources/extensions/gsd/auto-worktree-conflict-auto-resolve.ts +37 -0
- package/src/resources/extensions/gsd/auto-worktree-creation.ts +171 -0
- package/src/resources/extensions/gsd/auto-worktree-entry.ts +154 -0
- package/src/resources/extensions/gsd/auto-worktree-merge-already-merged.ts +161 -0
- package/src/resources/extensions/gsd/auto-worktree-merge-branch-head.ts +83 -0
- package/src/resources/extensions/gsd/auto-worktree-merge-cleanup.ts +132 -0
- package/src/resources/extensions/gsd/auto-worktree-merge-code-changes.ts +71 -0
- package/src/resources/extensions/gsd/auto-worktree-merge-db-ready.ts +112 -0
- package/src/resources/extensions/gsd/auto-worktree-merge-integration-branch.ts +99 -0
- package/src/resources/extensions/gsd/auto-worktree-merge-message.ts +166 -0
- package/src/resources/extensions/gsd/auto-worktree-merge-pre-teardown.ts +93 -0
- package/src/resources/extensions/gsd/auto-worktree-merge-stash.ts +213 -0
- package/src/resources/extensions/gsd/auto-worktree-merge.ts +391 -0
- package/src/resources/extensions/gsd/auto-worktree-milestone-shelter.ts +159 -0
- package/src/resources/extensions/gsd/auto-worktree-path-resolution.ts +6 -0
- package/src/resources/extensions/gsd/auto-worktree-path-utils.ts +10 -0
- package/src/resources/extensions/gsd/auto-worktree-resource-version.ts +44 -0
- package/src/resources/extensions/gsd/auto-worktree-runtime-cleanup.ts +119 -0
- package/src/resources/extensions/gsd/auto-worktree-session-registry.ts +71 -0
- package/src/resources/extensions/gsd/auto-worktree-sync.ts +370 -0
- package/src/resources/extensions/gsd/auto-worktree-teardown.ts +161 -0
- package/src/resources/extensions/gsd/auto-worktree.ts +69 -2429
- package/src/resources/extensions/gsd/auto.ts +11 -17
- package/src/resources/extensions/gsd/bootstrap/dynamic-tools.ts +1 -1
- package/src/resources/extensions/gsd/bootstrap/register-hooks.ts +20 -2
- package/src/resources/extensions/gsd/bootstrap/system-context.ts +1 -1
- package/src/resources/extensions/gsd/bootstrap/tool-call-loop-guard.ts +218 -27
- package/src/resources/extensions/gsd/codebase-generator.ts +7 -1
- package/src/resources/extensions/gsd/commands-handlers.ts +1 -1
- package/src/resources/extensions/gsd/commands-workflow-templates.ts +3 -3
- package/src/resources/extensions/gsd/db/queries.ts +9 -9
- package/src/resources/extensions/gsd/db/writers/cascades.ts +23 -1
- package/src/resources/extensions/gsd/doctor-engine-checks.ts +109 -10
- package/src/resources/extensions/gsd/flat-phase-migration.ts +34 -3
- package/src/resources/extensions/gsd/forensics.ts +1 -1
- package/src/resources/extensions/gsd/git-conflict-resolve.ts +2 -2
- package/src/resources/extensions/gsd/git-service.ts +53 -0
- package/src/resources/extensions/gsd/gsd-db.ts +33 -11
- package/src/resources/extensions/gsd/guided-flow.ts +26 -18
- package/src/resources/extensions/gsd/markdown-renderer.ts +15 -3
- package/src/resources/extensions/gsd/md-importer.ts +93 -22
- package/src/resources/extensions/gsd/migration-auto-check.ts +32 -3
- package/src/resources/extensions/gsd/milestone-merge-transaction.ts +15 -0
- package/src/resources/extensions/gsd/milestone-settlement.ts +1 -1
- package/src/resources/extensions/gsd/parallel-merge.ts +1 -1
- package/src/resources/extensions/gsd/parallel-orchestrator.ts +3 -1
- package/src/resources/extensions/gsd/paths.ts +21 -3
- package/src/resources/extensions/gsd/preferences-skills.ts +10 -16
- package/src/resources/extensions/gsd/preferences-types.ts +30 -0
- package/src/resources/extensions/gsd/preferences-validation.ts +69 -0
- package/src/resources/extensions/gsd/preferences.ts +20 -0
- package/src/resources/extensions/gsd/prompts/system.md +6 -6
- package/src/resources/extensions/gsd/quick.ts +11 -3
- package/src/resources/extensions/gsd/skill-discovery.ts +47 -23
- package/src/resources/extensions/gsd/slice-parallel-orchestrator.ts +3 -1
- package/src/resources/extensions/gsd/state/derive/from-db.ts +201 -157
- package/src/resources/extensions/gsd/tests/auto-artifact-paths.test.ts +169 -1
- package/src/resources/extensions/gsd/tests/auto-loop-no-copy-artifacts.test.ts +2 -1
- package/src/resources/extensions/gsd/tests/auto-recovery.test.ts +57 -0
- package/src/resources/extensions/gsd/tests/auto-start-clean-runtime-db-gated.test.ts +1 -1
- package/src/resources/extensions/gsd/tests/auto-warning-noise-regression.test.ts +2 -1
- package/src/resources/extensions/gsd/tests/auto-worktree-auto-resolve.test.ts +1 -1
- package/src/resources/extensions/gsd/tests/auto-worktree-merge-already-merged.test.ts +177 -0
- package/src/resources/extensions/gsd/tests/auto-worktree-merge-branch-head.test.ts +118 -0
- package/src/resources/extensions/gsd/tests/auto-worktree-merge-cleanup.test.ts +115 -0
- package/src/resources/extensions/gsd/tests/auto-worktree-merge-code-changes.test.ts +76 -0
- package/src/resources/extensions/gsd/tests/auto-worktree-merge-db-ready.test.ts +140 -0
- package/src/resources/extensions/gsd/tests/auto-worktree-merge-integration-branch.test.ts +123 -0
- package/src/resources/extensions/gsd/tests/auto-worktree-merge-message.test.ts +114 -0
- package/src/resources/extensions/gsd/tests/auto-worktree-merge-pre-teardown.test.ts +84 -0
- package/src/resources/extensions/gsd/tests/auto-worktree-merge-stash.test.ts +70 -0
- package/src/resources/extensions/gsd/tests/auto-worktree-milestone-shelter.test.ts +52 -0
- package/src/resources/extensions/gsd/tests/auto-worktree-registry.test.ts +6 -6
- package/src/resources/extensions/gsd/tests/auto-worktree-resource-version.test.ts +59 -0
- package/src/resources/extensions/gsd/tests/auto-worktree-untracked-content.test.ts +3 -1
- package/src/resources/extensions/gsd/tests/codebase-generator.test.ts +22 -0
- package/src/resources/extensions/gsd/tests/completed-units-metrics-sync.test.ts +1 -1
- package/src/resources/extensions/gsd/tests/derive-state-db.test.ts +4 -0
- package/src/resources/extensions/gsd/tests/doctor-scope-db-unavailable.test.ts +174 -0
- package/src/resources/extensions/gsd/tests/fast-forward-reused-milestone-branch.test.ts +1 -1
- package/src/resources/extensions/gsd/tests/flat-phase-migration.test.ts +71 -1
- package/src/resources/extensions/gsd/tests/flat-phase-renderer.test.ts +25 -1
- package/src/resources/extensions/gsd/tests/gsd-db.test.ts +40 -0
- package/src/resources/extensions/gsd/tests/import-completion-lost-update.test.ts +196 -0
- package/src/resources/extensions/gsd/tests/integration/all-milestones-complete-merge.test.ts +4 -6
- package/src/resources/extensions/gsd/tests/integration/auto-stash-merge.test.ts +2 -1
- package/src/resources/extensions/gsd/tests/integration/auto-worktree-milestone-merge.test.ts +3 -3
- package/src/resources/extensions/gsd/tests/integration/auto-worktree.test.ts +7 -8
- package/src/resources/extensions/gsd/tests/integration/feature-branch-lifecycle-integration.test.ts +2 -5
- package/src/resources/extensions/gsd/tests/integration/git-service.test.ts +76 -0
- package/src/resources/extensions/gsd/tests/integration/merge-cwd-restore.test.ts +1 -1
- package/src/resources/extensions/gsd/tests/integration/merge-preserve-worktree-on-preteardown-error.test.ts +1 -1
- package/src/resources/extensions/gsd/tests/integration/merge-strategy-regular.test.ts +2 -1
- package/src/resources/extensions/gsd/tests/integration/milestone-transition-worktree.test.ts +5 -7
- package/src/resources/extensions/gsd/tests/integration/workspace-collapse-integration.test.ts +2 -4
- package/src/resources/extensions/gsd/tests/journal-integration.test.ts +16 -15
- package/src/resources/extensions/gsd/tests/merge-closeout-consistency-gate.test.ts +1 -1
- package/src/resources/extensions/gsd/tests/merge-db-cycle.test.ts +1 -1
- package/src/resources/extensions/gsd/tests/merge-self-branch-guard.test.ts +1 -1
- package/src/resources/extensions/gsd/tests/migration-auto-check.test.ts +91 -0
- package/src/resources/extensions/gsd/tests/milestone-merge-stash-restore.test.ts +439 -0
- package/src/resources/extensions/gsd/tests/milestone-merge-transaction.test.ts +24 -0
- package/src/resources/extensions/gsd/tests/none-mode-gates.test.ts +1 -1
- package/src/resources/extensions/gsd/tests/originalbase-path-comparison.test.ts +3 -3
- package/src/resources/extensions/gsd/tests/parallel-worker-lock-contention.test.ts +1 -1
- package/src/resources/extensions/gsd/tests/preferences-worktree-sync.test.ts +1 -4
- package/src/resources/extensions/gsd/tests/preferences.test.ts +19 -0
- package/src/resources/extensions/gsd/tests/prefs-wizard-coverage.test.ts +5 -0
- package/src/resources/extensions/gsd/tests/reopen-slice.test.ts +52 -0
- package/src/resources/extensions/gsd/tests/runtime-invariant-modules.test.ts +20 -0
- package/src/resources/extensions/gsd/tests/skill-discovery.test.ts +46 -0
- package/src/resources/extensions/gsd/tests/stale-worktree-cwd.test.ts +3 -5
- package/src/resources/extensions/gsd/tests/stash-pop-gsd-conflict.test.ts +2 -1
- package/src/resources/extensions/gsd/tests/stash-queued-context-files.test.ts +2 -1
- package/src/resources/extensions/gsd/tests/steer-worktree-path.test.ts +1 -1
- package/src/resources/extensions/gsd/tests/sync-worktree-skip-current.test.ts +1 -1
- package/src/resources/extensions/gsd/tests/system-flat-phase-layout.test.ts +33 -0
- package/src/resources/extensions/gsd/tests/teardown-chdir-failure-clears-registry.test.ts +4 -4
- package/src/resources/extensions/gsd/tests/teardown-cleanup-parity.test.ts +2 -1
- package/src/resources/extensions/gsd/tests/teardown-failure-clears-registry.test.ts +4 -4
- package/src/resources/extensions/gsd/tests/tool-call-loop-guard.test.ts +196 -0
- package/src/resources/extensions/gsd/tests/uat-stuck-loop-orphaned-worktree.test.ts +1 -1
- package/src/resources/extensions/gsd/tests/unit-registry.test.ts +26 -1
- package/src/resources/extensions/gsd/tests/worktree-db-integration.test.ts +1 -1
- package/src/resources/extensions/gsd/tests/worktree-db-respawn-truncation.test.ts +1 -1
- package/src/resources/extensions/gsd/tests/worktree-db-same-file.test.ts +1 -3
- package/src/resources/extensions/gsd/tests/worktree-expected-warnings.test.ts +1 -1
- package/src/resources/extensions/gsd/tests/worktree-git-pathspec.test.ts +1 -1
- package/src/resources/extensions/gsd/tests/worktree-main-branch.test.ts +1 -1
- package/src/resources/extensions/gsd/tests/worktree-preferences-sync.test.ts +1 -1
- package/src/resources/extensions/gsd/tests/worktree-reentry.test.ts +2 -1
- package/src/resources/extensions/gsd/tests/worktree-sync-milestones.test.ts +5 -2
- package/src/resources/extensions/gsd/tests/worktree-sync-overwrite-loop.test.ts +1 -1
- package/src/resources/extensions/gsd/tests/worktree-sync-tasks.test.ts +1 -1
- package/src/resources/extensions/gsd/tool-contract.ts +29 -0
- package/src/resources/extensions/gsd/tools/reopen-slice.ts +5 -0
- package/src/resources/extensions/gsd/tools/reopen-task.ts +13 -5
- package/src/resources/extensions/gsd/unit-registry.ts +16 -2
- package/src/resources/extensions/gsd/unmerged-milestone-guard.ts +1 -1
- package/src/resources/extensions/gsd/validation-block-guard.ts +2 -1
- package/src/resources/extensions/gsd/workspace-git-preflight.ts +2 -2
- package/src/resources/extensions/gsd/worktree-command.ts +1 -1
- package/src/resources/extensions/gsd/worktree-lifecycle.ts +19 -15
- package/src/resources/extensions/gsd/worktree-reentry.ts +2 -1
- package/src/resources/extensions/shared/interview-ui.ts +58 -11
- package/src/resources/extensions/shared/tests/interview-ui-border.test.ts +32 -0
- /package/dist/web/standalone/.next/static/{L2UMqXELiDH3oz-7yguXp → gXoWec4UNqcR0tkuHwYCr}/_buildManifest.js +0 -0
- /package/dist/web/standalone/.next/static/{L2UMqXELiDH3oz-7yguXp → gXoWec4UNqcR0tkuHwYCr}/_ssgManifest.js +0 -0
|
@@ -0,0 +1,112 @@
|
|
|
1
|
+
// gsd-pi — Milestone merge DB readiness guard.
|
|
2
|
+
//
|
|
3
|
+
// Owns the invariant that, before leaving worktree context for a milestone
|
|
4
|
+
// merge, the project DB is the active DB, worktree DB state is reconciled, and
|
|
5
|
+
// canonical closeout state proves the milestone is safe to merge.
|
|
6
|
+
|
|
7
|
+
import { join } from "node:path";
|
|
8
|
+
|
|
9
|
+
import { CLOSEOUT_CONSISTENCY_BLOCKED_REASON } from "./closeout-consistency-gate.js";
|
|
10
|
+
import {
|
|
11
|
+
closeWorkflowDatabase,
|
|
12
|
+
getWorkflowDatabasePath,
|
|
13
|
+
openWorkflowDatabasePath,
|
|
14
|
+
} from "./db-workspace.js";
|
|
15
|
+
import { GSDError, GSD_GIT_ERROR } from "./errors.js";
|
|
16
|
+
import {
|
|
17
|
+
isDbAvailable,
|
|
18
|
+
reconcileWorktreeDb,
|
|
19
|
+
} from "./gsd-db.js";
|
|
20
|
+
import {
|
|
21
|
+
formatCloseoutProofBlock,
|
|
22
|
+
proveMilestoneCloseout,
|
|
23
|
+
} from "./milestone-closeout-proof.js";
|
|
24
|
+
import { resolveGsdPathContract } from "./paths.js";
|
|
25
|
+
import { _shouldReconcileWorktreeDb } from "./auto-worktree-cleanup.js";
|
|
26
|
+
import { logError } from "./workflow-logger.js";
|
|
27
|
+
|
|
28
|
+
export interface MilestoneDbReadyRequest {
|
|
29
|
+
milestoneId: string;
|
|
30
|
+
projectRoot: string;
|
|
31
|
+
worktreeCwd: string;
|
|
32
|
+
}
|
|
33
|
+
|
|
34
|
+
interface MergeDbReadyDeps {
|
|
35
|
+
closeWorkflowDatabase: typeof closeWorkflowDatabase;
|
|
36
|
+
formatCloseoutProofBlock: typeof formatCloseoutProofBlock;
|
|
37
|
+
getWorkflowDatabasePath: typeof getWorkflowDatabasePath;
|
|
38
|
+
isDbAvailable: typeof isDbAvailable;
|
|
39
|
+
logError: typeof logError;
|
|
40
|
+
openWorkflowDatabasePath: typeof openWorkflowDatabasePath;
|
|
41
|
+
proveMilestoneCloseout: typeof proveMilestoneCloseout;
|
|
42
|
+
reconcileWorktreeDb: typeof reconcileWorktreeDb;
|
|
43
|
+
resolveGsdPathContract: typeof resolveGsdPathContract;
|
|
44
|
+
shouldReconcileWorktreeDb: typeof _shouldReconcileWorktreeDb;
|
|
45
|
+
}
|
|
46
|
+
|
|
47
|
+
const defaultDeps: MergeDbReadyDeps = {
|
|
48
|
+
closeWorkflowDatabase,
|
|
49
|
+
formatCloseoutProofBlock,
|
|
50
|
+
getWorkflowDatabasePath,
|
|
51
|
+
isDbAvailable,
|
|
52
|
+
logError,
|
|
53
|
+
openWorkflowDatabasePath,
|
|
54
|
+
proveMilestoneCloseout,
|
|
55
|
+
reconcileWorktreeDb,
|
|
56
|
+
resolveGsdPathContract,
|
|
57
|
+
shouldReconcileWorktreeDb: _shouldReconcileWorktreeDb,
|
|
58
|
+
};
|
|
59
|
+
|
|
60
|
+
let deps: MergeDbReadyDeps = defaultDeps;
|
|
61
|
+
|
|
62
|
+
export function _setMergeDbReadyDepsForTests(
|
|
63
|
+
overrides: Partial<MergeDbReadyDeps>,
|
|
64
|
+
): void {
|
|
65
|
+
deps = { ...defaultDeps, ...overrides };
|
|
66
|
+
}
|
|
67
|
+
|
|
68
|
+
export function _resetMergeDbReadyDepsForTests(): void {
|
|
69
|
+
deps = defaultDeps;
|
|
70
|
+
}
|
|
71
|
+
|
|
72
|
+
function reconcileWorktreeDatabase(request: MilestoneDbReadyRequest): void {
|
|
73
|
+
const { milestoneId, projectRoot, worktreeCwd } = request;
|
|
74
|
+
const contract = deps.resolveGsdPathContract(worktreeCwd, projectRoot);
|
|
75
|
+
const worktreeDbPath = join(contract.worktreeGsd ?? join(worktreeCwd, ".gsd"), "gsd.db");
|
|
76
|
+
const mainDbPath = contract.projectDb;
|
|
77
|
+
|
|
78
|
+
try {
|
|
79
|
+
const activeDbPath = deps.getWorkflowDatabasePath();
|
|
80
|
+
if (activeDbPath && deps.shouldReconcileWorktreeDb(activeDbPath, mainDbPath)) {
|
|
81
|
+
deps.closeWorkflowDatabase();
|
|
82
|
+
if (!deps.openWorkflowDatabasePath(mainDbPath)) {
|
|
83
|
+
throw new Error(`cannot open project DB at ${mainDbPath}`);
|
|
84
|
+
}
|
|
85
|
+
}
|
|
86
|
+
if (deps.shouldReconcileWorktreeDb(worktreeDbPath, mainDbPath)) {
|
|
87
|
+
deps.reconcileWorktreeDb(mainDbPath, worktreeDbPath);
|
|
88
|
+
}
|
|
89
|
+
} catch (err) {
|
|
90
|
+
const message = `DB reconciliation failed before milestone ${milestoneId} merge: ${err instanceof Error ? err.message : String(err)}`;
|
|
91
|
+
deps.logError("worktree", message);
|
|
92
|
+
throw new GSDError(
|
|
93
|
+
GSD_GIT_ERROR,
|
|
94
|
+
`${message}. Recovery reason: ${CLOSEOUT_CONSISTENCY_BLOCKED_REASON}.`,
|
|
95
|
+
);
|
|
96
|
+
}
|
|
97
|
+
}
|
|
98
|
+
|
|
99
|
+
function assertCloseoutProof(milestoneId: string): void {
|
|
100
|
+
const closeoutProof = deps.proveMilestoneCloseout(milestoneId);
|
|
101
|
+
if (!closeoutProof.ok) {
|
|
102
|
+
throw new GSDError(GSD_GIT_ERROR, deps.formatCloseoutProofBlock(closeoutProof));
|
|
103
|
+
}
|
|
104
|
+
}
|
|
105
|
+
|
|
106
|
+
export function assertMilestoneDbReadyForMerge(
|
|
107
|
+
request: MilestoneDbReadyRequest,
|
|
108
|
+
): void {
|
|
109
|
+
if (!deps.isDbAvailable()) return;
|
|
110
|
+
reconcileWorktreeDatabase(request);
|
|
111
|
+
assertCloseoutProof(request.milestoneId);
|
|
112
|
+
}
|
|
@@ -0,0 +1,99 @@
|
|
|
1
|
+
// gsd-pi — Milestone merge integration-branch preparation.
|
|
2
|
+
//
|
|
3
|
+
// Owns branch-selection safety before the merge transaction touches the project
|
|
4
|
+
// root: resolve integration branch, fail closed on self-merge metadata, clear
|
|
5
|
+
// transient milestone state, reject detached HEAD, and checkout when needed.
|
|
6
|
+
|
|
7
|
+
import { GSDError, GSD_GIT_ERROR } from "./errors.js";
|
|
8
|
+
import {
|
|
9
|
+
resolveMilestoneIntegrationBranch,
|
|
10
|
+
} from "./git-service.js";
|
|
11
|
+
import {
|
|
12
|
+
nativeCheckoutBranch,
|
|
13
|
+
nativeDetectMainBranch,
|
|
14
|
+
nativeGetCurrentBranch,
|
|
15
|
+
} from "./native-git-bridge.js";
|
|
16
|
+
import { clearProjectRootStateFiles } from "./auto-worktree-cleanup.js";
|
|
17
|
+
|
|
18
|
+
export interface PrepareIntegrationBranchRequest {
|
|
19
|
+
milestoneBranch: string;
|
|
20
|
+
milestoneId: string;
|
|
21
|
+
prefs: Parameters<typeof resolveMilestoneIntegrationBranch>[2];
|
|
22
|
+
previousCwd: string;
|
|
23
|
+
projectRoot: string;
|
|
24
|
+
}
|
|
25
|
+
|
|
26
|
+
export interface PreparedIntegrationBranch {
|
|
27
|
+
mainBranch: string;
|
|
28
|
+
}
|
|
29
|
+
|
|
30
|
+
interface MergeIntegrationBranchDeps {
|
|
31
|
+
chdir: typeof process.chdir;
|
|
32
|
+
clearProjectRootStateFiles: typeof clearProjectRootStateFiles;
|
|
33
|
+
nativeCheckoutBranch: typeof nativeCheckoutBranch;
|
|
34
|
+
nativeDetectMainBranch: typeof nativeDetectMainBranch;
|
|
35
|
+
nativeGetCurrentBranch: typeof nativeGetCurrentBranch;
|
|
36
|
+
resolveMilestoneIntegrationBranch: typeof resolveMilestoneIntegrationBranch;
|
|
37
|
+
}
|
|
38
|
+
|
|
39
|
+
const defaultDeps: MergeIntegrationBranchDeps = {
|
|
40
|
+
chdir: process.chdir.bind(process),
|
|
41
|
+
clearProjectRootStateFiles,
|
|
42
|
+
nativeCheckoutBranch,
|
|
43
|
+
nativeDetectMainBranch,
|
|
44
|
+
nativeGetCurrentBranch,
|
|
45
|
+
resolveMilestoneIntegrationBranch,
|
|
46
|
+
};
|
|
47
|
+
|
|
48
|
+
let deps: MergeIntegrationBranchDeps = defaultDeps;
|
|
49
|
+
|
|
50
|
+
export function _setMergeIntegrationBranchDepsForTests(
|
|
51
|
+
overrides: Partial<MergeIntegrationBranchDeps>,
|
|
52
|
+
): void {
|
|
53
|
+
deps = { ...defaultDeps, ...overrides };
|
|
54
|
+
}
|
|
55
|
+
|
|
56
|
+
export function _resetMergeIntegrationBranchDepsForTests(): void {
|
|
57
|
+
deps = defaultDeps;
|
|
58
|
+
}
|
|
59
|
+
|
|
60
|
+
function normalizeLocalBranchRef(branch: string): string {
|
|
61
|
+
return branch.startsWith("refs/heads/")
|
|
62
|
+
? branch.slice("refs/heads/".length)
|
|
63
|
+
: branch;
|
|
64
|
+
}
|
|
65
|
+
|
|
66
|
+
export function prepareIntegrationBranchForMilestoneMerge(
|
|
67
|
+
request: PrepareIntegrationBranchRequest,
|
|
68
|
+
): PreparedIntegrationBranch {
|
|
69
|
+
const { milestoneBranch, milestoneId, prefs, previousCwd, projectRoot } = request;
|
|
70
|
+
const branchResolution = deps.resolveMilestoneIntegrationBranch(projectRoot, milestoneId, prefs);
|
|
71
|
+
const mainBranch = branchResolution.effectiveBranch ?? deps.nativeDetectMainBranch(projectRoot);
|
|
72
|
+
|
|
73
|
+
if (normalizeLocalBranchRef(mainBranch) === milestoneBranch) {
|
|
74
|
+
deps.chdir(previousCwd);
|
|
75
|
+
throw new GSDError(
|
|
76
|
+
GSD_GIT_ERROR,
|
|
77
|
+
`Resolved integration branch "${mainBranch}" is the same ref as milestone branch ` +
|
|
78
|
+
`"${milestoneBranch}" — refusing to self-merge. ${branchResolution.reason}. ` +
|
|
79
|
+
`Repair milestone integration metadata before retrying milestone completion.`,
|
|
80
|
+
);
|
|
81
|
+
}
|
|
82
|
+
|
|
83
|
+
deps.clearProjectRootStateFiles(projectRoot, milestoneId);
|
|
84
|
+
|
|
85
|
+
const currentBranchAtBase = deps.nativeGetCurrentBranch(projectRoot);
|
|
86
|
+
if (!currentBranchAtBase || currentBranchAtBase.length === 0) {
|
|
87
|
+
deps.chdir(previousCwd);
|
|
88
|
+
throw new GSDError(
|
|
89
|
+
GSD_GIT_ERROR,
|
|
90
|
+
`Project root is in detached HEAD state — cannot perform milestone merge. ` +
|
|
91
|
+
`Checkout an integration branch (e.g. \`git checkout ${mainBranch}\`) before resuming.`,
|
|
92
|
+
);
|
|
93
|
+
}
|
|
94
|
+
if (currentBranchAtBase !== mainBranch) {
|
|
95
|
+
deps.nativeCheckoutBranch(projectRoot, mainBranch);
|
|
96
|
+
}
|
|
97
|
+
|
|
98
|
+
return { mainBranch };
|
|
99
|
+
}
|
|
@@ -0,0 +1,166 @@
|
|
|
1
|
+
// gsd-pi — Milestone merge commit-message construction.
|
|
2
|
+
//
|
|
3
|
+
// Owns the completed-slice/task summary and milestone title fallback policy used
|
|
4
|
+
// by auto-worktree milestone merge commits and publication summaries.
|
|
5
|
+
|
|
6
|
+
import {
|
|
7
|
+
getMilestone,
|
|
8
|
+
getMilestoneSlices,
|
|
9
|
+
getSliceTasks,
|
|
10
|
+
isDbAvailable,
|
|
11
|
+
} from "./gsd-db.js";
|
|
12
|
+
|
|
13
|
+
interface CompletedTaskSummary {
|
|
14
|
+
id: string;
|
|
15
|
+
title: string;
|
|
16
|
+
}
|
|
17
|
+
|
|
18
|
+
interface CompletedSliceSummary {
|
|
19
|
+
id: string;
|
|
20
|
+
title: string;
|
|
21
|
+
tasks: CompletedTaskSummary[];
|
|
22
|
+
}
|
|
23
|
+
|
|
24
|
+
export interface MilestoneMergeMessageRequest {
|
|
25
|
+
milestoneId: string;
|
|
26
|
+
milestoneBranch: string;
|
|
27
|
+
roadmapContent: string;
|
|
28
|
+
}
|
|
29
|
+
|
|
30
|
+
export interface MilestoneMergeMessage {
|
|
31
|
+
commitMessage: string;
|
|
32
|
+
milestoneTitle: string;
|
|
33
|
+
sliceSummaries: string[];
|
|
34
|
+
}
|
|
35
|
+
|
|
36
|
+
interface MilestoneMergeMessageDeps {
|
|
37
|
+
isDbAvailable: typeof isDbAvailable;
|
|
38
|
+
getMilestone: typeof getMilestone;
|
|
39
|
+
getMilestoneSlices: typeof getMilestoneSlices;
|
|
40
|
+
getSliceTasks: typeof getSliceTasks;
|
|
41
|
+
}
|
|
42
|
+
|
|
43
|
+
const defaultDeps: MilestoneMergeMessageDeps = {
|
|
44
|
+
isDbAvailable,
|
|
45
|
+
getMilestone,
|
|
46
|
+
getMilestoneSlices,
|
|
47
|
+
getSliceTasks,
|
|
48
|
+
};
|
|
49
|
+
|
|
50
|
+
let deps: MilestoneMergeMessageDeps = defaultDeps;
|
|
51
|
+
|
|
52
|
+
export function _setMilestoneMergeMessageDepsForTests(
|
|
53
|
+
overrides: Partial<MilestoneMergeMessageDeps>,
|
|
54
|
+
): void {
|
|
55
|
+
deps = { ...defaultDeps, ...overrides };
|
|
56
|
+
}
|
|
57
|
+
|
|
58
|
+
export function _resetMilestoneMergeMessageDepsForTests(): void {
|
|
59
|
+
deps = defaultDeps;
|
|
60
|
+
}
|
|
61
|
+
|
|
62
|
+
function stripGsdDisplayPrefix(value: string | undefined | null, id: string): string | undefined {
|
|
63
|
+
const raw = String(value ?? "").trim();
|
|
64
|
+
if (!raw) return undefined;
|
|
65
|
+
|
|
66
|
+
const lower = raw.toLowerCase();
|
|
67
|
+
const idLower = id.toLowerCase();
|
|
68
|
+
if (lower.startsWith(`${idLower}:`)) {
|
|
69
|
+
return raw.slice(id.length + 1).trim() || undefined;
|
|
70
|
+
}
|
|
71
|
+
return raw;
|
|
72
|
+
}
|
|
73
|
+
|
|
74
|
+
function getDbCompletedSlices(milestoneId: string): CompletedSliceSummary[] {
|
|
75
|
+
if (!deps.isDbAvailable()) return [];
|
|
76
|
+
return deps.getMilestoneSlices(milestoneId)
|
|
77
|
+
.filter((slice) => slice.status === "complete")
|
|
78
|
+
.map((slice) => ({
|
|
79
|
+
id: slice.id,
|
|
80
|
+
title: stripGsdDisplayPrefix(slice.title, slice.id) ?? slice.id,
|
|
81
|
+
tasks: deps.getSliceTasks(milestoneId, slice.id)
|
|
82
|
+
.filter((task) => task.status === "complete")
|
|
83
|
+
.map((task) => ({
|
|
84
|
+
id: task.id,
|
|
85
|
+
title: stripGsdDisplayPrefix(task.title, task.id) ?? task.id,
|
|
86
|
+
})),
|
|
87
|
+
}));
|
|
88
|
+
}
|
|
89
|
+
|
|
90
|
+
function getRoadmapCompletedSlices(roadmapContent: string): CompletedSliceSummary[] {
|
|
91
|
+
if (!roadmapContent) return [];
|
|
92
|
+
|
|
93
|
+
const completedSlices: CompletedSliceSummary[] = [];
|
|
94
|
+
const sliceRe = /- \[x\] \*\*(\w+):\s*(.+?)\*\*/gi;
|
|
95
|
+
let match: RegExpExecArray | null;
|
|
96
|
+
while ((match = sliceRe.exec(roadmapContent)) !== null) {
|
|
97
|
+
completedSlices.push({ id: match[1], title: match[2], tasks: [] });
|
|
98
|
+
}
|
|
99
|
+
return completedSlices;
|
|
100
|
+
}
|
|
101
|
+
|
|
102
|
+
function getCompletedSlices(
|
|
103
|
+
milestoneId: string,
|
|
104
|
+
roadmapContent: string,
|
|
105
|
+
): CompletedSliceSummary[] {
|
|
106
|
+
const dbCompletedSlices = getDbCompletedSlices(milestoneId);
|
|
107
|
+
if (dbCompletedSlices.length > 0) return dbCompletedSlices;
|
|
108
|
+
return getRoadmapCompletedSlices(roadmapContent);
|
|
109
|
+
}
|
|
110
|
+
|
|
111
|
+
function getRoadmapMilestoneTitle(milestoneId: string, roadmapContent: string): string | undefined {
|
|
112
|
+
if (!roadmapContent) return undefined;
|
|
113
|
+
const titleMatch = roadmapContent.match(new RegExp(`^#\\s+${milestoneId}:\\s*(.+)`, "m"));
|
|
114
|
+
return titleMatch?.[1]?.trim() || undefined;
|
|
115
|
+
}
|
|
116
|
+
|
|
117
|
+
function getMilestoneTitle(milestoneId: string, roadmapContent: string): string {
|
|
118
|
+
const dbMilestone = deps.getMilestone(milestoneId);
|
|
119
|
+
const dbTitle = stripGsdDisplayPrefix(dbMilestone?.title, milestoneId);
|
|
120
|
+
return dbTitle ?? getRoadmapMilestoneTitle(milestoneId, roadmapContent) ?? milestoneId;
|
|
121
|
+
}
|
|
122
|
+
|
|
123
|
+
function buildCommitMessage(request: {
|
|
124
|
+
completedSlices: CompletedSliceSummary[];
|
|
125
|
+
milestoneBranch: string;
|
|
126
|
+
milestoneId: string;
|
|
127
|
+
milestoneTitle: string;
|
|
128
|
+
}): string {
|
|
129
|
+
const { completedSlices, milestoneBranch, milestoneId, milestoneTitle } = request;
|
|
130
|
+
const subject = `feat: ${milestoneTitle}`;
|
|
131
|
+
const milestoneContext = milestoneTitle === milestoneId
|
|
132
|
+
? `Milestone: ${milestoneId}`
|
|
133
|
+
: `Milestone: ${milestoneId} - ${milestoneTitle}`;
|
|
134
|
+
|
|
135
|
+
if (completedSlices.length === 0) {
|
|
136
|
+
return `${subject}\n\n${milestoneContext}\nGSD-Milestone: ${milestoneId}\nBranch: ${milestoneBranch}`;
|
|
137
|
+
}
|
|
138
|
+
|
|
139
|
+
const sliceLines = completedSlices
|
|
140
|
+
.map((slice) => `- ${slice.id}: ${slice.title}`)
|
|
141
|
+
.join("\n");
|
|
142
|
+
const taskLines = completedSlices
|
|
143
|
+
.flatMap((slice) => slice.tasks.map((task) => `- ${slice.id}/${task.id}: ${task.title}`))
|
|
144
|
+
.join("\n");
|
|
145
|
+
const taskBlock = taskLines ? `\n\nCompleted tasks:\n${taskLines}` : "";
|
|
146
|
+
|
|
147
|
+
return `${subject}\n\nCompleted slices:\n${sliceLines}${taskBlock}\n\n${milestoneContext}\nGSD-Milestone: ${milestoneId}\nBranch: ${milestoneBranch}`;
|
|
148
|
+
}
|
|
149
|
+
|
|
150
|
+
export function buildMilestoneMergeMessage(
|
|
151
|
+
request: MilestoneMergeMessageRequest,
|
|
152
|
+
): MilestoneMergeMessage {
|
|
153
|
+
const { milestoneId, milestoneBranch, roadmapContent } = request;
|
|
154
|
+
const completedSlices = getCompletedSlices(milestoneId, roadmapContent);
|
|
155
|
+
const milestoneTitle = getMilestoneTitle(milestoneId, roadmapContent);
|
|
156
|
+
return {
|
|
157
|
+
commitMessage: buildCommitMessage({
|
|
158
|
+
completedSlices,
|
|
159
|
+
milestoneBranch,
|
|
160
|
+
milestoneId,
|
|
161
|
+
milestoneTitle,
|
|
162
|
+
}),
|
|
163
|
+
milestoneTitle,
|
|
164
|
+
sliceSummaries: completedSlices.map((slice) => `### ${slice.id}\n${slice.title}`),
|
|
165
|
+
};
|
|
166
|
+
}
|
|
@@ -0,0 +1,93 @@
|
|
|
1
|
+
// gsd-pi — Milestone merge pre-teardown safety guard.
|
|
2
|
+
//
|
|
3
|
+
// Owns the final data-loss check before deleting a merged milestone worktree:
|
|
4
|
+
// if the milestone worktree still has uncommitted work, abort teardown so the
|
|
5
|
+
// milestone branch remains a recovery ref.
|
|
6
|
+
|
|
7
|
+
import { existsSync } from "node:fs";
|
|
8
|
+
|
|
9
|
+
import { debugLog } from "./debug-logger.js";
|
|
10
|
+
import { GSDError, GSD_GIT_ERROR } from "./errors.js";
|
|
11
|
+
import {
|
|
12
|
+
nativeGetCurrentBranch,
|
|
13
|
+
nativeWorkingTreeStatus,
|
|
14
|
+
} from "./native-git-bridge.js";
|
|
15
|
+
|
|
16
|
+
export interface PreTeardownSafetyRequest {
|
|
17
|
+
milestoneBranch: string;
|
|
18
|
+
previousCwd: string;
|
|
19
|
+
worktreeCwd: string;
|
|
20
|
+
}
|
|
21
|
+
|
|
22
|
+
interface PreTeardownSafetyDeps {
|
|
23
|
+
chdir: typeof process.chdir;
|
|
24
|
+
debugLog: typeof debugLog;
|
|
25
|
+
existsSync: typeof existsSync;
|
|
26
|
+
nativeGetCurrentBranch: typeof nativeGetCurrentBranch;
|
|
27
|
+
nativeWorkingTreeStatus: typeof nativeWorkingTreeStatus;
|
|
28
|
+
}
|
|
29
|
+
|
|
30
|
+
const defaultDeps: PreTeardownSafetyDeps = {
|
|
31
|
+
chdir: process.chdir.bind(process),
|
|
32
|
+
debugLog,
|
|
33
|
+
existsSync,
|
|
34
|
+
nativeGetCurrentBranch,
|
|
35
|
+
nativeWorkingTreeStatus,
|
|
36
|
+
};
|
|
37
|
+
|
|
38
|
+
let deps: PreTeardownSafetyDeps = defaultDeps;
|
|
39
|
+
|
|
40
|
+
export function _setPreTeardownSafetyDepsForTests(
|
|
41
|
+
overrides: Partial<PreTeardownSafetyDeps>,
|
|
42
|
+
): void {
|
|
43
|
+
deps = { ...defaultDeps, ...overrides };
|
|
44
|
+
}
|
|
45
|
+
|
|
46
|
+
export function _resetPreTeardownSafetyDepsForTests(): void {
|
|
47
|
+
deps = defaultDeps;
|
|
48
|
+
}
|
|
49
|
+
|
|
50
|
+
/**
|
|
51
|
+
* Abort teardown when the milestone worktree still has uncommitted changes.
|
|
52
|
+
*
|
|
53
|
+
* The guard is intentionally scoped to worktree paths that are still on the
|
|
54
|
+
* milestone branch. In branch-mode or parallel merge paths, `worktreeCwd` can
|
|
55
|
+
* point at the integration branch; blocking on that dirty state would conflate
|
|
56
|
+
* unrelated milestones and preserve the wrong recovery ref.
|
|
57
|
+
*/
|
|
58
|
+
export function assertMilestoneWorktreeCleanBeforeTeardown(
|
|
59
|
+
request: PreTeardownSafetyRequest,
|
|
60
|
+
): void {
|
|
61
|
+
const { milestoneBranch, previousCwd, worktreeCwd } = request;
|
|
62
|
+
if (!deps.existsSync(worktreeCwd)) return;
|
|
63
|
+
|
|
64
|
+
let preTeardownBranch: string | null = null;
|
|
65
|
+
try {
|
|
66
|
+
preTeardownBranch = deps.nativeGetCurrentBranch(worktreeCwd);
|
|
67
|
+
} catch (err) {
|
|
68
|
+
deps.debugLog("mergeMilestoneToMain", {
|
|
69
|
+
phase: "pre-teardown-branch-detect-failed",
|
|
70
|
+
error: String(err),
|
|
71
|
+
});
|
|
72
|
+
}
|
|
73
|
+
|
|
74
|
+
if (preTeardownBranch !== milestoneBranch) return;
|
|
75
|
+
|
|
76
|
+
try {
|
|
77
|
+
const dirtyCheck = deps.nativeWorkingTreeStatus(worktreeCwd);
|
|
78
|
+
if (!dirtyCheck) return;
|
|
79
|
+
|
|
80
|
+
deps.chdir(previousCwd);
|
|
81
|
+
throw new GSDError(
|
|
82
|
+
GSD_GIT_ERROR,
|
|
83
|
+
`Milestone worktree still has uncommitted changes after squash merge. ` +
|
|
84
|
+
`Aborting teardown to preserve ${milestoneBranch}. Status:\n${dirtyCheck}`,
|
|
85
|
+
);
|
|
86
|
+
} catch (err) {
|
|
87
|
+
if (err instanceof GSDError) throw err;
|
|
88
|
+
deps.debugLog("mergeMilestoneToMain", {
|
|
89
|
+
phase: "pre-teardown-dirty-check-error",
|
|
90
|
+
error: String(err),
|
|
91
|
+
});
|
|
92
|
+
}
|
|
93
|
+
}
|
|
@@ -0,0 +1,213 @@
|
|
|
1
|
+
// gsd-pi — Pre-merge stash handling for auto-worktree merge closeout.
|
|
2
|
+
//
|
|
3
|
+
// Owns the local-work protection around milestone merge: stash before merge,
|
|
4
|
+
// DB handle cycling for Windows, restoring on merge failures, and post-commit
|
|
5
|
+
// stash-pop conflict recovery.
|
|
6
|
+
|
|
7
|
+
import { execFileSync } from "node:child_process";
|
|
8
|
+
import { join } from "node:path";
|
|
9
|
+
|
|
10
|
+
import {
|
|
11
|
+
closeWorkflowDatabase,
|
|
12
|
+
getWorkflowDatabasePath,
|
|
13
|
+
openWorkflowDatabasePath,
|
|
14
|
+
} from "./db-workspace.js";
|
|
15
|
+
import {
|
|
16
|
+
nativeAddPaths,
|
|
17
|
+
nativeConflictFiles,
|
|
18
|
+
nativeLsFiles,
|
|
19
|
+
nativeRmForce,
|
|
20
|
+
} from "./native-git-bridge.js";
|
|
21
|
+
import {
|
|
22
|
+
gsdJsonlFilesWithConflictMarkers,
|
|
23
|
+
hasConflictMarkers,
|
|
24
|
+
popStashByRef,
|
|
25
|
+
stashAlreadyExistsFilesFromError,
|
|
26
|
+
stashRefFromError,
|
|
27
|
+
} from "./worktree-git-recovery.js";
|
|
28
|
+
import { logWarning } from "./workflow-logger.js";
|
|
29
|
+
|
|
30
|
+
export interface PreMergeStash {
|
|
31
|
+
stash(): void;
|
|
32
|
+
reopenDbAfterMerge(): void;
|
|
33
|
+
restoreForMergeFailure(): void;
|
|
34
|
+
restoreAfterCommit(): void;
|
|
35
|
+
}
|
|
36
|
+
|
|
37
|
+
export function createPreMergeStash(
|
|
38
|
+
basePath: string,
|
|
39
|
+
milestoneId: string,
|
|
40
|
+
cycleDbHandles: boolean,
|
|
41
|
+
): PreMergeStash {
|
|
42
|
+
let stashed = false;
|
|
43
|
+
let marker: string | null = null;
|
|
44
|
+
const dbPathToReopen = cycleDbHandles ? getWorkflowDatabasePath() : null;
|
|
45
|
+
|
|
46
|
+
return {
|
|
47
|
+
stash(): void {
|
|
48
|
+
closeDbBeforeStashIfNeeded(cycleDbHandles);
|
|
49
|
+
try {
|
|
50
|
+
const status = execFileSync("git", ["status", "--porcelain"], {
|
|
51
|
+
cwd: basePath,
|
|
52
|
+
stdio: ["ignore", "pipe", "pipe"],
|
|
53
|
+
encoding: "utf-8",
|
|
54
|
+
}).trim();
|
|
55
|
+
if (!status) return;
|
|
56
|
+
|
|
57
|
+
marker = createStashMarker(milestoneId);
|
|
58
|
+
execFileSync(
|
|
59
|
+
"git",
|
|
60
|
+
["stash", "push", "--include-untracked", "-m", `gsd: pre-merge stash for ${milestoneId} [${marker}]`],
|
|
61
|
+
{ cwd: basePath, stdio: ["ignore", "pipe", "pipe"], encoding: "utf-8" },
|
|
62
|
+
);
|
|
63
|
+
stashed = true;
|
|
64
|
+
} catch (err) {
|
|
65
|
+
// Stash failure is non-fatal — proceed without stash and let the merge
|
|
66
|
+
// report the dirty tree if it fails.
|
|
67
|
+
logWarning("worktree", `git stash failed: ${err instanceof Error ? err.message : String(err)}`);
|
|
68
|
+
}
|
|
69
|
+
},
|
|
70
|
+
|
|
71
|
+
reopenDbAfterMerge(): void {
|
|
72
|
+
if (!cycleDbHandles || !dbPathToReopen) return;
|
|
73
|
+
try {
|
|
74
|
+
openWorkflowDatabasePath(dbPathToReopen);
|
|
75
|
+
} catch (err) {
|
|
76
|
+
logWarning("worktree", `post-merge db reopen failed: ${err instanceof Error ? err.message : String(err)}`);
|
|
77
|
+
}
|
|
78
|
+
},
|
|
79
|
+
|
|
80
|
+
restoreForMergeFailure(): void {
|
|
81
|
+
if (!stashed) return;
|
|
82
|
+
try {
|
|
83
|
+
popStashByRef(basePath, marker);
|
|
84
|
+
} catch (err) {
|
|
85
|
+
logWarning("worktree", `git stash pop failed: ${err instanceof Error ? err.message : String(err)}`);
|
|
86
|
+
}
|
|
87
|
+
},
|
|
88
|
+
|
|
89
|
+
restoreAfterCommit(): void {
|
|
90
|
+
if (!stashed) return;
|
|
91
|
+
restoreStashAfterCommit(basePath, marker);
|
|
92
|
+
},
|
|
93
|
+
};
|
|
94
|
+
}
|
|
95
|
+
|
|
96
|
+
function closeDbBeforeStashIfNeeded(cycleDbHandles: boolean): void {
|
|
97
|
+
if (!cycleDbHandles) return;
|
|
98
|
+
try {
|
|
99
|
+
closeWorkflowDatabase();
|
|
100
|
+
} catch (err) {
|
|
101
|
+
logWarning("worktree", `pre-stash db close failed: ${err instanceof Error ? err.message : String(err)}`);
|
|
102
|
+
}
|
|
103
|
+
}
|
|
104
|
+
|
|
105
|
+
function createStashMarker(milestoneId: string): string {
|
|
106
|
+
return `gsd-pre-merge:${milestoneId}:${process.pid}:${Date.now()}:${process.hrtime.bigint().toString(36)}`;
|
|
107
|
+
}
|
|
108
|
+
|
|
109
|
+
function restoreStashAfterCommit(basePath: string, marker: string | null): void {
|
|
110
|
+
let stashRefForDrop: string | null = null;
|
|
111
|
+
try {
|
|
112
|
+
stashRefForDrop = popStashByRef(basePath, marker);
|
|
113
|
+
} catch (e) {
|
|
114
|
+
stashRefForDrop = stashRefFromError(e);
|
|
115
|
+
logWarning("worktree", `git stash pop failed, attempting conflict resolution: ${(e as Error).message}`);
|
|
116
|
+
resolveStashPopConflict(basePath, e, stashRefForDrop);
|
|
117
|
+
}
|
|
118
|
+
}
|
|
119
|
+
|
|
120
|
+
function resolveStashPopConflict(basePath: string, error: unknown, stashRefForDrop: string | null): void {
|
|
121
|
+
const uu = nativeConflictFiles(basePath);
|
|
122
|
+
const gsdUU = uu.filter((f) => f.startsWith(".gsd/"));
|
|
123
|
+
const nonGsdUU = uu.filter((f) => !f.startsWith(".gsd/"));
|
|
124
|
+
const stashPopMessage = error instanceof Error ? error.message : String(error);
|
|
125
|
+
const isUntrackedRestoreFailure = stashPopMessage.includes("could not restore untracked files from stash");
|
|
126
|
+
const gsdContentConflicts: string[] = [];
|
|
127
|
+
const alreadyExists = stashAlreadyExistsFilesFromError(error);
|
|
128
|
+
|
|
129
|
+
if (isUntrackedRestoreFailure) {
|
|
130
|
+
gsdContentConflicts.push(...gsdJsonlFilesWithConflictMarkers(basePath));
|
|
131
|
+
}
|
|
132
|
+
|
|
133
|
+
const gsdConflictFiles = [...new Set([...gsdUU, ...gsdContentConflicts])];
|
|
134
|
+
resolveGsdConflictFiles(basePath, gsdConflictFiles);
|
|
135
|
+
|
|
136
|
+
if (gsdConflictFiles.length > 0 && nonGsdUU.length === 0) {
|
|
137
|
+
dropStashIfNoNonGsdConflictsRemain(basePath, stashRefForDrop);
|
|
138
|
+
return;
|
|
139
|
+
}
|
|
140
|
+
|
|
141
|
+
if (gsdUU.length === 0 && nonGsdUU.length === 0 && alreadyExists.length > 0) {
|
|
142
|
+
dropRecordedStash(basePath, stashRefForDrop);
|
|
143
|
+
return;
|
|
144
|
+
}
|
|
145
|
+
|
|
146
|
+
if (nonGsdUU.length > 0) {
|
|
147
|
+
logWarning("reconcile", "Stash pop conflict on non-.gsd files after merge", {
|
|
148
|
+
files: nonGsdUU.join(", "),
|
|
149
|
+
});
|
|
150
|
+
return;
|
|
151
|
+
}
|
|
152
|
+
|
|
153
|
+
logWarning(
|
|
154
|
+
"worktree",
|
|
155
|
+
"git stash pop failed without resolvable conflict files; leaving stash for manual recovery",
|
|
156
|
+
);
|
|
157
|
+
}
|
|
158
|
+
|
|
159
|
+
function resolveGsdConflictFiles(basePath: string, conflictFiles: string[]): void {
|
|
160
|
+
for (const file of conflictFiles) {
|
|
161
|
+
try {
|
|
162
|
+
// Accept the committed (HEAD) version of the state file.
|
|
163
|
+
execFileSync("git", ["checkout", "HEAD", "--", file], {
|
|
164
|
+
cwd: basePath,
|
|
165
|
+
stdio: ["ignore", "pipe", "pipe"],
|
|
166
|
+
encoding: "utf-8",
|
|
167
|
+
});
|
|
168
|
+
nativeAddPaths(basePath, [file]);
|
|
169
|
+
} catch (err) {
|
|
170
|
+
// Last resort: remove the conflicted state file.
|
|
171
|
+
logWarning("worktree", `checkout HEAD failed for ${file}, removing: ${(err as Error).message}`);
|
|
172
|
+
nativeRmForce(basePath, [file]);
|
|
173
|
+
}
|
|
174
|
+
}
|
|
175
|
+
}
|
|
176
|
+
|
|
177
|
+
function dropStashIfNoNonGsdConflictsRemain(basePath: string, stashRefForDrop: string | null): void {
|
|
178
|
+
const remainingUnmerged = nativeConflictFiles(basePath);
|
|
179
|
+
const nonGsdUnmerged = remainingUnmerged.filter((f) => !f.startsWith(".gsd/"));
|
|
180
|
+
const markerCandidates = Array.from(new Set([
|
|
181
|
+
...nonGsdUnmerged,
|
|
182
|
+
...nativeLsFiles(basePath, "."),
|
|
183
|
+
])).filter((f) => !f.startsWith(".gsd/"));
|
|
184
|
+
const nonGsdMarkerConflicts = markerCandidates.filter((f) =>
|
|
185
|
+
hasConflictMarkers(join(basePath, f)),
|
|
186
|
+
);
|
|
187
|
+
const hasRemainingNonGsdConflicts = nonGsdUnmerged.length > 0 || nonGsdMarkerConflicts.length > 0;
|
|
188
|
+
if (hasRemainingNonGsdConflicts) {
|
|
189
|
+
const files = Array.from(new Set([...nonGsdUnmerged, ...nonGsdMarkerConflicts]));
|
|
190
|
+
logWarning("reconcile", "Leaving stash because non-.gsd conflicts remain after auto-resolution", {
|
|
191
|
+
files: files.join(", "),
|
|
192
|
+
});
|
|
193
|
+
return;
|
|
194
|
+
}
|
|
195
|
+
|
|
196
|
+
dropRecordedStash(basePath, stashRefForDrop);
|
|
197
|
+
}
|
|
198
|
+
|
|
199
|
+
function dropRecordedStash(basePath: string, stashRefForDrop: string | null): void {
|
|
200
|
+
if (!stashRefForDrop) {
|
|
201
|
+
logWarning("worktree", "recorded stash entry could not be resolved; skipping automatic drop");
|
|
202
|
+
return;
|
|
203
|
+
}
|
|
204
|
+
try {
|
|
205
|
+
execFileSync("git", ["stash", "drop", stashRefForDrop], {
|
|
206
|
+
cwd: basePath,
|
|
207
|
+
stdio: ["ignore", "pipe", "pipe"],
|
|
208
|
+
encoding: "utf-8",
|
|
209
|
+
});
|
|
210
|
+
} catch (err) {
|
|
211
|
+
logWarning("worktree", `git stash drop failed: ${err instanceof Error ? err.message : String(err)}`);
|
|
212
|
+
}
|
|
213
|
+
}
|