Fix additional codegen-check problem

Author: edburns

.github/prompts/agentic-merge-reference-impl.prompt.md

@@ -135,11 +135,11 @@ For each change in the reference implementation diff, determine:
 
 | reference implementation (.NET)                    | Java SDK Equivalent                                    |
 |------------------------------------|--------------------------------------------------------|
-| `dotnet/src/Client.cs`             | `src/main/java/com/github/copilot/sdk/CopilotClient.java` |
-| `dotnet/src/Session.cs`            | `src/main/java/com/github/copilot/sdk/CopilotSession.java` |
-| `dotnet/src/Types.cs`              | `src/main/java/com/github/copilot/sdk/types/*.java`    |
+| `dotnet/src/Client.cs`             | `src/main/java/com/github/copilot/CopilotClient.java` |
+| `dotnet/src/Session.cs`            | `src/main/java/com/github/copilot/CopilotSession.java` |
+| `dotnet/src/Types.cs`              | `src/main/java/com/github/copilot/types/*.java`    |
 | `dotnet/src/Generated/*.cs`        | ❌ **DO NOT TOUCH** `src/generated/java/**` — see top of this file |
-| `dotnet/test/*.cs`                 | `src/test/java/com/github/copilot/sdk/*Test.java`      |
+| `dotnet/test/*.cs`                 | `src/test/java/com/github/copilot/*Test.java`      |
 | `docs/getting-started.md`          | `README.md` and `src/site/markdown/*.md`               |
 | `docs/*.md` (new files)            | `src/site/markdown/*.md` + update `src/site/site.xml`  |
 | `sdk-protocol-version.json`        | (embedded in Java code or resource file)               |
@@ -230,7 +230,7 @@ git diff "$LAST_REFERENCE_IMPL_COMMIT"..origin/main --stat -- test/snapshots/
 
 For each new or modified test file in `dotnet/test/`:
 
-1. **Create corresponding Java test class** in `src/test/java/com/github/copilot/sdk/`
+1. **Create corresponding Java test class** in `src/test/java/com/github/copilot/`
 2. **Follow existing test patterns** - Look at existing tests like `PermissionsTest.java` for structure
 3. **Use the E2ETestContext** infrastructure for tests that need the test harness
 4. **Match snapshot directory names** - Test snapshots in `test/snapshots/` must match the directory name used in `ctx.configureForTest()`
@@ -239,10 +239,10 @@ For each new or modified test file in `dotnet/test/`:
 
 | reference implementation Test (.NET)        | Java SDK Test                                          |
 |-----------------------------|--------------------------------------------------------|
-| `dotnet/test/AskUserTests.cs`  | `src/test/java/com/github/copilot/sdk/AskUserTest.java`  |
-| `dotnet/test/HooksTests.cs`    | `src/test/java/com/github/copilot/sdk/HooksTest.java`    |
-| `dotnet/test/ClientTests.cs`   | `src/test/java/com/github/copilot/sdk/CopilotClientTest.java` |
-| `dotnet/test/*Tests.cs`        | `src/test/java/com/github/copilot/sdk/*Test.java`        |
+| `dotnet/test/AskUserTests.cs`  | `src/test/java/com/github/copilot/AskUserTest.java`  |
+| `dotnet/test/HooksTests.cs`    | `src/test/java/com/github/copilot/HooksTest.java`    |
+| `dotnet/test/ClientTests.cs`   | `src/test/java/com/github/copilot/CopilotClientTest.java` |
+| `dotnet/test/*Tests.cs`        | `src/test/java/com/github/copilot/*Test.java`        |
 
 ### Test Snapshot Compatibility
 
@@ -353,7 +353,7 @@ var session = client.createSession(
 
 Explain the request/response objects and their properties.
 
-See [FeatureHandler](apidocs/com/github/copilot/sdk/json/FeatureHandler.html) Javadoc for more details.
+See [FeatureHandler](apidocs/com/github/copilot/rpc/FeatureHandler.html) Javadoc for more details.
 ```
 
 ### Verify Documentation Consistency

.github/prompts/documentation-coverage.prompt.md

@@ -17,13 +17,13 @@ Extract all public classes, methods, and features from the SDK:
 
 ```bash
 # List all public classes in core package
-grep -l "public class\|public interface\|public enum" src/main/java/com/github/copilot/sdk/*.java
+grep -l "public class\|public interface\|public enum" src/main/java/com/github/copilot/*.java
 
 # List all public classes in json package (DTOs)
-grep -l "public class" src/main/java/com/github/copilot/sdk/json/*.java
+grep -l "public class" src/main/java/com/github/copilot/rpc/*.java
 
 # List all event types
-ls src/main/java/com/github/copilot/sdk/events/
+ls src/main/java/com/github/copilot/events/
 ```
 
 ### Step 2: Inventory Documentation
@@ -45,7 +45,7 @@ For each major feature area, determine if it's documented:
 Examine `CopilotClient.java` for public methods:
 
 ```bash
-grep "public.*(" src/main/java/com/github/copilot/sdk/CopilotClient.java | grep -v "@"
+grep "public.*(" src/main/java/com/github/copilot/CopilotClient.java | grep -v "@"
 ```
 
 | Method | Purpose | Documented In | Status |
@@ -66,7 +66,7 @@ grep "public.*(" src/main/java/com/github/copilot/sdk/CopilotClient.java | grep
 Examine `CopilotSession.java` for public methods:
 
 ```bash
-grep "public.*(" src/main/java/com/github/copilot/sdk/CopilotSession.java | grep -v "@"
+grep "public.*(" src/main/java/com/github/copilot/CopilotSession.java | grep -v "@"
 ```
 
 | Method | Purpose | Documented In | Status |
@@ -83,7 +83,7 @@ grep "public.*(" src/main/java/com/github/copilot/sdk/CopilotSession.java | grep
 Examine `SessionConfig.java` for configurable options:
 
 ```bash
-grep "public.*set\|private.*;" src/main/java/com/github/copilot/sdk/json/SessionConfig.java
+grep "public.*set\|private.*;" src/main/java/com/github/copilot/rpc/SessionConfig.java
 ```
 
 | Option | Purpose | Documented | Example Provided |
@@ -103,7 +103,7 @@ grep "public.*set\|private.*;" src/main/java/com/github/copilot/sdk/json/Session
 Check which events are documented:
 
 ```bash
-grep "TYPE_MAP.put" src/main/java/com/github/copilot/sdk/events/SessionEventParser.java
+grep "TYPE_MAP.put" src/main/java/com/github/copilot/events/SessionEventParser.java
 ```
 
 | Event Type | Event Class | Documented | Example |
@@ -118,7 +118,7 @@ grep "TYPE_MAP.put" src/main/java/com/github/copilot/sdk/events/SessionEventPars
 Check `SessionHooks.java` for hook types:
 
 ```bash
-grep "private.*Handler" src/main/java/com/github/copilot/sdk/json/SessionHooks.java
+grep "private.*Handler" src/main/java/com/github/copilot/rpc/SessionHooks.java
 ```
 
 | Hook | Handler Interface | Documented | Example |
@@ -193,11 +193,11 @@ Topics that warrant dedicated documentation:
 ## Key Files
 
 ### Source Code
-- `src/main/java/com/github/copilot/sdk/CopilotClient.java`
-- `src/main/java/com/github/copilot/sdk/CopilotSession.java`
-- `src/main/java/com/github/copilot/sdk/json/SessionConfig.java`
-- `src/main/java/com/github/copilot/sdk/json/SessionHooks.java`
-- `src/main/java/com/github/copilot/sdk/events/SessionEventParser.java`
+- `src/main/java/com/github/copilot/CopilotClient.java`
+- `src/main/java/com/github/copilot/CopilotSession.java`
+- `src/main/java/com/github/copilot/rpc/SessionConfig.java`
+- `src/main/java/com/github/copilot/rpc/SessionHooks.java`
+- `src/main/java/com/github/copilot/events/SessionEventParser.java`
 
 ### Documentation
 - `src/site/markdown/index.md` - Landing page

.github/prompts/test-coverage-assessment.prompt.md

@@ -17,10 +17,10 @@ First, examine the source code to identify all components that should be tested:
 
 ```bash
 # List all event classes
-ls src/main/java/com/github/copilot/sdk/events/
+ls src/main/java/com/github/copilot/events/
 
 # Check the event type mapping in SessionEventParser
-grep -n "TYPE_MAP.put" src/main/java/com/github/copilot/sdk/events/SessionEventParser.java
+grep -n "TYPE_MAP.put" src/main/java/com/github/copilot/events/SessionEventParser.java
 ```
 
 Extract the list of all registered event types from `SessionEventParser.java`.
@@ -30,7 +30,7 @@ Extract the list of all registered event types from `SessionEventParser.java`.
 Check `SessionHooks.java` for all available hook handlers:
 
 ```bash
-grep -E "private.*Handler" src/main/java/com/github/copilot/sdk/json/SessionHooks.java
+grep -E "private.*Handler" src/main/java/com/github/copilot/rpc/SessionHooks.java
 ```
 
 ### Step 3: Analyze Existing Tests
@@ -39,13 +39,13 @@ Examine the test files to understand current coverage:
 
 ```bash
 # List all test files
-ls src/test/java/com/github/copilot/sdk/
+ls src/test/java/com/github/copilot/
 
 # Check for event-related tests
-grep -r "import.*events\." src/test/java/com/github/copilot/sdk/ | grep -v "\.class"
+grep -r "import.*events\." src/test/java/com/github/copilot/ | grep -v "\.class"
 
 # Check for hook tests
-grep -l "SessionHooks\|Hook" src/test/java/com/github/copilot/sdk/*.java
+grep -l "SessionHooks\|Hook" src/test/java/com/github/copilot/*.java
 ```
 
 ### Step 4: Categorize Test Coverage
@@ -95,13 +95,13 @@ Prioritized list of tests to add:
 
 ## Key Files to Examine
 
-- `src/main/java/com/github/copilot/sdk/events/SessionEventParser.java` - Event type registry
-- `src/main/java/com/github/copilot/sdk/json/SessionHooks.java` - Hook definitions
-- `src/main/java/com/github/copilot/sdk/CopilotSession.java` - Hook handling logic
-- `src/test/java/com/github/copilot/sdk/SessionEventParserTest.java` - Event parsing tests
-- `src/test/java/com/github/copilot/sdk/SessionEventsE2ETest.java` - Event E2E tests
-- `src/test/java/com/github/copilot/sdk/HooksTest.java` - Hook tests
-- `src/test/java/com/github/copilot/sdk/SessionEventHandlingTest.java` - Event handling tests
+- `src/main/java/com/github/copilot/events/SessionEventParser.java` - Event type registry
+- `src/main/java/com/github/copilot/rpc/SessionHooks.java` - Hook definitions
+- `src/main/java/com/github/copilot/CopilotSession.java` - Hook handling logic
+- `src/test/java/com/github/copilot/SessionEventParserTest.java` - Event parsing tests
+- `src/test/java/com/github/copilot/SessionEventsE2ETest.java` - Event E2E tests
+- `src/test/java/com/github/copilot/HooksTest.java` - Hook tests
+- `src/test/java/com/github/copilot/SessionEventHandlingTest.java` - Event handling tests
 
 ## Verification
 

.github/workflows/codegen-agentic-fix.md

@@ -177,7 +177,7 @@ For each attempt:
 2. **Read the generated types** to understand what changed. Check the generated files that the handwritten code references:
    ```bash
    # Example: check what a generated type looks like now
-   cat src/generated/java/com/github/copilot/sdk/generated/rpc/<TypeName>.java
+   cat src/generated/java/com/github/copilot/generated/rpc/<TypeName>.java
    ```
 
 3. **Fix the affected source files.** You may modify files under:

.github/workflows/update-copilot-dependency.yml

@@ -61,7 +61,7 @@ jobs:
           cat > /tmp/verify-codegen-prompt.txt << 'PROMPT_EOF'
           You are running inside the copilot-sdk-java repository.
           The code generator has just run and produced Java source files under
-          src/generated/java/com/github/copilot/sdk/generated/rpc/.
+          src/generated/java/com/github/copilot/generated/rpc/.
 
           Your task is to spot-check the generated API classes against the source
           JSON schema to verify the generator produced correct output.
@@ -74,9 +74,9 @@ jobs:
              scripts/codegen/node_modules/@github/copilot/schemas/api.schema.json
 
           2. Pick these 3 generated API classes to verify:
-             - src/generated/java/com/github/copilot/sdk/generated/rpc/SessionToolsApi.java
-             - src/generated/java/com/github/copilot/sdk/generated/rpc/SessionUiApi.java
-             - src/generated/java/com/github/copilot/sdk/generated/rpc/SessionPermissionsApi.java
+             - src/generated/java/com/github/copilot/generated/rpc/SessionToolsApi.java
+             - src/generated/java/com/github/copilot/generated/rpc/SessionUiApi.java
+             - src/generated/java/com/github/copilot/generated/rpc/SessionPermissionsApi.java
 
           3. For each class, verify:
              - Every RPC method in the schema's corresponding namespace has a

scripts/codegen/java.ts

@@ -359,7 +359,7 @@ async function generateSessionEvents(schemaPath: string): Promise<void> {
 
     const variants = extractEventVariants(schema);
     const packageName = "com.github.copilot.generated";
-    const packageDir = `src/generated/java/com/github/copilot/sdk/generated`;
+    const packageDir = `src/generated/java/com/github/copilot/generated`;
 
     // Generate base SessionEvent class
     await generateSessionEventBaseClass(variants, packageName, packageDir);
@@ -941,7 +941,7 @@ async function generateRpcTypes(schemaPath: string): Promise<void> {
     }
 
     const packageName = "com.github.copilot.generated.rpc";
-    const packageDir = `src/generated/java/com/github/copilot/sdk/generated/rpc`;
+    const packageDir = `src/generated/java/com/github/copilot/generated/rpc`;
 
     // Collect all RPC methods from all sections
     const sections: [string, Record<string, unknown>][] = [];
@@ -1563,7 +1563,7 @@ async function generateRpcWrappers(schemaPath: string): Promise<void> {
     currentDefinitions = (schema.definitions ?? {}) as Record<string, JSONSchema7>;
 
     const packageName = "com.github.copilot.generated.rpc";
-    const packageDir = `src/generated/java/com/github/copilot/sdk/generated/rpc`;
+    const packageDir = `src/generated/java/com/github/copilot/generated/rpc`;
 
     // RpcCaller interface and shared ObjectMapper holder
     await generateRpcCallerInterface(packageName, packageDir);