-
Notifications
You must be signed in to change notification settings - Fork 4
Comparing changes
Open a pull request
base repository: github/copilot-sdk-java
base: d892bdb
head repository: github/copilot-sdk-java
compare: 9ec2478
- 17 commits
- 329 files changed
- 3 contributors
Commits on Apr 12, 2026
-
Configuration menu - View commit details
-
Copy full SHA for 6561490 - Browse repository at this point
Copy the full SHA 6561490View commit details -
Add Java code generation from JSON schemas
- Add scripts/codegen/java.ts - Java code generator from session-events and api schemas - Add scripts/codegen/package.json with @github/copilot dependency - Add scripts/codegen/package-lock.json - Generate src/generated/java/com/github/copilot/sdk/generated/ - 76 session event files - Generate src/generated/java/com/github/copilot/sdk/generated/rpc/ - 111 RPC type files - Update pom.xml: add build-helper-maven-plugin, exclude generated from Spotless - Update config/checkstyle/checkstyle.xml: exclude generated packages - Update .gitattributes: mark src/generated/java/** as generated - Add .github/workflows/codegen-check.yml - CI check workflow - Add .github/workflows/update-copilot-dependency.yml - update workflow - Update docs/WORKFLOWS.md with new workflow documentation - Update .gitignore to exclude scripts/codegen/node_modules/" Agent-Logs-Url: https://github.com/github/copilot-sdk-java/sessions/aee4deda-40d4-4ecd-a831-2af9cb9461e7 Co-authored-by: edburns <75821+edburns@users.noreply.github.com>
Configuration menu - View commit details
-
Copy full SHA for 3ef1b22 - Browse repository at this point
Copy the full SHA 3ef1b22View commit details
Commits on Apr 13, 2026
-
Add comment explaining npm install before npm ci in update workflow
Agent-Logs-Url: https://github.com/github/copilot-sdk-java/sessions/153bf73a-7c73-4f85-99be-ea7df4e5a2b6 Co-authored-by: edburns <75821+edburns@users.noreply.github.com>
Configuration menu - View commit details
-
Copy full SHA for 6e3c50c - Browse repository at this point
Copy the full SHA 6e3c50cView commit details -
Improve Java code generator: sealed hierarchy, records, Long for inte…
…ger, @JsonCreator Agent-Logs-Url: https://github.com/github/copilot-sdk-java/sessions/c76bbc9a-e73e-40cc-8dd4-7ecd223c16cd Co-authored-by: edburns <75821+edburns@users.noreply.github.com>
Configuration menu - View commit details
-
Copy full SHA for 75e5dba - Browse repository at this point
Copy the full SHA 75e5dbaView commit details
Commits on Apr 14, 2026
-
Add @JsonInclude(NON_NULL) to all generated event classes and warn on…
… Object fallbacks The code generator (scripts/codegen/java.ts) had two gaps identified during report-card grading against the Epic #69 rubric: 1. Silent Object fallbacks: When schemaTypeToJava() fell back to List<Object> (untyped array) or Map<String, Object> (untyped object without properties or additionalProperties), it did so silently. Added console.warn() at both sites so type erasure is visible during generation, matching the existing warnings for anyOf and unrecognized schema patterns. 2. Missing @JsonInclude(NON_NULL) on outer event classes: The annotation was present on inner data records and RPC records but missing from the SessionEvent sealed base class and all outer event variant classes (e.g., AssistantMessageEvent, ToolExecutionStartEvent). Added the import and annotation to generateSessionEventBaseClass() and generateEventVariantClass(). After regenerating, all 75 session event files and 111 RPC type files were updated. mvn verify passes with 556 tests, 0 failures, 0 errors.
Configuration menu - View commit details
-
Copy full SHA for e9c0bf7 - Browse repository at this point
Copy the full SHA e9c0bf7View commit details -
Migrate events package to auto-generated SessionEvent types
Replace hand-written com.github.copilot.sdk.events package with auto-generated com.github.copilot.sdk.generated types. - Replace AbstractSessionEvent with SessionEvent from generated package - Replace SessionEventParser with Jackson native polymorphic deserialization - Update all main source and test files to use generated types - Adapt ForwardCompatibilityTest to remove getOriginalType() tests - Update type assertions for renamed data records (FooData -> FooEventData) - Fix enum type comparisons in tests (elicitation mode, shutdown type, etc.) - Update site documentation to reference generated package - Delete old events package Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Co-authored-by: edburns <75821+edburns@users.noreply.github.com>
Configuration menu - View commit details
-
Copy full SHA for da9ec9c - Browse repository at this point
Copy the full SHA da9ec9cView commit details
Commits on Apr 15, 2026
-
Fix residual old-package references and rename SessionEventParserTest
The migration commit (da9ec9c) replaced com.github.copilot.sdk.events with com.github.copilot.sdk.generated across all src/ files but missed four non-source files that still referenced the old package: 1. jbang-example.java — updated imports from events.AssistantMessageEvent and events.SessionUsageInfoEvent to generated equivalents. 2. README.md — updated Quick Start imports to use generated package. 3. .github/copilot-instructions.md — updated Package Structure section to describe com.github.copilot.sdk.generated instead of the deleted com.github.copilot.sdk.events package. 4. config/spotbugs/spotbugs-exclude.xml — changed SpotBugs exclusion from com.github.copilot.sdk.events to com.github.copilot.sdk.generated, and updated the comment to match. Also renamed SessionEventParserTest.java to SessionEventDeserializationTest.java (class name updated to match) since the test no longer exercises the deleted SessionEventParser class — it tests Jackson-native deserialization via ObjectMapper.readValue(json, SessionEvent.class). Build verified: 553 tests, 0 failures, BUILD SUCCESS. were updated. mvn verify passes with 556 tests, 0 failures, 0 errors.
Configuration menu - View commit details
-
Copy full SHA for 7e7b283 - Browse repository at this point
Copy the full SHA 7e7b283View commit details -
Generate typed RPC wrapper classes (ServerRpc, SessionRpc) from api.s…
…chema.json Agent-Logs-Url: https://github.com/github/copilot-sdk-java/sessions/73b13ddd-f0ef-46e7-b0a2-ba8cf0926b5c Co-authored-by: edburns <75821+edburns@users.noreply.github.com>
Configuration menu - View commit details
-
Copy full SHA for 6d35942 - Browse repository at this point
Copy the full SHA 6d35942View commit details
Commits on Apr 16, 2026
-
Fix race condition in SessionEventsE2ETest: wait for own listener to …
…see session.idle Agent-Logs-Url: https://github.com/github/copilot-sdk-java/sessions/96b1590a-aa30-40ae-83ab-416a58e05e83 Co-authored-by: edburns <75821+edburns@users.noreply.github.com>
Configuration menu - View commit details
-
Copy full SHA for e4dc6e5 - Browse repository at this point
Copy the full SHA e4dc6e5View commit details -
Chunk 3: Wire generated RPC wrappers into CopilotClient and CopilotSe…
…ssion Agent-Logs-Url: https://github.com/github/copilot-sdk-java/sessions/77121eb8-79b7-46ee-8734-81054d4e19bf Co-authored-by: edburns <75821+edburns@users.noreply.github.com>
Configuration menu - View commit details
-
Copy full SHA for a2712a7 - Browse repository at this point
Copy the full SHA a2712a7View commit details -
Merge remote-tracking branch 'upstream/main' into copilot/copilot-pr-70
# Conflicts: # .gitignore
Configuration menu - View commit details
-
Copy full SHA for f21a5ae - Browse repository at this point
Copy the full SHA f21a5aeView commit details
Commits on Apr 17, 2026
-
Chunk 3 improvements: lazy getRpc() init, full internal RPC migration…
…, add tests Agent-Logs-Url: https://github.com/github/copilot-sdk-java/sessions/bb264a9e-45c2-4c7e-898d-79b38d3c178d Co-authored-by: edburns <75821+edburns@users.noreply.github.com>
Configuration menu - View commit details
-
Copy full SHA for cc3e09a - Browse repository at this point
Copy the full SHA cc3e09aView commit details -
Fix Windows CLI path resolution and extract shared TestUtil
modified: src/test/java/com/github/copilot/sdk/CopilotClientTest.java - Removed duplicate `getCliPath()` and `findCopilotInPath()` methods; delegates to `TestUtil.findCliPath()` - Removed unused imports (`BufferedReader`, `InputStreamReader`, `Path`, `Paths`) - Replaced 5 skip-guard blocks (`if (cliPath == null) return`) with `assertNotNull(cliPath, ...)` so tests fail instead of silently skipping modified: src/test/java/com/github/copilot/sdk/MetadataApiTest.java - Removed duplicate `getCliPath()` and `findCopilotInPath()` methods; delegates to `TestUtil.findCliPath()` - Removed unused imports (`BufferedReader`, `InputStreamReader`, `Path`, `Paths`) - Replaced 3 skip-guard blocks with `assertNotNull(cliPath, ...)` so tests fail instead of silently skipping new file: src/test/java/com/github/copilot/sdk/TestUtil.java - New package-private utility class with shared `findCliPath()` and `findCopilotInPath()` - `findCopilotInPath()` iterates all `where.exe` results on Windows and tries launching each candidate, fixing CreateProcess error 193 caused by a Linux ELF binary appearing first in PATH - Resolution order: PATH search, then COPILOT_CLI_PATH env var, then parent-directory walk for nodejs module Signed-off-by: Ed Burns <edburns@microsoft.com>
Configuration menu - View commit details
-
Copy full SHA for 5ba95f6 - Browse repository at this point
Copy the full SHA 5ba95f6View commit details -
Apply Spotless formatting fixes to TestUtil.java
modified: src/test/java/com/github/copilot/sdk/TestUtil.java - Reflow Javadoc line wrapping to comply with Eclipse formatter rules - Collapse multi-line ProcessBuilder chain to single line per Spotless Signed-off-by: Ed Burns <edburns@microsoft.com>
Configuration menu - View commit details
-
Copy full SHA for 265eec4 - Browse repository at this point
Copy the full SHA 265eec4View commit details -
Configuration menu - View commit details
-
Copy full SHA for cf7d988 - Browse repository at this point
Copy the full SHA cf7d988View commit details -
Forbid agentic sync from modifying src/generated/java/ files
Agent-Logs-Url: https://github.com/github/copilot-sdk-java/sessions/38fdf90e-c98a-4df6-8954-3c965bdfb1f8 Co-authored-by: edburns <75821+edburns@users.noreply.github.com>
Configuration menu - View commit details
-
Copy full SHA for ed1b1e3 - Browse repository at this point
Copy the full SHA ed1b1e3View commit details -
Remove duplicate Javadoc block on sendExpandedToolResult()
modified: src/main/java/com/github/copilot/sdk/CopilotSession.java - Remove the first of two identical Javadoc comment blocks that appeared in sequence above `sendExpandedToolResult()`. The method retains its single, correct Javadoc block describing the `ObjectNode` bypass for `session.tools.handlePendingToolCall`. Signed-off-by: Ed Burns <edburns@microsoft.com>
Configuration menu - View commit details
-
Copy full SHA for 9ec2478 - Browse repository at this point
Copy the full SHA 9ec2478View commit details
This comparison is taking too long to generate.
Unfortunately it looks like we can’t render this comparison for you right now. It might be too big, or there might be something weird with your repository.
You can try running this command locally to see the comparison on your machine:
git diff d892bdb...9ec2478