Search with ripgrep and bundle only the coinflip plugin#21
Closed
HiLleywyn wants to merge 1 commit into
Closed
Conversation
The agent shell tool now prefers rg for content searches, keeping grep, egrep and fgrep on the allowlist only as a fallback for when ripgrep cannot serve. The Docker image installs ripgrep so rg is on PATH. Only the coinflip worked example ships bundled now. The notes, tasks, events and groups productivity suite moves to install-only marketplace plugins, and the docs, help text and smoke tests follow.
Owner
Author
|
Superseded by #22, reopened on the renamed branch Generated by Claude Code |
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
2 participants
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.
Summary
shell.runtool now prefers ripgrep (rg) for content searches.grep,egrepandfgrepstay on the allowlist only as a fallback for the rare case ripgrep fails or cannot serve. The tool description tells the model to reach forrgfirst, and the Docker image installsripgrepsorgis onPATH.coinflipworked example ships bundled now. Thenotes,tasks,eventsandgroupsproductivity suite is removed fromplugins/and becomes install-only marketplace plugins (still available via.ai plugins install).README.md,plugins/README.mdand the in-bot/helpcatalogue no longer present the productivity suite as built in.Changes
ai/workspace.py--rgadded to the shell allowlist.ai/tools.py--shell.rundescription steers the model to ripgrep first, grep as fallback.Dockerfile-- installsripgrep.plugins/--events.lua,groups.lua,notes.lua,tasks.luaremoved;coinflip.luakept.README.md,plugins/README.md,cogs/meta.py-- docs and help reflect the new default plugin set.tests/test_smoke.py-- bundled-plugin assertions updated; the command-tree test no longer depends on a removed file.Test plan
python -m pytest tests/-- 148 passedrgruns through the sandboxedshell.runtool and thatgrep/egrep/fgrepremain available as a fallbackGenerated by Claude Code