Skip to content

Add Read, Glob, and Grep tool permissions to D4D Assistant#72

Merged
justaddcoffee merged 1 commit intomainfrom
fix/add-read-glob-grep-permissions
Nov 7, 2025
Merged

Add Read, Glob, and Grep tool permissions to D4D Assistant#72
justaddcoffee merged 1 commit intomainfrom
fix/add-read-glob-grep-permissions

Conversation

@justaddcoffee
Copy link
Collaborator

Summary

Adds essential file operation tools to the D4D Assistant's allowed tools list.

Problem

The D4D Assistant was getting permission errors in GitHub Actions:

Claude requested permissions to read from /tmp/ai-input/user-prompt.txt, 
but you haven't granted it

This happened because the Read tool wasn't in the allowed tools list.

Solution

Added three fundamental tools to claude-allowed-tools:

  • Read - Read files from the repository and system (including /tmp/ai-input/user-prompt.txt)
  • Glob - Find files by pattern when exploring the codebase
  • Grep - Search file contents during metadata extraction

Why These Are Needed

  • Read: Required to read the user's prompt and any files in the repository
  • Glob: Helps find schema files, examples, and other resources by pattern
  • Grep: Searches for specific content when extracting metadata

Without these tools, the assistant cannot perform basic file operations needed for D4D datasheet creation.

Related

🤖 Generated with Claude Code

The D4D Assistant needs these fundamental tools to operate:
- Read: Read files from the repository and /tmp/ai-input/user-prompt.txt
- Glob: Find files by pattern when exploring the codebase
- Grep: Search file contents for metadata extraction

Without Read permission, the assistant cannot even read the user's
prompt from /tmp/ai-input/user-prompt.txt, causing permission errors
in GitHub Actions.

This fixes the permission error:
'Claude requested permissions to read from /tmp/ai-input/user-prompt.txt,
but you haven't granted it'

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
@justaddcoffee justaddcoffee merged commit 8c7d889 into main Nov 7, 2025
4 checks passed
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