@warpgogol/forge 2.9.2 → 2.9.5

package/os/rfc/handlers/validate.ts

@@ -142,10 +142,14 @@ export async function runRfcValidate(
       }
       const errorCount = violations.filter((v) => v.severity === "error").length;
       const warnCount = violations.filter((v) => v.severity === "warning").length;
+      const parts: string[] = [];
+      if (errorCount > 0) parts.push(`${errorCount} error${errorCount === 1 ? "" : "s"}`);
+      if (warnCount > 0) parts.push(`${warnCount} warning${warnCount === 1 ? "" : "s"}`);
+      const counts = parts.join(", ");
       if (errorCount > 0) {
-        logger.error(`${errorCount} error(s), ${warnCount} warning(s)`);
+        logger.error(counts);
       } else {
-        logger.warn(`${warnCount} warning(s), 0 errors — passed`);
+        logger.warn(`${counts} — passed`);
       }
     }
   }
@@ -160,7 +164,15 @@ export async function runRfcValidate(
     },
     exitCode: hasErrors ? 1 : 0,
     summary: hasErrors
-      ? `${violations.filter((v) => v.severity === "error").length} error(s) found`
+      ? `${violations.filter((v) => v.severity === "error").length} error${violations.filter((v) => v.severity === "error").length === 1 ? "" : "s"} found`
       : `All ${filesToValidate.length} RFC(s) passed validation`,
+    nextSteps: hasErrors
+      ? [
+          {
+            action: `Fix the ${violations.filter((v) => v.severity === "error").length} error${violations.filter((v) => v.severity === "error").length === 1 ? "" : "s"} above, then re-run: pnpm exec forge run rfc.validate`,
+            kind: "required",
+          },
+        ]
+      : undefined,
   };
 }

package/os/rfc/rfc-0000-template.md

@@ -8,10 +8,9 @@ kind: architecture
 scope: workspace
 owners:
   - architecture
-# Set by the deciding human together with the status change (RFC-0335).
+# Set by the deciding human together with the status change.
 # Draft scaffolds must keep this empty; do not prefill a default identity.
-# Format: human:<handle> (agent:<id> reserved — see RFC-0335).
-# Default reviewer when none is specified by the operator: human:andrii-syrokomskyi
+# Format: human:<handle>
 reviewers: []
 createdAt: YYYY-MM-DD
 updatedAt: YYYY-MM-DD
@@ -26,8 +25,6 @@ related:
   # - DNA-1
   # - AP-3
   # - RFC-0005
-  # - PAGE-MANDATORY-ARTIFACTS
-  # - COMPONENT-THREE-WAY-MIRROR
 # RFC-0331: DNA invariants this RFC implements, protects, or extends.
 # Required for architecture/contract RFCs created on or after 2026-07-07.
 # Entries must match ^DNA-\d+$ and exist in docs/architecture-dna.md.
@@ -55,17 +52,16 @@ appsImpacted: []
 packagesImpacted: []
 successSignals: []
 nonGoals: []
-# RFC-0268: OPTIONAL machine-checkable acceptance probes, executed on-demand
-# via `pnpm exec werkstatt run rfc.acceptance.run --id <this-rfc-id>` (never
-# automatically inside build pipelines). Closed probe vocabulary — see
-# docs/rfcs/rfc-0268-make-rfc-acceptance-criteria-machine-checkable.md.
+# OPTIONAL machine-checkable acceptance probes, executed on-demand
+# via `pnpm exec forge rfc.acceptance.run --id <this-rfc-id>` (never
+# automatically inside build pipelines).
 # acceptance:
 #   - probe: run
-#     command: "site-kernel run some.command.validate --app warpgogol-com"
+#     command: "pnpm exec forge some.command.validate"
 #     expect:
 #       exitCode: 0
 #   - probe: file-exists
-#     path: "packages/share/src/some-new-module.ts"
+#     path: "packages/my-package/src/some-new-module.ts"
 #   - probe: command-registered
 #     name: "some.new.command"
 #   - probe: file-contains
@@ -119,8 +115,8 @@ nonGoals: []
 <!-- Show the exact command(s) as a user would type them:
 
 ```sh
-pnpm exec werkstatt run domain.command --app main
-pnpm exec werkstatt run domain.command --all --json
+pnpm exec forge domain.command
+pnpm exec forge domain.command --json
 ```
 
 Describe flags, arguments, and scope (app | workspace).
@@ -150,7 +146,7 @@ interface ExampleResult {
 
 | Path | Role |
 |---|---|
-| `src/pages/[lang]/**/*.astro` | Scanned for violations |
+| `src/**/*.ts` | Scanned for violations |
 | `docs/rfcs/index.json` | Updated by rfc.index.generate |
 -->
 
@@ -163,7 +159,7 @@ interface ExampleResult {
   "command": "domain.command",
   "status": "fail",
   "violations": [
-    { "file": "src/pages/de/legal.astro", "rule": "missing-schema", "message": "..." }
+    { "file": "src/some-file.ts", "rule": "missing-schema", "message": "..." }
   ]
 }
 ```
@@ -216,12 +212,12 @@ interface ExampleResult {
 
 - Agents MAY implement code changes ONLY when this RFC has status: accepted (or implemented).
 - Agents MAY transition this RFC from `accepted` to `implemented` per RFC-0224 preconditions; reference this RFC ID in commits.
-- For RFCs created on or after 2026-07-07 with acceptance probes: before stamping `implemented`, run
-  `site-kernel run rfc.verification.emit --id <this-rfc-id>` and commit the evidence file
-  in the same commit (RFC-0330 amended transition precondition).
+- For RFCs with acceptance probes: before stamping `implemented`, run
+  `pnpm exec forge rfc.verification.emit --id <this-rfc-id>` and commit the evidence file
+  in the same commit.
 - Agents MUST NOT weaken or remove enforcement rules established by this RFC
   without a new RFC that supersedes it.
 - If implementation reveals an invariant conflict, run
-  `site-kernel run rfc.supersede.propose --id <this-rfc-id> --reason "..." --invariant "DNA-N"`
-  instead of working around it (RFC-0334).
+  `pnpm exec forge rfc.supersede.propose --id <this-rfc-id> --reason "..." --invariant "DNA-N"`
+  instead of working around it.
 -->

package/os/session/handlers/validate.ts

@@ -227,10 +227,14 @@ export async function runSessionValidate(
       }
       const errorCount = violations.filter((v) => v.severity === "error").length;
       const warnCount = violations.filter((v) => v.severity === "warning").length;
+      const parts: string[] = [];
+      if (errorCount > 0) parts.push(`${errorCount} error${errorCount === 1 ? "" : "s"}`);
+      if (warnCount > 0) parts.push(`${warnCount} warning${warnCount === 1 ? "" : "s"}`);
+      const counts = parts.join(", ");
       if (errorCount > 0) {
-        logger.error(`${errorCount} error(s), ${warnCount} warning(s)`);
+        logger.error(counts);
       } else {
-        logger.warn(`${warnCount} warning(s), 0 errors — passed`);
+        logger.warn(`${counts} — passed`);
       }
     }
   }
@@ -244,7 +248,15 @@ export async function runSessionValidate(
     },
     exitCode: hasErrors ? 1 : 0,
     summary: hasErrors
-      ? `${violations.filter((v) => v.severity === "error").length} error(s) found`
+      ? `${violations.filter((v) => v.severity === "error").length} error${violations.filter((v) => v.severity === "error").length === 1 ? "" : "s"} found`
       : `All ${filesToValidate.length} session file(s) passed validation`,
+    nextSteps: hasErrors
+      ? [
+          {
+            action: `Fix the ${violations.filter((v) => v.severity === "error").length} error${violations.filter((v) => v.severity === "error").length === 1 ? "" : "s"} above, then re-run: pnpm exec forge run session.validate`,
+            kind: "required",
+          },
+        ]
+      : undefined,
   };
 }

package/os/spec/live-spec-list.ts

@@ -74,5 +74,13 @@ export async function runSpecLiveList(
     data: result,
     exitCode: 0,
     summary: `spec.live.list: ${entries.length} living spec(s)`,
+    nextSteps: entries.length > 0
+      ? [
+          {
+            action: `Validate living specs: pnpm exec forge run spec.live.validate`,
+            kind: "optional",
+          },
+        ]
+      : undefined,
   };
 }

package/os/spec/live-spec-merge.ts

@@ -323,7 +323,7 @@ export async function runSpecLiveMerge(
       );
     }
 
-    return { data: result, exitCode: 0, summary: `spec.live.merge: created ${domain} from ${rfcId}` };
+    return { data: result, exitCode: 0, summary: `spec.live.merge: created ${domain} from ${rfcId}`, nextSteps: [{ action: `Validate the living spec: pnpm exec forge run spec.live.validate`, kind: "optional" }] };
   }
 
   for (const heading of rfcHeadings) {
@@ -409,5 +409,5 @@ export async function runSpecLiveMerge(
     );
   }
 
-  return { data: result, exitCode: 0, summary: `spec.live.merge: modified ${domain} from ${rfcId}` };
+  return { data: result, exitCode: 0, summary: `spec.live.merge: modified ${domain} from ${rfcId}`, nextSteps: [{ action: `Validate the living spec: pnpm exec forge run spec.live.validate`, kind: "optional" }] };
 }

package/os/spec/live-spec-show.ts

@@ -113,5 +113,11 @@ export async function runSpecLiveShow(
     data: result,
     exitCode: 0,
     summary: `spec.live.show: ${domain}`,
+    nextSteps: [
+      {
+        action: `Validate the living spec: pnpm exec forge run spec.live.validate`,
+        kind: "optional",
+      },
+    ],
   };
 }

package/os/spec/live-spec-validate.ts

@@ -151,7 +151,7 @@ export async function runSpecLiveValidate(
 
   if (outputFormat === "pretty") {
     if (hasFailures) {
-      logger.error(`spec.live.validate: ${violations.length} violation(s) across ${specsChecked} spec(s)`);
+      logger.error(`spec.live.validate: ${violations.length} violation${violations.length === 1 ? "" : "s"} across ${specsChecked} spec(s)`);
       for (const v of violations) {
         logger.error(`  ${v.rule}: ${v.message}`);
       }
@@ -164,7 +164,7 @@ export async function runSpecLiveValidate(
     data: result,
     exitCode: hasFailures ? 1 : 0,
     summary: hasFailures
-      ? `spec.live.validate: ${violations.length} violation(s)`
+      ? `spec.live.validate: ${violations.length} violation${violations.length === 1 ? "" : "s"}`
       : `spec.live.validate: ${specsChecked} spec(s) — pass`,
   };
 }

package/os/spec/spec-materialize.ts

@@ -386,5 +386,11 @@ export async function runSpecMaterialize(
     },
     exitCode: 0,
     summary: `spec.materialize: created ${created.length} RFC(s) from spec '${specId}'`,
+    nextSteps: [
+      {
+        action: `Validate the created RFCs: pnpm exec forge run rfc.validate`,
+        kind: "optional",
+      },
+    ],
   };
 }

package/os/spec/spec-status.ts

@@ -280,5 +280,11 @@ export async function runSpecStatus(
     },
     exitCode: 0,
     summary: `spec.status: ${results.length} spec(s)`,
+    nextSteps: [
+      {
+        action: `Validate specs: pnpm exec forge run spec.validate`,
+        kind: "optional",
+      },
+    ],
   };
 }

package/os/spec/spec-validate.ts

@@ -490,7 +490,7 @@ export async function runSpecValidate(
         if (result.violations.length === 0) {
           logger.success(`spec.validate: ${result.id} — pass`);
         } else {
-          logger.error(`spec.validate: ${result.id} — ${result.violations.length} violation(s)`);
+          logger.error(`spec.validate: ${result.id} — ${result.violations.length} violation${result.violations.length === 1 ? "" : "s"}`);
           for (const v of result.violations) {
             logger.error(`  ${v.rule}: ${v.message}`);
           }

package/os/werkstatt/handlers/werkstatt-lock-recover.ts

@@ -168,6 +168,15 @@ export async function runWerkstattLockRecover(
     return {
       data: { recovered, failed },
       summary: `[werkstatt.lock.recover] ${recovered.length} artifact${recovered.length === 1 ? "" : "s"} recovered, ${failed.length} failure${failed.length === 1 ? "" : "s"}`,
+      nextSteps:
+        failed.length > 0
+          ? [
+              {
+                action: `Investigate the ${failed.length} failed recovery item(s) above and retry manually`,
+                kind: "required",
+              },
+            ]
+          : undefined,
     };
   } finally {
     await releaseLock(workspaceRoot, "werkstatt-recovery");

package/os/werkstatt/handlers/werkstatt-lock-status.ts

@@ -69,5 +69,14 @@ export async function runWerkstattLockStatus(
       staleCount,
     },
     summary: `[werkstatt.lock.status] ${locks.length} lock${locks.length === 1 ? "" : "s"}, ${staleCount} stale`,
+    nextSteps:
+      staleCount > 0
+        ? [
+            {
+              action: `Recover stale locks: pnpm exec forge run werkstatt.lock.recover`,
+              kind: "optional",
+            },
+          ]
+        : undefined,
   };
 }

package/os/workflow/handlers.ts

@@ -203,6 +203,15 @@ export async function runWorkflowList(
     data: { command: "workflow.list", workflows: entries },
     exitCode: 0,
     summary: `workflow.list: ${entries.length} workflow file(s) found`,
+    nextSteps:
+      entries.length === 0
+        ? [
+            {
+              action: `Create a workflow file in .devin/workflows/ or .windsurf/workflows/`,
+              kind: "optional",
+            },
+          ]
+        : undefined,
   };
 }
 
@@ -298,7 +307,7 @@ export async function runWorkflowLint(
     exitCode: violations.length > 0 ? 1 : 0,
     summary:
       violations.length > 0
-        ? `workflow.lint: ${violations.length} violation(s)`
+        ? `workflow.lint: ${violations.length} violation${violations.length === 1 ? "" : "s"}`
         : `workflow.lint: ${workflows.length} workflow file(s) valid`,
   };
 }