Repository navigation
Conversation
5 of 12 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
CI repair — 2026-10-21
At original head
03a9ea40536aa34ad4a0e6ad64a9f91ab014b333, pinned MDM compatibility #39650748844 failed while running the newly added three-case SQL fixture, before any worker assertions. The precise PostgreSQL error wasoperator is not unique: unknown - unknownattests/integrations/pg-mdm/time/routing.sql:240, caused by SQL operator precedence inoracle->'explanation' - 'escalation_level'. Commitb9285a5fixes the fixture by parenthesizing JSONB extraction, matching the existing passing one-case oracle:((oracle -> 'explanation') - 'escalation_level').No qualification requirement, case vector, timestamp boundary, SQL implementation, production path, permission, dependency pin, or assertion was relaxed. Current-head CI is pending: pinned compatibility #39653229584, MDM preview #39653229550, core #39653229680. Keep this PR draft until the new head is green.
Scope: deterministic three-case time frontier
Adds an independent full-row oracle to
tests/integrations/pg-mdm/time/routing.sql. Reuses the existing v0.49 explicit-timeroute_cases(..., captured_at)implementation and v0.47 fixture table. It preserves two original case identities, then tests three simultaneous cases with due times one second before, exactly equal to, and one second after a single sampled UTC instant. A single sample yields two eligible next-level rows and one ineligible row. Samples before equality and after the last deadline must return the exact independently constructed three-row vectors.Assertions
case_key, originalreview_id, configured level and reason; no cross-case filtering or duplicated work candidates.WINNERversusINELIGIBLE,action,priorityand immutablecompetitors.timestamptz, with equality due and no hidden wall-clock selection.Boundaries
This is deterministic multi-case evaluation, not runtime batch admissions, a real multi-case outage/catch-up, backlog fairness, latency/resource qualification or T49.06/T49.12 closure. No production SQL, policy semantics, grants, dependencies, worker limits or release artifacts change. Remains dependent on React #14 and issue #11. Keep draft until current-head CI passes.