forge-orkes 0.80.1 → 0.80.3

package/bin/create-forge.js

@@ -588,12 +588,21 @@ function extractDetectionBlock(guideContent) {
  * with the exact guide pointer. There are NO per-version code branches: a new
  * guide that ships with a conformant Detection block is auto-covered.
  *
- * Range: installed < v <= source. `installedVersion` is the project's
+ * Range: installed <= v <= source. `installedVersion` is the project's
  * settings.json forge.version captured BEFORE the version stamp; `source` is
  * pkgVersion (this package's package.json) — never the template settings.json
  * literal, which is a placeholder the installer stamps at write time. Non-
  * interactive: the bin only warns and points at the guide; it never applies.
  *
+ * Lower bound is INCLUSIVE (issue #43): the version stamp always advances
+ * unconditionally, even when a MIGRATE hit was surfaced but declined that
+ * run — so an exclusive `installed < v` permanently hides a same-numbered
+ * guide the moment its own version gets stamped without being applied. This
+ * is safe to widen because the guide's own `## Detection` block is the real
+ * gate (repo-state-based: tracked file, size, absence of the post-migration
+ * artifact) — the version range only decides which guides are worth asking;
+ * a genuinely-already-applied guide's Detection block reports a no-op.
+ *
  * Returns the applying guides as [{version, guide, reason}] — the human warning
  * and the --json `migrations` key render the same findings.
  */
@@ -606,7 +615,7 @@ function runPostUpgradeMigrationChecks(installedVersion) {
     installedVersion && installedVersion !== 'unknown' ? installedVersion : '0.0.0';
   const source = pkgVersion;
 
-  // Select in-range guides: installed < v <= source. Parse {v} from the filename.
+  // Select in-range guides: installed <= v <= source. Parse {v} from the filename.
   let guides;
   try {
     guides = fs
@@ -618,7 +627,7 @@ function runPostUpgradeMigrationChecks(installedVersion) {
       .filter(Boolean)
       .filter(
         (g) =>
-          compareVersions(installed, g.version) < 0 &&
+          compareVersions(installed, g.version) <= 0 &&
           compareVersions(g.version, source) <= 0
       )
       .sort((a, b) => compareVersions(a.version, b.version));

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "forge-orkes",
-  "version": "0.80.1",
+  "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

@@ -135,7 +135,12 @@ function extract_id(line,    w, arr, n, tok, digits, suffix) {
   # normalization. Labelled ids (M-EVT01, m-SEQPRIM01) must NOT be digit-
   # stripped — that would collapse distinct labelled ids onto the same
   # numeric suffix (m-EVT01 and m-SEQPRIM01 would both become m-01) — so
-  # their alphanumeric suffix is preserved verbatim (lowercased) instead.
+  # their alphanumeric suffix is preserved VERBATIM, case included (issue
+  # #44): only the "m-" prefix is normalized, exactly like the numeric path
+  # never invents a digit convention of its own. Forge doesn't own the
+  # label's spelling — a project's own casing convention (e.g. m-EVT01, all
+  # caps) must survive the shard, or citations elsewhere in the repo stop
+  # matching the shard filename.
   if (tok !~ /^[Mm]-?[A-Za-z0-9]+$/) return ""
   suffix = tok
   sub(/^[Mm]-?/, "", suffix)
@@ -145,7 +150,7 @@ function extract_id(line,    w, arr, n, tok, digits, suffix) {
     gsub(/[^0-9]/, "", digits)
     return "m-" digits
   }
-  return "m-" tolower(suffix)
+  return "m-" suffix
 }
 
 function add_id(id) {
@@ -163,6 +168,7 @@ BEGIN {
   cur_noid_key = ""
   n_ids = 0
   n_noid = 0
+  n_other_headings = 0
 }
 
 /^## / {
@@ -174,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
 }
@@ -308,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

@@ -159,11 +159,14 @@ src_after="$(cat "$SOURCE" | wc -c)"
 [ -n "$src_after" ] && pass "source context.md still readable after migration" || fail "source context.md unreadable after migration"
 grep -q "### M7 — Widget Registry" "$SOURCE" && pass "source context.md left untouched" || fail "source context.md was modified"
 
-# --- labelled milestone ids (issue #42) -------------------------------------
+# --- labelled milestone ids (issue #42, case preservation issue #44) --------
 # Non-numeric ids (M-EVT01, m-SEQPRIM01) must shard into distinct files —
 # digit-stripping them onto a shared "m-01" was the exact silent-collision
 # bug reported upstream. Purely-numeric ids must keep their existing
-# normalization (m-53 stays m-53, unaffected by the labelled-id path).
+# normalization (m-53 stays m-53, unaffected by the labelled-id path). The
+# label's own casing must survive verbatim (issue #44) — only the "m-"
+# prefix is normalized — since a project's own convention (e.g. an
+# all-caps label) is what the rest of its repo actually cites.
 
 LBLROOT="$(mktemp -d)"
 trap 'rm -rf "$ROOT" "$LBLROOT"' EXIT INT TERM
@@ -203,15 +206,22 @@ LBLOUT="$("$MIGRATE" --source "$LBLSRC" --dest "$LBLDEST" 2>&1)"
 LBLRC=$?
 [ "$LBLRC" -eq 0 ] && pass "labelled-id migration exits 0" || fail "labelled-id migration exits 0 (exit $LBLRC)"
 
-check_file "$LBLDEST/context/m-evt01.md" "context/m-evt01.md created (labelled id, lowercased)"
-check_file "$LBLDEST/context/m-seqprim01.md" "context/m-seqprim01.md created (distinct from m-evt01, not digit-collapsed)"
+check_file "$LBLDEST/context/m-EVT01.md" "context/m-EVT01.md created (labelled id, prefix normalized)"
+check_file "$LBLDEST/context/m-SEQPRIM01.md" "context/m-SEQPRIM01.md created (distinct from m-EVT01, not digit-collapsed)"
 check_file "$LBLDEST/context/m-53.md" "context/m-53.md still created (numeric id unaffected by labelled-id path)"
 
+# check_file alone can't prove case survived — macOS's default case-insensitive
+# FS makes m-evt01.md and m-EVT01.md the same lookup. `ls` returns the actual
+# stored name; grep -x is case-sensitive regardless of the FS.
+lbl_listing="$(ls "$LBLDEST/context" 2>/dev/null)"
+echo "$lbl_listing" | grep -qx "m-EVT01.md" && pass "m-EVT01.md label case preserved verbatim on disk (issue #44)" || fail "label case NOT preserved — directory listing: $lbl_listing"
+echo "$lbl_listing" | grep -qx "m-SEQPRIM01.md" && pass "m-SEQPRIM01.md label case preserved verbatim on disk (issue #44)" || fail "label case NOT preserved — directory listing: $lbl_listing"
+
 lbl_count="$(ls "$LBLDEST"/context/*.md 2>/dev/null | wc -l | tr -d ' ')"
 [ "$lbl_count" -eq 3 ] && pass "labelled + numeric ids produce 3 distinct files, none collapsed" || fail "labelled-id run produced $lbl_count file(s), want 3"
 
-evt="$(cat "$LBLDEST/context/m-evt01.md" 2>/dev/null)"
-check_has "m-evt01.md carries its own Deferred entry (labelled-prefix match)" "$evt" "a follow-up scoped to the labelled id"
+evt="$(cat "$LBLDEST/context/m-EVT01.md" 2>/dev/null)"
+check_has "m-EVT01.md carries its own Deferred entry (labelled-prefix match)" "$evt" "a follow-up scoped to the labelled id"
 
 # --- unresolvable id scheme guard (issue #42) --------------------------------
 # A context.md whose Locked Decisions headings don't match m-<number> or a
@@ -241,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.1",
+    "version": "0.80.3",
     "default_tier": "standard",
     "beads_integration": false,
     "context_gates": {