decision: RULES.md says DEBUG mode enables only the banner and the test phrase; it also logs at debug level #761

Open
opened 2026-11-10 20:55:16 +01:00 by clawbot · 3 comments
Collaborator

RULES.md is your audit checklist for AutistMask; agents never edit it. Its DEBUG Mode section says "DEBUG mode only enables: red banner + hardcoded test mnemonic". A debug build (make build-debug) also does two more things, both from your own commits of 26 February:

  • it logs at debug level, so each request the extension makes goes to the browser console (method, origin and JSON-RPC method name; never the path, query or body);
  • the red banner names the screen shown, for example "DEBUG / INSECURE (ConfirmTx)".

A release build does neither.

Options:

  • Change the line: "DEBUG mode only enables: red banner naming the current screen, debug-level logging, and the hardcoded test recovery phrase". A worker opens the PR for you to merge.
  • Change the code: a debug build logs at the normal level and its banner leaves out the screen name.

Recommendation: Change the line. Both are your own features and change nothing a user does.

Found in the RULES.md read on #672 (comment).

Model: opus-5-5

`RULES.md` is your audit checklist for AutistMask; agents never edit it. Its DEBUG Mode section says "DEBUG mode only enables: red banner + hardcoded test mnemonic". A debug build (`make build-debug`) also does two more things, both from your own commits of 26 February: - it logs at debug level, so each request the extension makes goes to the browser console (method, origin and JSON-RPC method name; never the path, query or body); - the red banner names the screen shown, for example "DEBUG / INSECURE (ConfirmTx)". A release build does neither. Options: - **Change the line**: "DEBUG mode only enables: red banner naming the current screen, debug-level logging, and the hardcoded test recovery phrase". A worker opens the PR for you to merge. - **Change the code**: a debug build logs at the normal level and its banner leaves out the screen name. Recommendation: **Change the line**. Both are your own features and change nothing a user does. Found in the `RULES.md` read on https://git.eeqj.de/sneak/AutistMask/issues/672#issuecomment-160312. Model: opus-5-5
sneak was assigned by clawbot 2026-11-10 20:55:16 +01:00
clawbot added this to the 1.0.0 milestone 2026-11-10 20:55:22 +01:00
Owner

approved

approved
Author
Collaborator

sneak's ruling (comment above, 13 November): Change the line. A worker opens a PR into next that changes the RULES.md DEBUG Mode line to: "DEBUG mode only enables: red banner naming the current screen, debug-level logging, and the hardcoded test recovery phrase". The PR touches only that line and is assigned to sneak to merge, as with #193.

Model: opus-5-5

**sneak's ruling** (comment above, 13 November): **Change the line**. A worker opens a PR into `next` that changes the `RULES.md` DEBUG Mode line to: "DEBUG mode only enables: red banner naming the current screen, debug-level logging, and the hardcoded test recovery phrase". The PR touches only that line and is assigned to sneak to merge, as with https://git.eeqj.de/sneak/AutistMask/issues/193. Model: opus-5-5
sneak was unassigned by clawbot 2026-11-13 19:39:17 +01:00
clawbot self-assigned this 2026-11-13 19:39:17 +01:00
Author
Collaborator

PR #775 (branch issue-761-rules-debug-mode-line) changes the RULES.md DEBUG Mode line to the approved wording.

Deviation: assigned to clawbot with needs-review, not to sneak, since workers never assign their PR to him.

Model: opus-5-5

PR https://git.eeqj.de/sneak/AutistMask/pulls/775 (branch `issue-761-rules-debug-mode-line`) changes the `RULES.md` DEBUG Mode line to the approved wording. Deviation: assigned to `clawbot` with `needs-review`, not to sneak, since workers never assign their PR to him. Model: opus-5-5
Sign in to join this conversation.
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: sneak/AutistMask#761