• Joined on 2026-02-08
clawbot created pull request sneak/vaultik#55 2026-03-19 14:08:26 +01:00
Remove all ctime usage and storage
clawbot pushed to remove-ctime at sneak/vaultik 2026-03-19 14:08:14 +01:00
1717677288 remove ctime column from schema, model, queries, scanner, and docs
clawbot created branch remove-ctime in sneak/vaultik 2026-03-19 14:08:14 +01:00
clawbot commented on pull request sneak/vaultik#47 2026-03-19 14:05:18 +01:00
fix: use deleteSnapshotFromLocalDB in syncWithRemote

Checked: FK enforcement IS enabled. database.go sets _foreign_keys=ON in the DSN and also runs PRAGMA foreign_keys = ON explicitly on each connection.

So with PRs [#44](https://git.eeqj.de/s…

clawbot commented on pull request sneak/vaultik#47 2026-03-19 14:04:56 +01:00
fix: use deleteSnapshotFromLocalDB in syncWithRemote

It depends on whether SQLite FK enforcement is actually enabled. SQLite requires PRAGMA foreign_keys = ON per connection — it's off by default, and many Go SQLite drivers don't enable it…

clawbot commented on pull request sneak/vaultik#46 2026-03-19 14:00:40 +01:00
schema: add ON DELETE CASCADE to snapshot_files.file_id and snapshot_blobs.blob_id FKs

Yes — the cascades go from parent to junction table:

  • Deleting a file → automatically deletes its snapshot_files junction rows (which link files to snapshots)
  • Deleting a blob →…
clawbot opened issue sneak/vaultik#54 2026-03-19 13:59:40 +01:00
Remove all ctime usage and storage
clawbot closed issue sneak/vaultik#13 2026-03-19 13:59:28 +01:00
ctime is in schema and model but verify scanner populates it correctly
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.goblob_fetch.go per [issue #52](https://git.eeqj.de/sneak/vaultik/issu…