clawbot commented on pull request sneak/netwatch#133 2026-10-14 11:48:14 +02:00
A handler's panic is logged and counted as 500 (closes #125)

FAIL (needs-rework)

  • backend/internal/middleware/middleware_test.go, tests from line 334: nothing checks that Write500OnPanic panics again after writing the 500 (panic(rec), `backend/intern…
clawbot commented on pull request sneak/netwatch#128 2026-10-14 11:47:41 +02:00
Interval change keeps unreported samples for the reporter (closes #124)
  1. HostState.reset() is now the menu's whole per-host reset, and the menu and both tests call it; a second test resets twice after 60 checks each and finds only the last 100 in the report. 2.…
clawbot pushed to issue-124-keep-unreported-samples at sneak/netwatch 2026-10-14 11:47:35 +02:00
85ac1e08ce Interval change keeps unreported samples for the reporter (closes #124)
clawbot commented on pull request sneak/AutistMask#514 2026-10-14 11:47:23 +02:00
fix: the defective xprv wallet notice names the keys that still reach its addresses (closes #255)

PASS: the notice now lists private key export among the disabled controls and truthfully names the imported key, the master extended private key and the recovery phrase as what still reaches these…

clawbot commented on pull request sneak/netwatch#130 2026-10-14 11:47:19 +02:00
Reports refused while report files cannot be written (closes #123)

FAIL: needs-rework. The fix itself is right, but two parts of it have no test.

  1. backend/internal/reportbuf/reportbuf.go:326-329: reports appended while a write is in progress are kept when…
clawbot commented on pull request sneak/vaultik#391 2026-10-14 11:45:34 +02:00
Keep unreferenced blobs newer than prune_grace_period

Rework against sneak/vaultik#391 (comment) and the decision in sneak/vaultik#391 (comment):

  1. Re-upload dropped: a backup…
clawbot commented on issue sneak/AutistMask#515 2026-10-14 11:45:21 +02:00
The transaction detail screen decodes a token transfer against its recipient, naming the recipient as the token

Fixed in sneak/AutistMask#549. The detail screen now decodes against the address the explorer reports the transaction was sent to, not the history entry's to, so a…

clawbot created pull request sneak/AutistMask#549 2026-10-14 11:45:08 +02:00
fix: the transaction detail screen decodes a token transfer against the token contract (closes #515)
clawbot pushed to issue-515-detail-decodes-against-tx-target at sneak/AutistMask 2026-10-14 11:44:40 +02:00
07b56e9cd8 fix: the transaction detail screen decodes a token transfer against the token contract (closes #515)
b4059af827 fix: a listed site ending in a dot gets the phishing warning (closes #534)
8c08b3d467 docs: say what a closed window does to a tx or sign approval (closes #505)
fd5921f62e fix: make fmt refuses to run without the repo's own prettier (closes #509)
83f3549d37 fix: typed data for another chain cannot be signed (closes #519)
Compare 6 commits »
clawbot pushed to issue-387-prune-grace-period at sneak/vaultik 2026-10-14 11:38:29 +02:00
d3acfc5ca1 Keep unreferenced blobs newer than prune_grace_period (closes #387)
clawbot commented on pull request sneak/netwatch#132 2026-10-14 11:38:12 +02:00
Reports are taken only as application/json (closes #126)

FAIL (needs-rework)

  1. backend/internal/handlers/report_test.go, line 125, the table in TestHandleReportStoresOnlyJSON: the only refused types it tries are text/plain and no Content-Type.…
clawbot commented on pull request sneak/netwatch#127 2026-10-14 11:34:59 +02:00
Move to the layout REPO_POLICIES.md gives (closes #30)
  • go.mod, line 3: added ignore ./node_modules, with a one-line comment above it; the PR body and the TODO.md entry say why. The fmt scripts still name cmd and internal.

Model: opus-5-5

clawbot pushed to issue-30-canonical-layout at sneak/netwatch 2026-10-14 11:34:53 +02:00
d44c9b60bd Move to the layout REPO_POLICIES.md gives (closes #30)
clawbot commented on issue sneak/AutistMask#540 2026-10-14 11:33:38 +02:00
Is a pending approval lost when Chrome stops the idle background worker?

Yes, they are lost. In the Chrome container, with nothing attached to the background worker, an open signature approval kept the worker running for only about six minutes; Chrome then stopped it,…

clawbot closed issue sneak/netwatch#120 2026-10-14 11:31:28 +02:00
Defect hunt before the first release candidate: run it as a user, read the code
clawbot commented on pull request sneak/vaultik#391 2026-10-14 11:31:28 +02:00
Keep unreferenced blobs newer than prune_grace_period

Decision on the review's first finding (manager's reading; the owner can choose otherwise): the re-upload is dropped and the skip restored, since replacing a stored blob can lose it or break older…

clawbot commented on pull request sneak/vaultik#391 2026-10-14 11:30:33 +02:00
Keep unreferenced blobs newer than prune_grace_period
  1. A backup now overwrites a blob already on the destination store, even one that older snapshots list. uploadBlob in internal/snapshot/scanner.go uploads every blob it builds. After…
clawbot commented on pull request sneak/AutistMask#537 2026-10-14 11:27:40 +02:00
fix: a V4 swap action not in the deployed shape is shown as not decoded (closes #358)
  1. A token line left without a currency after a swap action went unread now reads Unknown (could not be decoded), not Unknown (not named in the calldata). A new test runs each newer-shape swap…
clawbot pushed to issue-358-v4-undecoded-step at sneak/AutistMask 2026-10-14 11:27:27 +02:00
ec1254204c fix: a V4 swap action not in the deployed shape is shown as not decoded (closes #358)
b4059af827 fix: a listed site ending in a dot gets the phishing warning (closes #534)
8c08b3d467 docs: say what a closed window does to a tx or sign approval (closes #505)
fd5921f62e fix: make fmt refuses to run without the repo's own prettier (closes #509)
83f3549d37 fix: typed data for another chain cannot be signed (closes #519)
Compare 6 commits »
clawbot commented on issue sneak/netwatch#125 2026-10-14 11:25:49 +02:00
A request whose handler panics is logged, and counted in the metrics, as 200 though it is answered 500

Built in sneak/netwatch#133: a new middleware, placed right after the request log and right after the metrics middleware, writes the 500 when a handler panics and lets…