Repository navigation
feat(agents): allowed commands for agy bots, no Claude Code needed, resume checked - #246
Merged
Merged
Conversation
system.status lists enabledAgents. The new-bot dialog offers the agent when there is more than one, with the experimental warning and no model choice for non-Claude agents; the model, effort and context meter under the message box show only for Claude bots.
agy's input takes text blocks only, so images go as the saved file's path (the bot opens it with view_file). The usage of its last response sizes the context meter, estimated by model family, without Compact. bots.agent_model (migration 0027), bots.setAgentModel and agents.models; the app lists the agent's models under the message box, whose names carry the effort.
…esume checked
- bots.allowed_commands (migration 0028) and bots.update { allowedCommands }:
command prefixes written as allow rules in the agy bot's settings; the edit
dialog has the list, with a warning for *.
- A bot that does not run on Claude Code starts without it; the app shows the
'bots cannot start' and sign-in warnings only when a Claude bot exists.
- Resume with --conversation checked with the real agy 1.3.1; an unknown id
starts a new conversation with a warning.
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.
Closes the gaps found while using the agy bots (stacked on #244 and #245; their commits show until those merge).
Commands. In
-pagycannot ask, so everything but Botloft's tools is denied. The owner can list command prefixes per bot (bots.allowed_commands, migration 0028,bots.update { allowedCommands }); they are written ascommand(<prefix>)allow rules at the next start, and the bot restarts to use them. The edit dialog has the list with a plain warning (*allows anything). Only for non-Claude bots.No Claude Code. The supervisor required Claude Code for every bot. Now only Claude bots need it. The app shows 'Bots can't start' and the Claude sign-in warning only when there is a Claude bot (or no bot yet).
Resume. Checked with the real agy 1.3.1:
--conversation <id>in a second process, same home, remembered what the first was told; an unknown id prints a warning and continues in a new conversation, which the daemon stores at the first message.Not changed: the login and usage-limit errors of agy were not provoked (no safe way); they show as a failed turn with agy's text.
Tests:
agy_bot.rs(starts without Claude Code, commands reach the rules),agentBot.test.tsx; fmt, clippy -D warnings,cargo test --workspaceandpnpm checkpass.