Skip to content
This repository was archived by the owner on Sep 29, 2026. It is now read-only.

Replace private machine and agent names in help text, comments and tests - #186

Merged
myobie merged 1 commit into
mainfrom
agent/private-names
Sep 29, 2026
Merged

myobie merged 1 commit into
mainfrom
agent/private-names

Conversation

@myobie

@myobie myobie commented Sep 29, 2026

Copy link
Copy Markdown
Collaborator

Follows #185, which fixed the README. Running the private-names check (scripts/check-private-names.py from compoundingtech/pty) over this repository found 12 more lines:

  • pty attach|peek|send|list --help examples used a real machine as the --remote peer. They now use example-host, the same text the Rust pty's help fixtures carry.
  • Five comments credited a macOS measurement to a named agent and date. They now say "measured on a Mac" and keep the finding.
  • tests/env-isolation.test.ts and tests/filter.test.ts use example.pty and /home/example.
  • tests/remote-fabric.test.ts names the Linux failure by platform, not by machine.

No behaviour changes. The repository is archived again once this merges, so no CI workflow is added here.

The help examples for attach, peek, send and list used a real machine
as the --remote peer; they now use example-host, which also matches the
Rust pty's help fixtures byte for byte. Comments that credited a
measurement to a named agent on a date now say only where it was
measured. Test data uses example.pty and /home/example.
@myobie
myobie merged commit 9e7e1ac into main Sep 29, 2026
2 checks passed
@myobie
myobie deleted the agent/private-names branch September 29, 2026 17:00
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant