@windyroad/itil 0.27.0-preview.296 → 0.27.1-preview.300

package/.claude-plugin/plugin.json

@@ -1,5 +1,5 @@
 {
   "name": "wr-itil",
-  "version": "0.27.0",
+  "version": "0.27.1",
   "description": "ITIL-aligned IT service management for Claude Code"
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@windyroad/itil",
-  "version": "0.27.0-preview.296",
+  "version": "0.27.1-preview.300",
   "description": "ITIL-aligned IT service management for Claude Code (problem, and future incident/change skills)",
   "bin": {
     "windyroad-itil": "./bin/install.mjs"

package/skills/capture-problem/SKILL.md

@@ -125,7 +125,7 @@ Same P056-safe local_max + origin_max formula as `/wr-itil:manage-problem` Step
 # extends the same coverage to the origin tree.
 local_max=$(ls docs/problems/*.md docs/problems/*/*.md 2>/dev/null | sed 's|.*/||' | grep -oE '^[0-9]+' | sort -n | tail -1)
 origin_max=$(git ls-tree -r --name-only origin/main docs/problems/ 2>/dev/null | sed 's|.*/||' | 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 )))
 ```
 
 Log the renumber decision in the operation report if origin and local diverged.

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

@@ -131,7 +131,7 @@ teardown() {
   [ "$local_max" = "107" ]
 
   # 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" = "108" ]
 }
 
@@ -141,10 +141,29 @@ 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" ]
 }
 
+@test "capture-problem: 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/problems"
+  : > "$TMPROOT/docs/problems/098-foo.open.md"
+  : > "$TMPROOT/docs/problems/099-bar.open.md"
+
+  local_max=$(ls "$TMPROOT/docs/problems"/*.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" ]
+}
+
 # ---------------------------------------------------------------------------
 # Conservative duplicate-grep — title-only filename match, 3-keyword cap.
 # Architect Q1 verdict: title-only because conservative threshold rationale

package/skills/capture-rfc/SKILL.md

@@ -128,7 +128,7 @@ Same `max(local, origin) + 1` formula as `capture-problem` Step 3, scanning `doc
 ```bash
 local_max=$(ls docs/rfcs/RFC-*.md 2>/dev/null | sed 's|.*/RFC-||;s|-.*||' | grep -oE '^[0-9]+' | sort -n | tail -1)
 origin_max=$(git ls-tree --name-only origin/main docs/rfcs/ 2>/dev/null | sed 's|^docs/rfcs/RFC-||;s|-.*||' | 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 )))
 ```
 
 Log the renumber decision in the operation report if origin and local diverged.

package/skills/manage-problem/SKILL.md

@@ -331,7 +331,7 @@ local_max=$(ls docs/problems/*.md docs/problems/*/*.md 2>/dev/null | sed 's|.*/|
 origin_max=$(git ls-tree --name-only -r origin/main docs/problems/ 2>/dev/null | sed 's|.*/||' | 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 ticket created since the last fetch, the `git ls-tree origin/<base>` lookup catches it here and the renumber is automatic. Log the renumber decision in the operation report (e.g. "Bumped next ID from 042 → 043 to avoid collision with origin").