Skip to content

Commit 41b6344

Browse files
MajorTalclaude
andcommitted
fix(ci): publish-astro must build @run402/sdk before tsc -p astro
astro now imports types from @run402/sdk (the new ./release-slice helper) so a fresh npm ci leaves sdk/dist missing, and the type-check step fails finding "@run402/sdk" / "@run402/sdk/node" module types. Adds a build:core + build:sdk step between npm ci and the type-check so the dist tree referenced by @run402/sdk's `types` entry exists when tsc walks the astro project. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
1 parent 937142d commit 41b6344

1 file changed

Lines changed: 11 additions & 0 deletions

File tree

.github/workflows/publish-astro.yml

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -105,6 +105,17 @@ jobs:
105105
- name: Install dependencies
106106
run: npm ci
107107

108+
- name: Build @run402/sdk + core
109+
# The astro package imports types from `@run402/sdk` (e.g. the
110+
# release-slice helper, v1.1.0+). `@run402/sdk`'s `types` entry
111+
# points at `sdk/dist/` which doesn't exist after a fresh `npm ci`,
112+
# so the subsequent `tsc -p astro` would fail to resolve those
113+
# imports. Building core + sdk first stages the dist tree the
114+
# workspace symlink hands to TypeScript's module resolver.
115+
run: |
116+
npm run build:core
117+
npm run build:sdk
118+
108119
- name: Lint + type-check
109120
run: npx tsc --noEmit -p astro
110121

0 commit comments

Comments
 (0)