Skip to content

Synchronize Stage 12 observability column - #698

Merged
anth-volk merged 1 commit into
mainfrom
fix/stage12-observability-schema-sync
Sep 27, 2026
Merged

anth-volk merged 1 commit into
mainfrom
fix/stage12-observability-schema-sync

Conversation

@anth-volk

Copy link
Copy Markdown
Contributor

Fixes #697

Summary

  • add the nullable, 36-character observability identifier to the typed Stage 12 report record
  • synchronize the DML-only SQLAlchemy mapping with the API-owned table
  • exercise the nullable column during infrastructure validation
  • add focused contract, mapping, and persistence coverage

This PR contains no Alembic migration, DDL, or schema-changing runtime call. PolicyEngine/policyengine-api remains the sole owner of the database schema.

Verification

  • contract tests: 104 passed
  • persistence tests: 12 passed
  • executor tests: 594 passed, 24 skipped, 2 deselected
  • entrypoint tests: 147 passed
  • Ruff formatting and lint checks passed for every changed file

A direct per-package Pyright invocation could not resolve dependencies from the existing local package environments. The repository CI static-analysis job remains the canonical check.

@anth-volk
anth-volk marked this pull request as ready for review September 27, 2026 14:59
@anth-volk
anth-volk merged commit 0d7bbd9 into main Sep 27, 2026
11 checks passed
@anth-volk
anth-volk deleted the fix/stage12-observability-schema-sync branch September 27, 2026 14:59
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.

Synchronize Stage 12 report mapping with observability column

1 participant