@bevel-software/platform-core-backend 0.35.0 → 0.36.1

package/src/modules/{workflow → version-control}/git/__tests__/git.service.commitFile.test.ts

@@ -6,7 +6,7 @@ import fs from 'node:fs/promises';
 import path from 'node:path';
 import os from 'node:os';
 import type { WorkspaceService } from '../../../workspace/workspace.service.js';
-import { WorkflowHooks } from '../../workflow-hooks.js';
+import { VersionControlHooks } from '../../version-control-hooks.js';
 import { GitService } from '../git.service.js';
 
 /**
@@ -69,12 +69,12 @@ function stubWorkspaceService(workspaceId: string, workspaceDir: string): Worksp
   } as unknown as WorkspaceService;
 }
 
-function makeValidator(): { hooks: WorkflowHooks; runValidation: ReturnType<typeof vi.fn> } {
+function makeValidator(): { hooks: VersionControlHooks; runValidation: ReturnType<typeof vi.fn> } {
   const report = { ok: true, mustFix: [], warnings: [], rawOutput: '' };
   const runValidation = vi.fn(async () => report);
   // Registered as a commit-validation hook — the seam GitService consults
   // where it used to call the injected validator.
-  const hooks = new WorkflowHooks();
+  const hooks = new VersionControlHooks();
   hooks.onCommitValidation(() => runValidation());
   return { hooks, runValidation };
 }

package/src/modules/{workflow → version-control}/git/__tests__/git.service.createBranch.test.ts

@@ -6,7 +6,7 @@ import fs from 'node:fs/promises';
 import path from 'node:path';
 import os from 'node:os';
 import type { WorkspaceService } from '../../../workspace/workspace.service.js';
-import { WorkflowHooks } from '../../workflow-hooks.js';
+import { VersionControlHooks } from '../../version-control-hooks.js';
 import { GitService } from '../git.service.js';
 
 const execFileAsync = promisify(execFile);
@@ -90,7 +90,7 @@ describe('GitService.createBranch', () => {
     const { upstream } = await seedWorkspace(root, workspaceId);
     const svc = new GitService(
       stubWorkspaceService(workspaceId, path.join(root, workspaceId)),
-      new WorkflowHooks(),
+      new VersionControlHooks(),
       testKbContext(),
     );
 
@@ -133,7 +133,7 @@ describe('GitService.createBranch', () => {
     const repo = path.join(root, workspaceId, 'knowledge-base');
     const svc = new GitService(
       stubWorkspaceService(workspaceId, path.join(root, workspaceId)),
-      new WorkflowHooks(),
+      new VersionControlHooks(),
       testKbContext(),
     );
 

package/src/modules/{workflow → version-control}/git/__tests__/git.service.deleteBranch.test.ts

@@ -7,14 +7,14 @@ import path from 'node:path';
 import os from 'node:os';
 import type { AuthUser } from '@bevel-software/platform-shared';
 import type { WorkspaceService } from '../../../workspace/workspace.service.js';
-import { WorkflowHooks } from '../../workflow-hooks.js';
+import { VersionControlHooks } from '../../version-control-hooks.js';
 import type { IAccessControl } from '../../../access/access-control.interface.js';
 import { GitService } from '../git.service.js';
 import {
   BranchAuthorshipError,
   BranchDeleteRefusedError,
   ProtectedBranchError,
-  WorkflowValidationError,
+  VersionControlValidationError,
 } from '../../../../shared/domain-errors.js';
 
 const execFileAsync = promisify(execFile);
@@ -160,7 +160,7 @@ describe('GitService.deleteBranch — authorship + remote delete', () => {
   function makeService(): GitService {
     return new GitService(
       stubWorkspaceService(workspaceId, path.join(root, workspaceId)),
-      new WorkflowHooks(),
+      new VersionControlHooks(),
       testKbContext(),
     );
   }
@@ -276,7 +276,7 @@ describe('GitService.deleteBranch — authorship + remote delete', () => {
     const svc = makeService();
     await expect(
       svc.deleteBranch(workspaceId, 'alice/my-draft', ALICE),
-    ).rejects.toBeInstanceOf(WorkflowValidationError);
+    ).rejects.toBeInstanceOf(VersionControlValidationError);
   });
 
   it('onlyIfNoRemote: true bypasses the authorship check (orphan-tidy-up path stays open)', async () => {
@@ -305,7 +305,7 @@ describe('GitService.deleteBranch — authorship + remote delete', () => {
 
     await expect(
       svc.deleteBranch(workspaceId, 'alice/my-draft', ALICE, { onlyIfNoRemote: true }),
-    ).rejects.toBeInstanceOf(WorkflowValidationError);
+    ).rejects.toBeInstanceOf(VersionControlValidationError);
   });
 
   // Admin-can-delete: when the user is in the `Admin` role per `roles.yaml`
@@ -315,7 +315,7 @@ describe('GitService.deleteBranch — authorship + remote delete', () => {
   function makeServiceWithAccess(ac: IAccessControl): GitService {
     return new GitService(
       stubWorkspaceService(workspaceId, path.join(root, workspaceId)),
-      new WorkflowHooks(),
+      new VersionControlHooks(),
       testKbContext(),
       undefined,
       ac,
@@ -391,7 +391,7 @@ describe('GitService — branch state for a checked delete', () => {
   function makeService(): GitService {
     return new GitService(
       stubWorkspaceService(workspaceId, path.join(root, workspaceId)),
-      new WorkflowHooks(),
+      new VersionControlHooks(),
       testKbContext(),
     );
   }

package/src/modules/{workflow → version-control}/git/__tests__/git.service.diffFileAtCommit.test.ts

@@ -6,9 +6,9 @@ import fs from 'node:fs/promises';
 import path from 'node:path';
 import os from 'node:os';
 import type { WorkspaceService } from '../../../workspace/workspace.service.js';
-import { WorkflowHooks } from '../../workflow-hooks.js';
+import { VersionControlHooks } from '../../version-control-hooks.js';
 import { GitService } from '../git.service.js';
-import { WorkflowValidationError } from '../../../../shared/domain-errors.js';
+import { VersionControlValidationError } from '../../../../shared/domain-errors.js';
 
 const execFileAsync = promisify(execFile);
 const PROCESS_MAP_DIR = 'knowledge-base';
@@ -108,7 +108,7 @@ describe('GitService.diffFileAtCommit', () => {
     const { workspaceDir, repo } = await seedWorkspace(root, workspaceId);
     const svc = new GitService(
       stubWorkspaceService(workspaceId, workspaceDir),
-      new WorkflowHooks(),
+      new VersionControlHooks(),
       testKbContext(),
     );
 
@@ -125,7 +125,7 @@ describe('GitService.diffFileAtCommit', () => {
     const { workspaceDir, repo } = await seedWorkspace(root, workspaceId);
     const svc = new GitService(
       stubWorkspaceService(workspaceId, workspaceDir),
-      new WorkflowHooks(),
+      new VersionControlHooks(),
       testKbContext(),
     );
 
@@ -141,7 +141,7 @@ describe('GitService.diffFileAtCommit', () => {
     const { workspaceDir, repo } = await seedWorkspace(root, workspaceId);
     const svc = new GitService(
       stubWorkspaceService(workspaceId, workspaceDir),
-      new WorkflowHooks(),
+      new VersionControlHooks(),
       testKbContext(),
     );
 
@@ -157,7 +157,7 @@ describe('GitService.diffFileAtCommit', () => {
     const { workspaceDir } = await seedWorkspace(root, workspaceId);
     const svc = new GitService(
       stubWorkspaceService(workspaceId, workspaceDir),
-      new WorkflowHooks(),
+      new VersionControlHooks(),
       testKbContext(),
     );
 
@@ -170,13 +170,13 @@ describe('GitService.diffFileAtCommit', () => {
     const { workspaceDir } = await seedWorkspace(root, workspaceId);
     const svc = new GitService(
       stubWorkspaceService(workspaceId, workspaceDir),
-      new WorkflowHooks(),
+      new VersionControlHooks(),
       testKbContext(),
     );
 
     await expect(
       svc.diffFileAtCommit(workspaceId, '../etc/passwd', 'abcdef1'),
-    ).rejects.toBeInstanceOf(WorkflowValidationError);
+    ).rejects.toBeInstanceOf(VersionControlValidationError);
   });
 
   // Regression: pathspec literal handling. `git show <sha> -- '[New] memo.md'`
@@ -187,7 +187,7 @@ describe('GitService.diffFileAtCommit', () => {
     const { workspaceDir, repo } = await seedWorkspace(root, workspaceId);
     const svc = new GitService(
       stubWorkspaceService(workspaceId, workspaceDir),
-      new WorkflowHooks(),
+      new VersionControlHooks(),
       testKbContext(),
     );
     await runGit(repo, ['commit', '--allow-empty', '-m', 'init']);

package/src/modules/{workflow → version-control}/git/__tests__/git.service.diffFileBetweenBranches.test.ts

@@ -6,9 +6,9 @@ import fs from 'node:fs/promises';
 import path from 'node:path';
 import os from 'node:os';
 import type { WorkspaceService } from '../../../workspace/workspace.service.js';
-import { WorkflowHooks } from '../../workflow-hooks.js';
+import { VersionControlHooks } from '../../version-control-hooks.js';
 import { GitService } from '../git.service.js';
-import { WorkflowValidationError, BranchNameError } from '../../../../shared/domain-errors.js';
+import { VersionControlValidationError, BranchNameError } from '../../../../shared/domain-errors.js';
 import { GitRunError, type IGitRunner } from '../../../../shared/git.contract.js';
 import { NodeGitRunner } from '../node-git-runner.js';
 
@@ -101,7 +101,7 @@ describe('GitService.diffFileBetweenBranches', () => {
     const { workspaceDir, repo } = await seedWorkspace(root, workspaceId);
     const svc = new GitService(
       stubWorkspaceService(workspaceId, workspaceDir),
-      new WorkflowHooks(),
+      new VersionControlHooks(),
       testKbContext(),
     );
 
@@ -121,7 +121,7 @@ describe('GitService.diffFileBetweenBranches', () => {
 
   it('asks the caller\'s verdict at the two commits served, before the path is described at either', async () => {
     const { workspaceDir, repo } = await seedWorkspace(root, workspaceId);
-    const svc = new GitService(stubWorkspaceService(workspaceId, workspaceDir), new WorkflowHooks(), testKbContext());
+    const svc = new GitService(stubWorkspaceService(workspaceId, workspaceDir), new VersionControlHooks(), testKbContext());
     await commitFile(repo, 'Knowledge/Foo.md', 'line one\n', 'base');
     await commitFile(repo, 'Docs/a.md', 'a\n', 'a folder');
     const base = (await execFileAsync('git', ['rev-parse', 'HEAD'], { cwd: repo })).stdout.trim();
@@ -141,7 +141,7 @@ describe('GitService.diffFileBetweenBranches', () => {
     // A folder the caller may not read is refused as unreadable, never as a folder.
     const refuse = async (s: unknown): Promise<void> => {
       asked.push(s);
-      throw new WorkflowValidationError('not yours to read');
+      throw new VersionControlValidationError('not yours to read');
     };
     await expect(svc.diffFileBetweenBranches(workspaceId, 'Docs', 'current-company-state', 'alice/edit-foo', refuse)).rejects.toThrow(/not yours to read/);
     expect(asked).toEqual([sides, sides]);
@@ -165,7 +165,7 @@ describe('GitService.diffFileBetweenBranches', () => {
         return inner.run(cwd, args, opts as never);
       },
     } as unknown as IGitRunner;
-    const svc = new GitService(stubWorkspaceService(workspaceId, workspaceDir), new WorkflowHooks(), testKbContext(), undefined, null, gitRunner);
+    const svc = new GitService(stubWorkspaceService(workspaceId, workspaceDir), new VersionControlHooks(), testKbContext(), undefined, null, gitRunner);
     const asked: unknown[] = [];
     const authorize = async (s: unknown): Promise<void> => { asked.push(s); };
     for (const f of [
@@ -186,7 +186,7 @@ describe('GitService.diffFileBetweenBranches', () => {
     const { workspaceDir, repo } = await seedWorkspace(root, workspaceId);
     const svc = new GitService(
       stubWorkspaceService(workspaceId, workspaceDir),
-      new WorkflowHooks(),
+      new VersionControlHooks(),
       testKbContext(),
     );
 
@@ -208,7 +208,7 @@ describe('GitService.diffFileBetweenBranches', () => {
     const { workspaceDir, repo } = await seedWorkspace(root, workspaceId);
     const svc = new GitService(
       stubWorkspaceService(workspaceId, workspaceDir),
-      new WorkflowHooks(),
+      new VersionControlHooks(),
       testKbContext(),
     );
 
@@ -230,7 +230,7 @@ describe('GitService.diffFileBetweenBranches', () => {
     const { workspaceDir, repo } = await seedWorkspace(root, workspaceId);
     const svc = new GitService(
       stubWorkspaceService(workspaceId, workspaceDir),
-      new WorkflowHooks(),
+      new VersionControlHooks(),
       testKbContext(),
     );
 
@@ -272,7 +272,7 @@ describe('GitService.diffFileBetweenBranches', () => {
 
     const svc = new GitService(
       stubWorkspaceService(workspaceId, workspaceDir),
-      new WorkflowHooks(),
+      new VersionControlHooks(),
       testKbContext(),
     );
 
@@ -297,7 +297,7 @@ describe('GitService.diffFileBetweenBranches', () => {
     const { workspaceDir, repo } = await seedWorkspace(root, workspaceId);
     const svc = new GitService(
       stubWorkspaceService(workspaceId, workspaceDir),
-      new WorkflowHooks(),
+      new VersionControlHooks(),
       testKbContext(),
     );
 
@@ -322,7 +322,7 @@ describe('GitService.diffFileBetweenBranches', () => {
     const { workspaceDir, repo } = await seedWorkspace(root, workspaceId);
     const svc = new GitService(
       stubWorkspaceService(workspaceId, workspaceDir),
-      new WorkflowHooks(),
+      new VersionControlHooks(),
       testKbContext(),
     );
 
@@ -347,7 +347,7 @@ describe('GitService.diffFileBetweenBranches', () => {
     const { workspaceDir, repo } = await seedWorkspace(root, workspaceId);
     const svc = new GitService(
       stubWorkspaceService(workspaceId, workspaceDir),
-      new WorkflowHooks(),
+      new VersionControlHooks(),
       testKbContext(),
     );
 
@@ -370,7 +370,7 @@ describe('GitService.diffFileBetweenBranches', () => {
     const { workspaceDir, repo } = await seedWorkspace(root, workspaceId);
     const svc = new GitService(
       stubWorkspaceService(workspaceId, workspaceDir),
-      new WorkflowHooks(),
+      new VersionControlHooks(),
       testKbContext(),
     );
 
@@ -392,7 +392,7 @@ describe('GitService.diffFileBetweenBranches', () => {
     const { workspaceDir, repo } = await seedWorkspace(root, workspaceId);
     const svc = new GitService(
       stubWorkspaceService(workspaceId, workspaceDir),
-      new WorkflowHooks(),
+      new VersionControlHooks(),
       testKbContext(),
     );
 
@@ -417,7 +417,7 @@ describe('GitService.diffFileBetweenBranches', () => {
     const { workspaceDir } = await seedWorkspace(root, workspaceId);
     const svc = new GitService(
       stubWorkspaceService(workspaceId, workspaceDir),
-      new WorkflowHooks(),
+      new VersionControlHooks(),
       testKbContext(),
     );
 
@@ -428,7 +428,7 @@ describe('GitService.diffFileBetweenBranches', () => {
         'current-company-state',
         'current-company-state',
       ),
-    ).rejects.toBeInstanceOf(WorkflowValidationError);
+    ).rejects.toBeInstanceOf(VersionControlValidationError);
   });
 
   it('rejects an unknown branch', async () => {
@@ -436,7 +436,7 @@ describe('GitService.diffFileBetweenBranches', () => {
     await commitFile(repo, 'Knowledge/Foo.md', 'base\n', 'base');
     const svc = new GitService(
       stubWorkspaceService(workspaceId, workspaceDir),
-      new WorkflowHooks(),
+      new VersionControlHooks(),
       testKbContext(),
     );
 
@@ -447,14 +447,14 @@ describe('GitService.diffFileBetweenBranches', () => {
         'current-company-state',
         'does-not-exist',
       ),
-    ).rejects.toBeInstanceOf(WorkflowValidationError);
+    ).rejects.toBeInstanceOf(VersionControlValidationError);
   });
 
   it('rejects a path-traversal attempt', async () => {
     const { workspaceDir } = await seedWorkspace(root, workspaceId);
     const svc = new GitService(
       stubWorkspaceService(workspaceId, workspaceDir),
-      new WorkflowHooks(),
+      new VersionControlHooks(),
       testKbContext(),
     );
 
@@ -465,14 +465,14 @@ describe('GitService.diffFileBetweenBranches', () => {
         'current-company-state',
         'target-company-state',
       ),
-    ).rejects.toBeInstanceOf(WorkflowValidationError);
+    ).rejects.toBeInstanceOf(VersionControlValidationError);
   });
 
   it('rejects a branch name with shell metacharacters', async () => {
     const { workspaceDir } = await seedWorkspace(root, workspaceId);
     const svc = new GitService(
       stubWorkspaceService(workspaceId, workspaceDir),
-      new WorkflowHooks(),
+      new VersionControlHooks(),
       testKbContext(),
     );
 
@@ -494,7 +494,7 @@ describe('GitService.diffFileBetweenBranches', () => {
     const { workspaceDir, repo } = await seedWorkspace(root, workspaceId);
     const svc = new GitService(
       stubWorkspaceService(workspaceId, workspaceDir),
-      new WorkflowHooks(),
+      new VersionControlHooks(),
       testKbContext(),
     );
     await runGit(repo, ['commit', '--allow-empty', '-m', 'init']);

package/src/modules/{workflow → version-control}/git/__tests__/git.service.fileBytesAtCommit.test.ts

@@ -7,10 +7,10 @@ import { GitService } from '../git.service.js';
 import {
   VersionNotOnBranchError,
   VERSION_NOT_ON_BRANCH_MESSAGE,
-  WorkflowValidationError,
+  VersionControlValidationError,
 } from '../../../../shared/domain-errors.js';
 import { GitRunError, assertOneSpecPerLine } from '../../../../shared/git.contract.js';
-import { runGit, gitOut, stubWorkflowHooks, stubWorkspaceService } from './git-test-helpers.js';
+import { runGit, gitOut, stubVersionControlHooks, stubWorkspaceService } from './git-test-helpers.js';
 
 /**
  * A past version has to come back BYTE FOR BYTE, and only for saves on the
@@ -62,7 +62,7 @@ describe('GitService.fileBytesAtCommit', () => {
 
     svc = new GitService(
       stubWorkspaceService({ [workspaceId]: workspaceDir }),
-      stubWorkflowHooks(),
+      stubVersionControlHooks(),
       testKbContext(),
     );
   });
@@ -156,13 +156,13 @@ describe('GitService.fileBytesAtCommit', () => {
       // `assertValidRelativePath` — the shared one in
       // `packages/shared/src/workspace/filename.ts`, which throws a plain
       // `Error('Invalid path: …')`, and the kb-fs one in
-      // `modules/kb-fs/branch-name.ts`, which throws `WorkflowValidationError`
+      // `modules/kb-fs/branch-name.ts`, which throws `VersionControlValidationError`
       // (status 400). `GitService` imports the kb-fs one, and the 400 is what
       // makes the route answer 400 rather than 500, so the type is asserted
       // here deliberately and the message pins which of the two ran.
       await expect(
         svc.fileBytesAtCommit(workspaceId, injected, addSha, 'after'),
-      ).rejects.toThrow(WorkflowValidationError);
+      ).rejects.toThrow(VersionControlValidationError);
       await expect(
         svc.fileBytesAtCommit(workspaceId, injected, addSha, 'after'),
       ).rejects.toThrow(/control character/);
@@ -188,7 +188,7 @@ describe('GitService.fileBytesAtCommit', () => {
   it('refuses a sha that is not a sha', async () => {
     await expect(
       svc.fileBytesAtCommit(workspaceId, 'knowledge-base/logo.png', 'HEAD', 'after'),
-    ).rejects.toThrow(WorkflowValidationError);
+    ).rejects.toThrow(VersionControlValidationError);
   });
 
   it('refuses a save that is not in this branch\'s history, and a made-up one, alike', async () => {
@@ -229,7 +229,7 @@ describe('GitService.fileBytesAtCommit', () => {
       .catch((e: unknown) => e);
     // `GitRunError` specifically, not just `Error`: the route reads the error's
     // TYPE to decide its status, so a future re-wrap into
-    // `WorkflowValidationError` would turn this into a 400 about the caller's
+    // `VersionControlValidationError` would turn this into a 400 about the caller's
     // request rather than the logged 500 a broken repository earns, and a bare
     // `toBeInstanceOf(Error)` would not notice.
     expect(err).toBeInstanceOf(GitRunError);

package/src/modules/{workflow → version-control}/git/__tests__/git.service.fileContentsAtCommit.test.ts

@@ -6,7 +6,7 @@ import fs from 'node:fs/promises';
 import path from 'node:path';
 import os from 'node:os';
 import type { WorkspaceService } from '../../../workspace/workspace.service.js';
-import { WorkflowHooks } from '../../workflow-hooks.js';
+import { VersionControlHooks } from '../../version-control-hooks.js';
 import { GitService } from '../git.service.js';
 
 const execFileAsync = promisify(execFile);
@@ -86,7 +86,7 @@ describe('GitService.fileContentsAtCommit', () => {
     const { workspaceDir, repo } = await seedWorkspace(root, workspaceId);
     const svc = new GitService(
       stubWorkspaceService(workspaceId, workspaceDir),
-      new WorkflowHooks(),
+      new VersionControlHooks(),
       testKbContext(),
     );
 
@@ -123,7 +123,7 @@ describe('GitService.fileContentsAtCommit', () => {
     const { workspaceDir } = await seedWorkspace(root, workspaceId);
     const svc = new GitService(
       stubWorkspaceService(workspaceId, workspaceDir),
-      new WorkflowHooks(),
+      new VersionControlHooks(),
       testKbContext(),
     );
     await expect(

package/src/modules/{workflow → version-control}/git/__tests__/git.service.forkPoint.test.ts

@@ -4,7 +4,7 @@ import fs from 'node:fs/promises';
 import path from 'node:path';
 import os from 'node:os';
 import { GitService } from '../git.service.js';
-import { gitOut, runGit, stubWorkflowHooks, stubWorkspaceService } from './git-test-helpers.js';
+import { gitOut, runGit, stubVersionControlHooks, stubWorkspaceService } from './git-test-helpers.js';
 
 /**
  * A change request shows only what its author changed. When someone edits the
@@ -52,7 +52,7 @@ describe('GitService — fork point, behind, and Update', () => {
 
     git = new GitService(
       stubWorkspaceService({ [WS]: path.dirname(repo) }),
-      stubWorkflowHooks(),
+      stubVersionControlHooks(),
       testKbContext(),
     );
   });

package/src/modules/{workflow → version-control}/git/__tests__/git.service.history-guards.test.ts

@@ -4,7 +4,7 @@ import fs from 'node:fs/promises';
 import path from 'node:path';
 import os from 'node:os';
 import { GitService } from '../git.service.js';
-import { runGit, gitOut, stubWorkflowHooks, stubWorkspaceService } from './git-test-helpers.js';
+import { runGit, gitOut, stubVersionControlHooks, stubWorkspaceService } from './git-test-helpers.js';
 
 // The history surfaces are per-file: their read gate checked exactly one
 // path, but a directory pathspec walks every child — and `show <ref>:<dir>`
@@ -34,7 +34,7 @@ describe('GitService history guards', () => {
     sha = await gitOut(repo, ['rev-parse', 'HEAD']);
     svc = new GitService(
       stubWorkspaceService({ [workspaceId]: workspaceDir }),
-      stubWorkflowHooks(),
+      stubVersionControlHooks(),
       testKbContext(),
     );
   });

package/src/modules/{workflow → version-control}/git/__tests__/git.service.list-branches.test.ts

@@ -6,7 +6,7 @@ import fs from 'node:fs/promises';
 import path from 'node:path';
 import os from 'node:os';
 import type { WorkspaceService } from '../../../workspace/workspace.service.js';
-import { WorkflowHooks } from '../../workflow-hooks.js';
+import { VersionControlHooks } from '../../version-control-hooks.js';
 import { GitService } from '../git.service.js';
 import { NodeGitRunner } from '../node-git-runner.js';
 import type { IGitRunner } from '../../../../shared/git.contract.js';
@@ -84,7 +84,7 @@ describe('GitService.listBranches', () => {
     await seedWorkspace(root, workspaceId);
     const svc = new GitService(
       stubWorkspaceService(workspaceId, path.join(root, workspaceId)),
-      new WorkflowHooks(),
+      new VersionControlHooks(),
       testKbContext(),
     );
 
@@ -103,7 +103,7 @@ describe('GitService.listBranches', () => {
     await seedWorkspace(root, workspaceId);
     const svc = new GitService(
       stubWorkspaceService(workspaceId, path.join(root, workspaceId)),
-      new WorkflowHooks(),
+      new VersionControlHooks(),
       testKbContext(),
     );
 
@@ -117,7 +117,7 @@ describe('GitService.listBranches', () => {
     const { upstream } = await seedWorkspace(root, workspaceId);
     const svc = new GitService(
       stubWorkspaceService(workspaceId, path.join(root, workspaceId)),
-      new WorkflowHooks(),
+      new VersionControlHooks(),
       testKbContext(),
     );
     // Prime the cache: first call fetches and populates lastImplicitFetchAt.
@@ -133,7 +133,7 @@ describe('GitService.listBranches', () => {
 
     const freshSvc = new GitService(
       stubWorkspaceService(workspaceId, path.join(root, workspaceId)),
-      new WorkflowHooks(),
+      new VersionControlHooks(),
       testKbContext(),
     );
     const branches = await freshSvc.listBranches(workspaceId);
@@ -147,7 +147,7 @@ describe('GitService.listBranches', () => {
 
     const svc = new GitService(
       stubWorkspaceService(workspaceId, path.join(root, workspaceId)),
-      new WorkflowHooks(),
+      new VersionControlHooks(),
       testKbContext(),
     );
 
@@ -161,7 +161,7 @@ describe('GitService.listBranches', () => {
     await seedWorkspace(root, workspaceId);
     const svc = new GitService(
       stubWorkspaceService(workspaceId, path.join(root, workspaceId)),
-      new WorkflowHooks(),
+      new VersionControlHooks(),
       testKbContext(),
     );
 
@@ -180,7 +180,7 @@ describe('GitService.listBranches', () => {
     const { upstream } = await seedWorkspace(root, workspaceId);
     const svc = new GitService(
       stubWorkspaceService(workspaceId, path.join(root, workspaceId)),
-      new WorkflowHooks(),
+      new VersionControlHooks(),
       testKbContext(),
     );
     // Simulate a just-completed clone — the workspace is freshly fetched.
@@ -206,7 +206,7 @@ describe('GitService.listBranches', () => {
     await seedWorkspace(root, workspaceId);
     const svc = new GitService(
       stubWorkspaceService(workspaceId, path.join(root, workspaceId)),
-      new WorkflowHooks(),
+      new VersionControlHooks(),
       testKbContext(),
     );
     const seen: string[][] = [];
@@ -223,7 +223,7 @@ describe('GitService.listBranches', () => {
     await seedWorkspace(root, workspaceId);
     const svc = new GitService(
       stubWorkspaceService(workspaceId, path.join(root, workspaceId)),
-      new WorkflowHooks(),
+      new VersionControlHooks(),
       testKbContext(),
     );
     svc.setBranchesListedListener(() => { throw new Error('listener blew up'); });
@@ -254,7 +254,7 @@ describe('GitService.listBranches', () => {
     const { runner, commands } = recordingRunner();
     const svc = new GitService(
       stubWorkspaceService(workspaceId, path.join(root, workspaceId)),
-      new WorkflowHooks(),
+      new VersionControlHooks(),
       testKbContext(),
       undefined,
       null,
@@ -317,7 +317,7 @@ describe('GitService.branchCounts', () => {
   }
 
   function svcFor(runner: IGitRunner = new NodeGitRunner(), kb = testKbContext()): GitService {
-    return new GitService(stubWorkspaceService(draftWs, path.join(root, draftWs)), new WorkflowHooks(), kb, undefined, null, runner);
+    return new GitService(stubWorkspaceService(draftWs, path.join(root, draftWs)), new VersionControlHooks(), kb, undefined, null, runner);
   }
 
   it('counts the workspace\'s own branch against its upstream, with rev-list for that branch alone', async () => {

package/src/modules/{workflow → version-control}/git/__tests__/git.service.listTrackedFiles.test.ts

@@ -6,7 +6,7 @@ import fs from 'node:fs/promises';
 import path from 'node:path';
 import os from 'node:os';
 import type { WorkspaceService } from '../../../workspace/workspace.service.js';
-import { WorkflowHooks } from '../../workflow-hooks.js';
+import { VersionControlHooks } from '../../version-control-hooks.js';
 import { GitService } from '../git.service.js';
 
 const execFileAsync = promisify(execFile);
@@ -63,7 +63,7 @@ async function makeSvc(root: string, workspaceId: string) {
   const { repo } = await seedWorkspace(root, workspaceId);
   const svc = new GitService(
     stubWorkspaceService(workspaceId, path.join(root, workspaceId)),
-    new WorkflowHooks(),
+    new VersionControlHooks(),
     testKbContext(),
   );
   return { svc, repo };