Skip to content

OpenAI compatibility#21

Merged
jtoman merged 8 commits into
masterfrom
jtoman/openai-memory-backend
Jul 2, 2026
Merged

OpenAI compatibility#21
jtoman merged 8 commits into
masterfrom
jtoman/openai-memory-backend

Conversation

@jtoman

@jtoman jtoman commented Jun 1, 2026

Copy link
Copy Markdown
Contributor
  • Changes shape of input schema for memory tools targeting openAI. We need to actually define a proper schema, as gpt-5.* don't have some built in schema its trained on
  • re-introduce the invoke and ainvoke wrappers to paper over the difference in input list schemas (anthropic allows bare strings in lists, openai doesn't)
  • paper over "standardized" token usage format from LC. Exposed as a new entrypoint to not break existing upstream users.

jtoman added 3 commits May 26, 2026 17:01
Keeps a singular tool, but with a more structured tool schema. The "flat
schema of all possible fields" is an artifact of trying to fit
anthropic's internal schema into langgraph's infrastructure. OpenAI
we're defining the tooling, so we can do better.
Paper over differences in "standardized" (lol) usage metrics, and
normalize `list[dict | str]` and `list[dict]` (openai doesn't let you
intermix string and dicts within a list of content, booo)
@jtoman jtoman requested a review from naftali-g June 1, 2026 20:58
Comment thread graphcore/tools/memory.py Outdated
Comment thread graphcore/tools/memory.py Outdated
Comment thread graphcore/tools/memory.py Outdated
Comment thread graphcore/tools/memory.py Outdated
Comment thread graphcore/utils.py
Comment thread graphcore/tools/memory.py

@shellygr shellygr left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

looks like there are some comments by Naftali that you acknowledged but didn't fix yet.

otherwise lgtm

@jtoman

jtoman commented Jun 30, 2026

Copy link
Copy Markdown
Contributor Author

looks like there are some comments by Naftali that you acknowledged but didn't fix yet.

otherwise lgtm

you're right, I hallucinated that I did

@jtoman jtoman requested review from naftali-g and shellygr June 30, 2026 18:12
@jtoman jtoman merged commit 8b9d320 into master Jul 2, 2026
2 checks passed
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.

3 participants