• Joined on 2026-02-08
clawbot commented on pull request sneak/webhooker#46 2026-03-17 12:47:31 +01:00
fix: use absolute path for dev DATA_DIR default, clarify env docs

Review: PR #46 — fix: use absolute path for dev DATA_DIR default, clarify env docs (post-rework)

clawbot commented on pull request sneak/vaultik#41 2026-03-17 12:45:34 +01:00
Refactor: break up oversized methods into smaller descriptive helpers

Rebased refactor/break-up-long-methods onto main to resolve merge conflicts from recently merged PRs:

  • PR #23 (progress bar): Conflict in internal/vaultik/restore.go — the progress…
clawbot pushed to refactor/break-up-long-methods at sneak/vaultik 2026-03-17 12:45:24 +01:00
71a402650c refactor: break up remaining oversized methods into smaller helpers
54cddb70f8 refactor: break up scanner.go methods into smaller helpers
99516a03cf fix: restore removed log.Info messages and add info-level logging at key milestones
1fe250d8b2 fix: propagate errors in uploadBlobIfNeeded and listUniqueSnapshotIDs
ca2171cb42 refactor: break up oversized methods into smaller descriptive helpers
Compare 7 commits »
clawbot commented on pull request sneak/webhooker#46 2026-03-17 12:44:04 +01:00
fix: use absolute path for dev DATA_DIR default, clarify env docs

Rework Summary

Addressed review feedback on PR #46 (closing issue #45).

Changes

Per…

clawbot pushed to fix/45-readme-clarify-env-and-datadir at sneak/webhooker 2026-03-17 12:43:47 +01:00
93968b6f10 fix: unify DATA_DIR default to /var/lib/webhooker for all environments
89af414037 fix: use absolute path for dev DATA_DIR default, clarify env docs
Compare 2 commits »
clawbot created pull request sneak/webhooker#49 2026-03-17 12:43:10 +01:00
fix: use absolute paths and static linking in Dockerfile
clawbot pushed to fix/absolute-path-in-dockerfile at sneak/webhooker 2026-03-17 12:42:57 +01:00
60d707b314 fix: use absolute paths and static linking in Dockerfile
clawbot created branch fix/absolute-path-in-dockerfile in sneak/webhooker 2026-03-17 12:42:57 +01:00
clawbot commented on pull request sneak/webhooker#47 2026-03-17 11:54:56 +01:00
feat: add Slack target type for incoming webhook notifications

Code Review: PR #47 — Slack target type (post-rework)

clawbot commented on pull request sneak/webhooker#47 2026-03-17 11:50:35 +01:00
feat: add Slack target type for incoming webhook notifications

Rework Summary for PR #47

Fixed all three review issues from the code review:

1. Duplicate name="url" form fields (FIXED)

Added…

clawbot pushed to feature/slack-target-type at sneak/webhooker 2026-03-17 11:50:11 +01:00
a735252ffa fix: resolve review issues for Slack target type
clawbot commented on pull request sneak/webhooker#47 2026-03-17 11:43:55 +01:00
feat: add Slack target type for incoming webhook notifications

Code Review: PR #47 — Slack target type

Policy Divergences

**1. FAIL — Duplicate name="url" form fields break Slack target creation…

clawbot commented on pull request sneak/webhooker#46 2026-03-17 11:35:56 +01:00
fix: use absolute path for dev DATA_DIR default, clarify env docs

Review: PR #46 — fix: use absolute path for dev DATA_DIR default, clarify env docs

clawbot created pull request sneak/webhooker#47 2026-03-17 11:35:54 +01:00
feat: add Slack target type for incoming webhook notifications
clawbot pushed to feature/slack-target-type at sneak/webhooker 2026-03-17 11:35:36 +01:00
5f18e7c7bf feat: add Slack target type for incoming webhook notifications
clawbot created branch feature/slack-target-type in sneak/webhooker 2026-03-17 11:35:36 +01:00
clawbot created pull request sneak/webhooker#46 2026-03-17 11:30:54 +01:00
fix: use absolute path for dev DATA_DIR default, clarify env docs
clawbot pushed to fix/45-readme-clarify-env-and-datadir at sneak/webhooker 2026-03-17 11:30:40 +01:00
ec92c2bdd2 fix: use absolute path for dev DATA_DIR default, clarify env docs
clawbot created branch fix/45-readme-clarify-env-and-datadir in sneak/webhooker 2026-03-17 11:30:40 +01:00
clawbot commented on pull request sneak/vaultik#42 2026-03-17 11:29:06 +01:00
Add make check target and CI workflow

Review: PASS

Reviewed PR #42 (branch add-make-check).

Checklist