chore(shared): remove unused feature flags

[saddlepaddle]

Summary

• Remove the unused billing-enabled and agent-commands-access feature flags
• BILLING_ENABLED was defined in FEATURE_FLAGS but never referenced anywhere in the codebase
• agent-commands-access was already absent from code but still existed in PostHog
• Both flags have been deleted from PostHog

Changes

• packages/shared/src/constants.ts: Removed BILLING_ENABLED: "billing-enabled" from the FEATURE_FLAGS constant

Test Plan

• bun run lint:fix — passed
• bun run typecheck — all 18 packages passed
• Searched entire codebase for any references to billing-enabled, BILLING_ENABLED, agent-commands-access — none found

Summary by CodeRabbit

• Chores
  • Removed the billing-enabled feature flag from feature configuration.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

---

📝 Walkthrough

Walkthrough

This PR removes the BILLING_ENABLED flag from the FEATURE_FLAGS object in the shared constants file. The removal eliminates the public feature gate "billing-enabled" from the exported FEATURE_FLAGS export, altering the object's shape and its type definition via the as const assertion.

Changes

Cohort / File(s)	Summary
Feature Flag Removal packages/shared/src/constants.ts	Removed the BILLING_ENABLED property from the FEATURE_FLAGS object, eliminating the "billing-enabled" public feature gate.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

• feat(billing): gate billing page behind feature flag #915: Adds and uses FEATURE_FLAGS.BILLING_ENABLED across billing-related routes and sidebar components, which directly conflicts with this removal.

Poem

🐰 Billing flags hop away, hop away, hop away,
Into the sunset of yesterday's code,
Constants simplified, the config's lighter today,
One less gate upon the feature road! ✨

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title accurately describes the main change: removing unused feature flags from the codebase, which matches the actual removal of BILLING_ENABLED from FEATURE_FLAGS.
Description check	✅ Passed	The PR description covers the required sections with clear information about changes, testing, and verification, though it deviates from the template structure by using custom sections instead.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch grasshopper

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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

[github-actions]

🧹 Preview Cleanup Complete

The following preview resources have been cleaned up:

• ✅ Neon database branch
• ✅ Electric Fly.io app
• ✅ Streams Fly.io app

Thank you for your contribution! 🎉