@windyroad/architect 0.6.1 → 0.6.2-preview.300

package/.claude-plugin/plugin.json

@@ -1,5 +1,5 @@
 {
   "name": "wr-architect",
-  "version": "0.6.1",
+  "version": "0.6.2",
   "description": "Architecture decision enforcement for Claude Code"
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@windyroad/architect",
-  "version": "0.6.1",
+  "version": "0.6.2-preview.300",
   "description": "Architecture decision enforcement for AI coding agents",
   "bin": {
     "windyroad-architect": "./bin/install.mjs"

package/skills/capture-adr/SKILL.md

@@ -56,7 +56,7 @@ Same P056-safe `local_max + origin_max + 1` formula as `/wr-architect:create-adr
 ```bash
 local_max=$(ls docs/decisions/*.md 2>/dev/null | sed 's/.*\///' | grep -oE '^[0-9]+' | sort -n | tail -1)
 origin_max=$(git ls-tree --name-only origin/main docs/decisions/ 2>/dev/null | sed 's|^docs/decisions/||' | grep -oE '^[0-9]+' | sort -n | tail -1)
-next=$(printf '%03d' $(( $(echo -e "${local_max:-0}\n${origin_max:-0}" | sort -n | tail -1) + 1 )))
+next=$(printf '%03d' $(( 10#$(echo -e "${local_max:-0}\n${origin_max:-0}" | sort -n | tail -1) + 1 )))
 ```
 
 `--name-only` is required (P056): without it, each `git ls-tree` line includes the 40-char blob SHA which can contain three-digit runs that the digit-extraction regex false-matches.

package/skills/capture-adr/test/capture-adr.bats

@@ -97,7 +97,7 @@ teardown() {
   [ "$local_max" = "123" ]
 
   # No origin available in the fixture; default to 0 then increment.
-  next=$(printf '%03d' $(( $(echo -e "${local_max:-0}\n0" | sort -n | tail -1) + 1 )))
+  next=$(printf '%03d' $(( 10#$(echo -e "${local_max:-0}\n0" | sort -n | tail -1) + 1 )))
   [ "$next" = "124" ]
 }
 
@@ -107,7 +107,7 @@ teardown() {
               | sed 's/.*\///' \
               | grep -oE '^[0-9]+' \
               | sort -n | tail -1)
-  next=$(printf '%03d' $(( $(echo -e "${local_max:-0}\n0" | sort -n | tail -1) + 1 )))
+  next=$(printf '%03d' $(( 10#$(echo -e "${local_max:-0}\n0" | sort -n | tail -1) + 1 )))
   [ "$next" = "001" ]
 }
 
@@ -119,10 +119,29 @@ teardown() {
   local_max=50
   origin_max=175   # parallel session pushed ADR-175
 
-  next=$(printf '%03d' $(( $(echo -e "${local_max:-0}\n${origin_max:-0}" | sort -n | tail -1) + 1 )))
+  next=$(printf '%03d' $(( 10#$(echo -e "${local_max:-0}\n${origin_max:-0}" | sort -n | tail -1) + 1 )))
   [ "$next" = "176" ]
 }
 
+@test "capture-adr: next-ID handles 099 → 100 transition without octal-eval failure (P164)" {
+  # P164 regression: bash $(( ... )) parses leading-zero numbers as octal.
+  # `099` is invalid octal (digits >= 8). Without `10#` prefix, this fires:
+  #   bash: 099: value too great for base (error token is "099")
+  # The fix is the standard `10#` base-10 prefix on the inner $(echo ... | tail -1).
+  mkdir -p "$TMPROOT/docs/decisions"
+  : > "$TMPROOT/docs/decisions/098-foo.proposed.md"
+  : > "$TMPROOT/docs/decisions/099-bar.proposed.md"
+
+  local_max=$(ls "$TMPROOT/docs/decisions"/*.md 2>/dev/null \
+              | sed 's/.*\///' \
+              | grep -oE '^[0-9]+' \
+              | sort -n | tail -1)
+  [ "$local_max" = "099" ]
+
+  next=$(printf '%03d' $(( 10#$(echo -e "${local_max:-0}\n0" | sort -n | tail -1) + 1 )))
+  [ "$next" = "100" ]
+}
+
 # ---------------------------------------------------------------------------
 # Skeleton-fill MADR shape — capture-adr writes a deferred-placeholder ADR
 # at status: proposed. Load-bearing primitives:

package/skills/create-adr/SKILL.md

@@ -78,7 +78,7 @@ local_max=$(ls docs/decisions/*.md 2>/dev/null | sed 's/.*\///' | grep -oE '^[0-
 origin_max=$(git ls-tree --name-only origin/main docs/decisions/ 2>/dev/null | sed 's|^docs/decisions/||' | grep -oE '^[0-9]+' | sort -n | tail -1)
 
 # Take the max of the two and increment.
-next=$(printf '%03d' $(( $(echo -e "${local_max:-0}\n${origin_max:-0}" | sort -n | tail -1) + 1 )))
+next=$(printf '%03d' $(( 10#$(echo -e "${local_max:-0}\n${origin_max:-0}" | sort -n | tail -1) + 1 )))
 ```
 
 If the local choice would have collided with an origin ADR created since the last fetch, the `git ls-tree origin/<base>` lookup catches it here and the renumber is automatic. Log the renumber in the user-facing report (e.g. "Bumped next ADR number from 020 → 021 to avoid collision with origin").