feat: skill registry points at the 10 current skills in brightdata/skills - #27
Open
nirsha-brd wants to merge 3 commits into
Open
feat: skill registry points at the 10 current skills in brightdata/skills#27nirsha-brd wants to merge 3 commits into
nirsha-brd wants to merge 3 commits into
Conversation
This was referenced Aug 27, 2026
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
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
Replaces the skill installer registry with the 10 skills that brightdata/skills ships after the v2 stack (#32, #34, #35, #36): agent-onboarding, scrape, datasets, fetch, search, browser, billing, brightdata-cli, brightdata-mcp, brightdata-sdk.
Two files changed:
src/utils/skill-installer/brightdata-skills.ts(the registry, names, descriptions and URLs in the existing entry format) andsrc/commands/scrape.ts(one added line, see below). No other code touched.Why
The v2 stack makes brightdata/skills ship exactly these 10 skills and removes the 17 retired folders. The current registry lists 6 names that will no longer exist on main (data-feeds, bright-data-mcp, bright-data-best-practices, design-mirror, python-sdk-best-practices, scraper-builder). Each of those fails at install time with a per-skill fetch error. This PR points the menu at the real set.
Verified
tsccompiles the branch clean.skill add nonsense-nameprints the clean unknown-skill error listing the 10 names, exit 1.skill add datasets(in the registry, not on skills main until the stack merges) fails as a contained per-skill 404 with a clean summary, no crash, exit 1.skill add scrapeperforms a real install across 10 detected agents, exit 0.Sequencing
Safe to merge before or after the skills stack. Before: the not-yet-published names fail with the contained per-skill error shown above. After: all 10 install. Ideal order is skills first, then this, then a release.
Also in this PR
.alias('fetch')on the scrape command, sobdata fetch <url>works as the same Web Unlocker call.bdata scrapekeeps working forever. Verified: no name collision with any command, tsc clean, help showsscrape|fetch, full vitest suite green minus one pre-existing ipc flake that fails identically without this change, and an org-wide code search shows nothing anywhere invokesbdata fetchtoday, so the alias cannot break any consumer.