fix: preserve scheduler ownership and flush loop output - #83
Conversation
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
💡 Codex ReviewLines 676 to 677 in 6308bf6 When an existing owned automation is paused but uses valid noncanonical TOML—such as fclt/src/evolution-preflight.ts Lines 66 to 68 in 6308bf6 When an existing journal, writeback queue, or loop audit file is read-only or has a file-specific denying ACL while its parent remains writable, creating a temporary directory succeeds and preflight reports ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
|
🎉 This PR is included in version 2.30.6 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
Piped loop output below 64 KB could still be truncated because the output helper returned after console logging. Await stdout writes at every response size.
Native automation updates can drop fclt’s custom ownership marker. Subsequent loop management then rejects the existing schedule, and re-enabling could overwrite native edits. Store an identity-bound ownership receipt in existing machine-local state, preserve authored automation settings when re-enabling, and provide explicit repair for already-affected configured loops without activating them.
Related: HACK-1044. Builds on merged review-preflight recovery #82.
Validation:
Repair records a runtime ownership receipt only. It does not relocate state, enable a paused schedule, apply capability changes, or infer ownership from the display name alone.