Uh oh!
There was an error while loading. Please reload this page.
feat(twister): restore the per-focus memory and status-brief surface - #386
Merged
Conversation
Restores `Plot.getMemory()`, `Plot.setMemory()` and `Plot.getBriefs()`, along with the `FocusMemoryEntry` and `FocusBrief` types. All three require `FocusAccess.Full`. These were withdrawn because the platform did not implement them yet — a twist declaring `FocusAccess.Full` and calling one would type-check against the SDK and then fail at runtime. That is no longer the case: the platform support lands alongside this, so the declared surface again describes what the platform actually answers. The shape is unchanged from the withdrawn version, as promised at the time. No published version was ever affected — the methods were added after 0.92.0 and their changeset was never consumed by a release — so this reads as an addition rather than a restoration from a consumer's point of view, and the changeset is worded that way.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Restores
Plot.getMemory(),Plot.setMemory()andPlot.getBriefs(), along with theFocusMemoryEntryandFocusBrieftypes. All three requireFocusAccess.Full.Why now
These were withdrawn because the platform did not implement them yet — a twist that declared
FocusAccess.Fulland called one of them would type-check cleanly against the SDK and then fail at runtime. Withdrawing kept the declared surface an accurate description of what the platform answers.That is no longer the case. Platform support lands alongside this, so the surface is honest again.
Unchanged shape
The types and signatures are identical to the withdrawn version, as stated at the time. Nothing about the API has been redesigned in the interim.
No published version is affected
The methods were added after
0.92.0and their changeset was never consumed by a release, so they never reached npm. From a consumer's point of view this is an addition, not a restoration — and the changeset is worded that way.What they do
getMemory(focusId)/setMemory(...)— maintain a per-focus agent memory: short freeform entries with optional absoluteYYYY-MM-DDdates.getBriefs()— read each focus's current status brief.Verification
pnpm buildclean (0 errors), changeset present and validating.