@codedrifters/configulator 0.0.391 → 0.0.392

package/lib/index.mjs

@@ -14676,8 +14676,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.",
@@ -14816,14 +14817,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 ""`,
     "}",
     "",
@@ -14880,11 +14884,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)"',
     "')",
     "",
@@ -15056,14 +15061,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 ""`,
     "}",
     "",
@@ -15111,7 +15119,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)"',
     "  ')",
     "",
@@ -15211,7 +15219,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 |',
     '      (.labels | map(.name) | map(select(startswith("type:"))) | .[0] // "") as $type_label |',
     '      "\\(.number)\\t\\(.title)\\t\\($type_label)\\t\\($dep_line)"',
     "    ')",
@@ -15316,7 +15324,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)"',
     "    ')",
     "",