Michael Bleigh

Oh also I think we should put this work into a launch branch - I created `launch/mcp-redux` and made #2896 to squash my own work and create a chain of...

Big +1 to bringing `toolResult` back - in my experience models are pretty good at understanding JSON, so except in pretty complex circumstances I'm not sure there's really a need...

> Evaluating the functions inside of weightedArrayElements will make it impossible for the method to return functions itself, in case the functions itself are the expected values. Fair, it might...

> Could you explain this to me? As far as I'm concerned, functions that return another function is a pretty common pattern in JS: Honestly it's probably fine to close...

Thanks for the bump @njpearman! I don't think anyone is actively working on this yet although I'd hope it will come off the backlog soon. If you want to take...

> I generally like it - but a few questions: > > **1. Streaming with `generateX`** > > `streamGenerate()` today is `generateStream()`. Do to mean to change that, or just...

I think destructuring is probably what I'm leaning toward at the moment since it provides the best balance between "one-liner friendly" and "can still get at metadata". I didn't realize...

Hi @njpearman thanks for the great start! I probably should have given you some pointers before sending you off to write the PR, but a few high-level comments: 1. JSON...

> @mbleigh > > I've updated the image paths in `server.mdx` in this Pull Request (#352) to use absolute paths from the root (e.g., `/images/weather-alerts.png`) instead of relative paths. This...

Hi, honestly I think this just fell through the cracks - thanks for the PR! I think I'd take a slightly different approach to this. Instead of a `sendGenkitContext` option...