@b9g/libuild 0.1.18 → 0.1.20

package/CHANGELOG.md

@@ -2,13 +2,26 @@
 
 All notable changes to this project will be documented in this file.
 
-## [0.1.18] - 2025-12-01
+## [0.1.20] - 2025-12-08
 
-### Changed
-- **prepublishOnly guards** - Replaced `private: true` with `prepublishOnly` script guards as the standard publish protection mechanism. Dist packages now include helpful error messages when accidentally publishing from wrong directory. The `private` field is now properly copied to dist/ for actual private packages, enabling libuild to work with private packages while maintaining publish protection.
+### Fixed
+- **Ambient .d.ts export validation** - Fixed validation error when using `--save` with ambient .d.ts files. The build now correctly handles both `./src/` and `./dist/src/` paths for ambient declarations in exports.
+
+## [0.1.19] - 2025-12-08
+
+### Added
+- **Ambient .d.ts exports** - Ambient TypeScript declaration files (*.d.ts) in src/ are now automatically added to the exports map, making them discoverable and importable by consumers.
+- **Triple-slash references for ambient types** - Generated .d.ts files now include triple-slash reference directives to ambient .d.ts files, ensuring TypeScript can find ambient type declarations.
 
 ### Fixed
-- **Private package publishing** - Fixed bug where packages marked with `private: true` could still be published via `libuild publish`. npm now correctly handles private package rejection.
+- **Hard fail on invalid workspace package.json** - Invalid JSON in workspace dependencies now causes build to fail immediately with clear error instead of silently continuing with a warning.
+
+## [0.1.18] - 2025-12-02
+
+### Changed
+- **BREAKING: Scripts not copied to dist** - Scripts field is no longer copied to dist/package.json as they don't work correctly in the dist context (can't reference other scripts that were filtered out). This fixes the issue where prepublishOnly guards would block `libuild publish`. Users who need install scripts (postinstall, etc.) can manually add them to dist/package.json if needed.
+- **prepublishOnly guards for root** - Root package.json gets `prepublishOnly` guard (via `--save`) to prevent accidental publishing from root directory.
+- **Copy private field** - The `private` field is now properly copied from root to dist/, enabling actual private packages to work with libuild while npm prevents their publication.
 
 ## [0.1.17] - 2025-11-25
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@b9g/libuild",
-  "version": "0.1.18",
+  "version": "0.1.20",
   "description": "Zero-config library builds",
   "keywords": [
     "build",

package/src/libuild.cjs

@@ -212788,6 +212788,7 @@ ${diagnostics}`,
           });
         }
         await addTripleSlashReferences(tsFiles, options.outDir);
+        await addAmbientReferences(tsFiles, options.outDir, options.rootDir);
       });
     }
   };
@@ -212822,6 +212823,29 @@ ${content}`;
     await FS2.writeFile(jsPath, modifiedContent);
   }
 }
+async function addAmbientReferences(tsFiles, outDir, rootDir) {
+  let ambientFiles = [];
+  try {
+    const files = await FS2.readdir(rootDir);
+    ambientFiles = files.filter((f) => f.endsWith(".d.ts"));
+  } catch {
+    return;
+  }
+  if (ambientFiles.length === 0)
+    return;
+  for (const tsFile of tsFiles) {
+    const baseName = Path2.basename(tsFile, Path2.extname(tsFile));
+    const dtsPath = Path2.join(outDir, `${baseName}.d.ts`);
+    const dtsExists = await FS2.access(dtsPath).then(() => true).catch(() => false);
+    if (!dtsExists)
+      continue;
+    const content = await FS2.readFile(dtsPath, "utf-8");
+    const references = ambientFiles.map((ambientFile) => `/// <reference path="./${ambientFile}" />`).join("\n");
+    const modifiedContent = `${references}
+${content}`;
+    await FS2.writeFile(dtsPath, modifiedContent);
+  }
+}
 
 // src/libuild.ts
 function generateRuntimeBanner(pkg) {
@@ -213035,6 +213059,9 @@ async function generateExports(entries, mainEntry, options, existingExports = {}
       if (existing === "./package.json" || existing.endsWith("/package.json")) {
         return existing;
       }
+      if (existing.endsWith(".d.ts") && existing.match(/\.\/(?:dist\/)?src\/[^/]+\.d\.ts$/)) {
+        return existing;
+      }
       const match = existing.match(/\.\/src\/([^/]+\.(?:ts|js))$/);
       if (match) {
         const filename = match[1];
@@ -213391,18 +213418,14 @@ async function resolveWorkspaceVersion(packageName, workspaceSpec, cwd) {
         `./${packageBaseName}/package.json`
       ];
       for (const pkgPath of possiblePaths) {
-        try {
-          const resolvedPath = Path3.resolve(cwd, pkgPath);
-          if (await fileExists(resolvedPath)) {
-            const pkgContent = await FS3.readFile(resolvedPath, "utf-8");
-            const pkgJson = JSON.parse(pkgContent);
-            if (pkgJson.name === packageName) {
-              console.info(`  Resolved workspace:* dependency "${packageName}" to ^${pkgJson.version}`);
-              return `^${pkgJson.version}`;
-            }
+        const resolvedPath = Path3.resolve(cwd, pkgPath);
+        if (await fileExists(resolvedPath)) {
+          const pkgContent = await FS3.readFile(resolvedPath, "utf-8");
+          const pkgJson = JSON.parse(pkgContent);
+          if (pkgJson.name === packageName) {
+            console.info(`  Resolved workspace:* dependency "${packageName}" to ^${pkgJson.version}`);
+            return `^${pkgJson.version}`;
           }
-        } catch {
-          continue;
         }
       }
       console.warn(`\u26A0\uFE0F  WARNING: Could not resolve workspace dependency "${packageName}" - keeping as workspace:*`);
@@ -213791,9 +213814,21 @@ async function build2(cwd, save = false) {
     });
   }
   const autoDiscoveredFiles = [];
+  const ambientDtsFiles = [];
+  if (await fileExists(srcDir)) {
+    const srcFiles = await FS3.readdir(srcDir);
+    ambientDtsFiles.push(...srcFiles.filter((f) => f.endsWith(".d.ts")));
+  }
   console.info("  Generating package.json...");
   const cleanedPkg = await cleanPackageJSON(pkg, mainEntry, options, cwd, distDir);
   const exportsResult = await generateExports(entries, mainEntry, options, pkg.exports, distDir, allBinEntries);
+  for (const dtsFile of ambientDtsFiles) {
+    const baseName = Path3.basename(dtsFile, ".d.ts");
+    const exportKey = `./${baseName}.d.ts`;
+    if (!exportsResult.exports[exportKey]) {
+      exportsResult.exports[exportKey] = `./src/${dtsFile}`;
+    }
+  }
   cleanedPkg.exports = fixExportsForDist(exportsResult.exports);
   if (exportsResult.staleExports.length > 0) {
     console.warn(`\u26A0\uFE0F  WARNING: Found ${exportsResult.staleExports.length} stale export(s) pointing to missing src/ files:`);
@@ -213827,16 +213862,12 @@ async function build2(cwd, save = false) {
       await FS3.copyFile(srcPath, Path3.join(distDir, file));
     }
   }
-  if (await fileExists(srcDir)) {
-    const srcFiles = await FS3.readdir(srcDir);
-    const ambientDtsFiles = srcFiles.filter((f) => f.endsWith(".d.ts"));
-    if (ambientDtsFiles.length > 0) {
-      console.info(`  Copying ${ambientDtsFiles.length} ambient .d.ts file(s)...`);
-      for (const dtsFile of ambientDtsFiles) {
-        const srcPath = Path3.join(srcDir, dtsFile);
-        const destPath = Path3.join(distSrcDir, dtsFile);
-        await FS3.copyFile(srcPath, destPath);
-      }
+  if (ambientDtsFiles.length > 0) {
+    console.info(`  Copying ${ambientDtsFiles.length} ambient .d.ts file(s)...`);
+    for (const dtsFile of ambientDtsFiles) {
+      const srcPath = Path3.join(srcDir, dtsFile);
+      const destPath = Path3.join(distSrcDir, dtsFile);
+      await FS3.copyFile(srcPath, destPath);
     }
   }
   const commonFiles = ["README.md", "LICENSE", "CHANGELOG.md", "COPYING", "AUTHORS"];

package/src/libuild.js

@@ -181,6 +181,7 @@ ${diagnostics}`,
           });
         }
         await addTripleSlashReferences(tsFiles, options.outDir);
+        await addAmbientReferences(tsFiles, options.outDir, options.rootDir);
       });
     }
   };
@@ -215,6 +216,29 @@ ${content}`;
     await FS2.writeFile(jsPath, modifiedContent);
   }
 }
+async function addAmbientReferences(tsFiles, outDir, rootDir) {
+  let ambientFiles = [];
+  try {
+    const files = await FS2.readdir(rootDir);
+    ambientFiles = files.filter((f) => f.endsWith(".d.ts"));
+  } catch {
+    return;
+  }
+  if (ambientFiles.length === 0)
+    return;
+  for (const tsFile of tsFiles) {
+    const baseName = Path2.basename(tsFile, Path2.extname(tsFile));
+    const dtsPath = Path2.join(outDir, `${baseName}.d.ts`);
+    const dtsExists = await FS2.access(dtsPath).then(() => true).catch(() => false);
+    if (!dtsExists)
+      continue;
+    const content = await FS2.readFile(dtsPath, "utf-8");
+    const references = ambientFiles.map((ambientFile) => `/// <reference path="./${ambientFile}" />`).join("\n");
+    const modifiedContent = `${references}
+${content}`;
+    await FS2.writeFile(dtsPath, modifiedContent);
+  }
+}
 
 // src/libuild.ts
 function generateRuntimeBanner(pkg) {
@@ -428,6 +452,9 @@ async function generateExports(entries, mainEntry, options, existingExports = {}
       if (existing === "./package.json" || existing.endsWith("/package.json")) {
         return existing;
       }
+      if (existing.endsWith(".d.ts") && existing.match(/\.\/(?:dist\/)?src\/[^/]+\.d\.ts$/)) {
+        return existing;
+      }
       const match = existing.match(/\.\/src\/([^/]+\.(?:ts|js))$/);
       if (match) {
         const filename = match[1];
@@ -784,18 +811,14 @@ async function resolveWorkspaceVersion(packageName, workspaceSpec, cwd) {
         `./${packageBaseName}/package.json`
       ];
       for (const pkgPath of possiblePaths) {
-        try {
-          const resolvedPath = Path3.resolve(cwd, pkgPath);
-          if (await fileExists(resolvedPath)) {
-            const pkgContent = await FS3.readFile(resolvedPath, "utf-8");
-            const pkgJson = JSON.parse(pkgContent);
-            if (pkgJson.name === packageName) {
-              console.info(`  Resolved workspace:* dependency "${packageName}" to ^${pkgJson.version}`);
-              return `^${pkgJson.version}`;
-            }
+        const resolvedPath = Path3.resolve(cwd, pkgPath);
+        if (await fileExists(resolvedPath)) {
+          const pkgContent = await FS3.readFile(resolvedPath, "utf-8");
+          const pkgJson = JSON.parse(pkgContent);
+          if (pkgJson.name === packageName) {
+            console.info(`  Resolved workspace:* dependency "${packageName}" to ^${pkgJson.version}`);
+            return `^${pkgJson.version}`;
           }
-        } catch {
-          continue;
         }
       }
       console.warn(`\u26A0\uFE0F  WARNING: Could not resolve workspace dependency "${packageName}" - keeping as workspace:*`);
@@ -1184,9 +1207,21 @@ async function build2(cwd, save = false) {
     });
   }
   const autoDiscoveredFiles = [];
+  const ambientDtsFiles = [];
+  if (await fileExists(srcDir)) {
+    const srcFiles = await FS3.readdir(srcDir);
+    ambientDtsFiles.push(...srcFiles.filter((f) => f.endsWith(".d.ts")));
+  }
   console.info("  Generating package.json...");
   const cleanedPkg = await cleanPackageJSON(pkg, mainEntry, options, cwd, distDir);
   const exportsResult = await generateExports(entries, mainEntry, options, pkg.exports, distDir, allBinEntries);
+  for (const dtsFile of ambientDtsFiles) {
+    const baseName = Path3.basename(dtsFile, ".d.ts");
+    const exportKey = `./${baseName}.d.ts`;
+    if (!exportsResult.exports[exportKey]) {
+      exportsResult.exports[exportKey] = `./src/${dtsFile}`;
+    }
+  }
   cleanedPkg.exports = fixExportsForDist(exportsResult.exports);
   if (exportsResult.staleExports.length > 0) {
     console.warn(`\u26A0\uFE0F  WARNING: Found ${exportsResult.staleExports.length} stale export(s) pointing to missing src/ files:`);
@@ -1220,16 +1255,12 @@ async function build2(cwd, save = false) {
       await FS3.copyFile(srcPath, Path3.join(distDir, file));
     }
   }
-  if (await fileExists(srcDir)) {
-    const srcFiles = await FS3.readdir(srcDir);
-    const ambientDtsFiles = srcFiles.filter((f) => f.endsWith(".d.ts"));
-    if (ambientDtsFiles.length > 0) {
-      console.info(`  Copying ${ambientDtsFiles.length} ambient .d.ts file(s)...`);
-      for (const dtsFile of ambientDtsFiles) {
-        const srcPath = Path3.join(srcDir, dtsFile);
-        const destPath = Path3.join(distSrcDir, dtsFile);
-        await FS3.copyFile(srcPath, destPath);
-      }
+  if (ambientDtsFiles.length > 0) {
+    console.info(`  Copying ${ambientDtsFiles.length} ambient .d.ts file(s)...`);
+    for (const dtsFile of ambientDtsFiles) {
+      const srcPath = Path3.join(srcDir, dtsFile);
+      const destPath = Path3.join(distSrcDir, dtsFile);
+      await FS3.copyFile(srcPath, destPath);
     }
   }
   const commonFiles = ["README.md", "LICENSE", "CHANGELOG.md", "COPYING", "AUTHORS"];