Skip to content

fix: ignore control system metadata changefeed updates - #328

Merged
stakach merged 3 commits into
masterfrom
ai/146-control-system-metadata
Sep 16, 2026
Merged

stakach merged 3 commits into
masterfrom
ai/146-control-system-metadata

Conversation

@stakach

@stakach stakach commented Sep 16, 2026 •

Copy link
Copy Markdown
Member

ControlSystem updates confined to name, description, display_name, version or signage telemetry now persist without refreshing running drivers. Ignore automatic updated_at and generated search_vector changes as well, so ordinary metadata saves remain silent. Modules, code and other nonignored changes still notify, including mixed updates; inserts/deletes and other models retain their notifications.

Require released pg-orm 2.4.1 for explicit database-only columns and remove the temporary override. No core-side filter or schema migration is needed. If the earlier two-column policy is already installed, use the documented expected-current-policy replacement during coordinated subscriber upgrade.

Review and verification

  • Regression red reproduced all four metadata save notifications; intermediate tests exposed the derived search_vector dependency.
  • Focused green: 6 specs, zero failures/errors, including persisted metadata and PG notification barriers.
  • Full stable/unstable CI passed against the identical pg-orm implementation before release.
  • Independent implementation and release-adoption reviews approved. Formatting and Ameba pass (180 files).
  • Local dependency resolution confirms pg-orm 2.4.1. Final stable/unstable CI and all style/PR checks passed against the release at 95d236e: https://github.com/PlaceOS/models/actions/runs/35046243883.

Completes the follow-up scope in PlaceOS/local#146; prerequisite spider-gazelle/pg-orm#22 is merged and released.

@github-actions github-actions Bot added type: bug something isn't working and removed type: bug something isn't working labels Sep 16, 2026
@stakach
stakach marked this pull request as ready for review September 16, 2026 01:59
@github-actions github-actions Bot added type: bug something isn't working and removed type: bug something isn't working labels Sep 16, 2026
@stakach
stakach merged commit 6207e4e into master Sep 16, 2026
8 checks passed
@stakach
stakach deleted the ai/146-control-system-metadata branch September 16, 2026 02:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type: bug something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant