Skip to content

docs(status): record Bug A + Bug B closure and 6b9a3b5 prod smoke#22

Merged
isuttell merged 1 commit into
mainfrom
docs/bug-a-closure
May 22, 2026
Merged

docs(status): record Bug A + Bug B closure and 6b9a3b5 prod smoke#22
isuttell merged 1 commit into
mainfrom
docs/bug-a-closure

Conversation

@isuttell

@isuttell isuttell commented May 22, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Snapshot bumped to 6b9a3b5.
  • Verified State: pnpm smoke:production pass entry for 6b9a3b5, deploy run 26291734441; PR preview workflow promoted to Pass with PR fix(db): route nested transactions through drizzle.transaction (Bug A) #21 as the Wave 4 exemplar (it caught the jsonb regression before merge).
  • Recently Completed: Bug A entry (drizzle on TransactionSql + jsonb serializer side-effect) and Bug B entry (0003 idempotency).

Test plan

  • docs-only diff, no code paths touched
  • CI green required before merge

Summary by CodeRabbit

  • Documentation
    • Updated project status documentation with current deployment and verification results
    • Documented recently completed bug resolutions

Review Change Stack

- Snapshot: bump to 6b9a3b5.
- Verified State: record production smoke pass on 6b9a3b5 and
  promote PR #21 to the Wave 4 PR-preview-lifecycle exemplar - it
  exercised deploy-pr-preview + smoke:pr + pr-preview-cleanup end
  to end and caught the jsonb regression before merge.
- Recently Completed: new Bug A entry (drizzle on TransactionSql +
  jsonb serializer) and Bug B entry (0003 idempotency).

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
@isuttell isuttell temporarily deployed to pr-preview-22 May 22, 2026 13:54 — with GitHub Actions Inactive
@coderabbitai

coderabbitai Bot commented May 22, 2026

Copy link
Copy Markdown

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: e1b11846-0314-4a21-9b03-ff6bbd3d27bf

📥 Commits

Reviewing files that changed from the base of the PR and between 6b9a3b5 and be63273.

📒 Files selected for processing (1)
  • docs/ops/project-status.md

Walkthrough

This PR updates the operational status documentation (docs/ops/project-status.md) with current metadata, verification test results, and incident records. The snapshot section now reflects the latest main commit hashes and "Last updated" timestamp including recent drizzle/serializer fixes. The "Verified State" table was refreshed with corrected dates and notes for setup, verify, smoke, production deploy, security hardening, and PR preview/cleanup workflows. Two new "Recently Completed" incidents were added: Bug A documents an admin workspace create 500 error caused by nested Drizzle transaction construction and JSON serialization, with the fix involving tx-bound DrizzleDb routing and payload stringification; Bug B documents an idempotency issue in the 0003_rls_runtime.sql RLS migration, resolved by making policy creation and drop operations conditional.

Possibly related PRs

  • zaks-io/agent-paste#19: Both PRs modify docs/ops/project-status.md to update the Snapshot "main" commit SHA and Verified State results, with overlapping changes to the same documentation sections.
  • zaks-io/agent-paste#14: Both PRs update PR preview/cleanup workflow status and behavior notes in the "Verified State" table.

Poem

🐰 The rabbit hops through status logs so bright,
Bugs squashed to bits—now working right!
Nested transactions, migrations fixed with care,
Operational docs refreshed in the air! ✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and specifically summarizes the main changes: documenting the closure of Bug A and Bug B, and recording a production smoke test pass for commit 6b9a3b5. It directly reflects the content of the documentation update.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch docs/bug-a-closure

Comment @coderabbitai help to get the list of available commands and usage tips.

@isuttell isuttell merged commit 0b1b625 into main May 22, 2026
4 checks passed
@isuttell isuttell deleted the docs/bug-a-closure branch May 22, 2026 13:57
@github-actions

Copy link
Copy Markdown

agent-paste PR preview resources were cleaned up. The pr-preview-${context.issue.number} environment is left in place; remove it from the GitHub UI if desired.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant