Skip to content

Start iterating on #1682 On branch edburns/1682-java-tool-ergonomics#1692

Draft
edburns wants to merge 4 commits into
mainfrom
edburns/1682-java-tool-ergonomics
Draft

Start iterating on #1682 On branch edburns/1682-java-tool-ergonomics#1692
edburns wants to merge 4 commits into
mainfrom
edburns/1682-java-tool-ergonomics

Conversation

@edburns

@edburns edburns commented Jun 16, 2026

Copy link
Copy Markdown
Collaborator

new file: 1682-java-tool-ergonomics-prompts-remove-before-merge/20260615-prompts.md

edburns and others added 2 commits June 16, 2026 19:51
new file:   1682-java-tool-ergonomics-prompts-remove-before-merge/20260615-prompts.md

Signed-off-by: Ed Burns <edburns@microsoft.com>
Packages the knowledge of creating net-new Java E2E integration tests
with handcrafted replay proxy YAML snapshots into a reusable Copilot skill.

New files:

- .github/skills/new-java-e2e-test-yaml-and-test/SKILL.md
  Main skill instructions covering: YAML snapshot format, proxy matching
  logic, Java IT test template, verification commands, key classes/files
  reference table, and common pitfalls. Includes the critical constraint
  that Java's CapiProxy always sets GITHUB_ACTIONS=true so snapshots must
  be handcrafted rather than recorded.

- .github/skills/new-java-e2e-test-yaml-and-test/examples.md
  Two working examples from the codebase: a simple single-turn test
  (Botanica identity REPLACE) and a multi-turn test with tool calls
  (system message transform with view tool).

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@edburns edburns force-pushed the edburns/1682-java-tool-ergonomics branch from 6c3fe31 to 299c3e4 Compare June 16, 2026 23:52
@github-actions

Copy link
Copy Markdown
Contributor

🔍 Cross-SDK Consistency Review

Summary

This PR makes Java-only changes that do not create cross-SDK API inconsistencies:

  1. java/scripts/codegen/java.ts — adds generation of package-info.java files for com.github.copilot.generated and com.github.copilot.generated.rpc
  2. Two generated package-info.java files — package-level Javadoc for the auto-generated packages
  3. .github/skills/new-java-e2e-test-yaml-and-test/ — internal Copilot agent skill (not public API)

✅ No cross-SDK API changes

The package-info.java additions are a Java-specific idiom for package-level documentation. Other SDKs already have their own equivalent patterns (TypeScript uses file-level JSDoc in generated files, Python uses __init__.py, etc.) and don't need updates.

⚠️ Cleanup needed before merge

The PR still contains the development notes directory that the PR description itself flags for removal:

1682-java-tool-ergonomics-prompts-remove-before-merge/
  ├── 20260615-prompts.md
  ├── 20260616-prompts.md
  ├── #20260616-prompts.md#   ← looks like an Emacs backup file
  └── .#20260616-prompts.md  ← looks like an Emacs lock file

These files should be removed before this draft is marked ready for review. The Emacs backup/lock files (#...# and .#...) suggest they may have been accidentally committed — consider adding *~, #*#, and .#* to .gitignore if not already present.

Generated by SDK Consistency Review Agent for issue #1692 · sonnet46 929K ·

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant