docs(s-remind2): backfill squash e5b0d13 (#298)#299
Merged
Conversation
…less framing Record the S-remind2 slice (distinct task.due_final 2nd reminder): slice-history narrative + CLAUDE.md Recent-learnings bullet (dropping the oldest S-notify-5a to hold the ~8 cap) + Current-status pointer + migration head 0067→0068. Also correct the now-wrong 'Docker-less box' framing in docs/dev-workflow.md: Docker IS available on this Linux box via 'sg docker -c' (the agent user is in the docker group), which runs -m integration / /check-migrations / a real-stack migration live-smoke locally. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Backfills the slice docs for #298 (S-remind2 — a distinct second/final reminder under
task.due_final), mirroring the#296→#297pattern. Docs only.docs/slice-history.md— full S-remind2 narrative at the top of NOTIFICATIONS (the dedup-collapse cause, the distinct-key fix, migration 0068, the blast-radius existing-test audit, the named deferrals); migration-head line →0068 (next 0069).CLAUDE.md→ Recent learnings — one dense bullet (prepended), dropping the oldest S-notify-5a to hold the ~8 cap (its narrative lives in slice-history; added to the trim-comment list).CLAUDE.md→ Current status — the second reminder shipped; migration head0067 → 0068.docs/dev-workflow.md(+ a pointer in CLAUDE.md / slice-history) — corrects the now-wrong "Docker-less box" framing: Docker (29.x) + compose ARE installed; the agent's shell user is in thedockergroup, so wrap docker-touching commands insg docker -c "…"(an already-running login shell doesn't pick up the new group). This runs-m integration(testcontainers),/check-migrations, and a real-stack migration live-smoke locally — proven this slice (26 integration tests passed on a real PG16; a livealembic 0067↔0068round-trip on the populated AHT DB).🤖 Generated with Claude Code