Repository navigation
Conversation
…tion-capabilities
…tion-capabilities
…tion-capabilities
…tion-capabilities
trissim
marked this pull request as ready for review
October 21, 2026 17:05
trissim
enabled auto-merge
October 21, 2026 17:06
There was a problem hiding this comment.
Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.
An organization owner can choose the repository organization as the billing source for reviews requested by members with a paid Copilot license. Learn about Copilot code review billing.
auto-merge was automatically disabled
October 21, 2026 17:50
Pull request was closed
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.
P429: mutation capabilities (draft; native qualification pending)
Replaced owners
src).RowMutationrenders guards;TypedTablecollects capability implementations by MRO.ImmutableRowscomposesNoUpdate/NoDelete;AppendOnlyTablesinherits that behavior.GuardedUpdateMembership/ColumnGuard.ExecutionState.accepts_previousand SQLite ordinal guards consume the sameordinal_rulevalue. Deleted the unused_sql_edgesrenderer and four context entries fromcoordination_schema.py.OwnerRecordowns the repeated owner projection.ParticipantSnapshotretains its existingOwnerGenerationsrow; copied owner thread/generation fields are gone.ExecutionReferenceowns the three restrictive execution references.TableSchema.require_installedreplaces the three module-level installed-schema checks; families declare their error type.carry_table_triggersis the sole stored-trigger writer.carry_declared_schemanow handles tables/indexes/views, never triggers. OnlySnapshotMeta.ddl_digestmay change, transactionally after validation against the old stored schema; no application row changes. Unknown sidecar schemas are refused with their filename. Retire the cutover after installation.Capability → tables
RetryPartitionshares the after-write partition check on obligations/replay assessments. Ordered column declarations own nondecreasing/nonincreasing/nullable-order SQL.C: retained guard disposition
claim_batch_membersandcohort_delivery_receipts: membership in an unsealed batch; receipt/claim cardinality, ordinal order, selected recipients and sealed observer exclusion cross tables. Initial sealed value and one-way UPDATE now derive fromInitialOneWayColumn, not conditions.awareness_claim_generations: its exact batch member, assignment and owner generation must agree.current_executions: an owner-generation row must already exist.attempts: active execution, current owner generation and initial active attempt must agree. The existing deferred FK is not the same immediate lease-acquisition rule.execution_claims: assignment/claim belongs to the execution and its startable lifecycle.executions: optional attempt references must name a matching attempt; lifecycle/ordinal/revision rules are separate capabilities.obligations: wire acceptance, publication intent and publication receipt membership (including the approved UPDATE membership disposition). Lifecycle changes/edges, clocks and revision are no longer raw SQL rules.publication_intents: pending obligation membership;ValidatedInsertsowns invoking the existing declared validator.publication_receipts: execution/obligation/intent membership and failed-execution exclusion.native_context_journal/native_runtime_triage_sources: original native-input/context identity membership and permitted initial proof values.ImmutableRows(no copied trigger templates/messages). Existing CHECK/FK constraints remain unchanged; no DDL change was used to replace a trigger.Source and SQL evidence
Refactor-audit parser census: before4,355 classes; after4,577; zero parse omissions. The after inventory includes the now-initialized audit skill gitlink, so this is not a production-class growth measurement. Explicit runtime declaration discovery checks membership; AST does not prove dynamic bindings. NRA's separate tree-sitter frontend was unavailable in the shared interpreter.
All75 table/index definitions remain byte-identical; 110 derived triggers. Command:
Diff output: empty (0 bytes).
Verification actually run
test_table_mutation_capabilities.py/test_table_trigger_carry.py, plus1 one-shot main-trigger carry proof). Usedtimeout300,-n0,-o faulthandler_timeout=60, owned persistent basetemp and--noconftest: these tests do not launch native and this is not native qualification.Authorized reader edits
Schema-check consumers (15 production modules +1 test): cursor_owner.py, transcripts.py, optional_awareness_projection.py, selected_claim.py, attempt_recovery.py, native_source_cursor.py, cohort_foreground.py, proven_source_coverage.py, private_send_stage.py, historical_native_inputs.py, private_send_claim.py, native_prompt_binding.py, selected_participant.py, selected_tool_broker.py, coordination_cohort.py; tests/test_coordinated_runtime.py. Three old function-owning modules migrated in place.
Identity readers: attempt_start.py, attempt_store.py, input_drain.py, selected_participant.py, native_source_cursor.py, thread_management.py; associated tests/awareness construction; additionally execution_store.py as granted. Net lines per production file below.
Current source diff including EOF cleanup:40 files,+910/-1264, net−354. The meaningful implementation is pushed at89693f6e9 (remote matched); eight EOF/whitespace cleanups remain to join the next semantic/native-pin commit.
Remaining before ready
stack/pi-native.sha256using the build owners. Record before/after hashes; never hand-merge them.EngagedExecution.assignment_stateand ExecutionState-owned next-ordinal reader on integration; fold the reader’s remaining literal advancement value onto the ordinal-rule owner.Assumed: