Repository navigation
Conversation
Signed-off-by: aihop <4408344+hhyykk@users.noreply.github.com>
This branch has not been deployed
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.
Goal And Delivered Outcome
A managed Effect daemon that cannot bind its loopback listener exits with an unhandled Node error. The Python launcher then reports
runtime_exited_before_ready, retries permission denials, and directsstart-goalcallers toward reinstalling LoopX.The listener now publishes the existing safe startup envelope. Permission denials reach
EffectRuntimeHostPermissionError, so existing CLI/doctor/quota recovery and no-retry handling apply. Other listener failures retain the shared error classification and existing retry policy. The handler is removed once listening succeeds.start-goal --guidedchanges fromruntime_exited_before_readyand reinstall guidance toruntime_host_permission_deniedand host-approved runtime recovery.main, starting at3dd3c550afecfa0c525a34e8744a3cf06774ac94.Author Declaration
effect_runtime_server.ts::failListenerStartupeffect_runtime.py::_start_runtimeScope And Continuation
Complete within listener startup classification. No transport, persistence, authority, capability, or configuration changes. Existing error helpers are reused; a larger transport refactor is unnecessary for this repair. Successful managed Goal execution still requires host-approved loopback access; this PR does not remove that environment restriction.
Validation
a3d74b1b45b0d92d49c11cc2e050e1c5e9d51c15, unchanged.loopx checkscoped to the three changed filesstart-goal --guidedsucceeds on the connected test Goal; prior permission-denied run proved safe diagnostic/recovery projectionloopx canary premerge --from-git-diff --git-diff-base origin/main: 5 direct checks + 10 selected checks, no failures, warnings or advisory failuresThe selected real-runtime suites now qualify both failure and successful serving paths. Hosted CI and exact-head maintainer review remain separate; passing local premerge does not grant merge authority. No merge performed.
Frontend / Visual Evidence
UI impact: none. CLI error classification/recovery changes; no visual surface changed.
Type of Change / LoopX Area
Bug fix and regression test; control plane / host runtime integration.
Technical Direction
Bounded S10 reliability/diagnostics repair under the existing overall roadmap. Does not claim complete operations or long-horizon qualification.
Shared-authority RFC fixture impact
N/A: no authority provider, schema, promotion, or compatibility projection change.
Boundary Checklist