You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
This issue tracks all no-op runs from agentic workflows in this repository. Each workflow run that completes with a no-op message (indicating no action was needed) posts a comment here.
What is a No-Op?
A no-op (no operation) occurs when an agentic workflow runs successfully but determines that no action is required. For example:
A security scanner that finds no issues
An update checker that finds nothing to update
A monitoring workflow that finds everything is healthy
These are successful outcomes, not failures, and help provide transparency into workflow behavior.
How This Helps
This issue helps you:
Track workflows that ran but determined no action was needed
Distinguish between failures and intentional no-ops
Monitor workflow health by seeing when workflows decide not to act
No consistency issues found: PR #1988 is an automated code-generation update (@github/copilot → 1.0.71-1) that regenerates all SDK generated files from the same shared schema. All 6 SDKs (Node.js, Python, Go, .NET, Java, Rust) are updated consistently with the same new event (session.memory_changed), new fields (mcpMeta, workingDirectory), and un-deprecations. No cross-SDK parity gaps detected.
PR #2107 only modifies .NET E2E test code (AskUserE2ETests.cs), switching SendAsync+GetFinalAssistantMessageAsync to SendAndWaitAsync for flake-fix alignment with Node/Python equivalents. No public API surface was changed, no new SDK features were introduced, and no cross-SDK consistency issues were found.
PR #2109 modifies only one file (dotnet/test/E2E/SessionE2ETests.cs) — a test-only fix replacing SendAsync+GetFinalAssistantMessageAsync with SendAndWaitAsync to eliminate a race condition. No public API changes were made. No cross-SDK consistency issues to report.
PR #2120 is a Dependabot bump of GitHub Actions workflow dependencies (actions/checkout, node feature version, etc.). No SDK client code (nodejs/, python/, go/, dotnet/, java/, rust/) was modified. No cross-SDK consistency issues to report.
PR #1986 modifies only .NET test infrastructure files (dotnet/test/). No public SDK API surface is changed in any language. The changes fix in-process E2E transport coverage and test session lifecycle — these are .NET-specific concerns (FFI/in-process transport) with no equivalent in other SDK languages. No cross-SDK consistency issues found.
PR #2017 is a Dependabot dependency bump for jackson-databind 2.22.0 → 2.22.1 in java/pom.xml only. This is a Java-internal dependency version update with no API surface changes. No cross-SDK consistency issues to report.
SDK consistency review complete for PR #2170. This PR is a Java-only internal build restructuring (single-module → multi-module Maven reactor, moving sdk code into java/sdk/ subdirectory). No new API surface or behavior is introduced. No changes touch Node.js, Python, Go, .NET, or Rust SDKs. No cross-SDK consistency issues found — no comment needed.
PR #2175 reviewed for cross-SDK consistency. All 4 changed files are Java-only (java/sdk/pom.xml, NativeRuntimeLoader.java, copilot-runtime.properties, NativeRuntimeLoaderTest.java). This PR implements Java-specific FFI infrastructure for extracting and caching a native runtime.node binary from the JAR classpath — a concern unique to the JVM deployment model. No other SDK (Node.js, Python, Go, .NET, Rust) uses JNA or classpath-based native binary extraction. No cross-SDK consistency issues identified.
Cross-SDK consistency review complete for PR #2181. All 6 SDKs (Node.js, Python, Go, .NET, Java, Rust) are updated consistently: model references changed from gpt-4.1 to gpt-5.4 and tests for reasoning-effort scenarios now use isolated client contexts. Shared YAML snapshots are updated to match. No consistency gaps found.
PR #2193 is an automated codegen update regenerating schema-derived types across all six SDK implementations (Node.js, Python, Go, .NET, Java, Rust) simultaneously. All changed files are auto-generated from the same @github/copilot schema (1.0.78-2). No cross-SDK consistency issues found — the PR maintains parity by design.
PR #2229 adds only a CI validation fixture file and a temporary test file (nodejs/test/ci-validation.test.ts). No SDK client code was modified. No cross-SDK consistency review needed.
PR #2229 ("Test CI behavior") contains no changes to any SDK implementation (Node.js, Python, Go, .NET, Java, or Rust). All 30 changed files are workflow/infrastructure files. No cross-SDK consistency review needed.
PR #2236 adds Java-only Maven build infrastructure (copilot-native module) to package the native runtime.node binary as a classifier JAR. No public SDK API was added or modified. No cross-SDK consistency review needed.
PR #2243 is a Dependabot security update bumping the Node.js devDependency postcss from 8.5.15 to 8.5.25 (and its transitive dep nanoid 3.3.12 → 3.3.17). Only nodejs/package-lock.json is changed — no SDK public API or cross-language consistency concerns. No review action needed.
Cross-SDK consistency review complete for PR #2442 ("Update @github/copilot to 1.0.82"). This PR is an automated schema codegen update that consistently updates all six SDK implementations — Node.js/TypeScript, Python, Go, .NET, Java, and Rust — with the same new types and fields (AutoTier, SandboxEnforcementStatus, AssistantMessageToolRequestCaller, displayText on EnqueueCommandParams, etc.). All generated files across all language SDKs are in sync. No consistency issues found.
Cross-SDK consistency review complete for PR #2437 ("Add Auto routing tier options across all SDKs"). The PR adds capi.autoTier support consistently across all six SDKs:
Node.js: autoTier?: AutoTier in CapiSessionOptions (camelCase ✓)
Python: auto_tier: AutoTier in CapiSessionOptions, mapped to wire autoTier (snake_case ✓)
Go: AutoTier AutoTier in CapiSessionOptions with json:"autoTier,omitempty" (PascalCase exported ✓)
.NET: AutoTier? AutoTier in CapiSessionOptions with [JsonPropertyName("autoTier")] (PascalCase ✓)
Java: setAutoTier(AutoTier) fluent setter on CapiSessionOptions (camelCase method ✓)
Rust: auto_tier: Option<AutoTier> with with_auto_tier() builder (snake_case ✓)
All three tier values (efficiency, balance, intelligence) are exposed in all SDKs. Wire format is consistent (autoTier). Session events (session.start, session.resume) expose autoTier/data.auto_tier uniformly. No consistency issues found.
Reviewed PR #2480: all changes are CI/workflow infrastructure (.github/workflows/*.yml for dotnet, go, java, nodejs, python, rust SDK test workflows, required-checks.yml, and the sdk-consistency-review workflow itself), plus a minor Go e2e test file tweak and a Java pom.xml dependency version bump. No SDK client/API code was modified in any language implementation, so there are no cross-language feature parity or API consistency concerns to flag.
Reviewed PR #2480: changes are limited to CI workflow configuration (.github/workflows/*.yml for dotnet, go, java, nodejs, python, rust SDK test pipelines, plus the consistency-review workflow itself), a minor Go E2E test file (go/internal/e2e/rpc_server_e2e_test.go), and a Java pom.xml dependency version bump. No SDK client/public API code was added or modified in any language, so there is no cross-SDK feature-parity or API-design consistency concern to raise for this PR. No review comments needed.
Reviewed PR #2482: renames ClientInfo fields (editorName/editorVersion/extensionName/extensionVersion -> applicationName/applicationVersion/integrationName/integrationVersion) across all six SDKs (Node.js, Python, Go, .NET, Java, Rust). Verified the rename is applied consistently: same wire-format mapping preserved (still editorName/editorVersion/extensionName/extensionVersion on the wire), naming follows each language's idioms (camelCase, snake_case, PascalCase as appropriate), docs (docs/features/client-info.md, CHANGELOG.md) and tests updated in lockstep across all languages. No cross-SDK inconsistencies found - this PR maintains full feature parity.
PR #2482 renames client-info fields (editorName/editorVersion/extensionName/extensionVersion -> applicationName/applicationVersion/integrationName/integrationVersion) consistently across all six SDKs (Node.js, Python, Go, .NET, Java, Rust), plus docs and CHANGELOG. Verified each SDK follows its own naming convention (camelCase, snake_case, PascalCase as appropriate) and the wire format (editorName/editorVersion/extensionName/extensionVersion) is preserved unchanged for backward compatibility with the runtime handshake. Tests were updated in parallel in all six SDKs. No cross-SDK consistency issues found - this is a clean, coordinated rename.
Reviewed PR #2483: it only regenerates auto-generated RPC/session-event bindings (dotnet/src/Generated, go/rpc, java/sdk/src/generated/java) to add Java bindings for types (DiscoveredHook, HookOrigin, AgentModelPolicy, CompletionReceipt*, SessionCompletionReceiptEvent, FusionPhaseActivity*) that already exist in Node.js, Python, Go, and Rust generated code. No hand-written SDK client logic was touched. This is schema/codegen sync work, not a new feature — all six SDKs already have or now have equivalent generated types with language-appropriate naming (PascalCase Java records/enums matching TS interfaces, Python dataclasses, Go structs, Rust structs). No cross-SDK consistency issues found.
PR #2487 only bumps a dev dependency (js-yaml) in nodejs/package-lock.json. No SDK client code or public API surface is modified, so there are no cross-language consistency implications to review.
Reviewed PR #2483: changes are limited to auto-generated code (dotnet/src/Generated, go/rpc, java/sdk/src/generated) syncing new schema types (AgentModelPolicy, CompletionReceipt*, FusionPhaseActivity*, DiscoveredHook, HookOrigin, etc.) plus java/pom.xml, java codegen package.json/lock, and an unrelated CI workflow shard split for dotnet-sdk-tests.yml. Verified these types already exist in the Node.js, Python, and Rust generated files, so this PR is bringing .NET/Go/Java up to parity with the shared schema rather than introducing an inconsistency. No hand-written public API surface was changed. No cross-SDK consistency issues found — no review comments needed.
Reviewed PR #2483: it only touches auto-generated SDK binding files (dotnet/src/Generated, go/rpc/z*.go, go/zsession_events.go, java/sdk/src/generated/java/**, plus the java codegen package-lock/pom and a CI workflow shard update) — regenerated from an updated api/session-events schema (new types: AgentModelPolicy, DiscoveredHook/HookOrigin, FusionPhaseActivityKind/FusionPhasePlanStep, CompletionReceipt* types, assistant.fusion_phase_activity and session.completion_receipt events). Cross-checked Node.js (rpc.ts/session-events.ts), Python (rpc.py/session_events.py), Go (zrpc.go/zsession_events.go), and Rust (generated/api_types.rs, generated/session_events.rs) generated files: all already contain the equivalent types/events with language-appropriate naming (e.g. Node's CompletionReceiptEvent == Java's SessionCompletionReceiptEvent, both for session.completion_receipt). No cross-SDK feature parity gaps found — this PR is bringing .NET/Go/Java generated bindings up to date with a schema that Node/Python/Rust already reflect. No public API design inconsistencies to flag. No review comments needed.
Reviewed PR #2483: this is a schema-driven codegen regeneration touching all six SDKs (.NET, Go, Java, Node.js, Python, Rust). New RPC/session-event types (HooksDiscoverParams/Result, ServerHooksApi, AgentModelPolicy, CompletionReceipt* types, SessionCompletionReceiptEvent, SessionsReadPersistedEventsParams/Result) are present consistently across all generated files in every language, with parallel naming conventions per language idiom (PascalCase in .NET/Go/Java types, camelCase methods in Java/Node, snake_case in Python/Rust). The only non-generated code change is in scripts/codegen/rust.ts, which sanitizes [::] sequences in doc comments to avoid broken Rust intra-doc links — this is Rust-toolchain-specific (rustdoc doesn't parse [::] as plain text) and doesn't need equivalent changes in other languages. Accompanying test file updates (dotnet/test/E2E/RpcMcpAndSkillsE2ETests.cs, java RpcSessionStateExtrasE2ETest.java, rust rpc_session_state_extras.rs) and CI workflow shard changes appear routine and consistent. No cross-SDK parity issues found; no review comments needed.
Reviewed PR #2494 (Fix internal client-session handler codegen). The fix is scoped correctly: it applies filterNodeByVisibility to TypeScript, Go, and Python client-session handler codegen to exclude internal RPC methods from generated public handler interfaces — bringing them in line with C# codegen (scripts/codegen/csharp.ts), which already applies this same public-visibility filter at lines 2660-2661 prior to this PR. Java and Rust codegen do not generate client-session handler interfaces at all (no equivalent public API surface exists for internal client-session methods), so no changes are needed there for parity. New regression test (nodejs/test/client-api-codegen.test.ts) exercises TS/Go/Python emitters with a mixed public/internal schema and correctly asserts internal methods/types are excluded — good cross-language coverage for the three SDKs actually affected by the bug. No consistency issues found; this PR maintains parity across affected SDKs.
Reviewed PR #2468: it only adds test coverage across all six SDKs (.NET, Go, Java, Node.js, Python, Rust) verifying that the existing generated messageId field on QueuePendingItems/UserMessageData serializes as optional camelCase JSON. No source or generated code is modified, and equivalent tests were added symmetrically to every language. This confirms existing cross-SDK consistency rather than introducing any new or divergent behavior, so no consistency issues were found and no review comments are needed.
Reviewed PR #2463 (CLI distribution/versioning refactor: npm-loader → GitHub-releases-based bundling for Node.js, .NET, Python, Go, Rust, plus Java/Go test-harness unification, and an unrelated Go session event-channel race fix). Changes are consistently applied across all six SDKs: the adjacent runtime.node resolution fallback already exists in Go/Java/Rust and was added to Node.js, .NET, and Python for parity. CLI version resolution moved from package-lock.json to package.json (with legacy fallback) consistently in Go bundler and Rust build script. Test harnesses (Go, Java) now share a common pinned-CLI preparation flow matching Node.js. No cross-SDK feature-parity gaps or API naming inconsistencies found — this is internal build/packaging tooling, not public API surface, so no action needed.
Activity
github-actions commentedon Jul 14, 2026
SDK Consistency Review Agent
No consistency issues found: PR #1988 is an automated code-generation update (
@github/copilot→ 1.0.71-1) that regenerates all SDK generated files from the same shared schema. All 6 SDKs (Node.js, Python, Go, .NET, Java, Rust) are updated consistently with the same new event (session.memory_changed), new fields (mcpMeta,workingDirectory), and un-deprecations. No cross-SDK parity gaps detected.github-actions commentedon Jul 28, 2026
SDK Consistency Review Agent
PR #2107 only modifies .NET E2E test code (AskUserE2ETests.cs), switching SendAsync+GetFinalAssistantMessageAsync to SendAndWaitAsync for flake-fix alignment with Node/Python equivalents. No public API surface was changed, no new SDK features were introduced, and no cross-SDK consistency issues were found.
github-actions commentedon Jul 28, 2026
SDK Consistency Review Agent
PR #2109 modifies only one file (dotnet/test/E2E/SessionE2ETests.cs) — a test-only fix replacing SendAsync+GetFinalAssistantMessageAsync with SendAndWaitAsync to eliminate a race condition. No public API changes were made. No cross-SDK consistency issues to report.
github-actions commentedon Jul 29, 2026
SDK Consistency Review Agent
PR #2120 is a Dependabot bump of GitHub Actions workflow dependencies (actions/checkout, node feature version, etc.). No SDK client code (nodejs/, python/, go/, dotnet/, java/, rust/) was modified. No cross-SDK consistency issues to report.
github-actions commentedon Jul 30, 2026
SDK Consistency Review Agent
PR #1986 modifies only .NET test infrastructure files (dotnet/test/). No public SDK API surface is changed in any language. The changes fix in-process E2E transport coverage and test session lifecycle — these are .NET-specific concerns (FFI/in-process transport) with no equivalent in other SDK languages. No cross-SDK consistency issues found.
github-actions commentedon Jul 30, 2026
SDK Consistency Review Agent
PR #2017 is a Dependabot dependency bump for jackson-databind 2.22.0 → 2.22.1 in java/pom.xml only. This is a Java-internal dependency version update with no API surface changes. No cross-SDK consistency issues to report.
github-actions commentedon Jul 30, 2026
SDK Consistency Review Agent
SDK consistency review complete for PR #2170. This PR is a Java-only internal build restructuring (single-module → multi-module Maven reactor, moving sdk code into java/sdk/ subdirectory). No new API surface or behavior is introduced. No changes touch Node.js, Python, Go, .NET, or Rust SDKs. No cross-SDK consistency issues found — no comment needed.
github-actions commentedon Jul 30, 2026
SDK Consistency Review Agent
PR #2175 reviewed for cross-SDK consistency. All 4 changed files are Java-only (
java/sdk/pom.xml,NativeRuntimeLoader.java,copilot-runtime.properties,NativeRuntimeLoaderTest.java). This PR implements Java-specific FFI infrastructure for extracting and caching a nativeruntime.nodebinary from the JAR classpath — a concern unique to the JVM deployment model. No other SDK (Node.js, Python, Go, .NET, Rust) uses JNA or classpath-based native binary extraction. No cross-SDK consistency issues identified.github-actions commentedon Jul 31, 2026
SDK Consistency Review Agent
Cross-SDK consistency review complete for PR #2181. All 6 SDKs (Node.js, Python, Go, .NET, Java, Rust) are updated consistently: model references changed from
gpt-4.1togpt-5.4and tests for reasoning-effort scenarios now use isolated client contexts. Shared YAML snapshots are updated to match. No consistency gaps found.github-actions commentedon Aug 2, 2026
SDK Consistency Review Agent
PR #2193 is an automated codegen update regenerating schema-derived types across all six SDK implementations (Node.js, Python, Go, .NET, Java, Rust) simultaneously. All changed files are auto-generated from the same
@github/copilotschema (1.0.78-2). No cross-SDK consistency issues found — the PR maintains parity by design.github-actions commentedon Aug 3, 2026
SDK Consistency Review Agent
PR #2229 adds only a CI validation fixture file and a temporary test file (nodejs/test/ci-validation.test.ts). No SDK client code was modified. No cross-SDK consistency review needed.
github-actions commentedon Aug 3, 2026
SDK Consistency Review Agent
PR #2229 ("Test CI behavior") contains no changes to any SDK implementation (Node.js, Python, Go, .NET, Java, or Rust). All 30 changed files are workflow/infrastructure files. No cross-SDK consistency review needed.
github-actions commentedon Aug 3, 2026
SDK Consistency Review Agent
PR #2236 adds Java-only Maven build infrastructure (
copilot-nativemodule) to package the nativeruntime.nodebinary as a classifier JAR. No public SDK API was added or modified. No cross-SDK consistency review needed.github-actions commentedon Aug 4, 2026
SDK Consistency Review Agent
PR #2243 is a Dependabot security update bumping the Node.js devDependency
postcssfrom 8.5.15 to 8.5.25 (and its transitive depnanoid3.3.12 → 3.3.17). Onlynodejs/package-lock.jsonis changed — no SDK public API or cross-language consistency concerns. No review action needed.35 remaining items
github-actions commentedon Aug 30, 2026
SDK Consistency Review Agent
Cross-SDK consistency review complete for PR #2442 ("Update
@github/copilotto 1.0.82"). This PR is an automated schema codegen update that consistently updates all six SDK implementations — Node.js/TypeScript, Python, Go, .NET, Java, and Rust — with the same new types and fields (AutoTier, SandboxEnforcementStatus, AssistantMessageToolRequestCaller, displayText on EnqueueCommandParams, etc.). All generated files across all language SDKs are in sync. No consistency issues found.github-actions commentedon Sep 1, 2026
SDK Consistency Review Agent
Cross-SDK consistency review complete for PR #2372 ("Expose sandbox bypass across SDKs"). The PR adds
SandboxConfig.allowBypassto all six SDK implementations (Node.js:allowBypass?: boolean, Python:allow_bypass: bool | None, Go:AllowBypass *bool, .NET:bool? AllowBypass, Rust:pub allow_bypass: Option<bool>, Java:@JsonProperty("allowBypass") Boolean allowBypass) with consistent JSON key"allowBypass"and optional/nullable semantics everywhere. Tests and serialization coverage are added for each language. No cross-SDK consistency issues found.github-actions commentedon Sep 1, 2026
SDK Consistency Review Agent
Cross-SDK consistency review complete for PR #2437 ("Add Auto routing tier options across all SDKs"). The PR adds
capi.autoTiersupport consistently across all six SDKs:autoTier?: AutoTierinCapiSessionOptions(camelCase ✓)auto_tier: AutoTierinCapiSessionOptions, mapped to wireautoTier(snake_case ✓)AutoTier AutoTierinCapiSessionOptionswithjson:"autoTier,omitempty"(PascalCase exported ✓)AutoTier? AutoTierinCapiSessionOptionswith[JsonPropertyName("autoTier")](PascalCase ✓)setAutoTier(AutoTier)fluent setter onCapiSessionOptions(camelCase method ✓)auto_tier: Option<AutoTier>withwith_auto_tier()builder (snake_case ✓)All three tier values (
efficiency,balance,intelligence) are exposed in all SDKs. Wire format is consistent (autoTier). Session events (session.start,session.resume) exposeautoTier/data.auto_tieruniformly. No consistency issues found.github-actions commentedon Sep 2, 2026
SDK Consistency Review Agent
Reviewed PR #2480: all changes are CI/workflow infrastructure (.github/workflows/*.yml for dotnet, go, java, nodejs, python, rust SDK test workflows, required-checks.yml, and the sdk-consistency-review workflow itself), plus a minor Go e2e test file tweak and a Java pom.xml dependency version bump. No SDK client/API code was modified in any language implementation, so there are no cross-language feature parity or API consistency concerns to flag.
github-actions commentedon Sep 2, 2026
SDK Consistency Review Agent
Reviewed PR #2480: changes are limited to CI workflow configuration (.github/workflows/*.yml for dotnet, go, java, nodejs, python, rust SDK test pipelines, plus the consistency-review workflow itself), a minor Go E2E test file (go/internal/e2e/rpc_server_e2e_test.go), and a Java pom.xml dependency version bump. No SDK client/public API code was added or modified in any language, so there is no cross-SDK feature-parity or API-design consistency concern to raise for this PR. No review comments needed.
github-actions commentedon Sep 2, 2026
SDK Consistency Review Agent
Reviewed PR #2482: renames ClientInfo fields (editorName/editorVersion/extensionName/extensionVersion -> applicationName/applicationVersion/integrationName/integrationVersion) across all six SDKs (Node.js, Python, Go, .NET, Java, Rust). Verified the rename is applied consistently: same wire-format mapping preserved (still editorName/editorVersion/extensionName/extensionVersion on the wire), naming follows each language's idioms (camelCase, snake_case, PascalCase as appropriate), docs (docs/features/client-info.md, CHANGELOG.md) and tests updated in lockstep across all languages. No cross-SDK inconsistencies found - this PR maintains full feature parity.
github-actions commentedon Sep 2, 2026
SDK Consistency Review Agent
PR #2482 renames client-info fields (editorName/editorVersion/extensionName/extensionVersion -> applicationName/applicationVersion/integrationName/integrationVersion) consistently across all six SDKs (Node.js, Python, Go, .NET, Java, Rust), plus docs and CHANGELOG. Verified each SDK follows its own naming convention (camelCase, snake_case, PascalCase as appropriate) and the wire format (editorName/editorVersion/extensionName/extensionVersion) is preserved unchanged for backward compatibility with the runtime handshake. Tests were updated in parallel in all six SDKs. No cross-SDK consistency issues found - this is a clean, coordinated rename.
github-actions commentedon Sep 2, 2026
SDK Consistency Review Agent
Reviewed PR #2483: it only regenerates auto-generated RPC/session-event bindings (dotnet/src/Generated, go/rpc, java/sdk/src/generated/java) to add Java bindings for types (DiscoveredHook, HookOrigin, AgentModelPolicy, CompletionReceipt*, SessionCompletionReceiptEvent, FusionPhaseActivity*) that already exist in Node.js, Python, Go, and Rust generated code. No hand-written SDK client logic was touched. This is schema/codegen sync work, not a new feature — all six SDKs already have or now have equivalent generated types with language-appropriate naming (PascalCase Java records/enums matching TS interfaces, Python dataclasses, Go structs, Rust structs). No cross-SDK consistency issues found.
github-actions commentedon Sep 2, 2026
SDK Consistency Review Agent
PR #2487 only bumps a dev dependency (js-yaml) in nodejs/package-lock.json. No SDK client code or public API surface is modified, so there are no cross-language consistency implications to review.
github-actions commentedon Sep 2, 2026
SDK Consistency Review Agent
Reviewed PR #2483: changes are limited to auto-generated code (dotnet/src/Generated, go/rpc, java/sdk/src/generated) syncing new schema types (AgentModelPolicy, CompletionReceipt*, FusionPhaseActivity*, DiscoveredHook, HookOrigin, etc.) plus java/pom.xml, java codegen package.json/lock, and an unrelated CI workflow shard split for dotnet-sdk-tests.yml. Verified these types already exist in the Node.js, Python, and Rust generated files, so this PR is bringing .NET/Go/Java up to parity with the shared schema rather than introducing an inconsistency. No hand-written public API surface was changed. No cross-SDK consistency issues found — no review comments needed.
github-actions commentedon Sep 2, 2026
SDK Consistency Review Agent
Reviewed PR #2483: it only touches auto-generated SDK binding files (dotnet/src/Generated, go/rpc/z*.go, go/zsession_events.go, java/sdk/src/generated/java/**, plus the java codegen package-lock/pom and a CI workflow shard update) — regenerated from an updated api/session-events schema (new types: AgentModelPolicy, DiscoveredHook/HookOrigin, FusionPhaseActivityKind/FusionPhasePlanStep, CompletionReceipt* types, assistant.fusion_phase_activity and session.completion_receipt events). Cross-checked Node.js (rpc.ts/session-events.ts), Python (rpc.py/session_events.py), Go (zrpc.go/zsession_events.go), and Rust (generated/api_types.rs, generated/session_events.rs) generated files: all already contain the equivalent types/events with language-appropriate naming (e.g. Node's CompletionReceiptEvent == Java's SessionCompletionReceiptEvent, both for session.completion_receipt). No cross-SDK feature parity gaps found — this PR is bringing .NET/Go/Java generated bindings up to date with a schema that Node/Python/Rust already reflect. No public API design inconsistencies to flag. No review comments needed.
github-actions commentedon Sep 2, 2026
SDK Consistency Review Agent
Reviewed PR #2483: this is a schema-driven codegen regeneration touching all six SDKs (.NET, Go, Java, Node.js, Python, Rust). New RPC/session-event types (HooksDiscoverParams/Result, ServerHooksApi, AgentModelPolicy, CompletionReceipt* types, SessionCompletionReceiptEvent, SessionsReadPersistedEventsParams/Result) are present consistently across all generated files in every language, with parallel naming conventions per language idiom (PascalCase in .NET/Go/Java types, camelCase methods in Java/Node, snake_case in Python/Rust). The only non-generated code change is in scripts/codegen/rust.ts, which sanitizes
[::]sequences in doc comments to avoid broken Rust intra-doc links — this is Rust-toolchain-specific (rustdoc doesn't parse[::]as plain text) and doesn't need equivalent changes in other languages. Accompanying test file updates (dotnet/test/E2E/RpcMcpAndSkillsE2ETests.cs, java RpcSessionStateExtrasE2ETest.java, rust rpc_session_state_extras.rs) and CI workflow shard changes appear routine and consistent. No cross-SDK parity issues found; no review comments needed.github-actions commentedon Sep 3, 2026
SDK Consistency Review Agent
Reviewed PR #2494 (Fix internal client-session handler codegen). The fix is scoped correctly: it applies filterNodeByVisibility to TypeScript, Go, and Python client-session handler codegen to exclude internal RPC methods from generated public handler interfaces — bringing them in line with C# codegen (scripts/codegen/csharp.ts), which already applies this same public-visibility filter at lines 2660-2661 prior to this PR. Java and Rust codegen do not generate client-session handler interfaces at all (no equivalent public API surface exists for internal client-session methods), so no changes are needed there for parity. New regression test (nodejs/test/client-api-codegen.test.ts) exercises TS/Go/Python emitters with a mixed public/internal schema and correctly asserts internal methods/types are excluded — good cross-language coverage for the three SDKs actually affected by the bug. No consistency issues found; this PR maintains parity across affected SDKs.
github-actions commentedon Sep 3, 2026
SDK Consistency Review Agent
Reviewed PR #2468: it only adds test coverage across all six SDKs (.NET, Go, Java, Node.js, Python, Rust) verifying that the existing generated
messageIdfield on QueuePendingItems/UserMessageData serializes as optional camelCase JSON. No source or generated code is modified, and equivalent tests were added symmetrically to every language. This confirms existing cross-SDK consistency rather than introducing any new or divergent behavior, so no consistency issues were found and no review comments are needed.github-actions commentedon Sep 3, 2026
SDK Consistency Review Agent
Reviewed PR #2463 (CLI distribution/versioning refactor: npm-loader → GitHub-releases-based bundling for Node.js, .NET, Python, Go, Rust, plus Java/Go test-harness unification, and an unrelated Go session event-channel race fix). Changes are consistently applied across all six SDKs: the adjacent runtime.node resolution fallback already exists in Go/Java/Rust and was added to Node.js, .NET, and Python for parity. CLI version resolution moved from package-lock.json to package.json (with legacy fallback) consistently in Go bundler and Rust build script. Test harnesses (Go, Java) now share a common pinned-CLI preparation flow matching Node.js. No cross-SDK feature-parity gaps or API naming inconsistencies found — this is internal build/packaging tooling, not public API surface, so no action needed.