Fix shared schema comparison for Go codegen

[stephentoub]

Schema definitions shared between api.schema.json and session-events.schema.json can have identical wire shapes but different documentation. With the 1.0.49-0 Copilot runtime schemas, that caused Go codegen to miss ReasoningSummary as shared and emit duplicate declarations in the rpc package.

This updates shared-definition comparison to ignore documentation-only metadata (description and markdownDescription) while preserving the existing structural comparison for refs, enum values, and object shapes. The regression test now covers a shared enum with intentionally different descriptions.

[Copilot]

Pull request overview

Fixes Go codegen duplicate declarations by making shared-schema comparison ignore documentation-only fields (description, markdownDescription) so definitions with identical wire shapes but differing docs are still recognized as shared.

Changes:

• Skip description and markdownDescription keys during normalization in normalizeDefinitionForComparison.
• Update shared-codegen test fixtures to include intentionally divergent descriptions on a shared enum.

Show a summary per file

File	Description
scripts/codegen/utils.ts	Ignore doc-only metadata when comparing shared schema definitions.
nodejs/test/shared-codegen.test.ts	Adds divergent descriptions to shared enum to cover the fix.

Copilot's findings

• Files reviewed: 2/2 changed files
• Comments generated: 0

[github-actions]

Cross-SDK Consistency Review ✅

This PR makes changes only to shared codegen tooling (scripts/codegen/utils.ts) and its test — no public API surface is added or modified in any of the four SDK implementations (nodejs/src/, python/copilot/, go/, dotnet/src/).

Summary of changes:

• scripts/codegen/utils.ts: normalizeDefinitionForComparison now ignores description and markdownDescription fields when comparing schema definitions for structural equivalence. This correctly treats schemas that are wire-identical (same type/enum/shape) but have different human-readable documentation as the same shared type.
• nodejs/test/shared-codegen.test.ts: Regression test updated to cover shared enums with intentionally different descriptions.

No consistency concerns. The fix is to shared infrastructure that benefits all language code generators equally — it prevents duplicate type declarations in Go's generated RPC code when the upstream schema has the same type documented differently in api.schema.json vs. session-events.schema.json. No changes are needed in any other SDK.

Generated by SDK Consistency Review Agent for issue #1304 · ● 210.5K · ◷