Skip to content

Revert MCP registry branding (#63)#64

Open
tonychang04 wants to merge 1 commit into
masterfrom
ins-306-revert-registry-branding
Open

Revert MCP registry branding (#63)#64
tonychang04 wants to merge 1 commit into
masterfrom
ins-306-revert-registry-branding

Conversation

@tonychang04
Copy link
Copy Markdown
Member

@tonychang04 tonychang04 commented Jun 2, 2026

Reverts #63 — backs out the websiteUrl + logo icons added to server.json. Not needed for now.

🤖 Generated with Claude Code


Summary by cubic

Remove websiteUrl and icons from server.json to undo #63 and return the manifest to a minimal, standard MCP shape. This supports INS-306 by keeping branding and discovery data in /.well-known/mcp/server-card.json and avoiding non‑standard fields in server.json.

Written for commit 530e148. Summary will update on new commits.

Review in cubic

Summary by CodeRabbit

  • Chores
    • Removed unused website URL and icon fields from the server manifest.

Backs out the server.json websiteUrl + icons added in #63; not needed for now.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Jun 2, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: a7693936-f7da-415a-9bde-17ce5e5cf838

📥 Commits

Reviewing files that changed from the base of the PR and between 4d3886d and 530e148.

📒 Files selected for processing (1)
  • server.json
💤 Files with no reviewable changes (1)
  • server.json

Walkthrough

Removes the websiteUrl field and icons array from the MCP server manifest in server.json. All other manifest metadata including name, description, repository URL, version, remotes, and environment variables remain intact.

Changes

Cohort / File(s) Summary
Manifest Metadata Cleanup
server.json
Removes 7 lines: websiteUrl field (pointing to insforge.dev) and icons array (GitHub-hosted PNG reference) from MCP server manifest. All other manifest fields unchanged.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

  • InsForge/insforge-mcp#63: Directly modifies the same server.json manifest by adding/removing websiteUrl and icons fields.

Suggested reviewers

  • jwfing

Poem

🐰 A manifest so clean and lean,
Two fields removed from the scene,
No icons glint, no website shines,
Just pure essentials left behind.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and concisely describes the main change: reverting MCP registry branding additions (websiteUrl and icons) from server.json.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch ins-306-revert-registry-branding

Comment @coderabbitai help to get the list of available commands and usage tips.

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