Skip to content

Remove personal home paths - #42

Merged
kat3samsin merged 1 commit into
trunkfrom
agent/remove-personal-home-paths
Jul 14, 2026
Merged

Remove personal home paths#42
kat3samsin merged 1 commit into
trunkfrom
agent/remove-personal-home-paths

Conversation

@kat3samsin

Copy link
Copy Markdown
Owner

What changed

  • Derive SkimpyClaw report paths from the current user home directory.
  • Derive the default Obsidian vault path from homedir() while preserving the existing environment override.
  • Remove the personal home directory from the Discord task test fixture.

Why

The public repository contained machine-specific /Users/katre paths. They exposed personal environment details and made the defaults unnecessarily tied to one macOS account.

Impact

Runtime locations remain unchanged for the current user, while clean installations resolve paths for their own account. No tracked source file retains the personal path literal.

Validation

  • pnpm build
  • pnpm test — 83 files, 982 tests passed
  • Focused suites — 3 files, 39 tests passed
  • gitleaks staged-diff scan — clean
  • Claude review — no blockers

Manual testing

  1. Request an existing Chief newspaper and confirm the local artifact link resolves under ~/.skimpyclaw/reports/chief-daily-reader.
  2. Run the Mayora artifact repair path and confirm it uses the current home directory.
  3. Run morning without SKIMPYCLAW_OBSIDIAN_VAULT_ROOT and confirm Daily Note and Digest paths resolve under the current users Obsidian vault.

@kat3samsin
kat3samsin marked this pull request as ready for review July 14, 2026 17:17
@kat3samsin
kat3samsin merged commit 88ffb86 into trunk Jul 14, 2026
6 checks passed
@kat3samsin
kat3samsin deleted the agent/remove-personal-home-paths branch July 14, 2026 17:18
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