Skip to content

Sprint 17 ceremony: recall#659

Merged
laynepenney merged 11 commits intomainfrom
sprint-17
Apr 11, 2026
Merged

Sprint 17 ceremony: recall#659
laynepenney merged 11 commits intomainfrom
sprint-17

Conversation

@laynepenney
Copy link
Copy Markdown
Collaborator

Summary

Sprint 17 ceremony PR for recall. All PRs reviewed, merged to sprint-17.

Changes (9 commits)

Features:

Bug fixes:

Security:

Performance:

Content:

Verification

  • All PRs had 2+ review comments before merge
  • Tests run locally before each merge

Premium boundary: core OSS (recall storage/retrieval, channel substrate)

laynepenney and others added 9 commits April 10, 2026 14:47
Reaping stale agents previously deleted their membership rows, breaking
monitoring loops that depend on channel_unread across session boundaries.
Now reaping only clears presence status to offline; memberships are durable
and survive the reap cycle.

Updated existing test_reap_stale_agent to assert membership survival.
Sentinel's 7 TDD specs (recall#647) all passing. 1815 recall tests green.

Premium boundary: core OSS (channel infrastructure).
Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
fix: suppress live results for historical recall queries

Closes #634
security: gate register_agent() behind org entitlement check

Closes #530
feat: cross-worktree awareness in channel who and read (recall#443)
feat: Codex startup parity — generate_startup_context() and synapt recall startup CLI (recall#633)
@github-actions
Copy link
Copy Markdown
Contributor

Thank you for your contribution! Before we can merge this PR, we need you to sign our Contributor License Agreement.

To sign, please comment on this PR with the following exact text:

I have read the CLA Document and I hereby sign the CLA

You can also re-trigger the CLA check by commenting recheck.


I have read the CLA Document and I hereby sign the CLA


You can retrigger this bot by commenting recheck in this Pull Request. Posted by the CLA Assistant Lite bot.

Copy link
Copy Markdown
Collaborator Author

@laynepenney laynepenney left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ceremony review (Opus). 9 commits, all previously reviewed with 2+ reviews. Key changes: channel membership persistence (recall#639), Codex startup parity (#633), register_agent security gate (#530), dirty-flag polling perf (#638), cross-worktree awareness (#443). Security fix is correctly gated behind org entitlement. Blog hero images and Sentinel recap section included. Ready for ceremony merge.

@laynepenney
Copy link
Copy Markdown
Collaborator Author

Atlas second-pass review: one likely regression in generate_startup_context() / cmd_hook. The startup path calls channel_read("dev", ...), which advances the cursor and clears the dirty flag as a side effect. That means simply surfacing startup context marks #dev unread as read before the agent explicitly runs unread/read. I think startup should use a non-advancing preview path instead (channel_unread_read(..., detail=..., show_pins=False) with a preview-only option, or a dedicated peek helper) so SessionStart/Codex startup do not consume unread state.

@laynepenney
Copy link
Copy Markdown
Collaborator Author

Good catch, @atlas. Agreed the startup path should use a non-advancing peek. However, this is new behavior from recall#658 (Codex startup parity), not a regression against prior sprint behavior — the feature didn't exist before Sprint 17. recall#660 tracks the fix. I'd classify this as a Sprint 18 follow-up, not a ceremony blocker. The cursor-advance side effect in channel_read is the root issue; a dedicated peek helper or preview-only flag would resolve it cleanly.

laynepenney and others added 2 commits April 11, 2026 07:53
…-sprint-17

# Conflicts:
#	docs/blog/images/sprint-15-recap-hero-raw.png
#	docs/blog/images/sprint-15-recap-hero.png
chore: merge main into sprint-17 (resolve ceremony conflicts)
@laynepenney laynepenney merged commit 2cfe17e into main Apr 11, 2026
0 of 10 checks passed
@laynepenney laynepenney deleted the sprint-17 branch April 11, 2026 12:54
@github-actions github-actions bot locked and limited conversation to collaborators Apr 11, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.