@produck/agent-toolkit 0.11.0 → 0.11.1

package/README.md

@@ -26,7 +26,7 @@ What it does (in order):
 6. Deploys root `.c8rc.json` and root `c8` devDependency
    **Note:** The `produck:coverage` script in subpackages is for local and AI development use only. It is NOT enforced by organization CI or `.c8rc.json`. Only the root workspace (monorepo root) is subject to org-level coverage enforcement and `.c8rc.json`.
 7. Deploys root `.gitattributes`
-8. Deploys the pinned `produck:coverage` script and `c8` devDependency to each workspace package, and enforces `scripts.test` (generates a default `test` script when missing).
+8. Deploys the pinned `produck:coverage` script to each workspace package and enforces `scripts.test` (generates a default `test` script when missing). Workspace packages rely on root-level c8 devDependency (hoisted by npm workspaces).
 9. Deploys `.husky/pre-commit` and `.husky/commit-msg`
 
 After running, add the persistent enforcement entry to the repository
@@ -90,11 +90,14 @@ failure:
    (`produck:baseline`, `produck:commit:check`) plus shared pinned root
    devDependencies (`husky`, `lerna`, `@produck/agent-toolkit`)
 8. `sync-coverage` — deploy root `scripts.produck:coverage`, `.c8rc.json`, and
-   root `c8` devDependency, then deploy pinned `produck:coverage` script and
-   `c8` devDependency into each workspace package, and ensure each workspace
-   package has `scripts.test` (auto-generate a default value when missing)
-9. `sync-publish` — create default `lerna.json` when missing and deploy root
-   `scripts.produck:publish:check` plus `scripts.produck:publish`
+   root `c8` devDependency (root-only; does not touch workspace packages)
+9. `sync-workspace` — deploy pinned `produck:coverage` script to each
+   workspace package and enforce `scripts.test` (auto-generate a default value
+   when missing). Workspace packages rely on root-level c8 devDependency
+   (hoisted by npm workspaces) and must not duplicate c8 in their own
+   `devDependencies`
+10. `sync-publish` — create default `lerna.json` when missing and deploy root
+    `scripts.produck:publish:check` plus `scripts.produck:publish`
 
 Add to downstream repository root `package.json` for one-command enforcement:
 

package/bin/command/enforce-node-baseline/help.txt

@@ -12,13 +12,14 @@ Behavior:
     5) sync-lint
     6) sync-install
     7) sync-git
-    8) sync-coverage
-    9) sync-publish
+    8) sync-coverage (root-level only)
+    9) sync-workspace (workspace packages, scripts only)
+   10) sync-publish
   - Stops at first failed step and exits non-zero
   - Prints one combined JSON report for all executed steps
 
 Rules:
-  - --check runs non-mutating validation mode for step 2, step 3, step 4, step 5, step 6, step 7, step 8, and step 9
-  - --dry-run runs non-mutating preview mode for step 2, step 3, step 4, step 5, step 6, step 7, step 8, and step 9
+  - --check runs non-mutating validation mode for step 2 through step 10
+  - --dry-run runs non-mutating preview mode for step 2 through step 10
   - --check takes precedence over --dry-run
-  - --workspace filters coverage sync targets in step 8
+  - --workspace filters workspace package targets in step 9

package/bin/command/sync-workspace/help.txt

@@ -3,20 +3,20 @@ Usage:
     [--workspace <path>] ... [--check] [--dry-run] [--json <file>]
 
 Behavior:
-  - Adds or updates scripts.produck:coverage and devDependencies.c8
-    in each workspace package.json.
+  - Adds or updates scripts.produck:coverage in each workspace
+    package.json.
   - Enforces scripts.test in each workspace package.json.
   - If scripts.test is missing, generates:
     node -e "console.log('No tests configured')"
-  - The produck:coverage script in workspaces is for local and AI
-    development use only.
+  - The produck:coverage script in workspaces relies on root-level
+    c8 devDependency (hoisted by npm workspaces).
   - Organization-reserved script key is scripts.produck:coverage
-  - c8 version is resolved from organization tooling baseline file
 
 Rules:
   - When --workspace is omitted, root package.json workspaces are used
   - Single-level glob patterns (e.g. packages/*) are expanded automatically
   - Recursive glob patterns (e.g. packages/**) are rejected
-  - Workspace package.json files must pin devDependencies.c8 to baseline version
+  - Workspace packages must not duplicate root devDependencies;
+    c8 belongs in root package.json only
   - --check validates without writing and exits non-zero on mismatch
   - --dry-run prints planned changes without writing

package/bin/command/sync-workspace/index.mjs

@@ -111,10 +111,6 @@ function buildRequiredCoverageScript(baseline) {
   return coverageTemplate.replace(/\{c8\.version\}/g, c8Version);
 }
 
-function buildRequiredC8DevDependency(baseline) {
-  return String(baseline.tools.c8.version);
-}
-
 function expandGlobPatterns(cwd, entries) {
   const result = [];
   for (const entry of entries) {
@@ -182,13 +178,7 @@ function resolveWorkspacePaths(cwd, options) {
   return raw;
 }
 
-function reconcileWorkspace(
-  cwd,
-  workspacePath,
-  mode,
-  requiredCoverageScript,
-  requiredC8Version,
-) {
+function reconcileWorkspace(cwd, workspacePath, mode, requiredCoverageScript) {
   const packageDir = path.resolve(cwd, workspacePath);
   const packageJsonPath = path.resolve(packageDir, 'package.json');
 
@@ -202,14 +192,10 @@ function reconcileWorkspace(
     coverageScript: null,
     previousTestScript: null,
     testScript: null,
-    previousC8DevDependency: null,
-    c8DevDependency: null,
     matchesRequiredCoverageBefore: false,
     matchesRequiredCoverageAfter: false,
     hasRequiredTestScriptBefore: false,
     hasRequiredTestScriptAfter: false,
-    matchesRequiredC8DevDependencyBefore: false,
-    matchesRequiredC8DevDependencyAfter: false,
     updated: false,
     error: '',
   };
@@ -235,12 +221,6 @@ function reconcileWorkspace(
     !Array.isArray(pkg.scripts)
       ? { ...pkg.scripts }
       : {};
-  const devDependencies =
-    pkg.devDependencies &&
-    typeof pkg.devDependencies === 'object' &&
-    !Array.isArray(pkg.devDependencies)
-      ? { ...pkg.devDependencies }
-      : {};
 
   const previousCoverage =
     typeof scripts[REQUIRED_COVERAGE_SCRIPT_KEY] === 'string'
@@ -251,30 +231,22 @@ function reconcileWorkspace(
     scripts[REQUIRED_TEST_SCRIPT_KEY].trim() !== ''
       ? scripts[REQUIRED_TEST_SCRIPT_KEY]
       : null;
-  const previousC8DevDependency =
-    typeof devDependencies.c8 === 'string' ? devDependencies.c8 : null;
   result.previousCoverage = previousCoverage;
   result.previousTestScript = previousTestScript;
-  result.previousC8DevDependency = previousC8DevDependency;
   result.matchesRequiredCoverageBefore =
     previousCoverage === requiredCoverageScript;
   result.hasRequiredTestScriptBefore = previousTestScript !== null;
-  result.matchesRequiredC8DevDependencyBefore =
-    previousC8DevDependency === requiredC8Version;
 
   if (
     (!result.matchesRequiredCoverageBefore ||
-      !result.hasRequiredTestScriptBefore ||
-      !result.matchesRequiredC8DevDependencyBefore) &&
+      !result.hasRequiredTestScriptBefore) &&
     mode === 'sync'
   ) {
     scripts[REQUIRED_COVERAGE_SCRIPT_KEY] = requiredCoverageScript;
     if (!result.hasRequiredTestScriptBefore) {
       scripts[REQUIRED_TEST_SCRIPT_KEY] = DEFAULT_TEST_SCRIPT_VALUE;
     }
-    devDependencies.c8 = requiredC8Version;
     pkg.scripts = scripts;
-    pkg.devDependencies = devDependencies;
     fs.writeFileSync(
       packageJsonPath,
       `${JSON.stringify(pkg, null, 2)}\n`,
@@ -291,18 +263,12 @@ function reconcileWorkspace(
     mode === 'sync' && !result.hasRequiredTestScriptBefore
       ? DEFAULT_TEST_SCRIPT_VALUE
       : previousTestScript;
-  result.c8DevDependency =
-    mode === 'sync' && !result.matchesRequiredC8DevDependencyBefore
-      ? requiredC8Version
-      : previousC8DevDependency;
 
   result.matchesRequiredCoverageAfter =
     result.updated || result.matchesRequiredCoverageBefore;
   result.hasRequiredTestScriptAfter =
     (mode === 'sync' && !result.hasRequiredTestScriptBefore) ||
     result.hasRequiredTestScriptBefore;
-  result.matchesRequiredC8DevDependencyAfter =
-    result.updated || result.matchesRequiredC8DevDependencyBefore;
   return result;
 }
 
@@ -314,7 +280,6 @@ export function runSyncWorkspace(options) {
   const { baseline: toolingBaseline, toolingBaselinePath } =
     loadToolingBaseline();
   const requiredCoverageScript = buildRequiredCoverageScript(toolingBaseline);
-  const requiredC8Version = buildRequiredC8DevDependency(toolingBaseline);
 
   if (!fs.existsSync(cwd)) {
     console.error(`CWD does not exist: ${cwd}`);
@@ -334,7 +299,6 @@ export function runSyncWorkspace(options) {
     },
     requiredCoverageScript,
     requiredTestScript: DEFAULT_TEST_SCRIPT_VALUE,
-    requiredC8DevDependency: requiredC8Version,
     workspaces: workspacePaths,
     results: [],
     ok: true,
@@ -347,7 +311,6 @@ export function runSyncWorkspace(options) {
       workspacePath,
       effectiveMode,
       requiredCoverageScript,
-      requiredC8Version,
     );
     report.results.push(item);
 
@@ -358,9 +321,7 @@ export function runSyncWorkspace(options) {
 
     if (
       mode === 'check' &&
-      (!item.matchesRequiredCoverageAfter ||
-        !item.hasRequiredTestScriptAfter ||
-        !item.matchesRequiredC8DevDependencyAfter)
+      (!item.matchesRequiredCoverageAfter || !item.hasRequiredTestScriptAfter)
     ) {
       report.ok = false;
     }

package/package.json

@@ -31,6 +31,6 @@
     "test": "node test/index.mjs"
   },
   "type": "module",
-  "version": "0.11.0",
-  "gitHead": "736364d5debd737ee92440818c9d0671db1c6974"
+  "version": "0.11.1",
+  "gitHead": "6ed9aad7d47f4483bd865c15543ed0ca4d712d26"
 }