Skip to content

doc: clarify CompanyOverview.employees is a string type across all SDKs - #574

Merged
huacnlee merged 2 commits into
mainfrom
copilot/fix-employee-schema-type
Aug 17, 2026
Merged

doc: clarify CompanyOverview.employees is a string type across all SDKs#574
huacnlee merged 2 commits into
mainfrom
copilot/fix-employee-schema-type

Conversation

Copilot AI commented Aug 17, 2026

Copy link
Copy Markdown
Contributor

The Longbridge API returns employees as a JSON string (e.g. "10000"), not an integer. This was already correctly modeled as String/str in all SDK types, but the doc comments didn't make this explicit — leaving the door open for downstream tools (e.g. the MCP company tool) to infer or assume an integer type, causing outputSchema mismatches.

Changes

  • Rust, Python, Node.js SDK bindings — updated employees doc comment to state "returned as a string by the API, e.g. "10000""
  • Python type stub (openapi.pyi) — updated docstring accordingly
  • Java (CompanyOverview.java) — updated Javadoc to match
  • CHANGELOG — added entry under [Unreleased] → Fixed

The runtime fix (changing Option<i64>Option<String> in the MCP output schema) must be applied in the longbridge-mcp repository at src/tools/output/fundamental.rs.

  • Fixes #1184

Co-authored-by: huacnlee <5518+huacnlee@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix employee schema type in company outputSchema doc: clarify CompanyOverview.employees is a string type across all SDKs Aug 17, 2026
Copilot AI requested a review from huacnlee August 17, 2026 03:40
@huacnlee
huacnlee marked this pull request as ready for review August 17, 2026 03:43
@huacnlee
huacnlee merged commit 78cf6ae into main Aug 17, 2026
6 checks passed
@huacnlee
huacnlee deleted the copilot/fix-employee-schema-type branch August 17, 2026 03:43
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.

[MCP] company outputSchema declares employees as integer, but API returns string

2 participants