Skip to content

Merge pull request #527 from pushkarverma3698/main - #528

Merged
pushkarverma3698 merged 1 commit into
mainfrom
beta
Aug 20, 2026
Merged

Merge pull request #527 from pushkarverma3698/main#528
pushkarverma3698 merged 1 commit into
mainfrom
beta

Conversation

@pushkarverma3698

Copy link
Copy Markdown
Owner

chore: sync beta with main

What & why

Failing test first (v3 invariant #2)

Checklist

  • pnpm gate green (tsc + tests)
  • pnpm verify:arch green (no new architectural debt; baseline may only shrink)
  • Behaviour change checked against pnpm eval (routing/tool/HITL not regressed) — or N/A
  • No hardcoded secrets; no console.log in src/ (boot console.error excepted)
  • Errors fail loud (surfaced to founder), no swallowed errors or fake "Done."
  • HITL gate covers any new external side effect (email/LinkedIn/GitHub/file) — or N/A
  • Idempotency guard precedes any new external send; audit row only on real success — or N/A
  • New run-loop branches emit a seam (trace.event); new seam added to the Seam union — or N/A
  • Pure logic (routing/slicing/guards/parsing) is a unit-tested function, not a prompt instruction
  • Every fixed bug has a regression test on the real gateway path
  • "Why" comments on non-obvious functions
  • Docs synced (CLAUDE.md rule fix(personal): harden path-guard — symlink bypass + OOM protection #18): New ADRs, phase docs, or decisions → run pnpm brain:sync — or N/A

Observability impact

Verification (real path, not just unit tests — CLAUDE.md #19)

Rollback plan

@pushkarverma3698
pushkarverma3698 merged commit 53fc964 into main Aug 20, 2026
6 checks passed
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