Skip to content

feat(seer): Add scope methods to SeerWorkflowConfig - #127892

Draft
trevor-e wants to merge 1 commit into
telkins/feat/seer-config-scope-fieldsfrom
telkins/feat/seer-config-service
Draft

trevor-e wants to merge 1 commit into
telkins/feat/seer-config-scope-fieldsfrom
telkins/feat/seer-config-service

Conversation

@trevor-e

@trevor-e trevor-e commented Oct 16, 2026 •

Copy link
Copy Markdown
Member

Adds create_configurable, set_scope and get_project_ids to SeerWorkflowConfig, so V2 configs have one place to save and read their project and environment selection. Saving validates first (at least one project for Some, same-org active projects, valid environment names), then writes the flag, project rows and environments in one transaction. A config whose selected projects are all deleted covers no projects. It never widens to all.

Validating the JSON settings fields is left to each automation.

Refs AIML-3722

@trevor-e
trevor-e requested a review from a team as a code owner October 16, 2026 22:20
@linear-code

linear-code Bot commented Oct 16, 2026

Copy link
Copy Markdown

AIML-3722

@github-actions github-actions Bot added the Scope: Backend Automatically applied to PRs that change backend components label Oct 16, 2026
@trevor-e
trevor-e marked this pull request as draft October 16, 2026 22:23

@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.

Want reviews to match your repository better? Bugbot Learning can learn team-specific rules from PR activity. A team admin can enable Learning in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit f3db0d1. Configure here.

Comment thread src/sentry/seer/workflows/configs.py Outdated
@trevor-e
trevor-e force-pushed the telkins/feat/seer-config-scope-fields branch from e9b4d70 to 36b0ead Compare October 16, 2026 22:39
@trevor-e
trevor-e force-pushed the telkins/feat/seer-config-service branch from f3db0d1 to 84b2e4b Compare October 16, 2026 22:39
@sentry

sentry Bot commented Oct 16, 2026 •

Copy link
Copy Markdown
Contributor

Sentry Snapshot Testing

Name Added Removed Changed Renamed Unchanged Skipped Status
sentry-frontend
sentry-frontend
0 0 0 0 630 0 ✅ Unchanged

⚙️ sentry-frontend Snapshot Settings

@trevor-e
trevor-e force-pushed the telkins/feat/seer-config-service branch from 84b2e4b to 828919e Compare October 16, 2026 23:02
@trevor-e
trevor-e force-pushed the telkins/feat/seer-config-scope-fields branch from 36b0ead to fceaa5e Compare October 16, 2026 23:02
@trevor-e trevor-e changed the title feat(seer): Add automation config write and scope resolution service feat(seer): Add scope methods to SeerWorkflowConfig Oct 16, 2026
@trevor-e
trevor-e force-pushed the telkins/feat/seer-config-scope-fields branch from fceaa5e to ed5bd54 Compare October 16, 2026 23:06
@trevor-e
trevor-e force-pushed the telkins/feat/seer-config-service branch 2 times, most recently from 2e31abb to dea74b4 Compare October 16, 2026 23:08
@trevor-e
trevor-e force-pushed the telkins/feat/seer-config-scope-fields branch from ed5bd54 to c44f454 Compare October 16, 2026 23:08
@trevor-e
trevor-e force-pushed the telkins/feat/seer-config-service branch from dea74b4 to e37393f Compare October 16, 2026 23:16
@trevor-e
trevor-e force-pushed the telkins/feat/seer-config-scope-fields branch 2 times, most recently from d3cbbd3 to 2e14098 Compare October 16, 2026 23:30
@trevor-e
trevor-e force-pushed the telkins/feat/seer-config-service branch from e37393f to 6cde167 Compare October 16, 2026 23:30
@trevor-e
trevor-e force-pushed the telkins/feat/seer-config-scope-fields branch from 2e14098 to 9364996 Compare October 17, 2026 00:04
@trevor-e
trevor-e force-pushed the telkins/feat/seer-config-service branch from 6cde167 to 8105652 Compare October 17, 2026 00:04
@trevor-e
trevor-e force-pushed the telkins/feat/seer-config-scope-fields branch from 9364996 to 6c31dda Compare October 17, 2026 00:09
@trevor-e
trevor-e force-pushed the telkins/feat/seer-config-service branch 2 times, most recently from 82e9bde to b2e7e73 Compare October 17, 2026 00:12
SeerWorkflowConfig gains create_configurable, set_scope, and get_project_ids as the way to write and read a config's project and environment scope. A config whose selected projects are all gone covers no projects instead of widening to all. Validating a config's settings is left to each automation.

Refs AIML-3722
@trevor-e
trevor-e force-pushed the telkins/feat/seer-config-service branch from b2e7e73 to 03ad3be Compare October 17, 2026 00:15

This branch has not been deployed

No deployments
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.

1 participant