@orkestrel/scaffold 0.0.78 → 0.0.79

package/dist/bin/main.js

@@ -502,6 +502,55 @@ function manifestToWritableDependencies(manifest, blueprint) {
 	};
 }
 /**
+* Reads the manifest scripts a writing verb may write.
+*
+* @param manifest - The target manifest text.
+* @param blueprint - The workspace shape that supplies the planned scripts.
+* @returns The blueprint's writable scripts, followed by the planned `test` chain when the
+* declared chain is a generated predecessor of it.
+*
+* @remarks
+* A declared `test` chain is a generated predecessor when it runs fewer `&&` steps than the
+* planned chain and every step it runs is a planned step, in the planned order. That is the chain
+* a birth or an earlier write left before a structural fact planned another step, such as a root
+* `tests/setupBrowser.test.ts` proof planning `npm run test:setup:browser`, so the write lands the
+* planned chain in its place. A chain running a step the plan does not generate, or running the
+* planned steps in another order, is one the package owns, and no entry names it.
+*
+* The planned chain joins only when every step it adds runs a script the manifest declares or the
+* writable scripts supply. The `test:src` and `test:app` aggregates sit outside the writable
+* scripts, so a planned chain adding an aggregate the manifest lacks stays with the package rather
+* than landing a step that names a missing script.
+*/
+function manifestToWritableScripts(manifest, blueprint) {
+	const writable = blueprintToWritableScripts(blueprint);
+	const planned = blueprintToScripts(blueprint).test;
+	const parsed = parseJSON(manifest);
+	const scripts = isRecord(parsed) && isRecord(parsed.scripts) ? parsed.scripts : {};
+	const declared = scripts.test;
+	if (planned === void 0 || !isString(declared)) return writable;
+	const steps = declared.split("&&").map((step) => step.trim());
+	const expected = planned.split("&&").map((step) => step.trim());
+	if (steps.length >= expected.length) return writable;
+	let cursor = 0;
+	for (const step of steps) {
+		while (cursor < expected.length && expected[cursor] !== step) cursor += 1;
+		if (cursor === expected.length) return writable;
+		cursor += 1;
+	}
+	const supplied = new Set(writable.map(({ name }) => name));
+	for (const step of expected) {
+		if (steps.includes(step)) continue;
+		const name = /^npm run (\S+)$/.exec(step)?.[1];
+		if (name === void 0 || !supplied.has(name) && !isString(scripts[name])) return writable;
+	}
+	return [...writable, {
+		name: "test",
+		command: planned,
+		accepted: [declared]
+	}];
+}
+/**
 * Keeps only dependencies published by the Orkestrel fleet.
 *
 * @param dependencies - The declared dependencies to inspect.
@@ -1447,7 +1496,7 @@ var CLI = class {
 			if (refusal !== void 0) throw refusal;
 			const result = mergeResults(host.materializer.repair(plan, audit, target), host.materializer.declare({
 				pins: versions.pins,
-				scripts: blueprintToWritableScripts(blueprint)
+				scripts: manifestToWritableScripts(this.#manifest(target), blueprint)
 			}, target));
 			const terminalBlueprint = this.#derive(target);
 			const [measured] = this.#survey(host.materializer, terminalBlueprint, target, groups);
@@ -1549,7 +1598,7 @@ var CLI = class {
 			}
 			const declared = {
 				pins: manifestToWritableDependencies(this.#manifest(target), blueprint),
-				scripts: blueprintToWritableScripts(blueprint)
+				scripts: manifestToWritableScripts(this.#manifest(target), blueprint)
 			};
 			const local = mergeResults(host.materializer.repair(plan, audit, target), host.materializer.remove(plan, audit, command.dirty === true ? {
 				tracked: worktree.tracked,
@@ -1878,7 +1927,8 @@ var CLI = class {
 	}
 	#projectQuestion(target, blueprint, writing = false) {
 		const text = this.#manifest(target);
-		const manifest = replaceManifestScripts(text, blueprintToWritableScripts(blueprint)) ?? text;
+		const writable = writing ? manifestToWritableScripts(text, blueprint) : blueprintToWritableScripts(blueprint);
+		const manifest = replaceManifestScripts(text, writable) ?? text;
 		const planned = blueprintToRootVite(blueprint);
 		const parsed = parseJSON(manifest);
 		const scripts = isRecord(parsed) && isRecord(parsed.scripts) ? parsed.scripts : {};
@@ -1984,7 +2034,7 @@ var CLI = class {
 		const direct = names.map((project) => `test:${project}`);
 		const declared = direct.filter((script) => isString(written[script]));
 		const undeclared = direct.filter((script) => !isString(written[script]));
-		const remedies = [declared.length === 0 ? "" : `${declared.join(", ")} ${declared.length === 1 ? "is" : "are"} already declared, so the gate is missing rather than the script: invoke ${declared.length === 1 ? "it" : "each of them"} by name from the ${gateNames} chain.`, undeclared.length === 0 ? "" : `${undeclared.join(", ")} ${undeclared.length === 1 ? "is" : "are"} not declared, so the script is missing as well as the gate: declare ${undeclared.length === 1 ? "it" : "each of them"} and invoke ${undeclared.length === 1 ? "it" : "each of them"} by name from the ${gateNames} chain.`];
+		const remedies = !writing && manifestToWritableScripts(text, blueprint).some(({ name }) => name === "test") ? ["The test chain runs generated steps only, so repair and overwrite write the planned chain and every missing direct script."] : [declared.length === 0 ? "" : `${declared.join(", ")} ${declared.length === 1 ? "is" : "are"} already declared, so the gate is missing rather than the script: invoke ${declared.length === 1 ? "it" : "each of them"} by name from the ${gateNames} chain.`, undeclared.length === 0 ? "" : `${undeclared.join(", ")} ${undeclared.length === 1 ? "is" : "are"} not declared, so the script is missing as well as the gate: declare ${undeclared.length === 1 ? "it" : "each of them"} and invoke ${undeclared.length === 1 ? "it" : "each of them"} by name from the ${gateNames} chain.`];
 		return {
 			field: "projects",
 			message: `${writing ? "The configs group is blocked because " : ""}the manifest at ${target} does not reach ${names.length === 1 ? "a Vitest project" : "Vitest projects"} the planned configuration registers: ${names.join(", ")}. No chain from ${gateNames} invokes ${names.length === 1 ? "it" : "them"}. ${remedies.filter((remedy) => remedy !== "").join(" ")}${writing ? " Exclude configs from --groups to write another group." : ""}`,