Skip to content

Commit 63d3636

Browse files
committed
release: stop bundling computerd binary in npm tarball; default publish to unreleased tag
1 parent a7725ca commit 63d3636

2 files changed

Lines changed: 13 additions & 22 deletions

File tree

.github/workflows/release.yml

Lines changed: 10 additions & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -17,16 +17,16 @@
1717
# hop). Stable releases also move each registry's `:latest`
1818
# tag; prereleases only push the version-specific tag.
1919
#
20-
# The computerd binary is shipped two ways:
21-
# - inside the @cloudflare/computer npm tarball at dist/bin/
22-
# computerd-linux-x64, so consumers installing the package get it for
23-
# free; and
24-
# - as a single-layer container image, mirrored to both
25-
# ghcr.io/cloudflare/computer-computerd-linux-x64 and
26-
# registry.cloudflare.com/library/computer-computerd-linux-x64. The
27-
# example Dockerfiles pin the GHCR coordinate (anonymous pulls,
28-
# no Cloudflare-account dependency); the CF copy is kept for
29-
# consumers that prefer to stay on the in-platform registry.
20+
# The computerd binary ships as a single-layer container image,
21+
# mirrored to both ghcr.io/cloudflare/computer-computerd-linux-x64
22+
# and registry.cloudflare.com/library/computer-computerd-linux-x64.
23+
# The example Dockerfiles pin the GHCR coordinate (anonymous pulls,
24+
# no Cloudflare-account dependency); the CF copy is kept for
25+
# consumers that prefer to stay on the in-platform registry. It is
26+
# deliberately NOT bundled inside the @cloudflare/computer npm
27+
# tarball — nothing in the package resolves it at runtime, and
28+
# every consumer copies it out of the image via `COPY --from`, so
29+
# bundling only bloated the tarball by ~120 MB.
3030
#
3131
# npm packages use trusted publishing (OIDC); no NPM_TOKEN is
3232
# needed, only the workflow's `id-token: write` permission plus a
@@ -117,18 +117,6 @@ jobs:
117117
- name: Build computerd binary
118118
run: npm run build:bin --workspace @cloudflare/computerd
119119

120-
# Short-term bundling: ship the linux-x64 computerd binary inside
121-
# @cloudflare/computer under dist/bin/ — not registered as a
122-
# "bin" entry, so npm install won't try to symlink it on
123-
# non-linux hosts. Docker workflows reach in and copy it out
124-
# explicitly. Replace with a proper platform-package install
125-
# once we have more than one target.
126-
- name: Stage binary into @cloudflare/computer
127-
run: |
128-
mkdir -p packages/computer/dist/bin
129-
cp artifacts/computerd/computerd-linux-x64 packages/computer/dist/bin/computerd-linux-x64
130-
chmod +x packages/computer/dist/bin/computerd-linux-x64
131-
132120
- name: Sync versions
133121
run: node script/set-versions.mjs "${RELEASE_TAG}"
134122

packages/computer/package.json

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,9 @@
1010
},
1111
"private": false,
1212
"type": "module",
13+
"publishConfig": {
14+
"tag": "unreleased"
15+
},
1316
"exports": {
1417
".": {
1518
"types": "./dist/index.d.ts",

0 commit comments

Comments
 (0)