Skip to content

Commit 3e54dc3

Browse files
committed
fix(release): install dependencies before merging macOS updater manifests
The 'Publish GitHub Release' job invokes scripts/merge-update-manifests.ts which imports from @effect/platform-node. Without a prior bun install step the release runner has no node_modules, so the script fails with: error: Cannot find module '@effect/platform-node/NodeServices' from /home/runner/work/t3code/t3code/scripts/merge-update-manifests.ts Upstream's release job includes an 'Install dependencies' step between Setup Node and Download artifacts (they run merge-update-manifests via `node` too — same dep resolution requirement). Mirror that step in the fork's simplified release flow. Failed run: https://github.com/aaditagrawal/t3code/actions/runs/24624988376/job/72002885256
1 parent 52e015c commit 3e54dc3

1 file changed

Lines changed: 3 additions & 0 deletions

File tree

.github/workflows/release.yml

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -246,6 +246,9 @@ jobs:
246246
node-version-file: package.json
247247
registry-url: https://registry.npmjs.org
248248

249+
- name: Install dependencies
250+
run: bun install --frozen-lockfile
251+
249252
- name: Download all desktop artifacts
250253
uses: actions/download-artifact@v8
251254
with:

0 commit comments

Comments
 (0)