Skip to content

Integration: kind families, thread-name owners, disclosure mixins, Pi event phases, phase evidence, commander package - #2695

Merged
trissim merged 25 commits into
mainfrom
integration/next
Oct 23, 2026
Merged

trissim merged 25 commits into
mainfrom
integration/next

Conversation

@trissim

@trissim trissim commented Oct 23, 2026

Copy link
Copy Markdown
Contributor

One integration batch, merged on Tristan's go: #2682 (annotation disclosure mixins), #2667 (kind families; the commander deleted the len==1 scalar-DDL branch in WireOrigin.matches_sql, TIME-1), #2677 (thread-name owners), #2637 (fleet-awareness plan), #2689 (Pi event phase attributes), #2687 (phase evidence self-match), #2690 (commander package).

Each head was read by the commander. The single test run over the touched files is still finishing as the after-check; any red gets a named owner.

🤖 Generated with Claude Code

trissim and others added 25 commits October 23, 2026 01:54
On 2026-10-23 the bus was down and the commander kept the fleet map
by hand-run scripts. Each finding is a fact Core can derive from git,
order records or turn records. Phase 10 names the owner for each:
one holder per file at order time, drift and overlap standing rules,
a spinning UnitState member, unpushed and empty-PR standing rules,
review triggered by delivery, and lost tools as a bus fact.

The PlanPhase declaration is left to the phase's first order. Five
open PRs already edit plan_phases.py.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Public instruction disclosure retains credential filtering rather than withholding all text.
NativeMessages keeps its span-to-original-message lookup override.
Wire decoders retain distinct tag, retry, diagnostic and response-shape rules.
The eight existing NoExactCopies groups already passed before this change.
Keeps #2667's declared execution origins with #2677's owner-thread-free
signatures, drops the dead imports both left, and restores two undefined
test names (SavedView in test_s4_ownership, Path in test_claim_bus_read).

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
WireOrigin.matches_sql kept a second SQL spelling of one membership fact
(`origin = 'wire'` beside `origin IN (...)`) only so retained DDL stayed
byte-identical (TIME-1). It now always derives the IN list from the family;
the thread-name-owners carry rebuilds ExecutionRecord's generated columns,
and reconcile rebuilds the triggers and views.

The disclosure policies move into annotation_disclosure.py beside the
capabilities that return them, so the capabilities import them at module
level and declare their return type instead of importing inside each call to
dodge the working_memory_disclosure -> turn_context cycle.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Copilot AI balanced review requested due to automatic review settings October 23, 2026 08:42
@trissim
trissim merged commit a6a44e7 into main Oct 23, 2026
2 checks passed

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

@github-actions

Copy link
Copy Markdown

debt-tools at 12088cb, every finding:

grew LintFindings:tools/cutover/thread_name_owners.py +11
grew NoneAbsence:tools/cutover/thread_name_owners.py +1
grew Unformatted:tools/cutover/thread_name_owners.py +1
introduced NoneAbsence:tools/cutover/thread_name_owners.py at lines [51]

@github-actions

Copy link
Copy Markdown

debt-src at 12088cb, every finding:

grew FamilyFlattened:src/agent_comms/publisher.py +1
grew FamilyFlattened:src/agent_comms/response_conversation.py +1
grew FunctionLocalImport:src/agent_comms/coordination_tables/executions.py +1
grew FunctionLocalImport:src/agent_comms/phase_evidence.py +1
grew LintFindings:src/agent_comms/coordination_snapshot.py +1
grew LintFindings:src/agent_comms/coordination_tables/executions.py +3
grew LintFindings:src/agent_comms/coordination_tables/responses.py +2
grew LintFindings:src/agent_comms/external_transcript.py +1
grew LintFindings:src/agent_comms/messages.py +1
grew LintFindings:src/agent_comms/phase_evidence.py +1
grew LintFindings:src/agent_comms/pi_commands.py +1
grew LintFindings:src/agent_comms/private_sidecar.py +2
grew LintFindings:src/agent_comms/publisher.py +1
grew LintFindings:src/agent_comms/response_conversation.py +3
grew LintFindings:src/agent_comms/selected_claim.py +1
grew LintFindings:src/agent_comms/turn_context.py +1
grew LintFindings:src/agent_comms/working_memory_worker.py +1
grew NoneAbsence:src/agent_comms/pi_commands.py +1
grew NoneAbsence:src/agent_comms/pi_events.py +2
grew TypeErrors:src/agent_comms/coordination_response.py +1
grew TypeErrors:src/agent_comms/coordination_snapshot.py +1
grew TypeErrors:src/agent_comms/wake.py +2
grew Unformatted:src/agent_comms/annotation_disclosure.py +1
grew Unformatted:src/agent_comms/phase_evidence.py +1
introduced NoneAbsence:src/agent_comms/pi_commands.py at lines [188]
introduced NoneAbsence:src/agent_comms/pi_events.py at lines [62, 66]
introduced NoneAbsence:src/agent_comms/private_sidecar.py at lines [373, 389]

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.

2 participants