Skip to content

P241: integrate main with shared native load and growth owners - #1965

Merged
trissim merged 5 commits into
mainfrom
feat/cn-load
Oct 20, 2026
Merged

trissim merged 5 commits into
mainfrom
feat/cn-load

Conversation

@trissim

@trissim trissim commented Oct 19, 2026 •

Copy link
Copy Markdown
Contributor

P241: shared native/CLI load and growth

Rebased onto requested main 15ba6edce. Preserves CommanderContext, the current argument parser and the current checked-order lifecycle; no deleted module-level commander methods restored.

Owners and deletions

  • Existing capacity.Load serves the native load tool and the real CLI; total is projected from its counts.
  • Existing OrderRead.for_commander selects retained orders and validates holder incarnation. Full edit readers, count readers, load and the unjudged standing rule use that owner. Deleted competing CommsUnjudgedTool.readers/count.
  • Deleted heartbeat reader/writer, CLI writes and directory creation. Registry parentage/status plus retained commander orders determine receiving forks.
  • Existing ToolCaller capabilities select an explicit actor or the configured commander. CLI arguments use existing concrete caller states and FieldCodec normalization.
  • Existing MergeGrowth consumes one captured first-parent numstat stream; no implicit fetch or per-commit subprocess loop. Existing Repo owns checked bounded Git execution.

Checks

On production/test checkpoint d00ab51d7:

  • Background timeout 300 ... -X faulthandler -m pytest -q -o addopts='' tests/test_commander_capacity.py tests/test_commander_verbs.py tests/test_standing_rules.py: 49 passed.
  • Separate debt_ratchet.py --root src/agent_comms --base 15ba6edce --head d00ab51d7: rc 1 solely SourceLines +8. No introduced findings or other positive delta.
  • Separate --root tools: rc 0.
  • Checker/baselines unchanged. Shared installed interpreter; no environment creation or installation.
  • Existing audit Package/Repository AST acquisition over Core/tools/tests: zero parse omissions, before and after. Evidence: p241-2200-ast-before.txt, p241-ready-ast-after.txt in /home/ts/.cache/agent-scratch/pi-cn-phase2/. Syntactic receiver evidence does not prove dynamic resolution or runtime freshness.

Actual application reads

Configured saved-state native handler: 32 open, 30 overdue, 1860 unjudged, total 278.0. Real agent-comms-commander capacity immediately afterwards: 32 open, 30 overdue, 1861 unjudged, displayed total 278. The count increased by one between reads; not a frozen parity comparison. The focused retained-fixture test verifies exact native/CLI equality and no heartbeat store.

Receipts use p241-ready-{tests,src,tools,native,cli} in the same scratch directory. This proves the real CLI and native handler against configured saved state; it is not installed native transport/provider-process acceptance. Merger owns installation.

Failures disclosed

The 20:52 checkpoint had a CLI NameError; its retained tests were not green. Its ratchet commands omitted required --head and exited 2. During this rebase the fork field also needed correction to supported concrete caller representations. Those intermediate failures are retained, not replayed or hidden. The final 22:10 readiness deadline was missed.

Assumed: this latest delivery order is shared load/growth; broader timing/alert parity remains P244's separate unfinished work. SourceLines growth follows the standing nonblocking ruling. No overall plan completion or installation is claimed.

Copilot AI balanced review requested due to automatic review settings October 19, 2026 19:15

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.

@trissim

trissim commented Oct 19, 2026

Copy link
Copy Markdown
Contributor Author

P241 takeover checkpoint: source fix 1c53462; docs receipt follows in this PR. Main b793c73 is already an ancestor. No rebase/force-push or former-owner checkout edit.

Both final ratchets ran sequentially in the background under timeout 300 / Python -X faulthandler, using the unchanged main checker and existing development interpreter: src 1; tools 0. Only head>base key: SourceLines 158 -> 376 (+218). Lint/type/structural regressions and introduced-site counts are zero. No budget/baseline/suppression/cast was added.

34 focused tests pass. Real configured saved-state CLI reads pass for native load (38/35/1071, zero forks), growth (136 rows), and this thread's current saved turn/time. This is checkout-source live read verification, not installation or complete parity.

Precise blockers: no allocation for 218 additive production lines; the recorded stack interpreter cannot import refactor_audit; development Python has no pyright (ratchet uses mypy). Existing timing cannot classify model/build/check/test/wait/idle, and prior CheckRoot decision-publication acceptance remains unfinished. Commander must resolve allocation/scope and recorded-check interpreter; no self-authorized allowance.

Assumed: current P253 reporting policy wins; unknown categories remain unknown; existing published history and original saved sessions remain preserved.

Agent-Comms-Order: ff8a8c397475

@trissim trissim changed the title P241: native commander load, growth, decisions and unit time P241: one growth reader, no additive measurement stack Oct 19, 2026
@trissim trissim changed the title P241: one growth reader, no additive measurement stack P241: integrate main with shared native load and growth owners Oct 19, 2026
@trissim
trissim marked this pull request as ready for review October 20, 2026 02:13
trissim added a commit that referenced this pull request Oct 20, 2026
@trissim
trissim merged commit 63a67a5 into main Oct 20, 2026
0 of 3 checks passed
@trissim
trissim deleted the feat/cn-load branch October 20, 2026 02:32
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