hotfix(migrations): fix duplicate LLC Alembic revision IDs (20260523_022)

[mrveiss]

Problem

Three LLC migrations were merged concurrently with identical revision = "20260523_022", breaking alembic upgrade head:

• 20260523_022_create_llc_work_items.py (GH#8213, merged 08:56Z)
• 20260523_022_organization_llc_extensions.py (GH#8211, merged 09:33Z) ← DUPLICATE
• 20260523_022_llc_goals.py (GH#8212, merged 09:41Z) ← DUPLICATE

Fix

Creates a linear chain:

021 → 022 (work_items) → 023 (org_extensions) → 024 (goals)

Files renamed and revision/down_revision updated accordingly.

Impact on pending PRs

All pending LLC migration PRs must update to use revision >= 025 with down_revision = "20260523_024":

• PR feat(llc): immutable activity log — model, service, API, migration (GH#8216) #8457 (issue-8216): currently uses revision 025, down=022 → must change down to 024
• PR feat(llc): board approval gates — GH#8214 #8458 (issue-8214): currently uses revision 023, down=022 → must change both to 025/024
• PR feat(llc): per-agent budget enforcement — hard stop, soft alert, cost ingest (#8215) #8459 (issue-8215): currently uses revision 022 → must change to 025/024

🤖 Generated with Claude Code

[github-actions]

⚠️ SSOT Configuration Compliance: Violations Found

Metric	Count
Total Violations	7
SSOT Violations (high priority)	7
Other Violations	0

⚠️ 7 values have SSOT config equivalents!

These should be replaced with SSOT config imports:

Python:

from src.config.ssot_config import config
# Use: config.vm.main, config.port.backend, config.backend_url

TypeScript:

import config from '@/config/ssot-config'
// Use: config.vm.main, config.port.backend, config.backendUrl

📖 See SSOT_CONFIG_GUIDE.md for documentation.

[mrveiss]

Fix updated: corrected FK dependency ordering

The initial commit had create_llc_work_items at revision 022 and llc_goals at 024. This would still break because create_llc_work_items has sa.ForeignKey("llc_goals.id") — PostgreSQL requires the referenced table to exist at CREATE TABLE time.

Final correct chain:

021 (heartbeat_wakeup_merged_count)  ← existing anchor
022 (llc_goals)                      ← no FK deps; must run first
023 (organization_llc_extensions)    ← standalone column additions
024 (create_llc_work_items)          ← FK to llc_goals.id; runs after 022

Head is still 024 — downstream PRs (#8457, #8458, #8459) still need down_revision = "20260523_024" as specified in MVA-868.

[mrveiss]

Superseded by hotfix/llc-migration-chain-v2 — PR #8460 conflicts with Dev_new_gui because commit 6091fb4a5 (feat(llc): per-agent budget enforcement, GH#8215) landed after this PR was opened and introduced 025_llc_agent_budgets.py with down_revision = '20260523_022'. The v2 branch resolves all issues: same duplicate-ID fix (022→023/024 renaming), plus deferred FK (fk_llc_work_items_goal_id) wired correctly in 024_llc_goals.py, and the budget migration chained to 025 with down_revision = '20260523_024'. New PR opened from v2 instead.