Skip to content

Conversation

@internalbuilds
Copy link

migrates to modules that support zod 4.x, LanguageModelV2 and other dependency updates.

fixes #52

Copy link

@ulvidamirli ulvidamirli left a comment

Choose a reason for hiding this comment

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

I tested and all works for me.

@mishushakov
Copy link
Owner

hey @internalbuilds, thanks for the pr!
i am noticing that you've removed the generics and thus the methods have lost type-safety - this is actually a requirement for the project, so we need to make sure both Zod schemas and JSON Schemas are type-inferred correctly.

@mishushakov
Copy link
Owner

I have published my v5 work in progress branch (based on your PR here)
https://github.com/mishushakov/llm-scraper/tree/v5

the only problem is I am getting type errors due to output option from Vercel AI SDK

 ✓ tests/scraper.test.ts (8 tests) 58303ms
   ✓ scrapes top 5 stories from Hacker News  6597ms
   ✓ scrapes top 5 stories from Hacker News (image format)  6764ms
   ✓ scrapes top 5 stories from Hacker News (markdown format)  5178ms
   ✓ scrapes top 5 stories from Hacker News (raw html)  6300ms
   ✓ scrapes top 5 stories from Hacker News (code generation)  4939ms
   ✓ scrapes top 5 stories from Hacker News (json schema)  5424ms
   ✓ scrapes example.com (streaming)  2813ms
   ✓ scrapes top stories from Hacker News (streaming, array)  20265ms

 Test Files  1 passed (1)
      Tests  8 passed (8)
   Start at  22:56:17
   Duration  58.69s (transform 32ms, setup 0ms, collect 253ms, tests 58.30s, environment 0ms, prepare 31ms)

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.

llm-scraper incompatible with latest @ai-sdk/openai

3 participants