@thebassclef/lite 1.9.9 → 1.9.11

package/dist/lite/lib/state.sh

@@ -40,8 +40,9 @@ __state_check_binary_deps() {
   if [ "${__STATE_DEPS_CHECKED:-}" = "1" ]; then
     return 0
   fi
+  # Baseline deps — awk + jq required unconditionally
   local missing=""
-  for bin in awk python3 jq; do
+  for bin in awk jq; do
     command -v "$bin" >/dev/null 2>&1 || missing="$missing $bin"
   done
   if [ -n "$missing" ]; then
@@ -49,21 +50,63 @@ __state_check_binary_deps() {
 STATE ACCESSOR FAILURE
 Required binary missing from PATH:$missing
 
-These binaries are load-bearing for lib/state.sh accessors (Shape B
-YAML frontmatter extraction + JSON tooling).
+awk + jq are load-bearing for lib/state.sh accessors.
 
 Resolution:
-  macOS: brew install gawk python3 jq
-  Debian: sudo apt-get install gawk python3 jq
-
-Closes the bassclef#598 silent-failure class.
+  macOS: brew install gawk jq
+  Debian: sudo apt-get install gawk jq
 EOF
     return 4
   fi
+
+  # Per cli#308: YAML parser needs EITHER yq OR python3+PyYAML.
+  # Stock macOS has python3 but not PyYAML. Pre-cli#308, that mismatch
+  # caused silent failures where __state_check_binary_deps passed but
+  # the actual yaml parse failed with ModuleNotFoundError. Now we check
+  # that at least one working parser exists.
+  __state_check_yaml_parser || return 6
   export __STATE_DEPS_CHECKED=1
   return 0
 }
 
+# YAML parser check (per cli#308). Returns 0 if yq available OR
+# python3+yaml-module importable. Returns 6 if neither.
+__state_check_yaml_parser() {
+  # Cache the result per shell process.
+  if [ "${__STATE_YAML_PARSER:-}" = "yq" ] || [ "${__STATE_YAML_PARSER:-}" = "python3" ]; then
+    return 0
+  fi
+  # Prefer yq (faster + simpler contract).
+  if command -v yq >/dev/null 2>&1; then
+    export __STATE_YAML_PARSER=yq
+    return 0
+  fi
+  # Fall back to python3+PyYAML.
+  if command -v python3 >/dev/null 2>&1 && python3 -c "import yaml" >/dev/null 2>&1; then
+    export __STATE_YAML_PARSER=python3
+    return 0
+  fi
+  cat >&2 <<EOF
+STATE ACCESSOR FAILURE — no YAML parser available (per cli#308)
+
+lib/state.sh needs ONE of:
+  - yq (preferred — faster, no Python module needed)
+  - python3 + PyYAML module
+
+Both are missing on this machine.
+
+Resolution:
+  macOS (preferred): brew install yq
+  macOS (alt):       python3 -m pip install pyyaml
+  Debian: sudo apt-get install yq  OR  python3 -m pip install pyyaml
+
+Per cli#308 — this check closes the silent-failure class where
+python3 existed but PyYAML did not, and callers reported "goal not
+found" (wrong diagnosis).
+EOF
+  return 6
+}
+
 # yq2json — extract YAML frontmatter from markdown file, emit JSON.
 # Used by Shape B (markdown + YAML frontmatter + body) entity accessors.
 #
@@ -86,7 +129,7 @@ EOF
 # on invalid input) + @luminary michael-feathers (characterization
 # tests RED-first via fixtures/2026-09-06-silent-empty/).
 __state_yq2json() {
-  __state_check_binary_deps || return 4
+  __state_check_binary_deps || return $?
   local file="$1"
   if [ ! -f "$file" ]; then
     return 1
@@ -127,7 +170,23 @@ __state_yq2json() {
 
   local out
   local ec
-  out=$(awk '/^---$/{c++; next} c==1' "$file" | python3 -c "
+  # Per cli#308 — prefer yq when available, fall back to python3+PyYAML.
+  # __state_check_binary_deps (called at top of this function via return 4
+  # path) already resolved __STATE_YAML_PARSER to one of the two.
+  if [ "${__STATE_YAML_PARSER:-}" = "yq" ]; then
+    out=$(awk '/^---$/{c++; next} c==1' "$file" | yq -o=json '.' 2>&1)
+    ec=$?
+    if [ "$ec" -ne 0 ]; then
+      printf 'yaml parse failed (yq): %s\n' "$out" >&2
+      return 3
+    fi
+    # yq returns `null` for empty input; coerce to `{}` for consistency
+    # with python3 path.
+    if [ "$out" = "null" ]; then
+      out="{}"
+    fi
+  else
+    out=$(awk '/^---$/{c++; next} c==1' "$file" | python3 -c "
 import sys, yaml, json
 try:
     data = yaml.safe_load(sys.stdin) or {}
@@ -136,7 +195,8 @@ except Exception as e:
     print(f'yaml parse failed: {e}', file=sys.stderr)
     sys.exit(3)
 ")
-  ec=$?
+    ec=$?
+  fi
   # Instrument real-parse count for #1519 tests + observability.
   __STATE_YQ2JSON_PARSE_COUNT=$(( ${__STATE_YQ2JSON_PARSE_COUNT:-0} + 1 ))
   if [ "$ec" -ne 0 ]; then
@@ -185,7 +245,7 @@ __state_atomic_write() {
 # output) + @luminary michael-nygard (silent-fail wrap; write does not
 # crash caller).
 __state_yaml_write() {
-  __state_check_binary_deps || return 4
+  __state_check_binary_deps || return $?
   # Finding #6 (#2036): `local path` is a zsh-magic variable name — ties to
   # $PATH when sourced from zsh. Renamed to `local p` throughout this file.
   local p="$1"
@@ -1030,6 +1090,17 @@ state_reference_get() {
     marker)
       state_marker_get "$id"
       ;;
+    input-artifact)
+      # InputArtifact — Shape C (pure JSON) per state-spine.md L262.
+      # Added per sunj-labs/bassclef-cli#315: /objectory-decompose Step 1
+      # reads input-artifact via this accessor; without a case the call
+      # exited 5 and the SKILL could not read the artifact in lite.
+      if [ -f "docs/input-artifacts/${id}.json" ]; then
+        cat "docs/input-artifacts/${id}.json"
+      else
+        return 1
+      fi
+      ;;
     gate)
       # Gates derive from Markers; not addressable directly in v0
       echo "state_reference_get: gate type derives from Marker; not directly addressable in v0" >&2

package/dist/lite/scripts/interpret-input.sh

@@ -52,12 +52,14 @@ SOURCE=""
 TYPE="auto"
 PERSIST=0
 SLUG=""
+INTENT=""     # cli#319 cure — operator intent text; defaults to SOURCE in text mode
 
 while [ $# -gt 0 ]; do
   case "$1" in
     --type=*) TYPE="${1#--type=}"; shift ;;
     --persist) PERSIST=1; shift ;;
     --slug=*) SLUG="${1#--slug=}"; shift ;;
+    --intent=*) INTENT="${1#--intent=}"; shift ;;    # cli#319
     --help|-h) usage; exit 0 ;;
     --*) echo "interpret-input: unknown flag '$1'" >&2; usage >&2; exit 2 ;;
     *)
@@ -117,7 +119,15 @@ fi
 
 NOW=$(date -u +%Y-%m-%dT%H:%M:%SZ)
 
-STRUCTURAL_HINTS='{}'
+# cli#319 cure — default INTENT to SOURCE in text mode; empty otherwise.
+# /pick-luminaries matches intent against the luminary catalog; an empty
+# intent is explicitly valid (adopters may inject it later), but text input
+# carries operator intent in SOURCE already — reuse it as a safe default.
+if [ -z "$INTENT" ] && [ "$TYPE" = "text" ]; then
+  INTENT="$SOURCE"
+fi
+
+STRUCTURAL_HINTS='{"verb_goal_pairs": []}'
 
 case "$TYPE" in
   text)
@@ -149,7 +159,7 @@ case "$TYPE" in
     if [ -z "$REPO_LOCAL" ]; then
       # Source can't be resolved; return Phase B-style stub (still valid InputArtifact)
       EXTRACTED_TEXT="deferred to Phase C"
-      STRUCTURAL_HINTS='{}'
+      STRUCTURAL_HINTS='{"verb_goal_pairs": []}'
     else
       EXTRACT_OUT=$(bash "$REPO_ROOT/scripts/interpret-input/extract-repo.sh" "$REPO_LOCAL" 2>/dev/null)
       EXTRACT_RC=$?
@@ -160,10 +170,13 @@ case "$TYPE" in
 
       if [ "$EXTRACT_RC" != "0" ] || [ -z "$EXTRACT_OUT" ]; then
         EXTRACTED_TEXT="repo extraction returned no signal for $SOURCE"
-        STRUCTURAL_HINTS='{}'
+        STRUCTURAL_HINTS='{"verb_goal_pairs": []}'
       else
         EXTRACTED_TEXT=$(echo "$EXTRACT_OUT" | jq -r '.extracted_text')
-        STRUCTURAL_HINTS=$(echo "$EXTRACT_OUT" | jq -c '.structural_hints')
+        # cli#312 cure — merge default verb_goal_pairs: [] so downstream readers
+        # (/build, /launch templates) always find the key even when extract
+        # doesn't produce it. Extract wins on any key it does emit.
+        STRUCTURAL_HINTS=$(echo "$EXTRACT_OUT" | jq -c '{verb_goal_pairs: []} + .structural_hints')
       fi
     fi
     EXTERNAL_URLS='[]'
@@ -179,6 +192,7 @@ ARTIFACT=$(jq -n \
   --arg type "$TYPE" \
   --arg source "$SOURCE" \
   --arg extracted_text "$EXTRACTED_TEXT" \
+  --arg intent "$INTENT" \
   --arg created_at "$NOW" \
   --argjson external_urls "$EXTERNAL_URLS" \
   --argjson structural_hints "$STRUCTURAL_HINTS" \
@@ -187,6 +201,7 @@ ARTIFACT=$(jq -n \
     type: $type,
     source: $source,
     extracted_text: $extracted_text,
+    intent: $intent,
     structural_hints: $structural_hints,
     external_urls: $external_urls,
     created_at: $created_at

package/dist/lite/standards/lite-manifest-schema-changes.md

@@ -34,6 +34,121 @@ Every entry stays in this file for the life of the manifest. Never pruned. When
 
 ## Entries (newest first)
 
+### cli#319 — interpret-input writes intent field (v1.16.9 → v1.16.10, 2026-10-07)
+
+- **Change type:** content-change (1 file: `scripts/interpret-input.sh`). Script now emits `intent` field in input-artifact JSON. Prior shape omitted it.
+- **Fields:** 448 entries (unchanged). Content hash shifts on the `interpret-input` script slug.
+- **Version bump:** v1.16.9 → v1.16.10 (patch — content-only change on 1 slug; rides on top of cli#312 v1.16.9 bump).
+- **Rationale:** bassclef-cli#319 — Kunal cold-adopter smoke caught downstream readers (`/launch`, `/build`) needing the intent field + missing on artifact JSON. Per PR #2105.
+- **Downstream cure:** no action for adopters at exact-pin. Downstream readers now get a stable intent key.
+- **Old shape retired:** nothing retired. Additive key.
+
+
+### cli#312 — interpret-input emits verb_goal_pairs default (v1.16.8 → v1.16.9, 2026-10-07)
+
+- **Change type:** content-change (1 file: `scripts/interpret-input.sh`). Script now writes `structural_hints.verb_goal_pairs` as an empty array when no pairs detected. Prior code omitted the key.
+- **Fields:** 448 entries (unchanged). Content hash shifts on the `interpret-input` script slug.
+- **Version bump:** v1.16.8 → v1.16.9 (patch — content-only change on 1 slug; rides on top of cli#324 v1.16.8 bump).
+- **Rationale:** bassclef-cli#312 — Kunal cold-adopter smoke caught downstream readers tripping on missing key. Default-empty-array is defensive. Per PR #2104.
+- **Downstream cure:** no action for adopters at exact-pin. Downstream readers of `structural_hints.verb_goal_pairs` get a consistent array shape.
+- **Old shape retired:** nothing retired. Behavior is additive — the key always exists now; readers that handled missing-key still work.
+
+
+### cli#324 — /launch Phase 11 greenfield partial vs redesign full (v1.16.7 → v1.16.8, 2026-10-07)
+
+- **Change type:** content-change (1 file: `.claude/skills/launch/SKILL.md`). Phase 11 now distinguishes greenfield partial-fit from redesign full-fit. Prior behavior conflated both cases.
+- **Fields:** 448 entries (unchanged). Content hash shifts on the `launch` skill slug.
+- **Version bump:** v1.16.7 → v1.16.8 (patch — content-only change on 1 slug; rides on top of cli#322 v1.16.7 bump).
+- **Rationale:** bassclef-cli#324 — Kunal cold-adopter smoke caught `/launch` Phase 11 misreading greenfield intents as redesign full-reshape. Partial-fit path missing. Per PR #2103.
+- **Downstream cure:** no action for adopters at exact-pin. Phase 11 now reads adopter intent + routes to partial OR full. Prior sessions that hit full-reshape see no regression (full stays as-is).
+- **Old shape retired:** nothing retired. Phase 11 branch is additive.
+
+
+### cli#322 — /launch Phase 14 jargon gate (v1.16.6 → v1.16.7, 2026-10-07)
+
+- **Change type:** content-change (1 file: `.claude/skills/launch/SKILL.md`). Phase 14 template now runs a jargon gate on the output before writing. Prior behavior shipped jargon through.
+- **Fields:** 448 entries (unchanged). Content hash shifts on the `launch` skill slug.
+- **Version bump:** v1.16.6 → v1.16.7 (patch — content-only change on 1 slug; rides on top of cli#309 v1.16.6 bump).
+- **Rationale:** bassclef-cli#322 — Kunal cold-adopter smoke caught `/launch` Phase 14 shipping bassclef-internal jargon into adopter-facing output. Per PR #2100.
+- **Downstream cure:** no action for adopters at exact-pin. Phase 14 now surfaces jargon via gate before write.
+- **Old shape retired:** nothing retired. Gate is additive.
+
+
+### cli#309 — /build hosting_platform=none warns (v1.16.5 → v1.16.6, 2026-10-07)
+
+- **Change type:** content-change (1 file: `lib/composer-preflight.sh`). Helper now warns + continues when hosting_platform=none instead of refusing. Prior code refused unconditionally.
+- **Fields:** 448 entries (unchanged). Content hash shifts on the `composer-preflight` lib slug.
+- **Version bump:** v1.16.5 → v1.16.6 (patch — content-only change on 1 slug; rides on top of cli#311 v1.16.5 bump).
+- **Rationale:** bassclef-cli#309 — Kunal cold-adopter smoke caught `/build` refusing when adopter opted out of hosting. Local-only builds are a valid shape. Per PR #2101.
+- **Downstream cure:** no action for adopters at exact-pin. Local-only builds (hosting_platform=none) now warn + proceed.
+- **Old shape retired:** nothing retired. Behavior is additive — refuse stays for other invalid states.
+
+
+### cli#311 — artifact-ingestion-gate skip when parent_bet is null (v1.16.4 → v1.16.5, 2026-10-07)
+
+- **Change type:** content-change (1 file: `.claude/hooks/artifact-ingestion-gate.sh`). Hook now skips the goal-doc parent-walk when the active goal declares `parent_bet: null` in frontmatter. Prior behavior treated null as missing + fired BLOCK on orientation-gate.
+- **Fields:** 448 entries (unchanged). Content hash shifts on the `artifact-ingestion-gate` hook slug.
+- **Version bump:** v1.16.4 → v1.16.5 (patch — content-only change on 1 slug; no entry count change).
+- **Rationale:** bassclef-cli#311 — Kunal cold-adopter smoke caught the orientation-gate firing BLOCK on root-level goals (no parent). Null parent_bet is valid state for arc-root goals; the gate was treating it as unresolved reference. Per PR #2102.
+- **Downstream cure:** no action for adopters at exact-pin. Root goals with `parent_bet: null` stop firing BLOCK at session-start; existing goals with non-null parents still get the walk.
+- **Old shape retired:** nothing retired. Null-parent-bet branch is additive on top of existing walk logic.
+
+
+### #2036 Finding #10 — /last-block skill + marker format (v1.15.5 → v1.16.0, 2026-10-06)
+
+- **Change type:** content-add (4 new files: `.claude/skills/last-block/SKILL.md`, `lib/last-block.sh`, `.claude/rules/last-block-marker.md`, `.claude/hooks/session-reflection.d/97-last-block-count.sh`). New skill that prints the exact text of a BLOCK or guardrail banner from this session or prior. New marker format at `state/last-block/<slug>.md`.
+- **Fields:** 448 entries (+4 from 444). Four new slugs — skill `last-block`, lib `last-block`, rule `last-block-marker`, hook `97-last-block-count`.
+- **Version bump:** v1.15.5 → v1.16.0 (minor — content-add of 4 new lite slugs). Rides on top of Finding #7 which landed first (v1.15.4 → v1.15.5).
+- **Rationale:** Kunal v1.6.4 adopter run — "I saw a BLOCK banner scroll off the top of the terminal. No way to read it again." Finding #10 adds `/last-block` for recall + a hook that counts markers at session-reflection time. Per bassclef-upstream#2036 Finding #10 + PR #2061.
+- **Downstream cure:** no action for adopters at exact-pin. Four new files land on next sync — all `tier: lite` so every tier picks them up. No shape change on existing slugs.
+- **Old shape retired:** nothing retired (content-add only).
+
+
+### cli#305 — identifier-leak scrub self-exclude + commit author check (v1.15.3 → v1.15.4, 2026-10-04)
+
+- **Change type:** content-change (1 file: `.claude/hooks/pre-commit-identifier-leak-scrub.sh`). Check 2 now filters out hunks from the hook's own files, its test, the rule, and the pattern source. Check 3 added — scans `git var GIT_AUTHOR_IDENT` for the pattern set plus the `.local` suffix. BLOCK banner adds a resolution path pointing at the GitHub no-reply address.
+- **Fields:** 444 entries (unchanged). Content hash shifts on the `pre-commit-identifier-leak-scrub` hook slug.
+- **Version bump:** v1.15.3 → v1.15.4 (patch — content-only change on 1 slug; no entry count change). Rides on top of the Finding #7 bump (v1.15.3) when PRs land in order.
+- **Rationale:** Kunal cli#305 — first commit after `bassclef init` blocked on the check's own files; author identity leaked silently when `user.email` was unset on Mac. Class A identity leak sister to cli#313 + cli#320 shipped in v1.15.2. Per PR #2062 (planned).
+- **Downstream cure:** no action for adopters at exact-pin. Self-exclude paths are PATH-scoped, not pattern-scoped — leaks moved into non-self files still block. Author check is additive; adopters with clean emails see no change. Adopters whose git author ends in `.local` or carries a hostname pattern see the new BLOCK with a recipe pointing at `<username>@users.noreply.github.com`.
+- **Old shape retired:** nothing retired. Check 1 (message) + Check 2 (staged content, now filtered) + Check 3 (author) chain is additive.
+
+### #2036 Finding #7 — Playwright probe file-based fallback (v1.15.4 → v1.15.5, 2026-10-05)
+
+- **Change type:** content-change (1 file: `lib/capability-probe.sh`). `probe_mcp_playwright()` extended with four MCP config file fallback paths after the `CLAUDE_MCP_PLAYWRIGHT` env-var fast path. Fail-soft on missing `jq` + malformed JSON.
+- **Fields:** 444 entries (unchanged). Content hash shifts on the `capability-probe` lib slug.
+- **Version bump:** v1.15.4 → v1.15.5 (patch — content-change on existing lib slug per policy). Rebased on top of cli#305 which landed first.
+- **Rationale:** Kunal v1.6.4 adopter run installed Playwright MCP but did not set `CLAUDE_MCP_PLAYWRIGHT`; probe returned 1; `/riff` + `/build` skipped screenshots. Fallback now reads `.mcp.json`, project + user `.claude/settings.json`, and `~/.claude.json`. Per bassclef-upstream#2036 Finding #7 + PR #2060.
+- **Downstream cure:** nothing — byte-for-byte unchanged when `CLAUDE_MCP_PLAYWRIGHT` is set (fast path first). Adopters without the env var set gain file-based detection on the fly.
+- **Old shape retired:** immediately (new behavior is additive).
+
+### cli#313 + cli#320 — identity-leak sweep in luminary-pick + personas (v1.15.1 → v1.15.2, 2026-10-04)
+
+- **Change type:** content-change (2 files: `lib/luminary-pick.sh` + `.claude/skills/personas/SKILL.md`). luminary-pick basedir fallback moves from `$HOME/src/sunj-labs/bassclef` to `$CLAUDE_PROJECT_DIR` → `git rev-parse` → `pwd` chain, plus adds `.claude/luminaries/*.md` catalog fallback for lite tier. personas SKILL default slug changes from email-derived (`operator-[local-part]`) to plain `operator` (never from email).
+- **Fields:** 444 entries (unchanged). Content hash shifts on the `luminary-pick` lib slug + `personas` skill slug.
+- **Version bump:** v1.15.1 → v1.15.2 (patch; content-only on 2 slugs; no schema shape shift; no entry count change).
+- **Rationale:** bassclef-cli#313 (luminary-pick hardcoded eng path; adopters hit "catalog dir missing" every /launch Phase 3.5) + bassclef-cli#320 (operator slug derived from git email leaked GitHub account ID into committed docs). Kunal cold-adopter smoke caught both; shared class is "engineering assumption in cold paths" (per plan doc Slot 3).
+- **Downstream cure:** no action for adopters at exact-pin. Both changes are additive — lite tier adopters now get a working picker catalog; email-derived slugs never fire again. Operators wanting a display name beyond `operator` get an in-turn prompt instead.
+- **Old shape retired:** nothing retired. Both old shapes stay viable (BASSCLEF_DIR env var still honored first; manual slug entry still allowed via operator reply).
+
+### cli#307 — /build Phase 0 awk matches Step-N + WU-N (v1.15.0 → v1.15.1, 2026-10-04)
+
+- **Change type:** content-change (1 file: `.claude/skills/build/SKILL.md`). Phase 0 safety floor awk pattern + prose at L276 updated to recognize both `### Step-N` (new vocab per ADR-040 D1) and `### WU-N` (legacy).
+- **Fields:** 444 entries (unchanged). Content hash shifts on the `build` skill slug.
+- **Version bump:** v1.15.0 → v1.15.1 (patch; content-only change on 1 slug; no schema shape shift; no entry count change).
+- **Rationale:** bassclef-cli#307 — Kunal cold-adopter smoke caught the Phase 0 awk reading zero acceptance lines on Step-format specs. Partial-rename regression from ADR-040 D1 (Phase 1 grep got the rename, Phase 0 awk was missed).
+- **Downstream cure:** no action for adopters at exact-pin — the fix is additive (both heading forms match). Step-format specs now trigger the floor as designed. WU-format specs continue to work (characterization).
+- **Old shape retired:** nothing retired. The WU-only pattern leaves. The `(Step|WU)-` pattern stays indefinitely.
+
+### #2036 — install-written-paths manifest + lib + rule (v1.14.4 → v1.15.0, 2026-10-03)
+
+- **Change type:** content-add (3 new entries: `lib/install-written-paths.sh`, `.claude/rules/install-written-paths-mediation.md`, `.claude/hooks/session-reflection.d/83-install-written-paths-rebuild.sh`); content-change on `.claude/hooks/pre-commit-gate.sh` (filter block added for Kunal Finding #8 cure).
+- **Fields:** 441 entries → 444 entries (3 adds, 1 content_hash shift).
+- **Version bump:** v1.14.4 → v1.15.0 (content-add MINOR per discipline policy).
+- **Rationale:** bassclef-upstream#2036 Finding #8 structural cure. See `architecture/reviews/2026-10-03.md` + plan doc `docs/next-session-plan-2026-10-04-kunal-slots-5-6.md` Slot 5.
+- **Downstream cure:** no action for adopters at exact-pin — the new entries are additive and the lib's accessor API is new. Adopters pulling at `@thebassclef/lite@^1.15.0` get the cure automatically; the session-start fragment regenerates the install-written-paths manifest on first fresh-install session.
+- **Old shape retired:** nothing retired. All existing entries preserved.
+
 ### #1777 — friction hook pattern adds stage|launch new skill vocabulary (v1.14.2 → v1.14.3, 2026-10-01)
 
 - **Change type:** content-change (1 file: `post-skill-friction-check` hook)