Add FileParserPlugin example for AI SDK v5 #51
+229
−0
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This was referenced Nov 12, 2025
Contributor
Author
|
Warning This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
This stack of pull requests is managed by Graphite. Learn more about stacking. |
There was a problem hiding this comment.
Pull Request Overview
This PR adds comprehensive FileParserPlugin examples for AI SDK v5, demonstrating PDF processing capabilities through OpenRouter. The implementation includes test fixtures with verification codes and examples for both URL-based and data URI-based PDF processing.
Key Changes:
- Added two FileParserPlugin examples: URL-based PDF processing (with Claude 3.5 Sonnet) and multi-size PDF testing (with GPT-4o-mini)
- Created PDF fixtures (small, medium, large, xlarge) with JSON metadata containing verification codes
- Updated
@openrouter/ai-sdk-providerto v1.2.2 for improved schema validation
Reviewed Changes
Copilot reviewed 4 out of 8 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
typescript/ai-sdk-v5/src/plugin-file-parser/file-parser-pdf-url.ts |
New example demonstrating PDF processing via public URLs using Claude 3.5 Sonnet (native PDF support) |
typescript/ai-sdk-v5/src/plugin-file-parser/file-parser-all-sizes.ts |
New example testing multiple PDF sizes with explicit FileParserPlugin configuration for GPT-4o-mini |
typescript/ai-sdk-v5/src/plugin-file-parser/README.md |
Documentation for the FileParserPlugin examples |
typescript/ai-sdk-v5/package.json |
Dependency update to @openrouter/ai-sdk-provider v1.2.2 |
fixtures/pdfs/small.pdf |
Test fixture PDF file (33KB) with verification code for testing |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
subtleGradient
force-pushed
the
03-prompt-caching-aisdk
branch
from
08b17d0 to
5138ba6
Compare
subtleGradient
force-pushed
the
pdf-example-aisdk
branch
3 times, most recently
from
edcaa49 to
7bf86b2
Compare
subtleGradient
force-pushed
the
03-prompt-caching-aisdk
branch
from
42726ef to
6a73a21
Compare
- Add PDF fixtures (small, medium, large, xlarge) with verification codes - Create comprehensive file-parser example testing all PDF sizes - Uses AI SDK's file attachment format with automatic plugin enablement - Validates extraction of verification codes from PDFs
- Update AI SDK example to use shared fixtures from pdf-example-fetch - Use absolute paths so examples work from any directory - Read verification codes from JSON metadata (inherited from parent branch) - Add file-parser-pdf-url.ts example
- Remove 'any' type assertion, use proper type narrowing - Fix import order with biome autofix
- Switch model from Claude 3.5 to GPT-4o-mini - Explicitly configure FileParserPlugin with Mistral OCR - Add clarifying comment about plugin necessity
Add @ts-expect-error with FIXME comment for usage token properties.
Include both examples with running instructions as suggested by Copilot review.
Copilot feedback: Overview section was incomplete/misleading about plugin behavior. Solution: Remove Overview section - complete behavior docs are in example file headers.
subtleGradient
changed the base branch from
03-prompt-caching-aisdk
to
graphite-base/51
This was referenced Nov 12, 2025
subtleGradient
force-pushed
the
graphite-base/51
branch
from
6a73a21 to
0147fbe
Compare
subtleGradient
force-pushed
the
pdf-example-aisdk
branch
from
7bf86b2 to
34d735c
Compare
subtleGradient
changed the base branch from
graphite-base/51
to
03-prompt-caching-aisdk
louisgv
reviewed
Nov 13, 2025
|
|
||
| const usage = result.providerMetadata?.openrouter?.usage; | ||
| if (usage && typeof usage === 'object' && 'cost' in usage) { | ||
| const cost = usage.cost as number; |
There was a problem hiding this comment.
it should assert the type
assert(typeof usage.cost === 'number')
Or something like that, I think
louisgv
approved these changes
Nov 13, 2025
louisgv
reviewed
Nov 13, 2025
| console.log('URL: https://bitcoin.org/bitcoin.pdf'); | ||
| console.log(); | ||
|
|
||
| try { |
There was a problem hiding this comment.
Why do we need try/catch here btw if the entry crash it will simply crash right?
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Add FileParserPlugin example for AI SDK v5
Add shared fixtures module and JSON metadata
Fix stylecheck issues in AI SDK examples
Fix large.pdf OCR readability and switch to Claude 3.5 Sonnet
Update @openrouter/ai-sdk-provider to 1.2.2
Fixes schema validation error for file parser plugin responses.
The file type annotation is now properly supported in the response schema.
Update FileParserPlugin config for models without native PDF support
FileParserPlugin with Mistral OCR - Add clarifying comment about plugin
necessity