Skip to content

Commit 4641a0c

Browse files
ovflowdclaude
andcommitted
refactor: move platform workspaces from packages/ to apps/ and decouple site deps
Platform packages are internal deployment wrappers, not publishable artifacts. Moving them out of packages/ keeps the publish workflow's globs untouched and matches the repo convention (apps/* = internal, packages/* = published). Also removes @node-core/platform-vercel and @node-core/platform-cloudflare from apps/site's dependencies. They're now declared as optional peer dependencies so: - A standalone install (no DEPLOY_TARGET) pulls in zero platform deps. - Vercel's installCommand scopes install to @node-core/website and @node-core/platform-vercel (skipping Cloudflare/OpenNext deps). - The Cloudflare deploy workflow scopes install to @node-core/website and @node-core/platform-cloudflare (skipping @vercel/* deps). Co-Authored-By: Claude Opus 4.7 <[email protected]>
1 parent 0ac46dc commit 4641a0c

29 files changed

Lines changed: 100 additions & 90 deletions

.github/CODEOWNERS

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -27,10 +27,10 @@ turbo.json @nodejs/nodejs-website @nodejs/web-infra
2727
crowdin.yml @nodejs/web-infra
2828
apps/site/redirects.json @nodejs/web-infra
2929
apps/site/site.json @nodejs/web-infra
30-
packages/platform-cloudflare/wrangler.jsonc @nodejs/web-infra
31-
packages/platform-cloudflare/open-next.config.ts @nodejs/web-infra
32-
packages/platform-cloudflare/next.platform.config.mjs @nodejs/web-infra
33-
packages/platform-vercel/next.platform.config.mjs @nodejs/web-infra
30+
apps/cloudflare/wrangler.jsonc @nodejs/web-infra
31+
apps/cloudflare/open-next.config.ts @nodejs/web-infra
32+
apps/cloudflare/next.platform.config.mjs @nodejs/web-infra
33+
apps/vercel/next.platform.config.mjs @nodejs/web-infra
3434
apps/site/redirects.json @nodejs/web-infra
3535

3636
# Critical Documents

.github/workflows/tmp-cloudflare-open-next-deploy.yml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -50,18 +50,18 @@ jobs:
5050
cache: 'pnpm'
5151

5252
- name: Install packages
53-
run: pnpm install --frozen-lockfile
53+
run: pnpm install --frozen-lockfile --filter=@node-core/website... --filter=@node-core/platform-cloudflare...
5454

5555
- name: Build blog data
5656
working-directory: apps/site
5757
run: node --run build:blog-data
5858

5959
- name: Build open-next site
60-
working-directory: apps/site
60+
working-directory: apps/cloudflare
6161
run: node --run cloudflare:build:worker
6262

6363
- name: Deploy open-next site
64-
working-directory: apps/site
64+
working-directory: apps/cloudflare
6565
run: node --run cloudflare:deploy
6666
env:
6767
CF_WORKERS_SCRIPTS_API_TOKEN: ${{ secrets.CLOUDFLARE_API_TOKEN }}
File renamed without changes.
File renamed without changes.
File renamed without changes.
Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -14,12 +14,12 @@
1414
"repository": {
1515
"type": "git",
1616
"url": "https://github.com/nodejs/nodejs.org",
17-
"directory": "packages/platform-cloudflare"
17+
"directory": "apps/cloudflare"
1818
},
1919
"scripts": {
20-
"cloudflare:build:worker": "cd ../../apps/site && opennextjs-cloudflare build --openNextConfigPath ../../packages/platform-cloudflare/open-next.config.ts --config ../../packages/platform-cloudflare/wrangler.jsonc",
21-
"cloudflare:deploy": "cd ../../apps/site && opennextjs-cloudflare deploy --openNextConfigPath ../../packages/platform-cloudflare/open-next.config.ts --config ../../packages/platform-cloudflare/wrangler.jsonc",
22-
"cloudflare:preview": "cd ../../apps/site && wrangler dev --config ../../packages/platform-cloudflare/wrangler.jsonc",
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",
2323
"lint:types": "tsc --noEmit"
2424
},
2525
"dependencies": {

packages/platform-cloudflare/playwright.platform.config.d.ts renamed to apps/cloudflare/playwright.platform.config.d.ts

File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.

0 commit comments

Comments
 (0)