Repository navigation
Reduce current task-picker startup time by 4.66% (no new κ) - #6179
Merged
Mister-Meeseeks merged 1 commit intoOct 29, 2026
Merged
Mister-Meeseeks merged 1 commit into
Mister-Meeseeks merged 1 commit into
Conversation
This was referenced Oct 29, 2026
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.
Summary
The current task-picker CLI executes the frozen picker in the parent before launching the same source in a child. Defer the parent load until historical route validation needs it. Current cards keep the same complete output with less startup work.
Claim type: empirical tooling improvement, with a scoped equivalence argument and finite conformance evidence. The recorded complete-command median falls from 104.461 ms to 99.598 ms, 4.66% less wall time. No mathematical bound, supplier admission, new kappa or selected-frontier change is claimed. This contribution was produced by Nick Petalas.
Only one implementation line moves. The separate research package supplies the exact-source checker, benchmark driver, generated timing receipt, provenance, limitations and next question. The first improving implementation is frozen.
Evidence
(status, stdout, stderr)triple agrees. The independent direct-CLI control agrees. Every paired batch improves. Batch-average ranges are 103.434–110.213 ms before and 93.962–105.575 ms after; paired decreases range from 3.769 to 9.565 ms. Raw ranges overlap, so the paired observations and host-specific scope matter.637a0513b0ca75053fc62ddb99d1ac43eb1cb333, package metadata/replay, committed bytes/modes and declared source ancestry pass. The committed-diff planner selects only this package. The official ten-file export passes integrity checks; its optimized read-only Git-free replay passes with all eleven extracted file payloads unchanged, including the export receipt.Exact reproduction from the repository root:
Any CPython 3.12 executable can replace the local path. Expected markers are
PASS_CURRENT_PICKER_LAZY_VALIDATION,PASS_REGISTERED_RESEARCH_PACKAGEandPASS_RESEARCH_COMMIT_BYTES. To repeat timings, choose a scratch directory and a new receipt path:Limitations and credit
This changes the replay-runtime card from scientific replay to research onboarding startup. The ratio is an empirical result for the recorded source-executed CLI, with a common launcher on both sides. It is not a cross-host speed guarantee or an acceleration of the selected scientific replay. Historical mapped routes still need validation and have no performance claim. Arbitrary side-effectful replacement modules, external consumers of the removed incidental module global and concurrent source edits are outside the equivalence argument. No public helper API is documented for that global.
No native/selected cold replay, broad historical scientific suite, Lean, Docker, independent human review or proof-kernel acceptance was run or claimed. No mathematical proof obligation is presented as discharged. Python-managed allocation and authenticated source execution are bounded, documented Power of Ten deviations in the package.
Credit Ali Romman / #5378 for the wrapper, Mihir Srivastava / #4906 and #5520 for startup guidance and historical routing, and Douglas Colkitt and the retained sparse-reference contributors for the frozen picker and validation. Deferred initialization is standard programming practice; this contribution is its measured application at this CLI boundary.
Advance CLAIM. Canonical main refreshed unchanged at
3f0152a32eeebac0df11946dbd50cabe5a7f7839, priority revision 8. Cached overlap covered 319 open PRs and 5,871 Discussion records with complete remote pagination. Its sparse local scan covered only 91 of 8,567 note paths; separate main-tree and merged-PR searches covered the candidate terms. These are bounded overlap checks, not proof of novelty.Open next questions
Does startup loading still matter for historical mapped routes? Profile those complete commands before changing their execution boundary. The next change must preserve child failure propagation, source freshness and sparse-reference rejection. No broader startup change is included here.
Related open work
Completion refresh inspected the eleven other open PRs created by this account since the advance CLAIM at
2026-10-29T13:09:45Z. None computes this current-picker startup quantity. The other contributions cover diagnostic/certificate capabilities and mathematical constructions or obstructions. No additional scientific computations were run to compare their results.CI notes
At head
637a0513b0ca75053fc62ddb99d1ac43eb1cb333, run 40880971198 passed both hosted checks: plan and replay (research/current-picker-lazy-validation). The completed checks were read once and the PR head matches this commit. Hosted replay certifies the registered bounded normal/optimized correctness checks and recorded measurement bindings; it does not repeat or independently certify the timing ratio. The 4.66% reduction remains the recorded local empirical result. Independent human and formal proof-kernel review are unclaimed.Merge danger
Door: two-way.
Blast radius: onboarding.
The current-command wrapper and one isolated opt-in package change. Existing route and source checks remain in place, and the historical picker, catalogue, frozen evidence and selected frontier retain their bytes.