@elliemae/pui-cli 9.0.0-alpha.7 → 9.0.0-alpha.9

package/dist/esm/monorepo/utils.cjs

@@ -1,22 +1,30 @@
 const path = require('path');
-const { execSync } = require('child_process');
+const { execFileSync } = require('child_process');
 
 const WORKSPACE_DIR_ENV_VAR = 'NPM_CONFIG_WORKSPACE_DIR';
 const WORKSPACE_MANIFEST_FILENAME = 'pnpm-workspace.yaml';
 
+const findUpCli = path.join(
+  path.dirname(require.resolve('find-up-cli/package.json')),
+  'cli.js',
+);
+
 const getPNPMWorkspaceLocation = (cwd) => {
-  let location = null;
-   
   for (const fileName of [WORKSPACE_MANIFEST_FILENAME, 'pnpm-workspace.yml']) {
     try {
-      const result = execSync(`npx find-up ${fileName}`, { cwd });
-      location = result.toString().trim();
-      break;
+      const location = execFileSync(
+        process.execPath,
+        [findUpCli, fileName, `--cwd=${cwd}`],
+        { encoding: 'utf8' },
+      ).trim();
+      if (location) {
+        return location;
+      }
     } catch {
       // ignore
     }
   }
-  return location;
+  return null;
 };
 
 exports.findMonoRepoRoot = (cwd = process.cwd()) => {

package/dist/esm/monorepo/utils.js

@@ -1,18 +1,15 @@
 import path from "node:path";
-import { execSync } from "child_process";
+import { findUpSync } from "find-up";
 const WORKSPACE_DIR_ENV_VAR = "NPM_CONFIG_WORKSPACE_DIR";
 const WORKSPACE_MANIFEST_FILENAME = "pnpm-workspace.yaml";
 const getPNPMWorkspaceLocation = (cwd) => {
-  let location = null;
   for (const fileName of [WORKSPACE_MANIFEST_FILENAME, "pnpm-workspace.yml"]) {
-    try {
-      const result = execSync(`npx find-up ${fileName}`, { cwd });
-      location = result.toString().trim();
-      break;
-    } catch {
+    const location = findUpSync(fileName, { cwd });
+    if (location) {
+      return location;
     }
   }
-  return location;
+  return null;
 };
 const findMonoRepoRoot = (cwd = process.cwd()) => {
   const workspaceManifestDirEnvVar = process.env[WORKSPACE_DIR_ENV_VAR] ?? process.env[WORKSPACE_DIR_ENV_VAR.toLowerCase()];

package/dist/esm/skills/migrate-to-pui-cli-9/SKILL.md

@@ -1,15 +1,17 @@
 ---
 name: migrate-to-pui-cli-9
 description: >-
-  Migrate a PUI app or library to pui-cli 9 (Node 24, pnpm 11, ESLint 9 flat config).
-  Use when upgrading @elliemae/pui-cli, migrating from .eslintrc.cjs to eslint.config.mjs,
-  fixing ESLint 9 lint failures, or adopting the shared flat config from pui-cli.
+  Migrate a PUI app or library to pui-cli 9 (Node 24, pnpm 11, ESLint 10 flat config, Stylelint 17,
+  Husky 9, Vitest 4). Use when upgrading @elliemae/pui-cli, migrating from .eslintrc.cjs to
+  eslint.config.mjs, upgrading Husky 8 hooks, fixing Vitest 4 / coverage-v8 breakages, or adopting
+  shared configs from pui-cli. App/library production builds stay on Webpack.
 ---
 
 # Migrate to pui-cli 9
 
 Upgrades a PUI repo from pui-cli 8 (ESLint 8 + `.eslintrc.cjs`) to pui-cli 9
-(ESLint 9 + `eslint.config.mjs` + Node 24 + pnpm 11).
+(ESLint 10 + `eslint.config.mjs` + Node 24 + pnpm 11). Production bundling remains Webpack;
+Vitest 4 uses Vite only as the test runner when you run `pui-cli vitest`.
 
 ## Pre-flight: Toolchain
 
@@ -42,7 +44,7 @@ pnpm add -D @elliemae/pui-cli@9
 
 Re-run install and smoke-test build/test without ESLint changes yet if the bump is large.
 
-### Phase 3 — ESLint 9 flat config
+### Phase 3 — ESLint 10 flat config
 
 **React apps and libraries:**
 
@@ -95,13 +97,111 @@ export default [
 
 Remove overrides in a follow-up debt PR. Do not copy Airbnb or legacy `.eslintrc` rules.
 
-### Phase 5 — Verify
+### Phase 5 — Husky 9 (if `.husky/` exists)
+
+Skip this phase if the repo has no git hooks. Most PUI apps/libs copied hooks from pui-cli boilerplate and still use Husky 8.
+
+1. Upgrade and remove deprecated package:
+
+```bash
+pnpm add -D husky@9
+pnpm remove husky-init   # if present
+```
+
+2. Update `package.json` `prepare` script:
+
+```diff
+- "prepare": "[ -n \"$CI\" ] || husky install"
++ "prepare": "[ -n \"$CI\" ] || husky"
+```
+
+3. Simplify each hook in `.husky/` — remove the shebang and `husky.sh` source. Keep only the command(s):
+
+**`.husky/pre-commit`** (typical PUI repo):
+
+```sh
+pnpm -s dlx lint-staged
+```
+
+**`.husky/commit-msg`** (if commitlint is enabled):
+
+```sh
+pnpm exec commitlint --edit $1
+```
+
+```diff
+- #!/bin/sh
+- . "$(dirname "$0")/_/husky.sh"
+-
+  pnpm -s dlx lint-staged
+```
+
+4. Delete `.husky/.gitignore` if present (Husky 9 regenerates `.husky/_` on install).
+
+5. Run `pnpm prepare`, then verify hooks with a test commit.
+
+**Environment variable changes (Husky 9):**
+
+| Husky 8                                   | Husky 9       |
+| ----------------------------------------- | ------------- |
+| `HUSKY_SKIP_HOOKS` / `HUSKY_SKIP_INSTALL` | `HUSKY=0`     |
+| `HUSKY_GIT_PARAMS` in commit-msg hooks    | `$1`, `$2`, … |
+
+Reference hooks: [pui-cli/.husky](https://git.elliemae.io/platform-ui/pui-cli/tree/master/.husky)
+
+### Phase 6 — Stylelint 17 (if CSS / styled-components)
+
+1. Simplify `stylelint.config.cjs` — remove `stylelint-config-styled-components` workarounds.
+2. Optional ESM: `stylelint.config.mjs` importing `@elliemae/pui-cli/stylelint`.
+3. Run `pnpm exec pui-cli lint` (CSS pass).
+
+See [stylelint-migration.md](https://docs.pui.mortgagetech.q1.ice.com/cli/stylelint-migration) in pui-cli docs.
+
+### Phase 7 — Vitest 4 (if using `pui-cli vitest`)
+
+Skip if the repo uses Jest (`pui-cli test`) only.
+
+1. Align devDependencies with pui-cli 9 (Vitest 4 stack):
+
+```bash
+pnpm add -D vitest@4 @vitest/coverage-v8@4 vite@8 @vitejs/plugin-react@6 vite-tsconfig-paths@6
+pnpm remove @vitest/coverage-c8 @types/uuid   # when upgrading uuid to 14+
+```
+
+2. Prefer extending shared config:
+
+```typescript
+import { defineConfig, mergeConfig } from 'vitest/config';
+import { vitestConfig } from '@elliemae/pui-cli/vitest';
+
+export default mergeConfig(
+  vitestConfig,
+  defineConfig({
+    test: {
+      /* overrides */
+    },
+  }),
+);
+```
+
+3. Update custom Vitest 1.x options:
+
+| Old                       | New                          |
+| ------------------------- | ---------------------------- |
+| `test.deps.optimizer.web` | `test.deps.optimizer.client` |
+| `@vitest/coverage-c8`     | `@vitest/coverage-v8`        |
+
+4. If `tsconfig.json` extends `@elliemae/pui-cli/app.tsconfig.json` or `library.tsconfig.json`, ensure pui-cli 9 is installed — shared tsconfigs use `moduleResolution: "bundler"`. Custom tsconfigs importing `vitest/config` may need the same.
+
+5. Smoke-test: `pnpm exec pui-cli vitest --passWithNoTests`
+
+### Phase 8 — Verify
 
 - [ ] `pnpm exec pui-cli lint` — 0 errors
 - [ ] `pnpm exec pui-cli tscheck --files`
-- [ ] `pnpm test`
-- [ ] `pnpm run build` (or `pui-cli build`)
-- [ ] Pre-commit / lint-staged passes
+- [ ] `pnpm test` (or `pui-cli vitest` if applicable)
+- [ ] `pnpm run build` (or `pui-cli build`) — Webpack, unchanged
+- [ ] Pre-commit / lint-staged passes (after Husky 9 migration if applicable)
 - [ ] CI green on the target branch
 
 ## Common lint fixes after upgrade
@@ -114,6 +214,9 @@ Remove overrides in a follow-up debt PR. Do not copy Airbnb or legacy `.eslintrc
 | Stale `eslint-disable` comments      | Remove disables for rules no longer in config                          |
 | Test/fixture files flagged           | Shared config includes `lib/**/tests/**` globs — ensure pui-cli 9.0.0+ |
 | `.d.ts` files                        | pui-cli turns off `no-explicit-any` for `**/*.d.ts`                    |
+| Cannot find module `vite` in tscheck | Set `moduleResolution: "bundler"` or extend pui-cli 9 tsconfig         |
+| Vitest `optimizer.web` unknown       | Rename to `test.deps.optimizer.client` (Vitest 4)                      |
+| Coverage fails after upgrade         | Remove `@vitest/coverage-c8`; use `@vitest/coverage-v8@4`              |
 
 Full rule comparison: [eslint-rules-migration.md](https://docs.pui.mortgagetech.q1.ice.com/cli/eslint-rules-migration) (also at `docs/eslint-rules-migration.md` in pui-cli).
 
@@ -130,11 +233,12 @@ Skills are copied to `.cursor/skills/`, `.claude/skills/`, and `.github/skills/`
 ## What NOT to change
 
 - Application business logic — migration is tooling/config only
-- Webpack/babel config unless pui-cli 9 release notes require it
-- Prettier / Stylelint / commitlint configs unless pui-cli 9 bumps those presets
+- Webpack/babel production build config — pui-cli 9 does not migrate apps to Vite for bundling
+- Prettier / commitlint configs unless pui-cli 9 bumps those presets
 
 ## Additional resources
 
 - [pui-cli 9 migration guide](https://docs.pui.mortgagetech.q1.ice.com/cli/pui-cli-9-migration)
 - [ESLint rules migration guide](https://docs.pui.mortgagetech.q1.ice.com/cli/eslint-rules-migration)
+- [Stylelint migration guide](https://docs.pui.mortgagetech.q1.ice.com/cli/stylelint-migration)
 - Reference one-liner: `pui-react-boilerplate/eslint.config.mjs`

package/dist/esm/testing/vitest.config.js

@@ -14,11 +14,12 @@ const vitestConfig = defineConfig({
     include: ["./{app,lib}/**/*.{test,spec}.{js,mjs,cjs,ts,mts,cts,jsx,tsx}"],
     exclude: [...configDefaults.exclude, ".idea", ".git", ".cache", "e2e"],
     coverage: {
+      provider: "v8",
       reportsDirectory: "./reports/coverage"
     },
     deps: {
       optimizer: {
-        web: {
+        client: {
           include: ["app.config.json", "@elliemae/pui-app-sdk"]
         }
       }

package/dist/types/lib/commands/build.d.ts

@@ -1,4 +1,4 @@
-import { type CommandModule } from 'yargs';
+import type { CommandModule } from 'yargs';
 type Arguments = {
     service: boolean;
 };

package/dist/types/lib/commands/buildcdn.d.ts

@@ -1,2 +1,2 @@
-import { type CommandModule } from 'yargs';
+import type { CommandModule } from 'yargs';
 export declare const buildCDNCmd: CommandModule<Record<string, never>>;

package/dist/types/lib/commands/codemod.d.ts

@@ -1,4 +1,4 @@
-import { type CommandModule } from 'yargs';
+import type { CommandModule } from 'yargs';
 type Arguments = {
     transform: string;
 };

package/dist/types/lib/commands/gendoc.d.ts

@@ -1,2 +1,2 @@
-import { type CommandModule } from 'yargs';
+import type { CommandModule } from 'yargs';
 export declare const gendocCmd: CommandModule<Record<string, never>, Record<string, never>>;

package/dist/types/lib/commands/lint.d.ts

@@ -1,4 +1,4 @@
-import { type CommandModule } from 'yargs';
+import type { CommandModule } from 'yargs';
 type Arguments = {
     css: boolean;
     js: boolean;

package/dist/types/lib/commands/pack.d.ts

@@ -1,4 +1,4 @@
-import { type CommandModule } from 'yargs';
+import type { CommandModule } from 'yargs';
 type Arguments = {
     production: boolean;
     target: string;

package/dist/types/lib/commands/skills.d.ts

@@ -1,4 +1,4 @@
-import { type CommandModule } from 'yargs';
+import type { CommandModule } from 'yargs';
 type SkillTarget = 'cursor' | 'claude' | 'copilot';
 type TargetOption = SkillTarget | 'all';
 type Arguments = {

package/dist/types/lib/commands/start.d.ts

@@ -1,4 +1,4 @@
-import { type CommandModule } from 'yargs';
+import type { CommandModule } from 'yargs';
 type Arguments = {
     prod: boolean;
 };

package/dist/types/lib/commands/storybook.d.ts

@@ -1,4 +1,4 @@
-import { type CommandModule } from 'yargs';
+import type { CommandModule } from 'yargs';
 type Arguments = {
     build: boolean;
     docs: boolean;

package/dist/types/lib/commands/test.d.ts

@@ -1,4 +1,4 @@
-import { type CommandModule } from 'yargs';
+import type { CommandModule } from 'yargs';
 type Arguments = {
     fix: boolean;
     watch: boolean;

package/dist/types/lib/commands/tscheck.d.ts

@@ -1,4 +1,4 @@
-import { type CommandModule } from 'yargs';
+import type { CommandModule } from 'yargs';
 export type Arguments = {
     files: string[];
 };

package/dist/types/lib/commands/utils.d.ts

@@ -1,5 +1,46 @@
-import { execaCommand } from 'execa';
-export declare const exec: (command: string, options?: Parameters<typeof execaCommand>[1]) => Promise<import("execa").ExecaReturnValue<Buffer<ArrayBufferLike>>>;
+import { type Options } from 'execa';
+export declare const exec: <OptionsType extends Options = Options>(command: string, options?: OptionsType) => Promise<import("execa").Result<{
+    shell: true;
+} & {
+    preferLocal?: boolean;
+    localDir?: string | URL;
+    node?: boolean;
+    nodeOptions?: readonly string[];
+    nodePath?: string | URL;
+    shell?: boolean | string | URL;
+    cwd?: string | URL;
+    env?: Readonly<Partial<Record<string, string>>>;
+    extendEnv?: boolean;
+    input?: string | Uint8Array | import("stream").Readable;
+    inputFile?: string | URL;
+    stdin?: import("node_modules/execa/types/stdio/type.js").StdinOptionCommon<false>;
+    stdout?: import("node_modules/execa/types/stdio/type.js").StdoutStderrOptionCommon<false>;
+    stderr?: import("node_modules/execa/types/stdio/type.js").StdoutStderrOptionCommon<false>;
+    stdio: "inherit" | "pipe" | "ignore" | "overlapped" | readonly [import("node_modules/execa/types/stdio/type.js").StdinOptionCommon<false, false>, import("node_modules/execa/types/stdio/type.js").StdoutStderrOptionCommon<false, false>, import("node_modules/execa/types/stdio/type.js").StdoutStderrOptionCommon<false, false>, ...import("node_modules/execa/types/stdio/type.js").StdioExtraOptionCommon<false>[]];
+    all?: boolean;
+    encoding?: import("node_modules/execa/types/arguments/encoding-option.js").EncodingOption;
+    lines?: import("node_modules/execa/types/arguments/specific.js").FdGenericOption<boolean>;
+    stripFinalNewline?: import("node_modules/execa/types/arguments/specific.js").FdGenericOption<boolean>;
+    maxBuffer?: import("node_modules/execa/types/arguments/specific.js").FdGenericOption<number>;
+    buffer?: import("node_modules/execa/types/arguments/specific.js").FdGenericOption<boolean>;
+    ipc?: boolean | undefined;
+    serialization?: "json" | "advanced" | undefined;
+    ipcInput?: import("execa").Message | undefined;
+    verbose?: import("node_modules/execa/types/verbose.js").VerboseOption;
+    reject?: boolean;
+    timeout?: number;
+    cancelSignal?: AbortSignal | undefined;
+    gracefulCancel?: boolean | undefined;
+    forceKillAfterDelay?: number | boolean | undefined;
+    killSignal?: keyof import("os").SignalConstants | number;
+    detached?: boolean | undefined;
+    cleanup?: boolean | undefined;
+    uid?: number;
+    gid?: number;
+    argv0?: string;
+    windowsHide?: boolean;
+    windowsVerbatimArguments?: boolean;
+}>>;
 export declare const logInfo: {
     (...data: any[]): void;
     (message?: any, ...optionalParams: any[]): void;

package/dist/types/lib/commands/version.d.ts

@@ -1,4 +1,4 @@
-import { type CommandModule } from 'yargs';
+import type { CommandModule } from 'yargs';
 type Arguments = {
     deleteTags: boolean;
     useRegistry: boolean;

package/dist/types/lib/commands/vitest.d.ts

@@ -1,4 +1,4 @@
-import { type CommandModule } from 'yargs';
+import type { CommandModule } from 'yargs';
 type Arguments = {
     fix?: boolean;
     watch?: boolean;

package/dist/types/lib/index.d.cts

@@ -5,7 +5,7 @@ import { eslintFlatConfig } from "./lint-config/eslint/flat/react-export.mjs";
 import { eslintFlatConfigStrict } from "./lint-config/eslint/flat/react-export.mjs";
 import { esConfig as eslintBaseConfig } from "./lint-config/eslint/non-react.cjs";
 import { esReactConfig as eslintConfig } from "./lint-config/eslint/react.cjs";
-import { stylelintConfig } from "./lint-config/stylelint.config.cjs";
+import { stylelintConfig } from "./lint-config/stylelint/export.mjs";
 import { prettierConfig } from "./lint-config/prettier.config.cjs";
 import { commitlintConfig } from "./lint-config/commitlint.config.cjs";
 import { jestConfig } from "./testing/jest.config.cjs";

package/dist/types/lib/index.d.ts

@@ -6,7 +6,7 @@ export { eslintFlatConfig, eslintFlatConfigStrict, } from './lint-config/eslint/
 export { esConfig as eslintBaseConfig } from './lint-config/eslint/non-react.cjs';
 /** @deprecated Use eslintFlatConfig in eslint.config.mjs */
 export { esReactConfig as eslintConfig } from './lint-config/eslint/react.cjs';
-export { stylelintConfig } from './lint-config/stylelint.config.cjs';
+export { stylelintConfig } from './lint-config/stylelint/export.mjs';
 export { prettierConfig } from './lint-config/prettier.config.cjs';
 export { commitlintConfig } from './lint-config/commitlint.config.cjs';
 export { jestConfig } from './testing/jest.config.cjs';

package/dist/types/lib/lint-config/eslint/flat/compat.d.mts

@@ -1,4 +1,9 @@
+/** React — uses context.getFilename / getScope (eslint-plugin-react@7). */
+/** @type {import('eslint').ESLint.Plugin} */
+export const reactPlugin: import("eslint").ESLint.Plugin;
+/** Redux-saga — uses context.getScope in no-yield-in-race. */
 /** @type {import('eslint').ESLint.Plugin} */
 export const reduxSagaPlugin: import("eslint").ESLint.Plugin;
-/** Storybook flat presets with ESLint 10 compatibility shims applied. */
-export const storybookFlatConfigs: import("@eslint/compat").FixupConfigArray;
+/** Storybook flat presets — uses legacy context APIs (eslint-plugin-storybook@0.12). */
+/** @type {import('eslint').Linter.Config[]} */
+export const storybookFlatConfigs: import("eslint").Linter.Config[];

package/dist/types/lib/lint-config/stylelint/config.d.mts

@@ -0,0 +1,3 @@
+/** @type {import('stylelint').Config} */
+export const stylelintConfig: import("stylelint").Config;
+export default stylelintConfig;

package/dist/types/lib/lint-config/stylelint/export.d.mts

@@ -0,0 +1 @@
+export { stylelintConfig, default } from "./config.mjs";

package/dist/types/lib/lint-config/stylelint.config.d.cts

@@ -1,10 +1,2 @@
-export namespace stylelintConfig {
-    export let ignoreFiles: string[];
-    export let customSyntax: string;
-    let _extends: string[];
-    export { _extends as extends };
-    export let rules: {
-        "selector-type-no-unknown": null;
-        "function-no-unknown": null;
-    };
-}
+export { stylelintConfig };
+import { stylelintConfig } from "./stylelint/export.mjs";