chore: remove the pullRequestsPage org feature flag - #198
Conversation
The Pull requests page is available to every organization now, so the flag is gone from the monorepo. `src/host/scopes.ts` does not sync, so its copy of `OrgFeatures` has to drop the key here or the shared `lib/orgFeatures.ts` list stops type-checking against it. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01WBJ1qXTQEZ1aqcAVLQiCKZ
|
| Topic | Details | |||||||||
|---|---|---|---|---|---|---|---|---|---|---|
| Feature flag removal | Remove pullRequestsPage from OrgFeatures and ORG_FEATURE_KEYS so the server’s feature typing and validation match the monorepo.Modified files (2)
Latest Contributors(2)
| |||||||||
| Feature docs | Update documented --features and CURRENTS_MCP_FEATURES values to stop advertising the removed flag.Modified files (1)
Latest Contributors(2)
|
|
Understand this PR’s impact Explore downstream dependencies and potential security impact with Blast Radius. No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Essentials Run ID: 📒 Files selected for processing (3)
💤 Files with no reviewable changes (2)
Included review availability: 0 reviews are currently available. Your included PR review attempts over the past 7 days set your current allowance at 1 review per hour. 📝 WalkthroughWalkthroughThe change removes ChangesOrganization feature flag contract
Priority: ⬇️ Low Estimated code review effort: 1 (Trivial) | ~5 minutes Change: Other Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Comment |
The Pull requests page is available to every organization now, so
pullRequestsPageis gone from the monorepo (currents-dev/currents#3865).mcp-server/src/host/**is excluded from the source sync, so this repo's ownOrgFeatureshas to drop the key here. Without it, the sharedmcp-server/src/lib/orgFeatures.tslist — typed{ [K in OrgFeatureKey]: K }so the two cannot drift — no longer type-checks once the monorepo change lands, which is what the monorepo's sync check reports today.src/host/scopes.ts: droppedpullRequestsPagefromOrgFeatures.src/lib/orgFeatures.ts: dropped the same key, matching the monorepo copy that the next sync will overwrite this file with.README.md: the flag is no longer a name--features/CURRENTS_MCP_FEATURESaccepts, and the server exits on a name it does not recognize.Merge this before currents-dev/currents#3865.
npm run verifypasses locally: format, readme sync, types, build, 768 tests.🤖 Generated with Claude Code
https://claude.ai/code/session_01WBJ1qXTQEZ1aqcAVLQiCKZ
Need help on this PR? Tag
@codesmith-botwith what you need. Autofix is disabled.Summary by CodeRabbit
Documentation
pullRequestsPageand reflect the currently supported options.Changes
pullRequestsPageorganization feature flag from the public feature configuration and recognized feature list.