@dbarrett24/tsdown-config 1.0.1 → 1.0.2

package/dist/index.js

@@ -3,6 +3,35 @@ import { merge } from "es-toolkit";
 import fs from "fs";
 import path from "path";
 import { defineConfig } from "tsdown";
+//#region workspace-specifier.ts
+/**
+* Resolution of pnpm workspace protocol specifiers for the emitted `dist/package.json`.
+*
+* Kept in its own dependency-free module so it can be unit-tested without loading tsdown or
+* the pnpm lockfile reader. Its correctness is load-bearing at publish time: a specifier that
+* survives into the emitted manifest publishes successfully and is uninstallable for every
+* consumer, which is the most expensive failure this package can produce.
+*/
+/**
+* Convert a `workspace:` specifier into a publishable semver range.
+*
+* `workspace:*` pins the dependency exactly, which is right for packages that must move in
+* lockstep. `workspace:^` and `workspace:~` publish as the matching range, which is what a
+* package wants when consumers should be able to pick up the dependency's patch releases —
+* without it, a fix published to the dependency is unreachable behind an exact pin.
+*
+* @param specifier the raw dependency value, e.g. `workspace:^`
+* @param resolvedVersion the dependency's actual version, e.g. `1.2.3`
+*/
+const resolveWorkspaceSpecifier = (specifier, resolvedVersion) => {
+	if (specifier === "workspace:*") return resolvedVersion;
+	if (specifier === "workspace:^") return `^${resolvedVersion}`;
+	if (specifier === "workspace:~") return `~${resolvedVersion}`;
+	return specifier.replace(/^workspace:/, "");
+};
+/** True when a dependency value uses the pnpm workspace protocol. */
+const isWorkspaceSpecifier = (specifier) => specifier.startsWith("workspace:");
+//#endregion
 //#region index.ts
 const formatEmittedPackageJson = (obj) => `${JSON.stringify(obj, null, 4)}\n`;
 const makeConfig = (entries, clientEntries, optionOverrides) => {
@@ -68,18 +97,18 @@ const copyPackageJson = async () => {
 	const merged_deps = merge(lockfile_package_list?.dependencies ?? {}, lockfile_package_list?.devDependencies ?? {});
 	for (const item of Object.entries(newPackageJSON.peerDependencies)) {
 		const [dep, version] = item;
-		if (version.includes("workspace")) {
+		if (isWorkspaceSpecifier(version)) {
 			const package_path = merged_deps?.[dep].version.replace(/(?:link:|\/dist)/g, "") + "/package.json";
 			const packageJSON = JSON.parse(fs.readFileSync(package_path).toString());
-			newPackageJSON.peerDependencies[dep] = version.replace("workspace:*", packageJSON.version);
+			newPackageJSON.peerDependencies[dep] = resolveWorkspaceSpecifier(version, packageJSON.version);
 		}
 	}
 	for (const item of Object.entries(newPackageJSON.dependencies ?? {})) {
 		const [dep, version] = item;
-		if (version.includes("workspace")) {
+		if (isWorkspaceSpecifier(version)) {
 			const package_path = merged_deps?.[dep].version.replace(/(?:link:|\/dist)/g, "") + "/package.json";
 			const packageJSON = JSON.parse(fs.readFileSync(package_path).toString());
-			newPackageJSON.dependencies[dep] = version.replace("workspace:*", packageJSON.version);
+			newPackageJSON.dependencies[dep] = resolveWorkspaceSpecifier(version, packageJSON.version);
 		}
 	}
 	fs.writeFileSync("dist/package.json", formatEmittedPackageJson(newPackageJSON));

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@dbarrett24/tsdown-config",
-  "version": "1.0.1",
+  "version": "1.0.2",
   "description": "Shared tsdown configuration for building component libraries (publish-from-dist with generated wildcard exports)",
   "type": "module",
   "module": "dist/index.js",
@@ -23,7 +23,9 @@
     "typescript": "^6.0.3"
   },
   "devDependencies": {
+    "@swc/jest": "^0.2.39",
     "concurrently": "^8.2.2",
+    "jest": "^30.3.0",
     "@dbarrett24/typescript-config": "1.0.1"
   },
   "peerDependencies": {
@@ -53,9 +55,9 @@
     "lint-fix": "pnpm lint-oxlint-fix && pnpm lint-oxfmt-fix",
     "lint-oxfmt": "oxfmt --config ../../.oxfmtrc.json --check \"*.ts\"",
     "lint-oxfmt-fix": "oxfmt --config ../../.oxfmtrc.json --write \"*.ts\"",
-    "lint-oxlint": "oxlint index.ts tsdown.config.ts",
-    "lint-oxlint-fix": "oxlint --fix index.ts tsdown.config.ts",
+    "lint-oxlint": "oxlint index.ts tsdown.config.ts workspace-specifier.ts",
+    "lint-oxlint-fix": "oxlint --fix index.ts tsdown.config.ts workspace-specifier.ts",
     "ts-check": "tsc --noEmit",
-    "test": "echo 'No tests for tsdown-config'"
+    "test": "jest --passWithNoTests"
   }
 }