Skip to content

Deepen role runtime ownership - #234

Merged
dtcristo merged 3 commits into
mainfrom
agent/232-role-runtimes
Aug 8, 2026
Merged

dtcristo merged 3 commits into
mainfrom
agent/232-role-runtimes

Conversation

@xenorune

@xenorune xenorune commented Aug 8, 2026

Copy link
Copy Markdown
Member

Summary

  • add supported ClientRuntime and ServerRuntime process seams
  • move role orchestration, mode selection, shutdown, drain, ACME, and Managed-session ownership out of CLI callers
  • join Client workers and Server readiness, handshake, Visitor-intake, ACME, and session work on every terminal path
  • preserve the initiating error while teardown completes
  • contract binary-only crate exports and document the intentional public runtime inventory

Refs #232

Stack

Validation

  • cargo test --all-features
  • cargo clippy --all-targets --all-features -- -D warnings
  • cargo fmt --check
  • host actionlint
  • workflow wrapper attempted; remote Docker SSH timed out
  • dependency audit attempted; unchanged transitive event-listener 5.4.1 is covered by RUSTSEC-2026-0221

Review

  • Standards and Spec review completed from c81c61da
  • fixed Managed Client controller-first teardown, Server child joins, ACME error teardown, session drain, and public-interface inventory

dtcristo and others added 3 commits August 8, 2026 11:52
Refs #232

Co-authored-by: Xeno <xeno@xenorune.com>
Refs #232

Co-authored-by: Xeno <xeno@xenorune.com>
Refs #232

Co-authored-by: Xeno <xeno@xenorune.com>
@xenorune
xenorune force-pushed the agent/232-role-runtimes branch from af348a9 to 4a4369b Compare August 8, 2026 01:57
Base automatically changed from agent/232-active-visitor-work to main August 8, 2026 05:03
@dtcristo
dtcristo merged commit 6e0c40f into main Aug 8, 2026
6 of 8 checks passed
@dtcristo
dtcristo deleted the agent/232-role-runtimes branch August 8, 2026 05:05
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