Repository navigation
fix(codespaces): preserve execution across environment tools - #13
Merged
Merged
Conversation
Owner
Author
|
@greptileai review this PR against main. Please check Codespace execution targeting, remote process ownership, and web/mobile management controls. |
Owner
Author
|
@greptileai All three findings are fixed in fb6cc48, with regression coverage. Please review the updated head. Validation passed: 16 regression and adjacent tests, four affected-package typechecks, three selected React Compiler checks, format, lint, both unused-code checks, and generated GraphQL. Final Codex and personal CodeRabbit reviews found no remaining substantive issues. |
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.
Codespace threads could reuse host terminals, lose explicit terminal variables, and start fresh scheduled or webhook runs on the host. Host tools also confused MCP credential IDs with provider session IDs. Remote servers and terminal activity lacked target-aware status.
This change fixes the confirmed execution defects and adds the missing management controls:
Validation:
This remains a draft pending live client acceptance. Before/after management screenshots were captured through browser-test fixtures, not a live client. Native mobile and live Codespace acceptance remain unverified. The screenshots are retained locally; GitHub image attachment is still pending.
Greptile CLI review was attempted with the personal profile. The organization plan excludes per-use CLI reviews. The installed Greptile GitHub app reviewed the updated head fb6cc48 successfully: 54 files reviewed, zero new comments. Its three earlier findings are fixed with regression coverage and their review threads are resolved.
The documented provider, native-subagent, Git-import, and host-online limits remain unchanged.
Implemented with GPT-6.1 Sol in the Codex harness, with Codex subagents.


Confidence Score: 5/5
The reviewed changes appear safe to merge; no new blocking issue was found.Summary
This PR keeps terminals, scheduled tasks, and server controls on the selected computer.
Diagram
%%{init: {'theme': 'neutral'}}%% flowchart TD A[Discover servers] --> B{Thread state} B -->|Unsaved host draft| C[Scan host without thread ID] B -->|Unsaved Codespace draft| D[Do not scan host] B -->|Saved or unknown thread| E[Resolve target on server] E -->|Host| C E -->|Codespace| F[Scan remote checkout] E -->|Unavailable| G[Return disconnected result] F --> H[Exclude nested Git checkouts] C --> I[Stop uses discovery target] H --> IReviews (3) · Last reviewed commit: "fix(codespaces): scope server controls a..." · Reviewed by Greptile