Repository navigation
Integrate startup profiling and harden onDone CI fixture - #154
Merged
Merged
Conversation
Reduce the internal module graph without deferring RPC, tool registration, or recovery. Keep host packages external and index/API runtime modules shared. Add receipt-aware paired startup measurements and bounded property-worker CPU profiles with campaign provenance, failure retention, source identity, and explicit unknown counters.
The mismatched minimized path executed zero cases, correctly reported as incomplete despite a passing test. Use the seed/path combination observed to execute 100 cases and retain a worker profile.
perf: reduce startup module graph and profile fuzz workers
…zation' Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
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.
Changes
Integrate #152 into
master, preserving its published ancestry. #152 merged intochore/stack-next-release-deps; master85a9465does not contain those changes. Signed integration commitb0ce974has parents85a9465andfdbdf90; its tree exactly matches publishedb820312.Harden two additional positive onDone fixtures in
test/index.test.ts. Wait for the matching monitor's terminal receipt and actual message delivery instead of a fixed 500 ms sleep. Add a delayed-child case. Retain exact-one, outcome, output-safety, and 10-second deadline assertions. Production monitor/wake code is unchanged relative to #152.Evidence
Master CI 38562237909 passed Linux and failed Windows's custom onDone wake assertion: expected one message, observed zero. Terminal/delivery readiness was not reported; the original Windows cause remains unproved.
Before repair, the owned
sleep 1; echo 'monitor done'case failed at the same message assertion using the old wait strategy. Its recorded state had no terminal receipt or delivery yet. Four nearby controls passed. After repair, the same five cases pass. This is fixture-readiness hardening, not a demonstrated production defect.The integration retains #152's shared index/API bundle, external host packages and Node builtins, factory-time RPC, first-request schemas, factory-return admission, and bounded worker profiling. Compiled index/chunk hashes still match #152's measured bundle.
Validation
Hosted CI 38573211125 passes at exact head
7d3412127c592b2a4f323ba2f78e46b26271da55: Node 22.23.3, Linux 1,286 passed; Windows 1,279 passed / 7 skipped; both build and package checks pass. CodeQL 38573206627 passes at the same head. Windows POSIX skips are not native passes.