Skip to content

Commit 59bcfa6

Browse files
committed
fix: strip patchedDependencies from package.json in per-package CI checks
Individual package CI functions copy the root package.json and strip workspaces but not patchedDependencies, causing bun install to fail looking for the patches/ directory that isn't in the container. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 parent b8a4faf commit 59bcfa6

4 files changed

Lines changed: 5 additions & 5 deletions

File tree

.dagger/src/astro-opengraph-images.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@ export async function checkAstroOpengraphImages(
2121
.withExec([
2222
"bun",
2323
"-e",
24-
`const pkg = JSON.parse(await Bun.file('/workspace/package.json').text()); pkg.workspaces = ['packages/astro-opengraph-images', 'packages/eslint-config']; await Bun.write('/workspace/package.json', JSON.stringify(pkg));`,
24+
`const pkg = JSON.parse(await Bun.file('/workspace/package.json').text()); pkg.workspaces = ['packages/astro-opengraph-images', 'packages/eslint-config']; delete pkg.patchedDependencies; await Bun.write('/workspace/package.json', JSON.stringify(pkg));`,
2525
])
2626
// Package source
2727
.withDirectory(

.dagger/src/better-skill-capped.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,7 @@ export async function checkBetterSkillCapped(
3434
.withExec([
3535
"bun",
3636
"-e",
37-
`const pkg = JSON.parse(await Bun.file('/workspace/package.json').text()); pkg.workspaces = ['packages/better-skill-capped', 'packages/eslint-config']; await Bun.write('/workspace/package.json', JSON.stringify(pkg));`,
37+
`const pkg = JSON.parse(await Bun.file('/workspace/package.json').text()); pkg.workspaces = ['packages/better-skill-capped', 'packages/eslint-config']; delete pkg.patchedDependencies; await Bun.write('/workspace/package.json', JSON.stringify(pkg));`,
3838
])
3939
.withDirectory("/workspace/packages/better-skill-capped", mainSource)
4040
.withDirectory(

.dagger/src/sjer-red.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -54,7 +54,7 @@ function installDepsWithWebring(
5454
.withExec([
5555
"bun",
5656
"-e",
57-
`const pkg = JSON.parse(await Bun.file('/workspace/package.json').text()); pkg.workspaces = ['packages/sjer.red', 'packages/webring']; await Bun.write('/workspace/package.json', JSON.stringify(pkg));`,
57+
`const pkg = JSON.parse(await Bun.file('/workspace/package.json').text()); pkg.workspaces = ['packages/sjer.red', 'packages/webring']; delete pkg.patchedDependencies; await Bun.write('/workspace/package.json', JSON.stringify(pkg));`,
5858
])
5959
// Exclude sjer.red's own bun.lock — it resolves webring from npm.
6060
// Without any lockfile, bun generates fresh workspace resolution.
@@ -102,7 +102,7 @@ export async function checkSjerRed(source: Directory): Promise<string> {
102102
.withExec([
103103
"bun",
104104
"-e",
105-
`const pkg = JSON.parse(await Bun.file('/workspace/package.json').text()); pkg.workspaces = ['packages/sjer.red', 'packages/eslint-config', 'packages/webring']; await Bun.write('/workspace/package.json', JSON.stringify(pkg));`,
105+
`const pkg = JSON.parse(await Bun.file('/workspace/package.json').text()); pkg.workspaces = ['packages/sjer.red', 'packages/eslint-config', 'packages/webring']; delete pkg.patchedDependencies; await Bun.write('/workspace/package.json', JSON.stringify(pkg));`,
106106
])
107107
.withDirectory(
108108
"/workspace/packages/sjer.red",

.dagger/src/webring.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@ function getWebringContainer(source: Directory): Container {
2020
.withExec([
2121
"bun",
2222
"-e",
23-
`const pkg = JSON.parse(await Bun.file('/workspace/package.json').text()); pkg.workspaces = ['packages/webring', 'packages/eslint-config']; await Bun.write('/workspace/package.json', JSON.stringify(pkg));`,
23+
`const pkg = JSON.parse(await Bun.file('/workspace/package.json').text()); pkg.workspaces = ['packages/webring', 'packages/eslint-config']; delete pkg.patchedDependencies; await Bun.write('/workspace/package.json', JSON.stringify(pkg));`,
2424
])
2525
// Package source
2626
.withDirectory(

0 commit comments

Comments
 (0)