fix(ci): build @docmost/token-estimate before its non-nx consumers (#490)
The shared @docmost/token-estimate package (main: ./dist/index.js, dist/ gitignored) was never built in the CI jobs that bypass nx, so develop CI went red: - test.yml `test` (pnpm -r test): client vitest failed to resolve the import. - develop.yml `e2e-server` (direct jest e2e): server history-budget.ts hit TS2307 Cannot find module '@docmost/token-estimate'. The nx-driven jobs (build, e2e-mcp) stayed green via dependsOn: ^build. - test.yml: add "Build token-estimate" step to the `test` job. - develop.yml: add "Build token-estimate" step to the `e2e-server` job. - Dockerfile: ship packages/token-estimate/dist + package.json into the installer stage — apps/server depends on it (workspace:*) and imports it at runtime, so the prod image would otherwise crash with ERR_MODULE_NOT_FOUND (masked so far because publish/smoke never ran). Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
This commit is contained in:
@@ -159,6 +159,14 @@ jobs:
|
||||
- name: Build prosemirror-markdown
|
||||
run: pnpm --filter @docmost/prosemirror-markdown build
|
||||
|
||||
# @docmost/token-estimate is a shared workspace package the client vitest
|
||||
# suite resolves via its dist build (main: ./dist/index.js); dist/ is
|
||||
# gitignored and `pnpm -r test` does NOT honour nx `dependsOn: ^build`, so
|
||||
# build it before the recursive test run or the client suite fails with
|
||||
# "Failed to resolve import '@docmost/token-estimate'" (#490).
|
||||
- name: Build token-estimate
|
||||
run: pnpm --filter @docmost/token-estimate build
|
||||
|
||||
- name: Run unit tests
|
||||
run: pnpm -r test
|
||||
|
||||
|
||||
Reference in New Issue
Block a user