Skip to content

Commit 6dcd148

Browse files
Merge pull request #700 from corbitsdev/cl-7577-delete-13-empty-husk-packages
Keep deleted husk packages off the tree
2 parents 364061b + db5a845 commit 6dcd148

2 files changed

Lines changed: 58 additions & 0 deletions

File tree

‎scripts/checks/replaced-paths.txt‎

Lines changed: 18 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -13,3 +13,21 @@ scripts/checks/growth.ts
1313
scripts/checks/manifest.ts
1414
scripts/checks/routes.ts
1515
scripts/checks/lib/modules.ts
16+
packages/agent-workflow-authoring
17+
packages/cli
18+
packages/echo
19+
packages/folded-run-one-shot
20+
packages/folded-runs
21+
packages/hub-client
22+
packages/routines
23+
packages/routines-tools
24+
packages/sidecar-placement
25+
packages/task-dispatch-tools
26+
packages/task-planner
27+
packages/tasks
28+
packages/tasks-ui
29+
packages/workflow-catalog
30+
packages/workflow-deploy-source
31+
packages/workflow-freeze
32+
packages/workflow-host-actions
33+
packages/workflow-source

‎scripts/checks/test/deletion.test.ts‎

Lines changed: 40 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,5 @@
1+
import { existsSync, readFileSync } from "node:fs";
2+
import path from "node:path";
13
import { expect, test } from "bun:test";
24
import { auditReplacedPaths, parseLedger } from "../deletion";
35

@@ -27,3 +29,41 @@ test("only the surviving paths are reported", () => {
2729
expect(report.violations).toHaveLength(1);
2830
expect(report.violations[0]).toContain("alive.ts");
2931
});
32+
33+
// Empty leftover package dirs from the installer review (13) plus five more
34+
// found at pickup. Git already dropped them; the replacement ledger keeps
35+
// them from coming back as workspace members.
36+
const DELETED_HUSK_PACKAGES = [
37+
"packages/agent-workflow-authoring",
38+
"packages/cli",
39+
"packages/echo",
40+
"packages/folded-run-one-shot",
41+
"packages/folded-runs",
42+
"packages/hub-client",
43+
"packages/routines",
44+
"packages/routines-tools",
45+
"packages/sidecar-placement",
46+
"packages/task-dispatch-tools",
47+
"packages/task-planner",
48+
"packages/tasks",
49+
"packages/tasks-ui",
50+
"packages/workflow-catalog",
51+
"packages/workflow-deploy-source",
52+
"packages/workflow-freeze",
53+
"packages/workflow-host-actions",
54+
"packages/workflow-source",
55+
];
56+
57+
test("deleted husk packages stay on the replacement ledger and are gone", () => {
58+
const repoRoot = path.resolve(import.meta.dir, "../../..");
59+
const ledger = parseLedger(
60+
readFileSync(
61+
path.join(repoRoot, "scripts/checks/replaced-paths.txt"),
62+
"utf8",
63+
),
64+
);
65+
for (const husk of DELETED_HUSK_PACKAGES) {
66+
expect(ledger).toContain(husk);
67+
expect(existsSync(path.join(repoRoot, husk))).toBe(false);
68+
}
69+
});

0 commit comments

Comments
 (0)