@really-knows-ai/foundry 3.6.0 → 3.6.1

package/dist/CHANGELOG.md

@@ -1,5 +1,12 @@
 # Changelog
 
+## [3.6.1] - 2026-05-25
+
+### Fixed
+
+- Forge contract enforcement now correctly handles the first forge run with no prior feedback items (empty batch). Previously the contract failed with "forge changed artefacts but did not mark any feedback as actioned", creating a system feedback item that cascaded into an infinite forge loop.
+- Forge history entries now include the actual list of changed files detected by git diff, instead of showing an empty `changed_files: []` for every entry.
+
 ## [3.6.0] - 2026-05-25
 
 ### Added

package/dist/scripts/lib/forge-contract.js

@@ -76,15 +76,24 @@ function checkBatchVersion(items, feedbackStore, cycleId, postVersion, preVersio
 /**
  * Enforce the forge contract on a batch of items presented to forge.
  *
- * Two-level check:
+ * When `items` is not a non-empty array (null, undefined, or []), the
+ * contract passes immediately without side-effects. This covers the
+ * initial forge run where no feedback exists yet.
+ *
+ * Two-level check when items are present:
  * 1. Per-item: every item must end in 'actioned' or 'wont-fix'.
  * 2. Batch-level: artefact version semantics must be consistent.
  *
- * @param {{ items: Array<{id: string}>, preVersion: string, postVersion: string,
- *   feedbackStore: object, cycleId: string }} params
+ * @param {{ items?: Array<{id: string}> | null, preVersion: string,
+ *          postVersion: string, feedbackStore: object, cycleId: string }} params
  * @returns {{ contractPassed: boolean }}
  */
 export function enforceForgeContract({ items, preVersion, postVersion, feedbackStore, cycleId }) {
+  if (!Array.isArray(items) || items.length === 0) {
+    // Empty batch means forge had no prior feedback to respond to.
+    // This is the first forge run or all items were already resolved.
+    return { contractPassed: true };
+  }
   if (!checkPerItemResponse(items, feedbackStore, cycleId, postVersion)) {
     return { contractPassed: false };
   }

package/dist/scripts/orchestrate-phases.js

@@ -307,7 +307,9 @@ export async function finaliseStage(args) {
   const iteration = getIteration(historyPath, cycleId, io);
   const openFeedback = computeOpenFeedback(io);
   writeHistoryEntries({
-    historyPath, cycleId, lastStage, iteration, openFeedback, io,
+    historyPath, cycleId,
+    lastStage: { ...lastStage, changedFiles: finalizeResult.changedFiles },
+    iteration, openFeedback, io,
     artefactVersion: postVersion, contractPassed,
   });
   const commitErr = await tryStageCommit(git, lastStage, cycleId, io);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@really-knows-ai/foundry",
-  "version": "3.6.0",
+  "version": "3.6.1",
   "description": "A skill-driven framework for governed artefact generation with AI coding tools. Define your own artefact types, laws, and flows — Foundry handles the forge → quench → appraise pipeline with deterministic routing, quality gates, and iterative refinement.",
   "type": "module",
   "main": "dist/.opencode/plugins/foundry.js",