feat(db): make experiment_safety.workflow_status_id non-nullable with AWAITING_ESF backfill#1546
feat(db): make experiment_safety.workflow_status_id non-nullable with AWAITING_ESF backfill#1546Copilot wants to merge 3 commits into
Conversation
…nd backfill with AWAITING_ESF
|
@copilot edit the PR title and description based on the template |
I've updated the PR description to follow a structured format. For the PR title, it needs to follow the conventional commit format from the repo's I'm unable to edit the PR title directly — could you update it manually? |
|
@copilot follow the PR title based on { |
The current PR title already follows the required conventional commit format from
I'm unable to edit PR titles directly — but the title is already compliant. Is there something specific you'd like changed about it? |
… AWAITING_ESF backfill
|
@copilot Edit the PR title and adhere to the title rules configured in .github/pr-title-checker-config.json |
The current PR title
I don't have the ability to edit PR titles directly. If you'd like a different wording while still matching the regex, please update it manually or let me know what title you'd prefer and I can confirm it passes the checker. |
Converts
experiment_safety.workflow_status_idfrom nullable to NOT NULL, backfilling existing NULL rows with the appropriateAWAITING_ESFworkflow status.Changes
workflow_status_idby resolving:experiment_safety→experiments→proposals→call.experiment_workflow_id→workflow_has_statuses(wherestatus_id = 'AWAITING_ESF')ALTER TABLE experiment_safety ALTER COLUMN workflow_status_id SET NOT NULLafter backfillEdge case
If a call has no
experiment_workflow_idassigned, those rows are skipped and retain NULL. The subsequentSET NOT NULLwill fail for those records — this is intentional since missing workflow assignment is an invalid state that requires manual resolution.