feat(seer): Allow bulk-editing Code Review triggers

[ryan953]

[sentry]

Bug: A missing optional chaining operator (?.) on codeReviewTriggers will cause a TypeError when bulk-editing a repository that has null settings.
_{Severity: CRITICAL}

Suggested Fix

Add an optional chaining operator before the .includes call on line 168. The line should be changed to } else if (!repo.settings?.codeReviewTriggers?.includes(added)) { to prevent a TypeError when repo.settings is null.

Prompt for AI Agent

Review the code at the location below. A potential bug has been identified by an AI
agent. Verify if this is a real issue. If it is, propose a fix; if not, explain why it's
not valid.

Location:
static/gsApp/views/seerAutomation/components/repoTable/seerRepoTableHeader.tsx#L168

Potential issue: In the `handleBulkTriggers` function, the code attempts to access
`repo.settings?.codeReviewTriggers.includes(added)`. However, it is missing an optional
chaining operator (`?.`) before the `.includes` call. According to the type definitions,
`repo.settings` can be `null`. If a user performs a bulk edit on a repository where
`settings` is `null`, `repo.settings?.codeReviewTriggers` will evaluate to `undefined`,
and the subsequent attempt to call `.includes` on `undefined` will cause a `TypeError`,
crashing the component.

_{Did we get this right? 👍 / 👎 to inform future reviews.}

[sentry]

Bug: The currentTriggersValue calculation using .every() on an empty selectedRepos array results in a "vacuous truth", incorrectly populating the bulk edit trigger state during transient renders.
_{Severity: LOW}

Suggested Fix

Add a guard at the beginning of the useMemo hook for currentTriggersValue. If selectedRepos.length === 0, return an empty array [] to prevent the .every() logic from executing on an empty array and causing a vacuous truth.

Prompt for AI Agent

Review the code at the location below. A potential bug has been identified by an AI
agent. Verify if this is a real issue. If it is, propose a fix; if not, explain why it's
not valid.

Location:
static/gsApp/views/seerAutomation/components/repoTable/seerRepoTableHeader.tsx#L110-L116

Potential issue: The `currentTriggersValue` is calculated using
`Array.prototype.every()` on the `selectedRepos` array. When `selectedRepos` is empty,
`.every()` returns `true` due to vacuous truth. This can occur in a transient state
during a query change where selections exist (`isAnySelected` is true) but the filtered
repository list (`selectedRepos`) is temporarily empty. This results in the UI
incorrectly showing all bulk edit triggers as selected. Interacting with this incorrect
state can cause incorrect bulk update requests to be sent to the server.

[cursor]

Cursor Bugbot has reviewed your changes and found 2 potential issues.

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

^{Reviewed by Cursor Bugbot for commit 85e0a18. Configure here.}

[cursor]

Variables named some... actually use every() semantics

Low Severity

The variables someOnReadyForReview and someOnNewCommit are named with the prefix some but actually use .every(), which checks if ALL selected repos have the trigger. The naming directly contradicts the semantics — .some() returns true if any element matches, while .every() returns true only if all match. The every logic is correct for this feature (only show a trigger as selected when all repos have it), but the misleading names could cause a future developer to misunderstand the behavior.

 

^{Reviewed by Cursor Bugbot for commit 85e0a18. Configure here.}

[cursor]

Second union variant missing CodeReviewTrigger type update

Low Severity

The codeReviewTriggers field in the second union variant of RepositorySettings (line 19) still uses string[], while the third variant (line 24) was updated to CodeReviewTrigger[]. The handleBulkTriggers calls that pass only codeReviewTriggers without enabledCodeReview match the second variant, so they bypass the stricter CodeReviewTrigger type checking that the commit intended to introduce.

 

^{Reviewed by Cursor Bugbot for commit 85e0a18. Configure here.}