Repository navigation
Release git-cas v6.5.7 #117
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Changes from all commits
Commits
Show all changes
2 commits
Select commit
Hold shift + click to select a range
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
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
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
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
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
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
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
84 changes: 84 additions & 0 deletions
84
docs/design/0058-bounded-stream-session-reads/witness/release-candidate.md
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,84 @@ | ||
| # PERF-0058 v6.5.7 Release Candidate Witness | ||
|
|
||
| Date: 2026-08-23 | ||
|
|
||
| Issue: #115 | ||
|
|
||
| Implementation review: #116 | ||
|
|
||
| Release review: #117 | ||
|
|
||
| ## Scope | ||
|
|
||
| This witness records the pre-publication candidate for bounded session-backed | ||
| small stream reads. It does not claim that a `v6.5.7` tag, npm artifact, or | ||
| GitHub Release exists. This is an explicitly unpublished candidate. | ||
|
|
||
| The candidate: | ||
|
|
||
| - sets npm, JSR, and runtime package metadata to `6.5.7`; | ||
| - moves bounded stream-session reads from `Unreleased` to `6.5.7`; | ||
| - packages and links `docs/releases/v6.5.7.md`; | ||
| - records v6.5.6 publication evidence as immutable history; | ||
| - marks design 0058 landed after implementation merge #116; and | ||
| - leaves v6.5.7 tag and registry claims deliberately absent. | ||
|
|
||
| ## Implementation Provenance | ||
|
|
||
| | Capability | Review anchor | Commit | | ||
| | --- | --- | --- | | ||
| | Bounded stream-session route | #116 | `135a8ff416b12e6abede0b0f78a3e6ba00ca1255` | | ||
| | Deterministic process witness | #116 | `a34762c9ef1cf0bf4f717d2892d1ca0744f7e762` | | ||
| | Failure-safe real-Git cleanup | #116 | `c344d119fd2afb5f7c024b4912714acbfd156768` | | ||
|
|
||
| Implementation review #116 merged normally as | ||
| `1e30740c8670bf42b8bb863f8feb99a5e0f0f29b`. Its second parent is the exact | ||
| reviewed head `c344d119fd2afb5f7c024b4912714acbfd156768`. | ||
|
|
||
| The immutable versioned candidate-preparation commit is | ||
| `7631d597b1091f5cb5c29c7c7770a5a2bf435cc7`. It is the direct child of | ||
| implementation merge `1e30740c8670bf42b8bb863f8feb99a5e0f0f29b`; its tree is | ||
| the exact candidate tree that produced the 6,922-test verification record | ||
| below. Final release identity remains the normal merge of release review #117 | ||
| and the signed tag that peels to that merge. | ||
|
|
||
| ## Failure Witness | ||
|
|
||
| At the RED commit, repeated `readBlobStream()` calls still opened one one-shot | ||
| `cat-file` child per blob and never used the typed session content path. The | ||
| GREEN witness uses 32 deterministic 4,096-byte objects: fallback opens 32 | ||
| children, the session route opens one child, and both routes produce the same | ||
| semantic digest. | ||
|
|
||
| The 10 MiB + 1 byte case opens one persistent child for metadata and exactly | ||
| one one-shot child for content. It performs no session content read and leaves | ||
| no active session after close. | ||
|
|
||
| ## Verification | ||
|
|
||
| The implementation tree and the versioned release candidate each ran the | ||
| complete `npm run release:verify` method. Final candidate counts are: | ||
|
|
||
| | Gate | Result | Observed tests | | ||
| | --- | --- | ---: | | ||
| | Lint | PASS | - | | ||
| | Unit tests (Node) | PASS | 2,109 | | ||
| | Unit tests (Bun) | PASS | 2,108 | | ||
| | Unit tests (Deno) | PASS | 2,099 | | ||
| | Public type compatibility | PASS | - | | ||
| | Integration tests (Node) | PASS | 202 | | ||
| | Integration tests (Bun) | PASS | 202 | | ||
| | Integration tests (Deno) | PASS | 202 | | ||
| | Examples and build stamp | PASS | - | | ||
| | npm and JSR dry-runs | PASS | - | | ||
| | **Release method summary** | **PASS: 14/14 gates** | **6,922** | | ||
|
|
||
| Tag and publication evidence remain deliberately absent from this candidate | ||
| witness. | ||
|
|
||
| ## Publication Gate | ||
|
|
||
| Publication remains blocked until the release PR passes GitHub CI and review, | ||
| merges normally, the exact merged candidate passes full release verification, | ||
| a signed annotated `v6.5.7` tag peels to that reviewed merge, and the release | ||
| workflow publishes npm plus the final GitHub Release. | ||
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
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
37 changes: 37 additions & 0 deletions
37
.../method/retro/0058-bounded-stream-session-reads/bounded-stream-session-reads.md
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,37 @@ | ||
| # Retro — 0058 Bounded Stream Session Reads | ||
|
|
||
| ## Drift Check | ||
|
|
||
| - The cycle stayed inside the Git persistence adapter, deterministic process | ||
| diagnostics, release evidence, and compatibility documentation. | ||
| - It did not widen the public API, change storage or retention truth, cache | ||
| payloads in consumers, or buffer objects above the fixed ceiling. | ||
|
|
||
| ## What Shipped To Main | ||
|
|
||
| - Small `readBlobStream()` payloads at or below 10 MiB can reuse one persistent | ||
| typed `cat-file` session. | ||
| - Oversized content remains on exactly one genuine one-shot stream after a | ||
| bounded metadata lookup. | ||
| - Unit and SHA-1/SHA-256 real-Git tests prove route selection, exact bytes, | ||
| fixed-bound authority, fallbacks, and deterministic close. | ||
| - Implementation PR #116 merged normally as | ||
| `1e30740c8670bf42b8bb863f8feb99a5e0f0f29b`. | ||
|
|
||
| ## What Did Not Yet Ship | ||
|
|
||
| - The `v6.5.7` signed tag, npm artifact, provenance, and GitHub Release remain | ||
| deliberately absent until the reviewed release candidate merges. | ||
| - git-warp and Think remain untouched; their publication order begins only | ||
| after git-cas v6.5.7 is independently verified live. | ||
|
|
||
| ## Debt | ||
|
|
||
| - None added in git-cas. A persistent truly streaming `cat-file` protocol would | ||
| require separate plumbing ownership and evidence if a future workload | ||
| justifies it. | ||
|
|
||
| ## Cool Ideas | ||
|
|
||
| - Reuse the committed process-count schema when git-warp consumes v6.5.7 so | ||
| downstream improvement is compared structurally, not inferred from timing. |
Oops, something went wrong.
Oops, something went wrong.
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.
Uh oh!
There was an error while loading. Please reload this page.