Skip to content

Add classical code generation workflow for Java SDK and migrate to generated event types #70

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
Copilot wants to merge 17 commits into from
Closed

Add classical code generation workflow for Java SDK and migrate to generated event types #70

Changes from 1 commit
Commits
Show all changes
17 commits
Select commit Hold shift + click to select a range
6561490
Initial plan
Copilot Apr 12, 2026
3ef1b22
Add Java code generation from JSON schemas
Copilot Apr 12, 2026
6e3c50c
Add comment explaining npm install before npm ci in update workflow
Copilot Apr 13, 2026
75e5dba
Improve Java code generator: sealed hierarchy, records, Long for inte…
Copilot Apr 13, 2026
e9c0bf7
Add @JsonInclude(NON_NULL) to all generated event classes and warn on…
edburns Apr 14, 2026
da9ec9c
Migrate events package to auto-generated SessionEvent types
Copilot Apr 14, 2026
7e7b283
Fix residual old-package references and rename SessionEventParserTest
edburns Apr 15, 2026
6d35942
Generate typed RPC wrapper classes (ServerRpc, SessionRpc) from api.s…
Copilot Apr 15, 2026
e4dc6e5
Fix race condition in SessionEventsE2ETest: wait for own listener to …
Copilot Apr 16, 2026
a2712a7
Chunk 3: Wire generated RPC wrappers into CopilotClient and CopilotSe…
Copilot Apr 16, 2026
f21a5ae
Merge remote-tracking branch 'upstream/main' into copilot/copilot-pr-70
edburns Apr 16, 2026
cc3e09a
Chunk 3 improvements: lazy getRpc() init, full internal RPC migration…
Copilot Apr 17, 2026
5ba95f6
Fix Windows CLI path resolution and extract shared TestUtil
edburns Apr 17, 2026
265eec4
Apply Spotless formatting fixes to TestUtil.java
edburns Apr 17, 2026
cf7d988
Merge branch 'main' into copilot/copilot-pr-70
edburns Apr 17, 2026
ed1b1e3
Forbid agentic sync from modifying src/generated/java/ files
Copilot Apr 17, 2026
9ec2478
Remove duplicate Javadoc block on sendExpandedToolResult()
edburns Apr 17, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
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>
  • Loading branch information
@edburns
Copilot and edburns authored Apr 13, 2026
commit 6e3c50cb29ca56c1d0d8b6473db1aa537d723ac4
5 changes: 5 additions & 0 deletions .github/workflows/update-copilot-dependency.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -36,6 +36,11 @@ jobs:
env:
VERSION: ${{ inputs.version }}
working-directory: ./scripts/codegen
Copy link
Copy Markdown
Collaborator

@edburns edburns Apr 13, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@copilot add a comment explaining the necessity of the unusual step of running npm install before npm ci

Copy link
Copy Markdown
Contributor Author

Copilot AI Apr 13, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added a comment in 6e3c50c explaining the two-step approach: npm install updates package.json and package-lock.json to pin the new version, then npm ci does a clean reproducible install from the updated lock file. Running npm ci alone would fail because it cannot change a pinned version, and npm install alone leaves behind stale packages.

# npm install updates package.json and package-lock.json to the new
# version; npm ci (below) then does a clean, reproducible install from
# the updated lock file. Both steps are required: npm install alone
# leaves leftover packages, while npm ci alone cannot change the pinned
# version in the lock file.
run: npm install "@github/copilot@$VERSION"

- name: Install codegen dependencies
Expand Down
Loading