Skip to content

feat: Reorganize settings and add permissions viewer#1165

Merged
charlesvien merged 4 commits intomainfrom
03-10-reorganize_settings_and_add_permissions_viewer
Mar 10, 2026
Merged

feat: Reorganize settings and add permissions viewer#1165
charlesvien merged 4 commits intomainfrom
03-10-reorganize_settings_and_add_permissions_viewer

Conversation

@charlesvien
Copy link
Member

@charlesvien charlesvien commented Mar 10, 2026 •
edited
Loading

  1. Add getClaudePermissions tRPC endpoint to read allow/deny rules from ~/.claude/settings.json
  2. Create PermissionsSettings component showing allowed and denied tool permissions as color-coded badges
  3. Move "Keep awake" setting from Claude Code tab to General tab under new "Power" section
  4. Reorganize Claude Code settings into "Extensions" and "Permissions" sections with border-separated headers
  5. Apply consistent noBorder on last-in-section rows and update section header styling

Screenshot 2026-03-10 at 11.24.24 AM.png

@charlesvien charlesvien mentioned this pull request Mar 10, 2026
Closed
@charlesvien Graphite App
Copy link
Member Author

charlesvien commented Mar 10, 2026 •
edited
Loading

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

This was referenced Mar 10, 2026
Merged
Merged
@charlesvien charlesvien changed the title Reorganize settings and add permissions viewer feat: Reorganize settings and add permissions viewer Mar 10, 2026
@charlesvien charlesvien force-pushed the 03-10-reorganize_settings_and_add_permissions_viewer branch from 9e5d696 to e6250ea Compare March 10, 2026 16:34
@charlesvien charlesvien force-pushed the 03-09-sync_claude-agent-acp_upstream_v0.20.2_v0.21.0 branch from 60fe768 to 5b87c5a Compare March 10, 2026 16:34
This was referenced Mar 10, 2026
Merged
Merged
Merged
Merged
Closed
@charlesvien charlesvien marked this pull request as ready for review March 10, 2026 18:32
@charlesvien charlesvien force-pushed the 03-10-reorganize_settings_and_add_permissions_viewer branch from ed74519 to b9a6747 Compare March 10, 2026 18:33
k11kirky
k11kirky approved these changes Mar 10, 2026
View reviewed changes
Copy link
Contributor

@k11kirky k11kirky left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@charlesvien Graphite App
Copy link
Member Author

charlesvien commented Mar 10, 2026 •
edited
Loading

Merge activity

  • Mar 10, 8:28 PM UTC: A user started a stack merge that includes this pull request via Graphite.
  • Mar 10, 8:30 PM UTC: Graphite rebased this pull request as part of a merge.
  • Mar 10, 8:36 PM UTC: @charlesvien merged this pull request with Graphite.

@charlesvien charlesvien changed the base branch from 03-09-sync_claude-agent-acp_upstream_v0.20.2_v0.21.0 to graphite-base/1165 March 10, 2026 20:28
@charlesvien charlesvien changed the base branch from graphite-base/1165 to main March 10, 2026 20:29
@charlesvien charlesvien force-pushed the 03-10-reorganize_settings_and_add_permissions_viewer branch from b9a6747 to dbfeea8 Compare March 10, 2026 20:30
@charlesvien charlesvien merged commit b499906 into main Mar 10, 2026
15 checks passed
@charlesvien charlesvien deleted the 03-10-reorganize_settings_and_add_permissions_viewer branch March 10, 2026 20:36
@charlesvien charlesvien mentioned this pull request Mar 10, 2026
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@k11kirky k11kirky k11kirky approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@charlesvien @k11kirky