forge-orkes 0.80.2 → 0.80.4

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "forge-orkes",
-  "version": "0.80.2",
+  "version": "0.80.4",
   "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.4",
     "default_tier": "standard",
     "beads_integration": false,
     "context_gates": {

package/template/.forge/gitignore

@@ -34,13 +34,19 @@ dev-source
 # Orchestration locks
 *.lock
 
-# Render-on-read derived registries (0.53.0). Regenerated at every `forge` boot
-# from their sources (state/index.yml ← state/milestone-*.yml; streams/active.yml
-# ← streams/*.yml + active milestones). Local cache only — never committed, so a
-# committed projection can never drift stale from its source. Without these
-# entries a fresh install would re-commit them.
+# Render-on-read derived registries (0.53.0; context.md joined in 0.80.0/ADR-033).
+# Regenerated at every `forge` boot from their sources (state/index.yml ←
+# state/milestone-*.yml; streams/active.yml ← streams/*.yml + active milestones;
+# context.md ← context/*.md joined against state/index.yml). Local cache only —
+# never committed, so a committed projection can never drift stale from its
+# source. Without these entries a fresh install would re-commit them, and an
+# upgrade would silently re-track context.md after a project migrated off the
+# monolithic file (issue #46 — this line must ship canonically, not as a
+# per-project manual edit, since this file is overwritten wholesale on every
+# upgrade).
 state/index.yml
 streams/active.yml
+context.md
 
 # Notion board id-binding cache (ADR-020 derived class) — forge-id → page-id, rebuilt
 # by query-self-heal; never committed (a committed copy would leak private Notion page IDs).

package/template/.forge/migrations/0.80.0-context-md-sharding.md

@@ -123,19 +123,23 @@ What is **not** repeatable is the untrack step below — do that once.
 git rm --cached .forge/context.md
 ```
 
-Add `.forge/context.md` to whichever gitignore file already lists
-`.forge/state/index.yml` and `.forge/streams/active.yml` — grep for
-`index.yml` to find the right file (in this repo it's the root
-`.gitignore`, under the "Render-on-read derived registries" comment block;
-your project's may differ if it customized gitignore layout):
+If your project uses the shipped `.forge/.gitignore` default (no custom
+gitignore layout), a `create-forge` install/upgrade at 0.80.4+ already lists
+`context.md` next to `state/index.yml` and `streams/active.yml` in that file —
+nothing to add by hand, and it's safe against a future upgrade overwriting it
+(issue #46: before 0.80.4, `.forge/.gitignore` was wholesale re-synced from
+the canonical shipped file on every upgrade, silently reverting a manual
+addition here — the fix moved the line into the canonical source itself).
+
+If your project customized gitignore layout (e.g. this repo's own root
+`.gitignore`, not `.forge/.gitignore`), add `.forge/context.md` there by
+hand, next to the other two derived-registry entries, matching their
+existing comment block rather than starting a new one:
 
 ```bash
 grep -rn "index.yml" .gitignore .forge/.gitignore 2>/dev/null
 ```
 
-Add the line next to the other two derived-registry entries, matching their
-existing comment block rather than starting a new one.
-
 ### 3. Commit the migration output
 
 ```bash