Skip to content

Commit ecf659d

Browse files
ovflowdclaude
andcommitted
chore(cloudflare): drop cd-prefixed scripts and ignore wrangler state dir
Replace `cd ../site && ...` in the three cloudflare scripts with `pnpm --filter=@node-core/website exec ...` so pnpm sets cwd transparently (per Dario's review feedback). Ignore apps/cloudflare/.wrangler, which is where wrangler v4 writes its state dir relative to the config file. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
1 parent 93751a4 commit ecf659d

3 files changed

Lines changed: 4 additions & 5 deletions

File tree

.gitignore

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -43,6 +43,7 @@ dist/
4343
# Cloudflare Build Output
4444
apps/site/.open-next
4545
apps/site/.wrangler
46+
apps/cloudflare/.wrangler
4647

4748
## Playwright
4849
test-results

apps/cloudflare/package.json

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -17,9 +17,9 @@
1717
"directory": "apps/cloudflare"
1818
},
1919
"scripts": {
20-
"cloudflare:build:worker": "cd ../site && opennextjs-cloudflare build --openNextConfigPath ../cloudflare/open-next.config.ts --config ../cloudflare/wrangler.jsonc",
21-
"cloudflare:deploy": "cd ../site && opennextjs-cloudflare deploy --openNextConfigPath ../cloudflare/open-next.config.ts --config ../cloudflare/wrangler.jsonc",
22-
"cloudflare:preview": "cd ../site && wrangler dev --config ../cloudflare/wrangler.jsonc",
20+
"cloudflare:build:worker": "pnpm --filter=@node-core/website exec opennextjs-cloudflare build --openNextConfigPath ../cloudflare/open-next.config.ts --config ../cloudflare/wrangler.jsonc",
21+
"cloudflare:deploy": "pnpm --filter=@node-core/website exec opennextjs-cloudflare deploy --openNextConfigPath ../cloudflare/open-next.config.ts --config ../cloudflare/wrangler.jsonc",
22+
"cloudflare:preview": "pnpm --filter=@node-core/website exec wrangler dev --config ../cloudflare/wrangler.jsonc",
2323
"lint:types": "tsc --noEmit"
2424
},
2525
"dependencies": {

apps/cloudflare/wrangler.jsonc

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -31,8 +31,6 @@
3131
"head_sampling_rate": 1,
3232
},
3333
"build": {
34-
// Run the asset polyfiller from apps/site so that `pages`, `snippets`, and
35-
// the `.open-next` output directory resolve against the Next.js app.
3634
"cwd": "../site",
3735
"command": "wrangler-build-time-fs-assets-polyfilling --assets pages --assets snippets --assets-output-dir .open-next/assets",
3836
},

0 commit comments

Comments
 (0)