Skip to content
This repository was archived by the owner on Apr 21, 2026. It is now read-only.

feat(shared-constants): add MCP_UI_SERVER_NAME, deprecate MCP_UIFORGE_SERVER_NAME#137

Merged
LucasSantana-Dev merged 1 commit into
mainfrom
fix/shared-constants-rebrand
Mar 15, 2026
Merged

feat(shared-constants): add MCP_UI_SERVER_NAME, deprecate MCP_UIFORGE_SERVER_NAME#137
LucasSantana-Dev merged 1 commit into
mainfrom
fix/shared-constants-rebrand

Conversation

@LucasSantana-Dev
Copy link
Copy Markdown
Member

Summary

Add MCP_UI_SERVER_NAME as the canonical Forge Space name for the UI generation MCP server, with a backwards-compatible deprecation of the stale MCP_UIFORGE_SERVER_NAME.

Changes

patterns/shared-constants/mcp-protocol.ts

/** @deprecated Use MCP_UI_SERVER_NAME instead */
export const MCP_UIFORGE_SERVER_NAME = 'uiforge';

/** Canonical name for the UI generation MCP server (ui-mcp) */
export const MCP_UI_SERVER_NAME = 'forge-ui';

Tests

  • New: MCP_UI_SERVER_NAME should be the canonical Forge Space server name
  • Updated index test: MCP_UI_SERVER_NAME exported from barrel

Why Backwards-Compatible

MCP_UIFORGE_SERVER_NAME is kept with a @deprecated JSDoc tag so existing consumers don't break. They can migrate to MCP_UI_SERVER_NAME at their own pace.

Metrics

Metric Before After
Tests 599 600
Old constant 'uiforge' 'uiforge' (kept, deprecated)
New constant MCP_UI_SERVER_NAME = 'forge-ui'

…_SERVER_NAME

- Add MCP_UI_SERVER_NAME = 'forge-ui' as canonical name for ui-mcp server
- Mark MCP_UIFORGE_SERVER_NAME as @deprecated (backwards-compatible)
- Add tests: MCP_UI_SERVER_NAME is 'forge-ui', exported from index
- Total: 600 tests
@LucasSantana-Dev LucasSantana-Dev requested a review from a team as a code owner March 15, 2026 05:55
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Mar 15, 2026

Warning

Rate limit exceeded

@LucasSantana-Dev has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 16 minutes and 55 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 3b52b1d0-1554-4256-a0ce-5872f791209e

📥 Commits

Reviewing files that changed from the base of the PR and between 56681ec and 07ffb53.

📒 Files selected for processing (3)
  • patterns/shared-constants/__tests__/index.test.ts
  • patterns/shared-constants/__tests__/mcp-protocol.test.ts
  • patterns/shared-constants/mcp-protocol.ts
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/shared-constants-rebrand
📝 Coding Plan
  • Generate coding plan for human review comments

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@github-actions
Copy link
Copy Markdown

Project Scorecard


Scorecard: 84/100 (B)
────────────────────────────────────────
  security: 100/100 (A)
  quality: 80/100 (B) — 1 violations
  performance: 67/100 (D) — 1 violations
  compliance: 75/100 (C) — 1 violations
  dependency: 100/100 (A)

Recommendations:
  - Increase test coverage to meet the 80% threshold
  - Extend log retention to at least 90 days for compliance

@LucasSantana-Dev LucasSantana-Dev merged commit 2f3e2e0 into main Mar 15, 2026
23 checks passed
@LucasSantana-Dev LucasSantana-Dev deleted the fix/shared-constants-rebrand branch March 15, 2026 05:56
@sonarqubecloud
Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
E Maintainability Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant