-
Notifications
You must be signed in to change notification settings - Fork 3.2k
feat(ocr): added reducto and pulse for OCR #2843
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Greptile SummaryThis PR adds two new OCR tools—Reducto and Pulse—for extracting text from documents. Both implementations follow the established patterns in the codebase with proper authentication, file access verification, and comprehensive type definitions. Key additions:
Implementation highlights:
The code is well-structured, follows existing patterns, and includes appropriate error handling. Confidence Score: 5/5
Important Files Changed
Sequence DiagramsequenceDiagram
participant User
participant Block as Reducto/Pulse Block
participant Parser as Parser Tool
participant APIRoute as API Route
participant Auth as Authentication
participant Storage as File Storage
participant OCR as OCR API
User->>Block: Configure with file and settings
Block->>Parser: Invoke tool with parameters
Parser->>Parser: Validate inputs
Parser->>APIRoute: POST to parse endpoint
APIRoute->>Auth: Verify user authentication
Auth-->>APIRoute: Authentication successful
alt Internal workspace file
APIRoute->>Storage: Extract storage information
APIRoute->>Auth: Check file permissions
Auth-->>APIRoute: Permission granted
APIRoute->>Storage: Generate temporary download URL
Storage-->>APIRoute: Return temporary URL
else External file URL
APIRoute->>APIRoute: Use provided URL
end
APIRoute->>OCR: Send file for processing
OCR-->>APIRoute: Return extracted content
APIRoute->>Parser: Return results
Parser->>Parser: Transform response
Parser-->>Block: Deliver output
Block-->>User: Show extracted text
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
18 files reviewed, 1 comment
Summary
Type of Change
Testing
Tested manually
Checklist