feat!: SDK-only transport — remove legacy API path#30
Merged
Conversation
resumeSession(conversationId) instead of resumeSession(agentId) so that SDK-routed messages show up in the same conversation on app.letta.com. Written by Cameron ◯ Letta Code "All problems in computer science can be solved by another level of indirection." - David Wheeler
…e with SDK resumeSession(conversationId) silently fails (0 chars response) when the conversation was created via the raw Letta API. The SDK needs to manage its own conversation. Revert to agentId for now. Written by Cameron ◯ Letta Code "Move fast and fix things." - Unknown
Revert to resumeSession(conversationId) and log every stream message type to diagnose why the session returns 0 chars. Written by Cameron ◯ Letta Code "Debugging is twice as hard as writing the code." - Brian Kernighan
In SDK mode, sync_letta_memory.ts and plan_checkpoint.ts were still spawning legacy send_worker.ts — racing the SDK worker on the same conversation and causing 409 CONFLICT errors (0 chars response). - sync_letta_memory: skip early prompt notification in SDK mode - plan_checkpoint: skip entirely in SDK mode (Stop hook handles it) Written by Cameron ◯ Letta Code "Two threads enter, one thread leaves." - Mad Max, concurrent edition
Add Read, Glob, Grep to tool_guidelines so Sub knows it has filesystem access via the Letta Code SDK transport. Written by Cameron ◯ Letta Code "Know thyself." - Socrates
BREAKING CHANGE: The `LETTA_SDK_TOOLS=off` option is removed. All message delivery to Sub now goes through the Letta Code SDK. Users who need the old behavior should pin to a pre-SDK tag. Deleted: - scripts/send_worker.ts (legacy raw API worker) - scripts/plan_checkpoint.ts (was no-op in SDK mode) Removed: - Legacy branch in send_messages_to_letta.ts - Early prompt notification in sync_letta_memory.ts - sendMessageToConversation from conversation_utils.ts - Checkpoint hook from hooks.json Written by Cameron ◯ Letta Code "Perfection is achieved, not when there is nothing more to add, but when there is nothing left to take away." - Antoine de Saint-Exupéry
- Remove checkpoint docs, legacy `off` mode, and stale log file refs from README - Clean up verbose per-message stream logging in SDK worker (keep tool calls + errors) - Bump version to 2.0.0 in package.json and plugin.json Written by Cameron ◯ Letta Code "Less is more." - Ludwig Mies van der Rohe
off now means "no client-side tools" — Sub still receives transcripts via the SDK transport but can only use memory operations. Useful for listen-only agents that observe without filesystem access. Written by Cameron ◯ Letta Code "Listening is an art that requires attention over talent." - Dean Jackson
send_worker.log → send_worker_sdk.log Written by Cameron ◯ Letta Code "The devil is in the details." - Proverb
Written by Cameron ◯ Letta Code "Clean code always looks like it was written by someone who cares." - Robert C. Martin
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 free
to 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.
Summary
LETTA_SDK_TOOLS=off). All message delivery to Sub now goes through the Letta Code SDK exclusively.sync_letta_memory.tsandplan_checkpoint.tswere racing the SDK worker on the same conversation (409 errors). Legacy paths now removed entirely.resumeSession(conversationId)so SDK sessions appear in the correct conversation thread on app.letta.com.tool_guidelinesmemory block and bundled.affile to tell Sub about Read/Glob/Grep client-side tools.Breaking changes
LETTA_SDK_TOOLS=offis removed. The SDK is now required.send_worker.ts(legacy raw API worker) deletedplan_checkpoint.ts(was a no-op in SDK mode) deletedFiles deleted
scripts/send_worker.ts— legacy worker that sent messages via rawfetch()to the Letta APIscripts/plan_checkpoint.ts— PreToolUse checkpoint hook, was already a no-op in SDK modeNet diff
~750 lines deleted
Test plan
Written by Cameron ◯ Letta Code
"Perfection is achieved, not when there is nothing more to add, but when there is nothing left to take away." - Antoine de Saint-Exupéry