feat(idp): add ui-mcp/siza namespaces, expand feature-toggle tests#138
Conversation
- Add 'ui-mcp' and 'siza' as canonical ToggleNamespace values - Deprecate legacy 'uiforge-mcp' and 'uiforge-webapp' with backwards compat - Update CLI VALID_NAMESPACES and help text to show new names - Tests: 18 → 27 (+9 covering canonical namespaces, custom strategies, disabled filtering, empty-namespace edge case)
|
Warning Rate limit exceeded
⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the 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 configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (3)
✨ Finishing Touches🧪 Generate unit tests (beta)
📝 Coding Plan
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. Comment |
Project Scorecard |
|



Summary
Completes the Forge Space rebranding in the IDP feature-toggle module by adding canonical namespace names while keeping full backwards compatibility.
Changes
patterns/idp/feature-toggles/schema.ts'ui-mcp'and'siza'as canonicalToggleNamespacevalues'uiforge-mcp'and'uiforge-webapp'with deprecation commentspatterns/idp/feature-toggles/cli.ts'ui-mcp'and'siza'toVALID_NAMESPACESarrayTests: 18 → 27 (+9 new)
accepts ui-mcp namespace— creates toggle in canonical namespaceaccepts siza namespace— creates enabled toggle in siza namespaceaccepts legacy uiforge-mcp namespace for backwards compatfilters by ui-mcp namespacefilters by siza namespacepersists custom strategy config— gradual-rollout with parameterspersists user-ids strategyfilters by disabled (enabled: false)returns empty array for namespace with no togglesMetrics