refactor(adaptive-delivery): migrate mechanics to Python - #201
Merged
Merged
Conversation
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.
Why
Adaptive-delivery's preflight, host routing, and evaluation fixtures relied on Bash/AWK mechanics that prevented native Windows use. Closes #183.
What changed
Verification
bun run check:python— passes, including Ruff, strict mypy, package tests, and separate 95% statement/branch gates.PYTHONIOENCODING=cp1252and now passes with explicit UTF-8/LF console output. Final local package coverage is 99.87% statements and 99.28% branches.bun run lint,bun run lint:ts,bun run lint:shell,bun run typecheck,bun run check:decisions, andbun run check:docs— pass.bashand/bin/bash(3.2). Python CI passes, including Python 3.10–3.13 on Linux/macOS/Windows, fresh copied runtime-only installation on all three platforms, and Bash 3.2/5 regression scenarios.bun run check:docs:external— passes (86 external URLs).planning transfer does not turn missing repository policy into authority), reproduced on unchanged base commit50d796e.Review notes
The public commands now use
uv run --quiet --frozen --no-dev --project <absolute-plugin-backend>; obsolete Bash entrypoints are removed. Invalid JSON syntax uses standard-library line/column diagnostics; semantic validation and exit/status contracts are preserved. The unrelated discovery test remains unchanged; all PR CI checks pass.Checklist
CONTRIBUTING.md, including the contributionlicensing terms.
this change does not affect a capability invariant.
behavior.
change does not affect plugin content.
bun run check:python, or this change does not affect registeredPython packages or their repository quality infrastructure.