Skip to content

Implement phase 03 of merge to monorepo plan#1369

Draft
edburns wants to merge 1 commit into
mainfrom
edburns/80-java-monorepo-phase-03-seeking-review
Draft

Implement phase 03 of merge to monorepo plan#1369
edburns wants to merge 1 commit into
mainfrom
edburns/80-java-monorepo-phase-03-seeking-review

Conversation

@edburns
Copy link
Copy Markdown
Collaborator

@edburns edburns commented May 21, 2026

See the plan.

Work in this PR implements Phase 03: Publish Workflows of this plan.

Supersedes #1368

@edburns edburns requested a review from a team as a code owner May 21, 2026 23:58
Copilot AI review requested due to automatic review settings May 21, 2026 23:58
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Implements Phase 03 (“Publish Workflows”) of the Java-to-monorepo migration plan by adding/adjusting Java publishing automation and related release collateral so the Java SDK can be released independently from the monorepo.

Changes:

  • Updates Java publish workflows to run from ./java, generate GitHub releases, and use Java-scoped secrets.
  • Adds a Java smoke-test workflow (JDK 17 + JDK 25) that uses Copilot CLI to generate/build a sample project.
  • Adds Java release collateral (release notes template + changelog update scripts) and supporting operational docs/scripts for the migration workstream.
Show a summary per file
File Description
java/src/test/java/com/github/copilot/sdk/TestUtil.java Adjusts CLI path resolution precedence (env var vs PATH).
.github/workflows/java-publish-maven.yml Java Maven Central release workflow updates (working dir, tag format, release notes template path, removes deploy-site/latest-tag behavior).
.github/workflows/java-publish-snapshot.yml Adds working-directory: ./java defaults for SNAPSHOT publishing.
.github/workflows/java-smoke-test.yml Adds Copilot-driven smoke test workflow for JDK 17 and JDK 25.
.github/workflows/java.notes.template Adds Java GitHub Release notes template.
.github/scripts/release/update-changelog.sh Adds release-time changelog updater script.
.github/scripts/release/test-update-changelog.sh Adds tests for the changelog updater script.
80-java-monorepo-add-01-remove-before-merge/ghcpsp-90-gpg-key-archive.sh Adds GPG key handoff archive script (operational/migration support).
80-java-monorepo-add-01-remove-before-merge/ghcpsp-90-gpg-key-import.sh Adds companion recipient-side GPG import/verify script.
80-java-monorepo-add-01-remove-before-merge/ghcp-sp-95-enable-java-branch-protection.ps1 Adds ruleset creation helper script for branch protection investigation.
80-java-monorepo-add-01-remove-before-merge/ghcp-sp-95-branch-protection-findings-01.md Documents investigation findings re: branch protection and Maven release tooling.
80-java-monorepo-add-01-remove-before-merge/dd-3001935-phase-03-plan.md Adds Phase 03 execution prompt/plan doc.
80-java-monorepo-add-01-remove-before-merge/dd-3001051-phase-2-plan.md Adds Phase 2 execution prompt/plan doc.
80-java-monorepo-add-01-remove-before-merge/dd-2998002-make-the-monorepo-smart-about-java-plan.md Adds plan doc for adding Java skill/governance to monorepo Copilot configuration.
80-java-monorepo-add-01-remove-before-merge/dd-2997995-phase-1-plan.md Adds Phase 1 execution prompt/plan doc.
80-java-monorepo-add-01-remove-before-merge/dd-2989727-move-java-to-monorepo-plan.md Adds/updates the master migration plan doc.
80-java-monorepo-add-01-remove-before-merge/20260512-prompts.md Captures migration prompt history/context.
80-java-monorepo-add-01-remove-before-merge/20260513-prompts.md Captures migration prompt history/context.
80-java-monorepo-add-01-remove-before-merge/20260515-prompts.md Captures migration prompt history/context.
80-java-monorepo-add-01-remove-before-merge/20260518-prompts.md Captures migration prompt history/context.
80-java-monorepo-add-01-remove-before-merge/20260519-prompts.md Captures migration prompt history/context.
80-java-monorepo-add-01-remove-before-merge/20260519-java-sdk-tests-review.md Captures review notes for Phase 2 work.
80-java-monorepo-add-01-remove-before-merge/20260520-prompts.md Captures migration prompt history/context.

Copilot's findings

  • Files reviewed: 7/7 changed files
  • Comments generated: 9

Comment thread java/src/test/java/com/github/copilot/sdk/TestUtil.java
Comment thread .github/workflows/java-smoke-test.yml
Comment thread .github/workflows/java-smoke-test.yml
Comment thread .github/scripts/release/update-changelog.sh Outdated
Comment thread .github/workflows/java-publish-maven.yml
Comment thread .github/workflows/java.notes.template Outdated
Comment thread .github/workflows/java.notes.template Outdated
Comment thread .github/workflows/java.notes.template Outdated
Comment thread .github/scripts/release/test-update-changelog.sh
@edburns edburns marked this pull request as draft May 22, 2026 00:11
edburns added a commit that referenced this pull request May 22, 2026
Implement Phase 03 of the Java-to-monorepo migration plan, adapting
the Java SDK's publishing and testing automation for the monorepo
structure.

Changes:
- java-publish-maven.yml: add working-directory, use java/v tag prefix,
  scope secrets to Java
- java-publish-snapshot.yml: add working-directory for monorepo
- java-smoke-test.yml: new workflow running smoke tests on JDK 17 and
  JDK 25 with globally installed Copilot CLI (version pinned from
  pom.xml)
- java.notes.template: release notes template for GitHub Releases
- update-changelog.sh: script to update CHANGELOG.md during release,
  generating compare links with java/v tag prefix
- test-update-changelog.sh: tests for the changelog update script
- TestUtil.java: align findCliPath() Javadoc with actual resolution
  order (COPILOT_CLI_PATH first, then PATH search)
- pom.xml: update copilot-sdk-ref-impl version property
@edburns edburns force-pushed the edburns/80-java-monorepo-phase-03-seeking-review branch from dd8c5a2 to 30e1c34 Compare May 22, 2026 00:59
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.

2 participants