docs(matrix): cachekit-core envelope pre-scan on main, unreleased (LAB-3479) - #82
Conversation
Wire format (ByteStorage): the Rust (canonical cachekit-core) cell moves to in review with cachekit-core#80, which runs the Retrieve Flow step-2 pre-scan in ByteStorage::retrieve and executes decode-bounds.json 1.1.0 through it. Python and TypeScript cells and the TypeScript test-vectors cell stay warnings, name the core PR, and flip on each SDK's core bump.
|
Warning Review limit reachedNext included review available in 3 minutes. View limit detailsLimit details: You’ve used the included review currently available. You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. Review configuration: ⚙️ Run configurationConfiguration used: Repository: cachekit-io/protocol/.coderabbit.yaml Review profile: ASSERTIVE Plan: Advanced Run ID: 📒 Files selected for processing (2)
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:
|
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:
|
…e-pre-scan-matrix # Conflicts: # sdk-feature-matrix.md
Resolves the sdk-feature-matrix.md conflict with the decode-bounds 1.1.0 re-vendor (LAB-3481): main's Test vectors row and footnote 16 are kept, and this PR's cachekit-core#80 notes are applied on top of them. cachekit-core#80 has merged, so the Rust Wire format cell, footnote 16 and the changelog fragment use the merged-unreleased form instead of "in review". The TypeScript cells name both bindings that reach retrieve (cachekit-core-ts and cachekit-core-wasm).
|
@kody review --force |
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:
|
Updates the feature matrix for cachekit-io/cachekit-core#80, merged to core
main. It adds the Retrieve Flow step-2 structural pre-scan toByteStorage::retrieveand runsdecode-bounds.json1.1.0 through it in core CI.cachekit-core):cachekit-core-tsandcachekit-core-wasm.unpack.cachekit-corevendorsdecode-bounds.json1.1.0.Changelog entry:
changelog.d/20260929_lab-3479.md.tools/test_changelog_collect.py,check-version-floors.pyandcheck-spec-duplication.pypass locally.This PR is a documentation-only update to the SDK feature matrix. It records that the
cachekit-coreenvelope structural pre-scan (Retrieve Flow step 2) has merged to coremainvia cachekit-core#80. It has not been released: crates.io 0.6.0 does not include it.Changes to
sdk-feature-matrix.mdWire format (ByteStorage) row
cachekit-core):main,ByteStorage::retrieveandvalidatenow pre-scan the envelope bytes before the typedrmp_serde::from_sliceintoStorageEnvelope. The cell stays short of ✅ because crates.io 0.6.0 does not have the change.unpackin bothcachekit-core-tsandcachekit-core-wasm. The cell flips only when both bindings move to a core release that includes the pre-scan.Test vectors in CI row
ByteStorage::retrieve) now names the same release condition.unpackin both bindings. It requires two things: a core release carrying chore(changelog): one fragment file per entry in changelog.d/, collected at release (LAB-6153) #80, and the vectors being driven throughunpack.Footnote ¹⁶
cachekit-corevendorsdecode-bounds.json1.1.0 and drives it throughByteStorage::retrieveon coremain, which is unreleased.Header
Changes to
changelog.d/20260929_lab-3479.md(new)Public API references
No APIs are modified in this repository. The matrix now documents pre-scan behavior for the following
cachekit-coreentry points, on coremainonly:ByteStorage::retrieveByteStorage::validateunpack, exposed viacachekit-core-tsandcachekit-core-wasm