Skip to content

A caller that knows when this sitting began can say so - #57

Merged
ralyodio merged 1 commit into
mainfrom
feat/memory-since-this-episode
Nov 5, 2026
Merged

ralyodio merged 1 commit into
mainfrom
feat/memory-since-this-episode

Conversation

@ralyodio

@ralyodio ralyodio commented Nov 5, 2026

Copy link
Copy Markdown
Contributor

Follow-up to #55/#56. Bounding room memory to thirty minutes stops a new episode opening with last month's, but it does not stop it opening with the last take's — and a retake right after a failure is the normal case here, not the odd one:

episode started gap
8 17:10:09 —
9 17:31:… 21 min

Inside one thirty-minute window, so episode 9 would have been handed episode 8's transcript as its own memory — including a guest answer it never heard.

The leg cannot know where an episode begins; the show does. --memory-since (ISO or epoch ms) sets the line, and nothing before it belongs to this call — used both for the memory load and for the catch-up staleness guard from #56, so the two cannot disagree.

Without the flag the thirty-minute window stands, so a plain agent-phone join is unchanged. A value that does not parse is ignored rather than trusted: a bad flag must not blank the context a rejoin depends on ('tomorrow-ish' and '0' both fall back).

bun test 115 pass / 0 fail, tsc --noEmit clean.

Needs the matching one-line change in offtherecord to pass the episode's created_at; opening that next.

🤖 Generated with Claude Code

With the window bounded to thirty minutes (#55) a new episode no longer opens
holding last month's, but takes come back to back: episodes 8 and 9 were
twenty-one minutes apart in the same room, so the default would still have
handed the second one the first one's transcript - and a retake right after a
failure is the normal case, not the odd one.

The leg cannot know where an episode begins. The show does. So --memory-since
(ISO or epoch ms) sets the line, and nothing before it is this call's. Without
it, the thirty-minute window stands, so a plain `join` is unchanged. A value
that does not parse is ignored rather than trusted: a bad flag must not blank
the context a rejoin depends on.

115 pass, 0 fail; tsc clean.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@ralyodio
ralyodio merged commit dae12cd into main Nov 5, 2026
3 checks passed
@ralyodio
ralyodio deleted the feat/memory-since-this-episode branch November 5, 2026 17:53
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