Skip to content

Remove jq from the prerequisites - #58

Open
anakanemison wants to merge 1 commit into
mschaller/d5-transitive-definesfrom
mschaller/d6-jq-optional
Open

anakanemison wants to merge 1 commit into
mschaller/d5-transitive-definesfrom
mschaller/d6-jq-optional

Conversation

@anakanemison

@anakanemison anakanemison commented Oct 20, 2026 •

Copy link
Copy Markdown
Contributor

SKILL.md lists jq as required and says to stop and ask if it is missing, but no script uses it, and an unattended agent cannot ask. This removes it.

In any2bazel-lab, agents in containers without jq tried sudo apt-get install jq, downloaded a binary from GitHub (two runs), or spent calls establishing they did not need it. In six runs with an earlier version that kept jq as optional, none installed it.

Stacked on #57 (tracking issue: #35). pytest tests/: 186 passed (132 on main). Lab proposal: P28.

🤖 Generated with Claude Code

@armandomontanez

armandomontanez commented Oct 20, 2026 •

Copy link
Copy Markdown
Contributor

I added this preemptively since I was going to add template jq commands, but yeah this is technically not needed yet.

I suggest removing jq entirely from this list instead.

@anakanemison
anakanemison force-pushed the mschaller/d6-jq-optional branch from 656d604 to 29eb747 Compare October 26, 2026 13:30
SKILL.md lists jq among the tools every migration requires, and says to stop
and ask the user to install anything missing. No script in scripts/ calls
jq. An unattended agent cannot ask, so it improvises. In any2bazel-lab's
migration runs, in containers without jq, agents tried
`sudo apt-get install jq`, downloaded a jq binary from GitHub (two runs), or
spent three or four calls establishing they did not need it. One agent wrote
the finding down itself: "No script under the skill's scripts/ directory
invokes it".

jq was listed in advance of template jq commands that have not been added,
so this removes it. In the lab's six runs with an earlier version of this
change, which kept jq as an optional convenience, no agent installed or
downloaded it.

Lab-Proposal: P28
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@anakanemison
anakanemison removed this pull request from stack #59 November 2, 2026 17:49
@anakanemison
anakanemison force-pushed the mschaller/d6-jq-optional branch from 29eb747 to 2b6f61c Compare November 2, 2026 17:50
@anakanemison anakanemison changed the title Say that jq is optional (for discussion) Remove jq from the prerequisites Nov 2, 2026
@anakanemison
anakanemison added this pull request to stack #63 November 2, 2026 17:54
@anakanemison

Copy link
Copy Markdown
Contributor Author

I added this preemptively since I was going to add template jq commands, but yeah this is technically not needed yet.

I suggest removing jq entirely from this list instead.

Done.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants