Skip to content

Harness: a positional column for sweep-diff, a multi-word operand for the oracle, and a root guard - #158

Open
sadigaxund wants to merge 7 commits into
mainfrom
r12/harness
Open

sadigaxund wants to merge 7 commits into
mainfrom
r12/harness

Conversation

@sadigaxund

Copy link
Copy Markdown
Collaborator

Adds a positional column to xtask sweep-diff, teaches the existence oracle to attest a multi-word operand, and refuses to run as root without --allow-root. The 0.8.0 merge shipped a positional regression that the flag and subcommand columns could not see, so the instrument that is supposed to catch one gained the column it was missing.

Sweep numbers, full PATH, 2324 tools, against cf5912f:

  • Parser tree unchanged, as expected for an xtask-and-CLI branch: overall: IDENTICAL, 0 flag, 0 subcommand and 0 positional gains or losses, 0 field-level changes. All nine control tools byte-identical on their #fp2 records: git, gcc, aarch64-linux-gnu-g++-13, ar, pnpm, systemctl, tar, find, docker.
  • Existence oracle: existence-fabrication 80 tools to 76. The five tools cleared are exactly the fleet of this shape, measured off the baseline scoreboard as 5 tools and 6 findings: accessdb 1 to 0, gdk-pixbuf-thumbnailer 2 to 0, mknod 1 to 0, systemd-sysusers 1 to 0, systemd-tmpfiles 1 to 0. Xvfb carries a different shape and stays at 1.
  • Replaying the S-154 merged-versus-fixed scoreboard pair now puts 14 positionals across 13 tools on the gain bar, and the same 14 on the loss bar with the two files swapped, where before this branch both bars read zero.

sadigaxund and others added 7 commits September 13, 2026 20:46
Reads gains/losses off #fp2 entity ids, never a scoreboard
column, so a V1 or missing fingerprint reports unmeasured.
Field-level adds/removes now split positionals off flags.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
A synopsis run of 2-4 slots joins the same way the parser
reads it, so mknod, accessdb, gdk-pixbuf-thumbnailer,
systemd-sysusers and systemd-tmpfiles stop fabricating.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
needless_splitn on is_positional_id, and a comment block
over the 12-line ceiling in per_tool_fingerprint_diff.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Refuses before any probe when effective uid is 0, in main
before the TUI, --doctor, --report, or --review resolve a
tool. --allow-root lifts it. Adds spec §6 rule 10.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
The OPTIONS list was missing the flag the root refusal names.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Tracking the file exposed a comment-to-code ratio shape_guard
untracked bytes had hidden. Trims doc comments, no logic change.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
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