@warpgogol/forge 3.0.0 → 4.1.1

package/os/core/handlers/package-health.ts

@@ -8,6 +8,7 @@
 </MODULE_CONTRACT>
 <CHANGE_SUMMARY>
   <item>Initial: forge.package.health validator — engines, CI workflow, extract config, devDeps completeness.</item>
+  <item>PKG-HEALTH-05: reject versionBump: major in extract.config.yaml — prevents double-bump major version inflation.</item>
 </CHANGE_SUMMARY>
 */
 
@@ -156,6 +157,26 @@ export async function runPackageHealth(
         file: pkgDir,
         fixHint: "Create extract.config.yaml with git remote and extraction settings.",
       });
+    } else {
+      // CHECK 3b: versionBump must not be major (prevent accidental major version inflation)
+      try {
+        const configRaw = fs.readFileSync(extractConfigPath, "utf8");
+        const versionBumpMatch = configRaw.match(/^versionBump:\s*(\S+)/m);
+        if (versionBumpMatch && versionBumpMatch[1] === "major") {
+          violations.push({
+            ruleId: "PKG-HEALTH-05",
+            packageName,
+            severity: "error",
+            message:
+              "extract.config.yaml has versionBump: major — this causes double-bump when the monorepo version was already bumped manually. Use versionBump: patch instead.",
+            file: extractConfigPath,
+            fixHint:
+              "Change versionBump to patch (or minor). Manually bump the monorepo package.json version to the target instead.",
+          });
+        }
+      } catch {
+        // Read error — skip this check
+      }
     }
 
     // CHECK 4: devDependencies completeness — tools used in scripts must be declared

package/os/core/index.ts

@@ -10,4 +10,4 @@
 </CHANGE_SUMMARY>
 */
 
-export { forgeCoreModule } from "./core.module.ts";
+export { createForgeCoreModule } from "./core.module.ts";

package/os/exploration/exploration.module.ts

@@ -12,16 +12,17 @@
 
 import type { ForgeModule } from "../../src/forge-module.ts";
 
-export const forgeExplorationModule: ForgeModule = {
+export async function createForgeExplorationModule(): Promise<ForgeModule> {
+const { runExplorationList } = await import("./handlers/list.ts");
+    const { runExplorationShow } = await import("./handlers/show.ts");
+    const { runExplorationArchive } = await import("./handlers/archive.ts");
+  return {
   name: "forge-exploration",
   version: "0.1.0",
   runtime: "autonomous",
-  async register(registry) {
-    const { runExplorationList } = await import("./handlers/list.ts");
-    const { runExplorationShow } = await import("./handlers/show.ts");
-    const { runExplorationArchive } = await import("./handlers/archive.ts");
-
-    registry.registerCommand({
+    declarations: [],
+  commands: [
+    {
       name: "exploration.list",
       description:
         "List all exploration notes in docs/explorations/. Returns id, title, status, and createdAt for each note. " +
@@ -39,9 +40,8 @@ export const forgeExplorationModule: ForgeModule = {
         },
       },
       execute: runExplorationList,
-    });
-
-    registry.registerCommand({
+    },
+    {
       name: "exploration.show",
       description:
         "Show the full content of a single exploration note. Use --id <slug> to specify the note slug " +
@@ -60,9 +60,8 @@ export const forgeExplorationModule: ForgeModule = {
         },
       },
       execute: runExplorationShow,
-    });
-
-    registry.registerCommand({
+    },
+    {
       name: "exploration.archive",
       description:
         "Archive an exploration note by setting its status to 'archived'. Use --id <slug> to specify the note. " +
@@ -86,6 +85,10 @@ export const forgeExplorationModule: ForgeModule = {
         },
       },
       execute: runExplorationArchive,
-    });
-  },
-};
+    }
+  ],
+  pipelines: [
+
+  ]};
+}
+;

package/os/exploration/index.ts

@@ -6,11 +6,11 @@
 </non-goals>
 </MODULE_CONTRACT>
 <CHANGE_SUMMARY>
-  <item>RFC-0710: expose forgeExplorationModule and exploration types from the exploration domain.</item>
+  <item>RFC-0710: expose createForgeExplorationModule and exploration types from the exploration domain.</item>
 </CHANGE_SUMMARY>
 */
 
-export { forgeExplorationModule } from "./exploration.module.ts";
+export { createForgeExplorationModule } from "./exploration.module.ts";
 export { runExplorationList } from "./handlers/list.ts";
 export { runExplorationShow } from "./handlers/show.ts";
 export { runExplorationArchive } from "./handlers/archive.ts";

package/os/mission/index.ts

@@ -1 +1 @@
-export { forgeMissionModule } from "./mission.module.ts";
+export { createForgeMissionModule } from "./mission.module.ts";

package/os/mission/mission.module.ts

@@ -13,14 +13,15 @@
 
 import type { ForgeModule } from "../../src/forge-module.ts";
 
-export const forgeMissionModule: ForgeModule = {
+export async function createForgeMissionModule(): Promise<ForgeModule> {
+const { runMissionArchive } = await import("./handlers/archive.ts");
+  return {
   name: "forge-mission",
   version: "0.1.0",
   runtime: "autonomous",
-  async register(registry) {
-    const { runMissionArchive } = await import("./handlers/archive.ts");
-
-    registry.registerCommand({
+    declarations: [],
+  commands: [
+    {
       name: "mission.archive",
       description:
         "Move terminal-state mission directories (state: closed or aborted in " +
@@ -50,6 +51,10 @@ export const forgeMissionModule: ForgeModule = {
         },
       },
       execute: runMissionArchive,
-    });
-  },
-};
+    }
+  ],
+  pipelines: [
+
+  ]};
+}
+;

package/os/naming/index.ts

@@ -10,5 +10,5 @@
 </CHANGE_SUMMARY>
 */
 
-export { forgeNamingModule } from "./naming.module.ts";
+export { createForgeNamingModule } from "./naming.module.ts";
 export { runNamingConventionLint } from "./naming-convention.ts";

package/os/naming/naming-convention.ts

@@ -102,6 +102,7 @@ const NAMING_CONVENTION_EXEMPT_DIRS = new Set([
   "docs/performance", // Tool-generated screenshots and Lighthouse reports with timestamps
   "docs/specs", // Imported specification documents with their own naming convention
   "docs/rfcs/archive", // Archived RFCs — historical artifacts, cannot be renamed
+  "docs/metrics", // RFC-1053: generated metrics files with RFC/session IDs in filenames
 ]);
 
 /**

package/os/naming/naming.module.ts

@@ -12,13 +12,15 @@
 
 import type { ForgeModule } from "../../src/forge-module.ts";
 
-export const forgeNamingModule: ForgeModule = {
+export async function createForgeNamingModule(): Promise<ForgeModule> {
+const { runNamingConventionLint } = await import("./naming-convention.ts");
+  return {
   name: "forge-naming",
   version: "0.1.0",
   runtime: "autonomous",
-  async register(registry) {
-    const { runNamingConventionLint } = await import("./naming-convention.ts");
-    registry.registerCommand({
+    declarations: [],
+  commands: [
+    {
       name: "naming.convention.lint",
       contract: "naming",
       rules: [],
@@ -34,6 +36,10 @@ export const forgeNamingModule: ForgeModule = {
       },
       reads: ["packages/**/*.{ts,tsx}", "apps/**/*.{ts,tsx}", "services/**/*.{ts,tsx}"],
       execute: runNamingConventionLint,
-    });
-  },
-};
+    }
+  ],
+  pipelines: [
+
+  ]};
+}
+;

package/os/notes/index.ts

@@ -1,10 +1,10 @@
 /*
 <MODULE_CONTRACT>
-<purpose>Re-export forgeNotesModule and note validator handlers.</purpose>
+<purpose>Re-export createForgeNotesModule and note validator handlers.</purpose>
 </MODULE_CONTRACT>
 <CHANGE_SUMMARY>
   <item>RFC-0808: initial notes module index.</item>
 </CHANGE_SUMMARY>
 */
 
-export { forgeNotesModule } from "./notes.module.ts";
+export { createForgeNotesModule } from "./notes.module.ts";

package/os/notes/notes.module.ts

@@ -17,12 +17,8 @@ import type {
   ForgeRuntimeContext,
 } from "../../src/types.ts";
 
-export const forgeNotesModule: ForgeModule = {
-  name: "forge-notes",
-  version: "0.1.0",
-  runtime: "autonomous",
-  async register(registry) {
-    const { runNoteLinkValidate } = await import("../../src/validators/note-link-validate.ts");
+export async function createForgeNotesModule(): Promise<ForgeModule> {
+const { runNoteLinkValidate } = await import("../../src/validators/note-link-validate.ts");
     const { runNoteFrontmatterValidate } =
       await import("../../src/validators/note-frontmatter-validate.ts");
     const { runNoteOrphanDetect } = await import("../../src/validators/note-orphan-detect.ts");
@@ -47,8 +43,13 @@ export const forgeNotesModule: ForgeModule = {
     ): Promise<ForgeCommandResult> => {
       return runNoteOrphanDetect(input, context);
     };
-
-    registry.registerCommand({
+  return {
+  name: "forge-notes",
+  version: "0.1.0",
+  runtime: "autonomous",
+    declarations: [],
+  commands: [
+    {
       name: "note.link.validate",
       contract: "note",
       rules: [],
@@ -69,9 +70,8 @@ export const forgeNotesModule: ForgeModule = {
       reads: ["vault/**/*.md"],
       cacheable: false,
       execute: noteLinkValidateWrapper,
-    });
-
-    registry.registerCommand({
+    },
+    {
       name: "note.frontmatter.validate",
       contract: "note",
       rules: [],
@@ -92,9 +92,8 @@ export const forgeNotesModule: ForgeModule = {
       reads: ["vault/**/*.md"],
       cacheable: false,
       execute: noteFrontmatterValidateWrapper,
-    });
-
-    registry.registerCommand({
+    },
+    {
       name: "note.orphan.detect",
       description:
         "Detect orphan notes in a markdown note vault — notes with zero inbound wikilinks. Always exits zero (warnings, not errors).",
@@ -109,6 +108,10 @@ export const forgeNotesModule: ForgeModule = {
       reads: ["vault/**/*.md"],
       cacheable: false,
       execute: noteOrphanDetectWrapper,
-    });
-  },
-};
+    }
+  ],
+  pipelines: [
+
+  ]};
+}
+;

package/os/plan/index.ts

@@ -10,7 +10,7 @@
 </CHANGE_SUMMARY>
 */
 
-export { forgePlanModule } from "./plan.module.ts";
+export { createForgePlanModule } from "./plan.module.ts";
 export { runPlanArchive } from "./handlers/archive.ts";
 export {
   listPlanFiles,

package/os/plan/plan.module.ts

@@ -12,14 +12,15 @@
 
 import type { ForgeModule } from "../../src/forge-module.ts";
 
-export const forgePlanModule: ForgeModule = {
+export async function createForgePlanModule(): Promise<ForgeModule> {
+const { runPlanArchive } = await import("./handlers/archive.ts");
+  return {
   name: "forge-plan",
   version: "0.1.0",
   runtime: "autonomous",
-  async register(registry) {
-    const { runPlanArchive } = await import("./handlers/archive.ts");
-
-    registry.registerCommand({
+    declarations: [],
+  commands: [
+    {
       name: "plan.archive",
       description:
         "Move plan files whose parent RFC has terminal status " +
@@ -44,6 +45,10 @@ export const forgePlanModule: ForgeModule = {
         },
       },
       execute: runPlanArchive,
-    });
-  },
-};
+    }
+  ],
+  pipelines: [
+
+  ]};
+}
+;

package/os/plugin/plugin.module.ts

@@ -233,8 +233,9 @@ export const forgePluginModule: ForgeModule = {
   name: "forge-plugin",
   version: "0.1.0",
   runtime: "autonomous",
-  async register(registry) {
-    registry.registerCommand({
+    declarations: [],
+  commands: [
+    {
       name: "forge.plugin.validate",
       contract: "forge",
       rules: [],
@@ -246,9 +247,8 @@ export const forgePluginModule: ForgeModule = {
       reads: ["forge.yaml", "**/forge.plugin.yaml"],
       cacheable: false,
       execute: runPluginValidate,
-    });
-
-    registry.registerCommand({
+    },
+    {
       name: "forge.plugin.discover",
       description:
         "Enumerate all project-declared skill packs with valid forge.plugin.yaml manifests (RFC-0941). Returns pack id, version, prefix, and directory.",
@@ -258,6 +258,8 @@ export const forgePluginModule: ForgeModule = {
       reads: ["forge.yaml", "**/forge.plugin.yaml"],
       cacheable: false,
       execute: runPluginDiscover,
-    });
-  },
-};
+    }
+  ],
+  pipelines: [
+
+  ]};

package/os/program/program.module.ts

@@ -13,18 +13,19 @@ kernel registry (RFC-0856).</purpose>
 
 import type { ForgeModule } from "../../src/forge-module.ts";
 
-export const forgeProgramModule: ForgeModule = {
-  name: "forge-program",
-  version: "0.1.0",
-  runtime: "autonomous",
-
-  async register(registry) {
-    const { runValidate } = await import("./handlers/validate.ts");
+export async function createForgeProgramModule(): Promise<ForgeModule> {
+const { runValidate } = await import("./handlers/validate.ts");
     const { runSeal } = await import("./handlers/seal.ts");
     const { runLease } = await import("./handlers/lease.ts");
     const { runComplete } = await import("./handlers/complete.ts");
+  return {
+  name: "forge-program",
+  version: "0.1.0",
+  runtime: "autonomous",
 
-    registry.registerCommand({
+    declarations: [],
+  commands: [
+    {
       name: "program.packet.validate",
       contract: "program",
       rules: [],
@@ -57,9 +58,8 @@ export const forgeProgramModule: ForgeModule = {
       reads: ["docs/plans/**/program.yaml", "docs/plans/**/*.md"],
       cacheable: false,
       execute: runValidate,
-    });
-
-    registry.registerCommand({
+    },
+    {
       name: "program.packet.seal",
       description:
         "Steward finalizes a packet against the predecessor's completion commit, " +
@@ -99,9 +99,8 @@ export const forgeProgramModule: ForgeModule = {
       reads: ["docs/plans/**/program.yaml", "docs/plans/**/*.md"],
       cacheable: false,
       execute: runSeal,
-    });
-
-    registry.registerCommand({
+    },
+    {
       name: "program.packet.lease",
       description:
         "Manage the exclusive local executor lease for a sealed packet. " +
@@ -152,9 +151,8 @@ export const forgeProgramModule: ForgeModule = {
       reads: ["docs/plans/**/program.yaml", "docs/plans/**/*.md"],
       cacheable: false,
       execute: runLease,
-    });
-
-    registry.registerCommand({
+    },
+    {
       name: "program.packet.complete",
       description:
         "Steward validates the implementation range, writes the completion report, " +
@@ -210,6 +208,10 @@ export const forgeProgramModule: ForgeModule = {
       reads: ["docs/plans/**/program.yaml", "docs/plans/**/*.md"],
       cacheable: false,
       execute: runComplete,
-    });
-  },
-};
+    }
+  ],
+  pipelines: [
+
+  ]};
+}
+;

package/os/rfc/handlers/implement-stamp.ts

@@ -12,10 +12,12 @@ verification evidence, and atomically mutates RFC frontmatter.
 </non-goals>
 </MODULE_CONTRACT>
 <CHANGE_SUMMARY>
-  <item>RFC-0476: initial implementation.</item>
+  <item>RFC-0476: initial rfc.implement.stamp handler with acceptance criteria evaluation, evidence checks, and atomic status transition.</item>
+  <item>RFC-0268: acceptance probe evidence check before stamping.</item>
   <item>RFC-0756: auto-detect implementation commit when --implementation-commit is omitted.</item>
   <item>RFC-0795: add RFC-IMP-07 dependsOn dependency gate — blocks stamping when any dependsOn entry is not implemented.</item>
   <item>RFC-0997: add RFC-IMP-08 minimum-one-probe gate — blocks stamping for post-cutoff architecture/contract/command RFCs with no acceptance probes.</item>
+  <item>RFC-1053: integrate generateRfcMetrics after successful stamp (non-fatal, guarded by !isDryRun).</item>
 </CHANGE_SUMMARY>
 */
 
@@ -25,6 +27,7 @@ import { join, dirname } from "node:path";
 
 import { writeFileAtomic } from "../../../src/utils/fs-atomic.ts";
 import { parse as yamlParse } from "yaml";
+import { generateRfcMetrics } from "../../session/handlers/metrics-rfc.ts";
 
 import {
   listRfcFiles,
@@ -466,6 +469,18 @@ export async function runRfcImplementStamp(
     // Write the mutated RFC file atomically
     await writeFileAtomic(rfcFilePath, mutatedSource);
 
+    // ── RFC-1053: Generate per-RFC metrics (non-fatal, guarded by !isDryRun) ─
+    let metricsPath: string | undefined;
+    if (!isDryRun) {
+      try {
+        metricsPath = await generateRfcMetrics(workspaceRoot, targetId, logger);
+      } catch (metricsErr) {
+        logger.warn(
+          `[metrics] Failed to generate metrics for ${targetId}: ${(metricsErr as Error).message}`,
+        );
+      }
+    }
+
     // ── Re-emit evidence if probes exist (to update with new status) ─────────
     if (hasProbes && requiresEvidence && evidenceRelPath) {
       // Evidence already exists and passed — no need to re-emit on stamp
@@ -493,6 +508,7 @@ export async function runRfcImplementStamp(
           stampedAt,
           criteriaChecked: criteriaEval.totalChecked,
           ...(evidenceRelPath ? { evidencePath: evidenceRelPath } : {}),
+          ...(metricsPath ? { metricsPath } : {}),
         },
         violations: [],
       },

package/os/rfc/index.ts

@@ -10,7 +10,7 @@
 </CHANGE_SUMMARY>
 */
 
-export { forgeRfcModule } from "./rfc.module.ts";
+export { createForgeRfcModule } from "./rfc.module.ts";
 export {
   listRfcFiles,
   parseRfcFile,

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

@@ -218,8 +218,9 @@ interface ExampleResult {
 
 - [ ] AC-1: WHEN `<command> --json` is invoked, THE command SHALL return a JSON object matching the documented output schema (evidence: probe:AC-1 or test: <path>)
 - [ ] AC-2: THE `<command>` SHALL be registered in the kernel module with the correct name and scope (evidence: file: <module-path>)
-- [ ] AC-3: IF `<command>` receives invalid input, THEN THE command SHALL report a blocking error and exit non-zero (evidence: test: <path/to/test>)
-- [ ] AC-4: THE relevant `AGENTS.md` SHALL reference this RFC where agent behavior rules changed (evidence: file: <path:line>)
+- [ ] AC-3: IF `<command>` receives invalid input, THEN THE command SHALL report a blocking error (evidence: test: <path/to/test>)
+- [ ] AC-4: IF `<command>` receives invalid input, THEN THE command SHALL exit non-zero (evidence: test: <path/to/test>)
+- [ ] AC-5: THE relevant `AGENTS.md` SHALL reference this RFC where agent behavior rules changed (evidence: file: <path:line>)
 
 ## Implementation notes for agents