Proof Portal

Projects

jsonparser

ProbeLabs23 findings · 123 requirements
Showing the current git stateNo audit has been synced for this project yet — verification results (proof status, coverage, solver signals) appear once an audit is published.proof-demod79405e
All requirements
RequirementSYS-REQ-105SystemApproved

When GetUnsafeString is called on JSON containing values with Unicode edge cases (BOM, ZWJ, multi-byte sequences), the parser shall return the raw bytes as a Go string without corruption or panic, even though unescaping is skipped.

This requirement changed after its last recorded review, so approval is stale. Automated checks pass.
PriorityshallTypeguaranteeCategoryfunctionalComponentparserAssuranceEFindingsnone open

Specification

The requirement exactly as authored — its complete prose text and, where present, the formal FRETish sentence it compiles to.

Description

When GetUnsafeString is called on JSON containing values with Unicode edge cases (BOM, ZWJ, multi-byte sequences), the parser shall return the raw bytes as a Go string without corruption or panic, even though unescaping is skipped.

FRETish formula
the parser shall always satisfy !getunsafestring_input_has_unicode_edge_cases | getunsafestring_handles_unicode_edges_safely
View full formal model

Rationale & tags

Why this requirement exists, and how it is categorised.

GetUnsafeString returns raw bytes as a Go string. Multi-byte sequences at value boundaries must not cause slice-out-of-bounds panics.

getunsafestringedge_caseunicode

Verification & provenance

How this requirement was checked: the review trail, edit history, and the machine-analysis status terms (each ⓘ explains what it means).

Assurance levelE
Formalizationvalid
Strategyfretish

Review

Status
approved
Reviewer
Buger · lead_engineer
Reviewed
Aug 20, 2026, 13:11 UTC
Spec-conformance reviewed (agent:claude:spec-conformance); approved on owner authorization.

History

Created
Apr 21, 2026, 00:00 UTC · Claude · AI agent
Modified
Aug 20, 2026, 13:11 UTC · Buger

Change history

Every recorded revision of this requirement's source file — newest first, each with its commit message and the diff for that change.

Review history

Human and AI-agent approvals of this requirement — the 'why was this approved' lineage, each with the reviewer's justification and the code it cites.

  1. Claude:spec Conformance · AI agentApprovedSpec conformanceAug 20, 2026 · last monthREVIEW-85

    GetUnsafeString returns the raw value bytes via bytesToString and never invokes Unescape (parser.go:2034-2040), so the unescape-path demo defects (KI-14 solidus, KI-8/KI-15 surrogates) cannot corrupt or crash it. Get scans the enclosing JSON string structurally and is agnostic to interior UTF-8/BOM/ZWJ bytes, so TestGetUnsafeStringUnicodeEdgeCases (obligation_property_test.go:802) confirms no panic across BOM, raw multi-byte, emoji, and ZWJ values. The malformed-shape MCDC Row2 witness returns a clean "Key path not found" error rather than panicking, which the test explicitly accepts. All witness tests pass.

    Cited code (3)

Obligations

What this requirement must witness to be considered satisfied — the required evidence, and the tests that discharge each one.

The tests that discharge each obligation need a synced audit

The evidence matrix behind each obligation comes from the audit index, which is produced by running an audit — not read from git. Nothing here means unknown — not that the requirement has no obligations.

Formula evidence

The formal formula behind this requirement, the variables it is written over, and the tests that exercise it (each term is explained inline).

FRETish formula

the parser shall always satisfy !getunsafestring_input_has_unicode_edge_cases | getunsafestring_handles_unicode_edges_safely

Variables

NameTypeDirectionDescription
getunsafestring_input_has_unicode_edge_casesTrue when GetUnsafeString addresses a value with Unicode edge cases.
getunsafestring_handles_unicode_edges_safelyTrue when GetUnsafeString returns raw bytes as Go string without corruption for Unicode edge cases.

Witnesses· 3 scenarios total

  • TestMCDC_SYS_REQ_105_Row1_TriggerFalse
    exercises 1 condition scenario
  • TestMCDC_SYS_REQ_105_Row2_InvariantViolation
    exercises 1 condition scenario
  • TestGetUnsafeStringUnicodeEdgeCases
    exercises 1 condition scenario

MC/DC truth table· 3 rows

Each row assigns the formula's conditions (T/F) and shows the Result— the formula's value for that input row, not a test pass/fail. A row proves a condition when flipping only that condition flips the outcome. The test that covers each row is linked.

Covereda test exercises this rowExempteda reviewed mcdc:ignoreNo-actionfalse-result row satisfied by designUncoveredneeds a covering test
#getunsafestring_handles_unicode_edges_safelygetunsafestring_input_has_unicode_edge_casesResultProvesCovering test
1FFTgetunsafestring_input_has_unicode_edge_cases
2FTFgetunsafestring_handles_unicode_edges_safely
3TTTgetunsafestring_handles_unicode_edges_safely

Its place

How this requirement connects — what proves it, what it affects, and what it rests on. Authored links only here; automatically derived links come from the audit index.

Loading graph…

Trace evidence

The concrete artifacts linked to this requirement — implementing code, verifying tests, documents, and the findings raised against it.

No implementation linked

No source files are traced to this requirement yet.

Impact

Blast radius — authored trace links only (automatically derived links come from the audit index and aren't shown here).

If you change this

Requirements
0
Files
0
Tests
2
At-risk contracts
0

Tests to re-run (2)

  • mcdc_spec_witnesses_test.gomcdc_spec_witnesses_test.go
  • obligation_property_test.goobligation_property_test.go

What this rests on

Discussions

Discuss this with the proof team. Nothing changes in your audit automatically — you open a request and a staff member records any outcome inside the thread.

Sign in to discuss this with the proof team.Sign in