-
Notifications
You must be signed in to change notification settings - Fork 6
feat: linkedin #176
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
feat: linkedin #176
Conversation
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.
Pull Request Overview
This PR adds LinkedIn profile scraping functionality to the tee-worker using the Apify platform integration. The implementation provides LinkedIn profile search capabilities through a new LinkedInScraper that follows the existing patterns established for other scrapers.
- Adds LinkedIn job support via Apify API with comprehensive testing
- Fixes a type conversion bug in Twitter scraper error handling
- Updates project dependencies and build configurations
Reviewed Changes
Copilot reviewed 12 out of 13 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| internal/jobserver/jobserver.go | Registers LinkedIn job type in the job server |
| internal/jobs/twitter.go | Fixes string conversion bug in error message |
| internal/jobs/stats/stats.go | Adds LinkedIn-specific statistics tracking |
| internal/jobs/linkedinapify/ | New package for LinkedIn Apify client implementation |
| internal/jobs/linkedin*.go | Main LinkedIn scraper implementation and tests |
| internal/apify/actors.go | Adds LinkedIn actor configuration |
| go.mod | Updates Go version and dependencies |
| Makefile | Adds LinkedIn test target |
| Dockerfile | Updates base image reference |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
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.
Pull Request Overview
Copilot reviewed 13 out of 14 changed files in this pull request and generated no new comments.
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
rapidfix
left a comment
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.
Looking good here!
Pull Request Summary: LinkedIn Integration for TEE Worker
🎯 Overview
This PR implements comprehensive LinkedIn integration support in the
tee-workerrepository, enabling LinkedIn profile scraping and search capabilities through the Apify platform. The implementation follows the established patterns for other job types while adding LinkedIn-specific functionality.📊 Changes Summary
🚀 Key Features Added
1. LinkedIn Job Processing (
internal/jobs/linkedin.go)harvestapi~linkedin-profile-searchactorCapSearchByProfilecapability2. LinkedIn Apify Client (
internal/jobs/linkedinapify/client.go)3. Comprehensive Test Suite (
internal/jobs/linkedin_test.go)4. Enhanced Statistics Tracking (
internal/jobs/stats/stats.go)LinkedInProfiles: Tracks returned profile countLinkedInQueries: Tracks search query countLinkedInErrors: Tracks error occurrences5. Apify Actor Configuration (
internal/apify/actors.go)harvestapi~linkedin-profile-searchactor6. Job Server Integration (
internal/jobserver/jobserver.go)🔧 Technical Improvements
Build System Updates
test-linkedinMakefile targetCode Quality
Testing Infrastructure
🎯 LinkedIn Capabilities Enabled
🔄 Integration Points
Apify Platform
harvestapi~linkedin-profile-searchTEE Types Integration
Statistics & Monitoring
✅ Quality Assurance
Testing
Code Standards
Performance
🚀 Deployment Ready
This PR provides a complete LinkedIn integration solution that:
The implementation is production-ready and follows all established patterns and best practices within the TEE worker ecosystem.