Skip to content

fix(db): recognize deployed migration head - #1608

Closed
Komzpa wants to merge 1 commit into
mainfrom
fix/db-recognize-deployed-migration-head
Closed

fix(db): recognize deployed migration head#1608
Komzpa wants to merge 1 commit into
mainfrom
fix/db-recognize-deployed-migration-head

Conversation

@Komzpa

@Komzpa Komzpa commented Aug 5, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • add the Alembic revisions needed for a current main build to recognize the deployed database migration head
  • keep migration policy checks tolerant of older SQLite files that still carry legacy bridge columns or foreign-key wiring
  • unblock helper-built live stacks that combine this PR with fix(compact): preserve tool search pairs #1606 from origin/main

Validation

  • uv run --frozen python -c "from app.db.migrate import check_migration_policy; check_migration_policy(); print('policy_ok')"
  • uv run --frozen python -m app.db.migrate --db-url sqlite:///migration-smoke.db upgrade
  • uv run --frozen python -m app.db.migrate --db-url sqlite:///migration-smoke.db current
  • uv run --frozen python -m app.db.migrate --db-url sqlite:///migration-smoke.db check
  • git diff --check

@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@github-actions github-actions Bot added the db migration PR changes Alembic database migrations; maintainer must coordinate merge order label Aug 5, 2026
@Komzpa

Komzpa commented Aug 5, 2026

Copy link
Copy Markdown
Collaborator Author

Closing: this was opened for a local deployment-state repair and should not be an upstream PR.

@Komzpa Komzpa closed this Aug 5, 2026
@Komzpa
Komzpa deleted the fix/db-recognize-deployed-migration-head branch August 5, 2026 12:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

db migration PR changes Alembic database migrations; maintainer must coordinate merge order

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant