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

package/src/modules/{workflow → version-control}/git/__tests__/git.service.logForFile.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);
@@ -120,7 +120,7 @@ describe('GitService.logForFile', () => {
     const { workspaceDir, repo } = await seedWorkspace(root, workspaceId);
     const svc = new GitService(
       stubWorkspaceService(workspaceId, workspaceDir),
-      new WorkflowHooks(),
+      new VersionControlHooks(),
       testKbContext(),
     );
 
@@ -140,7 +140,7 @@ describe('GitService.logForFile', () => {
     const { workspaceDir, repo } = await seedWorkspace(root, workspaceId);
     const svc = new GitService(
       stubWorkspaceService(workspaceId, workspaceDir),
-      new WorkflowHooks(),
+      new VersionControlHooks(),
       testKbContext(),
     );
 
@@ -159,7 +159,7 @@ describe('GitService.logForFile', () => {
     const { workspaceDir, repo } = await seedWorkspace(root, workspaceId);
     const svc = new GitService(
       stubWorkspaceService(workspaceId, workspaceDir),
-      new WorkflowHooks(),
+      new VersionControlHooks(),
       testKbContext(),
     );
 
@@ -174,7 +174,7 @@ describe('GitService.logForFile', () => {
     const { workspaceDir, repo } = await seedWorkspace(root, workspaceId);
     const svc = new GitService(
       stubWorkspaceService(workspaceId, workspaceDir),
-      new WorkflowHooks(),
+      new VersionControlHooks(),
       testKbContext(),
     );
 
@@ -206,7 +206,7 @@ describe('GitService.logForFile', () => {
     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.mergeChangeRequest.test.ts

@@ -7,7 +7,7 @@ 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 { GitService } from '../git.service.js';
 import { mergeCommitSubject } from '../merge-commit.js';
 
@@ -96,7 +96,7 @@ describe('GitService.mergeChangeRequest', () => {
       await fs.writeFile(path.join(dir, 'feature.md'), 'new content\n');
     });
 
-    const git = new GitService(stubWorkspaceService(baseWsId, baseRepo), new WorkflowHooks(), testKbContext());
+    const git = new GitService(stubWorkspaceService(baseWsId, baseRepo), new VersionControlHooks(), testKbContext());
     const result = await git.mergeChangeRequest(
       baseWsId, 'alice/add', BASE, { subject: 'Add feature (#1)', body: 'Merged via Bevel' }, USER,
     );
@@ -134,7 +134,7 @@ describe('GitService.mergeChangeRequest', () => {
     await pushFeatureBranch(root, upstream, 'bob/first', async (dir) => {
       await fs.writeFile(path.join(dir, 'first.md'), 'first\n');
     });
-    const git = new GitService(stubWorkspaceService(baseWsId, baseRepo), new WorkflowHooks(), testKbContext());
+    const git = new GitService(stubWorkspaceService(baseWsId, baseRepo), new VersionControlHooks(), testKbContext());
     const first = await git.mergeChangeRequest(
       baseWsId, 'bob/first', BASE, { subject: mergeCommitSubject('First', 1), body: 'x' }, USER,
       { appliedChangeNumber: 1 },
@@ -181,7 +181,7 @@ describe('GitService.mergeChangeRequest', () => {
       await fs.writeFile(path.join(dir, 'feature.md'), 'new content\n');
     });
 
-    const git = new GitService(stubWorkspaceService(baseWsId, baseRepo), new WorkflowHooks(), testKbContext());
+    const git = new GitService(stubWorkspaceService(baseWsId, baseRepo), new VersionControlHooks(), testKbContext());
     const merge = () =>
       git.mergeChangeRequest(
         baseWsId, 'alice/add', BASE,
@@ -225,7 +225,7 @@ describe('GitService.mergeChangeRequest', () => {
     await fs.mkdir(path.join(root, staleWsId), { recursive: true });
     await runGit(root, ['clone', '-b', BASE, upstream, staleRepo]);
 
-    const git = new GitService(stubWorkspaceService(baseWsId, baseRepo), new WorkflowHooks(), testKbContext());
+    const git = new GitService(stubWorkspaceService(baseWsId, baseRepo), new VersionControlHooks(), testKbContext());
     const merged = await git.mergeChangeRequest(
       baseWsId, 'alice/add', BASE,
       { subject: mergeCommitSubject('Add feature', 7), body: 'Merged via Bevel' }, USER,
@@ -237,7 +237,7 @@ describe('GitService.mergeChangeRequest', () => {
     // Retired, as applying a change request retires it.
     await runGit(root, ['-C', upstream, 'branch', '-D', 'alice/add']);
 
-    const stale = new GitService(stubWorkspaceService(staleWsId, staleRepo), new WorkflowHooks(), testKbContext());
+    const stale = new GitService(stubWorkspaceService(staleWsId, staleRepo), new VersionControlHooks(), testKbContext());
     expect(await stale.appliedMergeCommitOnTarget(staleWsId, BASE, 7)).toBe(merged.mergeCommit);
     // A number nothing on the target was merged under owns nothing.
     expect(await stale.appliedMergeCommitOnTarget(staleWsId, BASE, 8)).toBeNull();
@@ -248,7 +248,7 @@ describe('GitService.mergeChangeRequest', () => {
     await pushFeatureBranch(root, upstream, 'alice/add', async (dir) => {
       await fs.writeFile(path.join(dir, 'feature.md'), 'new content\n');
     });
-    const git = new GitService(stubWorkspaceService(baseWsId, baseRepo), new WorkflowHooks(), testKbContext());
+    const git = new GitService(stubWorkspaceService(baseWsId, baseRepo), new VersionControlHooks(), testKbContext());
     const merged = await git.mergeChangeRequest(
       baseWsId, 'alice/add', BASE,
       { subject: mergeCommitSubject('Add feature', 7), body: 'Merged via Bevel' }, USER,
@@ -293,7 +293,7 @@ describe('GitService.mergeChangeRequest', () => {
     const sha = (await gitOut(old, ['rev-parse', 'HEAD'])).trim();
     expect((await gitOut(old, ['log', '-1', '--format=%s'])).trim()).toBe('Add feature');
 
-    const git = new GitService(stubWorkspaceService(baseWsId, baseRepo), new WorkflowHooks(), testKbContext());
+    const git = new GitService(stubWorkspaceService(baseWsId, baseRepo), new VersionControlHooks(), testKbContext());
     // With the title: the commit is the request's own, on the target and by sha.
     expect(await git.appliedMergeCommitOnTarget(baseWsId, BASE, 5, title)).toBe(sha);
     expect(await git.appliedChangeShas(baseWsId, { number: 5, mergeSha: sha, title }))
@@ -313,7 +313,7 @@ describe('GitService.mergeChangeRequest', () => {
     await pushFeatureBranch(root, upstream, 'alice/add', async (dir) => {
       await fs.writeFile(path.join(dir, 'feature.md'), 'new content\n');
     });
-    const git = new GitService(stubWorkspaceService(baseWsId, baseRepo), new WorkflowHooks(), testKbContext());
+    const git = new GitService(stubWorkspaceService(baseWsId, baseRepo), new VersionControlHooks(), testKbContext());
     const merged = await git.mergeChangeRequest(
       baseWsId, 'alice/add', BASE,
       { subject: mergeCommitSubject('Fix crash (#42)', 43), body: 'x' }, USER,
@@ -342,7 +342,7 @@ describe('GitService.mergeChangeRequest', () => {
       await fs.writeFile(path.join(dir, 'feature.md'), 'new content\n');
     });
 
-    const git = new GitService(stubWorkspaceService(baseWsId, baseRepo), new WorkflowHooks(), testKbContext());
+    const git = new GitService(stubWorkspaceService(baseWsId, baseRepo), new VersionControlHooks(), testKbContext());
     const merged = await git.mergeChangeRequest(
       baseWsId, 'alice/add', BASE,
       { subject: mergeCommitSubject('Add feature', 7), body: 'Merged via Bevel' }, USER,
@@ -373,7 +373,7 @@ describe('GitService.mergeChangeRequest', () => {
     await pushFeatureBranch(root, upstream, 'bob/first', async (dir) => {
       await fs.writeFile(path.join(dir, 'first.md'), 'first\n');
     });
-    const git = new GitService(stubWorkspaceService(baseWsId, baseRepo), new WorkflowHooks(), testKbContext());
+    const git = new GitService(stubWorkspaceService(baseWsId, baseRepo), new VersionControlHooks(), testKbContext());
     // #1's merge commit, whose BODY mentions #9 — the shape `--grep` matches and
     // the subject check must reject.
     await git.mergeChangeRequest(
@@ -408,7 +408,7 @@ describe('GitService.mergeChangeRequest', () => {
     await pushFeatureBranch(root, upstream, 'alice/add', async (dir) => {
       await fs.writeFile(path.join(dir, 'feature.md'), 'new content\n');
     });
-    const git = new GitService(stubWorkspaceService(baseWsId, baseRepo), new WorkflowHooks(), testKbContext());
+    const git = new GitService(stubWorkspaceService(baseWsId, baseRepo), new VersionControlHooks(), testKbContext());
     await expect(
       git.mergeChangeRequest(
         baseWsId, 'alice/add', BASE, { subject: 'Add feature\n\nand a note (#3)', body: 'x' }, USER,
@@ -429,7 +429,7 @@ describe('GitService.mergeChangeRequest', () => {
     await pushFeatureBranch(root, upstream, 'alice/add', async (dir) => {
       await fs.writeFile(path.join(dir, 'feature.md'), 'new content\n');
     });
-    const git = new GitService(stubWorkspaceService(baseWsId, baseRepo), new WorkflowHooks(), testKbContext());
+    const git = new GitService(stubWorkspaceService(baseWsId, baseRepo), new VersionControlHooks(), testKbContext());
     const result = await git.mergeChangeRequest(
       baseWsId, 'alice/add', BASE,
       { subject: mergeCommitSubject('Add feature\n\nand a note', 4), body: 'x' }, USER,
@@ -464,7 +464,7 @@ describe('GitService.mergeChangeRequest', () => {
     await runGit(root, ['clone', '-b', 'tmp-base-advance', upstream, advancer]);
     await runGit(advancer, ['push', 'origin', 'tmp-base-advance:' + BASE]);
 
-    const git = new GitService(stubWorkspaceService(baseWsId, baseRepo), new WorkflowHooks(), testKbContext());
+    const git = new GitService(stubWorkspaceService(baseWsId, baseRepo), new VersionControlHooks(), testKbContext());
     const result = await git.mergeChangeRequest(
       baseWsId, 'alice/edit', BASE, { subject: 'Edit (#2)', body: 'x' }, USER,
     );
@@ -498,7 +498,7 @@ describe('GitService.mergeChangeRequest', () => {
       // A save that has not been shared yet, exactly as a file tool leaves it.
       await fs.writeFile(path.join(baseRepo, 'base.md'), 'edited but not shared\n');
 
-      const git = new GitService(stubWorkspaceService(baseWsId, baseRepo), new WorkflowHooks(), testKbContext());
+      const git = new GitService(stubWorkspaceService(baseWsId, baseRepo), new VersionControlHooks(), testKbContext());
       await expect(
         git.mergeChangeRequest(
           baseWsId, 'alice/add', BASE, { subject: 'Add (#1)', body: 'x' }, USER,
@@ -523,7 +523,7 @@ describe('GitService.mergeChangeRequest', () => {
       await runGit(baseRepo, ['add', '-A']);
       await runGit(baseRepo, ['commit', '-m', 'local only']);
 
-      const git = new GitService(stubWorkspaceService(baseWsId, baseRepo), new WorkflowHooks(), testKbContext());
+      const git = new GitService(stubWorkspaceService(baseWsId, baseRepo), new VersionControlHooks(), testKbContext());
       await expect(
         git.mergeChangeRequest(
           baseWsId, 'alice/add', BASE, { subject: 'Add (#1)', body: 'x' }, USER,
@@ -538,7 +538,7 @@ describe('GitService.mergeChangeRequest', () => {
       await pushFeatureBranch(root, upstream, 'alice/add', async (dir) => {
         await fs.writeFile(path.join(dir, 'feature.md'), 'new content\n');
       });
-      const git = new GitService(stubWorkspaceService(baseWsId, baseRepo), new WorkflowHooks(), testKbContext());
+      const git = new GitService(stubWorkspaceService(baseWsId, baseRepo), new VersionControlHooks(), testKbContext());
       const result = await git.mergeChangeRequest(
         baseWsId, 'alice/add', BASE, { subject: 'Add (#1)', body: 'x' }, USER,
         { requireCleanTarget: true },
@@ -571,7 +571,7 @@ describe('GitService.mergeChangeRequest', () => {
       const publishedTip = (await gitOut(advancer, ['rev-parse', 'HEAD'])).trim();
       expect(staleHead).not.toBe(publishedTip);
 
-      const git = new GitService(stubWorkspaceService(baseWsId, baseRepo), new WorkflowHooks(), testKbContext());
+      const git = new GitService(stubWorkspaceService(baseWsId, baseRepo), new VersionControlHooks(), testKbContext());
       const seen: { sha: string; changedPaths: string[] }[] = [];
       const result = await git.mergeChangeRequest(
         baseWsId, 'alice/add', BASE, { subject: 'Add (#1)', body: 'x' }, USER,
@@ -589,7 +589,7 @@ describe('GitService.mergeChangeRequest', () => {
       await pushFeatureBranch(root, upstream, 'alice/add', async (dir) => {
         await fs.writeFile(path.join(dir, 'feature.md'), 'new content\n');
       });
-      const git = new GitService(stubWorkspaceService(baseWsId, baseRepo), new WorkflowHooks(), testKbContext());
+      const git = new GitService(stubWorkspaceService(baseWsId, baseRepo), new VersionControlHooks(), testKbContext());
       await expect(
         git.mergeChangeRequest(
           baseWsId, 'alice/add', BASE, { subject: 'Add (#1)', body: 'x' }, USER,
@@ -614,7 +614,7 @@ describe('GitService.mergeChangeRequest', () => {
         await runGit(dir, ['mv', 'roles.yaml', 'people.yaml']);
       });
 
-      const git = new GitService(stubWorkspaceService(baseWsId, baseRepo), new WorkflowHooks(), testKbContext());
+      const git = new GitService(stubWorkspaceService(baseWsId, baseRepo), new VersionControlHooks(), testKbContext());
       const seen: string[][] = [];
       const result = await git.mergeChangeRequest(
         baseWsId, 'alice/rename', BASE, { subject: 'Rename (#1)', body: 'x' }, USER,

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

@@ -7,8 +7,8 @@ import { GitService } from '../git.service.js';
 import { NodeGitRunner } from '../node-git-runner.js';
 import { GitRunError, type IGitRunner } from '../../../../shared/git.contract.js';
 import type { ReservedPushOps } from '@bevel-software/platform-shared';
-import { PullRebaseConflictError, WorkflowDomainError, WorkflowValidationError } from '../../../../shared/domain-errors.js';
-import { runGit, gitOut, stubWorkflowHooks, stubWorkspaceService } from './git-test-helpers.js';
+import { PullRebaseConflictError, VersionControlDomainError, VersionControlValidationError } from '../../../../shared/domain-errors.js';
+import { runGit, gitOut, stubVersionControlHooks, stubWorkspaceService } from './git-test-helpers.js';
 
 /**
  * `mergeIntoBranch`: the agent's merge, over real git. A conflict is described
@@ -24,7 +24,7 @@ const USER = { id: 'u1', email: 'ada@x.io', name: 'Ada Lovelace' };
 const allow = async (): Promise<void> => {};
 /**
  * The smallest publish a caller can bring: push, and on a rejection replay
- * onto the moved branch (the merge kept) and push once more. The workflow
+ * onto the moved branch (the merge kept) and push once more. The version-control service
  * facade brings its `pushWithRecovery` here; this stands in for it to show
  * the moves work inside the merge's reservation.
  */
@@ -107,7 +107,7 @@ describe('GitService.mergeIntoBranch', () => {
   });
 
   const service = () =>
-    new GitService(stubWorkspaceService({ [workspaceId]: path.join(root, workspaceId) }), stubWorkflowHooks(), testKbContext());
+    new GitService(stubWorkspaceService({ [workspaceId]: path.join(root, workspaceId) }), stubVersionControlHooks(), testKbContext());
 
   it('merges a target that moved into the draft as a merge commit by the user, and pushes nothing', async () => {
     const { repo, pusher } = await seed(root);
@@ -275,7 +275,7 @@ describe('GitService.mergeIntoBranch', () => {
     const svc = service();
     await expect(
       svc.mergeIntoBranch(workspaceId, DRAFT, TARGET, USER, { subject: 's', body: 'b', resolutions: [{ path: 'a.md' }], authorizeResolutions: allow }),
-    ).rejects.toBeInstanceOf(WorkflowValidationError);
+    ).rejects.toBeInstanceOf(VersionControlValidationError);
     await expect(
       svc.mergeIntoBranch(workspaceId, DRAFT, TARGET, USER, {
         subject: 's',
@@ -283,7 +283,7 @@ describe('GitService.mergeIntoBranch', () => {
         resolutions: [{ path: 'a.md', content: 'x', choose: 'ours' }],
         authorizeResolutions: allow,
       }),
-    ).rejects.toBeInstanceOf(WorkflowValidationError);
+    ).rejects.toBeInstanceOf(VersionControlValidationError);
   });
 
   it('refuses resolutions that nobody authorizes, before touching git', async () => {
@@ -306,9 +306,9 @@ describe('GitService.mergeIntoBranch', () => {
     const err = await service()
       .mergeIntoBranch(workspaceId, DRAFT, TARGET, USER, { subject: 's', body: 'b' })
       .catch((e: unknown) => e);
-    expect(err).toBeInstanceOf(WorkflowDomainError);
-    expect((err as WorkflowDomainError).status).toBe(409);
-    expect((err as WorkflowDomainError).payload).toMatchObject({ kind: 'branch-busy' });
+    expect(err).toBeInstanceOf(VersionControlDomainError);
+    expect((err as VersionControlDomainError).status).toBe(409);
+    expect((err as VersionControlDomainError).payload).toMatchObject({ kind: 'branch-busy' });
     expect(await fs.readFile(path.join(repo, 'shared.md'), 'utf-8')).toBe('unsaved edit\n');
   });
 
@@ -458,11 +458,11 @@ describe('GitService.mergeIntoBranch', () => {
             mergeHead: await gitOut(repo, ['rev-parse', '-q', '--verify', 'MERGE_HEAD']).then(() => true, () => false),
             sharedOnDisk: await fs.readFile(path.join(repo, 'shared.md'), 'utf-8'),
           });
-          throw new WorkflowDomainError('not yours to settle', 403);
+          throw new VersionControlDomainError('not yours to settle', 403);
         },
       })
       .catch((e: unknown) => e);
-    expect(err).toBeInstanceOf(WorkflowDomainError);
+    expect(err).toBeInstanceOf(VersionControlDomainError);
     // Asked with the target's tip as fetched by this very merge.
     expect(asked).toEqual([{ merge: { ours, theirs }, head: ours, mergeHead: false, sharedOnDisk: 'line 1\nline 2 from draft\nline 3\n' }]);
     expect((await gitOut(repo, ['rev-parse', 'HEAD'])).trim()).toBe(ours);
@@ -487,7 +487,7 @@ describe('GitService.mergeIntoBranch', () => {
     } as unknown as IGitRunner;
     const svc = new GitService(
       stubWorkspaceService({ [workspaceId]: path.join(root, workspaceId) }),
-      stubWorkflowHooks(),
+      stubVersionControlHooks(),
       testKbContext(),
       undefined,
       null,
@@ -643,7 +643,7 @@ describe('GitService.readFileAtBranch', () => {
     const { repo, pusher } = await seed(root);
     const theirs = await pushOn(pusher, TARGET, { 'other.md': 'other, changed on target\n', 'docs/a.md': 'a\n' }, 'target edits');
     await runGit(repo, ['fetch', '-q', 'origin']);
-    const svc = new GitService(stubWorkspaceService({ [workspaceId]: path.join(root, workspaceId) }), stubWorkflowHooks(), testKbContext());
+    const svc = new GitService(stubWorkspaceService({ [workspaceId]: path.join(root, workspaceId) }), stubVersionControlHooks(), testKbContext());
 
     expect(await svc.readFileAtBranch(workspaceId, 'knowledge-base/other.md', TARGET)).toEqual({
       sha: theirs,
@@ -660,7 +660,7 @@ describe('GitService.readFileAtBranch', () => {
     const asked: string[][] = [];
     const refuse = async (commits: string[]): Promise<void> => {
       asked.push(commits);
-      throw new WorkflowDomainError('not yours to read', 403);
+      throw new VersionControlDomainError('not yours to read', 403);
     };
     await expect(svc.readFileAtBranch(workspaceId, 'docs', TARGET, refuse)).rejects.toThrow(/not yours to read/);
     await expect(svc.readFileAtBranch(workspaceId, 'other.md', TARGET, refuse)).rejects.toThrow(/not yours to read/);
@@ -673,7 +673,7 @@ describe('GitService.readFileAtBranch', () => {
 
   it('reads the PUBLISHED tip, not a local head that has moved past it', async () => {
     const { repo } = await seed(root);
-    const svc = new GitService(stubWorkspaceService({ [workspaceId]: path.join(root, workspaceId) }), stubWorkflowHooks(), testKbContext());
+    const svc = new GitService(stubWorkspaceService({ [workspaceId]: path.join(root, workspaceId) }), stubVersionControlHooks(), testKbContext());
     const published = (await gitOut(repo, ['rev-parse', `origin/${DRAFT}`])).trim();
     const local = await commitLocally(repo, { 'other.md': 'not shared yet\n' }, 'local only');
     const asked: string[][] = [];
@@ -702,7 +702,7 @@ describe('GitService.readFileAtBranch', () => {
         return inner.run(cwd, args, opts as never);
       },
     } as unknown as IGitRunner;
-    const svc = new GitService(stubWorkspaceService({ [workspaceId]: path.join(root, workspaceId) }), stubWorkflowHooks(), testKbContext(), undefined, null, gitRunner);
+    const svc = new GitService(stubWorkspaceService({ [workspaceId]: path.join(root, workspaceId) }), stubVersionControlHooks(), testKbContext(), undefined, null, gitRunner);
     const asked: string[][] = [];
     const authorize = async (commits: string[]): Promise<void> => { asked.push(commits); };
     for (const f of [

package/src/modules/{workflow → version-control}/git/__tests__/git.service.pendingChanges.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 { parsePorcelainZ } from '../git.service.js';
 
@@ -72,7 +72,7 @@ async function makeSvc(root: string, workspaceId: string) {
   const { workspaceDir, repo } = await seedWorkspace(root, workspaceId);
   const svc = new GitService(
     stubWorkspaceService(workspaceId, workspaceDir),
-    new WorkflowHooks(),
+    new VersionControlHooks(),
     testKbContext(),
   );
   return { svc, repo };

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

@@ -7,10 +7,10 @@ import path from 'node:path';
 import os from 'node:os';
 import type { AuthUser } from '@bevel-software/platform-shared';
 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 post-merge refresh path: `WorkflowService.mergeChangeRequest` merges a
+ * The post-merge refresh path: `VersionControlService.mergeChangeRequest` merges a
  * change request onto `origin/<base>` and then pulls the BASE branch's own
  * workspace so its working tree (what every file read serves) isn't left behind
  * the remote.
@@ -109,7 +109,7 @@ describe('post-merge refresh of the target branch workspace', () => {
         [sourceWsId]: path.dirname(sourceRepo),
         [baseWsId]: path.dirname(baseRepo),
       }),
-      stubWorkflowHooks(),
+      stubVersionControlHooks(),
       testKbContext({ kbDirName: KB_DIR }),
     );
 
@@ -163,7 +163,7 @@ describe('post-merge refresh of the target branch workspace', () => {
         [sourceWsId]: path.dirname(sourceRepo),
         [baseWsId]: path.dirname(baseRepo),
       }),
-      stubWorkflowHooks(),
+      stubVersionControlHooks(),
       testKbContext({ kbDirName: KB_DIR }),
     );
 
@@ -254,7 +254,7 @@ describe('post-merge refresh of the target branch workspace', () => {
         [sourceWsId]: path.dirname(sourceRepo),
         [baseWsId]: path.dirname(baseRepo),
       }),
-      stubWorkflowHooks(),
+      stubVersionControlHooks(),
       testKbContext({ kbDirName: KB_DIR }),
     );
 

package/src/modules/{workflow → version-control}/git/__tests__/git.service.prFetchFailure.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);
@@ -71,7 +71,7 @@ function service(): GitService {
   const workspaces = {
     getWorkspacePath: async () => path.dirname(repo),
   } as unknown as WorkspaceService;
-  return new GitService(workspaces, new WorkflowHooks(), testKbContext());
+  return new GitService(workspaces, new VersionControlHooks(), testKbContext());
 }
 
 /** Everything the service logged at warn level, as one string per line. */

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

@@ -5,7 +5,7 @@ import path from 'node:path';
 import os from 'node:os';
 import { GitService } from '../git.service.js';
 import { PullRebaseConflictError } from '../../../../shared/domain-errors.js';
-import { runGit, gitOut, stubWorkflowHooks, stubWorkspaceService } from './git-test-helpers.js';
+import { runGit, gitOut, stubVersionControlHooks, stubWorkspaceService } from './git-test-helpers.js';
 
 /** See git.service.createBranch.test.ts — same prod-mirroring layout. */
 async function seedWorkspace(root: string, workspaceId: string): Promise<{
@@ -85,7 +85,7 @@ describe('GitService.pull', () => {
 
     const svc = new GitService(
       stubWorkspaceService({ [workspaceId]: path.join(root, workspaceId) }),
-      stubWorkflowHooks(),
+      stubVersionControlHooks(),
       testKbContext(),
     );
 
@@ -128,7 +128,7 @@ describe('GitService.pull', () => {
 
     const svc = new GitService(
       stubWorkspaceService({ [workspaceId]: path.join(root, workspaceId) }),
-      stubWorkflowHooks(),
+      stubVersionControlHooks(),
       testKbContext(),
     );
 
@@ -174,7 +174,7 @@ describe('GitService.pull', () => {
 
     const svc = new GitService(
       stubWorkspaceService({ [workspaceId]: path.join(root, workspaceId) }),
-      stubWorkflowHooks(),
+      stubVersionControlHooks(),
       testKbContext(),
     );
 
@@ -219,7 +219,7 @@ describe('GitService.pull', () => {
 
     const svc = new GitService(
       stubWorkspaceService({ [workspaceId]: path.join(root, workspaceId) }),
-      stubWorkflowHooks(),
+      stubVersionControlHooks(),
       testKbContext(),
     );
 
@@ -247,7 +247,7 @@ describe('GitService.pull', () => {
 
     const svc = new GitService(
       stubWorkspaceService({ [workspaceId]: path.join(root, workspaceId) }),
-      stubWorkflowHooks(),
+      stubVersionControlHooks(),
       testKbContext(),
     );
 
@@ -269,7 +269,7 @@ describe('GitService.pull', () => {
 
     const svc = new GitService(
       stubWorkspaceService({ [workspaceId]: path.join(root, workspaceId) }),
-      stubWorkflowHooks(),
+      stubVersionControlHooks(),
       testKbContext(),
     );
 
@@ -284,7 +284,7 @@ describe('GitService.pull', () => {
   // both rewrote the same file, so replaying the local commit conflicts. The
   // pull must (a) abort the rebase so the clone isn't left in a detached
   // apply state, (b) surface a TYPED error naming the contested paths so the
-  // workflow layer can queue background recovery, and (c) leave the local
+  // version-control layer can queue background recovery, and (c) leave the local
   // commit intact (it's someone's saved content).
   it('throws PullRebaseConflictError with the conflicted paths and aborts the rebase cleanly', async () => {
     const { upstream, repo } = await seedWorkspace(root, workspaceId);
@@ -311,7 +311,7 @@ describe('GitService.pull', () => {
 
     const svc = new GitService(
       stubWorkspaceService({ [workspaceId]: path.join(root, workspaceId) }),
-      stubWorkflowHooks(),
+      stubVersionControlHooks(),
       testKbContext(),
     );
 
@@ -338,7 +338,7 @@ describe('GitService.pull', () => {
     const { repo } = await seedWorkspace(root, workspaceId);
     const svc = new GitService(
       stubWorkspaceService({ [workspaceId]: path.join(root, workspaceId) }),
-      stubWorkflowHooks(),
+      stubVersionControlHooks(),
       testKbContext(),
     );
 
@@ -364,7 +364,7 @@ describe('GitService.pull', () => {
     const { repo } = await seedWorkspace(root, workspaceId);
     const svc = new GitService(
       stubWorkspaceService({ [workspaceId]: path.join(root, workspaceId) }),
-      stubWorkflowHooks(),
+      stubVersionControlHooks(),
       testKbContext(),
     );
 
@@ -474,7 +474,7 @@ describe('GitService.pull — an unpushed merge commit', () => {
   function svcFor(): GitService {
     return new GitService(
       stubWorkspaceService({ [workspaceId]: path.join(root, workspaceId) }),
-      stubWorkflowHooks(),
+      stubVersionControlHooks(),
       testKbContext(),
     );
   }

package/src/modules/{workflow → version-control}/git/__tests__/git.service.readFileAtRef.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);
@@ -59,7 +59,7 @@ describe('GitService.readFileAtRef', () => {
   function makeSvc(workspaceDir: string) {
     return new GitService(
       stubWorkspaceService(workspaceId, workspaceDir),
-      new WorkflowHooks(),
+      new VersionControlHooks(),
       testKbContext({ kbDirName: PROCESS_MAP_DIR }),
     );
   }
@@ -115,7 +115,7 @@ describe('GitService.commitAtRef', () => {
   function makeSvc(workspaceDir: string) {
     return new GitService(
       stubWorkspaceService(workspaceId, workspaceDir),
-      new WorkflowHooks(),
+      new VersionControlHooks(),
       testKbContext({ kbDirName: PROCESS_MAP_DIR }),
     );
   }

package/src/modules/{workflow → version-control}/git/__tests__/git.service.revertPathsAndPush.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';
 
 /**
  * `GitService.revertPathsAndPush` — the git half of "Delete folder and its
@@ -59,7 +59,7 @@ async function seed(root: string, branches: string[]): Promise<Fixture> {
     await runGit(repo, ['config', 'core.autocrlf', 'false']);
     repos[b] = repo;
   }
-  const svc = new GitService(stubWorkspaceService(dirs), stubWorkflowHooks(), testKbContext({ kbDirName: KB }));
+  const svc = new GitService(stubWorkspaceService(dirs), stubVersionControlHooks(), testKbContext({ kbDirName: KB }));
   return { upstream, base, repos, svc };
 }
 

package/src/modules/{workflow → version-control}/git/__tests__/git.service.status.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);
@@ -81,7 +81,7 @@ describe('GitService.status — hasUpstream', () => {
 
   it('reports hasUpstream=true on a tracked branch (cloned from origin)', async () => {
     const { workspaceDir } = await seedWorkspace(root, workspaceId);
-    const svc = new GitService(stubWorkspaceService(workspaceId, workspaceDir), new WorkflowHooks(), testKbContext());
+    const svc = new GitService(stubWorkspaceService(workspaceId, workspaceDir), new VersionControlHooks(), testKbContext());
 
     const status = await svc.status(workspaceId);
     expect(status.branch).toBe('current-company-state');
@@ -97,7 +97,7 @@ describe('GitService.status — hasUpstream', () => {
     await runGit(repo, ['checkout', '-b', 'alice/unpublished-draft']);
     await runGit(repo, ['commit', '--allow-empty', '-m', 'local-only edit']);
 
-    const svc = new GitService(stubWorkspaceService(workspaceId, workspaceDir), new WorkflowHooks(), testKbContext());
+    const svc = new GitService(stubWorkspaceService(workspaceId, workspaceDir), new VersionControlHooks(), testKbContext());
     const status = await svc.status(workspaceId);
 
     expect(status.branch).toBe('alice/unpublished-draft');
@@ -115,7 +115,7 @@ describe('GitService.status — hasUpstream', () => {
     await runGit(repo, ['commit', '--allow-empty', '-m', 'two']);
     await runGit(repo, ['push', '-u', 'origin', 'alice/published-draft']);
 
-    const svc = new GitService(stubWorkspaceService(workspaceId, workspaceDir), new WorkflowHooks(), testKbContext());
+    const svc = new GitService(stubWorkspaceService(workspaceId, workspaceDir), new VersionControlHooks(), testKbContext());
     const status = await svc.status(workspaceId);
 
     expect(status.branch).toBe('alice/published-draft');