chore: marketplace listing prep — licence field, 400x400 icon, llms-install.md, registry remotes by SavioBS629 · Pull Request #464 · browserstack/mcp-server · GitHub
Skip to content

chore: marketplace listing prep — licence field, 400x400 icon, llms-install.md, registry remotes - #464

Merged
gaurav-singh-9227 merged 4 commits into
browserstack:mainfrom
SavioBS629:chore/marketplace-listing-prep
Oct 7, 2026
Merged

gaurav-singh-9227 merged 4 commits into
browserstack:mainfrom
SavioBS629:chore/marketplace-listing-prep

Conversation

@SavioBS629

@SavioBS629 SavioBS629 commented Oct 6, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

All four cross-cutting engineering prep items from the Cline, Smithery and Kiro Marketplaces — MCP Server Listing Plan, No code changes. Four commits: one per concern, plus the README revert.

1. Licence field: ISC → AGPL-3.0-only (0d33a38)

  • package.json declares "license": "ISC", while the repo's LICENSE file is the full GNU AGPL-3.0 text and GitHub detects the project as AGPL-3.0.
  • The ISC value dates from the initial commit (19 Apr 2025) and is the npm init default; the AGPL LICENSE was added deliberately three days later. The field was never updated.
  • Today the npm page for @browserstack/mcp-server shows ISC, so downstream consumers and marketplace reviewers see a licence that contradicts the repo.
  • Sets the SPDX identifier AGPL-3.0-only in package.json and the lockfile's root entry.

2. 400×400 icon (e92772c)

  • Adds assets/icon-400.png: the official BrowserStack mark at 400×400, RGBA, transparent background, ~51 KB.
  • Source is the BrowserStack GitHub org avatar (460×460), downscaled with Lanczos. Not an upscale of the existing 200×200 file, which is the generic MCP glyph rather than the BrowserStack brand.
  • Cline's submission hard-requires a 400×400 PNG; Kiro and Smithery listings also show an icon. Once merged, this file's raw GitHub URL is what the Cline issue (fix(security): encode tool-argument IDs in API URL paths (block path-segment injection) #468 refresh), the Kiro feature request, and the Smithery listing will reference.

3. llms-install.md (22f1289)

  • Agent-oriented install guide at the repo root. Cline's reviewers test that Cline can set the server up from the README or llms-install.md alone; our README is 800+ lines and mixes marketing with setup.
  • Covers the hosted remote (https://mcp.browserstack.com/mcp, Streamable HTTP, OAuth discovery) and the local npx -y @browserstack/mcp-server@latest stdio install, each with configs for generic clients, VS Code and Kiro; optional env vars; a verification step; troubleshooting.

4. server.json remotes entry (22f1289)

  • Adds {"type": "streamable-http", "url": "https://mcp.browserstack.com/mcp"}. Registry mirrors (GitHub MCP Registry, Kiro) only advertise the npm install today.
  • Validated locally against the 2025-10-17 server.schema.json with ajv.
  • Checked the registry's validator: remotes only need HTTPS, no localhost, no template variables. There is no namespace/domain ownership check for remotes, so publishing under io.github.browserstack/* with a mcp.browserstack.com URL is fine and the release-time mcp-publisher publish step is unaffected.

5. README "Add to Kiro" links: moved to #463

Originally in commit 22f1289, reverted in f404f88. The Kiro links conflicted with the README rewrite in #457 and featuring a new client in the README is a product call, so they now ride the stacked README PR #463 where product can weigh in. The Kiro install config stays in llms-install.md here.

Review note

The licence change alters the string published to npm on the next release, so it should get a Legal/manager sign-off before merge. If the intended licence is in fact ISC, the correct fix is instead to replace the LICENSE file.

Not in this PR

  • Cline re-test from a fresh VS Code using only llms-install.md (QA step). Worth doing on this branch before the Cline issue is refreshed.
  • Smithery static server card at /.well-known/mcp/server-card.json. Lives in the remote-mcp-server wrapper and is only needed if Smithery's OAuth-aware scan fails; the hosted endpoint already returns the correct 401 + WWW-Authenticate and both well-known OAuth documents.
  • LICENSE has no BrowserStack copyright line and the README has no License section; cheap follow-ups once the licence is confirmed.

Test plan

  • npm run build passes (no source changes)
  • npm pack --dry-run shows the manifest with AGPL-3.0-only
  • assets/icon-400.png previews correctly on GitHub at 400×400
  • mcp-publisher publish succeeds on the next tag with the new remotes entry
  • Fresh Cline install from llms-install.md alone connects and runs a tool

🤖 Generated with Claude Code

SavioBS629 and others added 2 commits October 6, 2026 12:39
The "license" field has read ISC since the initial commit, which is the
npm init default and was never a deliberate choice. The LICENSE file added
three days later is the full AGPL-3.0 text and GitHub detects the repo as
AGPL-3.0, but every npm consumer of @browserstack/mcp-server currently sees
ISC on the package page. Align the manifest (and the lockfile's root entry)
with the actual licence using the SPDX identifier.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Cline's MCP Marketplace requires a 400x400 PNG; Kiro and Smithery also
display a server icon. The existing assets are a 200x200 generic MCP glyph
and a 1387x331 wordmark with a baked-in dark background, neither of which
fits. This is the official BrowserStack mark (the GitHub org avatar,
460x460 RGBA) downscaled with Lanczos to 400x400, transparent background.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Oct 6, 2026 •

Copy link
Copy Markdown

Marketplace listing prep (Cline, Smithery, Kiro), cross-cutting items 2
and 3 plus the Kiro step 1 from the ENG plan page.

- llms-install.md: agent-oriented install guide. Cline reviewers test
  that an agent can set the server up from README or llms-install.md
  alone; the README is 800+ lines of mixed marketing and setup. Covers
  the hosted OAuth remote and the npx stdio install for the common
  clients, optional env vars, a verification step and troubleshooting.
- server.json: add a `remotes` entry for https://mcp.browserstack.com/mcp
  (streamable-http). Registry mirrors (GitHub, Kiro) only advertise the
  npm install today. Validated against the 2025-10-17 server schema; the
  registry's remote validation only requires https, no localhost and no
  template variables, so the io.github namespace is unaffected.
- README: "Add to Kiro" deep links (hosted and local) beside the one-click
  VS Code/Cursor buttons in both places they appear, and a Kiro entry in
  the per-client config list.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
@SavioBS629 SavioBS629 changed the title Chore/marketplace listing prep chore: marketplace listing prep — licence field, 400x400 icon, llms-install.md, registry remotes, Add-to-Kiro links Oct 6, 2026
The README is being restructured in browserstack#457, and the Kiro config entry
conflicts with that rewrite. The links move to the stacked README PR
(browserstack#463) so product can weigh in on featuring Kiro; the Kiro install
config stays in llms-install.md.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
@SavioBS629 SavioBS629 changed the title chore: marketplace listing prep — licence field, 400x400 icon, llms-install.md, registry remotes, Add-to-Kiro links chore: marketplace listing prep — licence field, 400x400 icon, llms-install.md, registry remotes Oct 6, 2026
@gaurav-singh-9227
gaurav-singh-9227 merged commit f7db430 into browserstack:main Oct 7, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants