@theholocron/astromech 5.0.0-alpha.41 → 5.0.0-alpha.42

package/dist/index.mjs

@@ -792,10 +792,11 @@ const mkRunner = (tool, args = []) => ({
 });
 /**
 * Resolve `{ tool, args }` for a runner, applying `detect[]` against repo
-* files, then (#750) each candidate's `tool` name against `package.json`'s
-* own declared dependencies. Only called for `tool` / `detect` runners — the
-* caller handles `command` runners itself, so `local.detect` is present
-* whenever `local.tool` is not.
+* files, falling back (#750) to each candidate's `tool` name against
+* `package.json`'s own declared dependencies when neither matches by
+* filename. Only called for `tool` / `detect` runners — the caller handles
+* `command` runners itself, so `local.detect` is present whenever
+* `local.tool` is not.
 *
 * The dependency fallback matters once a Bucket A migration (#680) deletes
 * a build tool's own config file (`tsdown.config.ts`, `vite.config.ts`, …):
@@ -805,6 +806,19 @@ const mkRunner = (tool, args = []) => ({
 * package can't actually build with `tsdown` without it in
 * `dependencies`/`devDependencies`, so that's a signal at least as reliable
 * as the config file, and one deleting the file doesn't remove.
+*
+* **Per-candidate, not two full passes** (#750 follow-up — the first
+* shipped version got this wrong): `delivery.build`'s own `detect[]` ends
+* with `{ when: /^tsconfig\.json$/, tool: "tsc" }` as its last-resort
+* fallback, and `tsconfig.json` exists in essentially every package
+* regardless of build tool. A "check every candidate's filename, *then*
+* check every candidate's dependency" two-pass design would let that
+* always-present fallback file win over a real `tsdown`/`vite`/`rollup`
+* dependency match, the moment the earlier candidates' own config files are
+* gone — exactly backwards. Checking `(file match) || (dependency match)`
+* together for each candidate before moving to the next preserves the
+* intended precedence: a real signal for an *earlier* candidate, by either
+* means, still wins over a later candidate's mere filename presence.
 */
 function resolveRunner(local, cwd, listDir, readFile) {
 	if (local.tool) return mkRunner(local.tool, local.args);
@@ -814,9 +828,8 @@ function resolveRunner(local, cwd, listDir, readFile) {
 	} catch {
 		files = [];
 	}
-	for (const candidate of local.detect) if (files.some((f) => candidate.when.test(f))) return mkRunner(candidate.tool, candidate.args);
 	const deps = packageJsonDependencies(cwd, readFile);
-	for (const candidate of local.detect) if (deps.has(candidate.tool)) return mkRunner(candidate.tool, candidate.args);
+	for (const candidate of local.detect) if (files.some((f) => candidate.when.test(f)) || deps.has(candidate.tool)) return mkRunner(candidate.tool, candidate.args);
 }
 /** Every key from `package.json`'s `dependencies` + `devDependencies`, or an empty set on any read/parse failure. */
 function packageJsonDependencies(cwd, readFile) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@theholocron/astromech",
-  "version": "5.0.0-alpha.41",
+  "version": "5.0.0-alpha.42",
   "description": "The Holocron task runner — one task manifest drives `holocron run`, `holocron ci`, the CI workflows, package.json scripts, linters, and required checks.",
   "keywords": [
     "ci",
@@ -37,7 +37,7 @@
     "dist"
   ],
   "dependencies": {
-    "@theholocron/datapad": "5.0.0-alpha.41"
+    "@theholocron/datapad": "5.0.0-alpha.42"
   },
   "devDependencies": {
     "@theholocron/eslint-config": "^8.0.0",
@@ -53,7 +53,7 @@
     "tsdown": "^0.22.14",
     "typescript": "^5.9.3",
     "vitest": "^4.1.11",
-    "@theholocron/rollup-plugin-transform-template": "5.0.0-alpha.41"
+    "@theholocron/rollup-plugin-transform-template": "5.0.0-alpha.42"
   },
   "engines": {
     "node": ">=22"