Skip to content

feat(openclaw): gate agent-visible tools#2566

Open
Mijamind719 wants to merge 1 commit into
volcengine:mainfrom
Mijamind719:codex/openclaw-tool-governance
Open

feat(openclaw): gate agent-visible tools#2566
Mijamind719 wants to merge 1 commit into
volcengine:mainfrom
Mijamind719:codex/openclaw-tool-governance

Conversation

@Mijamind719

Copy link
Copy Markdown
Collaborator

Summary

  • add OpenViking tool allowlist/blocklist config for agent-visible tools
  • hide add_resource by default while keeping /add-resource command available
  • expose enableAddResourceTool for explicit opt-in imports

Testing

  • cd examples/openclaw-plugin && npm test
  • cd examples/openclaw-plugin && npm run typecheck
  • cd examples/openclaw-plugin && npm run build

Stacked PR plan: this is PR 1/5 for splitting #2386 into reviewable core features.

@github-actions

Copy link
Copy Markdown

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

🎫 Ticket compliance analysis ✅

2386 - Fully compliant

Compliant requirements:

  • Added enableAddResourceTool, enabledTools, disabledTools config options - add_resource is disabled by default and only registered when enableAddResourceTool=true - Implemented tool gating with registerOpenVikingTool wrapper - Added comprehensive tests for config parsing and tool registration
⏱️ Estimated effort to review: 2 🔵🔵⚪⚪⚪
🏅 Score: 95
🧪 PR contains tests
🔒 No security concerns identified
✅ No TODO sections
🔀 No multiple PR themes
⚡ No major issues detected

@github-actions

Copy link
Copy Markdown

PR Code Suggestions ✨

No code suggestions found for the PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Backlog

Development

Successfully merging this pull request may close these issues.

1 participant