Skip to content

Add nocopy.JSON#27

Merged
kelindar merged 2 commits into
masterfrom
codex/add-nocopy-json
Jul 25, 2026
Merged

Add nocopy.JSON#27
kelindar merged 2 commits into
masterfrom
codex/add-nocopy-json

Conversation

@kelindar

Copy link
Copy Markdown
Owner

Summary

Add nocopy.JSON, a raw JSON value that works in the same struct with both encoding/json and kelindar/binary.

  • JSON unmarshal copies input using json.RawMessage semantics
  • JSON marshal emits raw JSON rather than base64
  • binary decode reuses the underlying input slice through the existing byteSliceCodec

Tests

Coverage verifies JSON input ownership, raw JSON output, binary aliasing, and the normal nocopy binary round trip.

Checks

  • go test ./...
  • go vet ./...
  • git diff --check

@github-actions

Copy link
Copy Markdown
100.0%

passed

████████████████████100.0% on changed lines

9 lines changed   0 uncovered   1 file

 📂 1 file changed
File Coverage Uncovered Lines
🟢 nocopy/types.go 100.0%
 📋 Full diff-cover report

Diff Coverage

Diff: origin/master...HEAD, staged and unstaged changes

  • nocopy/types.go (100%)

Summary

  • Total: 9 lines
  • Missing: 0 lines
  • Coverage: 100%

🛡️ diff-cover-action

@coveralls

Copy link
Copy Markdown

Coverage Report for CI Build 30176981130

Coverage increased (+0.05%) to 90.315%

Details

  • Coverage increased (+0.05%) from the base build.
  • Patch coverage: 9 of 9 lines across 1 file are fully covered (100%).
  • No coverage regressions found.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

No coverage regressions found.


Coverage Stats

Coverage Status
Relevant Lines: 1776
Covered Lines: 1604
Line Coverage: 90.32%
Coverage Strength: 592.24 hits per line

💛 - Coveralls

@kelindar
kelindar marked this pull request as ready for review July 25, 2026 22:22
@kelindar
kelindar merged commit 5de7ec9 into master Jul 25, 2026
1 check passed
@kelindar
kelindar deleted the codex/add-nocopy-json branch July 25, 2026 22:23
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