[codex] add AgentHiFive integration#3144
Open
supersantux wants to merge 11 commits into
Open
Conversation
89c5a79 to
d83b59b
Compare
91d9fe9 to
53005b9
Compare
Add a skill that lets a running nanobot agent create new bot instances through a helper script. The agent collects instance name, channel type, and optional model from the user, then runs the script which: - Calls nanobot onboard to create config + workspace skeleton - Enables the target channel and sets workspace/model in config - Auto-assigns gateway/API ports if defaults are occupied - Validates config via Pydantic before saving - Reports required fields the user needs to fill in (e.g. bot token)
…el reference - Make SKILL.md platform-agnostic (remove Windows-only path rules) - Add 14-channel quick-reference table with required fields - Create references/channels.md with detailed per-channel config - Inherit model from parent config when not explicitly specified - Consolidate duplicate file reads in _patch_config - Add email channel consent_granted field documentation - Fix auto_reply_enabled default value (true, not false) - Add troubleshooting section to SKILL.md
- Revert unrelated docs change (default provider description) - Move olostep import from global scope to lazy import inside method - Revert formatting-only changes (__init__ signature, or "brave" defaults) - Update tests to mock via sys.modules instead of module-level globals
53005b9 to
32181fd
Compare
This was referenced Apr 29, 2026
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.
What changed
This PR adds the initial AgentHiFive integration for NanoBot.
It wires AgentHiFive into NanoBot as a protected MCP-backed backend, adds adapter-owned approval tracking and exact replay, introduces vault-managed Telegram and Slack channels, and extends the setup flow so users can bootstrap AgentHiFive, reconnect it, and toggle supported channels from a dedicated AgentHiFive setup path.
It also includes several integration hardening improvements that came out of end-to-end testing and review:
Why
The goal is to let NanoBot use AgentHiFive-managed connections and approval gates without pushing that complexity onto the user or the model.
This addresses a few concrete issues along the way:
User impact
Users can:
nanobot setup-agenthifiveValidation
Focused local checks run during preparation:
pytest -q tests/agent/test_runner.pypytest -q tests/channels/test_agenthifive_channel.pypytest -q tests/agent/test_agenthifive_auth.py tests/agent/test_agenthifive_hook.pypytest -q tests/cli/test_commands.py -k 'gateway_health_endpoint_binds_and_serves_expected_responses or serve_uses_api_config_defaults_and_workspace_override or setup_agenthifive or setup_channels_command_runs_channel_setup'Those focused checks passed locally.