feat: add url field to 9 search adapters (67% -> 97% coverage)#414
Merged
jackwener merged 4 commits intojackwener:mainfrom Mar 25, 2026
Merged
Conversation
Add a 'url' field to the TikTok search adapter output, constructed from the author's uniqueId and the video id returned by the API. This allows downstream consumers (AI agents, pipelines, scripts) to link directly to each video instead of only having the author handle. The URL format is: https://www.tiktok.com/@{author}/video/{videoId} Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Add url output to search commands that were missing direct links: YAML adapters: - hackernews: surface existing url from map step into columns - zhihu: pass computed url through map step into columns - linux-do: construct url from topic id - instagram: construct profile url from username - xueqiu: pass computed url through map step into columns TS adapters: - arxiv: surface existing url from parseEntries into return + columns - apple-podcasts: add collectionViewUrl from iTunes API - medium: add url to columns (already computed in utils) - weread: construct book url from bookId This brings search adapter url coverage from 67% to 97% (32/33). The only adapter without url is dictionary (word lookup, no URL concept). Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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
Add
urlfield to 9 search adapters that were missing direct links in their output.Why
Before this PR, only 22 out of 33 search adapters (67%) included a
urlfield. This meant downstream consumers (AI agents, data pipelines, scripts) could not link to the actual content for 11 adapters.After this PR, 32 out of 33 search adapters (97%) include
url. The only exception isdictionary(word lookup has no URL concept).Changes
YAML adapters (5 files)
urlwas already in map step but missing fromcolumnsurlwas computed in evaluate but stripped by map step; now passed throughhttps://linux.do/t/topic/{id}from topic idhttps://www.instagram.com/{username}from user dataurlwas computed in evaluate but stripped by map step; now passed throughTS adapters (4 files)
urlwas already inparseEntries()utils but not surfaced in return/columnscollectionViewUrlfrom iTunes API responseurlwas already computed inloadMediumPosts()utils; just add to columnshttps://weread.qq.com/web/bookDetail/{bookId}from book dataCoverage improvement
Risk
Minimal - all changes only add a new field to existing output. No existing fields or behavior are modified.