24 lines
987 B
Markdown
24 lines
987 B
Markdown
|
# Latest repo review
|
|||
|
|
|
|||
|
|
Most recent: **2026-05-30** → full report: `docs/reviews/2026-05-30-review.md`
|
|||
|
|
|
|||
|
|
| | high | medium | low | total |
|
|||
|
|
|---|---|---|---|---|
|
|||
|
|
| Auto-fixed | 2 | 3 | 2 | 7 |
|
|||
|
|
| Open | 4 | 4 | 9 | 17 |
|
|||
|
|
|
|||
|
|
Dominant theme: drift from this session's own changes — residual `.vault_pass`
|
|||
|
|
references after the Vaultwarden/rbw switch, and leftover PR/merge-request language
|
|||
|
|
after going trunk-based.
|
|||
|
|
|
|||
|
|
## Suggested follow-up prompt
|
|||
|
|
|
|||
|
|
> Remediate the boma 2026-05-30 review (`docs/reviews/2026-05-30-review.md`):
|
|||
|
|
> 1. Purge the residual `.vault_pass` references R1–R5 → the rbw/Vaultwarden flow.
|
|||
|
|
> 2. Decide the workflow model R6–R7 — I lean "keep deploy approval gates, drop the
|
|||
|
|
> PR/merge-request framing"; reconcile ADR-003/008 and CLAUDE.md to match.
|
|||
|
|
> 3. Resolve R8 — scaffold `base`/`docker_host` via `make new-role`, or correct
|
|||
|
|
> STATUS.md/roles/README.md to say the roles don't exist yet.
|
|||
|
|
> 4. Fix the Terraform `vlan_tag` wiring (R9).
|
|||
|
|
> Report on the rest.
|