Consume pinned portable v5 rule releases - #43
Merged
Merged
Conversation
zhaobozhen
force-pushed
the
codex/portable-rules
branch
from
September 20, 2026 06:28
dddf6e2 to
1be5ecf
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.
Consume a pinned portable v5 Rules release instead of maintaining an independent rule import contract. Verify the immutable artifact and metadata before generating shared lookup, localized-detail and icon data, retaining the existing web/Worker behavior and lazy loading.
Validation: targeted Node/Python importer tests, 19 shared matching fixtures, 2,736 literal lookups, integrity/locale/cache checks and offline generation passed. Pages build and all size budgets passed (core 483.1 KiB, details 2.18 MiB, icons 452.8 KiB, total JS 3.48 MiB, four initial requests). Portable entries are unchanged except release metadata. Unrelated UI/runtime tests and Worker dry-run were not repeated for the final metadata refresh; no deployment, Telegram or KV operation was performed.
After Rules 45 is published from
7b0a6f92806de7b6c7cf578049b28b555564c223, runnpm run rules:update -- --manifestto replace the local bootstrap with the verified remote lock, then run check/build/size gates. Merging to main invokes this repository's existing production deployment workflow; review that rollout deliberately.Related Rules migration: LibChecker/LibChecker-Rules#1386