Skip to content

fix(autofix): Read automation settings from detailed project#115239

Open
scttcper wants to merge 5 commits intomasterfrom
scttcper/autofix-detailed-project
Open

fix(autofix): Read automation settings from detailed project#115239
scttcper wants to merge 5 commits intomasterfrom
scttcper/autofix-detailed-project

Conversation

@scttcper
Copy link
Copy Markdown
Member

@scttcper scttcper commented May 8, 2026

Autofix coding agent CTAs were checking detailed project settings from project summary objects, so missing Seer automation fields could look enabled.

Use the detailed project for those reads and keep updates going through useUpdateProject with the original project. Matches the pattern from #115139.

part of #115024

The coding agent CTAs were checking Seer automation fields from the project prop, which can be a project summary and miss detailed settings. That made missing values look enabled in a few places.

Load the detailed project before deciding CTA state, and keep updates going through useUpdateProject with the original project. This follows the same pattern as the preprod settings fix.

Co-Authored-By: Codex GPT-5 <noreply@openai.com>
@github-actions github-actions Bot added the Scope: Frontend Automatically applied to PRs that change frontend components label May 8, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 8, 2026

📊 Type Coverage Diff

✅ No new type safety issues introduced. Coverage: 93.46%

ProjectSeer renders the coding agent CTAs, which now load detailed project data before checking automation settings. Add the detailed project mock to the settings tests, including the custom org setups that clear default mocks.

Co-Authored-By: Codex GPT-5 <noreply@openai.com>
@scttcper scttcper marked this pull request as ready for review May 8, 2026 23:04
@scttcper scttcper requested review from a team as code owners May 8, 2026 23:04
Comment thread static/app/views/issueDetails/streamline/sidebar/seerNotices.tsx Outdated
Copy link
Copy Markdown
Contributor

@cursor cursor Bot left a comment

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 9ea84e7. Configure here.

Comment thread static/app/views/issueDetails/streamline/sidebar/seerNotices.tsx Outdated
scttcper and others added 2 commits May 8, 2026 16:11
The notice should not decide the automation step from summary project data while the detailed project request is loading.

Keep the detailed project guard for onboarding state, but still use the detailed values for the handoff setup click.

Co-Authored-By: Codex GPT-5 <noreply@openai.com>
Comment thread static/app/views/issueDetails/streamline/sidebar/seerNotices.tsx
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant