pob 26.2.0 → 26.3.0

package/CHANGELOG.md

@@ -3,6 +3,17 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+## [26.3.0](https://github.com/christophehurpeau/pob/compare/pob@26.2.0...pob@26.3.0) (2025-02-02)
+
+### Features
+
+* add mjs and cjs to checkonly ([2666924](https://github.com/christophehurpeau/pob/commit/2666924e72980b7c9a5c1315ff7f8680642171ff))
+* improve typescript checkonly support ([3355104](https://github.com/christophehurpeau/pob/commit/3355104897428d9014b11402ed771a10b1b94064))
+
+### Bug Fixes
+
+* avoid deleting @babel/core ([022d74d](https://github.com/christophehurpeau/pob/commit/022d74d8e462962432cf0bc53d5f3b0d79f86769))
+
 ## [26.2.0](https://github.com/christophehurpeau/pob/compare/pob@26.1.1...pob@26.2.0) (2025-02-02)
 
 ### Features

package/lib/generators/common/babel/CommonBabelGenerator.js

@@ -241,7 +241,7 @@ export default class CommonBabelGenerator extends Generator {
         pkg.pob.bundler === "rollup-babel"
       ) {
         delete pkg.pob.babelEnvs;
-        if (pkg.pob.typescript !== true) {
+        if (!pkg.pob.typescript) {
           delete pkg.pob.envs;
           delete pkg.pob.entries;
           delete pkg.pob.jsx;
@@ -304,11 +304,14 @@ export default class CommonBabelGenerator extends Generator {
 
     /* dependencies */
 
-    packageUtils.addOrRemoveDevDependencies(
-      pkg,
-      useBabel || (pkg.peerDependencies && pkg.peerDependencies["@babel/core"]),
-      ["@babel/core"],
-    );
+    if (
+      useBabel ||
+      (pkg.peerDependencies && pkg.peerDependencies["@babel/core"])
+    ) {
+      packageUtils.addDevDependencies(pkg, ["@babel/core"]);
+    } else if (pkg.dependencies && pkg.dependencies["pob-babel"] && !useBabel) {
+      packageUtils.removeDevDependencies(pkg, ["@babel/core"]);
+    }
     packageUtils.addOrRemoveDevDependencies(pkg, useBabel, ["pob-babel"]);
 
     if (pkg.dependencies && pkg.dependencies["pob-babel"]) {
@@ -325,7 +328,8 @@ export default class CommonBabelGenerator extends Generator {
     packageUtils.addOrRemoveDevDependencies(
       pkg,
       (useBabel && pkg.pob.jsx) ||
-        (pkg.devDependencies?.["@babel/preset-react"] && isLibraryRollupPlugin),
+        (pkg.devDependencies?.["@babel/preset-react"] &&
+          (isLibraryRollupPlugin || pkg.name === "alouette-icons")),
       ["@babel/preset-react"],
     );
 

package/lib/generators/common/transpiler/CommonTranspilerGenerator.js

@@ -136,10 +136,10 @@ export default class CommonTranspilerGenerator extends Generator {
   default() {
     const pkg = this.fs.readJSON(this.destinationPath("package.json"));
     const withBabel = this.babelEnvs && this.babelEnvs.length > 0;
-    const withTypescript =
-      pkg.pob.typescript === true || withBabel || !!pkg.pob.bundler;
+    const withTypescript = pkg.pob.typescript || withBabel || !!pkg.pob.bundler;
     const bundler =
       withTypescript &&
+      pkg.pob?.typescript !== "check-only" &&
       (pkg.pob.rollup === false ||
       pkg.pob.bundler === "tsc" ||
       (!pkg.pob.bundler && !pkg.pob.typescript === true)
@@ -202,7 +202,11 @@ export default class CommonTranspilerGenerator extends Generator {
     });
 
     if (shouldBuildDefinitions) {
-      pkg.scripts.build += " && yarn run build:definitions";
+      if (pkg.scripts.build) {
+        pkg.scripts.build += " && yarn run build:definitions";
+      } else {
+        pkg.scripts.build = "yarn run build:definitions";
+      }
     } else if (!this.options.isApp && !bundler && !withTypescript) {
       // check definitions, but also force lerna to execute build:definitions in right order
       // example: nightingale-types depends on nightingale-levels
@@ -238,17 +242,23 @@ export default class CommonTranspilerGenerator extends Generator {
     );
     packageUtils.addOrRemoveDevDependencies(
       pkg,
-      bundler === "esbuild" && withTypescript,
+      bundler === "esbuild" &&
+        withTypescript &&
+        pkg.pob?.typescript !== "check-only",
       ["@pob/esbuild"],
     );
     packageUtils.addOrRemoveDevDependencies(
       pkg,
-      bundler === "rollup-esbuild" && withTypescript,
+      bundler === "rollup-esbuild" &&
+        withTypescript &&
+        pkg.pob?.typescript !== "check-only",
       ["@pob/rollup-esbuild"],
     );
     packageUtils.addOrRemoveDependencies(
       pkg,
-      (bundler === "tsc" || bundler === "rollup-typescript") && withTypescript,
+      (bundler === "tsc" || bundler === "rollup-typescript") &&
+        withTypescript &&
+        pkg.pob?.typescript !== "check-only",
       ["tslib"],
       "^",
     );
@@ -289,7 +299,7 @@ export default class CommonTranspilerGenerator extends Generator {
     }
 
     // if (!pkg.main || pkg.main.startsWith('./lib/')) {
-    if (bundler || withTypescript) {
+    if (bundler || (withTypescript && pkg.pob?.typescript !== "check-only")) {
       // see pkg.exports instead.
       delete pkg.main;
       if (!this.options.isApp) {
@@ -298,12 +308,12 @@ export default class CommonTranspilerGenerator extends Generator {
         pkg.types = `./${this.options.srcDirectory}/index.ts`;
       }
     } else {
-      if (!pkg.main) {
+      if (!pkg.main && !pkg.pob.entries) {
         pkg.exports = "./lib/index.js";
       }
       if (pkg.type === "module" && this.fs.exists("./lib/index.cjs")) {
         pkg.main = "./lib/index.cjs";
-      } else {
+      } else if (!pkg.pob?.entries || pkg.pob.entries.includes("index")) {
         pkg.main = "./lib/index.js";
       }
       if (!this.options.isApp || this.options.isAppLibrary) {
@@ -351,7 +361,7 @@ export default class CommonTranspilerGenerator extends Generator {
     }
 
     /* webpack 5 and node with ESM support */
-    if (bundler || withTypescript) {
+    if (bundler || (withTypescript && pkg.pob?.typescript !== "check-only")) {
       const omitTarget = bundler === "esbuild" || bundler === "tsc";
       pkg.exports = {
         "./package.json": "./package.json",
@@ -375,7 +385,7 @@ export default class CommonTranspilerGenerator extends Generator {
         };
 
         const defaultNodeEnv =
-          withBabel || withTypescript
+          withBabel || (withTypescript && pkg.pob?.typescript !== "check-only")
             ? envs.find((env) => env.target === "node")
             : undefined;
 

package/lib/generators/common/typescript/CommonTypescriptGenerator.js

@@ -237,20 +237,27 @@ export default class CommonTypescriptGenerator extends Generator {
               ]),
           );
 
+          const isTypescriptPackageMap = new Map(
+            [...packageLocations.entries()].map(
+              ([packageName, packageLocation]) => [
+                packageName,
+                existsSync(
+                  `${packageLocations.get(packageName)}/tsconfig.json`,
+                ) && existsSync(`${packageLocations.get(packageName)}/src`),
+              ],
+            ),
+          );
+
           monorepoPackageSrcPaths = [...packageLocations.entries()].map(
             ([packageName, packageLocation]) => [
               packageName,
               `${packageLocation}/${
-                existsSync(`${packageLocations.get(packageName)}/tsconfig.json`)
-                  ? "src"
-                  : "lib"
+                isTypescriptPackageMap.get(packageName) ? "src" : "lib"
               }`,
             ],
           );
           monorepoPackageReferences = yoConfig.pob.monorepo.packageNames
-            .filter((packageName) =>
-              existsSync(`${packageLocations.get(packageName)}/tsconfig.json`),
-            )
+            .filter((packageName) => isTypescriptPackageMap.get(packageName))
             .map((packageName) => packageLocations.get(packageName));
           // monorepoPackageBuildReferences = yoConfig.pob.monorepo.packageNames
           //   .filter((packageName) =>

package/lib/generators/common/typescript/templates/tsconfig.check-js.json.ejs

@@ -12,5 +12,5 @@
 <% } -%>
     "noEmit": true
   },
-  "include": ["<%= srcDirectory %>/**/*.js"]
+  "include": ["<%= srcDirectory %>/**/*.js","<%= srcDirectory %>/**/*.mjs","<%= srcDirectory %>/**/*.cjs", "<%= srcDirectory %>/**/*.d.ts"]
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pob",
-  "version": "26.2.0",
+  "version": "26.3.0",
   "description": "Pile of bones, library generator with git/babel/typescript/typedoc/readme/jest",
   "keywords": [
     "skeleton"