-
-
Notifications
You must be signed in to change notification settings - Fork 80.3k
skills/trello: requires.bins gates on jq but every body example also needs curl #94727
Copy link
Copy link
Closed
Closed
Copy link
Labels
P2Normal backlog priority with limited blast radius.Normal backlog priority with limited blast radius.clawsweeper:linked-pr-openClawSweeper found an open linked pull request for this issue.ClawSweeper found an open linked pull request for this issue.clawsweeper:no-new-fix-prClawSweeper does not recommend queueing a new automated fix PR for this issue.ClawSweeper does not recommend queueing a new automated fix PR for this issue.clawsweeper:source-reproClawSweeper found a high-confidence source-level issue reproduction.ClawSweeper found a high-confidence source-level issue reproduction.impact:otherThis issue has meaningful maintainer-visible impact outside the owned taxonomy.This issue has meaningful maintainer-visible impact outside the owned taxonomy.issue-rating: 🦞 diamond lobsterVery strong issue quality with high-confidence source-level or clear reproduction.Very strong issue quality with high-confidence source-level or clear reproduction.
Description
Metadata
Metadata
Assignees
Labels
P2Normal backlog priority with limited blast radius.Normal backlog priority with limited blast radius.clawsweeper:linked-pr-openClawSweeper found an open linked pull request for this issue.ClawSweeper found an open linked pull request for this issue.clawsweeper:no-new-fix-prClawSweeper does not recommend queueing a new automated fix PR for this issue.ClawSweeper does not recommend queueing a new automated fix PR for this issue.clawsweeper:source-reproClawSweeper found a high-confidence source-level issue reproduction.ClawSweeper found a high-confidence source-level issue reproduction.impact:otherThis issue has meaningful maintainer-visible impact outside the owned taxonomy.This issue has meaningful maintainer-visible impact outside the owned taxonomy.issue-rating: 🦞 diamond lobsterVery strong issue quality with high-confidence source-level or clear reproduction.Very strong issue quality with high-confidence source-level or clear reproduction.
Type
Fields
No fields configured for issues without a type.
Summary
The bundled
skills/trello/SKILL.mdbody usescurlin every example, but itsrequires.binsonly declares["jq"]. On a system withoutcurl, the doctor /openclaw skills checkwill mark the Trello skill as ready, but the very first command the agent issues from it will fail at the shell. The gating disagrees with the body.Affected versions
Reproduced on
openclaw@2026.6.8(HEADc33cec04d9onmain). The gap was introduced in commit 1bf7d2f3bd (docs: update trello skill requirements), which added"jq"torequires.binsbut did not add"curl". Every code example in the SKILL.md body — both before and after that commit — usescurl.Repro (source-level)
skills/trello/SKILL.mdonmain(HEADc33cec04d9). Frontmatter:skills/trello/SKILL.md:10)curlinvocations in the body:curl. None of them are reachable withoutcurlon$PATH.curl:skills/notion/SKILL.md:9—"requires": { "anyBins": ["ntn", "curl"] }skills/openai-whisper-api/SKILL.md—"requires": { "bins": ["curl", "node"], ... }Both correctly include
curlin their gating. Trello is the only bundled skill with this mismatch.Expected
Either:
skills/trello/SKILL.mdfrontmatter declarescurl(preferred — matches the actual body), orcurlcalls to a binary that is actually gated on.Actual
openclaw skills checkreports Trello as ready on acurl-less system; the agent only discovers the missing dep when it runs the first command andbashreturnscommand not found: curl.Suggested fix
One-line change to
skills/trello/SKILL.md:10:Optionally also add a
curlinstall entry to theinstallarray alongsidejq, mirroringnotion's pattern. On macOS and most Linux distroscurlis already preinstalled, so an install entry is nice-to-have rather than required.Notes
curl(i.e. ~everyone on macOS/Debian/Ubuntu/Fedora), single failure mode prevented for the rare-but-real Alpine / minimal-container case.c33cec04d9.