Skip to content

[CORRUPTED] Synthetic Benchmark PR #1906 - [bench] Add feature to subscription as REST API - #12

Open
amirzaushnizer wants to merge 8 commits into
base_pr_1906_20260621_2809from
corrupted_pr_1906_20260621_2809
Open

[CORRUPTED] Synthetic Benchmark PR #1906 - [bench] Add feature to subscription as REST API#12
amirzaushnizer wants to merge 8 commits into
base_pr_1906_20260621_2809from
corrupted_pr_1906_20260621_2809

Conversation

@amirzaushnizer

Copy link
Copy Markdown
Collaborator

Benchmark PR flowglad#1906

Type: Corrupted (contains bugs)

Original PR Title: [bench] Add feature to subscription as REST API
Original PR Description: Reconstructed for skills benchmark from flowglad#1906
Original PR URL: flowglad#1906

angihe93 and others added 8 commits February 3, 2026 18:50
…ept subscriptionId

- Step 1: Update input schema to use subscriptionId (id) instead of subscriptionItemId
- Step 2: Update helper to accept subscriptionId directly
- Step 3: Add OpenAPI metadata and output schema to procedure
- Step 4: Add route config for REST endpoint at POST /subscriptions/{id}/add-feature
- Step 5: Update AddSubscriptionFeatureModal to accept subscriptionId prop
- Step 6: Update form schema to use id instead of subscriptionItemId
- Step 7: Simplify form fields - remove subscription item selector
- Step 8: Update InnerSubscriptionPage to pass subscriptionId to modal
- Update tests to use new input format
- All 42 related tests pass

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
The OpenAPI path is /subscriptions/{id}/add-feature, so we need
routeSuffix: 'add-feature' to match (default would be add-feature-to-subscription).

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants