Skip to content

docs: scrub local directory paths from CLAUDE.md - #23

Merged
jmc-wander merged 1 commit into
mainfrom
chore/scrub-local-paths-and-key-names
Oct 3, 2026
Merged

jmc-wander merged 1 commit into
mainfrom
chore/scrub-local-paths-and-key-names

Conversation

@jmc-wander

@jmc-wander jmc-wander commented Oct 3, 2026 •

Copy link
Copy Markdown
Contributor

Summary

Removes local directory paths from CLAUDE.md:

  • The Quick Reference cd now uses path/to/pact instead of a local checkout path.
  • Removed a pointer to a personal note vault that only exists on one machine.

This PR changes documentation only. No code changed and no personal key names were found.

Test plan

  • Re-scanned for personal key names and local path patterns: no hits remain
  • Repo CI (ci.yml) runs on this PR

🤖 Generated with Claude Code

https://claude.ai/code/session_01C2cg2ZsxSjpJAXB9YB7sYn


View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need. Autofix is disabled.

Replace a checkout path with a generic placeholder and drop a pointer to
a personal note vault that has no meaning outside the owner's machine.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01C2cg2ZsxSjpJAXB9YB7sYn

@adaptcom adaptcom Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Confidence Score: 5/5

Summary

Replaces machine-specific documentation paths with portable guidance and removes an inaccessible personal-vault reference. Documentation-only risk is minimal, and CI passed.

Important Files Changed

File Overview
CLAUDE.md Scrubs developer-local paths while preserving shared setup and Kindex guidance

↻ Re-run review · View in Adapt

@jmc-wander
jmc-wander merged commit f27b38e into main Oct 3, 2026
3 checks passed
@jmc-wander
jmc-wander deleted the chore/scrub-local-paths-and-key-names branch October 4, 2026 17:46
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.

2 participants