@drafthq/draft 3.7.1 → 3.7.2

package/.claude-plugin/marketplace.json

@@ -12,7 +12,7 @@
       "name": "draft",
       "source": "./",
       "description": "Context-Driven Development: draft specs and plans before implementation. Structured workflows for features and fixes.",
-      "version": "3.7.1",
+      "version": "3.7.2",
       "author": {
         "name": "mayurpise"
       },

package/.claude-plugin/plugin.json

@@ -1,7 +1,7 @@
 {
   "name": "draft",
   "description": "Context-Driven Development: draft specs and plans before implementation. Structured workflows for features and fixes.",
-  "version": "3.7.1",
+  "version": "3.7.2",
   "author": {
     "name": "mayurpise"
   },

package/.cursor-plugin/plugin.json

@@ -2,7 +2,7 @@
   "name": "draft",
   "displayName": "Draft",
   "description": "Context-Driven Development: draft specs and plans before implementation. Structured workflows for features and fixes.",
-  "version": "3.7.1",
+  "version": "3.7.2",
   "skills": "./skills/",
   "agents": "./core/agents/",
   "author": {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@drafthq/draft",
-  "version": "3.7.1",
+  "version": "3.7.2",
   "description": "Context-Driven Development for AI coding agents — install Draft into Claude Code, Cursor, Codex, or opencode.",
   "bin": {
     "draft": "cli/bin/draft.js"

package/scripts/tools/hotspot-rank.sh

@@ -71,8 +71,20 @@ ARCH_JSON="$(memory_cli get_architecture \
 echo "$ARCH_JSON" | jq -e . >/dev/null 2>&1 || unavailable
 
 # Pre-computed complexity/cognitive/is_entry_point per symbol (merged onto fan-in).
-PROPS_JSON="$(gq_run "$PROJECT" "$(gq_q_node_props)" || echo '{"rows":[]}')"
-echo "$PROPS_JSON" | jq -e . >/dev/null 2>&1 || PROPS_JSON='{"rows":[]}'
+#
+# The hotspots themselves come from get_architecture above, which is fail-loud,
+# so a failure here costs enrichment rather than the whole result — worth keeping
+# the ranking for. But it must not pass silently: substituting an empty row set
+# gave every symbol complexity 0 and cognitive 0, folded those zeros into `score`,
+# and still reported source:"memory-graph", so a fan-in-only ranking was
+# indistinguishable from a fully measured one. Record the outcome instead, and
+# omit the fields rather than emitting measurements that were never taken.
+if PROPS_JSON="$(gq_run "$PROJECT" "$(gq_q_node_props)")"; then
+    ENRICHMENT="ok"
+else
+    PROPS_JSON='{"rows":[]}'
+    ENRICHMENT="unavailable"
+fi
 
 # Merge via temp files (the props row set can exceed argv limits on large repos).
 TMP_ARCH="$(mktemp)"; TMP_PROPS="$(mktemp)"
@@ -80,7 +92,8 @@ trap 'rm -f "$TMP_ARCH" "$TMP_PROPS"' EXIT
 printf '%s' "$ARCH_JSON" > "$TMP_ARCH"
 printf '%s' "$PROPS_JSON" > "$TMP_PROPS"
 
-jq -n --slurpfile arch "$TMP_ARCH" --slurpfile props "$TMP_PROPS" --argjson top "$TOP" '
+jq -n --slurpfile arch "$TMP_ARCH" --slurpfile props "$TMP_PROPS" --argjson top "$TOP" \
+      --arg enrichment "$ENRICHMENT" '
     ((($props[0].rows) // []) | map(select(.[0] != null))) as $prows
     | (reduce $prows[] as $r ({};
           .[$r[0]] = {c:((($r[1]) // "0") | tonumber? // 0),
@@ -89,10 +102,11 @@ jq -n --slurpfile arch "$TMP_ARCH" --slurpfile props "$TMP_PROPS" --argjson top
     | [ (($arch[0].hotspots) // [])[]
         | (.qualified_name) as $q
         | ($pmap[$q] // {c:0, cog:0, ep:false}) as $p
-        | {id:$q, name:.name, fanIn:(.fan_in // 0),
-           complexity:$p.c, cognitive:$p.cog,
-           score:((.fan_in // 0) + $p.c + $p.cog),
-           isEntryPoint:$p.ep} ]
+        | ({id:$q, name:.name, fanIn:(.fan_in // 0),
+            score:((.fan_in // 0) + $p.c + $p.cog)}
+           + (if $enrichment == "ok"
+              then {complexity:$p.c, cognitive:$p.cog, isEntryPoint:$p.ep}
+              else {} end)) ]
     | sort_by(-.score)
     | (if $top > 0 then .[0:$top] else . end) as $h
-    | {hotspots:$h, source:"memory-graph"}'
+    | {hotspots:$h, enrichment:$enrichment, source:"memory-graph"}'

package/scripts/tools/mermaid-from-graph.sh

@@ -57,25 +57,39 @@ if [[ ! -d "$REPO" ]]; then
     exit 1
 fi
 
+# $1: "unavailable" (engine/query failed) or "empty" (query ran, no such edges).
+# Swallowing a failed query with `{}` used to make both look identical, so an
+# engine failure and a repo with genuinely no cross-file imports produced the
+# same "graph not built" diagram — the reader could not tell which they had.
 stub() {
-    cat <<'EOF'
+    if [[ "${1:-unavailable}" == "empty" ]]; then
+        cat <<'EOF'
+```mermaid
+%% graph is built, but holds no edges of this kind — nothing to draw
+flowchart LR
+    empty["no edges"]
+```
+EOF
+    else
+        cat <<'EOF'
 ```mermaid
 %% graph data unavailable — index the repo with the graph engine first
 flowchart LR
     empty["graph not built"]
 ```
 EOF
+    fi
     exit 2
 }
 
-graph_bootstrap "$REPO" || stub
+graph_bootstrap "$REPO" || stub unavailable
 
 # module-deps: real IMPORTS edges (the auto-derived dependency graph). Self-imports
 # (src == dst) are dropped so the diagram is a true cross-file graph. Capped at 40
 # edges for readability.
 render_module_deps() {
-    local res; res="$(gq_run "$PROJECT" "$(gq_q_imports)" || echo '{}')"
-    local edges; edges="$(echo "${res:-{\}}" | jq -r '
+    local res; res="$(gq_run "$PROJECT" "$(gq_q_imports)")" || return 2
+    local edges; edges="$(echo "$res" | jq -r '
         [ (.rows // [])[] | {s:(.[0]|tostring), d:(.[1]|tostring)}
           | select(.s != "" and .d != "" and .s != .d) ]
         | unique | .[0:40][] | "    \"" + .s + "\" --> \"" + .d + "\""' 2>/dev/null || true)"
@@ -85,23 +99,25 @@ render_module_deps() {
 
 # co-change: FILE_CHANGES_WITH coupling (the prior module-deps proxy).
 render_co_change() {
-    local res; res="$(gq_run "$PROJECT" "$(gq_q_co_change)" || echo '{}')"
-    local edges; edges="$(echo "${res:-{\}}" | jq -r '(.rows // [])[] | "    \"" + (.[0]|tostring) + "\" --> \"" + (.[1]|tostring) + "\""' 2>/dev/null || true)"
+    local res; res="$(gq_run "$PROJECT" "$(gq_q_co_change)")" || return 2
+    local edges; edges="$(echo "$res" | jq -r '(.rows // [])[] | "    \"" + (.[0]|tostring) + "\" --> \"" + (.[1]|tostring) + "\""' 2>/dev/null || true)"
     if [[ -z "$edges" ]]; then return 1; fi
     printf '```mermaid\nflowchart LR\n%s\n```\n' "$edges"
 }
 
 render_proto_map() {
     local res; res="$(memory_cli get_architecture \
-        "$(jq -n --arg p "$PROJECT" '{project:$p, aspects:["routes"]}')" || echo '{}')"
-    local edges; edges="$(echo "${res:-{\}}" | jq -r '(.routes // [])[] | "    \"" + ((.method // "")|tostring) + " " + ((.path // "")|tostring) + "\" --> \"" + ((.handler // "?")|tostring) + "\""' 2>/dev/null || true)"
+        "$(jq -n --arg p "$PROJECT" '{project:$p, aspects:["routes"]}')" || true)"
+    [[ -n "$res" ]] && echo "$res" | jq -e . >/dev/null 2>&1 || return 2
+    local edges; edges="$(echo "$res" | jq -r '(.routes // [])[] | "    \"" + ((.method // "")|tostring) + " " + ((.path // "")|tostring) + "\" --> \"" + ((.handler // "?")|tostring) + "\""' 2>/dev/null || true)"
     if [[ -z "$edges" ]]; then return 1; fi
     printf '```mermaid\nflowchart LR\n%s\n```\n' "$edges"
 }
 
+# rc 2 = the query failed; rc 1 = it ran and returned nothing. Different stubs.
 case "$DIAGRAM" in
-    module-deps) render_module_deps || stub ;;
-    co-change)   render_co_change || stub ;;
-    proto-map)   render_proto_map || stub ;;
+    module-deps) render_module_deps || { rc=$?; [[ $rc -eq 1 ]] && stub empty || stub unavailable; } ;;
+    co-change)   render_co_change   || { rc=$?; [[ $rc -eq 1 ]] && stub empty || stub unavailable; } ;;
+    proto-map)   render_proto_map   || { rc=$?; [[ $rc -eq 1 ]] && stub empty || stub unavailable; } ;;
     *) echo "Unknown --diagram '$DIAGRAM' (expected module-deps|co-change|proto-map)" >&2; exit 1 ;;
 esac