sku 12.4.4 → 12.4.6

package/CHANGELOG.md

@@ -1,5 +1,17 @@
 # sku
 
+## 12.4.6
+
+### Patch Changes
+
+- Fix package manager detection in non-monorepos ([#907](https://github.com/seek-oss/sku/pull/907))
+
+## 12.4.5
+
+### Patch Changes
+
+- Bump some dependencies ([#904](https://github.com/seek-oss/sku/pull/904))
+
 ## 12.4.4
 
 ### Patch Changes

package/config/lintStaged/lintStagedConfig.js

@@ -2,16 +2,18 @@ const { isYarn } = require('../../lib/packageManager');
 const { lintExtensions } = require('../../lib/lint');
 const { getCommand } = require('@antfu/ni');
 
-const steps = {};
+/**
+ * @type {import('lint-staged').Config}
+ */
+const config = {
+  [`**/*.{${lintExtensions},md,less}`]: ['sku format', 'sku lint'],
+};
 
 // Yarn lock integrity check
 if (isYarn) {
-  steps['+(package.json|yarn.lock)'] = [
-    () => getCommand('yarn', 'install', ['--check-files']),
+  config['+(package.json|yarn.lock)'] = [
+    getCommand('yarn', 'install', ['--check-files']),
   ];
 }
 
-// Format & lint
-steps[`**/*.{${lintExtensions},md,less}`] = ['sku format', 'sku lint'];
-
-module.exports = steps;
+module.exports = config;

package/lib/packageManager.js

@@ -1,6 +1,8 @@
+const { existsSync } = require('node:fs');
+const { join } = require('node:path');
 const { cwd } = require('../lib/cwd');
 const { findRootSync } = require('@manypkg/find-root');
-const { getCommand, INSTALL_PAGE } = require('@antfu/ni');
+const { getCommand, INSTALL_PAGE, LOCKS } = require('@antfu/ni');
 
 const { sync: which } = require('which');
 const skuArgs = require('../config/args');
@@ -10,6 +12,19 @@ const skuArgs = require('../config/args');
 /** @type {Array<SupportedPackageManager>} */
 const supportedPackageManagers = ['yarn', 'pnpm', 'npm'];
 
+/** @type {Record<SupportedPackageManager, string>} */
+const lockfileForPackageManager = Object.fromEntries(
+  Object.entries(LOCKS)
+    .filter(([, packageManager]) =>
+      supportedPackageManagers.includes(packageManager),
+    )
+    .map(([lockfileName, packageManager]) => [packageManager, lockfileName]),
+);
+
+const supportedLockfiles = supportedPackageManagers.map(
+  (packageManager) => lockfileForPackageManager[packageManager],
+);
+
 /**
  * @param {SupportedPackageManager} commandName
  * @returns {SupportedPackageManager | null}
@@ -24,18 +39,28 @@ const detectPackageManager = () =>
 
 /**
  * Get the package manager and root directory of the project. If the project does not have a
- * project manager configured, a supported package manager will be detected in your `PATH`, and
+ * package manager configured, a supported package manager will be detected in your `PATH`, and
  * `rootDir` will be `null`.
  * @returns {{packageManager: SupportedPackageManager, rootDir: string | null}}
  */
 const getPackageManager = () => {
   let _packageManager = skuArgs?.packageManager;
 
+  // @manypkg/find-root only returns a tool if it finds a monorepo.
+  // If it finds a regular repo, it will return a 'root' tool, which is absolutely useless.
+  // So we need to detect the package manager ourselves. I'd use `detect` from from `@antfu/ni` or
+  // `detect-package-manager`, but they're async only and we can't make getPackageManager async.
   try {
-    const {
-      tool: { type: foundPackageManager },
-      rootDir,
-    } = findRootSync(cwd());
+    const { rootDir } = findRootSync(cwd());
+
+    let foundPackageManager;
+
+    for (const supportedLockfile of supportedLockfiles) {
+      if (existsSync(join(rootDir, supportedLockfile))) {
+        foundPackageManager = LOCKS[supportedLockfile];
+        break;
+      }
+    }
 
     if (!supportedPackageManagers.includes(foundPackageManager)) {
       throw new Error('Unsupported package manager found');
@@ -126,7 +151,7 @@ const getAddCommand = ({ type, logLevel, deps, exact }) => {
   return getCommand(packageManager, 'add', args);
 };
 
-const getInstallCommand = () => getCommand(packageManager, 'install', []);
+const getInstallCommand = () => getCommand(packageManager, 'install');
 
 const getWhyCommand = () => {
   const whyCommand = isPnpm ? 'why -r' : 'why';

package/lib/preCommit.js

@@ -1,11 +1,12 @@
 const chalk = require('chalk');
+
+const config = require('../config/lintStaged/lintStagedConfig');
 const lintStaged = require('lint-staged');
-const configPath = require.resolve('../config/lintStaged/lintStagedConfig');
 
 module.exports = async () => {
   let success = false;
   try {
-    success = await lintStaged({ configPath });
+    success = await lintStaged({ config });
   } catch (e) {
     console.error(chalk.red(e));
   }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "sku",
-  "version": "12.4.4",
+  "version": "12.4.6",
   "description": "Front-end development toolkit, powered by Webpack, Babel, CSS Modules, Less and Jest",
   "main": "index.js",
   "bin": {
@@ -71,13 +71,13 @@
     "cssnano": "^6.0.0",
     "death": "^1.1.0",
     "debug": "^4.3.1",
-    "dedent": "^0.7.0",
-    "didyoumean2": "^5.0.0",
+    "dedent": "^1.5.1",
+    "didyoumean2": "^6.0.1",
     "ejs": "^3.1.8",
     "empty-dir": "^3.0.0",
     "ensure-gitignore": "^1.1.2",
     "env-ci": "^7.0.0",
-    "esbuild": "^0.17.0",
+    "esbuild": "^0.19.7",
     "esbuild-register": "^3.3.3",
     "escape-string-regexp": "^4.0.0",
     "eslint": "^8.41.0",
@@ -128,7 +128,6 @@
   },
   "devDependencies": {
     "@types/cross-spawn": "^6.0.3",
-    "@types/dedent": "^0.7.0",
     "@types/express": "^4.17.11",
     "@types/react": "^18.2.3",
     "@types/react-dom": "^18.2.3",