Skip to content

fix(sidenote): park notes in .git/info, not a tracked file at the project root - #36

Merged
pallaoro merged 1 commit into
mainfrom
sidenotes-are-added-to-the
Sep 2, 2026
Merged

fix(sidenote): park notes in .git/info, not a tracked file at the project root#36
pallaoro merged 1 commit into
mainfrom
sidenotes-are-added-to-the

Conversation

@pallaoro

@pallaoro pallaoro commented Sep 2, 2026

Copy link
Copy Markdown
Member

/sidenote wrote to .claude/sidenotes.md at the project root, but only this
repo's own .gitignore excluded it — the installer never propagated that guard,
so in every consumer repo a git add -A swept personal parked thoughts into
unrelated PRs. And a gitignored file inside a worktree dies with the worktree,
which breaks the "survives this session" contract for worktree-based flows.

Write to $(git rev-parse --path-format=absolute --git-common-dir)/info/sidenotes.md
instead: git's own home for local, never-committed, per-repo data. No ignore
rule needed, one file shared by all worktrees, survives worktree removal.
Fallback outside a git repo stays .claude/sidenotes.md. The flush also reads a
legacy root file if present so old entries aren't silently dropped. README
synced.

…ject root

/sidenote wrote to .claude/sidenotes.md at the project root, but only this
repo's own .gitignore excluded it — the installer never propagated that guard,
so in every consumer repo a `git add -A` swept personal parked thoughts into
unrelated PRs. And a gitignored file inside a worktree dies with the worktree,
which breaks the "survives this session" contract for worktree-based flows.

Write to $(git rev-parse --path-format=absolute --git-common-dir)/info/sidenotes.md
instead: git's own home for local, never-committed, per-repo data. No ignore
rule needed, one file shared by all worktrees, survives worktree removal.
Fallback outside a git repo stays .claude/sidenotes.md. The flush also reads a
legacy root file if present so old entries aren't silently dropped. README
synced.
@pallaoro
pallaoro merged commit 8cca942 into main Sep 2, 2026
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