apps/Alex2Node
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

build: refuse unknown arguments; name the fix when typescript is missing

Browse source 
Two failures of scripts/build.mjs did not say what to change. `node scripts/build.mjs --outt /some/dir` ignored
the misspelt flag, exited 0 and replaced the tracked dist/. Without devDependencies (npm install --omit=dev in a
clone runs "prepare") the script died with a stack trace, because typescript was resolved at module level,
outside the try block.

The script now takes no arguments or `--out <dir>` and answers anything else with one line and the usage, exit 1.
A missing compiler gives "build: typescript is not installed: run npm install (it is a devDependency)". The body
moved into build(args), so the staging directory and the compiler path are no longer module-level variables.

Two tests on a temporary package: three refused argument lists, and a package without node_modules; both compare
the package before and after. No change to dist/. npm test: 30 pass in 9.5-10.2 s (load average 7 on 4 cores).

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
This commit is contained in:
David 2026-09-28 14:42:33 +00:00
parent ee311dab01
commit d96c6b8372
2 changed files with 71 additions and 27 deletions
Download patch file Download diff file Expand all files Collapse all files

33
test/build.test.js
View file

@ -11,8 +11,8 @@ const { spawnSync } = require("node:child_process");
const ROOT = path.join(__dirname, "..");
// A package directory with the build script, the three tsconfig files, src/index.ts and the dist/ of an earlier
// build; removed when the test ends.
function scratchPackage(t, source) {
// build; removed when the test ends. installed: false leaves node_modules out, as before npm install.
function scratchPackage(t, source, { installed = true } = {}) {
const dir = fs.mkdtempSync(path.join(os.tmpdir(), "alex2node-build-"));
t.after(() => fs.rmSync(dir, { recursive: true, force: true }));
for (const file of ["scripts/build.mjs", "tsconfig.json", "tsconfig.cjs.json", "tsconfig.esm.json", "src/index.ts"]) {
@ -24,7 +24,7 @@ function scratchPackage(t, source) {
fs.mkdirSync(path.join(dir, "dist", part), { recursive: true });
fs.writeFileSync(path.join(dir, "dist", part, "earlier.txt"), `${part} of the earlier build\n`);
}
fs.symlinkSync(path.join(ROOT, "node_modules"), path.join(dir, "node_modules"), "dir");
if (installed) fs.symlinkSync(path.join(ROOT, "node_modules"), path.join(dir, "node_modules"), "dir");
return dir;
}
@ -82,3 +82,30 @@ test("the second compiler run fails: what the first one wrote is dropped, dist/
assert.match(run.stderr, /^build: tsc -p tsconfig\.esm\.json failed/m);
assert.deepEqual(snapshot(dir), before);
});
test("an argument the script does not know is refused: a misspelt --out must not build into dist/", (t) => {
const dir = scratchPackage(t, "export const port = 1883;\n");
const before = snapshot(dir);
const elsewhere = path.join(dir, "elsewhere");
const refused = [
[["--outt", elsewhere], "unknown argument --outt"],
[["--out"], "--out needs a directory"],
[["--out", elsewhere, "--watch"], "unknown argument --watch"],
];
for (const [args, reason] of refused) {
const run = build(dir, ...args);
assert.equal(run.status, 1, args.join(" "));
assert.equal(run.stderr, `build: ${reason}; usage: node scripts/build.mjs [--out <dir>]\n`);
assert.deepEqual(snapshot(dir), before, args.join(" "));
}
});
test("without typescript the build says what to install, in one line", (t) => {
const dir = scratchPackage(t, "export const port = 1883;\n", { installed: false });
const before = snapshot(dir);
const run = build(dir);
assert.equal(run.status, 1);
assert.equal(run.stderr, "build: typescript is not installed: run npm install (it is a devDependency)\n");
assert.deepEqual(snapshot(dir), before);
});