Skip to content

ref(api): use token operationIds + summary for preprod/monitors/sentry_apps/notifications/seer/feedback endpoints#117403

Merged
betegon merged 5 commits into
masterfrom
api/operation-id-tokens-misc
Jun 18, 2026
Merged

ref(api): use token operationIds + summary for preprod/monitors/sentry_apps/notifications/seer/feedback endpoints#117403
betegon merged 5 commits into
masterfrom
api/operation-id-tokens-misc

Conversation

@betegon

@betegon betegon commented Jun 11, 2026

Copy link
Copy Markdown
Member

Summary

operationId→token migration, batch 3 of 3 (preprod, monitors, sentry_apps, notifications, seer, feedback — 35 ops / 22 files). Sentence → summary, operation_id → short camelCase REST token; docs titles/slugs unchanged (derived from summary).

Overrides:

  • startOrganizationIssueAutofix / getOrganizationIssueAutofixState (seer)
  • unified all preprod ops to PreprodArtifact casing (structural produced inconsistent Preprodartifacts), and fixed the latest-base singleton to getOrganizationPreprodArtifactSnapshotLatestBase (was a mislabeled list)

monitors map to the Detector/Monitor paths as-is; notifications-action ops are self-consistent.

Test plan

  • prek green; CI api docs test regenerates + validates the spec (--fail-on-warn).

Related

⚠️ Don't merge the batches simultaneously — the schema publish job has no concurrency guard (caused a partial-release race before). Space them out (merge one, wait for its openapi publish, then the next).

@github-actions github-actions Bot added the Scope: Backend Automatically applied to PRs that change backend components label Jun 11, 2026
…y_apps/notifications/seer/feedback

operationId->token migration, batch 3 of 3 (35 ops). Sentence -> summary=,
operation_id -> camelCase REST token.

Overrides: startOrganizationIssueAutofix / getOrganizationIssueAutofixState,
submitProjectUserFeedback->listProjectUserFeedback(structural); unified all
preprod ops to PreprodArtifact casing and fixed the latest-base singleton to
getOrganizationPreprodArtifactSnapshotLatestBase.
@betegon betegon force-pushed the api/operation-id-tokens-misc branch from 93ec00f to df07690 Compare June 17, 2026 11:58
@betegon betegon marked this pull request as ready for review June 17, 2026 17:51
@betegon betegon requested review from a team as code owners June 17, 2026 17:51

@cursor cursor Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit d92f7bb. Configure here.

The GET returns a single project config object (enabled + nested rule
flags), not a paginated collection, so list -> get to match the verb
convention and the parallel getProjectPreprodSizeAnalysisStatusCheckRules
endpoint. Summary (docs title/slug) unchanged.
@betegon betegon merged commit 7c789a1 into master Jun 18, 2026
64 of 65 checks passed
@betegon betegon deleted the api/operation-id-tokens-misc branch June 18, 2026 08:49
sehr-m pushed a commit that referenced this pull request Jun 23, 2026
…y_apps/notifications/seer/feedback endpoints (#117403)

## Summary

operationId→token migration, **batch 3 of 3** (preprod, monitors,
sentry_apps, notifications, seer, feedback — **35 ops / 22 files**).
Sentence → `summary`, `operation_id` → short camelCase REST token; docs
titles/slugs unchanged (derived from `summary`).

Overrides:
- `startOrganizationIssueAutofix` / `getOrganizationIssueAutofixState`
(seer)
- unified all preprod ops to `PreprodArtifact` casing (structural
produced inconsistent `Preprodartifacts`), and fixed the latest-base
singleton to `getOrganizationPreprodArtifactSnapshotLatestBase` (was a
mislabeled `list`)

monitors map to the `Detector`/`Monitor` paths as-is;
notifications-action ops are self-consistent.

## Test plan
- prek green; CI `api docs test` regenerates + validates the spec
(`--fail-on-warn`).

## Related
- Batch 1 (core+api): #117337 · Batch 2
(issues/integrations/releases/workflow): #117339
- Convention: #117285 · Merged: #117166 #117297 #117302

⚠️ Don't merge the batches simultaneously — the schema publish job has
no concurrency guard (caused a partial-release race before). Space them
out (merge one, wait for its `openapi` publish, then the next).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Backend Automatically applied to PRs that change backend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants