orbital-command 0.1.4 → 0.3.0
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/bin/orbital.js +676 -53
- package/dist/assets/PrimitivesConfig-CrmQXYh4.js +32 -0
- package/dist/assets/QualityGates-BbasOsF3.js +21 -0
- package/dist/assets/SessionTimeline-CGeJsVvy.js +1 -0
- package/dist/assets/Settings-oiM496mc.js +12 -0
- package/dist/assets/SourceControl-B1fP2nJL.js +41 -0
- package/dist/assets/WorkflowVisualizer-CWLYf-f0.js +74 -0
- package/dist/assets/arrow-down-CPy85_J6.js +6 -0
- package/dist/assets/charts-DbDg0Psc.js +68 -0
- package/dist/assets/circle-x-Cwz6ZQDV.js +6 -0
- package/dist/assets/file-text-C46Xr65c.js +6 -0
- package/dist/assets/formatDistanceToNow-BMqsSP44.js +1 -0
- package/dist/assets/globe-Cn2yNZUD.js +6 -0
- package/dist/assets/index-Aj4sV8Al.css +1 -0
- package/dist/assets/index-Bc9dK3MW.js +354 -0
- package/dist/assets/key-OPaNTWJ5.js +6 -0
- package/dist/assets/minus-GMsbpKym.js +6 -0
- package/dist/assets/shield-DwAFkDYI.js +6 -0
- package/dist/assets/ui-BmsSg9jU.js +53 -0
- package/dist/assets/useWorkflowEditor-BJkTX_NR.js +16 -0
- package/dist/assets/{vendor-Dzv9lrRc.js → vendor-Bqt8AJn2.js} +1 -1
- package/dist/assets/zap-DfbUoOty.js +11 -0
- package/dist/favicon.svg +1 -0
- package/dist/index.html +6 -5
- package/dist/server/server/__tests__/data-routes.test.js +124 -0
- package/dist/server/server/__tests__/helpers/db.js +17 -0
- package/dist/server/server/__tests__/helpers/mock-emitter.js +8 -0
- package/dist/server/server/__tests__/scope-routes.test.js +137 -0
- package/dist/server/server/__tests__/sprint-routes.test.js +102 -0
- package/dist/server/server/__tests__/workflow-routes.test.js +107 -0
- package/dist/server/server/config-migrator.js +138 -0
- package/dist/server/server/config.js +17 -2
- package/dist/server/server/database.js +27 -12
- package/dist/server/server/global-config.js +143 -0
- package/dist/server/server/index.js +882 -252
- package/dist/server/server/init.js +579 -194
- package/dist/server/server/launch.js +29 -0
- package/dist/server/server/manifest-types.js +8 -0
- package/dist/server/server/manifest.js +454 -0
- package/dist/server/server/migrate-legacy.js +229 -0
- package/dist/server/server/parsers/event-parser.test.js +117 -0
- package/dist/server/server/parsers/scope-parser.js +74 -28
- package/dist/server/server/parsers/scope-parser.test.js +230 -0
- package/dist/server/server/project-context.js +255 -0
- package/dist/server/server/project-emitter.js +41 -0
- package/dist/server/server/project-manager.js +297 -0
- package/dist/server/server/routes/config-routes.js +1 -3
- package/dist/server/server/routes/data-routes.js +22 -110
- package/dist/server/server/routes/dispatch-routes.js +15 -9
- package/dist/server/server/routes/git-routes.js +74 -0
- package/dist/server/server/routes/manifest-routes.js +319 -0
- package/dist/server/server/routes/scope-routes.js +37 -23
- package/dist/server/server/routes/sync-routes.js +134 -0
- package/dist/server/server/routes/version-routes.js +1 -15
- package/dist/server/server/routes/workflow-routes.js +9 -3
- package/dist/server/server/schema.js +2 -0
- package/dist/server/server/services/batch-orchestrator.js +26 -16
- package/dist/server/server/services/claude-session-service.js +17 -14
- package/dist/server/server/services/deploy-service.test.js +119 -0
- package/dist/server/server/services/event-service.js +64 -1
- package/dist/server/server/services/event-service.test.js +191 -0
- package/dist/server/server/services/gate-service.test.js +105 -0
- package/dist/server/server/services/git-service.js +108 -4
- package/dist/server/server/services/github-service.js +110 -2
- package/dist/server/server/services/readiness-service.test.js +190 -0
- package/dist/server/server/services/scope-cache.js +5 -1
- package/dist/server/server/services/scope-cache.test.js +142 -0
- package/dist/server/server/services/scope-service.js +217 -126
- package/dist/server/server/services/scope-service.test.js +137 -0
- package/dist/server/server/services/sprint-orchestrator.js +7 -6
- package/dist/server/server/services/sprint-service.js +21 -1
- package/dist/server/server/services/sprint-service.test.js +238 -0
- package/dist/server/server/services/sync-service.js +434 -0
- package/dist/server/server/services/sync-types.js +2 -0
- package/dist/server/server/services/telemetry-service.js +143 -0
- package/dist/server/server/services/workflow-service.js +26 -5
- package/dist/server/server/services/workflow-service.test.js +159 -0
- package/dist/server/server/settings-sync.js +284 -0
- package/dist/server/server/update-planner.js +279 -0
- package/dist/server/server/utils/cc-hooks-parser.js +3 -0
- package/dist/server/server/utils/cc-hooks-parser.test.js +86 -0
- package/dist/server/server/utils/dispatch-utils.js +77 -20
- package/dist/server/server/utils/dispatch-utils.test.js +182 -0
- package/dist/server/server/utils/logger.js +37 -3
- package/dist/server/server/utils/package-info.js +30 -0
- package/dist/server/server/utils/route-helpers.js +10 -0
- package/dist/server/server/utils/terminal-launcher.js +79 -25
- package/dist/server/server/utils/worktree-manager.js +13 -4
- package/dist/server/server/validator.js +230 -0
- package/dist/server/server/watchers/global-watcher.js +63 -0
- package/dist/server/server/watchers/scope-watcher.js +27 -12
- package/dist/server/server/wizard/config-editor.js +237 -0
- package/dist/server/server/wizard/detect.js +96 -0
- package/dist/server/server/wizard/doctor.js +115 -0
- package/dist/server/server/wizard/index.js +155 -0
- package/dist/server/server/wizard/phases/confirm.js +39 -0
- package/dist/server/server/wizard/phases/project-setup.js +90 -0
- package/dist/server/server/wizard/phases/setup-wizard.js +66 -0
- package/dist/server/server/wizard/phases/welcome.js +35 -0
- package/dist/server/server/wizard/phases/workflow-setup.js +22 -0
- package/dist/server/server/wizard/types.js +29 -0
- package/dist/server/server/wizard/ui.js +74 -0
- package/dist/server/shared/__fixtures__/workflow-configs.js +75 -0
- package/dist/server/shared/default-workflow.json +65 -0
- package/dist/server/shared/onboarding-tour.test.js +81 -0
- package/dist/server/shared/project-colors.js +24 -0
- package/dist/server/shared/workflow-config.test.js +84 -0
- package/dist/server/shared/workflow-engine.test.js +302 -0
- package/dist/server/shared/workflow-normalizer.js +101 -0
- package/dist/server/shared/workflow-normalizer.test.js +100 -0
- package/dist/server/src/components/onboarding/tour-steps.js +84 -0
- package/package.json +20 -15
- package/schemas/orbital.config.schema.json +16 -1
- package/scripts/postinstall.js +55 -7
- package/server/__tests__/data-routes.test.ts +149 -0
- package/server/__tests__/helpers/db.ts +19 -0
- package/server/__tests__/helpers/mock-emitter.ts +10 -0
- package/server/__tests__/scope-routes.test.ts +157 -0
- package/server/__tests__/sprint-routes.test.ts +118 -0
- package/server/__tests__/workflow-routes.test.ts +120 -0
- package/server/config-migrator.ts +163 -0
- package/server/config.ts +26 -2
- package/server/database.ts +35 -18
- package/server/global-config.ts +200 -0
- package/server/index.ts +975 -287
- package/server/init.ts +625 -182
- package/server/launch.ts +32 -0
- package/server/manifest-types.ts +145 -0
- package/server/manifest.ts +494 -0
- package/server/migrate-legacy.ts +290 -0
- package/server/parsers/event-parser.test.ts +135 -0
- package/server/parsers/scope-parser.test.ts +270 -0
- package/server/parsers/scope-parser.ts +79 -31
- package/server/project-context.ts +309 -0
- package/server/project-emitter.ts +50 -0
- package/server/project-manager.ts +369 -0
- package/server/routes/config-routes.ts +3 -5
- package/server/routes/data-routes.ts +28 -141
- package/server/routes/dispatch-routes.ts +19 -11
- package/server/routes/git-routes.ts +77 -0
- package/server/routes/manifest-routes.ts +388 -0
- package/server/routes/scope-routes.ts +29 -25
- package/server/routes/sync-routes.ts +175 -0
- package/server/routes/version-routes.ts +1 -16
- package/server/routes/workflow-routes.ts +9 -3
- package/server/schema.ts +2 -0
- package/server/services/batch-orchestrator.ts +24 -16
- package/server/services/claude-session-service.ts +16 -14
- package/server/services/deploy-service.test.ts +145 -0
- package/server/services/deploy-service.ts +2 -2
- package/server/services/event-service.test.ts +242 -0
- package/server/services/event-service.ts +92 -3
- package/server/services/gate-service.test.ts +131 -0
- package/server/services/gate-service.ts +2 -2
- package/server/services/git-service.ts +137 -4
- package/server/services/github-service.ts +120 -2
- package/server/services/readiness-service.test.ts +217 -0
- package/server/services/scope-cache.test.ts +167 -0
- package/server/services/scope-cache.ts +4 -1
- package/server/services/scope-service.test.ts +169 -0
- package/server/services/scope-service.ts +220 -126
- package/server/services/sprint-orchestrator.ts +7 -7
- package/server/services/sprint-service.test.ts +271 -0
- package/server/services/sprint-service.ts +27 -3
- package/server/services/sync-service.ts +482 -0
- package/server/services/sync-types.ts +77 -0
- package/server/services/telemetry-service.ts +195 -0
- package/server/services/workflow-service.test.ts +190 -0
- package/server/services/workflow-service.ts +29 -9
- package/server/settings-sync.ts +359 -0
- package/server/update-planner.ts +346 -0
- package/server/utils/cc-hooks-parser.test.ts +96 -0
- package/server/utils/cc-hooks-parser.ts +4 -0
- package/server/utils/dispatch-utils.test.ts +245 -0
- package/server/utils/dispatch-utils.ts +97 -27
- package/server/utils/logger.ts +40 -3
- package/server/utils/package-info.ts +32 -0
- package/server/utils/route-helpers.ts +12 -0
- package/server/utils/terminal-launcher.ts +85 -25
- package/server/utils/worktree-manager.ts +9 -4
- package/server/validator.ts +270 -0
- package/server/watchers/global-watcher.ts +77 -0
- package/server/watchers/scope-watcher.ts +21 -9
- package/server/wizard/config-editor.ts +248 -0
- package/server/wizard/detect.ts +104 -0
- package/server/wizard/doctor.ts +114 -0
- package/server/wizard/index.ts +187 -0
- package/server/wizard/phases/confirm.ts +45 -0
- package/server/wizard/phases/project-setup.ts +106 -0
- package/server/wizard/phases/setup-wizard.ts +78 -0
- package/server/wizard/phases/welcome.ts +43 -0
- package/server/wizard/phases/workflow-setup.ts +28 -0
- package/server/wizard/types.ts +56 -0
- package/server/wizard/ui.ts +93 -0
- package/shared/__fixtures__/workflow-configs.ts +80 -0
- package/shared/default-workflow.json +65 -0
- package/shared/onboarding-tour.test.ts +94 -0
- package/shared/project-colors.ts +24 -0
- package/shared/workflow-config.test.ts +111 -0
- package/shared/workflow-config.ts +7 -0
- package/shared/workflow-engine.test.ts +388 -0
- package/shared/workflow-normalizer.test.ts +119 -0
- package/shared/workflow-normalizer.ts +118 -0
- package/templates/hooks/end-session.sh +3 -1
- package/templates/hooks/orbital-emit.sh +2 -2
- package/templates/hooks/orbital-report-deploy.sh +4 -4
- package/templates/hooks/orbital-report-gates.sh +4 -4
- package/templates/hooks/orbital-scope-update.sh +1 -1
- package/templates/hooks/scope-create-cleanup.sh +2 -2
- package/templates/hooks/scope-create-gate.sh +0 -1
- package/templates/hooks/scope-helpers.sh +18 -0
- package/templates/hooks/scope-prepare.sh +66 -11
- package/templates/migrations/renames.json +1 -0
- package/templates/orbital.config.json +7 -2
- package/templates/settings-hooks.json +1 -1
- package/templates/skills/git-commit/SKILL.md +9 -4
- package/templates/skills/git-dev/SKILL.md +8 -3
- package/templates/skills/git-main/SKILL.md +8 -2
- package/templates/skills/git-production/SKILL.md +6 -2
- package/templates/skills/git-staging/SKILL.md +8 -3
- package/templates/skills/scope-create/SKILL.md +17 -3
- package/templates/skills/scope-fix-review/SKILL.md +6 -3
- package/templates/skills/scope-implement/SKILL.md +4 -1
- package/templates/skills/scope-post-review/SKILL.md +63 -5
- package/templates/skills/scope-pre-review/SKILL.md +5 -2
- package/templates/skills/scope-verify/SKILL.md +5 -3
- package/templates/skills/test-code-review/SKILL.md +41 -33
- package/templates/skills/test-scaffold/SKILL.md +222 -0
- package/dist/assets/WorkflowVisualizer-BZ21PIIF.js +0 -84
- package/dist/assets/charts-D__PA1zp.js +0 -72
- package/dist/assets/index-D1G6i0nS.css +0 -1
- package/dist/assets/index-DpItvKpf.js +0 -419
- package/dist/assets/ui-BvF022GT.js +0 -53
- package/index.html +0 -15
- package/postcss.config.js +0 -6
- package/src/App.tsx +0 -33
- package/src/components/AgentBadge.tsx +0 -40
- package/src/components/BatchPreflightModal.tsx +0 -115
- package/src/components/CardDisplayToggle.tsx +0 -74
- package/src/components/ColumnHeaderActions.tsx +0 -55
- package/src/components/ColumnMenu.tsx +0 -99
- package/src/components/DeployHistory.tsx +0 -141
- package/src/components/DispatchModal.tsx +0 -164
- package/src/components/DispatchPopover.tsx +0 -139
- package/src/components/DragOverlay.tsx +0 -25
- package/src/components/DriftSidebar.tsx +0 -140
- package/src/components/EnvironmentStrip.tsx +0 -88
- package/src/components/ErrorBoundary.tsx +0 -62
- package/src/components/FilterChip.tsx +0 -105
- package/src/components/GateIndicator.tsx +0 -33
- package/src/components/IdeaDetailModal.tsx +0 -190
- package/src/components/IdeaFormDialog.tsx +0 -113
- package/src/components/KanbanColumn.tsx +0 -201
- package/src/components/MarkdownRenderer.tsx +0 -114
- package/src/components/NeonGrid.tsx +0 -128
- package/src/components/PromotionQueue.tsx +0 -89
- package/src/components/ScopeCard.tsx +0 -234
- package/src/components/ScopeDetailModal.tsx +0 -255
- package/src/components/ScopeFilterBar.tsx +0 -152
- package/src/components/SearchInput.tsx +0 -102
- package/src/components/SessionPanel.tsx +0 -335
- package/src/components/SprintContainer.tsx +0 -303
- package/src/components/SprintDependencyDialog.tsx +0 -78
- package/src/components/SprintPreflightModal.tsx +0 -138
- package/src/components/StatusBar.tsx +0 -168
- package/src/components/SwimCell.tsx +0 -67
- package/src/components/SwimLaneRow.tsx +0 -94
- package/src/components/SwimlaneBoardView.tsx +0 -108
- package/src/components/VersionBadge.tsx +0 -139
- package/src/components/ViewModeSelector.tsx +0 -114
- package/src/components/config/AgentChip.tsx +0 -53
- package/src/components/config/AgentCreateDialog.tsx +0 -321
- package/src/components/config/AgentEditor.tsx +0 -175
- package/src/components/config/DirectoryTree.tsx +0 -582
- package/src/components/config/FileEditor.tsx +0 -550
- package/src/components/config/HookChip.tsx +0 -50
- package/src/components/config/StageCard.tsx +0 -198
- package/src/components/config/TransitionZone.tsx +0 -173
- package/src/components/config/UnifiedWorkflowPipeline.tsx +0 -216
- package/src/components/config/WorkflowPipeline.tsx +0 -161
- package/src/components/source-control/BranchList.tsx +0 -93
- package/src/components/source-control/BranchPanel.tsx +0 -105
- package/src/components/source-control/CommitLog.tsx +0 -100
- package/src/components/source-control/CommitRow.tsx +0 -47
- package/src/components/source-control/GitHubPanel.tsx +0 -110
- package/src/components/source-control/GitHubSetupGuide.tsx +0 -52
- package/src/components/source-control/GitOverviewBar.tsx +0 -101
- package/src/components/source-control/PullRequestList.tsx +0 -69
- package/src/components/source-control/WorktreeList.tsx +0 -80
- package/src/components/ui/badge.tsx +0 -41
- package/src/components/ui/button.tsx +0 -55
- package/src/components/ui/card.tsx +0 -78
- package/src/components/ui/dialog.tsx +0 -94
- package/src/components/ui/popover.tsx +0 -33
- package/src/components/ui/scroll-area.tsx +0 -54
- package/src/components/ui/separator.tsx +0 -28
- package/src/components/ui/tabs.tsx +0 -52
- package/src/components/ui/toggle-switch.tsx +0 -35
- package/src/components/ui/tooltip.tsx +0 -27
- package/src/components/workflow/AddEdgeDialog.tsx +0 -217
- package/src/components/workflow/AddListDialog.tsx +0 -201
- package/src/components/workflow/ChecklistEditor.tsx +0 -239
- package/src/components/workflow/CommandPrefixManager.tsx +0 -118
- package/src/components/workflow/ConfigSettingsPanel.tsx +0 -189
- package/src/components/workflow/DirectionSelector.tsx +0 -133
- package/src/components/workflow/DispatchConfigPanel.tsx +0 -180
- package/src/components/workflow/EdgeDetailPanel.tsx +0 -236
- package/src/components/workflow/EdgePropertyEditor.tsx +0 -251
- package/src/components/workflow/EditToolbar.tsx +0 -138
- package/src/components/workflow/HookDetailPanel.tsx +0 -250
- package/src/components/workflow/HookExecutionLog.tsx +0 -24
- package/src/components/workflow/HookSourceModal.tsx +0 -129
- package/src/components/workflow/HooksDashboard.tsx +0 -363
- package/src/components/workflow/ListPropertyEditor.tsx +0 -251
- package/src/components/workflow/MigrationPreviewDialog.tsx +0 -237
- package/src/components/workflow/MovementRulesPanel.tsx +0 -188
- package/src/components/workflow/NodeDetailPanel.tsx +0 -245
- package/src/components/workflow/PresetSelector.tsx +0 -414
- package/src/components/workflow/SkillCommandBuilder.tsx +0 -174
- package/src/components/workflow/WorkflowEdgeComponent.tsx +0 -145
- package/src/components/workflow/WorkflowNode.tsx +0 -147
- package/src/components/workflow/graphLayout.ts +0 -186
- package/src/components/workflow/mergeHooks.ts +0 -85
- package/src/components/workflow/useEditHistory.ts +0 -88
- package/src/components/workflow/useWorkflowEditor.ts +0 -262
- package/src/components/workflow/validateConfig.ts +0 -70
- package/src/hooks/useActiveDispatches.ts +0 -198
- package/src/hooks/useBoardSettings.ts +0 -170
- package/src/hooks/useCardDisplay.ts +0 -57
- package/src/hooks/useCcHooks.ts +0 -24
- package/src/hooks/useConfigTree.ts +0 -51
- package/src/hooks/useEnforcementRules.ts +0 -46
- package/src/hooks/useEvents.ts +0 -59
- package/src/hooks/useFileEditor.ts +0 -165
- package/src/hooks/useGates.ts +0 -57
- package/src/hooks/useIdeaActions.ts +0 -53
- package/src/hooks/useKanbanDnd.ts +0 -410
- package/src/hooks/useOrbitalConfig.ts +0 -54
- package/src/hooks/usePipeline.ts +0 -47
- package/src/hooks/usePipelineData.ts +0 -338
- package/src/hooks/useReconnect.ts +0 -25
- package/src/hooks/useScopeFilters.ts +0 -125
- package/src/hooks/useScopeSessions.ts +0 -44
- package/src/hooks/useScopes.ts +0 -67
- package/src/hooks/useSearch.ts +0 -67
- package/src/hooks/useSettings.tsx +0 -187
- package/src/hooks/useSocket.ts +0 -25
- package/src/hooks/useSourceControl.ts +0 -105
- package/src/hooks/useSprintPreflight.ts +0 -55
- package/src/hooks/useSprints.ts +0 -154
- package/src/hooks/useStatusBarHighlight.ts +0 -18
- package/src/hooks/useSwimlaneBoardSettings.ts +0 -104
- package/src/hooks/useTheme.ts +0 -9
- package/src/hooks/useTransitionReadiness.ts +0 -53
- package/src/hooks/useVersion.ts +0 -155
- package/src/hooks/useViolations.ts +0 -65
- package/src/hooks/useWorkflow.tsx +0 -125
- package/src/hooks/useZoomModifier.ts +0 -19
- package/src/index.css +0 -797
- package/src/layouts/DashboardLayout.tsx +0 -113
- package/src/lib/collisionDetection.ts +0 -20
- package/src/lib/scope-fields.ts +0 -61
- package/src/lib/swimlane.ts +0 -146
- package/src/lib/utils.ts +0 -15
- package/src/main.tsx +0 -19
- package/src/socket.ts +0 -11
- package/src/types/index.ts +0 -497
- package/src/views/AgentFeed.tsx +0 -339
- package/src/views/DeployPipeline.tsx +0 -59
- package/src/views/EnforcementView.tsx +0 -378
- package/src/views/PrimitivesConfig.tsx +0 -500
- package/src/views/QualityGates.tsx +0 -1012
- package/src/views/ScopeBoard.tsx +0 -454
- package/src/views/SessionTimeline.tsx +0 -516
- package/src/views/Settings.tsx +0 -183
- package/src/views/SourceControl.tsx +0 -95
- package/src/views/WorkflowVisualizer.tsx +0 -382
- package/tailwind.config.js +0 -161
- package/tsconfig.json +0 -25
- package/vite.config.ts +0 -49
|
@@ -0,0 +1,96 @@
|
|
|
1
|
+
import { describe, it, expect, afterEach } from 'vitest';
|
|
2
|
+
import { parseCcHooks } from './cc-hooks-parser.js';
|
|
3
|
+
import fs from 'fs';
|
|
4
|
+
import os from 'os';
|
|
5
|
+
import path from 'path';
|
|
6
|
+
|
|
7
|
+
describe('parseCcHooks', () => {
|
|
8
|
+
let tmpFile: string;
|
|
9
|
+
|
|
10
|
+
function writeSettings(data: Record<string, unknown>): string {
|
|
11
|
+
tmpFile = path.join(os.tmpdir(), `test-settings-${Date.now()}.json`);
|
|
12
|
+
fs.writeFileSync(tmpFile, JSON.stringify(data));
|
|
13
|
+
return tmpFile;
|
|
14
|
+
}
|
|
15
|
+
|
|
16
|
+
afterEach(() => {
|
|
17
|
+
if (tmpFile && fs.existsSync(tmpFile)) fs.unlinkSync(tmpFile);
|
|
18
|
+
});
|
|
19
|
+
|
|
20
|
+
it('parses all 4 event types', () => {
|
|
21
|
+
const file = writeSettings({
|
|
22
|
+
hooks: {
|
|
23
|
+
SessionStart: [{ hooks: [{ type: 'command', command: '$CLAUDE_PROJECT_DIR/.claude/hooks/init.sh', statusMessage: 'Initializing' }] }],
|
|
24
|
+
SessionEnd: [{ hooks: [{ type: 'command', command: '$CLAUDE_PROJECT_DIR/.claude/hooks/end.sh' }] }],
|
|
25
|
+
PreToolUse: [{ matcher: 'Edit', hooks: [{ type: 'command', command: '$CLAUDE_PROJECT_DIR/.claude/hooks/guard.sh', statusMessage: 'Checking' }] }],
|
|
26
|
+
PostToolUse: [{ hooks: [{ type: 'command', command: '$CLAUDE_PROJECT_DIR/.claude/hooks/log.sh' }] }],
|
|
27
|
+
},
|
|
28
|
+
});
|
|
29
|
+
const hooks = parseCcHooks(file);
|
|
30
|
+
expect(hooks).toHaveLength(4);
|
|
31
|
+
expect(hooks.map(h => h.event)).toEqual(['SessionStart', 'SessionEnd', 'PreToolUse', 'PostToolUse']);
|
|
32
|
+
});
|
|
33
|
+
|
|
34
|
+
it('extracts script path stripping $CLAUDE_PROJECT_DIR', () => {
|
|
35
|
+
const file = writeSettings({
|
|
36
|
+
hooks: {
|
|
37
|
+
SessionStart: [{ hooks: [{ type: 'command', command: '$CLAUDE_PROJECT_DIR/.claude/hooks/init-session.sh' }] }],
|
|
38
|
+
},
|
|
39
|
+
});
|
|
40
|
+
const hooks = parseCcHooks(file);
|
|
41
|
+
expect(hooks[0].scriptPath).toBe('.claude/hooks/init-session.sh');
|
|
42
|
+
expect(hooks[0].scriptName).toBe('init-session.sh');
|
|
43
|
+
});
|
|
44
|
+
|
|
45
|
+
it('derives ID from script filename', () => {
|
|
46
|
+
const file = writeSettings({
|
|
47
|
+
hooks: {
|
|
48
|
+
SessionStart: [{ hooks: [{ type: 'command', command: '$CLAUDE_PROJECT_DIR/.claude/hooks/init-session.sh' }] }],
|
|
49
|
+
},
|
|
50
|
+
});
|
|
51
|
+
const hooks = parseCcHooks(file);
|
|
52
|
+
expect(hooks[0].id).toBe('init-session');
|
|
53
|
+
});
|
|
54
|
+
|
|
55
|
+
it('extracts matcher from group', () => {
|
|
56
|
+
const file = writeSettings({
|
|
57
|
+
hooks: {
|
|
58
|
+
PreToolUse: [{ matcher: 'Edit', hooks: [{ type: 'command', command: 'test.sh' }] }],
|
|
59
|
+
},
|
|
60
|
+
});
|
|
61
|
+
const hooks = parseCcHooks(file);
|
|
62
|
+
expect(hooks[0].matcher).toBe('Edit');
|
|
63
|
+
});
|
|
64
|
+
|
|
65
|
+
it('sets matcher to null when absent', () => {
|
|
66
|
+
const file = writeSettings({
|
|
67
|
+
hooks: {
|
|
68
|
+
SessionStart: [{ hooks: [{ type: 'command', command: 'test.sh' }] }],
|
|
69
|
+
},
|
|
70
|
+
});
|
|
71
|
+
const hooks = parseCcHooks(file);
|
|
72
|
+
expect(hooks[0].matcher).toBeNull();
|
|
73
|
+
});
|
|
74
|
+
|
|
75
|
+
it('skips non-command entries', () => {
|
|
76
|
+
const file = writeSettings({
|
|
77
|
+
hooks: {
|
|
78
|
+
SessionStart: [{ hooks: [
|
|
79
|
+
{ type: 'command', command: 'test.sh' },
|
|
80
|
+
{ type: 'url', command: 'https://example.com' },
|
|
81
|
+
] }],
|
|
82
|
+
},
|
|
83
|
+
});
|
|
84
|
+
const hooks = parseCcHooks(file);
|
|
85
|
+
expect(hooks).toHaveLength(1);
|
|
86
|
+
});
|
|
87
|
+
|
|
88
|
+
it('returns empty array for file not found', () => {
|
|
89
|
+
expect(parseCcHooks('/tmp/nonexistent-settings.json')).toEqual([]);
|
|
90
|
+
});
|
|
91
|
+
|
|
92
|
+
it('returns empty array when no hooks key in settings', () => {
|
|
93
|
+
const file = writeSettings({ other: 'config' });
|
|
94
|
+
expect(parseCcHooks(file)).toEqual([]);
|
|
95
|
+
});
|
|
96
|
+
});
|
|
@@ -1,6 +1,9 @@
|
|
|
1
1
|
import { readFileSync } from 'node:fs';
|
|
2
|
+
import { createLogger } from './logger.js';
|
|
2
3
|
import type { CcHookEvent, CcHookParsed } from '../../shared/workflow-config.js';
|
|
3
4
|
|
|
5
|
+
const log = createLogger('config');
|
|
6
|
+
|
|
4
7
|
const CC_HOOK_EVENTS: CcHookEvent[] = ['SessionStart', 'SessionEnd', 'PreToolUse', 'PostToolUse'];
|
|
5
8
|
|
|
6
9
|
interface SettingsHookEntry {
|
|
@@ -36,6 +39,7 @@ export function parseCcHooks(settingsPath: string): CcHookParsed[] {
|
|
|
36
39
|
try {
|
|
37
40
|
raw = readFileSync(settingsPath, 'utf-8');
|
|
38
41
|
} catch {
|
|
42
|
+
log.debug('Settings file not found', { path: settingsPath });
|
|
39
43
|
return [];
|
|
40
44
|
}
|
|
41
45
|
|
|
@@ -0,0 +1,245 @@
|
|
|
1
|
+
import { describe, it, expect, beforeEach, afterEach, vi } from 'vitest';
|
|
2
|
+
import {
|
|
3
|
+
resolveDispatchEvent,
|
|
4
|
+
resolveActiveDispatchesForScope,
|
|
5
|
+
resolveAbandonedDispatchesForScope,
|
|
6
|
+
linkPidToDispatch,
|
|
7
|
+
resolveDispatchesByPid,
|
|
8
|
+
resolveDispatchesByDispatchId,
|
|
9
|
+
getActiveScopeIds,
|
|
10
|
+
getAbandonedScopeIds,
|
|
11
|
+
} from './dispatch-utils.js';
|
|
12
|
+
import { createTestDb } from '../__tests__/helpers/db.js';
|
|
13
|
+
import { createMockEmitter } from '../__tests__/helpers/mock-emitter.js';
|
|
14
|
+
import { WorkflowEngine } from '../../shared/workflow-engine.js';
|
|
15
|
+
import { CONFIG_WITH_HOOKS } from '../../shared/__fixtures__/workflow-configs.js';
|
|
16
|
+
import type Database from 'better-sqlite3';
|
|
17
|
+
import type { Emitter } from '../project-emitter.js';
|
|
18
|
+
|
|
19
|
+
// Mock isSessionPidAlive since it checks real OS processes
|
|
20
|
+
vi.mock('./terminal-launcher.js', () => ({
|
|
21
|
+
isSessionPidAlive: vi.fn().mockReturnValue(false),
|
|
22
|
+
launchInTerminal: vi.fn(),
|
|
23
|
+
buildSessionName: vi.fn(),
|
|
24
|
+
snapshotSessionPids: vi.fn().mockReturnValue([]),
|
|
25
|
+
discoverNewSession: vi.fn(),
|
|
26
|
+
renameSession: vi.fn(),
|
|
27
|
+
launchInCategorizedTerminal: vi.fn(),
|
|
28
|
+
}));
|
|
29
|
+
|
|
30
|
+
function insertDispatchEvent(db: Database.Database, overrides: Record<string, unknown> = {}) {
|
|
31
|
+
const id = overrides.id ?? `dispatch-${Date.now()}-${Math.random().toString(36).slice(2, 6)}`;
|
|
32
|
+
const data = JSON.stringify({
|
|
33
|
+
command: '/scope-implement 1',
|
|
34
|
+
transition: { from: 'backlog', to: 'active' },
|
|
35
|
+
...(overrides.data as Record<string, unknown> ?? {}),
|
|
36
|
+
});
|
|
37
|
+
db.prepare(
|
|
38
|
+
`INSERT INTO events (id, type, scope_id, session_id, agent, data, timestamp)
|
|
39
|
+
VALUES (?, 'DISPATCH', ?, ?, NULL, ?, ?)`
|
|
40
|
+
).run(
|
|
41
|
+
id,
|
|
42
|
+
overrides.scope_id ?? 1,
|
|
43
|
+
overrides.session_id ?? 'sess-1',
|
|
44
|
+
data,
|
|
45
|
+
overrides.timestamp ?? new Date().toISOString(),
|
|
46
|
+
);
|
|
47
|
+
return id as string;
|
|
48
|
+
}
|
|
49
|
+
|
|
50
|
+
describe('dispatch-utils', () => {
|
|
51
|
+
let db: Database.Database;
|
|
52
|
+
let cleanup: () => void;
|
|
53
|
+
let emitter: Emitter & { emit: ReturnType<typeof vi.fn> };
|
|
54
|
+
let engine: WorkflowEngine;
|
|
55
|
+
|
|
56
|
+
beforeEach(() => {
|
|
57
|
+
({ db, cleanup } = createTestDb());
|
|
58
|
+
emitter = createMockEmitter();
|
|
59
|
+
engine = new WorkflowEngine(CONFIG_WITH_HOOKS);
|
|
60
|
+
});
|
|
61
|
+
|
|
62
|
+
afterEach(() => {
|
|
63
|
+
cleanup?.();
|
|
64
|
+
});
|
|
65
|
+
|
|
66
|
+
// ─── resolveDispatchEvent() ───────────────────────────────
|
|
67
|
+
|
|
68
|
+
describe('resolveDispatchEvent()', () => {
|
|
69
|
+
it('marks dispatch as resolved with outcome', () => {
|
|
70
|
+
const id = insertDispatchEvent(db);
|
|
71
|
+
resolveDispatchEvent(db, emitter, id, 'completed');
|
|
72
|
+
|
|
73
|
+
const row = db.prepare('SELECT data FROM events WHERE id = ?').get(id) as { data: string };
|
|
74
|
+
const data = JSON.parse(row.data);
|
|
75
|
+
expect(data.resolved).toBeDefined();
|
|
76
|
+
expect(data.resolved.outcome).toBe('completed');
|
|
77
|
+
expect(data.resolved.at).toBeDefined();
|
|
78
|
+
});
|
|
79
|
+
|
|
80
|
+
it('emits dispatch:resolved event', () => {
|
|
81
|
+
const id = insertDispatchEvent(db);
|
|
82
|
+
resolveDispatchEvent(db, emitter, id, 'completed');
|
|
83
|
+
expect(emitter.emit).toHaveBeenCalledWith('dispatch:resolved', expect.objectContaining({ event_id: id }));
|
|
84
|
+
});
|
|
85
|
+
|
|
86
|
+
it('stores error message for failed dispatches', () => {
|
|
87
|
+
const id = insertDispatchEvent(db);
|
|
88
|
+
resolveDispatchEvent(db, emitter, id, 'failed', 'Timeout');
|
|
89
|
+
|
|
90
|
+
const row = db.prepare('SELECT data FROM events WHERE id = ?').get(id) as { data: string };
|
|
91
|
+
const data = JSON.parse(row.data);
|
|
92
|
+
expect(data.resolved.outcome).toBe('failed');
|
|
93
|
+
expect(data.resolved.error).toBe('Timeout');
|
|
94
|
+
});
|
|
95
|
+
});
|
|
96
|
+
|
|
97
|
+
// ─── resolveActiveDispatchesForScope() ────────────────────
|
|
98
|
+
|
|
99
|
+
describe('resolveActiveDispatchesForScope()', () => {
|
|
100
|
+
it('resolves all unresolved dispatches for a scope', () => {
|
|
101
|
+
insertDispatchEvent(db, { id: 'd1', scope_id: 42 });
|
|
102
|
+
insertDispatchEvent(db, { id: 'd2', scope_id: 42 });
|
|
103
|
+
insertDispatchEvent(db, { id: 'd3', scope_id: 99 }); // different scope
|
|
104
|
+
|
|
105
|
+
resolveActiveDispatchesForScope(db, emitter, 42, 'completed');
|
|
106
|
+
|
|
107
|
+
const rows = db.prepare("SELECT data FROM events WHERE scope_id = 42").all() as Array<{ data: string }>;
|
|
108
|
+
for (const row of rows) {
|
|
109
|
+
const data = JSON.parse(row.data);
|
|
110
|
+
expect(data.resolved).toBeDefined();
|
|
111
|
+
expect(data.resolved.outcome).toBe('completed');
|
|
112
|
+
}
|
|
113
|
+
|
|
114
|
+
// Scope 99 should NOT be resolved
|
|
115
|
+
const other = db.prepare("SELECT data FROM events WHERE scope_id = 99").get() as { data: string };
|
|
116
|
+
expect(JSON.parse(other.data).resolved).toBeUndefined();
|
|
117
|
+
});
|
|
118
|
+
|
|
119
|
+
it('handles no active dispatches gracefully', () => {
|
|
120
|
+
expect(() => resolveActiveDispatchesForScope(db, emitter, 999, 'completed')).not.toThrow();
|
|
121
|
+
});
|
|
122
|
+
});
|
|
123
|
+
|
|
124
|
+
// ─── resolveAbandonedDispatchesForScope() ─────────────────
|
|
125
|
+
|
|
126
|
+
describe('resolveAbandonedDispatchesForScope()', () => {
|
|
127
|
+
it('resolves abandoned dispatches as completed', () => {
|
|
128
|
+
insertDispatchEvent(db, { scope_id: 42, data: { resolved: true, outcome: 'abandoned' } });
|
|
129
|
+
const count = resolveAbandonedDispatchesForScope(db, emitter, 42);
|
|
130
|
+
expect(count).toBeGreaterThanOrEqual(0);
|
|
131
|
+
});
|
|
132
|
+
});
|
|
133
|
+
|
|
134
|
+
// ─── linkPidToDispatch() ──────────────────────────────────
|
|
135
|
+
|
|
136
|
+
describe('linkPidToDispatch()', () => {
|
|
137
|
+
it('stores PID in event data', () => {
|
|
138
|
+
const id = insertDispatchEvent(db);
|
|
139
|
+
linkPidToDispatch(db, id, 12345);
|
|
140
|
+
|
|
141
|
+
const row = db.prepare('SELECT data FROM events WHERE id = ?').get(id) as { data: string };
|
|
142
|
+
const data = JSON.parse(row.data);
|
|
143
|
+
expect(data.pid).toBe(12345);
|
|
144
|
+
});
|
|
145
|
+
|
|
146
|
+
it('handles non-existent event gracefully', () => {
|
|
147
|
+
expect(() => linkPidToDispatch(db, 'nonexistent', 12345)).not.toThrow();
|
|
148
|
+
});
|
|
149
|
+
});
|
|
150
|
+
|
|
151
|
+
// ─── resolveDispatchesByPid() ─────────────────────────────
|
|
152
|
+
|
|
153
|
+
describe('resolveDispatchesByPid()', () => {
|
|
154
|
+
it('resolves dispatches matching PID', () => {
|
|
155
|
+
const id = insertDispatchEvent(db, { scope_id: 1 });
|
|
156
|
+
linkPidToDispatch(db, id, 54321);
|
|
157
|
+
|
|
158
|
+
const ids = resolveDispatchesByPid(db, emitter, 54321);
|
|
159
|
+
expect(ids).toHaveLength(1);
|
|
160
|
+
expect(ids[0]).toBe(id);
|
|
161
|
+
|
|
162
|
+
const row = db.prepare('SELECT data FROM events WHERE id = ?').get(id) as { data: string };
|
|
163
|
+
expect(JSON.parse(row.data).resolved).toBeDefined();
|
|
164
|
+
});
|
|
165
|
+
|
|
166
|
+
it('returns empty array when no dispatches match PID', () => {
|
|
167
|
+
expect(resolveDispatchesByPid(db, emitter, 99999)).toHaveLength(0);
|
|
168
|
+
});
|
|
169
|
+
});
|
|
170
|
+
|
|
171
|
+
// ─── resolveDispatchesByDispatchId() ──────────────────────
|
|
172
|
+
|
|
173
|
+
describe('resolveDispatchesByDispatchId()', () => {
|
|
174
|
+
it('resolves single dispatch by event ID', () => {
|
|
175
|
+
const id = insertDispatchEvent(db);
|
|
176
|
+
const ids = resolveDispatchesByDispatchId(db, emitter, id);
|
|
177
|
+
expect(ids).toHaveLength(1);
|
|
178
|
+
expect(ids[0]).toBe(id);
|
|
179
|
+
});
|
|
180
|
+
|
|
181
|
+
it('returns empty array for non-existent dispatch', () => {
|
|
182
|
+
expect(resolveDispatchesByDispatchId(db, emitter, 'nonexistent')).toHaveLength(0);
|
|
183
|
+
});
|
|
184
|
+
});
|
|
185
|
+
|
|
186
|
+
// ─── getActiveScopeIds() ──────────────────────────────────
|
|
187
|
+
|
|
188
|
+
describe('getActiveScopeIds()', () => {
|
|
189
|
+
it('returns scope IDs with unresolved dispatches', () => {
|
|
190
|
+
insertDispatchEvent(db, { scope_id: 10 });
|
|
191
|
+
insertDispatchEvent(db, { scope_id: 20 });
|
|
192
|
+
|
|
193
|
+
const mockScopeService = {
|
|
194
|
+
getById: (id: number) => ({ id, status: 'active' }),
|
|
195
|
+
} as any;
|
|
196
|
+
|
|
197
|
+
const ids = getActiveScopeIds(db, mockScopeService, engine);
|
|
198
|
+
// May return empty if dispatches are stale (PID check fails), but shouldn't throw
|
|
199
|
+
expect(Array.isArray(ids)).toBe(true);
|
|
200
|
+
});
|
|
201
|
+
|
|
202
|
+
it('excludes scopes in terminal status', () => {
|
|
203
|
+
insertDispatchEvent(db, { scope_id: 30 });
|
|
204
|
+
const mockScopeService = {
|
|
205
|
+
getById: (id: number) => ({ id, status: 'shipped' }), // terminal
|
|
206
|
+
} as any;
|
|
207
|
+
|
|
208
|
+
const ids = getActiveScopeIds(db, mockScopeService, engine);
|
|
209
|
+
expect(ids).not.toContain(30);
|
|
210
|
+
});
|
|
211
|
+
});
|
|
212
|
+
|
|
213
|
+
// ─── getAbandonedScopeIds() ───────────────────────────────
|
|
214
|
+
|
|
215
|
+
describe('getAbandonedScopeIds()', () => {
|
|
216
|
+
it('returns recently abandoned scopes', () => {
|
|
217
|
+
insertDispatchEvent(db, {
|
|
218
|
+
scope_id: 50,
|
|
219
|
+
data: { resolved: true, outcome: 'abandoned', resolved_at: new Date().toISOString(), transition: { from: 'backlog', to: 'active' } },
|
|
220
|
+
});
|
|
221
|
+
|
|
222
|
+
const mockScopeService = {
|
|
223
|
+
getById: (id: number) => ({ id, status: 'active' }),
|
|
224
|
+
} as any;
|
|
225
|
+
|
|
226
|
+
const abandoned = getAbandonedScopeIds(db, mockScopeService, engine);
|
|
227
|
+
expect(Array.isArray(abandoned)).toBe(true);
|
|
228
|
+
});
|
|
229
|
+
|
|
230
|
+
it('excludes terminal scopes', () => {
|
|
231
|
+
insertDispatchEvent(db, {
|
|
232
|
+
scope_id: 60,
|
|
233
|
+
data: { resolved: true, outcome: 'abandoned', resolved_at: new Date().toISOString(), transition: { from: 'review', to: 'shipped' } },
|
|
234
|
+
});
|
|
235
|
+
|
|
236
|
+
const mockScopeService = {
|
|
237
|
+
getById: (id: number) => ({ id, status: 'shipped' }), // terminal
|
|
238
|
+
} as any;
|
|
239
|
+
|
|
240
|
+
const abandoned = getAbandonedScopeIds(db, mockScopeService, engine);
|
|
241
|
+
const ids = abandoned.map(a => a.scope_id);
|
|
242
|
+
expect(ids).not.toContain(60);
|
|
243
|
+
});
|
|
244
|
+
});
|
|
245
|
+
});
|
|
@@ -1,11 +1,11 @@
|
|
|
1
1
|
import type Database from 'better-sqlite3';
|
|
2
|
-
import type {
|
|
2
|
+
import type { Emitter } from '../project-emitter.js';
|
|
3
3
|
import type { ScopeService } from '../services/scope-service.js';
|
|
4
4
|
import type { WorkflowEngine } from '../../shared/workflow-engine.js';
|
|
5
5
|
import { isSessionPidAlive } from './terminal-launcher.js';
|
|
6
6
|
import { createLogger } from './logger.js';
|
|
7
7
|
|
|
8
|
-
const log = createLogger('dispatch');
|
|
8
|
+
const log = createLogger('dispatch-utils');
|
|
9
9
|
|
|
10
10
|
interface DispatchRow {
|
|
11
11
|
data: string;
|
|
@@ -15,7 +15,7 @@ interface DispatchRow {
|
|
|
15
15
|
/** Mark a DISPATCH event as resolved and emit socket notification. */
|
|
16
16
|
export function resolveDispatchEvent(
|
|
17
17
|
db: Database.Database,
|
|
18
|
-
io:
|
|
18
|
+
io: Emitter,
|
|
19
19
|
eventId: string,
|
|
20
20
|
outcome: 'completed' | 'failed' | 'abandoned',
|
|
21
21
|
error?: string,
|
|
@@ -42,10 +42,73 @@ export function resolveDispatchEvent(
|
|
|
42
42
|
});
|
|
43
43
|
}
|
|
44
44
|
|
|
45
|
+
/** Auto-revert scope status when a dispatch is abandoned, if the forward edge
|
|
46
|
+
* has autoRevert=true and the scope is still at the dispatch target.
|
|
47
|
+
* Safe: only reverts if the scope hasn't been moved since the dispatch.
|
|
48
|
+
* Returns true if revert was successful. */
|
|
49
|
+
function autoRevertAbandonedScope(
|
|
50
|
+
scopeService: ScopeService,
|
|
51
|
+
engine: WorkflowEngine,
|
|
52
|
+
scopeId: number,
|
|
53
|
+
data: Record<string, unknown>,
|
|
54
|
+
): boolean {
|
|
55
|
+
try {
|
|
56
|
+
const transition = data.transition as { from: string; to: string } | null;
|
|
57
|
+
if (!transition?.from || !transition?.to) return false;
|
|
58
|
+
|
|
59
|
+
const scope = scopeService.getById(scopeId);
|
|
60
|
+
// Only revert if scope is still at the dispatch target (hasn't been moved)
|
|
61
|
+
if (!scope || scope.status !== transition.to) return false;
|
|
62
|
+
|
|
63
|
+
const edge = engine.findEdge(transition.from, transition.to);
|
|
64
|
+
if (!edge?.autoRevert) return false;
|
|
65
|
+
|
|
66
|
+
const result = scopeService.updateStatus(scopeId, transition.from, 'rollback');
|
|
67
|
+
if (!result.ok) return false;
|
|
68
|
+
log.info('Auto-reverted abandoned dispatch', {
|
|
69
|
+
scopeId, from: transition.to, to: transition.from,
|
|
70
|
+
});
|
|
71
|
+
return true;
|
|
72
|
+
} catch (err) {
|
|
73
|
+
log.error('Auto-revert failed', { scopeId, error: String(err) });
|
|
74
|
+
return false;
|
|
75
|
+
}
|
|
76
|
+
}
|
|
77
|
+
|
|
78
|
+
/** Attempt auto-revert for an abandoned dispatch and clear the abandoned state if successful.
|
|
79
|
+
* Loads the dispatch event data, tries auto-revert, and re-resolves as 'completed' if the
|
|
80
|
+
* scope was successfully reverted. Returns true if auto-revert + clear succeeded. */
|
|
81
|
+
export function tryAutoRevertAndClear(
|
|
82
|
+
db: Database.Database,
|
|
83
|
+
io: Emitter,
|
|
84
|
+
scopeService: ScopeService,
|
|
85
|
+
engine: WorkflowEngine,
|
|
86
|
+
eventId: string,
|
|
87
|
+
): boolean {
|
|
88
|
+
const row = db.prepare('SELECT data, scope_id FROM events WHERE id = ?')
|
|
89
|
+
.get(eventId) as DispatchRow | undefined;
|
|
90
|
+
if (!row || row.scope_id == null) return false;
|
|
91
|
+
|
|
92
|
+
let data: Record<string, unknown>;
|
|
93
|
+
try {
|
|
94
|
+
data = JSON.parse(row.data);
|
|
95
|
+
} catch {
|
|
96
|
+
return false;
|
|
97
|
+
}
|
|
98
|
+
|
|
99
|
+
const reverted = autoRevertAbandonedScope(scopeService, engine, row.scope_id, data);
|
|
100
|
+
if (reverted) {
|
|
101
|
+
// Clear the abandoned state so getAbandonedScopeIds won't return this scope
|
|
102
|
+
resolveDispatchEvent(db, io, eventId, 'completed');
|
|
103
|
+
log.info('Cleared abandoned dispatch after auto-revert', { eventId, scope_id: row.scope_id });
|
|
104
|
+
}
|
|
105
|
+
return reverted;
|
|
106
|
+
}
|
|
107
|
+
|
|
45
108
|
/** Resolve all unresolved DISPATCH events for a given scope */
|
|
46
109
|
export function resolveActiveDispatchesForScope(
|
|
47
110
|
db: Database.Database,
|
|
48
|
-
io:
|
|
111
|
+
io: Emitter,
|
|
49
112
|
scopeId: number,
|
|
50
113
|
outcome: 'completed' | 'failed' | 'abandoned',
|
|
51
114
|
): void {
|
|
@@ -63,7 +126,7 @@ export function resolveActiveDispatchesForScope(
|
|
|
63
126
|
* Used by both recover and dismiss-abandoned routes to clear abandoned state. */
|
|
64
127
|
export function resolveAbandonedDispatchesForScope(
|
|
65
128
|
db: Database.Database,
|
|
66
|
-
io:
|
|
129
|
+
io: Emitter,
|
|
67
130
|
scopeId: number,
|
|
68
131
|
): number {
|
|
69
132
|
const rows = db.prepare(
|
|
@@ -104,15 +167,13 @@ export function linkPidToDispatch(
|
|
|
104
167
|
/** Resolve all unresolved DISPATCH events linked to a specific PID.
|
|
105
168
|
* Called when a SESSION_END event is received, indicating the Claude session
|
|
106
169
|
* process has exited and its dispatches should be cleared.
|
|
107
|
-
*
|
|
108
|
-
* NOTE: Does NOT revert scope status. Skills like /scope-implement intentionally
|
|
109
|
-
* keep scopes at the transition target (e.g. "implementing") after completion.
|
|
110
|
-
* Reverting on session end was destroying completed work and deleting scope files. */
|
|
170
|
+
* Returns the resolved event IDs so callers can attempt auto-revert. */
|
|
111
171
|
export function resolveDispatchesByPid(
|
|
112
172
|
db: Database.Database,
|
|
113
|
-
io:
|
|
173
|
+
io: Emitter,
|
|
114
174
|
pid: number,
|
|
115
|
-
|
|
175
|
+
outcome: 'completed' | 'abandoned' = 'abandoned',
|
|
176
|
+
): string[] {
|
|
116
177
|
const rows = db.prepare(
|
|
117
178
|
`SELECT id FROM events
|
|
118
179
|
WHERE type = 'DISPATCH'
|
|
@@ -121,33 +182,34 @@ export function resolveDispatchesByPid(
|
|
|
121
182
|
).all(pid) as Array<{ id: string }>;
|
|
122
183
|
|
|
123
184
|
for (const row of rows) {
|
|
124
|
-
resolveDispatchEvent(db, io, row.id,
|
|
185
|
+
resolveDispatchEvent(db, io, row.id, outcome);
|
|
125
186
|
}
|
|
126
187
|
|
|
127
|
-
return rows.
|
|
188
|
+
return rows.map(r => r.id);
|
|
128
189
|
}
|
|
129
190
|
|
|
130
191
|
/** Resolve all unresolved DISPATCH events linked to a specific dispatch ID.
|
|
131
192
|
* Called when a SESSION_END event includes dispatch_id from ORBITAL_DISPATCH_ID env var.
|
|
132
|
-
*
|
|
133
|
-
*
|
|
193
|
+
* Outcome depends on how the session ended: normal_exit → completed, otherwise → abandoned.
|
|
194
|
+
* Returns the resolved event IDs so callers can attempt auto-revert. */
|
|
134
195
|
export function resolveDispatchesByDispatchId(
|
|
135
196
|
db: Database.Database,
|
|
136
|
-
io:
|
|
197
|
+
io: Emitter,
|
|
137
198
|
dispatchId: string,
|
|
138
|
-
|
|
199
|
+
outcome: 'completed' | 'abandoned' = 'abandoned',
|
|
200
|
+
): string[] {
|
|
139
201
|
const row = db.prepare(
|
|
140
202
|
`SELECT id FROM events
|
|
141
203
|
WHERE id = ? AND type = 'DISPATCH' AND JSON_EXTRACT(data, '$.resolved') IS NULL`,
|
|
142
204
|
).get(dispatchId) as { id: string } | undefined;
|
|
143
205
|
|
|
144
|
-
if (!row) return
|
|
145
|
-
resolveDispatchEvent(db, io, row.id,
|
|
146
|
-
return
|
|
206
|
+
if (!row) return [];
|
|
207
|
+
resolveDispatchEvent(db, io, row.id, outcome);
|
|
208
|
+
return [row.id];
|
|
147
209
|
}
|
|
148
210
|
|
|
149
|
-
/** Fallback age threshold for dispatches without a linked PID (
|
|
150
|
-
const STALE_AGE_MS =
|
|
211
|
+
/** Fallback age threshold for dispatches without a linked PID (10 minutes). */
|
|
212
|
+
const STALE_AGE_MS = 10 * 60 * 1000;
|
|
151
213
|
|
|
152
214
|
/** Get all scope IDs that have actively running DISPATCH events.
|
|
153
215
|
* Uses PID liveness (process.kill(pid, 0)) when available, falls back to
|
|
@@ -209,6 +271,7 @@ export function getActiveScopeIds(db: Database.Database, scopeService: ScopeServ
|
|
|
209
271
|
try {
|
|
210
272
|
batchData = JSON.parse(batchRow.data);
|
|
211
273
|
} catch {
|
|
274
|
+
log.warn('Skipping unparseable batch dispatch event data', { data: batchRow.data });
|
|
212
275
|
continue;
|
|
213
276
|
}
|
|
214
277
|
|
|
@@ -242,11 +305,12 @@ export function getActiveScopeIds(db: Database.Database, scopeService: ScopeServ
|
|
|
242
305
|
* 3. No linked PID and dispatch older than STALE_AGE_MS (fallback)
|
|
243
306
|
* Called once at startup and periodically to clean up unresolved dispatches.
|
|
244
307
|
*
|
|
245
|
-
*
|
|
246
|
-
*
|
|
247
|
-
*
|
|
248
|
-
*
|
|
249
|
-
|
|
308
|
+
* When a dispatch is abandoned, auto-reverts scope status if the forward edge
|
|
309
|
+
* has autoRevert=true AND the scope is still at the dispatch target. This allows
|
|
310
|
+
* safe recovery for edges like backlog→implementing where the session crashed
|
|
311
|
+
* before doing meaningful work. Edges without autoRevert leave the scope in place
|
|
312
|
+
* for manual recovery from the dashboard. */
|
|
313
|
+
export function resolveStaleDispatches(db: Database.Database, io: Emitter, scopeService: ScopeService, engine: WorkflowEngine): number {
|
|
250
314
|
const cutoff = new Date(Date.now() - STALE_AGE_MS).toISOString();
|
|
251
315
|
|
|
252
316
|
// Single query on events only — split by cache status
|
|
@@ -288,6 +352,8 @@ export function resolveStaleDispatches(db: Database.Database, io: Server, scopeS
|
|
|
288
352
|
|
|
289
353
|
if (isStale) {
|
|
290
354
|
resolveDispatchEvent(db, io, row.id, 'abandoned');
|
|
355
|
+
// Try auto-revert; if successful, clear the abandoned state
|
|
356
|
+
tryAutoRevertAndClear(db, io, scopeService, engine, row.id);
|
|
291
357
|
resolved++;
|
|
292
358
|
}
|
|
293
359
|
}
|
|
@@ -306,6 +372,7 @@ export function resolveStaleDispatches(db: Database.Database, io: Server, scopeS
|
|
|
306
372
|
try {
|
|
307
373
|
batchData = JSON.parse(batchRow.data);
|
|
308
374
|
} catch {
|
|
375
|
+
log.warn('Skipping unparseable batch dispatch event data', { eventId: batchRow.id });
|
|
309
376
|
continue;
|
|
310
377
|
}
|
|
311
378
|
|
|
@@ -388,6 +455,9 @@ export function getAbandonedScopeIds(
|
|
|
388
455
|
const fromStatus = transition?.from as string ?? null;
|
|
389
456
|
const abandonedAt = resolved?.at as string ?? row.timestamp;
|
|
390
457
|
|
|
458
|
+
// Defense-in-depth: skip scopes already at their pre-dispatch status (already reverted)
|
|
459
|
+
if (fromStatus && scope.status === fromStatus) continue;
|
|
460
|
+
|
|
391
461
|
result.push({ scope_id: row.scope_id, from_status: fromStatus, abandoned_at: abandonedAt });
|
|
392
462
|
}
|
|
393
463
|
|
package/server/utils/logger.ts
CHANGED
|
@@ -33,8 +33,14 @@ const useColor = process.stdout.isTTY && !process.env.NO_COLOR;
|
|
|
33
33
|
const c = {
|
|
34
34
|
reset: useColor ? '\x1b[0m' : '',
|
|
35
35
|
dim: useColor ? '\x1b[2m' : '',
|
|
36
|
-
|
|
36
|
+
green: useColor ? '\x1b[32m' : '',
|
|
37
|
+
blue: useColor ? '\x1b[34m' : '',
|
|
38
|
+
magenta: useColor ? '\x1b[35m' : '',
|
|
37
39
|
cyan: useColor ? '\x1b[36m' : '',
|
|
40
|
+
white: useColor ? '\x1b[37m' : '',
|
|
41
|
+
gray: useColor ? '\x1b[90m' : '',
|
|
42
|
+
brightMagenta: useColor ? '\x1b[95m' : '',
|
|
43
|
+
brightCyan: useColor ? '\x1b[96m' : '',
|
|
38
44
|
yellow: useColor ? '\x1b[33m' : '',
|
|
39
45
|
red: useColor ? '\x1b[31m' : '',
|
|
40
46
|
};
|
|
@@ -53,6 +59,34 @@ const LEVEL_LABEL: Record<LogLevel, string> = {
|
|
|
53
59
|
error: 'ERROR',
|
|
54
60
|
};
|
|
55
61
|
|
|
62
|
+
// ─── Component Colors ───────────────────────────────────────
|
|
63
|
+
|
|
64
|
+
const COMPONENT_COLOR: Record<string, string> = {
|
|
65
|
+
// Scope (green)
|
|
66
|
+
'scope': c.green, 'scope-watcher': c.green,
|
|
67
|
+
// Dispatch (magenta)
|
|
68
|
+
'dispatch': c.magenta, 'dispatch-utils': c.magenta, 'batch': c.magenta,
|
|
69
|
+
// Git (blue)
|
|
70
|
+
'git': c.blue, 'worktree': c.blue,
|
|
71
|
+
// Workflow (cyan)
|
|
72
|
+
'workflow': c.cyan, 'sprint': c.cyan,
|
|
73
|
+
// Sync / Config (yellow)
|
|
74
|
+
'sync': c.yellow, 'config': c.yellow, 'manifest': c.yellow, 'global-config': c.yellow,
|
|
75
|
+
// Events (white)
|
|
76
|
+
'event': c.white, 'event-watcher': c.white, 'global-watcher': c.white,
|
|
77
|
+
// Infrastructure (white)
|
|
78
|
+
'server': c.white, 'central': c.white, 'database': c.white,
|
|
79
|
+
'project-context': c.white, 'project-manager': c.white, 'launch': c.white,
|
|
80
|
+
// Terminal (bright magenta)
|
|
81
|
+
'terminal': c.brightMagenta,
|
|
82
|
+
// Services (bright cyan)
|
|
83
|
+
'gate': c.brightCyan, 'deploy': c.brightCyan, 'telemetry': c.brightCyan, 'version': c.brightCyan,
|
|
84
|
+
};
|
|
85
|
+
|
|
86
|
+
function componentColor(name: string): string {
|
|
87
|
+
return COMPONENT_COLOR[name] ?? c.dim;
|
|
88
|
+
}
|
|
89
|
+
|
|
56
90
|
// ─── Formatting ─────────────────────────────────────────────
|
|
57
91
|
|
|
58
92
|
function timestamp(): string {
|
|
@@ -87,10 +121,13 @@ export interface Logger {
|
|
|
87
121
|
function write(level: LogLevel, component: string, msg: string, data?: Record<string, unknown>): void {
|
|
88
122
|
if (LEVEL_VALUE[level] < LEVEL_VALUE[currentLevel]) return;
|
|
89
123
|
|
|
90
|
-
const
|
|
124
|
+
const lvl = LEVEL_COLOR[level];
|
|
91
125
|
const label = LEVEL_LABEL[level];
|
|
92
126
|
const kv = formatData(data);
|
|
93
|
-
const
|
|
127
|
+
const cc = componentColor(component);
|
|
128
|
+
// Message uses level color for warn/error (urgency), component color otherwise
|
|
129
|
+
const mc = (level === 'warn' || level === 'error') ? lvl : cc;
|
|
130
|
+
const line = `${c.dim}${timestamp()}${c.reset} ${lvl}${label}${c.reset} ${cc}[${component}]${c.reset} ${mc}${msg}${c.reset}${c.dim}${kv}${c.reset}\n`;
|
|
94
131
|
|
|
95
132
|
if (level === 'warn' || level === 'error') {
|
|
96
133
|
process.stderr.write(line);
|
|
@@ -0,0 +1,32 @@
|
|
|
1
|
+
import path from 'path';
|
|
2
|
+
import fs from 'fs';
|
|
3
|
+
import { fileURLToPath } from 'url';
|
|
4
|
+
|
|
5
|
+
const __selfDir = path.dirname(fileURLToPath(import.meta.url));
|
|
6
|
+
|
|
7
|
+
/**
|
|
8
|
+
* Resolve the root directory of the orbital-command package itself.
|
|
9
|
+
* Walks up from the current file until it finds package.json.
|
|
10
|
+
*/
|
|
11
|
+
export function getOrbitalRoot(): string {
|
|
12
|
+
let dir = __selfDir;
|
|
13
|
+
for (let i = 0; i < 6; i++) {
|
|
14
|
+
if (fs.existsSync(path.join(dir, 'package.json'))) {
|
|
15
|
+
return dir;
|
|
16
|
+
}
|
|
17
|
+
dir = path.dirname(dir);
|
|
18
|
+
}
|
|
19
|
+
// Fallback: assume dev layout (server/utils/ → 2 levels up)
|
|
20
|
+
return path.resolve(__selfDir, '../..');
|
|
21
|
+
}
|
|
22
|
+
|
|
23
|
+
/** Read package version from package.json in the given root, or the orbital root. */
|
|
24
|
+
export function getPackageVersion(rootDir?: string): string {
|
|
25
|
+
try {
|
|
26
|
+
const dir = rootDir ?? getOrbitalRoot();
|
|
27
|
+
const pkg = JSON.parse(fs.readFileSync(path.join(dir, 'package.json'), 'utf-8'));
|
|
28
|
+
return pkg.version || '0.0.0';
|
|
29
|
+
} catch {
|
|
30
|
+
return '0.0.0';
|
|
31
|
+
}
|
|
32
|
+
}
|