Skip to content

Commit c04b36d

Browse files
committed
build(cutover): the ui cli spells the sourcesContent hook inline instead of importing the parent's (bill item 45 follow-up, objectstack#25803)
d2342a1 made ui/packages/cli/tsup.config.ts import ../../../scripts/tsup-drop-sources-content.mjs. The ui turbo-build-inputs guard (scripts/__tests__/turbo-build-inputs.test.ts) then found a build program reading a file no `$TURBO_ROOT$` input of the parked ui turbo config covers, and its on-disk matcher walks the ui/ root only, so a `$TURBO_ROOT$/../scripts/...` entry cannot satisfy it either. The hook is one assignment, so the config now spells it inline and reads nothing outside ui/. Measured: rebuild + check:sourcemap-no-sources-content -> 73 built packages, 777 maps, none embed source text; ui ESLint 0; turbo-build-inputs 46/46 (was 1 red); the three other ui turbo input guards and the pin tests of patch 61 pass. Co-Authored-By: Claude <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01RSDL3SFtJaKzcymvCD8RsU
1 parent 454818a commit c04b36d

1 file changed

Lines changed: 8 additions & 3 deletions

File tree

‎ui/packages/cli/tsup.config.ts‎

Lines changed: 8 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -8,8 +8,6 @@
88

99
import { defineConfig } from 'tsup';
1010

11-
import { dropSourcesContent } from '../../../scripts/tsup-drop-sources-content.mjs';
12-
1311
export default defineConfig({
1412
entry: {
1513
cli: 'src/cli.ts',
@@ -20,5 +18,12 @@ export default defineConfig({
2018
clean: true,
2119
sourcemap: true,
2220
shims: false,
23-
esbuildOptions: dropSourcesContent,
21+
// The parent repo's `check:sourcemap-no-sources-content` refuses a published
22+
// map that embeds the original source text (#16469). Its shared
23+
// `scripts/tsup-drop-sources-content.mjs` hook is spelled inline here rather
24+
// than imported, so this build reads nothing outside the ui/ workspace and
25+
// the ui turbo-build-inputs guard needs no `$TURBO_ROOT$/../` entry.
26+
esbuildOptions(options) {
27+
options.sourcesContent = false;
28+
},
2429
});

0 commit comments

Comments
 (0)