Skip to content

feat(me): pre-allow ~/.claude/handoff/ in setup template - #638

Merged
baleen37 merged 1 commit into
mainfrom
feat/me-setup-handoff-dir
May 11, 2026
Merged

feat(me): pre-allow ~/.claude/handoff/ in setup template#638
baleen37 merged 1 commit into
mainfrom
feat/me-setup-handoff-dir

Conversation

@baleen37

@baleen37 baleen37 commented Apr 29, 2026

Copy link
Copy Markdown
Owner

Summary

  • Add ~/.claude/handoff/ to permissions.additionalDirectories in the me:setup skill's settings.json template
  • Prevents the .claude/ sensitive-directory permission prompt from firing when me:handoff writes session handoff files

Test plan

  • Run me:setup on a fresh machine and confirm the new entry lands in ~/.claude/settings.json
  • Trigger me:handoff and confirm no permission prompt appears for ~/.claude/handoff/

Summary by CodeRabbit

  • Bug Fixes
    • Expanded directory access permissions to support additional functionality.

Add ~/.claude/handoff/ to permissions.additionalDirectories so the
me:handoff skill can write without triggering a per-session permission
prompt for the .claude/ sensitive-directory check.
@coderabbitai

coderabbitai Bot commented Apr 29, 2026

Copy link
Copy Markdown

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: ebecb278-2a4c-4c12-9eee-b22533aa1ad8

📥 Commits

Reviewing files that changed from the base of the PR and between dd58918 and 7883c0e.

📒 Files selected for processing (1)
  • plugins/me/skills/setup/settings.json

📝 Walkthrough

Walkthrough

The pull request adds a single configuration entry to the permissions settings, specifying ~/.claude/handoff/ as an additional directory. This enables directory-based permission handling for the handoff functionality without modifying existing allow/deny rules or default modes.

Changes

Cohort / File(s) Summary
Permissions Configuration
plugins/me/skills/setup/settings.json
Added additionalDirectories entry under permissions to include ~/.claude/handoff/ for directory access control.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Poem

🐰 A path through the woods, now granted access,
~/.claude/handoff/ emerges from darkness,
One line of permission, simple and pure,
Makes delegation safe, makes handoffs secure.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and specifically describes the main change: adding ~/.claude/handoff/ to the setup template's permissions.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/me-setup-handoff-dir

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share
Review rate limit: 0/1 reviews remaining, refill in 60 minutes.

Comment @coderabbitai help to get the list of available commands and usage tips.

@baleen37
baleen37 merged commit 261ae96 into main May 11, 2026
5 checks passed
@baleen37
baleen37 deleted the feat/me-setup-handoff-dir branch May 11, 2026 09:20
baleen-release-app Bot pushed a commit that referenced this pull request May 11, 2026
# [17.2.0](v17.1.2...v17.2.0) (2026-05-11)

### Features

* **me:** pre-allow ~/.claude/handoff/ in setup template ([#638](#638)) ([261ae96](261ae96))
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