Lars Grammel

Results 105 comments of Lars Grammel

I'd love to help with the adoption of new ES2015+ language features. I could tune/modify the bot ( https://p42.ai/ ) to fit the needs of D3 exactly, so all you'd...

What I had in mind is a configuration parameter similar to the skipTests parameter in surefire: http://maven.apache.org/plugins/maven-surefire-plugin/examples/skipping-test.html I've got a project with some builds where the coffeescript compilation is not...

There should always be a message at the point when text fragments start streaming in, so the root cause could be different. Can you share the code that caused this...

> I am curious though, in this `useAssistant()` code, how does it ensure that there's always a message in the array before hitting those `lastMessage` code? [This part is async...

> @lgrammel Sorry, I'm still confused. Isn't `assistant_control_data` enqueued prior to `forwardStream` being called here? > > https://github.com/vercel/ai/blob/fac9a6ee7c3aca975f7dd9f2f40cda1b82a1f8f3/packages/core/streams/assistant-response.ts#L88-L105 Yes, first there is a control data message, then the `assistant_message`, and...

> If the data control message comes first, then couldn't this fail since `messages` may be empty then `lastMessage` is undefined? > > https://github.com/vercel/ai/blob/main/packages/core/react/use-assistant.ts#L212-L223 Yes, good point. This would need...

@tchanxx Can you update the PR to only check in the control data part, and move the if check outside of setMessages?

There is a type in the URL, the correct URL is https://sdk.vercel.ai/docs/guides/providers/huggingface

I've tried to reproduce the error with the following example: ```ts // route.ts import { NextResponse } from 'next/server'; // IMPORTANT! Set the runtime to edge export const runtime =...

@jeremyphilemon @shuding not sure if I like this new version better. Replacing system messages with user seems somewhat strange, and I'm not sure if it might cause issues.