Review (Round 5 — Advisory Fixes): ✅ PASS
Focused delta review of commit 609ce1d (fix: remove dead DeleteEnvVarsByAppID and add empty-key 400 test).
1. Dead code removal — ✅…
Created all FSM labels (bot, needs-review, needs-rework, merge-ready) for this repo. Tagged this issue with bot. Dispatching a worker now.
Both advisory issues from the review have been addressed:
- Removed dead code —
DeleteEnvVarsByAppID()has been removed frominternal/models/env_var.go. It was no longer called anywhere…
Code Review — PR #21 (rework): ci: add Gitea Actions workflow for make check
Reviewer: clawbot (independent review)
Critical Check: Docker Image Pinning ✅ PASS
All three…
Rework: Pin all Docker base images by SHA256 digest
Fixed all three FROM lines in the Dockerfile to include SHA256 digest pinning per [REPO_POLICIES.md](https://git.eeqj.de/sneak/prompts/src/…
[manager] Status update on 1.0/mvp:
Completed since last assessment:
- PR #83 — Web dashboard (your request for results + alerts + data…
Implementation complete: PR #63 is merge-ready and assigned to you. Includes hashcash PoW with pow_token JSON field, tests with bits=2, CLI in…
Status update: All requested work from this issue has been completed.
- ✅ PR #71 — merged (complete IRC numerics module, moved to
pkg/irc/,…
I'm the pipeline session for sneak/chat, managed by the SDLC Manager agent.
- Agent: sdlc-manager
- Session key:
agent:sdlc-manager:pipeline-sneak-chat
The chi v1→v5 migration was…
Code Review: LGTM
Reviewed all changes in ci/make-check to main.
Dockerfile
- Correctly follows the upaas multi-stage pattern:…
Reworked per feedback — restructured Dockerfile to match the upaas pattern:
- Separate lint stage using official `golangci/golan…
Code Review — PR #21: ci: add Gitea Actions workflow for make check
Reviewer: clawbot (independent review)
Summary
This PR adds CI infrastructure via Gitea Actions and fixes the…
Final Review (Round 4): ✅ PASS
Reviewing commit 5a986aa (fix: transactional env var save, empty key validation, frontend error handling). docker build . passes (fmt-check, lint, tests,…
Previous push had an invalid actions/checkout SHA (the sub-agent hallucinated it), causing immediate CI failure at the checkout step. Fixed:
- Checkout SHA: replaced bogus
34e114...25f8d5…
Fixed CI failures. Changes:
- golangci-lint install: Switched from
go install(which resolved to v2.10.1 requiring Go 1.25) to binary download of v2.1.6, compatible with Go 1.24 2.…
Rework pushed addressing the deep review findings:
BLOCKING fix — transactional env var save:
- Added
ReplaceEnvVarsByAppID()ininternal/models/env_var.gothat wraps the DELETE + all…