@theholocron/astromech 5.0.0-alpha.171 → 5.0.0-alpha.172

package/dist/index.d.mts

@@ -383,6 +383,15 @@ interface WorkspacePackage {
   dir: string;
   version: string | undefined;
   dependencies: Record<string, string>;
+  /**
+   * A workspace:* devDependency can optionally count too (holocron#1071,
+   * workspaceDependencies' includeDevDependencies option). Optional so a
+   * caller holding the previous, dev-dependency-less shape (a plain
+   * object literal, or a value from outside this package's own version)
+   * keeps compiling and working -- a missing value is treated the same
+   * as `{}`.
+   */
+  devDependencies?: Record<string, string>;
 }
 interface Tree {
   readJson: (path: string) => unknown;

package/dist/index.mjs

@@ -2017,11 +2017,54 @@ const WAIT_TIMEOUT_MS = 3e5;
 const WAIT_INTERVAL_MS = 1e4;
 //#endregion
 //#region src/tasks/deploy-on-release/workspace-dependencies/workspace-dependencies.ts
-/** The `workspace:*` dependencies of `pkg`, as workspace packages (with their current version). */
-function workspaceDependencies(pkg, all) {
-	const names = new Set(Object.entries(pkg.dependencies).filter(([, spec]) => String(spec).startsWith("workspace:")).map(([name]) => name));
+/**
+* `pkg`'s direct `workspace:*` dependencies -- `dependencies`, plus
+* `devDependencies` when asked. `pkg.devDependencies` defaults to `{}`:
+* it's optional on `WorkspacePackage` precisely so a caller holding the
+* previous (dev-dependency-less) shape doesn't crash here.
+*/
+function directWorkspaceDependencies(pkg, all, includeDevDependencies) {
+	const entries = includeDevDependencies ? [...Object.entries(pkg.dependencies), ...Object.entries(pkg.devDependencies ?? {})] : Object.entries(pkg.dependencies);
+	const names = new Set(entries.filter(([, spec]) => String(spec).startsWith("workspace:")).map(([name]) => name));
 	return all.filter((candidate) => names.has(candidate.name));
 }
+/**
+* Every `workspace:*` dependency of `pkg`, transitively (holocron#1071) —
+* its own direct dependencies, and theirs, and so on — as workspace
+* packages (with their current version).
+*
+* `includeDevDependencies` defaults to `false`, matching the pre-#1071
+* behavior: a shared dev/build-time workspace package (a tsconfig, a
+* lint config, a build script) counts only when a caller opts in.
+* `defaultPaths` (computing paths that should trigger a rebuild) opts in;
+* `deployOnRelease`'s publish-wait list (waiting for npm to serve a
+* package's RUNTIME dependencies before deploying it) does not -- an
+* unpublished, possibly `private`, dev-only package would never satisfy
+* that wait and the deploy would time out (holocron#1071 review).
+*
+* A diamond (two packages depending on the same third one) contributes
+* that third package once, not twice — tracked by a visited set, seeded
+* with `pkg` itself so a dependency cycle (A -> B -> A) can't loop
+* forever either, even though a real workspace graph normally can't have
+* one.
+*/
+function workspaceDependencies(pkg, all, options) {
+	const includeDevDependencies = options?.includeDevDependencies ?? false;
+	const visited = /* @__PURE__ */ new Set([pkg.name]);
+	const result = [];
+	let frontier = directWorkspaceDependencies(pkg, all, includeDevDependencies);
+	while (frontier.length > 0) {
+		const next = [];
+		for (const dep of frontier) {
+			if (visited.has(dep.name)) continue;
+			visited.add(dep.name);
+			result.push(dep);
+			next.push(...directWorkspaceDependencies(dep, all, includeDevDependencies));
+		}
+		frontier = next;
+	}
+	return result;
+}
 //#endregion
 //#region src/tasks/deploy-on-release/default-paths/default-paths.ts
 /**
@@ -2033,7 +2076,7 @@ function workspaceDependencies(pkg, all) {
 * @returns Repo-relative prefixes, each ending in `/`.
 */
 function defaultPaths(repoRoot, pkg, all) {
-	return [pkg, ...workspaceDependencies(pkg, all)].map(({ dir }) => `${relative(repoRoot, dir).replace(/\\/g, "/")}/`);
+	return [pkg, ...workspaceDependencies(pkg, all, { includeDevDependencies: true })].map(({ dir }) => `${relative(repoRoot, dir).replace(/\\/g, "/")}/`);
 }
 //#endregion
 //#region src/tasks/deploy-on-release/deploy-targets/deploy-targets.ts
@@ -2131,7 +2174,8 @@ function workspacePackages(repoRoot, { readJson, listDirs }) {
 		name,
 		dir,
 		version: manifest.version,
-		dependencies: manifest.dependencies ?? {}
+		dependencies: manifest.dependencies ?? {},
+		devDependencies: manifest.devDependencies ?? {}
 	}] : []);
 }
 //#endregion

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@theholocron/astromech",
-  "version": "5.0.0-alpha.171",
+  "version": "5.0.0-alpha.172",
   "description": "The Holocron task runner — one task manifest drives `holocron run`, `holocron run ci`, the CI workflows, package.json scripts, linters, and required checks.",
   "keywords": [
     "ci",
@@ -48,7 +48,7 @@
     "@theholocron/fs-utils": "^1.16.0",
     "@theholocron/json-utils": "^1.16.0",
     "@theholocron/pkg-utils": "^1.16.0",
-    "@theholocron/datapad": "5.0.0-alpha.171"
+    "@theholocron/datapad": "5.0.0-alpha.172"
   },
   "devDependencies": {
     "@commitlint/config-conventional": "^21.2.3",