Skip to content

chore(privacy): stop publishing absolute home paths - #222

Merged
saagpatel merged 2 commits into
masterfrom
chore/redact-home-paths
Aug 11, 2026
Merged

chore(privacy): stop publishing absolute home paths#222
saagpatel merged 2 commits into
masterfrom
chore/redact-home-paths

Conversation

@saagpatel

Copy link
Copy Markdown
Owner

Replaces absolute home paths with ~ in tracked files, so the repository stops
publishing the operator's macOS account name and local directory layout.

This is one of 31 identical branches across the portfolio. Every one of them was
committed and pushed but none was ever opened as a pull request, so the paths
stayed on the default branch and stayed public. Merging this is what actually
removes them.

No credential, token, key material or personal name was exposed. What leaks is
the account name and the shape of the directories under it, which is a
fingerprinting signal rather than a breach.

Some occurrences are deliberately left in place, and should not be redacted in a
follow-up:

  • tests that assert a named file does not contain the path. The literal
    there is the needle the test searches for, not a leak.
  • sandbox-escape tests that attempt to read or write a real absolute host path.
    Redacting the path removes the thing the test proves.
  • sealed evidence artifacts pinned by a SHA-256 contract test. A text edit
    invalidates the proof rather than redacting it; those need a re-run that
    produces a fresh receipt, not a substitution.

Test fixtures that must stay a detectable private path use /Users/example/...
rather than ~, because the validator they exercise keys on /users|home|root/
and not on the account name. That keeps each assertion meaningful while naming
nobody.

github-actions Bot and others added 2 commits August 11, 2026 05:48
Removes 146 occurrences of the operator's absolute home path from documentation, prose and shell defaults in this public repo.
Functional config, test fixtures and generated files are untouched: $HOME is not expanded by JSON/YAML parsers and tests may assert the literal path. 1 such occurrences remain and need per-file review.
The earlier commit on this branch redacted documentation and stopped there.
Code, configuration, test fixtures and generated run artifacts still carried the
absolute home path, which is what the published branch actually exposes.

Substitutions used, and why they differ:
  ~                    prose, shell command strings, and opaque fixture values,
                       none of which resolve the path
  /Users/example/...   fixtures that must stay a detectable private path. The
                       portfolio-truth validator keys on /users|home|root/, not
                       on the account name, so an anonymous account keeps every
                       assertion meaningful while naming nobody

Left alone deliberately, because each is a test that enforces the very thing
this branch is doing and redacting it would remove what the test proves:
  package-surface.test.ts asserts five named operator surfaces do not contain
    the path; the literal there is the needle, not a leak
  the sealed local-execution evidence is SHA-256 pinned by its own contract test,
    so a text edit invalidates the proof rather than redacting it. That one needs
    a re-qualification run, not a substitution.

@chatgpt-codex-connector chatgpt-codex-connector 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: efef96dea4

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment on lines +11 to +12
- [src-tauri/src/commands/mod.rs](~/AssistSupport/src-tauri/src/commands/mod.rs) had grown into a multi-thousand-line command hotspot.
- [src-tauri/src/lib.rs](~/AssistSupport/src-tauri/src/lib.rs) still owned one giant `generate_handler!` block, so registration and permission coverage depended on the same brittle file shape.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Replace tilde link targets with repository-relative paths

When these Markdown files are viewed on GitHub or another renderer, ~ is not expanded to the reader's home directory; it is treated as a literal relative URL component, so these links resolve under a nonexistent path such as docs/adr/~/AssistSupport/.... The same replacement breaks the source and evidence links throughout the changed documentation; use paths relative to each document or repository-root URLs instead.

Useful? React with 👍 / 👎.

@saagpatel
saagpatel merged commit 17bea40 into master Aug 11, 2026
15 of 19 checks passed
@saagpatel
saagpatel deleted the chore/redact-home-paths branch August 11, 2026 20:02
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