forge-orkes 0.80.2 → 0.80.3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "forge-orkes",
-  "version": "0.80.2",
+  "version": "0.80.3",
   "description": "Set up the Forge meta-prompting framework for Claude Code in your project",
   "bin": {
     "create-forge": "./bin/create-forge.js"

package/template/.claude/hooks/forge-context-migrate.sh

@@ -168,6 +168,7 @@ BEGIN {
   cur_noid_key = ""
   n_ids = 0
   n_noid = 0
+  n_other_headings = 0
 }
 
 /^## / {
@@ -179,6 +180,10 @@ BEGIN {
   if ($0 ~ /^## Needs Resolution/) { section = "NEEDS_RESOLUTION"; next }
   if ($0 ~ /^## Amendment Log/) { section = "AMENDMENT"; next }
   section = "OTHER"
+  n_other_headings++
+  w = $0
+  sub(/^## /, "", w)
+  other_heading_order[n_other_headings] = w
   other_buf = other_buf $0 "\n"
   next
 }
@@ -313,6 +318,23 @@ END {
     exit 1
   }
 
+  # Unrecognized top-level `## ` sections (issue #45) are preserved verbatim
+  # in context-log.md's "Other Sections" by design — never guessed onto a
+  # milestone — but a large or multi-heading OTHER bucket usually means the
+  # source context.md uses a section vocabulary this tool doesn't know about
+  # (e.g. a project-specific "## Active Streams"), and that content silently
+  # missing its own per-milestone home is easy to overlook. Loud, not fatal:
+  # nothing is lost (it's right there in context-log.md), so this warns
+  # without changing the exit code.
+  if (n_other_headings > 0 && length(other_buf) > 500) {
+    headings_list = other_heading_order[1]
+    for (i = 2; i <= n_other_headings; i++) headings_list = headings_list ", " other_heading_order[i]
+    print "forge-context-migrate: WARNING — " length(other_buf) " byte(s) across " \
+      n_other_headings " unrecognized heading(s) (" headings_list ") landed in" \
+      " context-log.md's \"Other Sections\" instead of a per-milestone file — review" \
+      " it; if that content is milestone-scoped, re-file it into context/{id}.md by hand" > "/dev/stderr"
+  }
+
   print "forge-context-migrate: " n_ids " milestone id(s) processed"
 }
 AWKEOF

package/template/.claude/hooks/tests/forge-context-migrate.test.sh

@@ -251,6 +251,56 @@ BADRC=$?
 [ "$BADRC" -ne 0 ] && pass "unresolvable id scheme exits non-zero instead of silently producing 0 shards" || fail "unresolvable id scheme should exit non-zero (exit $BADRC)"
 check_has "unresolvable id scheme prints an ERROR line" "$BADOUT" "ERROR"
 
+# --- large unrecognized-section warning (issue #45) -------------------------
+# A project-specific top-level heading (e.g. "## Active Streams") this tool
+# doesn't know about is preserved verbatim in context-log.md's "Other
+# Sections" by design — but silently doing that with a large amount of
+# content is easy to overlook. Must warn (not fail — nothing is lost) when
+# the OTHER bucket is substantial, and stay silent when it's small/absent.
+# Deliberately alongside a resolving Locked Decisions id, so this warning
+# fires independent of (and isn't masked by) the zero-shards guard above.
+
+WARNROOT="$(mktemp -d)"
+trap 'rm -rf "$ROOT" "$LBLROOT" "$BADROOT" "$WARNROOT"' EXIT INT TERM
+WARNSRC="$WARNROOT/context.md"
+WARNDEST="$WARNROOT/.forge"
+mkdir -p "$WARNDEST"
+
+{
+  printf '## Locked Decisions\n\n### m-53 — numeric decision\n\n- **Decision**: fine.\n\n'
+  printf '## Active Streams\n\n### m-HC01 — some stream\n\n'
+  # 600 bytes — over the 500-byte warn floor
+  printf 'x%.0s' $(seq 1 600)
+  printf '\n\n## Needs Resolution\n\n(none)\n\n## Amendment Log\n\n(none)\n'
+} > "$WARNSRC"
+
+WARNOUT="$("$MIGRATE" --source "$WARNSRC" --dest "$WARNDEST" --dry-run 2>&1)"
+WARNRC=$?
+[ "$WARNRC" -eq 0 ] && pass "large unrecognized-section run still exits 0 (warning, not a failure)" || fail "large unrecognized-section run should exit 0 (exit $WARNRC)"
+check_has "large unrecognized-section run prints a WARNING line" "$WARNOUT" "WARNING"
+check_has "warning names the unrecognized heading" "$WARNOUT" "Active Streams"
+
+SMALLROOT="$(mktemp -d)"
+trap 'rm -rf "$ROOT" "$LBLROOT" "$BADROOT" "$WARNROOT" "$SMALLROOT"' EXIT INT TERM
+SMALLSRC="$SMALLROOT/context.md"
+SMALLDEST="$SMALLROOT/.forge"
+mkdir -p "$SMALLDEST"
+
+cat > "$SMALLSRC" <<'FIXTURE'
+## Locked Decisions
+
+### m-53 — numeric decision
+
+- **Decision**: fine.
+
+## Some Small Aside
+
+A short unrecognized note, well under the warn floor.
+FIXTURE
+
+SMALLOUT="$("$MIGRATE" --source "$SMALLSRC" --dest "$SMALLDEST" --dry-run 2>&1)"
+check_absent "a small unrecognized section stays silent (no WARNING)" "$SMALLOUT" "WARNING"
+
 # --- summary -----------------------------------------------------------------
 
 printf '\n%s passed, %s failed\n' "$PASSED" "$FAILED"

package/template/.claude/settings.json

@@ -1,6 +1,6 @@
 {
   "forge": {
-    "version": "0.80.2",
+    "version": "0.80.3",
     "default_tier": "standard",
     "beads_integration": false,
     "context_gates": {