Skip to content

fix: excessive permissions prompts#70

Merged
scode merged 1 commit intomainfrom
fix-permission-paths
Mar 1, 2026
Merged

fix: excessive permissions prompts#70
scode merged 1 commit intomainfrom
fix-permission-paths

Conversation

@scode
Copy link
Copy Markdown
Owner

@scode scode commented Mar 1, 2026

Claude Code's permissions.allow uses gitignore-style path matching:
/path is project-relative, //path is absolute, ~/path is
home-relative. The setup instructions were emitting bare absolute paths
like /Users/alice/.leiter/soul.md, which Claude Code interpreted as
project-relative — so the permission never matched and the user was
always prompted.

Add permission_path() which formats paths as ~/... (under $HOME)
or //... (elsewhere), and use it in setup/teardown instructions.

changelog: include

Copy link
Copy Markdown
Owner Author

scode commented Mar 1, 2026

This stack of pull requests is managed by Graphite. Learn more about stacking.

@scode scode force-pushed the fix-permission-paths branch from db6f2d9 to 1c7986c Compare March 1, 2026 03:04
@scode scode force-pushed the fix-permission-paths branch from 1c7986c to 61f4c53 Compare March 1, 2026 03:08
@scode scode changed the title fix: use Claude Code permission-compatible path format fix: permissions installed were incorrect Mar 1, 2026
@scode scode changed the title fix: permissions installed were incorrect fix: excessive permissions prompts Mar 1, 2026
@scode scode force-pushed the fix-permission-paths branch from 61f4c53 to f019221 Compare March 1, 2026 03:15
@scode scode force-pushed the fix-permission-paths branch from f019221 to 2022e90 Compare March 1, 2026 03:20
Copy link
Copy Markdown
Owner Author

scode commented Mar 1, 2026

Merge activity

  • Mar 1, 3:24 AM UTC: A user started a stack merge that includes this pull request via Graphite.
  • Mar 1, 3:25 AM UTC: Graphite rebased this pull request as part of a merge.
  • Mar 1, 3:26 AM UTC: @scode merged this pull request with Graphite.

@scode scode changed the base branch from auto-distill to graphite-base/70 March 1, 2026 03:24
@scode scode changed the base branch from graphite-base/70 to main March 1, 2026 03:24
Claude Code's permissions.allow uses gitignore-style path matching:
`/path` is project-relative, `//path` is absolute, `~/path` is
home-relative. The setup instructions were emitting bare absolute paths
like `/Users/alice/.leiter/soul.md`, which Claude Code interpreted as
project-relative — so the permission never matched and the user was
always prompted.

Add `permission_path()` which formats paths as `~/...` (under $HOME)
or `//...` (elsewhere), and use it in setup/teardown instructions.
@scode scode force-pushed the fix-permission-paths branch from 2022e90 to adb471a Compare March 1, 2026 03:25
@scode scode merged commit 1f37239 into main Mar 1, 2026
16 checks passed
@scode scode deleted the fix-permission-paths branch March 4, 2026 03:50
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