Repository navigation
Document agent.tools, the switches that turn off the agent's own tool groups, and get_agent_config - #795
Document agent.tools, the switches that turn off the agent's own tool groups, and get_agent_config#795DavidCockerill wants to merge 2 commits into
Conversation
… groups, and get_agent_config Documents HarperFast/harper#3993: `agent.tools.filesystem`, `agent.tools.schedule`, `agent.tools.inspector` and `agent.tools.bestPractices` remove a built-in agent tool group at startup, with the group's names kept reserved; `set_agent_config` rejects them; the new `get_agent_config` operation returns the configuration in effect and is outside the `agent` permission group and MCP, like `set_agent_config`. Co-Authored-By: Claude Fable 5.1 <[email protected]>
…veat for agent.tools, and say which get_agent_config keys are optional Co-Authored-By: Claude Fable 5.1 <[email protected]>
There was a problem hiding this comment.
Code Review
This pull request updates the documentation to introduce a new feature allowing users to turn off specific tool groups for the built-in Harper agent at startup (using agent.tools configuration for filesystem, schedule, inspector, and bestPractices). It also documents a new get_agent_config operation that retrieves the active agent configuration, including startup-only settings, and details how these changes affect roles, permissions, and MCP access. I have no feedback to provide as there are no review comments.
🚀 Preview DeploymentYour preview deployment is ready! 🔗 Preview URL: https://preview.harper-documentation.harperfabric.com/pr-795 This preview will update automatically when you push new commits. |
|
Closing: the reference docs are moving out of this repo (see #788), so the agent.tools and get_agent_config documentation for HarperFast/harper#3993 will land with the moved pages instead of here. The branch stays available for reuse. — written by Claude Fable 5.1 on David's behalf |
🧹 Preview CleanupThe preview deployment for this PR has been removed. |
📝 Summary
The built-in agent's tool groups (filesystem,
schedule_followup, the V8 inspector, the best-practices lookup) could not be turned off. Nothing returned the agent configuration in effect. HarperFast/harper#3993 adds theagent.toolsswitches and theget_agent_configoperation. This documents both using the harper branchclaude/3993-agent-tool-switches-9d827b(PR link to follow). Every behavior claim was checked against that branch's source and tests. Version badges arev5.4.0: the feature ships in the open 5.4 line.🔧 Changes
reference/configuration/options.md:toolsjoins theagentoption list. The runtime settings description lists it among the startup-only keys. A new "Turning off tool groups" section gives the YAML, a table of what each key removes, and these behaviors: a disabled group is removed rather than gated, and its names stay reserved; the keys apply only at startup. Set them through the config file,set_configuration,HARPER_SET_CONFIG, env vars or CLI flags.set_agent_configrejects them. ThehttpFetchcaveat about the agent re-enabling tools also applies; and a malformed value turns groups off rather than on. The agent introduction links to the section.reference/operations-api/operations.md: documents seven operations, two outside theagentgroup. The privilege warning says each group running at process privilege can now be removed at startup. Addsget_agent_configto the operation table.set_agent_configliststoolsamong the startup-only keys it rejects. Aget_agent_configsection covers the request, an example response, optional keys, and exclusion from theagentpermission group and MCP.reference/users-and-roles/overview.md: theagentgroup excludesget_agent_configtoo. Narrows the claim about what the group withholds (session reads still show a session'smodelandprovider).release-notes/v5-lincoln/5.4.md: a "Turning Off the Agent's Own Tool Groups" entry under Built-in Agent.✅ Verification
npm run format:checkandnpm run lintpass.get_agent_configkeys), then one delta round.npm run buildsucceeds, so every new link and anchor resolves.Refs HarperFast/harper#3993 · Refs #674 · Feature PR: the harper PR for HarperFast/harper#3993, linked here once it opens
🤖 Generated with Claude Code
Related PRs: #788 overlaps (it re-sources these four pages from the product repos; whichever lands second carries this delta into the assembled pages), 3 others independent
Complexity: easy
Review-Coverage: authored=claude; ran=gemini,codex; adjudicated=domain; blocked=muse(failed); declined=cursor-grok,cursor-composer; rounds=2; full=1 @ 8befb4f
Review-Attention: skim ~2m (decisions: malformed-tools-fail-closed-vs-fail-start, get-agent-config-outside-agent-group, drop-injected-tools-from-all-off-claim) @ 8befb4f