fix(autoplan): bind commit before split-array index in tasks aggregator (#2018)

[0xDevNinja]

Problem

/autoplan's Phase 4 implementation-tasks aggregator returned zero tasks on every run, even when valid per-phase tasks-<phase>-*.jsonl records existed for the current branch and recent commits. Users saw No actionable tasks emitted from any phase with no diagnostic.

Root cause

The branch+commit filter was:

select(.branch == $branch and ($commits | split("|") | index(.commit) != null))

Inside ($commits | split("|") | ...) the pipe's input is the split array, so index(.commit) tried to index that array with the string "commit":

jq: error (at <stdin>:1): Cannot index array with string "commit"

The surrounding jq ... >> "$ALL_JSONL" swallowed the error via 2>/dev/null, so the aggregator silently dropped every record. Repro'd on v1.58.1.0 (also affects v1.52.2.0).

Fix

Bind the record's .commit to a variable before entering the split-array pipe, so index() receives the commit string:

select(.branch == $branch and (.commit as $c | ($commits | split("|") | index($c)) != null))

One-line change in the source resolver (scripts/resolvers/tasks-section.ts), autoplan/SKILL.md regenerated via bun run gen:skill-docs. Added a short comment so the binding isn't "simplified" back into the bug later.

Tests

New free regression test test/autoplan-tasks-aggregator.test.ts extracts the shipped select(...) filter from autoplan/SKILL.md and runs it through jq:

• matching branch+commit → kept, no jq error
• non-matching commit → excluded, no error
• wrong branch → excluded, no error
• record missing commit field → excluded, no error
• asserts the filter no longer contains the bare index(.commit) form

5/5 pass. Full free suite green (bun test gen-skill-docs + skill-validation: 734 pass), gen-skill-docs freshness check clean.

Fixes #2018.

[trunk-io]

Merging to main in this repository is managed by Trunk.

• To merge this pull request, check the box to the left or comment /trunk merge below.

After your PR is submitted to the merge queue, this comment will be automatically updated with its status. If the PR fails, failure details will also be posted here

[time-attack]

@16francej ensures commits are real and dont get split can merge