fix(launchpad): elide a contributor's home directory from note 315 (#385) - #505
Open
serina-mcfall wants to merge 1 commit into
Open
fix(launchpad): elide a contributor's home directory from note 315 (#385)#505serina-mcfall wants to merge 1 commit into
serina-mcfall wants to merge 1 commit into
Conversation
) `du -sh` output in launchpad/Research/315-desktop-stdout-destination.md carried an absolute /Users/<name>/ path, publishing a contributor's local username in a public repository. It has been merged for some time, so the value is already public; this stops it persisting. Elided to `~`, which is the convention the note already uses in its own prose and in the adjacent `ls` command, so nothing it demonstrates changes -- 96K in that location still reads the same. The substitution is disclosed inline rather than made silently, because a reader would otherwise take `~` for literal `du` output. Closes #385 Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01LuDy9K8WDtowz5e2Uz2wjz Signed-off-by: Serina Mcfall <serina.mcfall@gmail.com>
serina-mcfall
marked this pull request as ready for review
August 23, 2026 20:02
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
launchpad/Research/315-desktop-stdout-destination.mdline 154 pasteddu -shoutput containing an absolute/Users/<name>/…path, publishing a contributor's local username in a public repository. This elides the home-directory prefix to~.Related issue
Closes #385
Issue type
Bug
Agent provenance
Observed behaviour
The note has been merged for some time, so the value is already public. This stops it persisting rather than containing it — there is nothing to contain.
Expected behaviour
No tracked file under
launchpad/carries a contributor's home-directory path.The change
One line, plus a one-line disclosure:
/Users/<name>/Library/…→~/Library/…~is the convention this note already uses — in its own prose ("resolves on macOS to~/Library/Application Support/xyz.block.buzz.app/agents/logs/") and in the very next command in the same block (ls ~/Library/Application\ Support/…). So the elision makes line 154 consistent with its neighbours rather than introducing a new style.Nothing the note demonstrates changes: 96K in that location still reads the same, and the directory is still identified unambiguously.
Why the substitution is disclosed rather than silent
The block is presented as a pasted terminal transcript.
dudoes not print~. Editing a transcript to look like output that was never produced is the same defect class this cohort's reviews keep filing against research notes — so the note now says what was changed, in one line, at the point of the change.Verification
Command run:
Raw output:
Scope note — this is one of two
I scanned every research note merged on
launchpad:315 is the only merged note affected, and this PR closes it.
A second instance exists in
launchpad/Research/345-test-output-disclosure.md, which is not merged — it is in PR #439, currently blocked. It is raised in that PR's review and is not fixed here, because fixing it on this branch would touch a file another open PR owns. Worth noting that #345 is itself a note about output disclosure, which is where the pattern is easiest to miss.Not verified
The scan pattern only catches
/Users/<name>/and/home/<name>/. It would not catch a Windows profile path (C:\Users\<name>), a bare username appearing outside a path, or a hostname. Those were not searched for here.🤖 Drafted by Claude Code (
claude-opus-5) for @serina-mcfall.