Skip to content

Two incompatible protobuf toolchains leak camelCase field names into Rust call sites #8

Description

@jhamill34

entities/core and entities/credentials use protobuf/protobuf-codegen (rust-protobuf 3.2), while entities/engine uses prost/tonic-build (prost 0.11 / tonic 0.9). rust-protobuf does not convert field names to snake_case, so consumers end up reading fields like resultsPath, startPage.value, maxLimit.value, connectorId, connectorOperation, outputSelectors directly in ordinary Rust code — non-idiomatic camelCase leaking into otherwise-idiomatic call sites. Seen throughout runners/api_caller/src/lib.rs and runners/filtered_runner/src/lib.rs.

Suggested fix: standardize on one protobuf toolchain workspace-wide — prost is the more idiomatic default and is already in use for entities/engine — and migrate entities/core/entities/credentials to match, which fixes the naming issue at the source rather than patching call sites.

Metadata

Metadata

Assignees

No one assigned

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions