Repository navigation
Shell→dag P6: host_identity_converge drops its sudo-hostnamectl ShellCommand{script} for a typed hostname effect (already on host_effect_apply_gated — the precedent pattern); nbd_proxy_serve_program RawLine body (&/trap/$!) → first-class typed background/trap statements OR a typed long-running-proce - #6588
Closed
briansrls wants to merge 2 commits into
Closed
briansrls wants to merge 2 commits into
briansrls wants to merge 2 commits into
Conversation
briansrls
marked this pull request as ready for review
July 14, 2026 07:03
1 of 2 tasks
Contributor
|
Superseded by #6619 (session sleek-otter-89): main merged, host_effect conflicts resolved, PR marked ready for review. |
briansrls
added a commit
that referenced
this pull request
Jul 14, 2026
…6619) * WIP: Shell→dag P6: host_identity_converge drops its sudo-hostnamectl ShellCom * WIP: Shell→dag P6: host_identity_converge drops its sudo-hostnamectl ShellCom * Relocate SetHostnameCas shell scaffold to host_effect_hostname_script Move concat-built hostnamectl transport out of host_effect_realize into a rostered Scaffold module (srv3_host_effect_script precedent); swap roster entry from host_identity_converge (now typed-only) to the new realize handler. Co-authored-by: Cursor <cursoragent@cursor.com> --------- Co-authored-by: Brian Searls <briansearls1@gmail.com> Co-authored-by: Cursor <cursoragent@cursor.com>
Contributor
|
Superseded by #6619 (merged) — the typed SetHostnameCas hostname effect landed there, adopting this PR's content after its author session archived. Closing to avoid a stale CONFLICTING duplicate. — sent from calm-ferret-849 |
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
1 participant
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.
Auto-opened by session-dashboard for session
sunny-fox-155.Pushing to
session/sunny-fox-155advances this PR.Worker attestation
Before flipping this PR to ready for review, confirm each item:
npm test,cargo test) and the result.Closes #Ndirective.Summary
TODO: replace this paragraph with one or two sentences naming the change and its motivation. Reviewers read this first.
Test plan