Repository navigation
chore(conformance): record cachekit-ts's encryption 1.5.0 and current python-frame copies (LAB-8699) - #187
Conversation
… python-frame copies
|
Warning Review limit reachedYou've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. Next included review available in 19 minutes. View limit detailsLimit details: You’ve used the included review currently available. Review configuration: ⚙️ Run configuration
📒 Files selected for processing (4)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Code Review Completed! 🔥The code review was successfully completed based on your current configurations. Kody Guide: Usage and ConfigurationInteracting with Kody
Providing Context (Files & MCPs)Add these hints in your PR description (or a comment) to unlock deeper checks:
Current Kody ConfigurationReview OptionsThe following review options are enabled or disabled:
|
…c52c45-187 # Conflicts: # conformance/coverage.md # sdk-feature-matrix.md
Code Review Completed! 🔥The code review was successfully completed based on your current configurations. Kody Guide: Usage and ConfigurationInteracting with Kody
Providing Context (Files & MCPs)Add these hints in your PR description (or a comment) to unlock deeper checks:
Current Kody ConfigurationReview OptionsThe following review options are enabled or disabled:
|
Records cachekit-ts's re-vendored fixtures from cachekit-io/cachekit-ts#230 (merged as
83e0566):encryption.json1.5.0 and the currentpython-frame.json, which carry cachekit-io/protocol#180 and cachekit-io/protocol#181. No spec or fixture change.conformance/sdks.jsonreads cachekit-ts at83e0566. All eight copies were re-hashed from that commit, and only two changed:encryption.jsonis now1a8a3735…961589andpython-frame.jsonb677d5f1…5600d0.conformance/coverage.mdis regenerated. cachekit-ts's copies of both fixtures are now the current revision. Its summary row goes from 9 / 73 / 12 (covered / partial / uncovered) to 9 / 78 / 7: ENC-1, ENC-2, ENC-3, ENC-7 and ENC-9 move fromuncoveredtopartial (gap), and ENC-5, ENC-6, ENC-10, PRE-16 to PRE-18, PRE-22, PRE-25, PRE-28 and IOP-19 move from a partial vector count topartial (gap). Every mapped vector is now vendored; only each requirement's recorded gap keeps it fromcovered. No other SDK's column changed.sdk-feature-matrix.md, the TypeScript Test vectors in CI cell records what the SDK's tests drive from each copy. No row that binds cachekit-ts fails, so it records no deviation. Note ¹⁶ records both copies. Its cachekit-tsencryption.jsonsentence, which still said1.3.0, now says1.5.0.changelog.d/20261009_lab-8699.md.Merged with
mainafter cachekit-io/protocol#186 (the cachekit-rs record) landed in the same three files. The matrix keepsmain's text and re-applies only the TypeScript clauses, andcoverage.mdis regenerated from the merged tree. The cachekit-ts moves are unchanged, and no other SDK's column moves.Verification
python3 tools/conformance.py check --base origin/main: OK, andcoverage.mdis current.python3 tools/test_conformance.py: all 284 cases pass, re-run on the merged tree.Closes LAB-8699