Repository navigation
[10/11] Honor watcher settings in MCP startup - #11
Merged
Merged
Conversation
This was referenced Oct 19, 2026
oubiwann
force-pushed
the
codex/pr-13-mcp-watch-settings
branch
from
October 19, 2026 22:32
486ea1e to
4d9dd84
Compare
oubiwann
force-pushed
the
codex/pr-12-cache-player-state-integrity
branch
from
October 19, 2026 22:32
615ca76 to
116797c
Compare
oubiwann
changed the base branch from
codex/pr-12-cache-player-state-integrity
to
main
October 19, 2026 22:33
This was referenced Oct 19, 2026
oubiwann
force-pushed
the
codex/pr-13-mcp-watch-settings
branch
6 times, most recently
from
October 20, 2026 00:29
e0c6abd to
c79fe82
Compare
oubiwann
marked this pull request as ready for review
October 20, 2026 00:32
Resolve one startup watcher configuration and pass it through both normal and headless MCP startup. Keep the legacy wrappers at the 500ms default and verify disabled watching through isolated live children. Co-authored-by: Codex <noreply@openai.com> Co-authored-by: Billo AI <ai-engineering@billo.systems>
oubiwann
force-pushed
the
codex/pr-13-mcp-watch-settings
branch
from
October 20, 2026 00:33
c79fe82 to
a429b3f
Compare
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.
Target:
main. Remaining merge order: #11 -> #12. Previous prerequisites are merged.Resolve the optional WatchConfig once from the selected save and forward it through normal and headless MCP startup. Honor disabled watching and a configured debounce interval while preserving legacy public entrypoints and their 500 ms defaults.
The branch also carries PR #4’s deterministic cache-test timestamp correction through its dependency history. Its own original implementation increment is preserved.
Original accepted-source validation: Real stdio/HTTP child sessions with enabled 137 ms settings and disabled watching, compatible legacy API behavior, recovery, and workspace gates. CDC review; closed slice ledger. These records describe the original accepted commit, not a new acceptance verdict for the PR repair.
Original source:
codex/mcp-watch-settingsat16c078725492744db22bae837d87c89489a6add5. Current dedicated PR branch:codex/pr-13-mcp-watch-settingsata429b3fdebecf81050ba354c5b88dae8071768fb. The accepted branches and active Arc06 worktree are preserved.Remaining scope / evidence limits: These are startup settings. Duplicate producers and complete live-cache ownership remain follow-on work; no configuration hot reload is claimed.
All open PRs in this series target main. Their branches retain their dependencies, so later diffs are cumulative until predecessors land. GitHub permits rebase merging only: after each merge, restack the remaining dedicated PR branches onto the updated main and require fresh successful CI before merging the next PR. Do not merge a PR into another feature branch.
Merge after #10. Next: #12. Full merge order.
No merges or releases were performed during this repair; Arc06/Project01 closure and reconciliation with ongoing work remain separate.
Rebase refresh after PR #13: rebased onto main at
f98d39ed78703d81c0a9481a24bb85571b2397bd, excluding the cache commits already landed there. The complete source tree is byte-identical to the pre-rebase head. The timestamp correction is retained, all successors carry the rebased ancestry, and the stack contains zero merge commits. Prior source validation remains applicable; new CI checks are pending.Rebase refresh after PR #4: current head
a429b3fdebecf81050ba354c5b88dae8071768fbdescends from main at35105a7f5adf2d6b48662d5be315caba37dc83cc. Already-landed identity commits were excluded from replay. This refresh preserves the complete source tree and cache-test correction; zero merge commits. Fresh CI is pending.Rebase refresh after PR #5: current head
a429b3fdebecf81050ba354c5b88dae8071768fbdescends from main atd319111a26e9ac7d304de48842f2fc41555cd82a. Already-landed prerequisite commits were excluded from replay. The complete source tree and cache-test correction are unchanged; zero merge commits. Prior validation remains applicable; fresh CI is pending.Rebase refresh after PR #6: current head
a429b3fdebecf81050ba354c5b88dae8071768fbdescends from main atb9b6841710babb26f59676b952c4005719b13148. Already-landed prerequisite commits were excluded from replay. The complete source tree and cache-test correction are unchanged; zero merge commits. Prior validation remains applicable; fresh CI is pending.Rebase refresh after PR #8: current head
a429b3fdebecf81050ba354c5b88dae8071768fbdescends from main atdaf4f5ebbd772b1e0000a4fcb8c0a87ecb7b9aad. Already-landed prerequisite commits were excluded from replay. The complete source tree and cache-test correction are unchanged; zero merge commits. Prior validation remains applicable; fresh CI is pending.Rebase refresh after PR #9: current head
a429b3fdebecf81050ba354c5b88dae8071768fbdescends from main at6e6ddca0a06412f3f8ef157ab2334bbc7868a122. Already-landed prerequisite commits were excluded from replay. The complete source tree and cache-test correction are unchanged; zero merge commits. Prior validation remains applicable; fresh CI is pending.Rebase refresh after PR #10: current head
a429b3fdebecf81050ba354c5b88dae8071768fbdescends from main at608efce757fd19eb4b4adfb45b38942d1d186436. Already-landed prerequisite commits were excluded from replay. The complete source tree and cache-test correction are unchanged; zero merge commits. Prior validation remains applicable; fresh CI is pending.