@codedrifters/configulator 0.0.390 → 0.0.392

package/lib/index.js

@@ -15024,8 +15024,9 @@ function renderUnblockDependentsSection(ud) {
     "",
     "1. Searches open issues for `Depends on: #<closed>` references in",
     "   their body.",
-    "2. For each match, re-reads every dependency listed in the",
-    "   `Depends on:` line and checks whether they are all closed.",
+    "2. For each match, re-reads every dependency listed across",
+    "   **all** `Depends on:` lines in the body (bodies routinely list",
+    "   one dependency per line) and checks whether they are all closed.",
     "3. If **all** dependencies are closed, flips the dependent from",
     "   `status:blocked` to `status:ready` and posts a one-line comment",
     "   citing the resolving issue.",
@@ -15164,14 +15165,17 @@ function renderUnblockDependentsScript(ud) {
     "",
     "# \u2500\u2500 helpers \u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500",
     "",
-    '# Extract issue numbers from a "Depends on:" line.',
-    '# Returns space-separated numbers, or empty string for "(none)".',
+    '# Extract issue numbers from a "Depends on:" text. The argument may be a',
+    '# single line ("Depends on: #1, #2, #3") OR several Depends-on lines',
+    "# joined into one string: bodies routinely list one dependency per line",
+    "# (issue #881), so callers union every matching line before calling this.",
+    "# Returns the space-separated union of every #<n> reference, or an empty",
+    '# string when the text has no concrete references (e.g. a lone "(none)"',
+    '# sentinel). References are extracted directly, so a stray "(none)"',
+    "# concatenated with a real dep line never suppresses the real dependency",
+    "# \u2014 only reference-free text counts as having no dependencies.",
     "parse_deps() {",
     '  local line="$1"',
-    `  if echo "$line" | grep -qi '(none)'; then`,
-    '    echo ""',
-    "    return",
-    "  fi",
     `  echo "$line" | grep -oE '#[0-9]+' | tr -d '#' | tr '\\n' ' ' || echo ""`,
     "}",
     "",
@@ -15228,11 +15232,12 @@ function renderUnblockDependentsScript(ud) {
     "  exit 0",
     "fi",
     "",
-    "# Extract the `Depends on:` line per candidate and stream as",
-    "# tab-separated number / title / dep-line tuples.",
+    "# Union every `Depends on:` line per candidate (a body may list one",
+    "# dependency per line) into a single space-joined dep string and stream",
+    "# as tab-separated number / title / dep-line tuples.",
     `matches=$(echo "$dependents" | jq -r '`,
     "  .[] |",
-    '  (.body | split("\\n") | map(select(test("Depends on:"; "i"))) | .[0] // "") as $dep_line |',
+    '  (.body | split("\\n") | map(select(test("Depends on:"; "i"))) | join(" ")) as $dep_line |',
     '  "\\(.number)\\t\\(.title)\\t\\($dep_line)"',
     "')",
     "",
@@ -15404,14 +15409,17 @@ function buildCheckBlockedScript(tiers, scopeGate, _runRatio) {
     "",
     "# \u2500\u2500 helpers \u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500",
     "",
-    '# Extract issue numbers from a "Depends on:" line.',
-    '# Returns space-separated numbers, or empty string for "(none)".',
+    '# Extract issue numbers from a "Depends on:" text. The argument may be a',
+    '# single line ("Depends on: #1, #2, #3") OR several Depends-on lines',
+    "# joined into one string: bodies routinely list one dependency per line",
+    "# (issue #881), so callers union every matching line before calling this.",
+    "# Returns the space-separated union of every #<n> reference, or an empty",
+    '# string when the text has no concrete references (e.g. a lone "(none)"',
+    '# sentinel). References are extracted directly, so a stray "(none)"',
+    "# concatenated with a real dep line never suppresses the real dependency",
+    "# \u2014 only reference-free text counts as having no dependencies.",
     "parse_deps() {",
     '  local line="$1"',
-    `  if echo "$line" | grep -qi '(none)'; then`,
-    '    echo ""',
-    "    return",
-    "  fi",
     `  echo "$line" | grep -oE '#[0-9]+' | tr -d '#' | tr '\\n' ' ' || echo ""`,
     "}",
     "",
@@ -15459,7 +15467,7 @@ function buildCheckBlockedScript(tiers, scopeGate, _runRatio) {
     "  local issue_data",
     `  issue_data=$(echo "$issues" | jq -r '`,
     "    .[] |",
-    '    (.body | split("\\n") | map(select(test("Depends on:"; "i"))) | .[0] // "") as $dep_line |',
+    '    (.body | split("\\n") | map(select(test("Depends on:"; "i"))) | join(" ")) as $dep_line |',
     '    "\\(.number)\\t\\($dep_line)"',
     "  ')",
     "",
@@ -15559,13 +15567,18 @@ function buildCheckBlockedScript(tiers, scopeGate, _runRatio) {
     "    local issue_data",
     `    issue_data=$(echo "$issues" | jq -r '`,
     "      .[] |",
-    '      (.body | split("\\n") | map(select(test("Depends on:"; "i"))) | .[0] // "") as $dep_line |',
+    '      (.body | split("\\n") | map(select(test("Depends on:"; "i"))) | join(" ")) as $dep_line |',
     '      (.labels | map(.name) | map(select(startswith("type:"))) | .[0] // "") as $type_label |',
-    '      "\\(.number)\\t\\(.title)\\t\\($dep_line)\\t\\($type_label)"',
+    '      "\\(.number)\\t\\(.title)\\t\\($type_label)\\t\\($dep_line)"',
     "    ')",
     "",
     '    local results=""',
-    "    while IFS=$'\\t' read -r num title dep_line type_label; do",
+    "    # Field order: number, title, type_label, dep_line. dep_line is the",
+    "    # only optional field, so it MUST stay last: an absent Depends-on line",
+    "    # yields an empty trailing field, whereas an empty non-last field",
+    "    # collapses under IFS=tab and shifts every later field left, dropping",
+    "    # type_label off the end (#884). Matches the cmd_stale blocked scan.",
+    "    while IFS=$'\\t' read -r num title type_label dep_line; do",
     '      [[ -z "$num" ]] && continue',
     "",
     '      local deps=""',
@@ -15659,7 +15672,7 @@ function buildCheckBlockedScript(tiers, scopeGate, _runRatio) {
     "    local bl_data",
     `    bl_data=$(echo "$bl_issues" | jq -r '`,
     "      .[] |",
-    '      (.body | split("\\n") | map(select(test("Depends on:"; "i"))) | .[0] // "") as $dep_line |',
+    '      (.body | split("\\n") | map(select(test("Depends on:"; "i"))) | join(" ")) as $dep_line |',
     '      "\\(.number)\\t\\(.title)\\t\\(.updatedAt)\\t\\($dep_line)"',
     "    ')",
     "",