Skip to content

Conversation

@pokey
Copy link
Owner

@pokey pokey commented Oct 8, 2025

  • Migrate to be a PR against langchain-ai once feat(langchain/createAgent): Add support for modelSettings to middleware langchain-ai/langchainjs#9110 is merged
  • Add a proper description
  • Add todo to switch to using BaseStore instead of having new FileSystemProvider interface, and default to using graph if no store provided
  • Probably export codeExecutionAnthropic instead of codeExecution. Internally can just wrap existing impl. Something like:
    export const codeExecutionMiddlewareAnthropic = (fileProvider) => codeExecutionMiddleware(
      new AnthropicContainerProvider(),
      fileProvider
    );
    
    // ...then clients do:
    
    const middleware = codeExecutionMiddlewareAnthropic(
      new MemoryFileProvider()
    );
    This gives us flexibility to switch to anthropic-specific and openai-specific middleware impl in the future because current container abstraction is awkward
  • Note about forking off file system middleware, potentially using one from deepagent
  • Then current codeExecution middleware abstraction goes away and anthropic-specific and openai-specific middleware just use file system middleware. Rest of stuff can become utility functions?
  • Daytona becomes example
  • Look into intermittent issue with extra tools example. Something about constructing paths is confusing the llm ie whehter or not to include the /tmp/ prefix. See https://smith.langchain.com/public/79d05f34-2924-4707-b0ae-73b8da3d9a9d/r

dqbd and others added 30 commits August 7, 2025 01:13
@pokey pokey changed the title Add OpenAI container provider Add code execution middleware Oct 8, 2025
name: "codeExecutionMiddleware",
stateSchema,
tools: containerProvider.tools,
beforeModel: async (state) => {
Copy link
Owner Author

Choose a reason for hiding this comment

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

Prob want a way for middleware to shut itself off if the model is unsupported, eg Anthropic middleware with OpenAI model

@pokey pokey force-pushed the pokey/code-execution branch from 06102b5 to 1e2612f Compare October 14, 2025 15:30
@pokey pokey force-pushed the pokey/bases/code-execution branch from 0dd69d5 to 33feab1 Compare October 14, 2025 15:30
@pokey pokey force-pushed the pokey/code-execution branch 6 times, most recently from a70bb99 to bdbfa73 Compare October 15, 2025 12:44
Co-authored-by: Christian Bromann <[email protected]>
Co-authored-by: Eugene Yurtsev <[email protected]>
Co-authored-by: Jorge Lanzarotti <[email protected]>
Co-authored-by: itaismith <[email protected]>
Co-authored-by: Dylan Dignan <[email protected]>
Co-authored-by: Xuma <[email protected]>
Co-authored-by: Jack Ye <[email protected]>
Co-authored-by: Uriel Chemouni <[email protected]>
Co-authored-by: Rafael Reis <[email protected]>
Co-authored-by: Mikias Tilahun Abebe <[email protected]>
Co-authored-by: jacoblee93 <[email protected]>
Co-authored-by: Sriram Nyshadham <[email protected]>
Co-authored-by: Lucian Mocanu <[email protected]>
Co-authored-by: Elliot Scribner <[email protected]>
Co-authored-by: anadi45 <[email protected]>
Co-authored-by: ghdcksgml1 <[email protected]>
Co-authored-by: Caspar Broekhuizen <[email protected]>
Co-authored-by: Rito <[email protected]>
Co-authored-by: David Duong <[email protected]>
Co-authored-by: Yohan Lasorsa <[email protected]>
Co-authored-by: Duda Nogueira <[email protected]>
Co-authored-by: Piotr <[email protected]>
Co-authored-by: Matt Blum <[email protected]>
Co-authored-by: Ranjeet Baraik <[email protected]>
Co-authored-by: Zac Lovoy <[email protected]>
Co-authored-by: J3tCh3n <[email protected]>
Co-authored-by: Yury Kozyrev <[email protected]>
Co-authored-by: Yury Kozyrev <[email protected]>
Co-authored-by: Pokey Rule <[email protected]>
Co-authored-by: Durran Jordan <[email protected]>
Co-authored-by: rliu6915 <[email protected]>
Co-authored-by: Dag7 <[email protected]>
@pokey pokey force-pushed the pokey/code-execution branch from bdbfa73 to 8fcaa72 Compare October 16, 2025 10:11
@pokey pokey force-pushed the pokey/code-execution branch from 8fcaa72 to 8eff2cb Compare October 16, 2025 10:22
@pokey
Copy link
Owner Author

pokey commented Oct 16, 2025

Closing in favour of langchain-ai#9209

@pokey pokey closed this Oct 16, 2025
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.