Skip to content

Fix protobuf null inference for Ballerina-generated Struct values#491

Open
Delaksan-Sritharan wants to merge 1 commit into
LDFLK:mainfrom
Delaksan-Sritharan:fix/137-protobuf-null-struct-inference
Open

Fix protobuf null inference for Ballerina-generated Struct values#491
Delaksan-Sritharan wants to merge 1 commit into
LDFLK:mainfrom
Delaksan-Sritharan:fix/137-protobuf-null-struct-inference

Conversation

@Delaksan-Sritharan

Copy link
Copy Markdown

Description

Summary

Fixes protobuf null handling for Ballerina-generated google.protobuf.Struct values in the inference engine.

Changes

  • Treat structpb.Value objects with unset Kind as JSON null
  • Update schema generation to support scalar nulls represented with nil kind
  • Update storage inference to classify single-field structs with nil kind as scalar null
  • Add regression tests for unset-kind null handling in:
    • pkg/schema
    • pkg/storageinference

Verification

  • go test ./pkg/schema ./pkg/storageinference

Issue

Closes: #137

@gemini-code-assist

Copy link
Copy Markdown
Contributor

Caution

The consumer version of Gemini Code Assist on GitHub has been sunset. All code review activity has officially ceased.

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.

1 participant