Consolidate agent install flags into --install-agent / --uninstall-agent, add Droid support#96
Open
uepoch wants to merge 3 commits into
Open
Consolidate agent install flags into --install-agent / --uninstall-agent, add Droid support#96uepoch wants to merge 3 commits into
uepoch wants to merge 3 commits into
Conversation
Consolidate the eight --install-<agent> / --uninstall-<agent> boolean flags into two value-accepting flags that take the agent name: colgrep --install-agent <opencode|claude|codex|hermes> colgrep --uninstall-agent <opencode|claude|codex|hermes> The agent names are backed by a clap ValueEnum, so invalid values are caught at parse time. Dispatch functions in install/mod.rs map enum variants to the existing per-agent install/uninstall functions. Old flags (--install-opencode, --install-claude-code, etc.) are kept as hidden backward-compatible aliases that don't appear in --help output. Success messages now reference the new flag names.
Install colgrep as a skill for Factory's Droid agent at ~/.factory/skills/colgrep/SKILL.md, following Droid's skill-based extension model rather than the AGENTS.md injection used by other agents. The skill includes YAML frontmatter with name/description so the Droid can auto-discover and invoke it, and the same colgrep usage reference shared across all agents.
Author
|
Not sure if I can fix the |
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
1 participant
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
Unified
--install-agent/--uninstall-agentflagsAll agent install and uninstall flags now go through two value-accepting flags backed by a
clap::ValueEnum. Invalid agent names are caught at parse time with a clear error listing valid options.Droid (Factory) integration
Installs colgrep as a proper Factory skill at
~/.factory/skills/colgrep/SKILL.mdwith YAML frontmatter, following Droid's skill discovery model. The Droid auto-discovers and invokes the skill when code search is needed, and users can also trigger it with/colgrep. Uninstall removes the skill directory entirely.Backward compatibility
--install-opencode,--install-claude-code,--install-codex,--install-hermesand their uninstall counterparts all continue to work but are hidden from--help.If required, I can clear the old code, but I wanted to avoid breaking existing flows