Skip to content

Add Claude rules explicitly denying access to likely-sensitive files - #84

Merged
briskt merged 1 commit into
mainfrom
feature/add-claude-constraints
Oct 9, 2026
Merged

briskt merged 1 commit into
mainfrom
feature/add-claude-constraints

Conversation

@forevermatt

Copy link
Copy Markdown
Contributor

Added

  • Add Claude rules explicitly denying access to likely-sensitive files

PR checklist:

  • Add/update test(s) for any new/changed actions
  • Bump version number of any changed actions
  • Update README.md

I have my system-wide CLAUDE.md file check for this file each time I start work on a folder, to ensure at least this minimal level of protection is in place against Claude reading credentials. I'm glad to share the entry I have in my CLAUDE.md to make this happen, if anyone is interested.

@forevermatt
forevermatt requested a review from a team as a code owner October 5, 2026 16:34
@forevermatt
forevermatt requested review from ethancanne, hobbitronics, jason-jackson and mtompset and removed request for a team October 5, 2026 16:34
@forevermatt

Copy link
Copy Markdown
Contributor Author

I added this as I was starting to set up to have Claude upgrade nodemailer (per the other PR on this repo), but I'm not sure it's worth the time, given that Pipedream is shutting down workflows. Thoughts?

@jason-jackson jason-jackson left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Likely not needed, but if we port it to another service, could remind us to have something there.

@briskt
briskt merged commit 184968a into main Oct 9, 2026
5 checks passed
@briskt
briskt deleted the feature/add-claude-constraints branch October 9, 2026 04:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants