Skip to content

fix: restore bin/ dir for release SBOM step#55

Merged
monstercameron merged 1 commit into
masterfrom
v4
Jun 29, 2026
Merged

fix: restore bin/ dir for release SBOM step#55
monstercameron merged 1 commit into
masterfrom
v4

Conversation

@monstercameron

Copy link
Copy Markdown
Owner

Restores bin/README.md (deleted in the v4 squash) so the bin/ directory exists on a fresh checkout; the release workflow's CycloneDX SBOM step writes bin/sbom.cyclonedx.json and was failing without it.

🤖 Generated with Claude Code

.gitignore keeps bin/ present only via the un-ignored bin/README.md
(`/bin/**` + `!/bin/README.md`). That file was deleted in the v4 squash, so a
fresh checkout had no bin/ dir and the release workflow's "Generate CycloneDX
SBOM" step (which writes bin/sbom.cyclonedx.json) failed with
"cannot find the path specified", blocking the release job.

Restore bin/README.md to keep the directory tracked, as before v4.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@monstercameron monstercameron merged commit e05efbd into master Jun 29, 2026
monstercameron pushed a commit that referenced this pull request Jul 6, 2026
passphraseEncryptor memoized every salt->derived-key pair with no bound.
Sealing reuses this instance's single salt, but decryption derives from
each ciphertext's HEADER salt, so decrypting data sealed by many replicas
(each with its own random salt) grew the cache — and retained a derived
key (sensitive material) — one entry per distinct salt, without limit.
Cap the memo at 128 entries with FIFO eviction; a later hit on an evicted
salt simply re-runs PBKDF2.

Pinned by TestDeriveKeyCacheIsBounded. Assessment of the rest of #55:
localfirst presence already TTL-expires peers; the text CRDT's tombstone
map growth is the standard CRDT tradeoff whose safe GC needs causal-
stability tracking (an owner-level feature, not a contained fix).

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants