feat: tiktok search #13
Merged
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
Contributor
There was a problem hiding this comment.
Pull Request Overview
Implements TikTok search functionality by adding support for both query-based and trending video searches using Apify actors. The PR defines type structures to enable tee-worker and tee-indexer integrations.
- Adds comprehensive type definitions for TikTok search results from both query and trending APIs
- Extends job capabilities to support TikTok search operations alongside existing transcription
- Updates argument handling to support multiple TikTok operation types with proper validation
Reviewed Changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| types/tiktok.go | Defines comprehensive TikTok data structures for search results from both query and trending APIs |
| types/jobs.go | Adds TikTok search capabilities and updates job configuration to support new search operations |
| args/unmarshaller.go | Updates TikTok argument unmarshalling to handle multiple operation types with capability-based routing |
| args/tiktok.go | Implements new argument types for TikTok search operations with validation logic |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
rapidfix
reviewed
Aug 22, 2025
mcamou
reviewed
Aug 22, 2025
Collaborator
|
moving back to in progress as seems there are still things to do here (PR feedback and merge conflict) |
mcamou
approved these changes
Aug 26, 2025
Collaborator
mcamou
left a comment
mcamou
left a comment
There was a problem hiding this comment.
Excellent! Thank you for your patience!
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.
What
Implements tiktok search (both by query, and by trending), using the following apify actors:
This is just for types to support tee-worker and tee-indexer integrations!
Why
We need a way for the user to be able to search tiktok videos -> this defines the needed types to support the two ways of searching
Fixes: https://github.com/masa-finance/tee-indexer/issues/299