Skip to content

feat: Add WebSearch to always-allowed permissions for installed repos#174

Merged
nhorton merged 5 commits intomainfrom
add-websearch-permission
Jan 27, 2026
Merged

feat: Add WebSearch to always-allowed permissions for installed repos#174
nhorton merged 5 commits intomainfrom
add-websearch-permission

Conversation

@nhorton
Copy link
Contributor

@nhorton nhorton commented Jan 27, 2026

Summary

  • Adds WebSearch to the required permissions for installed repos
  • Refactors permission loading to use a template file instead of hardcoded Python

Changes

  1. New file: src/deepwork/templates/claude/settings.json - Contains permissions in a structure that mirrors the target settings.json for easy merging
  2. Refactored: ClaudeAdapter.sync_permissions() now loads permissions from the template file
  3. Updated tests: Dynamically read expected permissions from the template file

Benefits

  • Permissions are now declarative configuration rather than code
  • Easier to review/audit what permissions DeepWork requests
  • Adding new permissions only requires editing the JSON file

Test plan

  • All 51 adapter tests pass
  • Permissions are correctly loaded from template file
  • Tests dynamically verify all permissions from the template

🤖 Generated with Claude Code

When DeepWork is installed into a user's repository, WebSearch is now
automatically added to the permissions.allow list in settings.json.
This enables research-based jobs to use web search without permission prompts.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@nhorton nhorton force-pushed the add-websearch-permission branch from c7594b5 to 32e6fcc Compare January 27, 2026 20:28
nhorton and others added 4 commits January 27, 2026 13:40
- Add templates/claude/settings.json with permissions structure that mirrors
  the target settings.json format for easy merging
- Update ClaudeAdapter to load permissions from the template file instead of
  hardcoding them in Python
- Update tests to dynamically read expected permissions from the template file

This makes permissions declarative configuration rather than code, making it
easier to review and modify what permissions DeepWork requests.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Add assertion that permissions count > 2 to ensure template is being read
- Remove changelog entries since these are internal changes, not user-facing

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@nhorton nhorton merged commit 98b823b into main Jan 27, 2026
4 checks passed
@nhorton nhorton deleted the add-websearch-permission branch January 27, 2026 21:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant