atxtechbro / atxtechbro/dotfiles

retro: Fix PR #600 - only add permissions we actually use

Open
#602 0 comments 0 reactions 0 assignees View on GitHub
claude-code permissions retro
Dominant language
Shell
Stars
27
Forks
2
PR merge metrics
No merged PRs in 30d

Description

## Retro Findings from PR #600

The implementation added too many speculative permissions instead of following tracer bullets approach.

## Problems
1. **Added untested tools** - Added Read, LS, Glob, Grep, Task, TodoRead, etc. without knowing if they actually prompt
2. **No confidence** - "there are many tools here i have never heard of"
3. **Should add one-by-one** - As we hit actual prompts, not speculatively

## Correct Approach
- Start minimal
- Add permissions ONLY when we get prompted
- Test each addition
- Build confidence through actual use

## Action Items
- [ ] Revert to minimal set of KNOWN prompting tools
- [ ] Document which specific prompts we're solving
- [ ] Add new permissions incrementally as needed
- [ ] Each addition should reference the actual prompt that triggered it

## Example of Better Approach
```json
"allow": [
// Only tools we've actually been prompted for:
"mcp__filesystem__list_files", // prompted when doing X
"LS", // prompted when listing directories
// Add more as we discover them
]
```

Related: #601 (documentation bloat)
Principle: tracer-bullets

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by inspecting the current permission configuration and PR #600, then compare the added permissions with the actual prompts they address. Revert to the minimal known set, document each supported prompt, and ensure future additions reference a prompt that triggered them.

Written by the indexing model from the issue text.

Assessment

Tech stack
bash, shell
Domain
tooling
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
30/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.