Repository navigation
Bug: committed-host-identity-leaks #728
Description
- Work Mode: full-bug
Summary
Two consolidated findings: a committed project file and five committed agent-memory files leak a real developer account name and/or employer name into version control. Consolidated into one issue rather than two, since both are the same class of defect (plain-text identity leakage in a tracked file, not a runtime/log leak) and fixing them is a single coordinated sweep, not two independent efforts.
Environment
- OS/version: Windows 11 Pro (repo default)
- Python version: n/a - repo-wide, spans a
.csprojand Markdown agent-memory files - Command/flags used: n/a - findings are from direct
git grep/git showinspection - Data source or fixture: n/a
Steps to Reproduce
Not applicable - both findings are static content inspections. See "Actual Behavior."
Expected Behavior
No committed, tracked file should contain a real account name, machine name, or employer name in plain text. This is already an established repo-wide convention (see .claude/agent-memory/_shared_no_absolute_host_paths.md) that these two findings violate.
Actual Behavior
1. TaskMaster/TaskMaster.csproj:37 commits a personal publish path. <PublishUrl>C:\Users\DanMoisan\OneDrive - The Real Good Food Company\TM\</PublishUrl> - both the developer's Windows account name and their employer's organization name (via a OneDrive commercial folder name) are committed to version control, visible in every clone, the public GitHub UI, and the full history. This is the same class of leak already tracked by issue #602 (scoped to runtime exception messages), but worse in durability: a log message exists on one machine, while a committed project file republishes the identifiers to every reader indefinitely, and rewriting it out of history is disruptive in a way fixing a log message is not. (Source: #628.)
2. Five .claude/agent-memory/** files contain the real account name and/or machine name in plain text, confirmed by direct git grep against origin/main:
.claude/agent-memory/epic-orchestrator/feedback_measure_whole_volume_before_blaming_worktrees.md.claude/agent-memory/feature-review/project_464-review-residuals.md.claude/agent-memory/feature-review/project_488-review-residuals.md.claude/agent-memory/orchestrator/angle-bracket-redaction-breaks-trx-xml.md.claude/agent-memory/orchestrator/collect-pr-context-lands-in-main-checkout.md
These files should follow the same no-absolute-host-paths convention already documented and applied elsewhere in this same memory tree. (Source: #685.)
Note: issue #671 (TRX evidence-hygiene sweep producing malformed XML as a side effect of host-token redaction) is a related but mechanically distinct defect - a tool that redacts identity information incorrectly, rather than un-redacted identity information sitting in a file. Deliberately left standalone rather than folded in here, since #727 (filed earlier this session) already cross-references #671 by number as the umbrella tracker for that specific sweep-tool defect.
Logs / Screenshots
- Attached minimal logs or screenshot
- Snippet: n/a - see file/line citations above, each confirmed directly against
origin/mainon 2026-09-02.
Impact / Severity
- Blocker
- High
- Medium
- Low
Medium: no functional defect, but a real, durable, publicly-visible privacy/identity leak in a repository history that (per #628's own text) is disruptive to remove retroactively - the cost of delay compounds, unlike a log message.
Source
From: docs/features/potential/2026-09-02-committed-host-identity-leaks.md
Activity
docs(potential): record promoted records for consolidated issues #728-…
drmoisan commentedon Sep 28, 2026
Verified fixed by PR #881 (item #873, commit aa6bf5f).
TaskMaster/TaskMaster.csproj:37is now<PublishUrl>publish\</PublishUrl>, and the five named agent-memory files contain no account or host token. Other.claude/**files have leaked the account name since this issue was filed; that push-down-owned portion is tracked upstream in #932. Closing as fixed by #881.