Repository navigation
A skill's text calls its command by its full name - #2047
Merged
Merged
Conversation
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…ly beside its own command Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The first step of
@openagt/init(#2023): a skill's command is no longer installed in the project.👤
npx @openagt/skill-tickets@0.1 list. Nothing is installed in the project.🤖 automated · Opus 5.5, effort high
The problem
npx tickets list. That runs our command only where the package is installed in the project.npxdownloads whatever holds the nameticketson npm and runs it. Eight of our short names belong to strangers there.What changed
npx @openagt/skill-tickets@0.1 list. This repository's 12 tracked copies follow.0.0.2to0.1.0. A range over0.0.xmatches one version only;@0.1matches every0.1.x, so a fix reaches the agents by itself.npxruns a copy it finds in the checkout, so the full name runs the dashboard's own copy: nothing downloaded, no network, nothing to publish first.Picks of mine
@0.1, where the issue wrote@^1. Two reasons. In zsh with the extended-glob option on,^is a pattern sign and the command fails with "no matches found". AndMEMORY.mdsays the version line stays in0.x. Asked in chat; changing to1is a small follow-up.branches, the full name is left to npm, so that tool is never run by mistake.Known
npxdoes not see the link at the checkout's root and asks npm. That fails until0.1.0is published, in this repository too.npx agent-runner,npx agent-scheduler) keep their short names. They run with the dashboard's own tools ahead on the PATH, and are for the init pull request.node_modules/.bin, and a cleanup leaves it there.To publish, when this is merged (never by an agent)
0.1.0: the 25@openagt/skill-*and@openagt/agent-scheduler. Nothing breaks before that, except the sub-package case above.Checked
git statusshows nothing in the checkout or the project.npx --offlineitself.^in zsh, the0.xdecision, the wrong tool running beside anotherbranches, and three test gaps. All acted on.DECISIONS.md
In the eight command skills (the name changes), writing down the pick above:
In
packages/skill-branches/DECISIONS.md, the bullet about the command link, rewritten: