Skip to content

refactor(saga): convert sync_scan to std.saga.department#1273

Merged
ElonSG merged 1 commit into
devfrom
refactor/saga-handler-sync-scan
Jun 20, 2026
Merged

refactor(saga): convert sync_scan to std.saga.department#1273
ElonSG merged 1 commit into
devfrom
refactor/saga-handler-sync-scan

Conversation

@ElonSG

@ElonSG ElonSG commented Jun 20, 2026

Copy link
Copy Markdown
Contributor

Convert sync_scan to std.saga.department: done=false, act = old body byte-for-byte, wrap preserved, FULL spec incl fanout = { devloop_branch_tick } preserved. Also widens slicer dry-run test to selected_count [0-2] (ratchet drained to 1 entry) with word-boundary anchor. saga-handler 2->1. sshx-reviewed; full test 9 packages 0 failed.

⟦AI:FKST⟧

🤖 Generated with Claude Code

Convert github-devloop sync_scan to std.saga.department: done=false, act = old
pipeline body byte-for-byte, wrap=core.wrap_pipeline_failure name="sync_scan". The
FULL spec is preserved: consumes, produces, fanout = { "devloop_branch_tick" }, and
stall_window (applying the fanout-preservation lesson from the prior slice's review).
saga-handler.allowlist (G10 2->1).

Also widens the slicer dry-run test assertion from r"- selected_count: [02]" to
r"- selected_count: [0-2]\b": selected_count = min(slice_size=2, remaining_count), and
draining saga-handler to 1 remaining entry makes the dry-run select 1 site. The \b anchor
keeps it from matching multi-digit values (e.g. 20). sshx-reviewed (architecture approve,
quality/quality comment, no reject); full test 9 packages 0 failed.

⟦AI:FKST⟧

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_0163uaeg6woT12QeZE1cAkn6
@codecov

codecov Bot commented Jun 20, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@ElonSG ElonSG merged commit 72b397b into dev Jun 20, 2026
5 checks passed
@ElonSG ElonSG deleted the refactor/saga-handler-sync-scan branch June 20, 2026 15:31
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.

2 participants