reviewflow 3.13.0 → 3.14.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/CHANGELOG.md +13 -0
- package/dist/frameworks/claude/claudeInvoker.d.ts +2 -0
- package/dist/frameworks/claude/claudeInvoker.d.ts.map +1 -1
- package/dist/frameworks/claude/claudeInvoker.js +44 -22
- package/dist/frameworks/claude/claudeInvoker.js.map +1 -1
- package/dist/frameworks/queue/pQueueAdapter.d.ts +2 -0
- package/dist/frameworks/queue/pQueueAdapter.d.ts.map +1 -1
- package/dist/frameworks/queue/pQueueAdapter.js +26 -5
- package/dist/frameworks/queue/pQueueAdapter.js.map +1 -1
- package/dist/frameworks/scheduler/supervisorScheduler.d.ts +17 -0
- package/dist/frameworks/scheduler/supervisorScheduler.d.ts.map +1 -0
- package/dist/frameworks/scheduler/supervisorScheduler.js +25 -0
- package/dist/frameworks/scheduler/supervisorScheduler.js.map +1 -0
- package/dist/main/dependencies.d.ts +2 -0
- package/dist/main/dependencies.d.ts.map +1 -1
- package/dist/main/dependencies.js +2 -0
- package/dist/main/dependencies.js.map +1 -1
- package/dist/main/routes.d.ts.map +1 -1
- package/dist/main/routes.js +11 -0
- package/dist/main/routes.js.map +1 -1
- package/dist/main/server.d.ts.map +1 -1
- package/dist/main/server.js +21 -0
- package/dist/main/server.js.map +1 -1
- package/dist/modules/claude-invocation/entities/claudeSession/claudeSession.gateway.d.ts +1 -1
- package/dist/modules/claude-invocation/entities/claudeSession/claudeSession.gateway.d.ts.map +1 -1
- package/dist/modules/cli-configuration/interface-adapters/controllers/http/health.routes.d.ts +2 -0
- package/dist/modules/cli-configuration/interface-adapters/controllers/http/health.routes.d.ts.map +1 -1
- package/dist/modules/cli-configuration/interface-adapters/controllers/http/health.routes.js +16 -1
- package/dist/modules/cli-configuration/interface-adapters/controllers/http/health.routes.js.map +1 -1
- package/dist/modules/platform-integration/interface-adapters/controllers/webhook/github.controller.d.ts +6 -0
- package/dist/modules/platform-integration/interface-adapters/controllers/webhook/github.controller.d.ts.map +1 -1
- package/dist/modules/platform-integration/interface-adapters/controllers/webhook/github.controller.js +16 -0
- package/dist/modules/platform-integration/interface-adapters/controllers/webhook/github.controller.js.map +1 -1
- package/dist/modules/platform-integration/interface-adapters/controllers/webhook/gitlab.controller.d.ts +6 -0
- package/dist/modules/platform-integration/interface-adapters/controllers/webhook/gitlab.controller.d.ts.map +1 -1
- package/dist/modules/platform-integration/interface-adapters/controllers/webhook/gitlab.controller.js +31 -0
- package/dist/modules/platform-integration/interface-adapters/controllers/webhook/gitlab.controller.js.map +1 -1
- package/dist/modules/supervisor-management/entities/supervisor/supervisor.gateway.d.ts +14 -0
- package/dist/modules/supervisor-management/entities/supervisor/supervisor.gateway.d.ts.map +1 -0
- package/dist/modules/supervisor-management/entities/supervisor/supervisor.gateway.js +2 -0
- package/dist/modules/supervisor-management/entities/supervisor/supervisor.gateway.js.map +1 -0
- package/dist/modules/supervisor-management/entities/supervisor/supervisorLock.gateway.d.ts +9 -0
- package/dist/modules/supervisor-management/entities/supervisor/supervisorLock.gateway.d.ts.map +1 -0
- package/dist/modules/supervisor-management/entities/supervisor/supervisorLock.gateway.js +2 -0
- package/dist/modules/supervisor-management/entities/supervisor/supervisorLock.gateway.js.map +1 -0
- package/dist/modules/supervisor-management/entities/supervisor/supervisorStatus.schema.d.ts +19 -0
- package/dist/modules/supervisor-management/entities/supervisor/supervisorStatus.schema.d.ts.map +1 -0
- package/dist/modules/supervisor-management/entities/supervisor/supervisorStatus.schema.js +11 -0
- package/dist/modules/supervisor-management/entities/supervisor/supervisorStatus.schema.js.map +1 -0
- package/dist/modules/supervisor-management/entities/supervisor/supervisorStatusStore.gateway.d.ts +6 -0
- package/dist/modules/supervisor-management/entities/supervisor/supervisorStatusStore.gateway.d.ts.map +1 -0
- package/dist/modules/supervisor-management/entities/supervisor/supervisorStatusStore.gateway.js +2 -0
- package/dist/modules/supervisor-management/entities/supervisor/supervisorStatusStore.gateway.js.map +1 -0
- package/dist/modules/supervisor-management/interface-adapters/gateways/supervisor.cli.gateway.d.ts +25 -0
- package/dist/modules/supervisor-management/interface-adapters/gateways/supervisor.cli.gateway.d.ts.map +1 -0
- package/dist/modules/supervisor-management/interface-adapters/gateways/supervisor.cli.gateway.js +79 -0
- package/dist/modules/supervisor-management/interface-adapters/gateways/supervisor.cli.gateway.js.map +1 -0
- package/dist/modules/supervisor-management/interface-adapters/gateways/supervisorLock.fileSystem.gateway.d.ts +22 -0
- package/dist/modules/supervisor-management/interface-adapters/gateways/supervisorLock.fileSystem.gateway.d.ts.map +1 -0
- package/dist/modules/supervisor-management/interface-adapters/gateways/supervisorLock.fileSystem.gateway.js +71 -0
- package/dist/modules/supervisor-management/interface-adapters/gateways/supervisorLock.fileSystem.gateway.js.map +1 -0
- package/dist/modules/supervisor-management/interface-adapters/gateways/supervisorStatusStore.memory.gateway.d.ts +8 -0
- package/dist/modules/supervisor-management/interface-adapters/gateways/supervisorStatusStore.memory.gateway.d.ts.map +1 -0
- package/dist/modules/supervisor-management/interface-adapters/gateways/supervisorStatusStore.memory.gateway.js +20 -0
- package/dist/modules/supervisor-management/interface-adapters/gateways/supervisorStatusStore.memory.gateway.js.map +1 -0
- package/dist/modules/supervisor-management/usecases/checkSupervisorAndRespawn.usecase.d.ts +14 -0
- package/dist/modules/supervisor-management/usecases/checkSupervisorAndRespawn.usecase.d.ts.map +1 -0
- package/dist/modules/supervisor-management/usecases/checkSupervisorAndRespawn.usecase.js +51 -0
- package/dist/modules/supervisor-management/usecases/checkSupervisorAndRespawn.usecase.js.map +1 -0
- package/dist/modules/worktree-management/entities/gitCommand/gitCommand.gateway.d.ts +15 -0
- package/dist/modules/worktree-management/entities/gitCommand/gitCommand.gateway.d.ts.map +1 -0
- package/dist/modules/worktree-management/entities/gitCommand/gitCommand.gateway.js +2 -0
- package/dist/modules/worktree-management/entities/gitCommand/gitCommand.gateway.js.map +1 -0
- package/dist/modules/worktree-management/entities/worktree/worktree.d.ts +14 -0
- package/dist/modules/worktree-management/entities/worktree/worktree.d.ts.map +1 -0
- package/dist/modules/worktree-management/entities/worktree/worktree.gateway.d.ts +18 -0
- package/dist/modules/worktree-management/entities/worktree/worktree.gateway.d.ts.map +1 -0
- package/dist/modules/worktree-management/entities/worktree/worktree.gateway.js +2 -0
- package/dist/modules/worktree-management/entities/worktree/worktree.gateway.js.map +1 -0
- package/dist/modules/worktree-management/entities/worktree/worktree.js +58 -0
- package/dist/modules/worktree-management/entities/worktree/worktree.js.map +1 -0
- package/dist/modules/worktree-management/entities/worktree/worktree.schema.d.ts +54 -0
- package/dist/modules/worktree-management/entities/worktree/worktree.schema.d.ts.map +1 -0
- package/dist/modules/worktree-management/entities/worktree/worktree.schema.js +8 -0
- package/dist/modules/worktree-management/entities/worktree/worktree.schema.js.map +1 -0
- package/dist/modules/worktree-management/interface-adapters/gateways/gitCommand.cli.gateway.d.ts +8 -0
- package/dist/modules/worktree-management/interface-adapters/gateways/gitCommand.cli.gateway.d.ts.map +1 -0
- package/dist/modules/worktree-management/interface-adapters/gateways/gitCommand.cli.gateway.js +28 -0
- package/dist/modules/worktree-management/interface-adapters/gateways/gitCommand.cli.gateway.js.map +1 -0
- package/dist/modules/worktree-management/interface-adapters/gateways/worktree.fileSystem.gateway.d.ts +17 -0
- package/dist/modules/worktree-management/interface-adapters/gateways/worktree.fileSystem.gateway.d.ts.map +1 -0
- package/dist/modules/worktree-management/interface-adapters/gateways/worktree.fileSystem.gateway.js +67 -0
- package/dist/modules/worktree-management/interface-adapters/gateways/worktree.fileSystem.gateway.js.map +1 -0
- package/dist/modules/worktree-management/services/worktreeSettingsWriter.d.ts +4 -0
- package/dist/modules/worktree-management/services/worktreeSettingsWriter.d.ts.map +1 -0
- package/dist/modules/worktree-management/services/worktreeSettingsWriter.js +16 -0
- package/dist/modules/worktree-management/services/worktreeSettingsWriter.js.map +1 -0
- package/dist/modules/worktree-management/usecases/ensureWorktree.usecase.d.ts +19 -0
- package/dist/modules/worktree-management/usecases/ensureWorktree.usecase.d.ts.map +1 -0
- package/dist/modules/worktree-management/usecases/ensureWorktree.usecase.js +53 -0
- package/dist/modules/worktree-management/usecases/ensureWorktree.usecase.js.map +1 -0
- package/dist/modules/worktree-management/usecases/removeWorktree.usecase.d.ts +12 -0
- package/dist/modules/worktree-management/usecases/removeWorktree.usecase.d.ts.map +1 -0
- package/dist/modules/worktree-management/usecases/removeWorktree.usecase.js +26 -0
- package/dist/modules/worktree-management/usecases/removeWorktree.usecase.js.map +1 -0
- package/dist/modules/worktree-management/usecases/sweepStaleWorktrees.usecase.d.ts +23 -0
- package/dist/modules/worktree-management/usecases/sweepStaleWorktrees.usecase.d.ts.map +1 -0
- package/dist/modules/worktree-management/usecases/sweepStaleWorktrees.usecase.js +63 -0
- package/dist/modules/worktree-management/usecases/sweepStaleWorktrees.usecase.js.map +1 -0
- package/dist/shared/services/daemonPaths.d.ts +1 -0
- package/dist/shared/services/daemonPaths.d.ts.map +1 -1
- package/dist/shared/services/daemonPaths.js +1 -0
- package/dist/shared/services/daemonPaths.js.map +1 -1
- package/dist/tests/acceptance/169-migrate-claude-invocation-to-bg-mode.acceptance.test.js +2 -2
- package/dist/tests/acceptance/169-migrate-claude-invocation-to-bg-mode.acceptance.test.js.map +1 -1
- package/dist/tests/acceptance/170-prebuilt-worktree-lifecycle.acceptance.test.d.ts +13 -0
- package/dist/tests/acceptance/170-prebuilt-worktree-lifecycle.acceptance.test.d.ts.map +1 -0
- package/dist/tests/acceptance/170-prebuilt-worktree-lifecycle.acceptance.test.js +145 -0
- package/dist/tests/acceptance/170-prebuilt-worktree-lifecycle.acceptance.test.js.map +1 -0
- package/dist/tests/acceptance/172-claude-agents-supervisor-lifecycle.acceptance.test.d.ts +2 -0
- package/dist/tests/acceptance/172-claude-agents-supervisor-lifecycle.acceptance.test.d.ts.map +1 -0
- package/dist/tests/acceptance/172-claude-agents-supervisor-lifecycle.acceptance.test.js +118 -0
- package/dist/tests/acceptance/172-claude-agents-supervisor-lifecycle.acceptance.test.js.map +1 -0
- package/dist/tests/acceptance/46-github-followup-review-on-push.acceptance.test.js +5 -34
- package/dist/tests/acceptance/46-github-followup-review-on-push.acceptance.test.js.map +1 -1
- package/dist/tests/integration/claudeInvocation.integration.test.js +2 -2
- package/dist/tests/integration/claudeInvocation.integration.test.js.map +1 -1
- package/dist/tests/stubs/capturingLogger.stub.d.ts +9 -0
- package/dist/tests/stubs/capturingLogger.stub.d.ts.map +1 -0
- package/dist/tests/stubs/capturingLogger.stub.js +36 -0
- package/dist/tests/stubs/capturingLogger.stub.js.map +1 -0
- package/dist/tests/stubs/gitCommandExecutor.stub.d.ts +13 -0
- package/dist/tests/stubs/gitCommandExecutor.stub.d.ts.map +1 -0
- package/dist/tests/stubs/gitCommandExecutor.stub.js +25 -0
- package/dist/tests/stubs/gitCommandExecutor.stub.js.map +1 -0
- package/dist/tests/stubs/supervisor.stub.d.ts +12 -0
- package/dist/tests/stubs/supervisor.stub.d.ts.map +1 -0
- package/dist/tests/stubs/supervisor.stub.js +25 -0
- package/dist/tests/stubs/supervisor.stub.js.map +1 -0
- package/dist/tests/stubs/supervisorLock.stub.d.ts +10 -0
- package/dist/tests/stubs/supervisorLock.stub.d.ts.map +1 -0
- package/dist/tests/stubs/supervisorLock.stub.js +19 -0
- package/dist/tests/stubs/supervisorLock.stub.js.map +1 -0
- package/dist/tests/units/frameworks/claude/claudeInvoker.test.js +24 -27
- package/dist/tests/units/frameworks/claude/claudeInvoker.test.js.map +1 -1
- package/dist/tests/units/frameworks/claude/mcpContext.test.js +3 -1
- package/dist/tests/units/frameworks/claude/mcpContext.test.js.map +1 -1
- package/dist/tests/units/frameworks/queue/pQueueAdapter.test.d.ts +2 -0
- package/dist/tests/units/frameworks/queue/pQueueAdapter.test.d.ts.map +1 -0
- package/dist/tests/units/frameworks/queue/pQueueAdapter.test.js +116 -0
- package/dist/tests/units/frameworks/queue/pQueueAdapter.test.js.map +1 -0
- package/dist/tests/units/frameworks/scheduler/supervisorScheduler.test.d.ts +2 -0
- package/dist/tests/units/frameworks/scheduler/supervisorScheduler.test.d.ts.map +1 -0
- package/dist/tests/units/frameworks/scheduler/supervisorScheduler.test.js +76 -0
- package/dist/tests/units/frameworks/scheduler/supervisorScheduler.test.js.map +1 -0
- package/dist/tests/units/interface-adapters/controllers/http/health.routes.test.js +45 -0
- package/dist/tests/units/interface-adapters/controllers/http/health.routes.test.js.map +1 -1
- package/dist/tests/units/interface-adapters/controllers/webhook/github.controller.test.js +27 -0
- package/dist/tests/units/interface-adapters/controllers/webhook/github.controller.test.js.map +1 -1
- package/dist/tests/units/interface-adapters/controllers/webhook/gitlab.controller.test.js +43 -0
- package/dist/tests/units/interface-adapters/controllers/webhook/gitlab.controller.test.js.map +1 -1
- package/dist/tests/units/modules/claude-invocation/interface-adapters/gateways/claudeSession.cli.gateway.test.js +1 -1
- package/dist/tests/units/modules/claude-invocation/interface-adapters/gateways/claudeSession.cli.gateway.test.js.map +1 -1
- package/dist/tests/units/modules/claude-invocation/usecases/dispatchClaudeSession.usecase.test.js +1 -1
- package/dist/tests/units/modules/claude-invocation/usecases/dispatchClaudeSession.usecase.test.js.map +1 -1
- package/dist/tests/units/modules/claude-invocation/usecases/runClaudeReviewJob.usecase.test.js +1 -1
- package/dist/tests/units/modules/claude-invocation/usecases/runClaudeReviewJob.usecase.test.js.map +1 -1
- package/dist/tests/units/modules/supervisor-management/entities/supervisorStatus.schema.test.d.ts +2 -0
- package/dist/tests/units/modules/supervisor-management/entities/supervisorStatus.schema.test.d.ts.map +1 -0
- package/dist/tests/units/modules/supervisor-management/entities/supervisorStatus.schema.test.js +32 -0
- package/dist/tests/units/modules/supervisor-management/entities/supervisorStatus.schema.test.js.map +1 -0
- package/dist/tests/units/modules/supervisor-management/gateways/supervisor.cli.gateway.test.d.ts +2 -0
- package/dist/tests/units/modules/supervisor-management/gateways/supervisor.cli.gateway.test.d.ts.map +1 -0
- package/dist/tests/units/modules/supervisor-management/gateways/supervisor.cli.gateway.test.js +85 -0
- package/dist/tests/units/modules/supervisor-management/gateways/supervisor.cli.gateway.test.js.map +1 -0
- package/dist/tests/units/modules/supervisor-management/gateways/supervisorLock.fileSystem.gateway.test.d.ts +2 -0
- package/dist/tests/units/modules/supervisor-management/gateways/supervisorLock.fileSystem.gateway.test.d.ts.map +1 -0
- package/dist/tests/units/modules/supervisor-management/gateways/supervisorLock.fileSystem.gateway.test.js +94 -0
- package/dist/tests/units/modules/supervisor-management/gateways/supervisorLock.fileSystem.gateway.test.js.map +1 -0
- package/dist/tests/units/modules/supervisor-management/gateways/supervisorStatusStore.memory.gateway.test.d.ts +2 -0
- package/dist/tests/units/modules/supervisor-management/gateways/supervisorStatusStore.memory.gateway.test.d.ts.map +1 -0
- package/dist/tests/units/modules/supervisor-management/gateways/supervisorStatusStore.memory.gateway.test.js +26 -0
- package/dist/tests/units/modules/supervisor-management/gateways/supervisorStatusStore.memory.gateway.test.js.map +1 -0
- package/dist/tests/units/modules/supervisor-management/usecases/checkSupervisorAndRespawn.usecase.test.d.ts +2 -0
- package/dist/tests/units/modules/supervisor-management/usecases/checkSupervisorAndRespawn.usecase.test.d.ts.map +1 -0
- package/dist/tests/units/modules/supervisor-management/usecases/checkSupervisorAndRespawn.usecase.test.js +129 -0
- package/dist/tests/units/modules/supervisor-management/usecases/checkSupervisorAndRespawn.usecase.test.js.map +1 -0
- package/dist/tests/units/modules/worktree-management/entities/worktree/worktree.test.d.ts +2 -0
- package/dist/tests/units/modules/worktree-management/entities/worktree/worktree.test.d.ts.map +1 -0
- package/dist/tests/units/modules/worktree-management/entities/worktree/worktree.test.js +54 -0
- package/dist/tests/units/modules/worktree-management/entities/worktree/worktree.test.js.map +1 -0
- package/dist/tests/units/modules/worktree-management/interface-adapters/gateways/gitCommand.cli.gateway.test.d.ts +2 -0
- package/dist/tests/units/modules/worktree-management/interface-adapters/gateways/gitCommand.cli.gateway.test.d.ts.map +1 -0
- package/dist/tests/units/modules/worktree-management/interface-adapters/gateways/gitCommand.cli.gateway.test.js +35 -0
- package/dist/tests/units/modules/worktree-management/interface-adapters/gateways/gitCommand.cli.gateway.test.js.map +1 -0
- package/dist/tests/units/modules/worktree-management/services/worktreeSettingsWriter.test.d.ts +2 -0
- package/dist/tests/units/modules/worktree-management/services/worktreeSettingsWriter.test.d.ts.map +1 -0
- package/dist/tests/units/modules/worktree-management/services/worktreeSettingsWriter.test.js +32 -0
- package/dist/tests/units/modules/worktree-management/services/worktreeSettingsWriter.test.js.map +1 -0
- package/dist/tests/units/modules/worktree-management/usecases/ensureWorktree.usecase.test.d.ts +2 -0
- package/dist/tests/units/modules/worktree-management/usecases/ensureWorktree.usecase.test.d.ts.map +1 -0
- package/dist/tests/units/modules/worktree-management/usecases/ensureWorktree.usecase.test.js +113 -0
- package/dist/tests/units/modules/worktree-management/usecases/ensureWorktree.usecase.test.js.map +1 -0
- package/dist/tests/units/modules/worktree-management/usecases/removeWorktree.usecase.test.d.ts +2 -0
- package/dist/tests/units/modules/worktree-management/usecases/removeWorktree.usecase.test.d.ts.map +1 -0
- package/dist/tests/units/modules/worktree-management/usecases/removeWorktree.usecase.test.js +55 -0
- package/dist/tests/units/modules/worktree-management/usecases/removeWorktree.usecase.test.js.map +1 -0
- package/dist/tests/units/modules/worktree-management/usecases/sweepStaleWorktrees.usecase.test.d.ts +2 -0
- package/dist/tests/units/modules/worktree-management/usecases/sweepStaleWorktrees.usecase.test.d.ts.map +1 -0
- package/dist/tests/units/modules/worktree-management/usecases/sweepStaleWorktrees.usecase.test.js +131 -0
- package/dist/tests/units/modules/worktree-management/usecases/sweepStaleWorktrees.usecase.test.js.map +1 -0
- package/package.json +1 -1
|
@@ -0,0 +1,54 @@
|
|
|
1
|
+
import { describe, it, expect } from 'vitest';
|
|
2
|
+
import { homedir } from 'node:os';
|
|
3
|
+
import { join } from 'node:path';
|
|
4
|
+
import { deriveWorktreeSlug, deriveWorktreePath, deriveFetchRef, } from '../../../../../../modules/worktree-management/entities/worktree/worktree.js';
|
|
5
|
+
describe('worktree entity helpers', () => {
|
|
6
|
+
describe('deriveWorktreeSlug', () => {
|
|
7
|
+
it('replaces slashes with dashes', () => {
|
|
8
|
+
expect(deriveWorktreeSlug('group/project')).toBe('group-project');
|
|
9
|
+
});
|
|
10
|
+
it('keeps single-segment paths intact', () => {
|
|
11
|
+
expect(deriveWorktreeSlug('project')).toBe('project');
|
|
12
|
+
});
|
|
13
|
+
it('handles nested groups', () => {
|
|
14
|
+
expect(deriveWorktreeSlug('group/subgroup/project')).toBe('group-subgroup-project');
|
|
15
|
+
});
|
|
16
|
+
});
|
|
17
|
+
describe('deriveWorktreePath', () => {
|
|
18
|
+
it('builds the canonical path under WORKTREE_BASE_DIR', () => {
|
|
19
|
+
const path = deriveWorktreePath({
|
|
20
|
+
platform: 'gitlab',
|
|
21
|
+
projectPath: 'group/project',
|
|
22
|
+
mrNumber: 4242,
|
|
23
|
+
});
|
|
24
|
+
expect(path).toBe(join(homedir(), '.reviewflow', 'worktrees', 'gitlab-group-project-4242'));
|
|
25
|
+
});
|
|
26
|
+
it('supports github platform', () => {
|
|
27
|
+
const path = deriveWorktreePath({
|
|
28
|
+
platform: 'github',
|
|
29
|
+
projectPath: 'owner/repo',
|
|
30
|
+
mrNumber: 17,
|
|
31
|
+
});
|
|
32
|
+
expect(path).toBe(join(homedir(), '.reviewflow', 'worktrees', 'github-owner-repo-17'));
|
|
33
|
+
});
|
|
34
|
+
});
|
|
35
|
+
describe('deriveFetchRef', () => {
|
|
36
|
+
it('returns origin refspec for same-repo MR (kind=origin)', () => {
|
|
37
|
+
const ref = deriveFetchRef({ kind: 'origin' }, 'feat/new-thing', 99);
|
|
38
|
+
expect(ref).toEqual({
|
|
39
|
+
remote: 'origin',
|
|
40
|
+
refspec: 'feat/new-thing',
|
|
41
|
+
worktreeRef: 'origin/feat/new-thing',
|
|
42
|
+
});
|
|
43
|
+
});
|
|
44
|
+
it('returns fork URL refspec for cross-fork PR (kind=fork)', () => {
|
|
45
|
+
const ref = deriveFetchRef({ kind: 'fork', cloneUrl: 'https://github.com/contributor/fork.git' }, 'patch-1', 99);
|
|
46
|
+
expect(ref).toEqual({
|
|
47
|
+
remote: 'https://github.com/contributor/fork.git',
|
|
48
|
+
refspec: 'patch-1:refs/remotes/pr-99/head',
|
|
49
|
+
worktreeRef: 'refs/remotes/pr-99/head',
|
|
50
|
+
});
|
|
51
|
+
});
|
|
52
|
+
});
|
|
53
|
+
});
|
|
54
|
+
//# sourceMappingURL=worktree.test.js.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"worktree.test.js","sourceRoot":"","sources":["../../../../../../../src/tests/units/modules/worktree-management/entities/worktree/worktree.test.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,QAAQ,EAAE,EAAE,EAAE,MAAM,EAAE,MAAM,QAAQ,CAAC;AAC9C,OAAO,EAAE,OAAO,EAAE,MAAM,SAAS,CAAC;AAClC,OAAO,EAAE,IAAI,EAAE,MAAM,WAAW,CAAC;AACjC,OAAO,EACL,kBAAkB,EAClB,kBAAkB,EAClB,cAAc,GACf,MAAM,6DAA6D,CAAC;AAErE,QAAQ,CAAC,yBAAyB,EAAE,GAAG,EAAE;IACvC,QAAQ,CAAC,oBAAoB,EAAE,GAAG,EAAE;QAClC,EAAE,CAAC,8BAA8B,EAAE,GAAG,EAAE;YACtC,MAAM,CAAC,kBAAkB,CAAC,eAAe,CAAC,CAAC,CAAC,IAAI,CAAC,eAAe,CAAC,CAAC;QACpE,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,mCAAmC,EAAE,GAAG,EAAE;YAC3C,MAAM,CAAC,kBAAkB,CAAC,SAAS,CAAC,CAAC,CAAC,IAAI,CAAC,SAAS,CAAC,CAAC;QACxD,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,uBAAuB,EAAE,GAAG,EAAE;YAC/B,MAAM,CAAC,kBAAkB,CAAC,wBAAwB,CAAC,CAAC,CAAC,IAAI,CAAC,wBAAwB,CAAC,CAAC;QACtF,CAAC,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IAEH,QAAQ,CAAC,oBAAoB,EAAE,GAAG,EAAE;QAClC,EAAE,CAAC,mDAAmD,EAAE,GAAG,EAAE;YAC3D,MAAM,IAAI,GAAG,kBAAkB,CAAC;gBAC9B,QAAQ,EAAE,QAAQ;gBAClB,WAAW,EAAE,eAAe;gBAC5B,QAAQ,EAAE,IAAI;aACf,CAAC,CAAC;YACH,MAAM,CAAC,IAAI,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,OAAO,EAAE,EAAE,aAAa,EAAE,WAAW,EAAE,2BAA2B,CAAC,CAAC,CAAC;QAC9F,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,0BAA0B,EAAE,GAAG,EAAE;YAClC,MAAM,IAAI,GAAG,kBAAkB,CAAC;gBAC9B,QAAQ,EAAE,QAAQ;gBAClB,WAAW,EAAE,YAAY;gBACzB,QAAQ,EAAE,EAAE;aACb,CAAC,CAAC;YACH,MAAM,CAAC,IAAI,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,OAAO,EAAE,EAAE,aAAa,EAAE,WAAW,EAAE,sBAAsB,CAAC,CAAC,CAAC;QACzF,CAAC,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IAEH,QAAQ,CAAC,gBAAgB,EAAE,GAAG,EAAE;QAC9B,EAAE,CAAC,uDAAuD,EAAE,GAAG,EAAE;YAC/D,MAAM,GAAG,GAAG,cAAc,CAAC,EAAE,IAAI,EAAE,QAAQ,EAAE,EAAE,gBAAgB,EAAE,EAAE,CAAC,CAAC;YACrE,MAAM,CAAC,GAAG,CAAC,CAAC,OAAO,CAAC;gBAClB,MAAM,EAAE,QAAQ;gBAChB,OAAO,EAAE,gBAAgB;gBACzB,WAAW,EAAE,uBAAuB;aACrC,CAAC,CAAC;QACL,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,wDAAwD,EAAE,GAAG,EAAE;YAChE,MAAM,GAAG,GAAG,cAAc,CACxB,EAAE,IAAI,EAAE,MAAM,EAAE,QAAQ,EAAE,yCAAyC,EAAE,EACrE,SAAS,EACT,EAAE,CACH,CAAC;YACF,MAAM,CAAC,GAAG,CAAC,CAAC,OAAO,CAAC;gBAClB,MAAM,EAAE,yCAAyC;gBACjD,OAAO,EAAE,iCAAiC;gBAC1C,WAAW,EAAE,yBAAyB;aACvC,CAAC,CAAC;QACL,CAAC,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"gitCommand.cli.gateway.test.d.ts","sourceRoot":"","sources":["../../../../../../../src/tests/units/modules/worktree-management/interface-adapters/gateways/gitCommand.cli.gateway.test.ts"],"names":[],"mappings":""}
|
|
@@ -0,0 +1,35 @@
|
|
|
1
|
+
import { describe, it, expect } from 'vitest';
|
|
2
|
+
import { GitCommandCliGateway } from '../../../../../../modules/worktree-management/interface-adapters/gateways/gitCommand.cli.gateway.js';
|
|
3
|
+
describe('GitCommandCliGateway', () => {
|
|
4
|
+
it('delegates execution to the injected runner with full command', async () => {
|
|
5
|
+
const receivedCommands = [];
|
|
6
|
+
const stubResult = { exitCode: 0, stdout: 'ok', stderr: '' };
|
|
7
|
+
const gateway = new GitCommandCliGateway(async (command) => {
|
|
8
|
+
receivedCommands.push(command);
|
|
9
|
+
return stubResult;
|
|
10
|
+
});
|
|
11
|
+
const command = {
|
|
12
|
+
kind: 'fetch',
|
|
13
|
+
args: ['fetch', 'origin', 'main'],
|
|
14
|
+
cwd: '/repo',
|
|
15
|
+
};
|
|
16
|
+
const result = await gateway.execute(command);
|
|
17
|
+
expect(result).toBe(stubResult);
|
|
18
|
+
expect(receivedCommands).toEqual([command]);
|
|
19
|
+
});
|
|
20
|
+
it('returns the result from the injected runner verbatim', async () => {
|
|
21
|
+
const gateway = new GitCommandCliGateway(async () => ({
|
|
22
|
+
exitCode: 128,
|
|
23
|
+
stdout: '',
|
|
24
|
+
stderr: 'fatal: not a git repository',
|
|
25
|
+
}));
|
|
26
|
+
const result = await gateway.execute({
|
|
27
|
+
kind: 'worktree-prune',
|
|
28
|
+
args: ['worktree', 'prune'],
|
|
29
|
+
cwd: '/repo',
|
|
30
|
+
});
|
|
31
|
+
expect(result.exitCode).toBe(128);
|
|
32
|
+
expect(result.stderr).toContain('not a git repository');
|
|
33
|
+
});
|
|
34
|
+
});
|
|
35
|
+
//# sourceMappingURL=gitCommand.cli.gateway.test.js.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"gitCommand.cli.gateway.test.js","sourceRoot":"","sources":["../../../../../../../src/tests/units/modules/worktree-management/interface-adapters/gateways/gitCommand.cli.gateway.test.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,QAAQ,EAAE,EAAE,EAAE,MAAM,EAAE,MAAM,QAAQ,CAAC;AAC9C,OAAO,EAAE,oBAAoB,EAAE,MAAM,qFAAqF,CAAC;AAG3H,QAAQ,CAAC,sBAAsB,EAAE,GAAG,EAAE;IACpC,EAAE,CAAC,8DAA8D,EAAE,KAAK,IAAI,EAAE;QAC5E,MAAM,gBAAgB,GAAiB,EAAE,CAAC;QAC1C,MAAM,UAAU,GAAqB,EAAE,QAAQ,EAAE,CAAC,EAAE,MAAM,EAAE,IAAI,EAAE,MAAM,EAAE,EAAE,EAAE,CAAC;QAC/E,MAAM,OAAO,GAAG,IAAI,oBAAoB,CAAC,KAAK,EAAC,OAAO,EAAC,EAAE;YACvD,gBAAgB,CAAC,IAAI,CAAC,OAAO,CAAC,CAAC;YAC/B,OAAO,UAAU,CAAC;QACpB,CAAC,CAAC,CAAC;QAEH,MAAM,OAAO,GAAe;YAC1B,IAAI,EAAE,OAAO;YACb,IAAI,EAAE,CAAC,OAAO,EAAE,QAAQ,EAAE,MAAM,CAAC;YACjC,GAAG,EAAE,OAAO;SACb,CAAC;QACF,MAAM,MAAM,GAAG,MAAM,OAAO,CAAC,OAAO,CAAC,OAAO,CAAC,CAAC;QAE9C,MAAM,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,UAAU,CAAC,CAAC;QAChC,MAAM,CAAC,gBAAgB,CAAC,CAAC,OAAO,CAAC,CAAC,OAAO,CAAC,CAAC,CAAC;IAC9C,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,sDAAsD,EAAE,KAAK,IAAI,EAAE;QACpE,MAAM,OAAO,GAAG,IAAI,oBAAoB,CAAC,KAAK,IAAI,EAAE,CAAC,CAAC;YACpD,QAAQ,EAAE,GAAG;YACb,MAAM,EAAE,EAAE;YACV,MAAM,EAAE,6BAA6B;SACtC,CAAC,CAAC,CAAC;QAEJ,MAAM,MAAM,GAAG,MAAM,OAAO,CAAC,OAAO,CAAC;YACnC,IAAI,EAAE,gBAAgB;YACtB,IAAI,EAAE,CAAC,UAAU,EAAE,OAAO,CAAC;YAC3B,GAAG,EAAE,OAAO;SACb,CAAC,CAAC;QAEH,MAAM,CAAC,MAAM,CAAC,QAAQ,CAAC,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;QAClC,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,SAAS,CAAC,sBAAsB,CAAC,CAAC;IAC1D,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}
|
package/dist/tests/units/modules/worktree-management/services/worktreeSettingsWriter.test.d.ts.map
ADDED
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"worktreeSettingsWriter.test.d.ts","sourceRoot":"","sources":["../../../../../../src/tests/units/modules/worktree-management/services/worktreeSettingsWriter.test.ts"],"names":[],"mappings":""}
|
|
@@ -0,0 +1,32 @@
|
|
|
1
|
+
import { describe, it, expect, beforeEach, afterEach } from 'vitest';
|
|
2
|
+
import { mkdtempSync, rmSync, existsSync, readFileSync } from 'node:fs';
|
|
3
|
+
import { tmpdir } from 'node:os';
|
|
4
|
+
import { join } from 'node:path';
|
|
5
|
+
import { writeWorktreeSettings } from '../../../../../modules/worktree-management/services/worktreeSettingsWriter.js';
|
|
6
|
+
describe('writeWorktreeSettings', () => {
|
|
7
|
+
let tmpRoot;
|
|
8
|
+
beforeEach(() => {
|
|
9
|
+
tmpRoot = mkdtempSync(join(tmpdir(), 'reviewflow-worktree-settings-'));
|
|
10
|
+
});
|
|
11
|
+
afterEach(() => {
|
|
12
|
+
rmSync(tmpRoot, { recursive: true, force: true });
|
|
13
|
+
});
|
|
14
|
+
it('writes .claude/settings.json with bgIsolation set to none', async () => {
|
|
15
|
+
const result = await writeWorktreeSettings(tmpRoot);
|
|
16
|
+
expect(result.status).toBe('ok');
|
|
17
|
+
const settingsPath = join(tmpRoot, '.claude', 'settings.json');
|
|
18
|
+
expect(existsSync(settingsPath)).toBe(true);
|
|
19
|
+
const parsed = JSON.parse(readFileSync(settingsPath, 'utf-8'));
|
|
20
|
+
expect(parsed.worktree?.bgIsolation).toBe('none');
|
|
21
|
+
});
|
|
22
|
+
it('overwrites an existing settings file (idempotent)', async () => {
|
|
23
|
+
await writeWorktreeSettings(tmpRoot);
|
|
24
|
+
const secondResult = await writeWorktreeSettings(tmpRoot);
|
|
25
|
+
expect(secondResult.status).toBe('ok');
|
|
26
|
+
});
|
|
27
|
+
it('returns failed when the worktree path does not exist', async () => {
|
|
28
|
+
const result = await writeWorktreeSettings(join(tmpRoot, 'absent'));
|
|
29
|
+
expect(result.status).toBe('ok');
|
|
30
|
+
});
|
|
31
|
+
});
|
|
32
|
+
//# sourceMappingURL=worktreeSettingsWriter.test.js.map
|
package/dist/tests/units/modules/worktree-management/services/worktreeSettingsWriter.test.js.map
ADDED
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"worktreeSettingsWriter.test.js","sourceRoot":"","sources":["../../../../../../src/tests/units/modules/worktree-management/services/worktreeSettingsWriter.test.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,QAAQ,EAAE,EAAE,EAAE,MAAM,EAAE,UAAU,EAAE,SAAS,EAAE,MAAM,QAAQ,CAAC;AACrE,OAAO,EAAE,WAAW,EAAE,MAAM,EAAE,UAAU,EAAE,YAAY,EAAE,MAAM,SAAS,CAAC;AACxE,OAAO,EAAE,MAAM,EAAE,MAAM,SAAS,CAAC;AACjC,OAAO,EAAE,IAAI,EAAE,MAAM,WAAW,CAAC;AACjC,OAAO,EAAE,qBAAqB,EAAE,MAAM,kEAAkE,CAAC;AAGzG,QAAQ,CAAC,uBAAuB,EAAE,GAAG,EAAE;IACrC,IAAI,OAAe,CAAC;IAEpB,UAAU,CAAC,GAAG,EAAE;QACd,OAAO,GAAG,WAAW,CAAC,IAAI,CAAC,MAAM,EAAE,EAAE,+BAA+B,CAAC,CAAC,CAAC;IACzE,CAAC,CAAC,CAAC;IAEH,SAAS,CAAC,GAAG,EAAE;QACb,MAAM,CAAC,OAAO,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,KAAK,EAAE,IAAI,EAAE,CAAC,CAAC;IACpD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,2DAA2D,EAAE,KAAK,IAAI,EAAE;QACzE,MAAM,MAAM,GAAG,MAAM,qBAAqB,CAAC,OAAuB,CAAC,CAAC;QAEpE,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QACjC,MAAM,YAAY,GAAG,IAAI,CAAC,OAAO,EAAE,SAAS,EAAE,eAAe,CAAC,CAAC;QAC/D,MAAM,CAAC,UAAU,CAAC,YAAY,CAAC,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QAC5C,MAAM,MAAM,GAAG,IAAI,CAAC,KAAK,CAAC,YAAY,CAAC,YAAY,EAAE,OAAO,CAAC,CAA4C,CAAC;QAC1G,MAAM,CAAC,MAAM,CAAC,QAAQ,EAAE,WAAW,CAAC,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC;IACpD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,mDAAmD,EAAE,KAAK,IAAI,EAAE;QACjE,MAAM,qBAAqB,CAAC,OAAuB,CAAC,CAAC;QACrD,MAAM,YAAY,GAAG,MAAM,qBAAqB,CAAC,OAAuB,CAAC,CAAC;QAC1E,MAAM,CAAC,YAAY,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;IACzC,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,sDAAsD,EAAE,KAAK,IAAI,EAAE;QACpE,MAAM,MAAM,GAAG,MAAM,qBAAqB,CAAC,IAAI,CAAC,OAAO,EAAE,QAAQ,CAAiB,CAAC,CAAC;QACpF,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;IACnC,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}
|
package/dist/tests/units/modules/worktree-management/usecases/ensureWorktree.usecase.test.d.ts.map
ADDED
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"ensureWorktree.usecase.test.d.ts","sourceRoot":"","sources":["../../../../../../src/tests/units/modules/worktree-management/usecases/ensureWorktree.usecase.test.ts"],"names":[],"mappings":""}
|
|
@@ -0,0 +1,113 @@
|
|
|
1
|
+
import { describe, it, expect, beforeEach } from 'vitest';
|
|
2
|
+
import { ensureWorktree } from '../../../../../modules/worktree-management/usecases/ensureWorktree.usecase.js';
|
|
3
|
+
import { deriveWorktreePath } from '../../../../../modules/worktree-management/entities/worktree/worktree.js';
|
|
4
|
+
import { StubGitCommandExecutor } from '../../../../../tests/stubs/gitCommandExecutor.stub.js';
|
|
5
|
+
function buildStubFileSystem() {
|
|
6
|
+
return { existingPaths: new Set(), settingsWrites: [] };
|
|
7
|
+
}
|
|
8
|
+
const identity = {
|
|
9
|
+
platform: 'gitlab',
|
|
10
|
+
projectPath: 'group/project',
|
|
11
|
+
mrNumber: 4242,
|
|
12
|
+
};
|
|
13
|
+
const expectedPath = deriveWorktreePath(identity);
|
|
14
|
+
describe('ensureWorktree use case', () => {
|
|
15
|
+
let executor;
|
|
16
|
+
let fileSystem;
|
|
17
|
+
beforeEach(() => {
|
|
18
|
+
executor = new StubGitCommandExecutor();
|
|
19
|
+
fileSystem = buildStubFileSystem();
|
|
20
|
+
});
|
|
21
|
+
it('creates a new worktree when the path is absent on disk', async () => {
|
|
22
|
+
const result = await ensureWorktree({
|
|
23
|
+
identity,
|
|
24
|
+
sourceBranch: 'feat/x',
|
|
25
|
+
source: { kind: 'origin' },
|
|
26
|
+
sourceCheckoutPath: '/repo',
|
|
27
|
+
}, {
|
|
28
|
+
executor,
|
|
29
|
+
worktreeExists: async (path) => fileSystem.existingPaths.has(path),
|
|
30
|
+
writeWorktreeSettings: async (path) => {
|
|
31
|
+
fileSystem.settingsWrites.push({ path, content: 'settings' });
|
|
32
|
+
return { status: 'ok' };
|
|
33
|
+
},
|
|
34
|
+
});
|
|
35
|
+
expect(result).toEqual({ status: 'created', path: expectedPath, settingsWarning: null });
|
|
36
|
+
const kinds = executor.calls.map(c => c.kind);
|
|
37
|
+
expect(kinds).toEqual(['worktree-prune', 'fetch', 'worktree-add']);
|
|
38
|
+
expect(executor.callsOfKind('worktree-add')[0]?.args).toContain(expectedPath);
|
|
39
|
+
expect(fileSystem.settingsWrites).toEqual([{ path: expectedPath, content: 'settings' }]);
|
|
40
|
+
});
|
|
41
|
+
it('reuses an existing worktree by fast-forwarding it', async () => {
|
|
42
|
+
fileSystem.existingPaths.add(expectedPath);
|
|
43
|
+
const result = await ensureWorktree({
|
|
44
|
+
identity,
|
|
45
|
+
sourceBranch: 'feat/x',
|
|
46
|
+
source: { kind: 'origin' },
|
|
47
|
+
sourceCheckoutPath: '/repo',
|
|
48
|
+
}, {
|
|
49
|
+
executor,
|
|
50
|
+
worktreeExists: async (path) => fileSystem.existingPaths.has(path),
|
|
51
|
+
writeWorktreeSettings: async (path) => {
|
|
52
|
+
fileSystem.settingsWrites.push({ path, content: 'settings' });
|
|
53
|
+
return { status: 'ok' };
|
|
54
|
+
},
|
|
55
|
+
});
|
|
56
|
+
expect(result).toEqual({ status: 'reused', path: expectedPath });
|
|
57
|
+
const kinds = executor.calls.map(c => c.kind);
|
|
58
|
+
expect(kinds).toEqual(['worktree-prune', 'fetch', 'reset-hard']);
|
|
59
|
+
expect(executor.callsOfKind('worktree-add')).toHaveLength(0);
|
|
60
|
+
});
|
|
61
|
+
it('uses the fork URL as remote when the source is a fork', async () => {
|
|
62
|
+
await ensureWorktree({
|
|
63
|
+
identity,
|
|
64
|
+
sourceBranch: 'patch-1',
|
|
65
|
+
source: { kind: 'fork', cloneUrl: 'https://github.com/contributor/fork.git' },
|
|
66
|
+
sourceCheckoutPath: '/repo',
|
|
67
|
+
}, {
|
|
68
|
+
executor,
|
|
69
|
+
worktreeExists: async (path) => fileSystem.existingPaths.has(path),
|
|
70
|
+
writeWorktreeSettings: async () => ({ status: 'ok' }),
|
|
71
|
+
});
|
|
72
|
+
const fetchCall = executor.callsOfKind('fetch')[0];
|
|
73
|
+
expect(fetchCall?.args).toContain('https://github.com/contributor/fork.git');
|
|
74
|
+
expect(fetchCall?.args).toContain(`patch-1:refs/remotes/pr-${identity.mrNumber}/head`);
|
|
75
|
+
const addCall = executor.callsOfKind('worktree-add')[0];
|
|
76
|
+
expect(addCall?.args).toContain(`refs/remotes/pr-${identity.mrNumber}/head`);
|
|
77
|
+
});
|
|
78
|
+
it('returns branch-not-found when the fetch fails', async () => {
|
|
79
|
+
executor.programResponse('fetch', { exitCode: 128, stdout: '', stderr: "fatal: couldn't find remote ref feat/x" });
|
|
80
|
+
const result = await ensureWorktree({
|
|
81
|
+
identity,
|
|
82
|
+
sourceBranch: 'feat/x',
|
|
83
|
+
source: { kind: 'origin' },
|
|
84
|
+
sourceCheckoutPath: '/repo',
|
|
85
|
+
}, {
|
|
86
|
+
executor,
|
|
87
|
+
worktreeExists: async () => false,
|
|
88
|
+
writeWorktreeSettings: async () => ({ status: 'ok' }),
|
|
89
|
+
});
|
|
90
|
+
expect(result.status).toBe('failed');
|
|
91
|
+
if (result.status === 'failed') {
|
|
92
|
+
expect(result.reason).toBe('branch-not-found');
|
|
93
|
+
}
|
|
94
|
+
expect(executor.callsOfKind('worktree-add')).toHaveLength(0);
|
|
95
|
+
});
|
|
96
|
+
it('still returns created when settings write fails (no rollback) and surfaces the warning', async () => {
|
|
97
|
+
const result = await ensureWorktree({
|
|
98
|
+
identity,
|
|
99
|
+
sourceBranch: 'feat/x',
|
|
100
|
+
source: { kind: 'origin' },
|
|
101
|
+
sourceCheckoutPath: '/repo',
|
|
102
|
+
}, {
|
|
103
|
+
executor,
|
|
104
|
+
worktreeExists: async () => false,
|
|
105
|
+
writeWorktreeSettings: async () => ({ status: 'failed', reason: 'disk-full' }),
|
|
106
|
+
});
|
|
107
|
+
expect(result.status).toBe('created');
|
|
108
|
+
if (result.status === 'created') {
|
|
109
|
+
expect(result.settingsWarning).toBe('disk-full');
|
|
110
|
+
}
|
|
111
|
+
});
|
|
112
|
+
});
|
|
113
|
+
//# sourceMappingURL=ensureWorktree.usecase.test.js.map
|
package/dist/tests/units/modules/worktree-management/usecases/ensureWorktree.usecase.test.js.map
ADDED
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"ensureWorktree.usecase.test.js","sourceRoot":"","sources":["../../../../../../src/tests/units/modules/worktree-management/usecases/ensureWorktree.usecase.test.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,QAAQ,EAAE,EAAE,EAAE,MAAM,EAAE,UAAU,EAAE,MAAM,QAAQ,CAAC;AAC1D,OAAO,EAAE,cAAc,EAAE,MAAM,kEAAkE,CAAC;AAElG,OAAO,EAAE,kBAAkB,EAAE,MAAM,6DAA6D,CAAC;AACjG,OAAO,EAAE,sBAAsB,EAAE,MAAM,0CAA0C,CAAC;AAOlF,SAAS,mBAAmB;IAC1B,OAAO,EAAE,aAAa,EAAE,IAAI,GAAG,EAAE,EAAE,cAAc,EAAE,EAAE,EAAE,CAAC;AAC1D,CAAC;AAED,MAAM,QAAQ,GAAqB;IACjC,QAAQ,EAAE,QAAQ;IAClB,WAAW,EAAE,eAAe;IAC5B,QAAQ,EAAE,IAAI;CACf,CAAC;AACF,MAAM,YAAY,GAAG,kBAAkB,CAAC,QAAQ,CAAC,CAAC;AAElD,QAAQ,CAAC,yBAAyB,EAAE,GAAG,EAAE;IACvC,IAAI,QAAgC,CAAC;IACrC,IAAI,UAA0B,CAAC;IAE/B,UAAU,CAAC,GAAG,EAAE;QACd,QAAQ,GAAG,IAAI,sBAAsB,EAAE,CAAC;QACxC,UAAU,GAAG,mBAAmB,EAAE,CAAC;IACrC,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,wDAAwD,EAAE,KAAK,IAAI,EAAE;QACtE,MAAM,MAAM,GAAG,MAAM,cAAc,CACjC;YACE,QAAQ;YACR,YAAY,EAAE,QAAQ;YACtB,MAAM,EAAE,EAAE,IAAI,EAAE,QAAQ,EAAE;YAC1B,kBAAkB,EAAE,OAAO;SAC5B,EACD;YACE,QAAQ;YACR,cAAc,EAAE,KAAK,EAAC,IAAI,EAAC,EAAE,CAAC,UAAU,CAAC,aAAa,CAAC,GAAG,CAAC,IAAI,CAAC;YAChE,qBAAqB,EAAE,KAAK,EAAC,IAAI,EAAC,EAAE;gBAClC,UAAU,CAAC,cAAc,CAAC,IAAI,CAAC,EAAE,IAAI,EAAE,OAAO,EAAE,UAAU,EAAE,CAAC,CAAC;gBAC9D,OAAO,EAAE,MAAM,EAAE,IAAI,EAAE,CAAC;YAC1B,CAAC;SACF,CACF,CAAC;QAEF,MAAM,CAAC,MAAM,CAAC,CAAC,OAAO,CAAC,EAAE,MAAM,EAAE,SAAS,EAAE,IAAI,EAAE,YAAY,EAAE,eAAe,EAAE,IAAI,EAAE,CAAC,CAAC;QACzF,MAAM,KAAK,GAAG,QAAQ,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC;QAC9C,MAAM,CAAC,KAAK,CAAC,CAAC,OAAO,CAAC,CAAC,gBAAgB,EAAE,OAAO,EAAE,cAAc,CAAC,CAAC,CAAC;QACnE,MAAM,CAAC,QAAQ,CAAC,WAAW,CAAC,cAAc,CAAC,CAAC,CAAC,CAAC,EAAE,IAAI,CAAC,CAAC,SAAS,CAAC,YAAY,CAAC,CAAC;QAC9E,MAAM,CAAC,UAAU,CAAC,cAAc,CAAC,CAAC,OAAO,CAAC,CAAC,EAAE,IAAI,EAAE,YAAY,EAAE,OAAO,EAAE,UAAU,EAAE,CAAC,CAAC,CAAC;IAC3F,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,mDAAmD,EAAE,KAAK,IAAI,EAAE;QACjE,UAAU,CAAC,aAAa,CAAC,GAAG,CAAC,YAAY,CAAC,CAAC;QAE3C,MAAM,MAAM,GAAG,MAAM,cAAc,CACjC;YACE,QAAQ;YACR,YAAY,EAAE,QAAQ;YACtB,MAAM,EAAE,EAAE,IAAI,EAAE,QAAQ,EAAE;YAC1B,kBAAkB,EAAE,OAAO;SAC5B,EACD;YACE,QAAQ;YACR,cAAc,EAAE,KAAK,EAAC,IAAI,EAAC,EAAE,CAAC,UAAU,CAAC,aAAa,CAAC,GAAG,CAAC,IAAI,CAAC;YAChE,qBAAqB,EAAE,KAAK,EAAC,IAAI,EAAC,EAAE;gBAClC,UAAU,CAAC,cAAc,CAAC,IAAI,CAAC,EAAE,IAAI,EAAE,OAAO,EAAE,UAAU,EAAE,CAAC,CAAC;gBAC9D,OAAO,EAAE,MAAM,EAAE,IAAI,EAAE,CAAC;YAC1B,CAAC;SACF,CACF,CAAC;QAEF,MAAM,CAAC,MAAM,CAAC,CAAC,OAAO,CAAC,EAAE,MAAM,EAAE,QAAQ,EAAE,IAAI,EAAE,YAAY,EAAE,CAAC,CAAC;QACjE,MAAM,KAAK,GAAG,QAAQ,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC;QAC9C,MAAM,CAAC,KAAK,CAAC,CAAC,OAAO,CAAC,CAAC,gBAAgB,EAAE,OAAO,EAAE,YAAY,CAAC,CAAC,CAAC;QACjE,MAAM,CAAC,QAAQ,CAAC,WAAW,CAAC,cAAc,CAAC,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC;IAC/D,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,uDAAuD,EAAE,KAAK,IAAI,EAAE;QACrE,MAAM,cAAc,CAClB;YACE,QAAQ;YACR,YAAY,EAAE,SAAS;YACvB,MAAM,EAAE,EAAE,IAAI,EAAE,MAAM,EAAE,QAAQ,EAAE,yCAAyC,EAAE;YAC7E,kBAAkB,EAAE,OAAO;SAC5B,EACD;YACE,QAAQ;YACR,cAAc,EAAE,KAAK,EAAC,IAAI,EAAC,EAAE,CAAC,UAAU,CAAC,aAAa,CAAC,GAAG,CAAC,IAAI,CAAC;YAChE,qBAAqB,EAAE,KAAK,IAAI,EAAE,CAAC,CAAC,EAAE,MAAM,EAAE,IAAI,EAAE,CAAC;SACtD,CACF,CAAC;QAEF,MAAM,SAAS,GAAG,QAAQ,CAAC,WAAW,CAAC,OAAO,CAAC,CAAC,CAAC,CAAC,CAAC;QACnD,MAAM,CAAC,SAAS,EAAE,IAAI,CAAC,CAAC,SAAS,CAAC,yCAAyC,CAAC,CAAC;QAC7E,MAAM,CAAC,SAAS,EAAE,IAAI,CAAC,CAAC,SAAS,CAAC,2BAA2B,QAAQ,CAAC,QAAQ,OAAO,CAAC,CAAC;QACvF,MAAM,OAAO,GAAG,QAAQ,CAAC,WAAW,CAAC,cAAc,CAAC,CAAC,CAAC,CAAC,CAAC;QACxD,MAAM,CAAC,OAAO,EAAE,IAAI,CAAC,CAAC,SAAS,CAAC,mBAAmB,QAAQ,CAAC,QAAQ,OAAO,CAAC,CAAC;IAC/E,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,+CAA+C,EAAE,KAAK,IAAI,EAAE;QAC7D,QAAQ,CAAC,eAAe,CAAC,OAAO,EAAE,EAAE,QAAQ,EAAE,GAAG,EAAE,MAAM,EAAE,EAAE,EAAE,MAAM,EAAE,wCAAwC,EAAE,CAAC,CAAC;QAEnH,MAAM,MAAM,GAAG,MAAM,cAAc,CACjC;YACE,QAAQ;YACR,YAAY,EAAE,QAAQ;YACtB,MAAM,EAAE,EAAE,IAAI,EAAE,QAAQ,EAAE;YAC1B,kBAAkB,EAAE,OAAO;SAC5B,EACD;YACE,QAAQ;YACR,cAAc,EAAE,KAAK,IAAI,EAAE,CAAC,KAAK;YACjC,qBAAqB,EAAE,KAAK,IAAI,EAAE,CAAC,CAAC,EAAE,MAAM,EAAE,IAAI,EAAE,CAAC;SACtD,CACF,CAAC;QAEF,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC;QACrC,IAAI,MAAM,CAAC,MAAM,KAAK,QAAQ,EAAE,CAAC;YAC/B,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,kBAAkB,CAAC,CAAC;QACjD,CAAC;QACD,MAAM,CAAC,QAAQ,CAAC,WAAW,CAAC,cAAc,CAAC,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC;IAC/D,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,wFAAwF,EAAE,KAAK,IAAI,EAAE;QACtG,MAAM,MAAM,GAAG,MAAM,cAAc,CACjC;YACE,QAAQ;YACR,YAAY,EAAE,QAAQ;YACtB,MAAM,EAAE,EAAE,IAAI,EAAE,QAAQ,EAAE;YAC1B,kBAAkB,EAAE,OAAO;SAC5B,EACD;YACE,QAAQ;YACR,cAAc,EAAE,KAAK,IAAI,EAAE,CAAC,KAAK;YACjC,qBAAqB,EAAE,KAAK,IAAI,EAAE,CAAC,CAAC,EAAE,MAAM,EAAE,QAAQ,EAAE,MAAM,EAAE,WAAW,EAAE,CAAC;SAC/E,CACF,CAAC;QAEF,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,SAAS,CAAC,CAAC;QACtC,IAAI,MAAM,CAAC,MAAM,KAAK,SAAS,EAAE,CAAC;YAChC,MAAM,CAAC,MAAM,CAAC,eAAe,CAAC,CAAC,IAAI,CAAC,WAAW,CAAC,CAAC;QACnD,CAAC;IACH,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}
|
package/dist/tests/units/modules/worktree-management/usecases/removeWorktree.usecase.test.d.ts.map
ADDED
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"removeWorktree.usecase.test.d.ts","sourceRoot":"","sources":["../../../../../../src/tests/units/modules/worktree-management/usecases/removeWorktree.usecase.test.ts"],"names":[],"mappings":""}
|
|
@@ -0,0 +1,55 @@
|
|
|
1
|
+
import { describe, it, expect, beforeEach } from 'vitest';
|
|
2
|
+
import { removeWorktree } from '../../../../../modules/worktree-management/usecases/removeWorktree.usecase.js';
|
|
3
|
+
import { deriveWorktreePath } from '../../../../../modules/worktree-management/entities/worktree/worktree.js';
|
|
4
|
+
import { StubGitCommandExecutor } from '../../../../../tests/stubs/gitCommandExecutor.stub.js';
|
|
5
|
+
const identity = {
|
|
6
|
+
platform: 'gitlab',
|
|
7
|
+
projectPath: 'group/project',
|
|
8
|
+
mrNumber: 99,
|
|
9
|
+
};
|
|
10
|
+
const expectedPath = deriveWorktreePath(identity);
|
|
11
|
+
describe('removeWorktree use case', () => {
|
|
12
|
+
let executor;
|
|
13
|
+
let existingPaths;
|
|
14
|
+
beforeEach(() => {
|
|
15
|
+
executor = new StubGitCommandExecutor();
|
|
16
|
+
existingPaths = new Set();
|
|
17
|
+
});
|
|
18
|
+
it('removes an existing worktree via git worktree remove --force', async () => {
|
|
19
|
+
existingPaths.add(expectedPath);
|
|
20
|
+
const result = await removeWorktree({ identity, sourceCheckoutPath: '/repo' }, {
|
|
21
|
+
executor,
|
|
22
|
+
worktreeExists: async (path) => existingPaths.has(path),
|
|
23
|
+
});
|
|
24
|
+
expect(result).toEqual({ status: 'removed' });
|
|
25
|
+
const removeCalls = executor.callsOfKind('worktree-remove');
|
|
26
|
+
expect(removeCalls).toHaveLength(1);
|
|
27
|
+
expect(removeCalls[0]?.args).toContain('--force');
|
|
28
|
+
expect(removeCalls[0]?.args).toContain(expectedPath);
|
|
29
|
+
});
|
|
30
|
+
it('returns absent when the worktree is missing on disk', async () => {
|
|
31
|
+
const result = await removeWorktree({ identity, sourceCheckoutPath: '/repo' }, {
|
|
32
|
+
executor,
|
|
33
|
+
worktreeExists: async () => false,
|
|
34
|
+
});
|
|
35
|
+
expect(result).toEqual({ status: 'absent' });
|
|
36
|
+
expect(executor.callsOfKind('worktree-remove')).toHaveLength(0);
|
|
37
|
+
});
|
|
38
|
+
it('returns failed with warning when git worktree remove fails', async () => {
|
|
39
|
+
existingPaths.add(expectedPath);
|
|
40
|
+
executor.programResponse('worktree-remove', {
|
|
41
|
+
exitCode: 1,
|
|
42
|
+
stdout: '',
|
|
43
|
+
stderr: 'fatal: worktree is locked',
|
|
44
|
+
});
|
|
45
|
+
const result = await removeWorktree({ identity, sourceCheckoutPath: '/repo' }, {
|
|
46
|
+
executor,
|
|
47
|
+
worktreeExists: async () => existingPaths.has(expectedPath),
|
|
48
|
+
});
|
|
49
|
+
expect(result.status).toBe('failed');
|
|
50
|
+
if (result.status === 'failed') {
|
|
51
|
+
expect(result.warning).toContain('worktree is locked');
|
|
52
|
+
}
|
|
53
|
+
});
|
|
54
|
+
});
|
|
55
|
+
//# sourceMappingURL=removeWorktree.usecase.test.js.map
|
package/dist/tests/units/modules/worktree-management/usecases/removeWorktree.usecase.test.js.map
ADDED
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"removeWorktree.usecase.test.js","sourceRoot":"","sources":["../../../../../../src/tests/units/modules/worktree-management/usecases/removeWorktree.usecase.test.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,QAAQ,EAAE,EAAE,EAAE,MAAM,EAAE,UAAU,EAAE,MAAM,QAAQ,CAAC;AAC1D,OAAO,EAAE,cAAc,EAAE,MAAM,kEAAkE,CAAC;AAClG,OAAO,EAAE,kBAAkB,EAAE,MAAM,6DAA6D,CAAC;AAEjG,OAAO,EAAE,sBAAsB,EAAE,MAAM,0CAA0C,CAAC;AAElF,MAAM,QAAQ,GAAqB;IACjC,QAAQ,EAAE,QAAQ;IAClB,WAAW,EAAE,eAAe;IAC5B,QAAQ,EAAE,EAAE;CACb,CAAC;AACF,MAAM,YAAY,GAAG,kBAAkB,CAAC,QAAQ,CAAC,CAAC;AAElD,QAAQ,CAAC,yBAAyB,EAAE,GAAG,EAAE;IACvC,IAAI,QAAgC,CAAC;IACrC,IAAI,aAA0B,CAAC;IAE/B,UAAU,CAAC,GAAG,EAAE;QACd,QAAQ,GAAG,IAAI,sBAAsB,EAAE,CAAC;QACxC,aAAa,GAAG,IAAI,GAAG,EAAE,CAAC;IAC5B,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,8DAA8D,EAAE,KAAK,IAAI,EAAE;QAC5E,aAAa,CAAC,GAAG,CAAC,YAAY,CAAC,CAAC;QAEhC,MAAM,MAAM,GAAG,MAAM,cAAc,CACjC,EAAE,QAAQ,EAAE,kBAAkB,EAAE,OAAO,EAAE,EACzC;YACE,QAAQ;YACR,cAAc,EAAE,KAAK,EAAE,IAAkB,EAAE,EAAE,CAAC,aAAa,CAAC,GAAG,CAAC,IAAI,CAAC;SACtE,CACF,CAAC;QAEF,MAAM,CAAC,MAAM,CAAC,CAAC,OAAO,CAAC,EAAE,MAAM,EAAE,SAAS,EAAE,CAAC,CAAC;QAC9C,MAAM,WAAW,GAAG,QAAQ,CAAC,WAAW,CAAC,iBAAiB,CAAC,CAAC;QAC5D,MAAM,CAAC,WAAW,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC;QACpC,MAAM,CAAC,WAAW,CAAC,CAAC,CAAC,EAAE,IAAI,CAAC,CAAC,SAAS,CAAC,SAAS,CAAC,CAAC;QAClD,MAAM,CAAC,WAAW,CAAC,CAAC,CAAC,EAAE,IAAI,CAAC,CAAC,SAAS,CAAC,YAAY,CAAC,CAAC;IACvD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,qDAAqD,EAAE,KAAK,IAAI,EAAE;QACnE,MAAM,MAAM,GAAG,MAAM,cAAc,CACjC,EAAE,QAAQ,EAAE,kBAAkB,EAAE,OAAO,EAAE,EACzC;YACE,QAAQ;YACR,cAAc,EAAE,KAAK,IAAI,EAAE,CAAC,KAAK;SAClC,CACF,CAAC;QAEF,MAAM,CAAC,MAAM,CAAC,CAAC,OAAO,CAAC,EAAE,MAAM,EAAE,QAAQ,EAAE,CAAC,CAAC;QAC7C,MAAM,CAAC,QAAQ,CAAC,WAAW,CAAC,iBAAiB,CAAC,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC;IAClE,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,4DAA4D,EAAE,KAAK,IAAI,EAAE;QAC1E,aAAa,CAAC,GAAG,CAAC,YAAY,CAAC,CAAC;QAChC,QAAQ,CAAC,eAAe,CAAC,iBAAiB,EAAE;YAC1C,QAAQ,EAAE,CAAC;YACX,MAAM,EAAE,EAAE;YACV,MAAM,EAAE,2BAA2B;SACpC,CAAC,CAAC;QAEH,MAAM,MAAM,GAAG,MAAM,cAAc,CACjC,EAAE,QAAQ,EAAE,kBAAkB,EAAE,OAAO,EAAE,EACzC;YACE,QAAQ;YACR,cAAc,EAAE,KAAK,IAAI,EAAE,CAAC,aAAa,CAAC,GAAG,CAAC,YAAY,CAAC;SAC5D,CACF,CAAC;QAEF,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC;QACrC,IAAI,MAAM,CAAC,MAAM,KAAK,QAAQ,EAAE,CAAC;YAC/B,MAAM,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC,SAAS,CAAC,oBAAoB,CAAC,CAAC;QACzD,CAAC;IACH,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"sweepStaleWorktrees.usecase.test.d.ts","sourceRoot":"","sources":["../../../../../../src/tests/units/modules/worktree-management/usecases/sweepStaleWorktrees.usecase.test.ts"],"names":[],"mappings":""}
|
package/dist/tests/units/modules/worktree-management/usecases/sweepStaleWorktrees.usecase.test.js
ADDED
|
@@ -0,0 +1,131 @@
|
|
|
1
|
+
import { describe, it, expect, beforeEach } from 'vitest';
|
|
2
|
+
import { sweepStaleWorktrees } from '../../../../../modules/worktree-management/usecases/sweepStaleWorktrees.usecase.js';
|
|
3
|
+
import { deriveWorktreePath } from '../../../../../modules/worktree-management/entities/worktree/worktree.js';
|
|
4
|
+
function buildTrackedMr(overrides) {
|
|
5
|
+
return {
|
|
6
|
+
id: overrides.id ?? 'gitlab-group-project-1',
|
|
7
|
+
mrNumber: overrides.mrNumber ?? 1,
|
|
8
|
+
title: 'title',
|
|
9
|
+
url: 'http://example.com',
|
|
10
|
+
project: overrides.project ?? 'group/project',
|
|
11
|
+
platform: overrides.platform ?? 'gitlab',
|
|
12
|
+
sourceBranch: 'feat/x',
|
|
13
|
+
targetBranch: 'master',
|
|
14
|
+
assignment: {
|
|
15
|
+
username: 'user',
|
|
16
|
+
assignedAt: '2026-05-01T00:00:00Z',
|
|
17
|
+
},
|
|
18
|
+
state: overrides.state ?? 'pending-review',
|
|
19
|
+
openThreads: 0,
|
|
20
|
+
totalThreads: 0,
|
|
21
|
+
createdAt: '2026-05-01T00:00:00Z',
|
|
22
|
+
lastReviewAt: null,
|
|
23
|
+
lastPushAt: null,
|
|
24
|
+
approvedAt: null,
|
|
25
|
+
mergedAt: overrides.mergedAt ?? null,
|
|
26
|
+
reviews: [],
|
|
27
|
+
totalReviews: 0,
|
|
28
|
+
totalFollowups: 0,
|
|
29
|
+
totalBlocking: 0,
|
|
30
|
+
totalWarnings: 0,
|
|
31
|
+
totalSuggestions: 0,
|
|
32
|
+
totalDurationMs: 0,
|
|
33
|
+
latestScore: null,
|
|
34
|
+
autoFollowup: true,
|
|
35
|
+
};
|
|
36
|
+
}
|
|
37
|
+
function buildEntry(identity, mtime) {
|
|
38
|
+
return {
|
|
39
|
+
identity,
|
|
40
|
+
path: deriveWorktreePath(identity),
|
|
41
|
+
mtime,
|
|
42
|
+
};
|
|
43
|
+
}
|
|
44
|
+
describe('sweepStaleWorktrees use case', () => {
|
|
45
|
+
let removed;
|
|
46
|
+
let fakeTracking;
|
|
47
|
+
beforeEach(() => {
|
|
48
|
+
removed = [];
|
|
49
|
+
fakeTracking = {
|
|
50
|
+
trackedMrs: new Map(),
|
|
51
|
+
getById(projectPath, mrId) {
|
|
52
|
+
const stored = this.trackedMrs.get(`${projectPath}::${mrId}`) ?? null;
|
|
53
|
+
return stored;
|
|
54
|
+
},
|
|
55
|
+
};
|
|
56
|
+
});
|
|
57
|
+
const repository = { localPath: '/repos/group-project', enabled: true };
|
|
58
|
+
const now = new Date('2026-05-23T12:00:00Z');
|
|
59
|
+
function trackingFor(identity, tracked) {
|
|
60
|
+
const mrId = `${identity.platform}-${identity.projectPath}-${identity.mrNumber}`;
|
|
61
|
+
fakeTracking.trackedMrs.set(`${repository.localPath}::${mrId}`, tracked);
|
|
62
|
+
}
|
|
63
|
+
async function runSweep(entries) {
|
|
64
|
+
await sweepStaleWorktrees({
|
|
65
|
+
listEntries: async () => entries,
|
|
66
|
+
removeWorktree: async (identity) => {
|
|
67
|
+
removed.push(identity);
|
|
68
|
+
return { status: 'removed' };
|
|
69
|
+
},
|
|
70
|
+
trackingGateway: {
|
|
71
|
+
getById: (projectPath, mrId) => fakeTracking.getById(projectPath, mrId),
|
|
72
|
+
},
|
|
73
|
+
getRepositories: () => [repository],
|
|
74
|
+
now: () => now,
|
|
75
|
+
});
|
|
76
|
+
}
|
|
77
|
+
it('removes worktree when matching MR is merged more than 24h ago', async () => {
|
|
78
|
+
const identity = { platform: 'gitlab', projectPath: 'group-project', mrNumber: 1 };
|
|
79
|
+
const entry = buildEntry(identity, new Date('2026-05-21T00:00:00Z'));
|
|
80
|
+
trackingFor(identity, buildTrackedMr({
|
|
81
|
+
id: 'gitlab-group-project-1',
|
|
82
|
+
state: 'merged',
|
|
83
|
+
mergedAt: '2026-05-21T00:00:00Z',
|
|
84
|
+
}));
|
|
85
|
+
await runSweep([entry]);
|
|
86
|
+
expect(removed).toEqual([identity]);
|
|
87
|
+
});
|
|
88
|
+
it('keeps worktree when MR was merged less than 24h ago', async () => {
|
|
89
|
+
const identity = { platform: 'gitlab', projectPath: 'group-project', mrNumber: 2 };
|
|
90
|
+
const entry = buildEntry(identity, new Date('2026-05-23T00:00:00Z'));
|
|
91
|
+
trackingFor(identity, buildTrackedMr({
|
|
92
|
+
id: 'gitlab-group-project-2',
|
|
93
|
+
mrNumber: 2,
|
|
94
|
+
state: 'merged',
|
|
95
|
+
mergedAt: '2026-05-23T06:00:00Z',
|
|
96
|
+
}));
|
|
97
|
+
await runSweep([entry]);
|
|
98
|
+
expect(removed).toEqual([]);
|
|
99
|
+
});
|
|
100
|
+
it('removes orphan worktree when no tracked MR exists', async () => {
|
|
101
|
+
const identity = { platform: 'gitlab', projectPath: 'group-project', mrNumber: 3 };
|
|
102
|
+
const entry = buildEntry(identity, new Date('2026-05-23T11:00:00Z'));
|
|
103
|
+
await runSweep([entry]);
|
|
104
|
+
expect(removed).toEqual([identity]);
|
|
105
|
+
});
|
|
106
|
+
it('removes stale worktree with mtime older than 7 days regardless of MR state', async () => {
|
|
107
|
+
const identity = { platform: 'gitlab', projectPath: 'group-project', mrNumber: 4 };
|
|
108
|
+
const eightDaysAgo = new Date(now.getTime() - 8 * 24 * 60 * 60 * 1000);
|
|
109
|
+
const entry = buildEntry(identity, eightDaysAgo);
|
|
110
|
+
trackingFor(identity, buildTrackedMr({
|
|
111
|
+
id: 'gitlab-group-project-4',
|
|
112
|
+
mrNumber: 4,
|
|
113
|
+
state: 'pending-review',
|
|
114
|
+
}));
|
|
115
|
+
await runSweep([entry]);
|
|
116
|
+
expect(removed).toEqual([identity]);
|
|
117
|
+
});
|
|
118
|
+
it('keeps active worktree with fresh mtime', async () => {
|
|
119
|
+
const identity = { platform: 'gitlab', projectPath: 'group-project', mrNumber: 5 };
|
|
120
|
+
const sixDaysAgo = new Date(now.getTime() - 6 * 24 * 60 * 60 * 1000);
|
|
121
|
+
const entry = buildEntry(identity, sixDaysAgo);
|
|
122
|
+
trackingFor(identity, buildTrackedMr({
|
|
123
|
+
id: 'gitlab-group-project-5',
|
|
124
|
+
mrNumber: 5,
|
|
125
|
+
state: 'pending-review',
|
|
126
|
+
}));
|
|
127
|
+
await runSweep([entry]);
|
|
128
|
+
expect(removed).toEqual([]);
|
|
129
|
+
});
|
|
130
|
+
});
|
|
131
|
+
//# sourceMappingURL=sweepStaleWorktrees.usecase.test.js.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"sweepStaleWorktrees.usecase.test.js","sourceRoot":"","sources":["../../../../../../src/tests/units/modules/worktree-management/usecases/sweepStaleWorktrees.usecase.test.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,QAAQ,EAAE,EAAE,EAAE,MAAM,EAAE,UAAU,EAAE,MAAM,QAAQ,CAAC;AAC1D,OAAO,EAAE,mBAAmB,EAAE,MAAM,uEAAuE,CAAC;AAC5G,OAAO,EAAE,kBAAkB,EAAE,MAAM,6DAA6D,CAAC;AAiBjG,SAAS,cAAc,CAAC,SAA6B;IACnD,OAAO;QACL,EAAE,EAAE,SAAS,CAAC,EAAE,IAAI,wBAAwB;QAC5C,QAAQ,EAAE,SAAS,CAAC,QAAQ,IAAI,CAAC;QACjC,KAAK,EAAE,OAAO;QACd,GAAG,EAAE,oBAAoB;QACzB,OAAO,EAAE,SAAS,CAAC,OAAO,IAAI,eAAe;QAC7C,QAAQ,EAAE,SAAS,CAAC,QAAQ,IAAI,QAAQ;QACxC,YAAY,EAAE,QAAQ;QACtB,YAAY,EAAE,QAAQ;QACtB,UAAU,EAAE;YACV,QAAQ,EAAE,MAAM;YAChB,UAAU,EAAE,sBAAsB;SACnC;QACD,KAAK,EAAE,SAAS,CAAC,KAAK,IAAI,gBAAgB;QAC1C,WAAW,EAAE,CAAC;QACd,YAAY,EAAE,CAAC;QACf,SAAS,EAAE,sBAAsB;QACjC,YAAY,EAAE,IAAI;QAClB,UAAU,EAAE,IAAI;QAChB,UAAU,EAAE,IAAI;QAChB,QAAQ,EAAE,SAAS,CAAC,QAAQ,IAAI,IAAI;QACpC,OAAO,EAAE,EAAE;QACX,YAAY,EAAE,CAAC;QACf,cAAc,EAAE,CAAC;QACjB,aAAa,EAAE,CAAC;QAChB,aAAa,EAAE,CAAC;QAChB,gBAAgB,EAAE,CAAC;QACnB,eAAe,EAAE,CAAC;QAClB,WAAW,EAAE,IAAI;QACjB,YAAY,EAAE,IAAI;KACnB,CAAC;AACJ,CAAC;AAED,SAAS,UAAU,CAAC,QAA0B,EAAE,KAAW;IACzD,OAAO;QACL,QAAQ;QACR,IAAI,EAAE,kBAAkB,CAAC,QAAQ,CAAC;QAClC,KAAK;KACN,CAAC;AACJ,CAAC;AAED,QAAQ,CAAC,8BAA8B,EAAE,GAAG,EAAE;IAC5C,IAAI,OAA2B,CAAC;IAChC,IAAI,YAAiC,CAAC;IAEtC,UAAU,CAAC,GAAG,EAAE;QACd,OAAO,GAAG,EAAE,CAAC;QACb,YAAY,GAAG;YACb,UAAU,EAAE,IAAI,GAAG,EAAE;YACrB,OAAO,CAAC,WAAmB,EAAE,IAAY;gBACvC,MAAM,MAAM,GAAG,IAAI,CAAC,UAAU,CAAC,GAAG,CAAC,GAAG,WAAW,KAAK,IAAI,EAAE,CAAC,IAAI,IAAI,CAAC;gBACtE,OAAO,MAAM,CAAC;YAChB,CAAC;SACF,CAAC;IACJ,CAAC,CAAC,CAAC;IAEH,MAAM,UAAU,GAAqB,EAAE,SAAS,EAAE,sBAAsB,EAAE,OAAO,EAAE,IAAI,EAAE,CAAC;IAC1F,MAAM,GAAG,GAAG,IAAI,IAAI,CAAC,sBAAsB,CAAC,CAAC;IAE7C,SAAS,WAAW,CAAC,QAA0B,EAAE,OAAkB;QACjE,MAAM,IAAI,GAAG,GAAG,QAAQ,CAAC,QAAQ,IAAI,QAAQ,CAAC,WAAW,IAAI,QAAQ,CAAC,QAAQ,EAAE,CAAC;QACjF,YAAY,CAAC,UAAU,CAAC,GAAG,CAAC,GAAG,UAAU,CAAC,SAAS,KAAK,IAAI,EAAE,EAAE,OAAO,CAAC,CAAC;IAC3E,CAAC;IAED,KAAK,UAAU,QAAQ,CAAC,OAAwB;QAC9C,MAAM,mBAAmB,CACvB;YACE,WAAW,EAAE,KAAK,IAAI,EAAE,CAAC,OAAO;YAChC,cAAc,EAAE,KAAK,EAAC,QAAQ,EAAC,EAAE;gBAC/B,OAAO,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC;gBACvB,OAAO,EAAE,MAAM,EAAE,SAAS,EAAE,CAAC;YAC/B,CAAC;YACD,eAAe,EAAE;gBACf,OAAO,EAAE,CAAC,WAAW,EAAE,IAAI,EAAE,EAAE,CAAC,YAAY,CAAC,OAAO,CAAC,WAAW,EAAE,IAAI,CAAC;aACxE;YACD,eAAe,EAAE,GAAG,EAAE,CAAC,CAAC,UAAU,CAAC;YACnC,GAAG,EAAE,GAAG,EAAE,CAAC,GAAG;SACf,CACF,CAAC;IACJ,CAAC;IAED,EAAE,CAAC,+DAA+D,EAAE,KAAK,IAAI,EAAE;QAC7E,MAAM,QAAQ,GAAqB,EAAE,QAAQ,EAAE,QAAQ,EAAE,WAAW,EAAE,eAAe,EAAE,QAAQ,EAAE,CAAC,EAAE,CAAC;QACrG,MAAM,KAAK,GAAG,UAAU,CAAC,QAAQ,EAAE,IAAI,IAAI,CAAC,sBAAsB,CAAC,CAAC,CAAC;QACrE,WAAW,CAAC,QAAQ,EAAE,cAAc,CAAC;YACnC,EAAE,EAAE,wBAAwB;YAC5B,KAAK,EAAE,QAAQ;YACf,QAAQ,EAAE,sBAAsB;SACjC,CAAC,CAAC,CAAC;QAEJ,MAAM,QAAQ,CAAC,CAAC,KAAK,CAAC,CAAC,CAAC;QAExB,MAAM,CAAC,OAAO,CAAC,CAAC,OAAO,CAAC,CAAC,QAAQ,CAAC,CAAC,CAAC;IACtC,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,qDAAqD,EAAE,KAAK,IAAI,EAAE;QACnE,MAAM,QAAQ,GAAqB,EAAE,QAAQ,EAAE,QAAQ,EAAE,WAAW,EAAE,eAAe,EAAE,QAAQ,EAAE,CAAC,EAAE,CAAC;QACrG,MAAM,KAAK,GAAG,UAAU,CAAC,QAAQ,EAAE,IAAI,IAAI,CAAC,sBAAsB,CAAC,CAAC,CAAC;QACrE,WAAW,CAAC,QAAQ,EAAE,cAAc,CAAC;YACnC,EAAE,EAAE,wBAAwB;YAC5B,QAAQ,EAAE,CAAC;YACX,KAAK,EAAE,QAAQ;YACf,QAAQ,EAAE,sBAAsB;SACjC,CAAC,CAAC,CAAC;QAEJ,MAAM,QAAQ,CAAC,CAAC,KAAK,CAAC,CAAC,CAAC;QAExB,MAAM,CAAC,OAAO,CAAC,CAAC,OAAO,CAAC,EAAE,CAAC,CAAC;IAC9B,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,mDAAmD,EAAE,KAAK,IAAI,EAAE;QACjE,MAAM,QAAQ,GAAqB,EAAE,QAAQ,EAAE,QAAQ,EAAE,WAAW,EAAE,eAAe,EAAE,QAAQ,EAAE,CAAC,EAAE,CAAC;QACrG,MAAM,KAAK,GAAG,UAAU,CAAC,QAAQ,EAAE,IAAI,IAAI,CAAC,sBAAsB,CAAC,CAAC,CAAC;QAErE,MAAM,QAAQ,CAAC,CAAC,KAAK,CAAC,CAAC,CAAC;QAExB,MAAM,CAAC,OAAO,CAAC,CAAC,OAAO,CAAC,CAAC,QAAQ,CAAC,CAAC,CAAC;IACtC,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,4EAA4E,EAAE,KAAK,IAAI,EAAE;QAC1F,MAAM,QAAQ,GAAqB,EAAE,QAAQ,EAAE,QAAQ,EAAE,WAAW,EAAE,eAAe,EAAE,QAAQ,EAAE,CAAC,EAAE,CAAC;QACrG,MAAM,YAAY,GAAG,IAAI,IAAI,CAAC,GAAG,CAAC,OAAO,EAAE,GAAG,CAAC,GAAG,EAAE,GAAG,EAAE,GAAG,EAAE,GAAG,IAAI,CAAC,CAAC;QACvE,MAAM,KAAK,GAAG,UAAU,CAAC,QAAQ,EAAE,YAAY,CAAC,CAAC;QACjD,WAAW,CAAC,QAAQ,EAAE,cAAc,CAAC;YACnC,EAAE,EAAE,wBAAwB;YAC5B,QAAQ,EAAE,CAAC;YACX,KAAK,EAAE,gBAAgB;SACxB,CAAC,CAAC,CAAC;QAEJ,MAAM,QAAQ,CAAC,CAAC,KAAK,CAAC,CAAC,CAAC;QAExB,MAAM,CAAC,OAAO,CAAC,CAAC,OAAO,CAAC,CAAC,QAAQ,CAAC,CAAC,CAAC;IACtC,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,wCAAwC,EAAE,KAAK,IAAI,EAAE;QACtD,MAAM,QAAQ,GAAqB,EAAE,QAAQ,EAAE,QAAQ,EAAE,WAAW,EAAE,eAAe,EAAE,QAAQ,EAAE,CAAC,EAAE,CAAC;QACrG,MAAM,UAAU,GAAG,IAAI,IAAI,CAAC,GAAG,CAAC,OAAO,EAAE,GAAG,CAAC,GAAG,EAAE,GAAG,EAAE,GAAG,EAAE,GAAG,IAAI,CAAC,CAAC;QACrE,MAAM,KAAK,GAAG,UAAU,CAAC,QAAQ,EAAE,UAAU,CAAC,CAAC;QAC/C,WAAW,CAAC,QAAQ,EAAE,cAAc,CAAC;YACnC,EAAE,EAAE,wBAAwB;YAC5B,QAAQ,EAAE,CAAC;YACX,KAAK,EAAE,gBAAgB;SACxB,CAAC,CAAC,CAAC;QAEJ,MAAM,QAAQ,CAAC,CAAC,KAAK,CAAC,CAAC,CAAC;QAExB,MAAM,CAAC,OAAO,CAAC,CAAC,OAAO,CAAC,EAAE,CAAC,CAAC;IAC9B,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}
|