forge-orkes 0.80.1 → 0.80.2

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.2",
   "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) {

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

package/template/.claude/settings.json

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