clawbot

clawbot commented on issue sneak/vaultik#13

2026-03-19 13:59:28 +01:00

ctime is in schema and model but verify scanner populates it correctly

Closing: sneak decided to remove ctime entirely rather than fix its population. New issue will track ctime removal from schema, model, and scanner.

clawbot closed pull request sneak/vaultik#48

2026-03-19 13:59:19 +01:00

fix: populate ctime from actual filesystem stats instead of mtime fallback

clawbot commented on pull request sneak/vaultik#48

2026-03-19 13:59:19 +01:00

fix: populate ctime from actual filesystem stats instead of mtime fallback

Understood. Closing this PR — will open a new issue to remove all ctime usage and storage from the codebase.

clawbot commented on pull request sneak/vaultik#48

2026-03-19 09:36:02 +01:00

fix: populate ctime from actual filesystem stats instead of mtime fallback

You're right, I was wrong about that. Vaultik hashes on mtime change, not every scan — ctime has nothing to do with the scanning/hashing path.

Corrected take: the only value ctime provides is as…

clawbot pushed to refactor/dockerfile-pinned-linter-image at sneak/webhooker

2026-03-19 06:26:51 +01:00

32a9170428 refactor: use pinned golangci-lint Docker image for linting

d771fe14df fix: detect TLS per-request in CSRF middleware to fix login (#54)

33e2140a5a feat: redirect root path based on auth state (#52)

Compare 3 commits »

clawbot commented on pull request sneak/vaultik#53

2026-03-19 00:33:51 +01:00

Rename blob_fetch_stub.go to blob_fetch.go

Review: PASS ✅

PR #53 correctly implements issue #52 — rename blob_fetch_stub.go → `blob_fetch.…

clawbot commented on pull request sneak/vaultik#49

2026-03-19 00:33:42 +01:00

fix: remove destructive sync from ListSnapshots

Review: PASS ✅ (post-rebase)

clawbot commented on pull request sneak/vaultik#53

2026-03-19 00:32:38 +01:00

Rename blob_fetch_stub.go to blob_fetch.go

Rework: Rebased onto current main

The branch was based on a stale main (before PR #41 was merged), causing the diff to include reversed…

clawbot pushed to rename-blob-fetch-stub at sneak/vaultik

2026-03-19 00:32:22 +01:00

d0818e800b rename blob_fetch_stub.go to blob_fetch.go

ac2f21a89d Refactor: break up oversized methods into smaller descriptive helpers (#41)

Compare 2 commits »

clawbot commented on pull request sneak/vaultik#49

2026-03-19 00:31:52 +01:00

fix: remove destructive sync from ListSnapshots

Rebased onto main (on top of PR #39 and PR #41).

Conflict was in reconcileLocalWithRemote — the…

clawbot pushed to fix/list-snapshots-no-destructive-side-effect at sneak/vaultik

2026-03-19 00:31:31 +01:00

a36b314c79 fix: remove destructive sync from ListSnapshots

ac2f21a89d Refactor: break up oversized methods into smaller descriptive helpers (#41)

8c59f55096 fix: verify blob hash after download and decryption (closes #5) (#39)

Compare 3 commits »

clawbot commented on pull request sneak/vaultik#53

2026-03-19 00:28:28 +01:00

Rename blob_fetch_stub.go to blob_fetch.go

Review: FAIL ❌

PR #53 claims to be a pure rename of blob_fetch_stub.go → blob_fetch.go per [issue #52](https://git.eeqj.de/sneak/vaultik/issu…

clawbot created pull request sneak/vaultik#53

2026-03-19 00:26:49 +01:00

Rename blob_fetch_stub.go to blob_fetch.go

clawbot pushed to rename-blob-fetch-stub at sneak/vaultik

2026-03-19 00:26:42 +01:00

8d3a3ab7a1 rename blob_fetch_stub.go to blob_fetch.go

clawbot created branch rename-blob-fetch-stub in sneak/vaultik

2026-03-19 00:26:42 +01:00

clawbot commented on pull request sneak/vaultik#44

2026-03-19 00:25:21 +01:00

fix: add ON DELETE CASCADE to uploads FK on snapshot_id

It's the other direction — we don't. The ON DELETE CASCADE is on the FK from uploads.snapshot_id → snapshots.id, so:

Deleting a snapshot → automatically deletes its **upload…

clawbot opened issue sneak/vaultik#52

2026-03-19 00:22:57 +01:00

Rename blob_fetch_stub.go to blob_fetch.go

clawbot commented on pull request sneak/vaultik#39

2026-03-19 00:22:44 +01:00

fix: verify blob hash after download and decryption (closes #5)

Good catch — it shouldn't be. The file contains production code (hashVerifyReader, FetchAndDecryptBlob) but the original worker named it _stub.go and nobody caught the misleading name…

clawbot commented on pull request sneak/vaultik#51

2026-03-18 05:59:30 +01:00

feat: per-name purge filtering for snapshot purge

Review: PASS

clawbot commented on pull request sneak/prompts#19

2026-03-18 05:56:56 +01:00

style: strengthen constructor naming and Params struct rules

Code Review: PR #19 — style: strengthen constructor naming and Params struct rules

Policy Divergences

No policy violations found.

-…