@mandujs/core 0.54.31 → 0.55.0-beta.1
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/README.md +654 -654
- package/package.json +106 -210
- package/scripts/postinstall-lock.ts +153 -153
- package/src/a11y/__tests__/run-audit.test.ts +333 -333
- package/src/a11y/fix-hints.ts +76 -76
- package/src/a11y/index.ts +18 -18
- package/src/a11y/run-audit.ts +15 -15
- package/src/a11y/types.ts +125 -125
- package/src/agent/__tests__/apply.test.ts +224 -0
- package/src/agent/__tests__/context.test.ts +97 -98
- package/src/agent/apply.ts +1031 -0
- package/src/agent/context.ts +552 -552
- package/src/agent/index.ts +4 -3
- package/src/agent/plan.ts +161 -243
- package/src/agent/repair.ts +239 -162
- package/src/agent/sync.ts +197 -198
- package/src/agent/types.ts +242 -94
- package/src/agent/verify.ts +55 -55
- package/src/auth/__tests__/login.test.ts +1 -1
- package/src/auth/__tests__/password.test.ts +15 -5
- package/src/auth/__tests__/reset.test.ts +2 -2
- package/src/auth/__tests__/tokens.test.ts +274 -274
- package/src/auth/__tests__/verification.test.ts +274 -274
- package/src/auth/index.ts +76 -76
- package/src/auth/login.ts +225 -225
- package/src/auth/password.ts +14 -2
- package/src/auth/reset.ts +243 -243
- package/src/auth/tokens.ts +612 -612
- package/src/auth/verification.ts +253 -253
- package/src/brain/__tests__/redactor.test.ts +94 -94
- package/src/brain/adapters/__tests__/_helpers.ts +83 -83
- package/src/brain/adapters/__tests__/anthropic-oauth.test.ts +196 -196
- package/src/brain/adapters/__tests__/chatgpt-auth.test.ts +193 -193
- package/src/brain/adapters/__tests__/openai-oauth.test.ts +209 -209
- package/src/brain/adapters/__tests__/resolver.test.ts +143 -143
- package/src/brain/adapters/anthropic-oauth.ts +1 -1
- package/src/brain/adapters/chatgpt-auth.ts +300 -300
- package/src/brain/adapters/index.ts +319 -319
- package/src/brain/adapters/oauth-flow.ts +439 -439
- package/src/brain/consent.ts +240 -240
- package/src/brain/credentials.ts +396 -396
- package/src/brain/doctor/analyzer.ts +7 -7
- package/src/bundler/__snapshots__/build.test.ts.snap +5 -5
- package/src/bundler/__tests__/build-runner.ts +166 -166
- package/src/bundler/__tests__/client-boundary-transform.test.ts +524 -524
- package/src/bundler/__tests__/cold-start.test.ts +60 -60
- package/src/bundler/__tests__/css.test.ts +49 -20
- package/src/bundler/__tests__/dev-reliability.test.ts +619 -619
- package/src/bundler/__tests__/extended-watch.test.ts +711 -711
- package/src/bundler/__tests__/fast-refresh.test.ts +24 -24
- package/src/bundler/__tests__/generation.test.ts +447 -0
- package/src/bundler/__tests__/hdr.test.ts +24 -18
- package/src/bundler/__tests__/hmr-client.test.ts +62 -24
- package/src/bundler/__tests__/jsx-runtime-shim.test.ts +179 -140
- package/src/bundler/__tests__/manifest-schema.test.ts +305 -266
- package/src/bundler/__tests__/prod-smoke.test.ts +138 -138
- package/src/bundler/__tests__/reverse-import-graph.test.ts +42 -42
- package/src/bundler/__tests__/slot-dispatch.test.ts +573 -573
- package/src/bundler/__tests__/url-cap-and-slot-regex.test.ts +286 -286
- package/src/bundler/__tests__/vendor-cache.test.ts +455 -455
- package/src/bundler/analyzer.ts +15 -15
- package/src/bundler/budget.ts +404 -404
- package/src/bundler/build.test.ts +898 -915
- package/src/bundler/build.ts +113 -24
- package/src/bundler/client-boundary-transform.ts +977 -977
- package/src/bundler/css.ts +65 -44
- package/src/bundler/dev.ts +105 -73
- package/src/bundler/fast-refresh-preamble.ts +47 -47
- package/src/bundler/generation.ts +602 -0
- package/src/bundler/index.ts +3 -3
- package/src/bundler/manifest-schema.ts +55 -40
- package/src/bundler/plugins/__tests__/block-generated-imports.test.ts +13 -13
- package/src/bundler/plugins/__tests__/react-compiler-config.test.ts +83 -83
- package/src/bundler/plugins/__tests__/react-compiler-lint.test.ts +110 -110
- package/src/bundler/plugins/block-generated-imports.ts +16 -15
- package/src/bundler/plugins/index.ts +83 -83
- package/src/bundler/plugins/react-compiler-config.ts +108 -108
- package/src/bundler/plugins/react-compiler-lint.ts +253 -253
- package/src/bundler/plugins/react-compiler.ts +162 -162
- package/src/bundler/prerender.ts +29 -9
- package/src/bundler/reverse-import-graph.ts +339 -339
- package/src/bundler/safe-build.test.ts +1 -1
- package/src/bundler/safe-build.ts +103 -103
- package/src/bundler/scenario-matrix.ts +229 -229
- package/src/bundler/types.ts +58 -56
- package/src/bundler/vendor-cache-types.ts +130 -130
- package/src/bundler/vendor-cache.ts +526 -526
- package/src/change/snapshot.ts +18 -18
- package/src/client/Form.tsx +105 -105
- package/src/client/Link.tsx +9 -9
- package/src/client/__tests__/props-serialization.test.ts +37 -37
- package/src/client/__tests__/use-sse.test.ts +153 -153
- package/src/client/globals.ts +1 -1
- package/src/client/hooks.ts +362 -362
- package/src/client/hydrate.ts +2 -2
- package/src/client/index.ts +1 -1
- package/src/client/island.ts +79 -79
- package/src/client/prefetch-helper.ts +55 -55
- package/src/client/props-serialization.ts +233 -233
- package/src/client/runtime-entry.ts +598 -598
- package/src/client/runtime.ts +1 -1
- package/src/client/serialize.ts +50 -50
- package/src/client/use-fetch.ts +6 -6
- package/src/client/use-head.ts +197 -197
- package/src/client/use-sse.ts +378 -378
- package/src/client/window-state.ts +101 -101
- package/src/components/Image-compat.ts +3 -0
- package/src/components/Image.tsx +162 -162
- package/src/config/validate.ts +1 -1
- package/src/config/watcher.ts +311 -311
- package/src/constants.ts +40 -40
- package/src/content/collection.ts +9 -9
- package/src/content/content-layer.ts +7 -7
- package/src/content/data-store.ts +245 -245
- package/src/content/frontmatter.ts +189 -189
- package/src/content/generate-types.ts +1 -1
- package/src/content/index.ts +2 -2
- package/src/content/loader-context.ts +171 -171
- package/src/content/loaders/api.ts +216 -216
- package/src/content/loaders/file.ts +172 -172
- package/src/content/loaders/glob.ts +253 -253
- package/src/content/loaders/index.ts +34 -34
- package/src/content/loaders/types.ts +137 -137
- package/src/content/meta-store.ts +209 -209
- package/src/content/prebuild.test.ts +571 -571
- package/src/content/prebuild.ts +636 -636
- package/src/content/schema.ts +20 -20
- package/src/content/sidebar.ts +630 -630
- package/src/content/slug.ts +110 -110
- package/src/content/types.ts +282 -282
- package/src/content/watcher.ts +135 -135
- package/src/contract/client-safe.test.ts +42 -42
- package/src/contract/client-safe.ts +114 -114
- package/src/contract/define.ts +11 -11
- package/src/contract/index.ts +1 -1
- package/src/contract/normalize.test.ts +276 -276
- package/src/contract/normalize.ts +410 -410
- package/src/contract/registry.test.ts +206 -206
- package/src/contract/route-helpers.ts +1 -1
- package/src/contract/rpc.ts +443 -443
- package/src/contract/types.ts +58 -58
- package/src/db/__tests__/db.test.ts +9 -9
- package/src/db/index.ts +2 -2
- package/src/db/migrations/history-table.ts +345 -345
- package/src/db/migrations/index.ts +3 -3
- package/src/db/migrations/lock.ts +324 -324
- package/src/db/migrations/runner.ts +650 -650
- package/src/deploy/cache.ts +140 -140
- package/src/deploy/compile/vercel.ts +344 -344
- package/src/deploy/index.ts +87 -87
- package/src/deploy/inference/brain.ts +268 -268
- package/src/deploy/inference/context.ts +82 -82
- package/src/deploy/inference/filling-extract.ts +245 -245
- package/src/deploy/inference/heuristic.ts +182 -182
- package/src/deploy/intent.ts +173 -173
- package/src/deploy/plan.ts +178 -178
- package/src/design/__tests__/agents-link.test.ts +109 -109
- package/src/design/__tests__/extract-patch-diff.test.ts +265 -265
- package/src/design/__tests__/lint.test.ts +110 -110
- package/src/design/__tests__/parser.test.ts +195 -195
- package/src/design/__tests__/tailwind-theme.test.ts +229 -229
- package/src/design/agents-link.ts +165 -165
- package/src/design/diff.ts +138 -138
- package/src/design/extract.ts +285 -285
- package/src/design/index.ts +102 -102
- package/src/design/lint.ts +209 -209
- package/src/design/parser.ts +555 -555
- package/src/design/patch.ts +241 -241
- package/src/design/scaffold.ts +147 -147
- package/src/design/tailwind-theme.ts +441 -441
- package/src/design/types.ts +210 -210
- package/src/desktop/__tests__/smoke.test.ts +2 -2
- package/src/desktop/__tests__/webview-fallback.test.ts +12 -32
- package/src/desktop/__tests__/window.test.ts +20 -99
- package/src/desktop/__tests__/worker.test.ts +266 -266
- package/src/desktop/index.ts +43 -43
- package/src/desktop/types.ts +158 -158
- package/src/desktop/webview-fallback.ts +17 -4
- package/src/desktop/window.ts +17 -8
- package/src/desktop/worker.ts +180 -180
- package/src/dev-error-overlay/__tests__/overlay-injector.test.ts +241 -241
- package/src/dev-error-overlay/index.ts +30 -30
- package/src/dev-error-overlay/overlay-injector.ts +243 -243
- package/src/dev-error-overlay/overlay-styles.ts +52 -52
- package/src/dev-error-overlay/types.ts +66 -66
- package/src/devtools/ai/context-builder.ts +375 -375
- package/src/devtools/ai/index.ts +25 -25
- package/src/devtools/ai/mcp-connector.ts +25 -25
- package/src/devtools/client/catchers/error-catcher.ts +344 -344
- package/src/devtools/client/catchers/index.ts +18 -18
- package/src/devtools/client/components/index.ts +39 -39
- package/src/devtools/client/components/mandu-character.tsx +331 -331
- package/src/devtools/client/components/panel/errors-panel.tsx +259 -259
- package/src/devtools/client/components/panel/guard-panel.tsx +30 -30
- package/src/devtools/client/components/panel/islands-panel.tsx +16 -16
- package/src/devtools/client/components/panel/network-panel.tsx +291 -291
- package/src/devtools/client/components/panel/panel-container.tsx +1 -1
- package/src/devtools/client/components/panel/preview-panel.tsx +46 -46
- package/src/devtools/client/filters/context-filters.ts +282 -282
- package/src/devtools/client/filters/index.ts +16 -16
- package/src/devtools/client/index.ts +63 -63
- package/src/devtools/hook/create-hook.ts +207 -207
- package/src/devtools/hook/index.ts +13 -13
- package/src/devtools/index.ts +439 -439
- package/src/devtools/init.ts +265 -265
- package/src/devtools/protocol.ts +237 -237
- package/src/devtools/server/index.ts +17 -17
- package/src/devtools/types.ts +35 -35
- package/src/devtools/worker/index.ts +25 -25
- package/src/devtools/worker/redaction-worker.ts +233 -233
- package/src/diagnose/__tests__/checks.test.ts +136 -136
- package/src/diagnose/checks.ts +185 -185
- package/src/diagnose/index.ts +17 -17
- package/src/diagnose/run.ts +10 -10
- package/src/diagnose/types.ts +53 -53
- package/src/email/__tests__/email.test.ts +355 -355
- package/src/email/index.ts +282 -282
- package/src/email/smtp.ts +64 -64
- package/src/error/domains.ts +265 -265
- package/src/error/types.ts +6 -6
- package/src/errors/extractor.ts +409 -409
- package/src/errors/index.ts +19 -19
- package/src/filling/__tests__/session-sqlite.test.ts +5 -1
- package/src/filling/auth.ts +308 -308
- package/src/filling/body-parse.test.ts +60 -60
- package/src/filling/cookie-codec.ts +5 -3
- package/src/filling/deps.ts +265 -265
- package/src/filling/head-method.test.ts +154 -154
- package/src/filling/session-sqlite.ts +617 -617
- package/src/filling/sse.ts +5 -5
- package/src/filling/ws.ts +78 -78
- package/src/generator/generate.ts +30 -30
- package/src/generator/index.ts +3 -3
- package/src/generator/templates.test.ts +48 -48
- package/src/generator/templates.ts +219 -219
- package/src/guard/__tests__/design-inline-class.test.ts +219 -219
- package/src/guard/__tests__/tsgolint-bridge.test.ts +347 -347
- package/src/guard/analyzer.ts +360 -360
- package/src/guard/auto-correct.ts +1 -1
- package/src/guard/check.ts +39 -24
- package/src/guard/config-guard.ts +13 -13
- package/src/guard/contract-guard.ts +9 -9
- package/src/guard/define-rule.ts +243 -243
- package/src/guard/design-inline-class.ts +393 -393
- package/src/guard/file-type.test.ts +24 -24
- package/src/guard/fs-routes-policy.ts +51 -51
- package/src/guard/graph.ts +1 -1
- package/src/guard/healing.ts +36 -36
- package/src/guard/index.ts +11 -11
- package/src/guard/presets/atomic.ts +70 -70
- package/src/guard/presets/clean.ts +77 -77
- package/src/guard/presets/fsd.ts +79 -79
- package/src/guard/presets/hexagonal.ts +68 -68
- package/src/guard/reporter.ts +442 -442
- package/src/guard/rule-presets.ts +379 -379
- package/src/guard/semantic-slots.ts +1 -1
- package/src/guard/suggestions.ts +358 -358
- package/src/guard/tsgolint-bridge.ts +512 -512
- package/src/guard/types.ts +348 -348
- package/src/guard/watcher.ts +405 -405
- package/src/i18n/define.ts +126 -126
- package/src/i18n/index.ts +52 -52
- package/src/i18n/message-registry.ts +173 -173
- package/src/i18n/types.ts +112 -112
- package/src/id/__tests__/id.test.ts +3 -3
- package/src/id/index.ts +105 -105
- package/src/index.ts +9 -48
- package/src/internal/client-boundary.ts +266 -266
- package/src/internal/index.ts +2 -2
- package/src/kitchen/api/agent-devtools-api.ts +779 -779
- package/src/kitchen/api/bundle-inspector.ts +179 -0
- package/src/kitchen/api/errors-grouping.ts +126 -126
- package/src/kitchen/api/file-api.ts +11 -11
- package/src/kitchen/kitchen-handler.ts +36 -0
- package/src/kitchen/kitchen-ui.ts +456 -0
- package/src/logging/index.ts +22 -22
- package/src/logging/transports.ts +365 -365
- package/src/middleware/bridge.ts +147 -147
- package/src/middleware/compose.ts +134 -134
- package/src/middleware/compress.ts +62 -62
- package/src/middleware/cors.ts +47 -47
- package/src/middleware/csrf.ts +328 -328
- package/src/middleware/define.ts +132 -132
- package/src/middleware/jwt.ts +134 -134
- package/src/middleware/logger.ts +58 -58
- package/src/middleware/oauth/__tests__/oauth.test.ts +1 -1
- package/src/middleware/oauth/index.ts +505 -505
- package/src/middleware/oauth/providers.ts +115 -115
- package/src/middleware/rate-limit/index.ts +522 -522
- package/src/middleware/rate-limit/sqlite-store.ts +382 -382
- package/src/middleware/scheduler-cron.ts +96 -96
- package/src/middleware/secure/__tests__/secure.test.ts +360 -360
- package/src/middleware/secure/csp.ts +193 -193
- package/src/middleware/session.ts +174 -174
- package/src/middleware/timeout.ts +55 -55
- package/src/observability/logger-adapter.ts +36 -36
- package/src/observability/sqlite-store.ts +254 -254
- package/src/openapi/generator.ts +1 -1
- package/src/openapi/openapi.test.ts +43 -43
- package/src/perf/__tests__/user-marks.test.ts +354 -354
- package/src/perf/index.ts +133 -133
- package/src/perf/user-marks.ts +1 -1
- package/src/plugins/__tests__/lifecycle-integration.test.ts +272 -272
- package/src/plugins/__tests__/runner.test.ts +409 -409
- package/src/plugins/define.ts +124 -124
- package/src/plugins/examples/dep-check-plugin.ts +80 -80
- package/src/plugins/examples/prerender-cache-plugin.ts +111 -111
- package/src/plugins/examples/sitemap-plugin.ts +65 -65
- package/src/plugins/runner.ts +361 -361
- package/src/plugins/types.ts +368 -368
- package/src/report/index.ts +1 -1
- package/src/resource/__tests__/generator.test.ts +7 -7
- package/src/resource/__tests__/schema.test.ts +14 -14
- package/src/resource/ddl/__tests__/diff.test.ts +639 -639
- package/src/resource/ddl/__tests__/emit.test.ts +165 -165
- package/src/resource/ddl/__tests__/snapshot.test.ts +499 -499
- package/src/resource/ddl/emit.ts +146 -146
- package/src/resource/ddl/persistence-types.ts +218 -218
- package/src/resource/ddl/type-map.ts +223 -223
- package/src/resource/ddl/types.ts +232 -232
- package/src/resource/generator-repo.ts +630 -630
- package/src/resource/generator-schema.ts +11 -11
- package/src/resource/generators/slot.ts +72 -72
- package/src/resource/schema.ts +21 -21
- package/src/router/client-entry.test.ts +227 -227
- package/src/router/client-entry.ts +218 -218
- package/src/router/fs-patterns.test.ts +96 -96
- package/src/router/fs-routes.test.ts +532 -532
- package/src/router/fs-routes.ts +53 -53
- package/src/router/fs-scanner.ts +216 -216
- package/src/router/fs-types.ts +19 -19
- package/src/router/route-source-analyzer.ts +521 -521
- package/src/routes/index.ts +74 -74
- package/src/routes/metadata-routes.ts +427 -427
- package/src/routes/types.ts +341 -341
- package/src/runtime/__tests__/devtools-adapter.test.ts +68 -68
- package/src/runtime/__tests__/error-boundary-redaction.test.ts +141 -141
- package/src/runtime/__tests__/hdr-client.test.ts +223 -223
- package/src/runtime/__tests__/http-errors.test.ts +117 -117
- package/src/runtime/__tests__/inline-client-hydration.test.ts +237 -237
- package/src/runtime/__tests__/not-found.test.ts +152 -152
- package/src/runtime/__tests__/observability-lifecycle.test.ts +103 -103
- package/src/runtime/__tests__/page-render-response.test.ts +517 -517
- package/src/runtime/__tests__/request-middleware.test.ts +70 -70
- package/src/runtime/__tests__/searchparams-page-props.test.ts +81 -81
- package/src/runtime/adapter.ts +47 -47
- package/src/runtime/boundary.tsx +252 -252
- package/src/runtime/cache.ts +494 -494
- package/src/runtime/compose.ts +222 -222
- package/src/runtime/devtools-adapter.ts +68 -68
- package/src/runtime/escape.ts +34 -34
- package/src/runtime/fast-refresh-runtime.ts +322 -322
- package/src/runtime/fast-refresh-types.ts +1 -1
- package/src/runtime/handler.ts +65 -65
- package/src/runtime/handlers.ts +30 -11
- package/src/runtime/http-errors.ts +113 -113
- package/src/runtime/image-handler.ts +1 -1
- package/src/runtime/lifecycle.ts +381 -381
- package/src/runtime/logger.test.ts +345 -345
- package/src/runtime/middleware.ts +264 -264
- package/src/runtime/not-found.ts +93 -93
- package/src/runtime/observability-lifecycle.ts +290 -290
- package/src/runtime/openapi-endpoint.ts +236 -236
- package/src/runtime/page-render-response.ts +287 -274
- package/src/runtime/ppr.ts +74 -74
- package/src/runtime/rate-limit.ts +1 -1
- package/src/runtime/redirect.ts +1 -1
- package/src/runtime/registry.ts +171 -171
- package/src/runtime/request-middleware.ts +31 -31
- package/src/runtime/router.test.ts +4 -4
- package/src/runtime/router.ts +10 -10
- package/src/runtime/server.ts +39 -20
- package/src/runtime/shims.ts +48 -48
- package/src/runtime/ssr.ts +43 -17
- package/src/runtime/static-files.ts +369 -289
- package/src/runtime/streaming-ssr.ts +219 -180
- package/src/runtime/trace.ts +144 -144
- package/src/scheduler/__tests__/scheduler.test.ts +12 -11
- package/src/scheduler/index.ts +11 -5
- package/src/scheduler/validate.ts +169 -169
- package/src/seo/index.ts +219 -219
- package/src/seo/integration/ssr.ts +306 -306
- package/src/seo/render/basic.ts +435 -435
- package/src/seo/render/index.ts +143 -143
- package/src/seo/render/jsonld.ts +539 -539
- package/src/seo/render/opengraph.ts +197 -197
- package/src/seo/render/robots.ts +116 -116
- package/src/seo/render/sitemap.ts +137 -137
- package/src/seo/render/twitter.ts +127 -127
- package/src/seo/resolve/opengraph.ts +143 -143
- package/src/seo/resolve/robots.ts +73 -73
- package/src/seo/resolve/title.ts +94 -94
- package/src/seo/resolve/twitter.ts +73 -73
- package/src/seo/resolve/url.ts +104 -104
- package/src/seo/routes/index.ts +290 -290
- package/src/seo/types.ts +588 -588
- package/src/slot/validator.ts +39 -39
- package/src/spec/schema.ts +35 -35
- package/src/storage/s3/__tests__/s3.test.ts +479 -479
- package/src/storage/s3/index.ts +412 -412
- package/src/testing/__tests__/assertions.test.ts +632 -632
- package/src/testing/__tests__/reporter.test.ts +454 -454
- package/src/testing/assertions.ts +986 -986
- package/src/testing/db.ts +157 -157
- package/src/testing/index.ts +9 -3
- package/src/testing/lcov.ts +192 -0
- package/src/testing/mocks.ts +203 -203
- package/src/testing/session.ts +190 -190
- package/src/types/branded.ts +56 -56
- package/src/types/index.ts +1 -1
- package/src/utils/safe-io.ts +188 -188
- package/src/utils/string-safe.ts +298 -298
- package/src/watcher/__tests__/watcher.test.ts +59 -59
- package/src/watcher/watcher.ts +61 -61
|
@@ -1,571 +1,571 @@
|
|
|
1
|
-
/**
|
|
2
|
-
* Tests for `content/prebuild.ts` (Issue #196).
|
|
3
|
-
*
|
|
4
|
-
* We exercise:
|
|
5
|
-
* - `discoverPrebuildScripts` globbing + sort determinism
|
|
6
|
-
* - `shouldAutoPrebuild` activation policy (content/ OR scripts/prebuild-*.ts)
|
|
7
|
-
* - `runPrebuildScripts` with an injected spawn hook (no actual subprocess)
|
|
8
|
-
* covering the success path, per-script fail-fast, timeout surface, and
|
|
9
|
-
* the empty-discovery no-op fast path.
|
|
10
|
-
*
|
|
11
|
-
* Spawning a real `bun` subprocess is intentionally avoided — those paths
|
|
12
|
-
* belong in the CLI-level integration test. At this layer the Bun runtime
|
|
13
|
-
* is mocked so the test doubles as a correctness guarantee for any Node
|
|
14
|
-
* environment that later injects its own spawn hook.
|
|
15
|
-
*/
|
|
16
|
-
|
|
17
|
-
import { describe, it, expect, beforeEach, afterEach } from "bun:test";
|
|
18
|
-
import fs from "node:fs";
|
|
19
|
-
import os from "node:os";
|
|
20
|
-
import path from "node:path";
|
|
21
|
-
|
|
22
|
-
import {
|
|
23
|
-
discoverPrebuildScripts,
|
|
24
|
-
shouldAutoPrebuild,
|
|
25
|
-
runPrebuildScripts,
|
|
26
|
-
PrebuildError,
|
|
27
|
-
PrebuildTimeoutError,
|
|
28
|
-
resolvePrebuildTimeout,
|
|
29
|
-
DEFAULT_PREBUILD_TIMEOUT_MS,
|
|
30
|
-
PREBUILD_TIMEOUT_ENV,
|
|
31
|
-
defaultSpawn,
|
|
32
|
-
type SpawnHook,
|
|
33
|
-
} from "./prebuild";
|
|
34
|
-
|
|
35
|
-
const PREFIX = path.join(os.tmpdir(), "mandu-prebuild-test-");
|
|
36
|
-
|
|
37
|
-
function mktmp(prefix = ""): string {
|
|
38
|
-
return fs.mkdtempSync(PREFIX + prefix);
|
|
39
|
-
}
|
|
40
|
-
|
|
41
|
-
function writeFile(dir: string, rel: string, content = "// stub\n"): string {
|
|
42
|
-
const abs = path.join(dir, rel);
|
|
43
|
-
fs.mkdirSync(path.dirname(abs), { recursive: true });
|
|
44
|
-
fs.writeFileSync(abs, content);
|
|
45
|
-
return abs;
|
|
46
|
-
}
|
|
47
|
-
|
|
48
|
-
// ---------------------------------------------------------------------------
|
|
49
|
-
// discoverPrebuildScripts
|
|
50
|
-
// ---------------------------------------------------------------------------
|
|
51
|
-
|
|
52
|
-
describe("discoverPrebuildScripts", () => {
|
|
53
|
-
let dir = "";
|
|
54
|
-
beforeEach(() => { dir = mktmp("discover-"); });
|
|
55
|
-
afterEach(() => { fs.rmSync(dir, { recursive: true, force: true }); });
|
|
56
|
-
|
|
57
|
-
it("returns empty when scripts/ does not exist", () => {
|
|
58
|
-
expect(discoverPrebuildScripts(dir)).toEqual([]);
|
|
59
|
-
});
|
|
60
|
-
|
|
61
|
-
it("returns empty when scripts/ exists but has no prebuild files", () => {
|
|
62
|
-
writeFile(dir, "scripts/build.ts");
|
|
63
|
-
writeFile(dir, "scripts/README.md");
|
|
64
|
-
expect(discoverPrebuildScripts(dir)).toEqual([]);
|
|
65
|
-
});
|
|
66
|
-
|
|
67
|
-
it("finds prebuild-<name>.ts files", () => {
|
|
68
|
-
writeFile(dir, "scripts/prebuild-docs.ts");
|
|
69
|
-
writeFile(dir, "scripts/prebuild-seo.ts");
|
|
70
|
-
const found = discoverPrebuildScripts(dir);
|
|
71
|
-
expect(found).toHaveLength(2);
|
|
72
|
-
expect(found[0]).toContain("prebuild-docs.ts");
|
|
73
|
-
expect(found[1]).toContain("prebuild-seo.ts");
|
|
74
|
-
});
|
|
75
|
-
|
|
76
|
-
it("sorts lexicographically so numeric prefix controls ordering", () => {
|
|
77
|
-
writeFile(dir, "scripts/prebuild-20-second.ts");
|
|
78
|
-
writeFile(dir, "scripts/prebuild-10-first.ts");
|
|
79
|
-
writeFile(dir, "scripts/prebuild-99-third.ts");
|
|
80
|
-
const found = discoverPrebuildScripts(dir).map((p) => path.basename(p));
|
|
81
|
-
expect(found).toEqual([
|
|
82
|
-
"prebuild-10-first.ts",
|
|
83
|
-
"prebuild-20-second.ts",
|
|
84
|
-
"prebuild-99-third.ts",
|
|
85
|
-
]);
|
|
86
|
-
});
|
|
87
|
-
|
|
88
|
-
it("accepts .tsx, .js, .mjs extensions", () => {
|
|
89
|
-
writeFile(dir, "scripts/prebuild-a.tsx");
|
|
90
|
-
writeFile(dir, "scripts/prebuild-b.js");
|
|
91
|
-
writeFile(dir, "scripts/prebuild-c.mjs");
|
|
92
|
-
const found = discoverPrebuildScripts(dir).map((p) => path.basename(p));
|
|
93
|
-
expect(found).toEqual(["prebuild-a.tsx", "prebuild-b.js", "prebuild-c.mjs"]);
|
|
94
|
-
});
|
|
95
|
-
|
|
96
|
-
it("rejects non-prebuild filenames even if in scripts/", () => {
|
|
97
|
-
writeFile(dir, "scripts/my-prebuild.ts"); // doesn't start with prebuild
|
|
98
|
-
writeFile(dir, "scripts/prebuild.py"); // wrong extension
|
|
99
|
-
writeFile(dir, "scripts/prebuild-valid.ts");
|
|
100
|
-
const found = discoverPrebuildScripts(dir).map((p) => path.basename(p));
|
|
101
|
-
expect(found).toEqual(["prebuild-valid.ts"]);
|
|
102
|
-
});
|
|
103
|
-
|
|
104
|
-
it("honors custom scriptsDir", () => {
|
|
105
|
-
writeFile(dir, "custom/prebuild-1.ts");
|
|
106
|
-
expect(discoverPrebuildScripts(dir, "custom")).toHaveLength(1);
|
|
107
|
-
});
|
|
108
|
-
});
|
|
109
|
-
|
|
110
|
-
// ---------------------------------------------------------------------------
|
|
111
|
-
// shouldAutoPrebuild
|
|
112
|
-
// ---------------------------------------------------------------------------
|
|
113
|
-
|
|
114
|
-
describe("shouldAutoPrebuild", () => {
|
|
115
|
-
let dir = "";
|
|
116
|
-
beforeEach(() => { dir = mktmp("policy-"); });
|
|
117
|
-
afterEach(() => { fs.rmSync(dir, { recursive: true, force: true }); });
|
|
118
|
-
|
|
119
|
-
it("false when neither content/ nor prebuild scripts exist", () => {
|
|
120
|
-
expect(shouldAutoPrebuild(dir)).toBe(false);
|
|
121
|
-
});
|
|
122
|
-
|
|
123
|
-
it("true when content/ directory exists (even if empty)", () => {
|
|
124
|
-
fs.mkdirSync(path.join(dir, "content"));
|
|
125
|
-
expect(shouldAutoPrebuild(dir)).toBe(true);
|
|
126
|
-
});
|
|
127
|
-
|
|
128
|
-
it("true when a prebuild script exists but content/ does not", () => {
|
|
129
|
-
writeFile(dir, "scripts/prebuild-docs.ts");
|
|
130
|
-
expect(shouldAutoPrebuild(dir)).toBe(true);
|
|
131
|
-
});
|
|
132
|
-
|
|
133
|
-
it("false when content/ is a file, not a directory", () => {
|
|
134
|
-
fs.writeFileSync(path.join(dir, "content"), "not a dir");
|
|
135
|
-
expect(shouldAutoPrebuild(dir)).toBe(false);
|
|
136
|
-
});
|
|
137
|
-
});
|
|
138
|
-
|
|
139
|
-
// ---------------------------------------------------------------------------
|
|
140
|
-
// runPrebuildScripts with mock spawn
|
|
141
|
-
// ---------------------------------------------------------------------------
|
|
142
|
-
|
|
143
|
-
function makeMockSpawn(
|
|
144
|
-
policy: (scriptPath: string) => { exitCode: number | null; durationMs: number } | Promise<{ exitCode: number | null; durationMs: number }>,
|
|
145
|
-
): SpawnHook & { calls: Array<{ scriptPath: string; cwd: string; timeoutMs: number }> } {
|
|
146
|
-
const calls: Array<{ scriptPath: string; cwd: string; timeoutMs: number }> = [];
|
|
147
|
-
const hook: SpawnHook = async (args) => {
|
|
148
|
-
calls.push(args);
|
|
149
|
-
return await policy(args.scriptPath);
|
|
150
|
-
};
|
|
151
|
-
return Object.assign(hook, { calls });
|
|
152
|
-
}
|
|
153
|
-
|
|
154
|
-
describe("runPrebuildScripts", () => {
|
|
155
|
-
let dir = "";
|
|
156
|
-
beforeEach(() => { dir = mktmp("run-"); });
|
|
157
|
-
afterEach(() => { fs.rmSync(dir, { recursive: true, force: true }); });
|
|
158
|
-
|
|
159
|
-
it("returns { ran: 0 } when no scripts exist — silent no-op", async () => {
|
|
160
|
-
const spawn = makeMockSpawn(() => ({ exitCode: 0, durationMs: 5 }));
|
|
161
|
-
const result = await runPrebuildScripts({ rootDir: dir, spawn });
|
|
162
|
-
expect(result.ran).toBe(0);
|
|
163
|
-
expect(spawn.calls).toHaveLength(0);
|
|
164
|
-
});
|
|
165
|
-
|
|
166
|
-
it("runs discovered scripts in order", async () => {
|
|
167
|
-
writeFile(dir, "scripts/prebuild-b.ts");
|
|
168
|
-
writeFile(dir, "scripts/prebuild-a.ts");
|
|
169
|
-
const order: string[] = [];
|
|
170
|
-
const spawn = makeMockSpawn(async (p) => {
|
|
171
|
-
order.push(path.basename(p));
|
|
172
|
-
return { exitCode: 0, durationMs: 1 };
|
|
173
|
-
});
|
|
174
|
-
const result = await runPrebuildScripts({ rootDir: dir, spawn });
|
|
175
|
-
expect(result.ran).toBe(2);
|
|
176
|
-
expect(order).toEqual(["prebuild-a.ts", "prebuild-b.ts"]);
|
|
177
|
-
});
|
|
178
|
-
|
|
179
|
-
it("throws PrebuildError with exitCode on first non-zero exit", async () => {
|
|
180
|
-
writeFile(dir, "scripts/prebuild-1.ts");
|
|
181
|
-
writeFile(dir, "scripts/prebuild-2.ts");
|
|
182
|
-
const spawn = makeMockSpawn((p) => ({
|
|
183
|
-
exitCode: path.basename(p) === "prebuild-1.ts" ? 3 : 0,
|
|
184
|
-
durationMs: 1,
|
|
185
|
-
}));
|
|
186
|
-
let caught: unknown;
|
|
187
|
-
try {
|
|
188
|
-
await runPrebuildScripts({ rootDir: dir, spawn });
|
|
189
|
-
} catch (e) {
|
|
190
|
-
caught = e;
|
|
191
|
-
}
|
|
192
|
-
expect(caught).toBeInstanceOf(PrebuildError);
|
|
193
|
-
const err = caught as PrebuildError;
|
|
194
|
-
expect(err.exitCode).toBe(3);
|
|
195
|
-
expect(err.scriptPath).toContain("prebuild-1.ts");
|
|
196
|
-
// prebuild-2.ts must NOT have been invoked (fail-fast chain semantics).
|
|
197
|
-
expect(spawn.calls).toHaveLength(1);
|
|
198
|
-
});
|
|
199
|
-
|
|
200
|
-
it("wraps non-PrebuildError spawn rejections", async () => {
|
|
201
|
-
writeFile(dir, "scripts/prebuild.ts");
|
|
202
|
-
const spawn: SpawnHook = async () => {
|
|
203
|
-
throw new Error("spawn ENOENT");
|
|
204
|
-
};
|
|
205
|
-
let caught: unknown;
|
|
206
|
-
try {
|
|
207
|
-
await runPrebuildScripts({ rootDir: dir, spawn });
|
|
208
|
-
} catch (e) {
|
|
209
|
-
caught = e;
|
|
210
|
-
}
|
|
211
|
-
expect(caught).toBeInstanceOf(PrebuildError);
|
|
212
|
-
expect((caught as PrebuildError).message).toContain("spawn ENOENT");
|
|
213
|
-
});
|
|
214
|
-
|
|
215
|
-
it("invokes onStart / onFinish callbacks for each script", async () => {
|
|
216
|
-
writeFile(dir, "scripts/prebuild-a.ts");
|
|
217
|
-
writeFile(dir, "scripts/prebuild-b.ts");
|
|
218
|
-
const starts: string[] = [];
|
|
219
|
-
const finishes: Array<{ name: string; code: number | null }> = [];
|
|
220
|
-
await runPrebuildScripts({
|
|
221
|
-
rootDir: dir,
|
|
222
|
-
spawn: async () => ({ exitCode: 0, durationMs: 1 }),
|
|
223
|
-
onStart: (p, i, total) => {
|
|
224
|
-
starts.push(`${i + 1}/${total}:${path.basename(p)}`);
|
|
225
|
-
},
|
|
226
|
-
onFinish: (r) => {
|
|
227
|
-
finishes.push({ name: path.basename(r.scriptPath), code: r.exitCode });
|
|
228
|
-
},
|
|
229
|
-
});
|
|
230
|
-
expect(starts).toEqual(["1/2:prebuild-a.ts", "2/2:prebuild-b.ts"]);
|
|
231
|
-
expect(finishes).toEqual([
|
|
232
|
-
{ name: "prebuild-a.ts", code: 0 },
|
|
233
|
-
{ name: "prebuild-b.ts", code: 0 },
|
|
234
|
-
]);
|
|
235
|
-
});
|
|
236
|
-
|
|
237
|
-
it("threads timeoutMs through to the spawn hook", async () => {
|
|
238
|
-
writeFile(dir, "scripts/prebuild-1.ts");
|
|
239
|
-
const spawn = makeMockSpawn(() => ({ exitCode: 0, durationMs: 1 }));
|
|
240
|
-
await runPrebuildScripts({
|
|
241
|
-
rootDir: dir,
|
|
242
|
-
spawn,
|
|
243
|
-
timeoutMs: 5_000,
|
|
244
|
-
});
|
|
245
|
-
expect(spawn.calls[0].timeoutMs).toBe(5_000);
|
|
246
|
-
});
|
|
247
|
-
|
|
248
|
-
it("uses default 2-minute timeout when not overridden", async () => {
|
|
249
|
-
writeFile(dir, "scripts/prebuild-1.ts");
|
|
250
|
-
const spawn = makeMockSpawn(() => ({ exitCode: 0, durationMs: 1 }));
|
|
251
|
-
await runPrebuildScripts({ rootDir: dir, spawn });
|
|
252
|
-
expect(spawn.calls[0].timeoutMs).toBe(2 * 60 * 1000);
|
|
253
|
-
});
|
|
254
|
-
});
|
|
255
|
-
|
|
256
|
-
// ---------------------------------------------------------------------------
|
|
257
|
-
// resolvePrebuildTimeout (Issue #203)
|
|
258
|
-
// ---------------------------------------------------------------------------
|
|
259
|
-
|
|
260
|
-
describe("resolvePrebuildTimeout", () => {
|
|
261
|
-
const originalEnv = process.env[PREBUILD_TIMEOUT_ENV];
|
|
262
|
-
afterEach(() => {
|
|
263
|
-
if (originalEnv === undefined) delete process.env[PREBUILD_TIMEOUT_ENV];
|
|
264
|
-
else process.env[PREBUILD_TIMEOUT_ENV] = originalEnv;
|
|
265
|
-
});
|
|
266
|
-
|
|
267
|
-
it("returns DEFAULT_PREBUILD_TIMEOUT_MS (120s) when no override", () => {
|
|
268
|
-
delete process.env[PREBUILD_TIMEOUT_ENV];
|
|
269
|
-
expect(resolvePrebuildTimeout()).toBe(DEFAULT_PREBUILD_TIMEOUT_MS);
|
|
270
|
-
expect(DEFAULT_PREBUILD_TIMEOUT_MS).toBe(120_000);
|
|
271
|
-
});
|
|
272
|
-
|
|
273
|
-
it("explicit arg wins over env var", () => {
|
|
274
|
-
process.env[PREBUILD_TIMEOUT_ENV] = "5000";
|
|
275
|
-
expect(resolvePrebuildTimeout(1234)).toBe(1234);
|
|
276
|
-
});
|
|
277
|
-
|
|
278
|
-
it("env var is used when no explicit arg", () => {
|
|
279
|
-
process.env[PREBUILD_TIMEOUT_ENV] = "7500";
|
|
280
|
-
expect(resolvePrebuildTimeout()).toBe(7500);
|
|
281
|
-
});
|
|
282
|
-
|
|
283
|
-
it("ignores invalid env var (non-numeric) and falls back to default", () => {
|
|
284
|
-
process.env[PREBUILD_TIMEOUT_ENV] = "not-a-number";
|
|
285
|
-
expect(resolvePrebuildTimeout()).toBe(DEFAULT_PREBUILD_TIMEOUT_MS);
|
|
286
|
-
});
|
|
287
|
-
|
|
288
|
-
it("ignores invalid env var (zero / negative) and falls back to default", () => {
|
|
289
|
-
process.env[PREBUILD_TIMEOUT_ENV] = "0";
|
|
290
|
-
expect(resolvePrebuildTimeout()).toBe(DEFAULT_PREBUILD_TIMEOUT_MS);
|
|
291
|
-
process.env[PREBUILD_TIMEOUT_ENV] = "-5";
|
|
292
|
-
expect(resolvePrebuildTimeout()).toBe(DEFAULT_PREBUILD_TIMEOUT_MS);
|
|
293
|
-
});
|
|
294
|
-
|
|
295
|
-
it("ignores non-positive explicit arg", () => {
|
|
296
|
-
delete process.env[PREBUILD_TIMEOUT_ENV];
|
|
297
|
-
expect(resolvePrebuildTimeout(0)).toBe(DEFAULT_PREBUILD_TIMEOUT_MS);
|
|
298
|
-
expect(resolvePrebuildTimeout(-1)).toBe(DEFAULT_PREBUILD_TIMEOUT_MS);
|
|
299
|
-
});
|
|
300
|
-
});
|
|
301
|
-
|
|
302
|
-
// ---------------------------------------------------------------------------
|
|
303
|
-
// Issue #203 — Timeout + error surface regressions
|
|
304
|
-
// ---------------------------------------------------------------------------
|
|
305
|
-
|
|
306
|
-
describe("runPrebuildScripts — timeout surface (Issue #203)", () => {
|
|
307
|
-
let dir = "";
|
|
308
|
-
beforeEach(() => { dir = mktmp("timeout-"); });
|
|
309
|
-
afterEach(() => { fs.rmSync(dir, { recursive: true, force: true }); });
|
|
310
|
-
|
|
311
|
-
it("honours MANDU_PREBUILD_TIMEOUT_MS env override when timeoutMs omitted", async () => {
|
|
312
|
-
const originalEnv = process.env[PREBUILD_TIMEOUT_ENV];
|
|
313
|
-
process.env[PREBUILD_TIMEOUT_ENV] = "9999";
|
|
314
|
-
try {
|
|
315
|
-
writeFile(dir, "scripts/prebuild-1.ts");
|
|
316
|
-
const spawn = makeMockSpawn(() => ({ exitCode: 0, durationMs: 1 }));
|
|
317
|
-
await runPrebuildScripts({ rootDir: dir, spawn });
|
|
318
|
-
expect(spawn.calls[0].timeoutMs).toBe(9999);
|
|
319
|
-
} finally {
|
|
320
|
-
if (originalEnv === undefined) delete process.env[PREBUILD_TIMEOUT_ENV];
|
|
321
|
-
else process.env[PREBUILD_TIMEOUT_ENV] = originalEnv;
|
|
322
|
-
}
|
|
323
|
-
});
|
|
324
|
-
|
|
325
|
-
it("explicit timeoutMs wins over env var", async () => {
|
|
326
|
-
const originalEnv = process.env[PREBUILD_TIMEOUT_ENV];
|
|
327
|
-
process.env[PREBUILD_TIMEOUT_ENV] = "9999";
|
|
328
|
-
try {
|
|
329
|
-
writeFile(dir, "scripts/prebuild-1.ts");
|
|
330
|
-
const spawn = makeMockSpawn(() => ({ exitCode: 0, durationMs: 1 }));
|
|
331
|
-
await runPrebuildScripts({ rootDir: dir, spawn, timeoutMs: 2222 });
|
|
332
|
-
expect(spawn.calls[0].timeoutMs).toBe(2222);
|
|
333
|
-
} finally {
|
|
334
|
-
if (originalEnv === undefined) delete process.env[PREBUILD_TIMEOUT_ENV];
|
|
335
|
-
else process.env[PREBUILD_TIMEOUT_ENV] = originalEnv;
|
|
336
|
-
}
|
|
337
|
-
});
|
|
338
|
-
|
|
339
|
-
it("defaultSpawn throws PrebuildTimeoutError with script name + limit when the script runs longer than timeout", async () => {
|
|
340
|
-
// Script sleeps 300ms — we set timeout 80ms, so the timer must win.
|
|
341
|
-
// We use `Bun.sleep` + `process.exit(0)` so the script exits cleanly
|
|
342
|
-
// if somehow the kill is skipped (no zombie in test harness).
|
|
343
|
-
const scriptAbs = writeFile(
|
|
344
|
-
dir,
|
|
345
|
-
"scripts/prebuild-slow.ts",
|
|
346
|
-
"await Bun.sleep(300);\nprocess.exit(0);\n",
|
|
347
|
-
);
|
|
348
|
-
|
|
349
|
-
let caught: unknown;
|
|
350
|
-
try {
|
|
351
|
-
await defaultSpawn({ scriptPath: scriptAbs, cwd: dir, timeoutMs: 80 });
|
|
352
|
-
} catch (e) {
|
|
353
|
-
caught = e;
|
|
354
|
-
}
|
|
355
|
-
expect(caught).toBeInstanceOf(PrebuildTimeoutError);
|
|
356
|
-
expect(caught).toBeInstanceOf(PrebuildError); // subclass relationship
|
|
357
|
-
const err = caught as PrebuildTimeoutError;
|
|
358
|
-
expect(err.timeoutMs).toBe(80);
|
|
359
|
-
expect(err.scriptPath).toContain("prebuild-slow.ts");
|
|
360
|
-
// Message contract: includes script name + timeout limit + override hint.
|
|
361
|
-
expect(err.message).toContain("prebuild-slow.ts");
|
|
362
|
-
expect(err.message).toContain("80ms");
|
|
363
|
-
expect(err.message).toContain("dev.prebuildTimeoutMs");
|
|
364
|
-
expect(err.message).toContain(PREBUILD_TIMEOUT_ENV);
|
|
365
|
-
// Regression beacon: message MUST NOT contain the opaque
|
|
366
|
-
// "non-Error thrown" string Issue #203 was reported against.
|
|
367
|
-
expect(err.message).not.toContain("non-Error thrown");
|
|
368
|
-
});
|
|
369
|
-
|
|
370
|
-
it("runPrebuildScripts surfaces PrebuildTimeoutError via injected spawn", async () => {
|
|
371
|
-
writeFile(dir, "scripts/prebuild-slow.ts");
|
|
372
|
-
const spawn: SpawnHook = async (args) => {
|
|
373
|
-
throw new PrebuildTimeoutError({
|
|
374
|
-
scriptPath: args.scriptPath,
|
|
375
|
-
timeoutMs: args.timeoutMs,
|
|
376
|
-
durationMs: args.timeoutMs,
|
|
377
|
-
});
|
|
378
|
-
};
|
|
379
|
-
let caught: unknown;
|
|
380
|
-
try {
|
|
381
|
-
await runPrebuildScripts({ rootDir: dir, spawn, timeoutMs: 100 });
|
|
382
|
-
} catch (e) {
|
|
383
|
-
caught = e;
|
|
384
|
-
}
|
|
385
|
-
expect(caught).toBeInstanceOf(PrebuildTimeoutError);
|
|
386
|
-
const err = caught as PrebuildTimeoutError;
|
|
387
|
-
expect(err.timeoutMs).toBe(100);
|
|
388
|
-
expect(err.scriptPath).toContain("prebuild-slow.ts");
|
|
389
|
-
});
|
|
390
|
-
});
|
|
391
|
-
|
|
392
|
-
describe("runPrebuildScripts — error preservation (Issue #203)", () => {
|
|
393
|
-
let dir = "";
|
|
394
|
-
beforeEach(() => { dir = mktmp("err-surface-"); });
|
|
395
|
-
afterEach(() => { fs.rmSync(dir, { recursive: true, force: true }); });
|
|
396
|
-
|
|
397
|
-
it("preserves the inner Error message ('boom') when a spawn rejection occurs", async () => {
|
|
398
|
-
writeFile(dir, "scripts/prebuild.ts");
|
|
399
|
-
const inner = new Error("boom");
|
|
400
|
-
const spawn: SpawnHook = async () => {
|
|
401
|
-
throw inner;
|
|
402
|
-
};
|
|
403
|
-
let caught: unknown;
|
|
404
|
-
try {
|
|
405
|
-
await runPrebuildScripts({ rootDir: dir, spawn });
|
|
406
|
-
} catch (e) {
|
|
407
|
-
caught = e;
|
|
408
|
-
}
|
|
409
|
-
expect(caught).toBeInstanceOf(PrebuildError);
|
|
410
|
-
const err = caught as PrebuildError & { cause?: unknown };
|
|
411
|
-
// The inner message MUST appear verbatim — the whole point of #203.
|
|
412
|
-
expect(err.message).toContain("boom");
|
|
413
|
-
// Regression beacon.
|
|
414
|
-
expect(err.message).not.toContain("non-Error thrown");
|
|
415
|
-
// The inner error is attached as `.cause` so debug tooling can
|
|
416
|
-
// recover its stack.
|
|
417
|
-
expect(err.cause).toBe(inner);
|
|
418
|
-
});
|
|
419
|
-
|
|
420
|
-
it("preserves stack via .cause when inner Error has a stack", async () => {
|
|
421
|
-
writeFile(dir, "scripts/prebuild.ts");
|
|
422
|
-
const inner = new Error("kaboom");
|
|
423
|
-
const originalStack = inner.stack;
|
|
424
|
-
expect(originalStack).toBeTruthy();
|
|
425
|
-
const spawn: SpawnHook = async () => {
|
|
426
|
-
throw inner;
|
|
427
|
-
};
|
|
428
|
-
let caught: unknown;
|
|
429
|
-
try {
|
|
430
|
-
await runPrebuildScripts({ rootDir: dir, spawn });
|
|
431
|
-
} catch (e) {
|
|
432
|
-
caught = e;
|
|
433
|
-
}
|
|
434
|
-
const err = caught as PrebuildError & { cause?: Error };
|
|
435
|
-
expect(err.cause).toBe(inner);
|
|
436
|
-
expect(err.cause?.stack).toBe(originalStack);
|
|
437
|
-
});
|
|
438
|
-
|
|
439
|
-
it("handles non-Error rejections (string) without producing 'non-Error thrown'", async () => {
|
|
440
|
-
writeFile(dir, "scripts/prebuild.ts");
|
|
441
|
-
const spawn: SpawnHook = async () => {
|
|
442
|
-
// Raw string throw — the pathological case Issue #203 was about.
|
|
443
|
-
throw "spawn blew up";
|
|
444
|
-
};
|
|
445
|
-
let caught: unknown;
|
|
446
|
-
try {
|
|
447
|
-
await runPrebuildScripts({ rootDir: dir, spawn });
|
|
448
|
-
} catch (e) {
|
|
449
|
-
caught = e;
|
|
450
|
-
}
|
|
451
|
-
expect(caught).toBeInstanceOf(PrebuildError);
|
|
452
|
-
const err = caught as PrebuildError & { cause?: unknown };
|
|
453
|
-
// The raw string survives into `.message`.
|
|
454
|
-
expect(err.message).toContain("spawn blew up");
|
|
455
|
-
expect(err.message).not.toContain("non-Error thrown");
|
|
456
|
-
expect(err.cause).toBe("spawn blew up");
|
|
457
|
-
});
|
|
458
|
-
|
|
459
|
-
it("handles non-Error rejections (object) without producing 'non-Error thrown'", async () => {
|
|
460
|
-
writeFile(dir, "scripts/prebuild.ts");
|
|
461
|
-
const raw = { code: "EBADF", info: "fd closed" };
|
|
462
|
-
const spawn: SpawnHook = async () => {
|
|
463
|
-
throw raw;
|
|
464
|
-
};
|
|
465
|
-
let caught: unknown;
|
|
466
|
-
try {
|
|
467
|
-
await runPrebuildScripts({ rootDir: dir, spawn });
|
|
468
|
-
} catch (e) {
|
|
469
|
-
caught = e;
|
|
470
|
-
}
|
|
471
|
-
const err = caught as PrebuildError & { cause?: unknown };
|
|
472
|
-
expect(err.message).toContain("EBADF");
|
|
473
|
-
expect(err.message).not.toContain("non-Error thrown");
|
|
474
|
-
expect(err.cause).toBe(raw);
|
|
475
|
-
});
|
|
476
|
-
|
|
477
|
-
it("includes stdout/stderr tails in PrebuildError.message for non-zero exit", async () => {
|
|
478
|
-
writeFile(dir, "scripts/prebuild.ts");
|
|
479
|
-
const stderrSample = Array.from({ length: 15 }, (_, i) => `err-line-${i + 1}`).join("\n");
|
|
480
|
-
const stdoutSample = Array.from({ length: 12 }, (_, i) => `out-line-${i + 1}`).join("\n");
|
|
481
|
-
const spawn: SpawnHook = async () => ({
|
|
482
|
-
exitCode: 1,
|
|
483
|
-
durationMs: 5,
|
|
484
|
-
stdoutTail: stdoutSample,
|
|
485
|
-
stderrTail: stderrSample,
|
|
486
|
-
});
|
|
487
|
-
let caught: unknown;
|
|
488
|
-
try {
|
|
489
|
-
await runPrebuildScripts({ rootDir: dir, spawn });
|
|
490
|
-
} catch (e) {
|
|
491
|
-
caught = e;
|
|
492
|
-
}
|
|
493
|
-
expect(caught).toBeInstanceOf(PrebuildError);
|
|
494
|
-
const err = caught as PrebuildError;
|
|
495
|
-
// Exactly last 10 stderr lines are kept.
|
|
496
|
-
expect(err.message).toContain("err-line-15");
|
|
497
|
-
expect(err.message).toContain("err-line-6"); // the 10-from-last
|
|
498
|
-
expect(err.message).not.toContain("err-line-5"); // trimmed
|
|
499
|
-
// Stdout tail is also present (10 lines out of 12).
|
|
500
|
-
expect(err.message).toContain("out-line-12");
|
|
501
|
-
expect(err.message).toContain("out-line-3");
|
|
502
|
-
expect(err.message).not.toContain("out-line-2");
|
|
503
|
-
// Structured fields carry the full captured tails.
|
|
504
|
-
expect(err.stderrTail).toBe(stderrSample);
|
|
505
|
-
expect(err.stdoutTail).toBe(stdoutSample);
|
|
506
|
-
});
|
|
507
|
-
|
|
508
|
-
it("omits empty tail sections when the spawn hook didn't capture", async () => {
|
|
509
|
-
writeFile(dir, "scripts/prebuild.ts");
|
|
510
|
-
const spawn: SpawnHook = async () => ({ exitCode: 2, durationMs: 3 });
|
|
511
|
-
let caught: unknown;
|
|
512
|
-
try {
|
|
513
|
-
await runPrebuildScripts({ rootDir: dir, spawn });
|
|
514
|
-
} catch (e) {
|
|
515
|
-
caught = e;
|
|
516
|
-
}
|
|
517
|
-
const err = caught as PrebuildError;
|
|
518
|
-
// The "exited with code 2" prefix is present but no "--- stderr ---" sections.
|
|
519
|
-
expect(err.message).toContain("exited with code 2");
|
|
520
|
-
expect(err.message).not.toContain("--- stderr");
|
|
521
|
-
expect(err.message).not.toContain("--- stdout");
|
|
522
|
-
});
|
|
523
|
-
|
|
524
|
-
it("normalizes Windows-style CRLF line endings in stderr tail", async () => {
|
|
525
|
-
writeFile(dir, "scripts/prebuild.ts");
|
|
526
|
-
const crlf = ["a", "b", "c", "d"].join("\r\n");
|
|
527
|
-
const spawn: SpawnHook = async () => ({
|
|
528
|
-
exitCode: 1,
|
|
529
|
-
durationMs: 1,
|
|
530
|
-
stderrTail: crlf,
|
|
531
|
-
});
|
|
532
|
-
let caught: unknown;
|
|
533
|
-
try {
|
|
534
|
-
await runPrebuildScripts({ rootDir: dir, spawn });
|
|
535
|
-
} catch (e) {
|
|
536
|
-
caught = e;
|
|
537
|
-
}
|
|
538
|
-
const err = caught as PrebuildError;
|
|
539
|
-
// All 4 lines survive (under the 10-line cap).
|
|
540
|
-
expect(err.message).toContain("a\nb\nc\nd");
|
|
541
|
-
// No raw \r\n survived.
|
|
542
|
-
expect(err.message).not.toContain("\r\n");
|
|
543
|
-
});
|
|
544
|
-
});
|
|
545
|
-
|
|
546
|
-
describe("PrebuildTimeoutError shape (Issue #203)", () => {
|
|
547
|
-
it("is a subclass of PrebuildError so existing `instanceof PrebuildError` callers still match", () => {
|
|
548
|
-
const err = new PrebuildTimeoutError({
|
|
549
|
-
scriptPath: "/tmp/foo/scripts/prebuild.ts",
|
|
550
|
-
timeoutMs: 500,
|
|
551
|
-
durationMs: 500,
|
|
552
|
-
});
|
|
553
|
-
expect(err).toBeInstanceOf(PrebuildTimeoutError);
|
|
554
|
-
expect(err).toBeInstanceOf(PrebuildError);
|
|
555
|
-
expect(err).toBeInstanceOf(Error);
|
|
556
|
-
expect(err.name).toBe("PrebuildTimeoutError");
|
|
557
|
-
expect(err.exitCode).toBeNull();
|
|
558
|
-
});
|
|
559
|
-
|
|
560
|
-
it("message names the script + limit + override paths", () => {
|
|
561
|
-
const err = new PrebuildTimeoutError({
|
|
562
|
-
scriptPath: "/repo/scripts/prebuild-seed.ts",
|
|
563
|
-
timeoutMs: 250,
|
|
564
|
-
durationMs: 250,
|
|
565
|
-
});
|
|
566
|
-
expect(err.message).toContain("prebuild-seed.ts");
|
|
567
|
-
expect(err.message).toContain("250ms");
|
|
568
|
-
expect(err.message).toContain("dev.prebuildTimeoutMs");
|
|
569
|
-
expect(err.message).toContain(PREBUILD_TIMEOUT_ENV);
|
|
570
|
-
});
|
|
571
|
-
});
|
|
1
|
+
/**
|
|
2
|
+
* Tests for `content/prebuild.ts` (Issue #196).
|
|
3
|
+
*
|
|
4
|
+
* We exercise:
|
|
5
|
+
* - `discoverPrebuildScripts` globbing + sort determinism
|
|
6
|
+
* - `shouldAutoPrebuild` activation policy (content/ OR scripts/prebuild-*.ts)
|
|
7
|
+
* - `runPrebuildScripts` with an injected spawn hook (no actual subprocess)
|
|
8
|
+
* covering the success path, per-script fail-fast, timeout surface, and
|
|
9
|
+
* the empty-discovery no-op fast path.
|
|
10
|
+
*
|
|
11
|
+
* Spawning a real `bun` subprocess is intentionally avoided — those paths
|
|
12
|
+
* belong in the CLI-level integration test. At this layer the Bun runtime
|
|
13
|
+
* is mocked so the test doubles as a correctness guarantee for any Node
|
|
14
|
+
* environment that later injects its own spawn hook.
|
|
15
|
+
*/
|
|
16
|
+
|
|
17
|
+
import { describe, it, expect, beforeEach, afterEach } from "bun:test";
|
|
18
|
+
import fs from "node:fs";
|
|
19
|
+
import os from "node:os";
|
|
20
|
+
import path from "node:path";
|
|
21
|
+
|
|
22
|
+
import {
|
|
23
|
+
discoverPrebuildScripts,
|
|
24
|
+
shouldAutoPrebuild,
|
|
25
|
+
runPrebuildScripts,
|
|
26
|
+
PrebuildError,
|
|
27
|
+
PrebuildTimeoutError,
|
|
28
|
+
resolvePrebuildTimeout,
|
|
29
|
+
DEFAULT_PREBUILD_TIMEOUT_MS,
|
|
30
|
+
PREBUILD_TIMEOUT_ENV,
|
|
31
|
+
defaultSpawn,
|
|
32
|
+
type SpawnHook,
|
|
33
|
+
} from "./prebuild";
|
|
34
|
+
|
|
35
|
+
const PREFIX = path.join(os.tmpdir(), "mandu-prebuild-test-");
|
|
36
|
+
|
|
37
|
+
function mktmp(prefix = ""): string {
|
|
38
|
+
return fs.mkdtempSync(PREFIX + prefix);
|
|
39
|
+
}
|
|
40
|
+
|
|
41
|
+
function writeFile(dir: string, rel: string, content = "// stub\n"): string {
|
|
42
|
+
const abs = path.join(dir, rel);
|
|
43
|
+
fs.mkdirSync(path.dirname(abs), { recursive: true });
|
|
44
|
+
fs.writeFileSync(abs, content);
|
|
45
|
+
return abs;
|
|
46
|
+
}
|
|
47
|
+
|
|
48
|
+
// ---------------------------------------------------------------------------
|
|
49
|
+
// discoverPrebuildScripts
|
|
50
|
+
// ---------------------------------------------------------------------------
|
|
51
|
+
|
|
52
|
+
describe("discoverPrebuildScripts", () => {
|
|
53
|
+
let dir = "";
|
|
54
|
+
beforeEach(() => { dir = mktmp("discover-"); });
|
|
55
|
+
afterEach(() => { fs.rmSync(dir, { recursive: true, force: true }); });
|
|
56
|
+
|
|
57
|
+
it("returns empty when scripts/ does not exist", () => {
|
|
58
|
+
expect(discoverPrebuildScripts(dir)).toEqual([]);
|
|
59
|
+
});
|
|
60
|
+
|
|
61
|
+
it("returns empty when scripts/ exists but has no prebuild files", () => {
|
|
62
|
+
writeFile(dir, "scripts/build.ts");
|
|
63
|
+
writeFile(dir, "scripts/README.md");
|
|
64
|
+
expect(discoverPrebuildScripts(dir)).toEqual([]);
|
|
65
|
+
});
|
|
66
|
+
|
|
67
|
+
it("finds prebuild-<name>.ts files", () => {
|
|
68
|
+
writeFile(dir, "scripts/prebuild-docs.ts");
|
|
69
|
+
writeFile(dir, "scripts/prebuild-seo.ts");
|
|
70
|
+
const found = discoverPrebuildScripts(dir);
|
|
71
|
+
expect(found).toHaveLength(2);
|
|
72
|
+
expect(found[0]).toContain("prebuild-docs.ts");
|
|
73
|
+
expect(found[1]).toContain("prebuild-seo.ts");
|
|
74
|
+
});
|
|
75
|
+
|
|
76
|
+
it("sorts lexicographically so numeric prefix controls ordering", () => {
|
|
77
|
+
writeFile(dir, "scripts/prebuild-20-second.ts");
|
|
78
|
+
writeFile(dir, "scripts/prebuild-10-first.ts");
|
|
79
|
+
writeFile(dir, "scripts/prebuild-99-third.ts");
|
|
80
|
+
const found = discoverPrebuildScripts(dir).map((p) => path.basename(p));
|
|
81
|
+
expect(found).toEqual([
|
|
82
|
+
"prebuild-10-first.ts",
|
|
83
|
+
"prebuild-20-second.ts",
|
|
84
|
+
"prebuild-99-third.ts",
|
|
85
|
+
]);
|
|
86
|
+
});
|
|
87
|
+
|
|
88
|
+
it("accepts .tsx, .js, .mjs extensions", () => {
|
|
89
|
+
writeFile(dir, "scripts/prebuild-a.tsx");
|
|
90
|
+
writeFile(dir, "scripts/prebuild-b.js");
|
|
91
|
+
writeFile(dir, "scripts/prebuild-c.mjs");
|
|
92
|
+
const found = discoverPrebuildScripts(dir).map((p) => path.basename(p));
|
|
93
|
+
expect(found).toEqual(["prebuild-a.tsx", "prebuild-b.js", "prebuild-c.mjs"]);
|
|
94
|
+
});
|
|
95
|
+
|
|
96
|
+
it("rejects non-prebuild filenames even if in scripts/", () => {
|
|
97
|
+
writeFile(dir, "scripts/my-prebuild.ts"); // doesn't start with prebuild
|
|
98
|
+
writeFile(dir, "scripts/prebuild.py"); // wrong extension
|
|
99
|
+
writeFile(dir, "scripts/prebuild-valid.ts");
|
|
100
|
+
const found = discoverPrebuildScripts(dir).map((p) => path.basename(p));
|
|
101
|
+
expect(found).toEqual(["prebuild-valid.ts"]);
|
|
102
|
+
});
|
|
103
|
+
|
|
104
|
+
it("honors custom scriptsDir", () => {
|
|
105
|
+
writeFile(dir, "custom/prebuild-1.ts");
|
|
106
|
+
expect(discoverPrebuildScripts(dir, "custom")).toHaveLength(1);
|
|
107
|
+
});
|
|
108
|
+
});
|
|
109
|
+
|
|
110
|
+
// ---------------------------------------------------------------------------
|
|
111
|
+
// shouldAutoPrebuild
|
|
112
|
+
// ---------------------------------------------------------------------------
|
|
113
|
+
|
|
114
|
+
describe("shouldAutoPrebuild", () => {
|
|
115
|
+
let dir = "";
|
|
116
|
+
beforeEach(() => { dir = mktmp("policy-"); });
|
|
117
|
+
afterEach(() => { fs.rmSync(dir, { recursive: true, force: true }); });
|
|
118
|
+
|
|
119
|
+
it("false when neither content/ nor prebuild scripts exist", () => {
|
|
120
|
+
expect(shouldAutoPrebuild(dir)).toBe(false);
|
|
121
|
+
});
|
|
122
|
+
|
|
123
|
+
it("true when content/ directory exists (even if empty)", () => {
|
|
124
|
+
fs.mkdirSync(path.join(dir, "content"));
|
|
125
|
+
expect(shouldAutoPrebuild(dir)).toBe(true);
|
|
126
|
+
});
|
|
127
|
+
|
|
128
|
+
it("true when a prebuild script exists but content/ does not", () => {
|
|
129
|
+
writeFile(dir, "scripts/prebuild-docs.ts");
|
|
130
|
+
expect(shouldAutoPrebuild(dir)).toBe(true);
|
|
131
|
+
});
|
|
132
|
+
|
|
133
|
+
it("false when content/ is a file, not a directory", () => {
|
|
134
|
+
fs.writeFileSync(path.join(dir, "content"), "not a dir");
|
|
135
|
+
expect(shouldAutoPrebuild(dir)).toBe(false);
|
|
136
|
+
});
|
|
137
|
+
});
|
|
138
|
+
|
|
139
|
+
// ---------------------------------------------------------------------------
|
|
140
|
+
// runPrebuildScripts with mock spawn
|
|
141
|
+
// ---------------------------------------------------------------------------
|
|
142
|
+
|
|
143
|
+
function makeMockSpawn(
|
|
144
|
+
policy: (scriptPath: string) => { exitCode: number | null; durationMs: number } | Promise<{ exitCode: number | null; durationMs: number }>,
|
|
145
|
+
): SpawnHook & { calls: Array<{ scriptPath: string; cwd: string; timeoutMs: number }> } {
|
|
146
|
+
const calls: Array<{ scriptPath: string; cwd: string; timeoutMs: number }> = [];
|
|
147
|
+
const hook: SpawnHook = async (args) => {
|
|
148
|
+
calls.push(args);
|
|
149
|
+
return await policy(args.scriptPath);
|
|
150
|
+
};
|
|
151
|
+
return Object.assign(hook, { calls });
|
|
152
|
+
}
|
|
153
|
+
|
|
154
|
+
describe("runPrebuildScripts", () => {
|
|
155
|
+
let dir = "";
|
|
156
|
+
beforeEach(() => { dir = mktmp("run-"); });
|
|
157
|
+
afterEach(() => { fs.rmSync(dir, { recursive: true, force: true }); });
|
|
158
|
+
|
|
159
|
+
it("returns { ran: 0 } when no scripts exist — silent no-op", async () => {
|
|
160
|
+
const spawn = makeMockSpawn(() => ({ exitCode: 0, durationMs: 5 }));
|
|
161
|
+
const result = await runPrebuildScripts({ rootDir: dir, spawn });
|
|
162
|
+
expect(result.ran).toBe(0);
|
|
163
|
+
expect(spawn.calls).toHaveLength(0);
|
|
164
|
+
});
|
|
165
|
+
|
|
166
|
+
it("runs discovered scripts in order", async () => {
|
|
167
|
+
writeFile(dir, "scripts/prebuild-b.ts");
|
|
168
|
+
writeFile(dir, "scripts/prebuild-a.ts");
|
|
169
|
+
const order: string[] = [];
|
|
170
|
+
const spawn = makeMockSpawn(async (p) => {
|
|
171
|
+
order.push(path.basename(p));
|
|
172
|
+
return { exitCode: 0, durationMs: 1 };
|
|
173
|
+
});
|
|
174
|
+
const result = await runPrebuildScripts({ rootDir: dir, spawn });
|
|
175
|
+
expect(result.ran).toBe(2);
|
|
176
|
+
expect(order).toEqual(["prebuild-a.ts", "prebuild-b.ts"]);
|
|
177
|
+
});
|
|
178
|
+
|
|
179
|
+
it("throws PrebuildError with exitCode on first non-zero exit", async () => {
|
|
180
|
+
writeFile(dir, "scripts/prebuild-1.ts");
|
|
181
|
+
writeFile(dir, "scripts/prebuild-2.ts");
|
|
182
|
+
const spawn = makeMockSpawn((p) => ({
|
|
183
|
+
exitCode: path.basename(p) === "prebuild-1.ts" ? 3 : 0,
|
|
184
|
+
durationMs: 1,
|
|
185
|
+
}));
|
|
186
|
+
let caught: unknown;
|
|
187
|
+
try {
|
|
188
|
+
await runPrebuildScripts({ rootDir: dir, spawn });
|
|
189
|
+
} catch (e) {
|
|
190
|
+
caught = e;
|
|
191
|
+
}
|
|
192
|
+
expect(caught).toBeInstanceOf(PrebuildError);
|
|
193
|
+
const err = caught as PrebuildError;
|
|
194
|
+
expect(err.exitCode).toBe(3);
|
|
195
|
+
expect(err.scriptPath).toContain("prebuild-1.ts");
|
|
196
|
+
// prebuild-2.ts must NOT have been invoked (fail-fast chain semantics).
|
|
197
|
+
expect(spawn.calls).toHaveLength(1);
|
|
198
|
+
});
|
|
199
|
+
|
|
200
|
+
it("wraps non-PrebuildError spawn rejections", async () => {
|
|
201
|
+
writeFile(dir, "scripts/prebuild.ts");
|
|
202
|
+
const spawn: SpawnHook = async () => {
|
|
203
|
+
throw new Error("spawn ENOENT");
|
|
204
|
+
};
|
|
205
|
+
let caught: unknown;
|
|
206
|
+
try {
|
|
207
|
+
await runPrebuildScripts({ rootDir: dir, spawn });
|
|
208
|
+
} catch (e) {
|
|
209
|
+
caught = e;
|
|
210
|
+
}
|
|
211
|
+
expect(caught).toBeInstanceOf(PrebuildError);
|
|
212
|
+
expect((caught as PrebuildError).message).toContain("spawn ENOENT");
|
|
213
|
+
});
|
|
214
|
+
|
|
215
|
+
it("invokes onStart / onFinish callbacks for each script", async () => {
|
|
216
|
+
writeFile(dir, "scripts/prebuild-a.ts");
|
|
217
|
+
writeFile(dir, "scripts/prebuild-b.ts");
|
|
218
|
+
const starts: string[] = [];
|
|
219
|
+
const finishes: Array<{ name: string; code: number | null }> = [];
|
|
220
|
+
await runPrebuildScripts({
|
|
221
|
+
rootDir: dir,
|
|
222
|
+
spawn: async () => ({ exitCode: 0, durationMs: 1 }),
|
|
223
|
+
onStart: (p, i, total) => {
|
|
224
|
+
starts.push(`${i + 1}/${total}:${path.basename(p)}`);
|
|
225
|
+
},
|
|
226
|
+
onFinish: (r) => {
|
|
227
|
+
finishes.push({ name: path.basename(r.scriptPath), code: r.exitCode });
|
|
228
|
+
},
|
|
229
|
+
});
|
|
230
|
+
expect(starts).toEqual(["1/2:prebuild-a.ts", "2/2:prebuild-b.ts"]);
|
|
231
|
+
expect(finishes).toEqual([
|
|
232
|
+
{ name: "prebuild-a.ts", code: 0 },
|
|
233
|
+
{ name: "prebuild-b.ts", code: 0 },
|
|
234
|
+
]);
|
|
235
|
+
});
|
|
236
|
+
|
|
237
|
+
it("threads timeoutMs through to the spawn hook", async () => {
|
|
238
|
+
writeFile(dir, "scripts/prebuild-1.ts");
|
|
239
|
+
const spawn = makeMockSpawn(() => ({ exitCode: 0, durationMs: 1 }));
|
|
240
|
+
await runPrebuildScripts({
|
|
241
|
+
rootDir: dir,
|
|
242
|
+
spawn,
|
|
243
|
+
timeoutMs: 5_000,
|
|
244
|
+
});
|
|
245
|
+
expect(spawn.calls[0].timeoutMs).toBe(5_000);
|
|
246
|
+
});
|
|
247
|
+
|
|
248
|
+
it("uses default 2-minute timeout when not overridden", async () => {
|
|
249
|
+
writeFile(dir, "scripts/prebuild-1.ts");
|
|
250
|
+
const spawn = makeMockSpawn(() => ({ exitCode: 0, durationMs: 1 }));
|
|
251
|
+
await runPrebuildScripts({ rootDir: dir, spawn });
|
|
252
|
+
expect(spawn.calls[0].timeoutMs).toBe(2 * 60 * 1000);
|
|
253
|
+
});
|
|
254
|
+
});
|
|
255
|
+
|
|
256
|
+
// ---------------------------------------------------------------------------
|
|
257
|
+
// resolvePrebuildTimeout (Issue #203)
|
|
258
|
+
// ---------------------------------------------------------------------------
|
|
259
|
+
|
|
260
|
+
describe("resolvePrebuildTimeout", () => {
|
|
261
|
+
const originalEnv = process.env[PREBUILD_TIMEOUT_ENV];
|
|
262
|
+
afterEach(() => {
|
|
263
|
+
if (originalEnv === undefined) delete process.env[PREBUILD_TIMEOUT_ENV];
|
|
264
|
+
else process.env[PREBUILD_TIMEOUT_ENV] = originalEnv;
|
|
265
|
+
});
|
|
266
|
+
|
|
267
|
+
it("returns DEFAULT_PREBUILD_TIMEOUT_MS (120s) when no override", () => {
|
|
268
|
+
delete process.env[PREBUILD_TIMEOUT_ENV];
|
|
269
|
+
expect(resolvePrebuildTimeout()).toBe(DEFAULT_PREBUILD_TIMEOUT_MS);
|
|
270
|
+
expect(DEFAULT_PREBUILD_TIMEOUT_MS).toBe(120_000);
|
|
271
|
+
});
|
|
272
|
+
|
|
273
|
+
it("explicit arg wins over env var", () => {
|
|
274
|
+
process.env[PREBUILD_TIMEOUT_ENV] = "5000";
|
|
275
|
+
expect(resolvePrebuildTimeout(1234)).toBe(1234);
|
|
276
|
+
});
|
|
277
|
+
|
|
278
|
+
it("env var is used when no explicit arg", () => {
|
|
279
|
+
process.env[PREBUILD_TIMEOUT_ENV] = "7500";
|
|
280
|
+
expect(resolvePrebuildTimeout()).toBe(7500);
|
|
281
|
+
});
|
|
282
|
+
|
|
283
|
+
it("ignores invalid env var (non-numeric) and falls back to default", () => {
|
|
284
|
+
process.env[PREBUILD_TIMEOUT_ENV] = "not-a-number";
|
|
285
|
+
expect(resolvePrebuildTimeout()).toBe(DEFAULT_PREBUILD_TIMEOUT_MS);
|
|
286
|
+
});
|
|
287
|
+
|
|
288
|
+
it("ignores invalid env var (zero / negative) and falls back to default", () => {
|
|
289
|
+
process.env[PREBUILD_TIMEOUT_ENV] = "0";
|
|
290
|
+
expect(resolvePrebuildTimeout()).toBe(DEFAULT_PREBUILD_TIMEOUT_MS);
|
|
291
|
+
process.env[PREBUILD_TIMEOUT_ENV] = "-5";
|
|
292
|
+
expect(resolvePrebuildTimeout()).toBe(DEFAULT_PREBUILD_TIMEOUT_MS);
|
|
293
|
+
});
|
|
294
|
+
|
|
295
|
+
it("ignores non-positive explicit arg", () => {
|
|
296
|
+
delete process.env[PREBUILD_TIMEOUT_ENV];
|
|
297
|
+
expect(resolvePrebuildTimeout(0)).toBe(DEFAULT_PREBUILD_TIMEOUT_MS);
|
|
298
|
+
expect(resolvePrebuildTimeout(-1)).toBe(DEFAULT_PREBUILD_TIMEOUT_MS);
|
|
299
|
+
});
|
|
300
|
+
});
|
|
301
|
+
|
|
302
|
+
// ---------------------------------------------------------------------------
|
|
303
|
+
// Issue #203 — Timeout + error surface regressions
|
|
304
|
+
// ---------------------------------------------------------------------------
|
|
305
|
+
|
|
306
|
+
describe("runPrebuildScripts — timeout surface (Issue #203)", () => {
|
|
307
|
+
let dir = "";
|
|
308
|
+
beforeEach(() => { dir = mktmp("timeout-"); });
|
|
309
|
+
afterEach(() => { fs.rmSync(dir, { recursive: true, force: true }); });
|
|
310
|
+
|
|
311
|
+
it("honours MANDU_PREBUILD_TIMEOUT_MS env override when timeoutMs omitted", async () => {
|
|
312
|
+
const originalEnv = process.env[PREBUILD_TIMEOUT_ENV];
|
|
313
|
+
process.env[PREBUILD_TIMEOUT_ENV] = "9999";
|
|
314
|
+
try {
|
|
315
|
+
writeFile(dir, "scripts/prebuild-1.ts");
|
|
316
|
+
const spawn = makeMockSpawn(() => ({ exitCode: 0, durationMs: 1 }));
|
|
317
|
+
await runPrebuildScripts({ rootDir: dir, spawn });
|
|
318
|
+
expect(spawn.calls[0].timeoutMs).toBe(9999);
|
|
319
|
+
} finally {
|
|
320
|
+
if (originalEnv === undefined) delete process.env[PREBUILD_TIMEOUT_ENV];
|
|
321
|
+
else process.env[PREBUILD_TIMEOUT_ENV] = originalEnv;
|
|
322
|
+
}
|
|
323
|
+
});
|
|
324
|
+
|
|
325
|
+
it("explicit timeoutMs wins over env var", async () => {
|
|
326
|
+
const originalEnv = process.env[PREBUILD_TIMEOUT_ENV];
|
|
327
|
+
process.env[PREBUILD_TIMEOUT_ENV] = "9999";
|
|
328
|
+
try {
|
|
329
|
+
writeFile(dir, "scripts/prebuild-1.ts");
|
|
330
|
+
const spawn = makeMockSpawn(() => ({ exitCode: 0, durationMs: 1 }));
|
|
331
|
+
await runPrebuildScripts({ rootDir: dir, spawn, timeoutMs: 2222 });
|
|
332
|
+
expect(spawn.calls[0].timeoutMs).toBe(2222);
|
|
333
|
+
} finally {
|
|
334
|
+
if (originalEnv === undefined) delete process.env[PREBUILD_TIMEOUT_ENV];
|
|
335
|
+
else process.env[PREBUILD_TIMEOUT_ENV] = originalEnv;
|
|
336
|
+
}
|
|
337
|
+
});
|
|
338
|
+
|
|
339
|
+
it("defaultSpawn throws PrebuildTimeoutError with script name + limit when the script runs longer than timeout", async () => {
|
|
340
|
+
// Script sleeps 300ms — we set timeout 80ms, so the timer must win.
|
|
341
|
+
// We use `Bun.sleep` + `process.exit(0)` so the script exits cleanly
|
|
342
|
+
// if somehow the kill is skipped (no zombie in test harness).
|
|
343
|
+
const scriptAbs = writeFile(
|
|
344
|
+
dir,
|
|
345
|
+
"scripts/prebuild-slow.ts",
|
|
346
|
+
"await Bun.sleep(300);\nprocess.exit(0);\n",
|
|
347
|
+
);
|
|
348
|
+
|
|
349
|
+
let caught: unknown;
|
|
350
|
+
try {
|
|
351
|
+
await defaultSpawn({ scriptPath: scriptAbs, cwd: dir, timeoutMs: 80 });
|
|
352
|
+
} catch (e) {
|
|
353
|
+
caught = e;
|
|
354
|
+
}
|
|
355
|
+
expect(caught).toBeInstanceOf(PrebuildTimeoutError);
|
|
356
|
+
expect(caught).toBeInstanceOf(PrebuildError); // subclass relationship
|
|
357
|
+
const err = caught as PrebuildTimeoutError;
|
|
358
|
+
expect(err.timeoutMs).toBe(80);
|
|
359
|
+
expect(err.scriptPath).toContain("prebuild-slow.ts");
|
|
360
|
+
// Message contract: includes script name + timeout limit + override hint.
|
|
361
|
+
expect(err.message).toContain("prebuild-slow.ts");
|
|
362
|
+
expect(err.message).toContain("80ms");
|
|
363
|
+
expect(err.message).toContain("dev.prebuildTimeoutMs");
|
|
364
|
+
expect(err.message).toContain(PREBUILD_TIMEOUT_ENV);
|
|
365
|
+
// Regression beacon: message MUST NOT contain the opaque
|
|
366
|
+
// "non-Error thrown" string Issue #203 was reported against.
|
|
367
|
+
expect(err.message).not.toContain("non-Error thrown");
|
|
368
|
+
});
|
|
369
|
+
|
|
370
|
+
it("runPrebuildScripts surfaces PrebuildTimeoutError via injected spawn", async () => {
|
|
371
|
+
writeFile(dir, "scripts/prebuild-slow.ts");
|
|
372
|
+
const spawn: SpawnHook = async (args) => {
|
|
373
|
+
throw new PrebuildTimeoutError({
|
|
374
|
+
scriptPath: args.scriptPath,
|
|
375
|
+
timeoutMs: args.timeoutMs,
|
|
376
|
+
durationMs: args.timeoutMs,
|
|
377
|
+
});
|
|
378
|
+
};
|
|
379
|
+
let caught: unknown;
|
|
380
|
+
try {
|
|
381
|
+
await runPrebuildScripts({ rootDir: dir, spawn, timeoutMs: 100 });
|
|
382
|
+
} catch (e) {
|
|
383
|
+
caught = e;
|
|
384
|
+
}
|
|
385
|
+
expect(caught).toBeInstanceOf(PrebuildTimeoutError);
|
|
386
|
+
const err = caught as PrebuildTimeoutError;
|
|
387
|
+
expect(err.timeoutMs).toBe(100);
|
|
388
|
+
expect(err.scriptPath).toContain("prebuild-slow.ts");
|
|
389
|
+
});
|
|
390
|
+
});
|
|
391
|
+
|
|
392
|
+
describe("runPrebuildScripts — error preservation (Issue #203)", () => {
|
|
393
|
+
let dir = "";
|
|
394
|
+
beforeEach(() => { dir = mktmp("err-surface-"); });
|
|
395
|
+
afterEach(() => { fs.rmSync(dir, { recursive: true, force: true }); });
|
|
396
|
+
|
|
397
|
+
it("preserves the inner Error message ('boom') when a spawn rejection occurs", async () => {
|
|
398
|
+
writeFile(dir, "scripts/prebuild.ts");
|
|
399
|
+
const inner = new Error("boom");
|
|
400
|
+
const spawn: SpawnHook = async () => {
|
|
401
|
+
throw inner;
|
|
402
|
+
};
|
|
403
|
+
let caught: unknown;
|
|
404
|
+
try {
|
|
405
|
+
await runPrebuildScripts({ rootDir: dir, spawn });
|
|
406
|
+
} catch (e) {
|
|
407
|
+
caught = e;
|
|
408
|
+
}
|
|
409
|
+
expect(caught).toBeInstanceOf(PrebuildError);
|
|
410
|
+
const err = caught as PrebuildError & { cause?: unknown };
|
|
411
|
+
// The inner message MUST appear verbatim — the whole point of #203.
|
|
412
|
+
expect(err.message).toContain("boom");
|
|
413
|
+
// Regression beacon.
|
|
414
|
+
expect(err.message).not.toContain("non-Error thrown");
|
|
415
|
+
// The inner error is attached as `.cause` so debug tooling can
|
|
416
|
+
// recover its stack.
|
|
417
|
+
expect(err.cause).toBe(inner);
|
|
418
|
+
});
|
|
419
|
+
|
|
420
|
+
it("preserves stack via .cause when inner Error has a stack", async () => {
|
|
421
|
+
writeFile(dir, "scripts/prebuild.ts");
|
|
422
|
+
const inner = new Error("kaboom");
|
|
423
|
+
const originalStack = inner.stack;
|
|
424
|
+
expect(originalStack).toBeTruthy();
|
|
425
|
+
const spawn: SpawnHook = async () => {
|
|
426
|
+
throw inner;
|
|
427
|
+
};
|
|
428
|
+
let caught: unknown;
|
|
429
|
+
try {
|
|
430
|
+
await runPrebuildScripts({ rootDir: dir, spawn });
|
|
431
|
+
} catch (e) {
|
|
432
|
+
caught = e;
|
|
433
|
+
}
|
|
434
|
+
const err = caught as PrebuildError & { cause?: Error };
|
|
435
|
+
expect(err.cause).toBe(inner);
|
|
436
|
+
expect(err.cause?.stack).toBe(originalStack);
|
|
437
|
+
});
|
|
438
|
+
|
|
439
|
+
it("handles non-Error rejections (string) without producing 'non-Error thrown'", async () => {
|
|
440
|
+
writeFile(dir, "scripts/prebuild.ts");
|
|
441
|
+
const spawn: SpawnHook = async () => {
|
|
442
|
+
// Raw string throw — the pathological case Issue #203 was about.
|
|
443
|
+
throw "spawn blew up";
|
|
444
|
+
};
|
|
445
|
+
let caught: unknown;
|
|
446
|
+
try {
|
|
447
|
+
await runPrebuildScripts({ rootDir: dir, spawn });
|
|
448
|
+
} catch (e) {
|
|
449
|
+
caught = e;
|
|
450
|
+
}
|
|
451
|
+
expect(caught).toBeInstanceOf(PrebuildError);
|
|
452
|
+
const err = caught as PrebuildError & { cause?: unknown };
|
|
453
|
+
// The raw string survives into `.message`.
|
|
454
|
+
expect(err.message).toContain("spawn blew up");
|
|
455
|
+
expect(err.message).not.toContain("non-Error thrown");
|
|
456
|
+
expect(err.cause).toBe("spawn blew up");
|
|
457
|
+
});
|
|
458
|
+
|
|
459
|
+
it("handles non-Error rejections (object) without producing 'non-Error thrown'", async () => {
|
|
460
|
+
writeFile(dir, "scripts/prebuild.ts");
|
|
461
|
+
const raw = { code: "EBADF", info: "fd closed" };
|
|
462
|
+
const spawn: SpawnHook = async () => {
|
|
463
|
+
throw raw;
|
|
464
|
+
};
|
|
465
|
+
let caught: unknown;
|
|
466
|
+
try {
|
|
467
|
+
await runPrebuildScripts({ rootDir: dir, spawn });
|
|
468
|
+
} catch (e) {
|
|
469
|
+
caught = e;
|
|
470
|
+
}
|
|
471
|
+
const err = caught as PrebuildError & { cause?: unknown };
|
|
472
|
+
expect(err.message).toContain("EBADF");
|
|
473
|
+
expect(err.message).not.toContain("non-Error thrown");
|
|
474
|
+
expect(err.cause).toBe(raw);
|
|
475
|
+
});
|
|
476
|
+
|
|
477
|
+
it("includes stdout/stderr tails in PrebuildError.message for non-zero exit", async () => {
|
|
478
|
+
writeFile(dir, "scripts/prebuild.ts");
|
|
479
|
+
const stderrSample = Array.from({ length: 15 }, (_, i) => `err-line-${i + 1}`).join("\n");
|
|
480
|
+
const stdoutSample = Array.from({ length: 12 }, (_, i) => `out-line-${i + 1}`).join("\n");
|
|
481
|
+
const spawn: SpawnHook = async () => ({
|
|
482
|
+
exitCode: 1,
|
|
483
|
+
durationMs: 5,
|
|
484
|
+
stdoutTail: stdoutSample,
|
|
485
|
+
stderrTail: stderrSample,
|
|
486
|
+
});
|
|
487
|
+
let caught: unknown;
|
|
488
|
+
try {
|
|
489
|
+
await runPrebuildScripts({ rootDir: dir, spawn });
|
|
490
|
+
} catch (e) {
|
|
491
|
+
caught = e;
|
|
492
|
+
}
|
|
493
|
+
expect(caught).toBeInstanceOf(PrebuildError);
|
|
494
|
+
const err = caught as PrebuildError;
|
|
495
|
+
// Exactly last 10 stderr lines are kept.
|
|
496
|
+
expect(err.message).toContain("err-line-15");
|
|
497
|
+
expect(err.message).toContain("err-line-6"); // the 10-from-last
|
|
498
|
+
expect(err.message).not.toContain("err-line-5"); // trimmed
|
|
499
|
+
// Stdout tail is also present (10 lines out of 12).
|
|
500
|
+
expect(err.message).toContain("out-line-12");
|
|
501
|
+
expect(err.message).toContain("out-line-3");
|
|
502
|
+
expect(err.message).not.toContain("out-line-2");
|
|
503
|
+
// Structured fields carry the full captured tails.
|
|
504
|
+
expect(err.stderrTail).toBe(stderrSample);
|
|
505
|
+
expect(err.stdoutTail).toBe(stdoutSample);
|
|
506
|
+
});
|
|
507
|
+
|
|
508
|
+
it("omits empty tail sections when the spawn hook didn't capture", async () => {
|
|
509
|
+
writeFile(dir, "scripts/prebuild.ts");
|
|
510
|
+
const spawn: SpawnHook = async () => ({ exitCode: 2, durationMs: 3 });
|
|
511
|
+
let caught: unknown;
|
|
512
|
+
try {
|
|
513
|
+
await runPrebuildScripts({ rootDir: dir, spawn });
|
|
514
|
+
} catch (e) {
|
|
515
|
+
caught = e;
|
|
516
|
+
}
|
|
517
|
+
const err = caught as PrebuildError;
|
|
518
|
+
// The "exited with code 2" prefix is present but no "--- stderr ---" sections.
|
|
519
|
+
expect(err.message).toContain("exited with code 2");
|
|
520
|
+
expect(err.message).not.toContain("--- stderr");
|
|
521
|
+
expect(err.message).not.toContain("--- stdout");
|
|
522
|
+
});
|
|
523
|
+
|
|
524
|
+
it("normalizes Windows-style CRLF line endings in stderr tail", async () => {
|
|
525
|
+
writeFile(dir, "scripts/prebuild.ts");
|
|
526
|
+
const crlf = ["a", "b", "c", "d"].join("\r\n");
|
|
527
|
+
const spawn: SpawnHook = async () => ({
|
|
528
|
+
exitCode: 1,
|
|
529
|
+
durationMs: 1,
|
|
530
|
+
stderrTail: crlf,
|
|
531
|
+
});
|
|
532
|
+
let caught: unknown;
|
|
533
|
+
try {
|
|
534
|
+
await runPrebuildScripts({ rootDir: dir, spawn });
|
|
535
|
+
} catch (e) {
|
|
536
|
+
caught = e;
|
|
537
|
+
}
|
|
538
|
+
const err = caught as PrebuildError;
|
|
539
|
+
// All 4 lines survive (under the 10-line cap).
|
|
540
|
+
expect(err.message).toContain("a\nb\nc\nd");
|
|
541
|
+
// No raw \r\n survived.
|
|
542
|
+
expect(err.message).not.toContain("\r\n");
|
|
543
|
+
});
|
|
544
|
+
});
|
|
545
|
+
|
|
546
|
+
describe("PrebuildTimeoutError shape (Issue #203)", () => {
|
|
547
|
+
it("is a subclass of PrebuildError so existing `instanceof PrebuildError` callers still match", () => {
|
|
548
|
+
const err = new PrebuildTimeoutError({
|
|
549
|
+
scriptPath: "/tmp/foo/scripts/prebuild.ts",
|
|
550
|
+
timeoutMs: 500,
|
|
551
|
+
durationMs: 500,
|
|
552
|
+
});
|
|
553
|
+
expect(err).toBeInstanceOf(PrebuildTimeoutError);
|
|
554
|
+
expect(err).toBeInstanceOf(PrebuildError);
|
|
555
|
+
expect(err).toBeInstanceOf(Error);
|
|
556
|
+
expect(err.name).toBe("PrebuildTimeoutError");
|
|
557
|
+
expect(err.exitCode).toBeNull();
|
|
558
|
+
});
|
|
559
|
+
|
|
560
|
+
it("message names the script + limit + override paths", () => {
|
|
561
|
+
const err = new PrebuildTimeoutError({
|
|
562
|
+
scriptPath: "/repo/scripts/prebuild-seed.ts",
|
|
563
|
+
timeoutMs: 250,
|
|
564
|
+
durationMs: 250,
|
|
565
|
+
});
|
|
566
|
+
expect(err.message).toContain("prebuild-seed.ts");
|
|
567
|
+
expect(err.message).toContain("250ms");
|
|
568
|
+
expect(err.message).toContain("dev.prebuildTimeoutMs");
|
|
569
|
+
expect(err.message).toContain(PREBUILD_TIMEOUT_ENV);
|
|
570
|
+
});
|
|
571
|
+
});
|