[PM-33899] Release schedule on terminal subscription operations

[amorask-bitwarden]

🎟️ Tracking

https://bitwarden.atlassian.net/browse/PM-33899

📔 Objective

Introduces IPriceIncreaseScheduler — a shared service that consolidates subscription schedule creation and release logic behind the PM32645_DeferPriceMigrationToRenewal feature flag. Previously, schedule creation lived as a private method in UpcomingInvoiceHandler and release guards were inlined at each call site.

Schedule(Subscription) — Creates a two-phase subscription schedule that defers a price increase to the renewal date. Resolves the subscriber type via SubscriberId metadata to determine the correct migration path (Premium or Families), builds the appropriate Phase 2 configuration (target price + discount), and creates the schedule in Stripe. Returns bool so callers can skip downstream processing (e.g., renewal emails) when a schedule already exists.

Release(string customerId, string subscriptionId) — Releases any active schedule for the subscription, cancelling a pending deferred price increase. Logs and re-throws on failure, requiring manual release via the Stripe Dashboard.

Call sites:

• UpcomingInvoiceHandler — calls Schedule during invoice.upcoming processing (replaces the private SchedulePriceMigrationAsync)
• SubscriptionUpdatedHandler — calls Release when a subscription goes unpaid; calls Schedule when a subscription recovers (recreates the schedule)
• UpgradePremiumToOrganizationCommand — calls Release before upgrading to an org plan
• SetUpSponsorshipCommand — calls Release before applying a sponsorship

📸 Screenshots

[github-actions]

Checkmarx One – Scan Summary & Details – 5a48ff82-724b-4a29-accc-e338e5e7bcdc

Great job! No new security vulnerabilities introduced in this pull request

[codecov]

Codecov Report

❌ Patch coverage is 83.02752% with 37 lines in your changes missing coverage. Please review.
✅ Project coverage is 57.94%. Comparing base (89f6e43) to head (9cd250d).
⚠️ Report is 11 commits behind head on main.

Files with missing lines	Patch %	Lines
src/Core/Billing/Pricing/PriceIncreaseScheduler.cs	81.77%	31 Missing and 4 partials ⚠️
...Services/Implementations/UpcomingInvoiceHandler.cs	60.00%	0 Missing and 2 partials ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #7305      +/-   ##
==========================================
+ Coverage   57.91%   57.94%   +0.03%     
==========================================
  Files        2044     2045       +1     
  Lines       90130    90243     +113     
  Branches     8018     8026       +8     
==========================================
+ Hits        52199    52295      +96     
- Misses      36066    36080      +14     
- Partials     1865     1868       +3     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[sbrown-livefront]

🧹 ❇️ Abstracts a lot from that handler. Good stuff!