@izkac/forgekit 0.3.2 → 0.3.3

package/package.json

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

package/src/e2e.mjs

@@ -60,7 +60,9 @@ if (!sessionId) {
 }
 
 const { dir, session } = loadSession(sessionId);
-const file = e2ePath({ session, sessionDir: dir });
+// init writes: target the live change dir only (never fall back into the
+// archive). run/check/status read: allow the archive fallback.
+const file = e2ePath({ session, sessionDir: dir, forWrite: sub === 'init' });
 
 if (sub === 'init') {
   try {

package/src/integrity.mjs

@@ -98,12 +98,18 @@ function liveOrArchived(changesDir, change) {
 export function resolveChangeDir(opts = {}) {
   const cwd = opts.cwd ?? process.cwd();
   const session = opts.session ?? null;
+  // Write callers (spine/e2e init) must always target the LIVE change dir —
+  // scaffolding into an archived record would corrupt frozen history. Only
+  // read callers (checks, gates) fall back to the archive.
+  const forWrite = opts.forWrite === true;
   const change = session && isNonEmptyString(session.openspecChange) ? session.openspecChange : null;
   if (!change) return null;
 
   const openspecChanges = path.join(cwd, 'openspec', 'changes');
   const openspecDir = path.join(openspecChanges, change);
-  if (session.planType === 'openspec') return liveOrArchived(openspecChanges, change);
+  if (session.planType === 'openspec') {
+    return forWrite ? openspecDir : liveOrArchived(openspecChanges, change);
+  }
 
   let specsRoot = DEFAULT_SPECS_DIR;
   try {
@@ -113,11 +119,14 @@ export function resolveChangeDir(opts = {}) {
   }
   const specsChanges = path.join(cwd, specsRoot, 'changes');
   const specsDir = path.join(specsChanges, change);
-  if (session.planType === 'specs') return liveOrArchived(specsChanges, change);
+  if (session.planType === 'specs') {
+    return forWrite ? specsDir : liveOrArchived(specsChanges, change);
+  }
 
   // planType unknown — prefer whichever exists (live first, then archived)
   if (fs.existsSync(openspecDir)) return openspecDir;
   if (fs.existsSync(specsDir)) return specsDir;
+  if (forWrite) return openspecDir;
   return (
     findArchivedChangeDir(openspecChanges, change) ??
     findArchivedChangeDir(specsChanges, change) ??

package/src/integrity.test.mjs

@@ -136,12 +136,36 @@ test('resolveChangeDir: falls back to the archived copy after archive', () => {
         recursive: true,
       });
       assert.equal(resolveChangeDir({ cwd, session }), archived);
+
+      // forWrite (spine/e2e init) never follows into the archive — writes must
+      // target the live change dir so scaffolding can't corrupt frozen history.
+      assert.equal(resolveChangeDir({ cwd, session, forWrite: true }), liveDir);
     } finally {
       fs.rmSync(cwd, { recursive: true, force: true });
     }
   }
 });
 
+test('spinePath/e2ePath forWrite target the live dir even after archive', () => {
+  const cwd = tmp('forge-write-guard-');
+  try {
+    const session = { planType: 'openspec', openspecChange: 'add-customer-registry' };
+    const changesDir = path.join(cwd, 'openspec', 'changes');
+    // Only the archived copy exists — mimics running init after archive.
+    const archived = path.join(changesDir, 'archive', '2026-07-20-add-customer-registry');
+    fs.mkdirSync(archived, { recursive: true });
+    const liveDir = path.join(changesDir, 'add-customer-registry');
+
+    // Read path resolves the archive; write path stays on the (absent) live dir.
+    assert.equal(spinePath({ cwd, session }), path.join(archived, 'spine.json'));
+    assert.equal(spinePath({ cwd, session, forWrite: true }), path.join(liveDir, 'spine.json'));
+    assert.equal(e2ePath({ cwd, session }), path.join(archived, 'e2e.json'));
+    assert.equal(e2ePath({ cwd, session, forWrite: true }), path.join(liveDir, 'e2e.json'));
+  } finally {
+    fs.rmSync(cwd, { recursive: true, force: true });
+  }
+});
+
 test('runIntegrityChecks: passes after archive (spine resolves in archive dir)', () => {
   const cwd = tmp('forge-int-archived-');
   try {

package/src/spine.mjs

@@ -48,7 +48,9 @@ if (!sessionId) {
 }
 
 const { dir, session } = loadSession(sessionId);
-const file = spinePath({ session, sessionDir: dir });
+// init writes: target the live change dir only (never fall back into the
+// archive). check/status read: allow the archive fallback.
+const file = spinePath({ session, sessionDir: dir, forWrite: sub === 'init' });
 
 if (sub === 'init') {
   try {