@izkac/forgekit 0.1.6 → 0.1.7

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@izkac/forgekit",
-  "version": "0.1.6",
+  "version": "0.1.7",
   "description": "Forgekit CLIs — forgekit (install), forge (workflow), review (code review)",
   "type": "module",
   "bin": {

package/src/init.mjs

@@ -100,7 +100,8 @@ Options:
   --no-adr          Disable ADRs for this project
   --adr-dir <path>  ADR directory (default: ${DEFAULT_ADR_DIR} or ~/.forgekit preference)
   --overlay         Also run \`forge overlay\` (OpenSpec vendor patches)
-  --force, -f       Overwrite existing template files
+  --force, -f       Force re-scaffold of ADR/specs docs (managed command,
+                    rule, and hook files always refresh to the latest template)
   --cwd <path>      Project root (default: cwd)
   --help
 
@@ -124,16 +125,22 @@ export function resolveTemplatesRoot() {
 }
 
 /**
+ * Copy a forgekit-managed template file. These are regenerated pointers
+ * (forge-* commands/rules/hooks) with no user-owned content, so re-running
+ * `forge init` refreshes them in place — that's how template fixes propagate.
  * @param {string} src
  * @param {string} dest
- * @param {{ force?: boolean }} opts
+ * @param {{ force?: boolean }} _opts
  */
-function copyFile(src, dest, opts) {
+function copyFile(src, dest, _opts) {
   fs.mkdirSync(path.dirname(dest), { recursive: true });
-  if (fs.existsSync(dest) && !opts.force) {
-    return 'skipped';
+  const next = fs.readFileSync(src);
+  if (fs.existsSync(dest)) {
+    if (fs.readFileSync(dest).equals(next)) return 'unchanged';
+    fs.writeFileSync(dest, next);
+    return 'updated';
   }
-  fs.copyFileSync(src, dest);
+  fs.writeFileSync(dest, next);
   return 'written';
 }
 

package/src/init.test.mjs

@@ -3,7 +3,7 @@ import assert from 'node:assert/strict';
 import fs from 'node:fs';
 import os from 'node:os';
 import path from 'node:path';
-import { parseArgs, initProject, rememberedAgents } from './init.mjs';
+import { parseArgs, initProject, rememberedAgents, resolveTemplatesRoot } from './init.mjs';
 import { installSkillsToAgents } from './install.mjs';
 import { saveUserConfig } from './config.mjs';
 
@@ -35,6 +35,37 @@ test('rememberedAgents unions install config, installed skills, and project wiri
   }
 });
 
+test('re-running init refreshes stale managed rule files in place', () => {
+  const cwd = fs.mkdtempSync(path.join(os.tmpdir(), 'forgekit-refresh-'));
+  try {
+    initProject(['claude'], { cwd, adr: false, planEngine: null });
+    const rule = path.join(cwd, '.claude', 'rules', 'forge.md');
+    // Simulate an older install with a stale reference.
+    fs.writeFileSync(rule, 'Full workflow: forgekit `docs/forge.md`\n', 'utf8');
+
+    const report = initProject(['claude'], { cwd, adr: false, planEngine: null });
+    const updated = fs.readFileSync(rule, 'utf8');
+    assert.ok(!updated.includes('forgekit `docs/forge.md`'), 'stale ref replaced');
+    assert.ok(updated.includes('~/.claude/skills/forge/docs/forge.md'), 'points to global skill doc');
+    assert.ok(
+      report.files.some((f) => f.file.includes('forge.md') && f.status === 'updated'),
+      'reports the refresh as updated',
+    );
+  } finally {
+    fs.rmSync(cwd, { recursive: true, force: true });
+  }
+});
+
+test('thin-rule templates are engine-neutral (no hardcoded OpenSpec-only flow)', () => {
+  const root = resolveTemplatesRoot();
+  for (const rel of ['claude/rules/forge.md', 'cursor/rules/forge.mdc', 'codex/rules/forge.md']) {
+    const body = fs.readFileSync(path.join(root, ...rel.split('/')), 'utf8');
+    assert.ok(!/Forge = OpenSpec/.test(body), `${rel} still says "Forge = OpenSpec"`);
+    assert.ok(body.includes('forge change new'), `${rel} missing built-in specs command`);
+    assert.ok(body.includes('/opsx:propose'), `${rel} missing OpenSpec command`);
+  }
+});
+
 test('initProject wires templated envs and marks the rest skill-only', () => {
   const cwd = fs.mkdtempSync(path.join(os.tmpdir(), 'forgekit-init-'));
   try {

package/vendor/templates/project/claude/commands/forge.md

@@ -2,7 +2,7 @@
 name: /forge
 description: Forge — start or resume disciplined development (brainstorm → plan → build → review)
 category: Workflow
-tags: [workflow, forge, openspec]
+tags: [workflow, forge, planning]
 ---
 
 Run the **Forge** workflow for substantial work.

package/vendor/templates/project/claude/rules/forge.md

@@ -1,16 +1,20 @@
-# Forge (thin rule)
-
-Full workflow: `~/.claude/skills/forge/docs/forge.md` · Skill: user-installed **Forge** (`~/.claude/skills/forge/SKILL.md` after `forge install`).
-
-**Default:** triage before implementation. Substantial work → **Forge**
-(brainstorm → OpenSpec → subagent TDD implement → review).
-
-**Forge = OpenSpec:** after brainstorm, proceed directly to `/opsx:propose` — do not ask for a plan mode. Work too small for OpenSpec should skip Forge (`/forge:skip` or direct execution).
-
-**Skip Forge** only when work is trivial OR user sends **`/forge:skip`**.
-
-Scratch sessions: `.forge/sessions/` (14-day retention). Active pointer: `.forge/active.json`.
-
-CLI: `forge new`, `forge status`, `forge prefs`, `forge models`, `forge phase`, `forge doctor`.
-
-Do not edit vendor OpenSpec skills — Forge orchestrates them. Workflow skills are bundled under the Forge skill’s `skills/` folder.
+# Forge (thin rule)
+
+Full workflow: `~/.claude/skills/forge/docs/forge.md` · Skill: user-installed **Forge** (`~/.claude/skills/forge/SKILL.md` after `forge install`).
+
+**Default:** triage before implementation. Substantial work → **Forge**
+(brainstorm → plan → subagent TDD implement → review).
+
+**Planning engine:** this project's engine is recorded in `.forge/config.json` (`plan.engine`). After brainstorm, create the change spec directly — do not ask for a plan mode:
+- OpenSpec → `/opsx:propose`
+- built-in specs → `forge change new <slug>`
+
+Work too small to spec should skip Forge (`/forge:skip` or direct execution).
+
+**Skip Forge** only when work is trivial OR user sends **`/forge:skip`**.
+
+Scratch sessions: `.forge/sessions/` (14-day retention). Active pointer: `.forge/active.json`.
+
+CLI: `forge new`, `forge status`, `forge prefs`, `forge models`, `forge phase`, `forge doctor`.
+
+Do not edit vendor planning-engine skills — Forge orchestrates them. Workflow skills are bundled under the Forge skill's `skills/` folder.

package/vendor/templates/project/codex/rules/forge.md

@@ -1,10 +1,12 @@
-# Forge (thin rule)
-
-Full workflow: `~/.codex/skills/forge/docs/forge.md` · Skill: user-installed **Forge** (`~/.codex/skills/forge/SKILL.md` after `forge install`).
-
-**Default:** triage before implementation. Substantial work → **Forge**
-(brainstorm → OpenSpec → subagent TDD implement → review).
-
-**Forge = OpenSpec:** after brainstorm, proceed directly to OpenSpec propose. Skip with `/forge:skip` or when work is trivial.
-
-Scratch: `.forge/` · CLI: `forge new|status|prefs|models|phase|doctor`
+# Forge (thin rule)
+
+Full workflow: `~/.codex/skills/forge/docs/forge.md` · Skill: user-installed **Forge** (`~/.codex/skills/forge/SKILL.md` after `forge install`).
+
+**Default:** triage before implementation. Substantial work → **Forge**
+(brainstorm → plan → subagent TDD implement → review).
+
+**Planning engine:** recorded in `.forge/config.json` (`plan.engine`). After brainstorm, create the change directly — OpenSpec → `/opsx:propose`, built-in specs → `forge change new <slug>`. Skip with `/forge:skip` or when work is trivial.
+
+Scratch: `.forge/` · CLI: `forge new|status|prefs|models|phase|doctor`
+
+Do not edit vendor planning-engine skills — Forge orchestrates them.

package/vendor/templates/project/cursor/rules/forge.mdc

@@ -1,21 +1,25 @@
----
-description: Forge triage and workflow (thin rule)
-alwaysApply: true
----
-
-# Forge (thin rule)
-
-Full workflow: `~/.cursor/skills/forge/docs/forge.md` · Skill: user-installed **Forge** (`~/.cursor/skills/forge/SKILL.md` after `forge install`).
-
-**Default:** triage before implementation. Substantial work → **Forge**
-(brainstorm → OpenSpec → subagent TDD implement → review).
-
-**Forge = OpenSpec:** after brainstorm, proceed directly to `/opsx:propose` — do not ask for a plan mode. Work too small for OpenSpec should skip Forge (`/forge:skip` or direct execution).
-
-**Skip Forge** only when work is trivial OR user sends **`/forge:skip`**.
-
-Scratch sessions: `.forge/sessions/` (14-day retention). Active pointer: `.forge/active.json`.
-
-CLI: `forge new`, `forge status`, `forge prefs`, `forge models`, `forge phase`, `forge doctor`.
-
-Do not edit vendor OpenSpec skills — Forge orchestrates them. Workflow skills are bundled under the Forge skill’s `skills/` folder.
+---
+description: Forge triage and workflow (thin rule)
+alwaysApply: true
+---
+
+# Forge (thin rule)
+
+Full workflow: `~/.cursor/skills/forge/docs/forge.md` · Skill: user-installed **Forge** (`~/.cursor/skills/forge/SKILL.md` after `forge install`).
+
+**Default:** triage before implementation. Substantial work → **Forge**
+(brainstorm → plan → subagent TDD implement → review).
+
+**Planning engine:** this project's engine is recorded in `.forge/config.json` (`plan.engine`). After brainstorm, create the change spec directly — do not ask for a plan mode:
+- OpenSpec → `/opsx:propose`
+- built-in specs → `forge change new <slug>`
+
+Work too small to spec should skip Forge (`/forge:skip` or direct execution).
+
+**Skip Forge** only when work is trivial OR user sends **`/forge:skip`**.
+
+Scratch sessions: `.forge/sessions/` (14-day retention). Active pointer: `.forge/active.json`.
+
+CLI: `forge new`, `forge status`, `forge prefs`, `forge models`, `forge phase`, `forge doctor`.
+
+Do not edit vendor planning-engine skills — Forge orchestrates them. Workflow skills are bundled under the Forge skill's `skills/` folder.