Skip to content

Conversation

@jamesbornholt
Copy link

Without this, external_directory permissions prompts, at least for read tools, didn't show a path at all. You could figure it out by seeing which tool usages were pending, but that was annoying especially if it was a different session triggering the prompt. The "always allow" confirmation screen did include the path though.

  • Update bash tool to ask for each external directory individually (consistent with read/edit/patch tools)
  • Populate metadata.parentDir in bash tool's external_directory request
  • Display absolute path for external directories instead of relative

- Update bash tool to ask for each external directory individually
  (consistent with read/edit/patch tools)
- Populate metadata.parentDir in bash tool's external_directory request
- Use metadata.parentDir in permission UI instead of non-existent input().path
- Display absolute path for external directories instead of relative
@github-actions
Copy link
Contributor

github-actions bot commented Jan 8, 2026

The following comment was made by an LLM, it may be inaccurate:

Based on my search, I found one PR that is closely related to PR #7272:

Related PR:

This PR is related as both address improving the clarity and information displayed in external_directory permission prompts, though they may be targeting different tools (write tool vs bash tool).

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