@shazhou/proman-core 0.9.0 → 0.9.2

package/CHANGELOG.md

@@ -1,5 +1,36 @@
 # Changelog
 
+## 0.9.2 — 2026-06-13
+
+- Fix build regression: cli type falls back to tsc when no build script
+  
+  Since the monorepo refactor, `proman build` dispatched `pnpm run build`
+  for all `cli`-type packages. This broke consumers (ocas, uwf) whose cli
+  packages have no build script — they relied on proman's built-in
+  `tsc --build`.
+  
+  Now cli-type packages check for a build script in package.json first.
+  If present, use `pnpm run build`; otherwise fall back to `tsc --build`
+  (same as lib/api types).
+
+## 0.9.1 — 2026-06-13
+
+- Update release workflow for monorepo and fix smoke test workspace deps
+  
+  **release.yaml v2**: Updated for monorepo structure — uses `proman bump` and
+  `proman publish` instead of raw `npm version`/`pnpm publish`. Preflight checks
+  all packages' versions on npm. Publisher verifies `workspace:*` was resolved
+  correctly after publish.
+  
+  **smoke-test**: `smokeTestTarball` now accepts `workspacePackages` map and
+  symlinks workspace dependencies into extracted tarball's `node_modules/`,
+  fixing smoke test failure for packages with workspace deps (e.g. CLI → core).
+  
+  **smoke-test parse fix**: Extract `.tgz` filename from pnpm pack verbose output
+  via regex instead of treating entire stdout as filename.
+  
+  **docs**: Renamed `lint` → `check` across all documentation and pre-push hook.
+
 ## 0.9.0 — 2026-06-13
 
 - Refactor proman into a monorepo with `@shazhou/proman-core` and `@shazhou/proman` packages.

package/dist/commands/dev.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"dev.d.ts","sourceRoot":"","sources":["../../src/commands/dev.ts"],"names":[],"mappings":"AAUA,OAAO,EAA4B,KAAK,OAAO,EAAE,MAAM,iBAAiB,CAAA;AAExE,MAAM,MAAM,iBAAiB,GAAG;IAC9B,GAAG,EAAE,MAAM,CAAA;IACX,KAAK,CAAC,EAAE,OAAO,CAAA;IACf;;;0EAGsE;IACtE,KAAK,CAAC,EAAE,OAAO,CAAA;CAChB,CAAA;AAMD,wBAAsB,KAAK,CAAC,IAAI,EAAE,iBAAiB,GAAG,OAAO,CAAC,IAAI,CAAC,CAuElE;AAsBD,wBAAsB,QAAQ,CAAC,IAAI,EAAE,iBAAiB,GAAG,OAAO,CAAC,IAAI,CAAC,CAuBrE;AAED,wBAAsB,KAAK,CAAC,IAAI,EAAE,iBAAiB,GAAG,OAAO,CAAC,IAAI,CAAC,CAyBlE;AAED,wBAAsB,MAAM,CAAC,IAAI,EAAE,iBAAiB,GAAG,OAAO,CAAC,IAAI,CAAC,CAInE"}
+{"version":3,"file":"dev.d.ts","sourceRoot":"","sources":["../../src/commands/dev.ts"],"names":[],"mappings":"AAUA,OAAO,EAA4B,KAAK,OAAO,EAAE,MAAM,iBAAiB,CAAA;AAExE,MAAM,MAAM,iBAAiB,GAAG;IAC9B,GAAG,EAAE,MAAM,CAAA;IACX,KAAK,CAAC,EAAE,OAAO,CAAA;IACf;;;0EAGsE;IACtE,KAAK,CAAC,EAAE,OAAO,CAAA;CAChB,CAAA;AAMD,wBAAsB,KAAK,CAAC,IAAI,EAAE,iBAAiB,GAAG,OAAO,CAAC,IAAI,CAAC,CA+ElE;AAsBD,wBAAsB,QAAQ,CAAC,IAAI,EAAE,iBAAiB,GAAG,OAAO,CAAC,IAAI,CAAC,CAuBrE;AAED,wBAAsB,KAAK,CAAC,IAAI,EAAE,iBAAiB,GAAG,OAAO,CAAC,IAAI,CAAC,CAyBlE;AAED,wBAAsB,MAAM,CAAC,IAAI,EAAE,iBAAiB,GAAG,OAAO,CAAC,IAAI,CAAC,CAInE"}

package/dist/commands/dev.js

@@ -49,10 +49,18 @@ export async function build(opts) {
             case 'webui':
                 argv = pnpmExec('vite', 'build');
                 break;
-            case 'cli':
-                // cli → use package's own build script (may be esbuild, tsc, etc.)
-                argv = ['pnpm', 'run', 'build'];
+            case 'cli': {
+                // cli → prefer package's own build script if it exists,
+                // otherwise fall back to tsc --build (same as lib/api)
+                const pkgJsonPath = join(pkgDir, 'package.json');
+                let hasBuildScript = false;
+                if (existsSync(pkgJsonPath)) {
+                    const pkgJson = JSON.parse(readFileSync(pkgJsonPath, 'utf-8'));
+                    hasBuildScript = pkgJson.scripts?.build != null;
+                }
+                argv = hasBuildScript ? ['pnpm', 'run', 'build'] : pnpmExec('tsc', '--build');
                 break;
+            }
             default:
                 // lib | api → tsc --build
                 argv = pnpmExec('tsc', '--build');

package/dist/commands/publish.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"publish.d.ts","sourceRoot":"","sources":["../../src/commands/publish.ts"],"names":[],"mappings":"AAGA,OAAO,EAAgB,KAAK,MAAM,EAAE,MAAM,iBAAiB,CAAA;AAC3D,OAAO,EAIL,KAAK,gBAAgB,EACrB,KAAK,SAAS,EACd,KAAK,OAAO,EACb,MAAM,iBAAiB,CAAA;AAGxB,YAAY,EAAE,MAAM,EAAE,MAAM,iBAAiB,CAAA;AAC7C,YAAY,EAAE,SAAS,EAAE,MAAM,iBAAiB,CAAA;AAEhD,MAAM,MAAM,cAAc,GAAG;IAC3B,SAAS,CAAC,EAAE,OAAO,CAAA;IACnB,GAAG,CAAC,EAAE,MAAM,CAAA;IACZ,GAAG,CAAC,EAAE,MAAM,CAAA;IACZ,GAAG,CAAC,EAAE,SAAS,CAAA;IACf,aAAa,CAAC,EAAE,gBAAgB,CAAA;IAChC,KAAK,CAAC,EAAE,OAAO,CAAA;CAChB,CAAA;AAoBD;;;GAGG;AACH,wBAAsB,OAAO,CAAC,IAAI,GAAE,cAAmB,GAAG,OAAO,CAAC,IAAI,CAAC,CAuHtE"}
+{"version":3,"file":"publish.d.ts","sourceRoot":"","sources":["../../src/commands/publish.ts"],"names":[],"mappings":"AAGA,OAAO,EAAgB,KAAK,MAAM,EAAE,MAAM,iBAAiB,CAAA;AAC3D,OAAO,EAIL,KAAK,gBAAgB,EACrB,KAAK,SAAS,EACd,KAAK,OAAO,EACb,MAAM,iBAAiB,CAAA;AAGxB,YAAY,EAAE,MAAM,EAAE,MAAM,iBAAiB,CAAA;AAC7C,YAAY,EAAE,SAAS,EAAE,MAAM,iBAAiB,CAAA;AAEhD,MAAM,MAAM,cAAc,GAAG;IAC3B,SAAS,CAAC,EAAE,OAAO,CAAA;IACnB,GAAG,CAAC,EAAE,MAAM,CAAA;IACZ,GAAG,CAAC,EAAE,MAAM,CAAA;IACZ,GAAG,CAAC,EAAE,SAAS,CAAA;IACf,aAAa,CAAC,EAAE,gBAAgB,CAAA;IAChC,KAAK,CAAC,EAAE,OAAO,CAAA;CAChB,CAAA;AAoBD;;;GAGG;AACH,wBAAsB,OAAO,CAAC,IAAI,GAAE,cAAmB,GAAG,OAAO,CAAC,IAAI,CAAC,CA4HtE"}

package/dist/commands/publish.js

@@ -75,7 +75,12 @@ export async function publish(opts = {}) {
         }
         // Smoke test: validate tarball before publishing
         try {
-            await smokeTestTarball(pkgDir, spawn);
+            // Build workspace package map for symlink resolution
+            const workspacePackages = {};
+            for (const pkg of cfg.packages) {
+                workspacePackages[pkg.name] = resolve(cwd, pkg.path);
+            }
+            await smokeTestTarball(pkgDir, spawn, workspacePackages);
         }
         catch (err) {
             const message = err.message;

package/dist/utils/smoke-test.d.ts

@@ -1,7 +1,17 @@
 import type { SpawnFn } from './npm.js';
+/**
+ * Map of workspace package names → absolute paths on disk.
+ * Used to symlink workspace dependencies into the smoke test directory
+ * so bin commands can resolve them without npm install.
+ */
+export type WorkspacePackages = Record<string, string>;
 /**
  * Smoke test a package tarball by extracting it and running bin commands.
  * Validates that the packaged artifact actually works before publishing.
+ *
+ * @param workspacePackages - Map of workspace package names to their absolute
+ *   paths. When provided, workspace dependencies are symlinked into the
+ *   extracted tarball's node_modules so bin commands can resolve them.
  */
-export declare function smokeTestTarball(pkgDir: string, spawn: SpawnFn): Promise<void>;
+export declare function smokeTestTarball(pkgDir: string, spawn: SpawnFn, workspacePackages?: WorkspacePackages): Promise<void>;
 //# sourceMappingURL=smoke-test.d.ts.map

package/dist/utils/smoke-test.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"smoke-test.d.ts","sourceRoot":"","sources":["../../src/utils/smoke-test.ts"],"names":[],"mappings":"AAGA,OAAO,KAAK,EAAE,OAAO,EAAE,MAAM,UAAU,CAAA;AAQvC;;;GAGG;AACH,wBAAsB,gBAAgB,CAAC,MAAM,EAAE,MAAM,EAAE,KAAK,EAAE,OAAO,GAAG,OAAO,CAAC,IAAI,CAAC,CA+DpF"}
+{"version":3,"file":"smoke-test.d.ts","sourceRoot":"","sources":["../../src/utils/smoke-test.ts"],"names":[],"mappings":"AAGA,OAAO,KAAK,EAAE,OAAO,EAAE,MAAM,UAAU,CAAA;AASvC;;;;GAIG;AACH,MAAM,MAAM,iBAAiB,GAAG,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,CAAA;AAEtD;;;;;;;GAOG;AACH,wBAAsB,gBAAgB,CACpC,MAAM,EAAE,MAAM,EACd,KAAK,EAAE,OAAO,EACd,iBAAiB,CAAC,EAAE,iBAAiB,GACpC,OAAO,CAAC,IAAI,CAAC,CAqFf"}

package/dist/utils/smoke-test.js

@@ -1,11 +1,15 @@
-import { readFile, rm } from 'node:fs/promises';
+import { mkdir, readFile, rm, symlink } from 'node:fs/promises';
 import { tmpdir } from 'node:os';
 import { join } from 'node:path';
 /**
  * Smoke test a package tarball by extracting it and running bin commands.
  * Validates that the packaged artifact actually works before publishing.
+ *
+ * @param workspacePackages - Map of workspace package names to their absolute
+ *   paths. When provided, workspace dependencies are symlinked into the
+ *   extracted tarball's node_modules so bin commands can resolve them.
  */
-export async function smokeTestTarball(pkgDir, spawn) {
+export async function smokeTestTarball(pkgDir, spawn, workspacePackages) {
     // Read package.json to check for bin entries
     const pkgJsonPath = join(pkgDir, 'package.json');
     const pkgJsonText = await readFile(pkgJsonPath, 'utf8');
@@ -25,10 +29,13 @@ export async function smokeTestTarball(pkgDir, spawn) {
     if (packResult.code !== 0) {
         throw new Error(`pnpm pack failed: ${packResult.stderr || packResult.stdout}`);
     }
-    const tarballName = packResult.stdout.trim();
-    if (!tarballName) {
-        throw new Error('pnpm pack did not return tarball filename');
+    // pnpm pack outputs verbose info (📦, file list, etc.)
+    // Extract the .tgz filename from the output
+    const tgzMatch = packResult.stdout.match(/[\w@.-]+\.tgz/);
+    if (!tgzMatch) {
+        throw new Error(`pnpm pack did not return tarball filename. Output: ${packResult.stdout}`);
     }
+    const tarballName = tgzMatch[0];
     // Step 2: Extract tarball to temp directory
     const { mkdtemp } = await import('node:fs/promises');
     const testDir = await mkdtemp(join(tmpdir(), 'proman-smoke-'));
@@ -41,7 +48,25 @@ export async function smokeTestTarball(pkgDir, spawn) {
         }
         // pnpm pack creates a 'package/' directory inside the tarball
         const extractedPkgDir = join(testDir, 'package');
-        // Step 3: Test each bin entry
+        // Step 3: Symlink workspace dependencies into node_modules
+        if (workspacePackages) {
+            const deps = pkgJson.dependencies ?? {};
+            const nodeModulesDir = join(extractedPkgDir, 'node_modules');
+            for (const [depName, depPath] of Object.entries(workspacePackages)) {
+                if (depName in deps) {
+                    // Handle scoped packages: @scope/name → node_modules/@scope/name
+                    const segments = depName.split('/');
+                    if (segments.length === 2) {
+                        await mkdir(join(nodeModulesDir, segments[0]), { recursive: true });
+                    }
+                    else {
+                        await mkdir(nodeModulesDir, { recursive: true });
+                    }
+                    await symlink(depPath, join(nodeModulesDir, depName), 'dir');
+                }
+            }
+        }
+        // Step 4: Test each bin entry
         for (const [binName, binPath] of Object.entries(binEntries)) {
             const binFullPath = join(extractedPkgDir, binPath);
             const binTestResult = await spawn(['node', binFullPath, '--version'], extractedPkgDir);
@@ -52,7 +77,7 @@ export async function smokeTestTarball(pkgDir, spawn) {
         }
     }
     finally {
-        // Step 4: Always clean up temp directory and tarball
+        // Step 5: Always clean up temp directory and tarball
         await rm(testDir, { recursive: true, force: true });
         await rm(join(pkgDir, tarballName), { force: true });
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@shazhou/proman-core",
-  "version": "0.9.0",
+  "version": "0.9.2",
   "type": "module",
   "description": "Core library for TypeScript monorepo management",
   "keywords": [

package/src/commands/dev.ts

@@ -74,10 +74,18 @@ export async function build(opts: DevCommandOptions): Promise<void> {
       case 'webui':
         argv = pnpmExec('vite', 'build')
         break
-      case 'cli':
-        // cli → use package's own build script (may be esbuild, tsc, etc.)
-        argv = ['pnpm', 'run', 'build']
+      case 'cli': {
+        // cli → prefer package's own build script if it exists,
+        // otherwise fall back to tsc --build (same as lib/api)
+        const pkgJsonPath = join(pkgDir, 'package.json')
+        let hasBuildScript = false
+        if (existsSync(pkgJsonPath)) {
+          const pkgJson = JSON.parse(readFileSync(pkgJsonPath, 'utf-8'))
+          hasBuildScript = pkgJson.scripts?.build != null
+        }
+        argv = hasBuildScript ? ['pnpm', 'run', 'build'] : pnpmExec('tsc', '--build')
         break
+      }
       default:
         // lib | api → tsc --build
         argv = pnpmExec('tsc', '--build')

package/src/commands/publish.ts

@@ -113,7 +113,12 @@ export async function publish(opts: PublishOptions = {}): Promise<void> {
 
     // Smoke test: validate tarball before publishing
     try {
-      await smokeTestTarball(pkgDir, spawn)
+      // Build workspace package map for symlink resolution
+      const workspacePackages: Record<string, string> = {}
+      for (const pkg of cfg.packages) {
+        workspacePackages[pkg.name] = resolve(cwd, pkg.path)
+      }
+      await smokeTestTarball(pkgDir, spawn, workspacePackages)
     } catch (err) {
       const message = (err as Error).message
       const published = publishablePackages.slice(0, i).map((p) => p.name)

package/src/utils/smoke-test.ts

@@ -1,4 +1,4 @@
-import { readFile, rm } from 'node:fs/promises'
+import { mkdir, readFile, rm, symlink } from 'node:fs/promises'
 import { tmpdir } from 'node:os'
 import { join } from 'node:path'
 import type { SpawnFn } from './npm.js'
@@ -7,13 +7,29 @@ type PackageJson = {
   name: string
   version: string
   bin?: string | Record<string, string>
+  dependencies?: Record<string, string>
 }
 
+/**
+ * Map of workspace package names → absolute paths on disk.
+ * Used to symlink workspace dependencies into the smoke test directory
+ * so bin commands can resolve them without npm install.
+ */
+export type WorkspacePackages = Record<string, string>
+
 /**
  * Smoke test a package tarball by extracting it and running bin commands.
  * Validates that the packaged artifact actually works before publishing.
+ *
+ * @param workspacePackages - Map of workspace package names to their absolute
+ *   paths. When provided, workspace dependencies are symlinked into the
+ *   extracted tarball's node_modules so bin commands can resolve them.
  */
-export async function smokeTestTarball(pkgDir: string, spawn: SpawnFn): Promise<void> {
+export async function smokeTestTarball(
+  pkgDir: string,
+  spawn: SpawnFn,
+  workspacePackages?: WorkspacePackages,
+): Promise<void> {
   // Read package.json to check for bin entries
   const pkgJsonPath = join(pkgDir, 'package.json')
   const pkgJsonText = await readFile(pkgJsonPath, 'utf8')
@@ -39,10 +55,13 @@ export async function smokeTestTarball(pkgDir: string, spawn: SpawnFn): Promise<
     throw new Error(`pnpm pack failed: ${packResult.stderr || packResult.stdout}`)
   }
 
-  const tarballName = packResult.stdout.trim()
-  if (!tarballName) {
-    throw new Error('pnpm pack did not return tarball filename')
+  // pnpm pack outputs verbose info (📦, file list, etc.)
+  // Extract the .tgz filename from the output
+  const tgzMatch = packResult.stdout.match(/[\w@.-]+\.tgz/)
+  if (!tgzMatch) {
+    throw new Error(`pnpm pack did not return tarball filename. Output: ${packResult.stdout}`)
   }
+  const tarballName = tgzMatch[0]
 
   // Step 2: Extract tarball to temp directory
   const { mkdtemp } = await import('node:fs/promises')
@@ -59,7 +78,26 @@ export async function smokeTestTarball(pkgDir: string, spawn: SpawnFn): Promise<
     // pnpm pack creates a 'package/' directory inside the tarball
     const extractedPkgDir = join(testDir, 'package')
 
-    // Step 3: Test each bin entry
+    // Step 3: Symlink workspace dependencies into node_modules
+    if (workspacePackages) {
+      const deps = pkgJson.dependencies ?? {}
+      const nodeModulesDir = join(extractedPkgDir, 'node_modules')
+
+      for (const [depName, depPath] of Object.entries(workspacePackages)) {
+        if (depName in deps) {
+          // Handle scoped packages: @scope/name → node_modules/@scope/name
+          const segments = depName.split('/')
+          if (segments.length === 2) {
+            await mkdir(join(nodeModulesDir, segments[0] as string), { recursive: true })
+          } else {
+            await mkdir(nodeModulesDir, { recursive: true })
+          }
+          await symlink(depPath, join(nodeModulesDir, depName), 'dir')
+        }
+      }
+    }
+
+    // Step 4: Test each bin entry
     for (const [binName, binPath] of Object.entries(binEntries)) {
       const binFullPath = join(extractedPkgDir, binPath)
       const binTestResult = await spawn(['node', binFullPath, '--version'], extractedPkgDir)
@@ -72,7 +110,7 @@ export async function smokeTestTarball(pkgDir: string, spawn: SpawnFn): Promise<
       }
     }
   } finally {
-    // Step 4: Always clean up temp directory and tarball
+    // Step 5: Always clean up temp directory and tarball
     await rm(testDir, { recursive: true, force: true })
     await rm(join(pkgDir, tarballName), { force: true })
   }

package/tests/dev.test.ts

@@ -67,7 +67,7 @@ describe('build command', () => {
     // lib: tsc --build
     expectExec(calls[0]?.argv, 'tsc', ['--build'])
     expect(calls[0]?.cwd).toBe(resolve(FIX('typed'), 'packages/core'))
-    // cli: pnpm run build
+    // cli: pnpm run build (has build script in package.json)
     expect(calls[1]?.argv).toEqual(['pnpm', 'run', 'build'])
     expect(calls[1]?.cwd).toBe(resolve(FIX('typed'), 'packages/mycli'))
     // webui: vite build

package/tests/fixtures/typed/packages/mycli/package.json

@@ -0,0 +1,7 @@
+{
+  "name": "@test/mycli",
+  "version": "0.1.0",
+  "scripts": {
+    "build": "esbuild src/cli.ts --bundle --outfile=dist/cli.js"
+  }
+}