@savvy-web/tsdown-plugins 1.0.1 → 1.1.1

package/build/build-target-groups.js

@@ -108,6 +108,7 @@ async function buildTargetGroups(options) {
 				transform: options.transform,
 				sourceDir: options.cwd,
 				dual: options.dualExports ?? js.format.includes("cjs"),
+				emitDts: options.emitDts !== false,
 				...options.subdirExports !== void 0 ? { subdirExports: options.subdirExports } : {},
 				...options.exeRewrite !== void 0 ? { exeRewrite: options.exeRewrite } : {}
 			}) : void 0;
@@ -141,7 +142,7 @@ async function buildTargetGroups(options) {
 				]
 			}));
 			const dtsNeverBundle = [...partExternals ?? [], ...partDtsExternals ?? []];
-			if (Object.keys(dts.entry).length === 0) continue;
+			if (options.emitDts === false || Object.keys(dts.entry).length === 0) continue;
 			const dtsDeps = dtsNeverBundle.length > 0 || partBundleNodeModules || dts.bundledPackages ? { deps: {
 				...dtsNeverBundle.length > 0 ? { neverBundle: dtsNeverBundle } : {},
 				...partBundleNodeModules ? {

package/index.d.ts

@@ -68,9 +68,14 @@ interface ExeRewrite {
  *
  * Export keys in `subdirExports` are built into an isolated `<key>/index.*` subdir (e.g. an
  * RSPress `./runtime`), so their conditions gain an `/index` segment.
+ *
+ * `emitDts` gates the `types` condition on every GENERATED (non-ambient) TS export: pass `false`
+ * when the build's dts pass was skipped (issue #198) so the manifest does not point at a `.d.ts`
+ * that was never written. Defaults to `true`. Ambient `.d.ts` exports (hand-authored, copied
+ * verbatim regardless of the dts pass) are unaffected — their `types` condition is always kept.
  * @public
  */
-declare function transformExports(exports: unknown, dual?: DualExports, subdirExports?: ReadonlySet<string>): unknown;
+declare function transformExports(exports: unknown, dual?: DualExports, subdirExports?: ReadonlySet<string>, emitDts?: boolean): unknown;
 /**
  * Rewrite bin: TS targets to bin/[command].js (string to bin/cli.js); strip leading ./ otherwise.
  *
@@ -93,6 +98,12 @@ interface TransformManifestOptions {
   readonly subdirExports?: ReadonlySet<string> | undefined;
   /** When set, rewrite exports/bin values equal to `source` to the SEA path and add it to `files`. */
   readonly exeRewrite?: ExeRewrite | undefined;
+  /**
+   * Whether the build's dts pass ran. `false` omits the `types` condition from generated (non-
+   * ambient) TS exports — the manifest must not point at a `.d.ts` that was never written
+   * (issue #198). Defaults to `true`.
+   */
+  readonly emitDts?: boolean | undefined;
 }
 /**
  * Apply the full standard manifest transform (excluding catalog resolution, done upstream).
@@ -124,6 +135,8 @@ interface BuildEmittedManifestOptions {
   readonly subdirExports?: ReadonlySet<string> | undefined;
   /** When set, rewrite exports/bin values equal to the exe source to the SEA path and add it to `files`. */
   readonly exeRewrite?: ExeRewrite | undefined;
+  /** Whether the dts pass ran; `false` omits `types` conditions from the emitted manifest (issue #198). Defaults to `true`. */
+  readonly emitDts?: boolean | undefined;
 }
 /**
  * Compute the final manifest bytes for a TargetGroup (catalog resolution + standard transforms).
@@ -147,6 +160,8 @@ interface EmitManifestOptions {
   readonly subdirExports?: ReadonlySet<string> | undefined;
   /** When set, rewrite exports/bin values equal to the exe source to the SEA path and add it to `files`. */
   readonly exeRewrite?: ExeRewrite | undefined;
+  /** Whether the dts pass ran; `false` omits `types` conditions from the emitted manifest (issue #198). Defaults to `true`. */
+  readonly emitDts?: boolean | undefined;
 }
 /**
  * Rolldown plugin: emit the transformed package.json + LICENSE/README into the output pkg/ root.
@@ -660,9 +675,16 @@ interface BuildTargetGroupsOptions {
    * Emit a per-module (`unbundle: true`) declaration tree into `dist/prod/<id>/declarations/` per
    * group, in addition to the bundled dts in `pkg/`. API Extractor's diagnostics-run input for the
    * meta pass. Prod-only; the bundler sets it for `--target prod`. Absent → no third pass
-   * (byte-identical to the two-pass default). Not captured by the collector.
+   * (byte-identical to the two-pass default). Not captured by the collector. Has no effect when
+   * `emitDts` is `false` — the declarations pass exists only to feed the dts-derived meta pass.
    */
   readonly emitDeclarations?: boolean | undefined;
+  /**
+   * When `false`, skip BOTH the bundled per-entry dts pass (Pass 2) and the prod per-module
+   * declarations pass (Pass 3) — no TypeScript compiler load. The JS pass (Pass 1) and
+   * `copyPublicDir` still run. Defaults to `true` (current two/three-pass behavior).
+   */
+  readonly emitDts?: boolean | undefined;
 }
 /**
  * Run tsdown.build() per TargetGroup. Composable so the escape hatch gets multi-group too.

package/manifest/emit-manifest.js

@@ -4,6 +4,24 @@ import { join } from "node:path";
 import { readFile } from "node:fs/promises";
 
 //#region src/manifest/emit-manifest.ts
+const DEPENDENCY_FIELDS = [
+	"dependencies",
+	"devDependencies",
+	"peerDependencies",
+	"optionalDependencies"
+];
+const isCatalogOrWorkspaceSpec = (spec) => typeof spec === "string" && (spec.startsWith("catalog:") || spec.startsWith("workspace:"));
+/**
+* Whether any of `pkg`'s four dependency fields carries at least one `catalog:`/`workspace:`
+* specifier. `resolveManifest` returns a manifest with none of these unchanged, so callers can
+* skip the CatalogResolver + pnpm-workspace + lockfile assembly entirely when this is false.
+*/
+function manifestNeedsCatalogResolution(pkg) {
+	return DEPENDENCY_FIELDS.some((field) => {
+		const deps = pkg[field];
+		return typeof deps === "object" && deps !== null && Object.values(deps).some(isCatalogOrWorkspaceSpec);
+	});
+}
 /**
 * Compute the final manifest bytes for a TargetGroup (catalog resolution + standard transforms).
 *
@@ -11,7 +29,7 @@ import { readFile } from "node:fs/promises";
 */
 async function buildEmittedManifest(options) {
 	const { pkg, targetGroup, devManifest, transform } = options;
-	const shouldResolve = targetGroup.isProd || devManifest === "resolve";
+	const shouldResolve = (targetGroup.isProd || devManifest === "resolve") && manifestNeedsCatalogResolution(pkg);
 	let base = pkg;
 	if (shouldResolve) base = await resolveManifest(pkg);
 	base = {
@@ -25,7 +43,8 @@ async function buildEmittedManifest(options) {
 		}) : void 0,
 		dual: options.dual ?? false,
 		subdirExports: options.subdirExports,
-		exeRewrite: options.exeRewrite
+		exeRewrite: options.exeRewrite,
+		emitDts: options.emitDts ?? true
 	});
 }
 /**
@@ -45,7 +64,8 @@ function emitManifest(options) {
 				transform: options.transform,
 				dual: options.dual,
 				subdirExports: options.subdirExports,
-				exeRewrite: options.exeRewrite
+				exeRewrite: options.exeRewrite,
+				emitDts: options.emitDts
 			});
 			this.emitFile({
 				type: "asset",
@@ -65,4 +85,4 @@ function emitManifest(options) {
 }
 
 //#endregion
-export { buildEmittedManifest, emitManifest };
+export { buildEmittedManifest, emitManifest, manifestNeedsCatalogResolution };

package/manifest/transform.js

@@ -77,9 +77,14 @@ const toBuiltDts = (exportKey, subdirExports) => toBuiltJs(exportKey, subdirExpo
 const toBuiltCjs = (exportKey, subdirExports) => toBuiltJs(exportKey, subdirExports).replace(/\.js$/, ".cjs");
 const isDeclarationFile = (p) => p.endsWith(".d.ts") || p.endsWith(".d.cts") || p.endsWith(".d.mts");
 const isTs = (p) => !isDeclarationFile(p) && (p.endsWith(".ts") || p.endsWith(".tsx"));
-/** Build the conditions object for a TS export target (adds require when dual-format). */
-const tsConditions = (exportKey, dual, subdirExports) => ({
-	types: toBuiltDts(exportKey, subdirExports),
+/**
+* Build the conditions object for a TS export target (adds require when dual-format). `emitDts`
+* gates the `types` condition: when `false` (the build's dts pass was skipped, see issue #198),
+* omitting `types` avoids pointing the published manifest at a declaration file that was never
+* written. Defaults to `true` (current behavior, byte-identical).
+*/
+const tsConditions = (exportKey, dual, subdirExports, emitDts = true) => ({
+	...emitDts ? { types: toBuiltDts(exportKey, subdirExports) } : {},
 	import: toBuiltJs(exportKey, subdirExports),
 	...dual ? { require: toBuiltCjs(exportKey, subdirExports) } : {}
 });
@@ -103,10 +108,15 @@ function stripPublicPrefix(value) {
 *
 * Export keys in `subdirExports` are built into an isolated `<key>/index.*` subdir (e.g. an
 * RSPress `./runtime`), so their conditions gain an `/index` segment.
+*
+* `emitDts` gates the `types` condition on every GENERATED (non-ambient) TS export: pass `false`
+* when the build's dts pass was skipped (issue #198) so the manifest does not point at a `.d.ts`
+* that was never written. Defaults to `true`. Ambient `.d.ts` exports (hand-authored, copied
+* verbatim regardless of the dts pass) are unaffected — their `types` condition is always kept.
 * @public
 */
-function transformExports(exports, dual = false, subdirExports) {
-	if (typeof exports === "string") return isTs(exports) ? tsConditions(".", isDualKey(dual, "."), subdirExports) : stripPublicPrefix(exports);
+function transformExports(exports, dual = false, subdirExports, emitDts = true) {
+	if (typeof exports === "string") return isTs(exports) ? tsConditions(".", isDualKey(dual, "."), subdirExports, emitDts) : stripPublicPrefix(exports);
 	if (exports && typeof exports === "object") {
 		const out = {};
 		for (const [key, value] of Object.entries(exports)) {
@@ -120,8 +130,8 @@ function transformExports(exports, dual = false, subdirExports) {
 				out[key] = { types: `./${ambientOutName(key, cls.source)}` };
 				continue;
 			}
-			if (typeof value === "string" && isTs(value)) out[key] = tsConditions(key, isDualKey(dual, key), subdirExports);
-			else out[key] = transformExports(value, dual, subdirExports);
+			if (typeof value === "string" && isTs(value)) out[key] = tsConditions(key, isDualKey(dual, key), subdirExports, emitDts);
+			else out[key] = transformExports(value, dual, subdirExports, emitDts);
 		}
 		return out;
 	}
@@ -184,7 +194,7 @@ function transformManifest(pkg, options = {}) {
 	}
 	if (result.exports !== void 0 && result.exports !== null) {
 		const original = result.exports;
-		const transformed = transformExports(original, options.dual ?? false, options.subdirExports);
+		const transformed = transformExports(original, options.dual ?? false, options.subdirExports, options.emitDts ?? true);
 		const asMap = typeof original === "string" ? { ".": transformed } : { ...transformed };
 		if (!("./package.json" in asMap)) asMap["./package.json"] = "./package.json";
 		result.exports = asMap;

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@savvy-web/tsdown-plugins",
-	"version": "1.0.1",
+	"version": "1.1.1",
 	"private": false,
 	"description": "Interface-only tsdown/rolldown plugin pack powering @savvy-web/bundler",
 	"homepage": "https://github.com/savvy-web/systems/tree/main/packages/tsdown-plugins",
@@ -35,7 +35,7 @@
 		"@microsoft/tsdoc": "^0.16.0",
 		"@microsoft/tsdoc-config": "^0.18.1",
 		"deep-equal": "^2.2.3",
-		"json-schema-effect": "^0.2.4",
+		"json-schema-effect": "^0.3.0",
 		"picocolors": "^1.1.1",
 		"sort-package-json": "^4.0.0",
 		"std-env": "^4.1.0",