Proof Portal

Projects

jsonparser

ProbeLabs23 findings · 123 requirements

The fastest JSON parser for Go — formally verified with ReqProof (real library, master).

All requirements
RequirementSYS-REQ-047SystemReview

When the current path segment is a negative array index (e.g., "[-1]"), Get shall return the defined not-found result because negative indexing is not supported by the JSON path syntax.

This requirement changed after its last recorded review, so approval is stale. Automated checks pass and 1/1 obligations are satisfied.
PriorityshallTypeguaranteeCategoryfunctionalComponentparserAssuranceEFindingsnone open

Specification

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

Description

When the current path segment is a negative array index (e.g., "[-1]"), Get shall return the defined not-found result because negative indexing is not supported by the JSON path syntax.

FRETish formula
the parser shall always satisfy !path_segment_is_negative_array_index | returns_not_found_for_negative_array_index
View full formal model

Rationale & tags

Why this requirement exists, and how it is categorised.

Negative array indices are not part of the documented path syntax. The parser should deterministically handle them rather than producing undefined behavior.

array-indexedge-case

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
Realizabilityrealizable
Vacuitychecked_ok
Strategyfretish

Review

Status
approved
Reviewer
Leonidbugaev
Reviewed
Apr 23, 2026, 00:00 UTC

History

Created
Apr 14, 2026, 18:00 UTC · Claude · AI agent
Modified
Jul 26, 2026, 14:54 UTC · Created via CLI

Change history

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

Every recorded revision of this object's source — newest first, each with the commit message and its file-level diff.

Obligations

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

1 obligation · 1 discharged

Browse the catalogue
Discharged
If it were violatedHigh

Get on path [-1] is parsed by strconv.Atoi as -1 and then used as data[i-1] or an array element index, wrapping to a large unsigned index and panicking with index-out-of-range.

Discharging evidence1/1 required witnessed
  • nominalrequiredpresent
    Covered by 1 test

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 !path_segment_is_negative_array_index | returns_not_found_for_negative_array_index

Variables

NameTypeDirectionDescription
path_segment_is_negative_array_indexTrue when the current path segment is a negative array index such as "[-1]".
returns_not_found_for_negative_array_indexTrue when Get returns the defined not-found result for a negative array index because negative indexing is not supported.

Witnesses· 3 scenarios total

  • TestMCDC_SYS_REQ_047_Row1_TriggerFalse
    exercises 1 condition scenario
  • TestMCDC_SYS_REQ_047_Row2_InvariantViolation
    exercises 1 condition scenario
  • TestMCDC_SYS_REQ_047_Row3_NegativeNotFound
    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
#path_segment_is_negative_array_indexreturns_not_found_for_negative_array_indexResultProvesCovering test
1FFTpath_segment_is_negative_array_index
2TFFpath_segment_is_negative_array_index
3TTTreturns_not_found_for_negative_array_index

Its place

This requirement shown inside its trace neighbourhood — the parents it satisfies, the code and tests attached to it, and its findings.

Loading graph…

Trace evidence

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

Impact

Blast radius — if you change this requirement, what else may need re-checking, and what it in turn depends on.

If you change this

Requirements
0
Files
1
Tests
3
At-risk contracts
0

Files to re-check (1)

  • parser.goparser.go

Tests to re-run (3)

  • deep_spec_test.godeep_spec_test.go
  • mcdc_spec_witnesses_test.gomcdc_spec_witnesses_test.go
  • obligation_evidence_test.goobligation_evidence_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