Skip to content

Align CivicCore dependency with the published v1.2.1 release (DevHarmonics delivery) - #77

Merged
scottconverse merged 11 commits into
mainfrom
devharmonics/60e79db9
Jul 19, 2026
Merged

scottconverse merged 11 commits into
mainfrom
devharmonics/60e79db9

Conversation

@scottconverse

Copy link
Copy Markdown
Contributor

DevHarmonics reviewed delivery

Base: main (a960bba)
Head: devharmonics/60e79db9 (91c0730)
Run: 60e79db9-fd02-44d9-b840-eef24b85d3db

This pull request is intentionally draft. DevHarmonics does not merge automatically.

@scottconverse scottconverse changed the title Outcome: Complete the CivicCore v1.2.1 alignment in the release pipeline surfaces that the first bounded change did not Align CivicCore dependency with the published v1.2.1 release (DevHarmonics delivery) Jul 16, 2026
@scottconverse

Copy link
Copy Markdown
Contributor Author

Execution evidence for this change

This delivery was produced from source analysis, and the run that produced it
recorded plainly that "no fresh tests/scripts were executed, so validation claims
are source-defined or historical receipts only."
It has now been executed.

The pin cannot move without this PR's test change, and that is provable.

civiccore 1.2.1 adds civiccore_0003_local_task_queue, which becomes the new
Alembic head. tests/test_milestone_2_schema_and_migrations.py:458 asserted the
old head, so on main the upgrade fails:

AssertionError: assert 'civiccore_0003_local_task_queue' == 'civiccore_0002_llm'
FAILED tests/test_milestone_2_schema_and_migrations.py::test_alembic_command_upgrades_real_pgvector_database

Measured in a clean venv (pip install -e ".[dev]"), with Docker available so the
real pgvector migration smoke actually runs:

Branch civiccore Result
main 1.2.0 (pinned) 225 passed, 1 skipped
main 1.2.1 1 failed — migration head assertion
this branch 1.2.1 225 passed, 1 skipped

The single remaining skip is unrelated (gemma3:12b not available from local Ollama).

The checksum is independently verified. The 8dde2940… sha256 in this PR
matches both the published SHA256SUMS.txt on the v1.2.1 release and a fresh
sha256sum of the downloaded wheel.

One thing worth knowing beyond this PR

Without Docker, both pins look identically green — 223 passed either way. The
test that catches cross-repository migration drift self-skips with "Docker CLI is
not available for the real pgvector migration smoke."
That means a routine local
run cannot see this class of incompatibility at all, and neither can any CI job
without a Docker service. That is a gap worth closing separately from this PR.

🤖 Generated with Claude Code

@scottconverse
scottconverse marked this pull request as ready for review July 19, 2026 05:55
@scottconverse
scottconverse merged commit 05994fe into main Jul 19, 2026
1 check passed
@scottconverse
scottconverse deleted the devharmonics/60e79db9 branch July 19, 2026 05:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant