Edburns/1810 java tool ergonomics tool as lambda seeking review - #1895
Merged
edburns merged 58 commits intoJul 2, 2026
Merged
Conversation
Your branch is up to date with 'upstream/edburns/1810-java-tool-ergonomics-tool-as-lambda'. Changes to be committed: (use "git restore --staged <file>..." to unstage) modified: 1810-java-tool-ergonomics-tool-as-lambda-remove-before-merge/1810-ignorance-reduction-for-implementation-plan.md modified: 1810-java-tool-ergonomics-tool-as-lambda-remove-before-merge/20260628-prompts.md new file: 1810-java-tool-ergonomics-tool-as-lambda-remove-before-merge/20260629-prompts.md Signed-off-by: Ed Burns <edburns@microsoft.com>
…Phase 03. modified: 1810-java-tool-ergonomics-tool-as-lambda-remove-before-merge/1810-ignorance-reduction-for-implementation-plan.md modified: 1810-java-tool-ergonomics-tool-as-lambda-remove-before-merge/20260629-prompts.md Signed-off-by: Ed Burns <edburns@microsoft.com>
modified: 1810-java-tool-ergonomics-tool-as-lambda-remove-before-merge/1810-ignorance-reduction-for-implementation-plan.md - Check off the things already done. modified: 1810-java-tool-ergonomics-tool-as-lambda-remove-before-merge/20260629-prompts.md - GUTDODP
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Introduce `com.github.copilot.tool.Param<T>` — an immutable, fluent runtime parameter metadata class for inline/lambda tool definitions. Validation behavior: - Rejects blank name/description - Rejects required=true with non-empty defaultValue - Validates default values against declared Class<T> Includes comprehensive unit tests (ParamTest, 24 cases). Updates Phase 4.1 checkbox in the implementation plan.
Automates the lifecycle of a child Task issue from 'assigned to Copilot' through CI approval and review-agent feedback resolution, stopping just before marking the PR as Ready for Review. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
On branch edburns/1810-java-tool-ergonomics-tool-as-lambda modified: .github/skills/shepherd-task-to-ready/SKILL.md modified: 1810-java-tool-ergonomics-tool-as-lambda-remove-before-merge/20260630-prompts.md Signed-off-by: Ed Burns <edburns@microsoft.com>
…ix base branch - Use 'gh run rerun' instead of fork-only approve API endpoint - Ignore expected 'Block remove-before-merge paths' workflow failure - Verify and fix PR base branch after creation (Copilot may target main) Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Avoids race condition where Copilot targets main instead of the specified base branch. Instruction is added to issue body before assignment; base branch verification remains as fallback. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
The model should not mark checklist items as complete — that is the human DRI's responsibility. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
…to-ready Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
On branch edburns/1810-java-tool-ergonomics-tool-as-lambda modified: 1810-java-tool-ergonomics-tool-as-lambda-remove-before-merge/20260630-prompts.md Signed-off-by: Ed Burns <edburns@microsoft.com>
Follow-up skill that takes a PR from Ready for Review through Copilot code review comment resolution (done locally) and merge to the specified base branch. Max 20 iterations. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
…2) (#1857) * Initial plan * feat(java): implement ToolDefinition.from* overloads for lambda-defined tools (Phase 4.2) Co-authored-by: edburns <75821+edburns@users.noreply.github.com> * Fix review comments: typed defaults, array items schema, primitive cast - buildSchemaFromParams: parse default values to declared type before placing in JSON schema (avoids String defaults for numeric/boolean) - schemaForClass: emit items schema for Java array types using getComponentType() for schema fidelity - coerceDefaultValue: use boxed valueOf() instead of type.cast() for primitive types to avoid ClassCastException Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Fix review round 3: ToolResultObject passthrough, Optional* schema - formatResult: pass ToolResultObject through directly instead of JSON-serializing, preserving structured result semantics - schemaForClass: add OptionalInt/OptionalLong/OptionalDouble support to match compile-time SchemaGenerator behavior Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Fix review round 4: Optional* coercion in coerceArg - Return OptionalInt.empty()/OptionalLong.empty()/OptionalDouble.empty() for missing non-required params instead of null - Construct Optional*.of(...) from Number when value is present - Avoids NPE and aligns with annotation-processor behavior Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Fix review round 5: null-future guards, Optional* cast safety, @SInCE tags - All async fromAsync*/fromAsyncWithToolInvocation handlers now check for null future and return failedFuture with clear NPE message - Optional* coercion catches ClassCastException for non-numeric values and throws IllegalArgumentException with diagnostic message - Fixed @SInCE 1.0.2 -> 1.0.6 on all new API entries (15 in ToolDefinition, 1 in Param) Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: edburns <75821+edburns@users.noreply.github.com> Co-authored-by: Ed Burns <edburns@microsoft.com> Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
This was referenced Jul 13, 2026
This was referenced Jul 20, 2026
This was referenced Jul 27, 2026
edburns
added a commit
that referenced
this pull request
Aug 4, 2026
* ADR-006 * Plan * GUTDODP * GUTDODP * On branch edburns/1810-java-tool-ergonomics-tool-as-lambda GOTDODP Your branch is up to date with 'upstream/edburns/1810-java-tool-ergonomics-tool-as-lambda'. Changes to be committed: (use "git restore --staged <file>..." to unstage) modified: 1810-java-tool-ergonomics-tool-as-lambda-remove-before-merge/1810-ignorance-reduction-for-implementation-plan.md modified: 1810-java-tool-ergonomics-tool-as-lambda-remove-before-merge/20260628-prompts.md new file: 1810-java-tool-ergonomics-tool-as-lambda-remove-before-merge/20260629-prompts.md Signed-off-by: Ed Burns <edburns@microsoft.com> * On branch edburns/1810-java-tool-ergonomics-tool-as-lambda Completed Phase 03. modified: 1810-java-tool-ergonomics-tool-as-lambda-remove-before-merge/1810-ignorance-reduction-for-implementation-plan.md modified: 1810-java-tool-ergonomics-tool-as-lambda-remove-before-merge/20260629-prompts.md Signed-off-by: Ed Burns <edburns@microsoft.com> * GUTDODP * GUTDODP * On branch edburns/1810-java-tool-ergonomics-tool-as-lambda modified: 1810-java-tool-ergonomics-tool-as-lambda-remove-before-merge/1810-ignorance-reduction-for-implementation-plan.md - Check off the things already done. modified: 1810-java-tool-ergonomics-tool-as-lambda-remove-before-merge/20260629-prompts.md - GUTDODP * Add Phase 4 checklist linked to child issues Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Initial plan * Add Param<T> public API type for lambda-defined tools (#1839) Introduce `com.github.copilot.tool.Param<T>` — an immutable, fluent runtime parameter metadata class for inline/lambda tool definitions. Validation behavior: - Rejects blank name/description - Rejects required=true with non-empty defaultValue - Validates default values against declared Class<T> Includes comprehensive unit tests (ParamTest, 24 cases). Updates Phase 4.1 checkbox in the implementation plan. * Add Float/Short/Byte default validation test coverage for Param<T> * GUTDODP * Add shepherd-task-to-ready skill Automates the lifecycle of a child Task issue from 'assigned to Copilot' through CI approval and review-agent feedback resolution, stopping just before marking the PR as Ready for Review. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Iterate the skill On branch edburns/1810-java-tool-ergonomics-tool-as-lambda modified: .github/skills/shepherd-task-to-ready/SKILL.md modified: 1810-java-tool-ergonomics-tool-as-lambda-remove-before-merge/20260630-prompts.md Signed-off-by: Ed Burns <edburns@microsoft.com> * Iterate skill * Spotless * Mark 4.1 as complete * Update shepherd skill: rerun for approval, ignore expected failure, fix base branch - Use 'gh run rerun' instead of fork-only approve API endpoint - Ignore expected 'Block remove-before-merge paths' workflow failure - Verify and fix PR base branch after creation (Copilot may target main) Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Skill: prepend base branch instruction before assigning to Copilot Avoids race condition where Copilot targets main instead of the specified base branch. Instruction is added to issue body before assignment; base branch verification remains as fallback. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Iterate the skill * Skill: add guard against editing plan/checklist files The model should not mark checklist items as complete — that is the human DRI's responsibility. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Skill: clarify checklist editing is out of scope, not DRI-specific Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * GUTDODP * Rename skill: shepherd-task-to-ready → shepherd-task-from-assignment-to-ready Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Working to prompt the second skill. On branch edburns/1810-java-tool-ergonomics-tool-as-lambda modified: 1810-java-tool-ergonomics-tool-as-lambda-remove-before-merge/20260630-prompts.md Signed-off-by: Ed Burns <edburns@microsoft.com> * Add shepherd-task-from-ready-to-merged-to-base skill Follow-up skill that takes a PR from Ready for Review through Copilot code review comment resolution (done locally) and merge to the specified base branch. Max 20 iterations. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Find the PR * feat(java): implement ToolDefinition.from* lambda overloads (Phase 4.2) (#1857) * Initial plan * feat(java): implement ToolDefinition.from* overloads for lambda-defined tools (Phase 4.2) Co-authored-by: edburns <75821+edburns@users.noreply.github.com> * Fix review comments: typed defaults, array items schema, primitive cast - buildSchemaFromParams: parse default values to declared type before placing in JSON schema (avoids String defaults for numeric/boolean) - schemaForClass: emit items schema for Java array types using getComponentType() for schema fidelity - coerceDefaultValue: use boxed valueOf() instead of type.cast() for primitive types to avoid ClassCastException Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Fix review round 3: ToolResultObject passthrough, Optional* schema - formatResult: pass ToolResultObject through directly instead of JSON-serializing, preserving structured result semantics - schemaForClass: add OptionalInt/OptionalLong/OptionalDouble support to match compile-time SchemaGenerator behavior Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Fix review round 4: Optional* coercion in coerceArg - Return OptionalInt.empty()/OptionalLong.empty()/OptionalDouble.empty() for missing non-required params instead of null - Construct Optional*.of(...) from Number when value is present - Avoids NPE and aligns with annotation-processor behavior Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Fix review round 5: null-future guards, Optional* cast safety, @SInCE tags - All async fromAsync*/fromAsyncWithToolInvocation handlers now check for null future and return failedFuture with clear NPE message - Optional* coercion catches ClassCastException for non-numeric values and throws IllegalArgumentException with diagnostic message - Fixed @SInCE 1.0.2 -> 1.0.6 on all new API entries (15 in ToolDefinition, 1 in Param) Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: edburns <75821+edburns@users.noreply.github.com> Co-authored-by: Ed Burns <edburns@microsoft.com> Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Skill: add GraphQL thread resolution to Step 8 Use resolveReviewThread mutation to programmatically mark review threads as resolved after replying, instead of requiring manual resolution in the UI. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * On branch edburns/1810-java-tool-ergonomics-tool-as-lambda modified: .github/skills/shepherd-task-from-ready-to-merged-to-base/SKILL.md Refine skill for resolving comments. modified: 1810-java-tool-ergonomics-tool-as-lambda-remove-before-merge/20260630-prompts.md - GUTDODP Signed-off-by: Ed Burns <edburns@microsoft.com> * On branch edburns/1810-java-tool-ergonomics-tool-as-lambda modified: .github/skills/shepherd-task-from-ready-to-merged-to-base/SKILL.md - Instruct the agent to close the issue after merging. modified: 1810-java-tool-ergonomics-tool-as-lambda-remove-before-merge/20260630-prompts.md - GUTDODP Signed-off-by: Ed Burns <edburns@microsoft.com> * Add shepherd-task skill: end-to-end orchestrator Invokes shepherd-task-from-assignment-to-ready then shepherd-task-from-ready-to-merged-to-base in sequence. Only proceeds to Phase 2 if Phase 1 succeeds. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * On branch edburns/1810-java-tool-ergonomics-tool-as-lambda modified: .github/skills/shepherd-task/SKILL.md - Tell the agent to mark the task as complete. modified: 1810-java-tool-ergonomics-tool-as-lambda-remove-before-merge/20260630-prompts.md - GUTDODP Signed-off-by: Ed Burns <edburns@microsoft.com> * Invoke skill result * fix: use --body-file to preserve markdown formatting when prepending to issue body The previous approach using inline --body with shell variable interpolation stripped newlines from the original issue body, causing the Copilot cloud agent to receive a wall of unformatted text and misinterpret the assignment. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * fix: add idempotency guard for base branch prepend in shepherd skill Skip the issue body prepend if it already starts with '**Base branch:**', preventing double-prepending on retries after partial failures. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * feat: add /compact between Phase 1 and Phase 2 in shepherd-task skill Reduces context window usage before entering the review-iteration-heavy Phase 2, retaining only essential state (PR number, branch, inputs). Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * feat(java): implement ParamSchema + ParamCoercion internals for Param (Phase 4.3) (#1877) * Initial plan * feat(java): implement schema + coercion internals for Param (Phase 4.3) Fixes #1841 Adds two package-private internal helper classes in com.github.copilot.tool: - ParamSchema: runtime JSON Schema generation from Param<?> metadata. buildSchema() validates duplicate names; forType() mirrors the compile-time SchemaGenerator using java.lang.reflect.Class instead of javax.lang.model. - ParamCoercion: runtime argument coercion using existing ObjectMapper policy. coerce() resolves args → default → empty-optional → required-error in order. coerceDefault() parses string defaults into declared Java types. emptyOptionalOrNull() returns empty Optional variants for optional primitives. Both classes are package-private per resolution 3.8 (no public internal helpers). coerce() takes Map<String,Object> to avoid a cyclic rpc dependency. Updates Phase 4.3 checkbox in plan file. Co-authored-by: edburns <75821+edburns@users.noreply.github.com> * fix(java): address Copilot code review comments on ParamSchema/ParamCoercion - Add null guard for varargs array in buildSchema() - Clarify ParamSchema class Javadoc: simplified counterpart, not full parity - Clarify forType() Javadoc: flat type mapping only, no generics resolution - Clarify coerceDefault() Javadoc: ObjectMapper fallback is safety net only Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * fix(java): correct ParamSchema Javadoc - arrays do produce items schema Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: edburns <75821+edburns@users.noreply.github.com> Co-authored-by: Ed Burns <edburns@microsoft.com> Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * fix(skills): improve PR discovery with multi-strategy polling The shepherd skill's PR polling only matched by title/branch regex, which fails when Copilot uses descriptive names without the issue number. Now uses three strategies per iteration: A) Issue timeline API for cross-referenced PRs (most reliable) B) PR body search for 'Fixes #N' references C) Title/branch regex match (original fallback) Also increased timeout from 10 to 15 minutes since Copilot can take 5-12 minutes to produce a PR. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * feat: add shepherd-task shell scripts (PowerShell + bash) Orchestrates two separate copilot --yolo sessions for Phase 1 and Phase 2, with gh CLI verification between phases. Avoids context window exhaustion by using independent copilot instances instead of /compact. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * [Java] Tool-as-lambda 4.4: Add unit tests for API behavior and validation (#1879) * Initial plan * Phase 4.4: Add ToolDefinitionLambdaTest – unit tests for lambda tool API behavior and validation Co-authored-by: edburns <75821+edburns@users.noreply.github.com> * Address Copilot review: tighten test assertions - requiredParam test: assert IllegalArgumentException directly (not generic Exception via .get()) and verify message mentions param name - resultFormatting test: parse result as JSON and assert specific fields instead of loose string contains check Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: edburns <75821+edburns@users.noreply.github.com> Co-authored-by: Ed Burns <edburns@microsoft.com> Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * fix: strip remote prefix when comparing merged base branch name GitHub's baseRefName API never includes the remote prefix (e.g., upstream/), so normalize BaseBranch before comparison to avoid false failures. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * On branch edburns/1810-java-tool-ergonomics-tool-as-lambda new file: 1810-java-tool-ergonomics-tool-as-lambda-remove-before-merge/20260701-prompts.md modified: 1810-java-tool-ergonomics-tool-as-lambda-remove-before-merge/1810-ignorance-reduction-for-implementation-plan.md - When writing the new E2E test, rely on the existing skill. Signed-off-by: Ed Burns <edburns@microsoft.com> * [Java] Add replay-proxy E2E coverage for inline lambda-defined tools (#1881) * Initial plan * Add Java lambda-based E2E tool definition coverage Co-authored-by: edburns <75821+edburns@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: edburns <75821+edburns@users.noreply.github.com> * Extract workflow approval into reusable sub-skill Extract Steps 4-5 (approve pending workflow runs and wait for completion) from shepherd-task-from-assignment-to-ready into a new standalone skill: shepherd-task-approve-workflows-and-wait-for-completion. The original skill now invokes the sub-skill by reference. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Add workflow approval calls to ready-to-merged skill Insert invocations of shepherd-task-approve-workflows-and-wait-for-completion before gathering review comments (Step 5), before re-requesting review (Step 11), and before final checks (Step 14). Renumber all steps sequentially (0-21). Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * On branch edburns/1810-java-tool-ergonomics-tool-as-lambda modified: 1810-java-tool-ergonomics-tool-as-lambda-remove-before-merge/20260701-prompts.md - GUTDODP Signed-off-by: Ed Burns <edburns@microsoft.com> * Use agent_assignment API to guarantee base branch on Copilot assignment Replace the body-prepend workaround with the REST API's agent_assignment.base_branch field when assigning issues to Copilot. This is the programmatic equivalent of selecting the branch in the GitHub UI and guarantees Copilot creates its topic branch from BASE_BRANCH instead of defaulting to main. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Revert "Use agent_assignment API to guarantee base branch on Copilot assignment" This reverts commit 2a45faf. * Strengthen base branch enforcement for Copilot assignment Add three layers of reinforcement to ensure Copilot uses the correct base branch: 1. More prominent body prepend using GitHub IMPORTANT callout syntax with explicit DO NOT instructions 2. Reinforcing comment posted immediately after assignment 3. Stronger fallback: if PR targets wrong base, fix it AND request Copilot rebase with a changes-requested review This is critical because issue descriptions reference plan files that only exist on the feature branch, not on main. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * On branch edburns/1810-java-tool-ergonomics-tool-as-lambda Your branch is up to date with 'upstream/edburns/1810-java-tool-ergonomics-tool-as-lambda'. Changes not staged for commit: (use "git add <file>..." to update what will be committed) (use "git restore <file>..." to discard changes in working directory) modified: 1810-java-tool-ergonomics-tool-as-lambda-remove-before-merge/20260701-prompts.md no changes added to commit (use "git add" and/or "git commit -a") Signed-off-by: Ed Burns <edburns@microsoft.com> * [Java] Align inline tool docs with final lambda API and ADR links (#1885) * Initial plan * [Java] Align inline tool docs with final lambda API and ADR links - README: Added inline lambda tool authoring section with ToolDefinition.from(...) examples - Documented Param.of(...) required/default behavior and fluent modifiers - ADR-006: Updated to reflect final API (Param.of vs Params.of/ParamDef) - ADR-006: Added ADR-005 cross-reference and README coverage note - Plan: Marked Phase 4.6 as complete Fixes #1884 Co-authored-by: edburns <75821+edburns@users.noreply.github.com> * fix: add CompletableFuture import to async snippet and remove invalid ToolDefer.ALWAYS - Added missing import statement to make the async handler code example self-contained and compilable. - Removed ALWAYS from ToolDefer values list; enum only has NONE, AUTO, NEVER. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: edburns <75821+edburns@users.noreply.github.com> Co-authored-by: Ed Burns <edburns@microsoft.com> Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Removing these files from this topic branch Skills: • `.github/skills/shepherd-task-approve-workflows-and-wait-for-completion/SKILL.md` • `.github/skills/shepherd-task-from-assignment-to-ready/SKILL.md` • `.github/skills/shepherd-task-from-ready-to-merged-to-base/SKILL.md` • `.github/skills/shepherd-task/SKILL.md` Scripts: • `1810-java-tool-ergonomics-tool-as-lambda-remove-before-merge/shepherd-task.ps1` • `1810-java-tool-ergonomics-tool-as-lambda-remove-before-merge/shepherd-task.sh` This work is now being tracked in **Feature** #1893. * Remove prompts before seeking review * java: delegate ToolDefinition schema/coercion to ParamSchema and ParamCoercion Move ParamSchema and ParamCoercion from com.github.copilot.tool to com.github.copilot.rpc (package-private). Update all from*/fromAsync*/ fromWithToolInvocation*/fromAsyncWithToolInvocation* overloads to delegate to these helpers instead of inline private methods. Remove dead private methods from ToolDefinition: buildSchemaFromParams, schemaForClass, coerceArg, coerceDefaultValue, emptyOptionalOrNull, requireNonNullParam, requireUniqueParamNames. Addresses PR reviewer feedback about unused classes and duplicated logic. * java: cache getConfiguredMapper() in local variable per invocation Capture the singleton ObjectMapper once at the top of each from* factory method instead of calling getConfiguredMapper() multiple times. The lambda closes over the local, making it explicit that schema building, coercion, and result formatting all use the same mapper instance. Addresses Copilot AI reviewer suggestion (discussion r3514594600). --------- Signed-off-by: Ed Burns <edburns@microsoft.com> Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: edburns <75821+edburns@users.noreply.github.com>
edburns
added a commit
that referenced
this pull request
Aug 4, 2026
* ADR-006 * Plan * GUTDODP * GUTDODP * On branch edburns/1810-java-tool-ergonomics-tool-as-lambda GOTDODP Your branch is up to date with 'upstream/edburns/1810-java-tool-ergonomics-tool-as-lambda'. Changes to be committed: (use "git restore --staged <file>..." to unstage) modified: 1810-java-tool-ergonomics-tool-as-lambda-remove-before-merge/1810-ignorance-reduction-for-implementation-plan.md modified: 1810-java-tool-ergonomics-tool-as-lambda-remove-before-merge/20260628-prompts.md new file: 1810-java-tool-ergonomics-tool-as-lambda-remove-before-merge/20260629-prompts.md Signed-off-by: Ed Burns <edburns@microsoft.com> * On branch edburns/1810-java-tool-ergonomics-tool-as-lambda Completed Phase 03. modified: 1810-java-tool-ergonomics-tool-as-lambda-remove-before-merge/1810-ignorance-reduction-for-implementation-plan.md modified: 1810-java-tool-ergonomics-tool-as-lambda-remove-before-merge/20260629-prompts.md Signed-off-by: Ed Burns <edburns@microsoft.com> * GUTDODP * GUTDODP * On branch edburns/1810-java-tool-ergonomics-tool-as-lambda modified: 1810-java-tool-ergonomics-tool-as-lambda-remove-before-merge/1810-ignorance-reduction-for-implementation-plan.md - Check off the things already done. modified: 1810-java-tool-ergonomics-tool-as-lambda-remove-before-merge/20260629-prompts.md - GUTDODP * Add Phase 4 checklist linked to child issues Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Initial plan * Add Param<T> public API type for lambda-defined tools (#1839) Introduce `com.github.copilot.tool.Param<T>` — an immutable, fluent runtime parameter metadata class for inline/lambda tool definitions. Validation behavior: - Rejects blank name/description - Rejects required=true with non-empty defaultValue - Validates default values against declared Class<T> Includes comprehensive unit tests (ParamTest, 24 cases). Updates Phase 4.1 checkbox in the implementation plan. * Add Float/Short/Byte default validation test coverage for Param<T> * GUTDODP * Add shepherd-task-to-ready skill Automates the lifecycle of a child Task issue from 'assigned to Copilot' through CI approval and review-agent feedback resolution, stopping just before marking the PR as Ready for Review. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Iterate the skill On branch edburns/1810-java-tool-ergonomics-tool-as-lambda modified: .github/skills/shepherd-task-to-ready/SKILL.md modified: 1810-java-tool-ergonomics-tool-as-lambda-remove-before-merge/20260630-prompts.md Signed-off-by: Ed Burns <edburns@microsoft.com> * Iterate skill * Spotless * Mark 4.1 as complete * Update shepherd skill: rerun for approval, ignore expected failure, fix base branch - Use 'gh run rerun' instead of fork-only approve API endpoint - Ignore expected 'Block remove-before-merge paths' workflow failure - Verify and fix PR base branch after creation (Copilot may target main) Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Skill: prepend base branch instruction before assigning to Copilot Avoids race condition where Copilot targets main instead of the specified base branch. Instruction is added to issue body before assignment; base branch verification remains as fallback. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Iterate the skill * Skill: add guard against editing plan/checklist files The model should not mark checklist items as complete — that is the human DRI's responsibility. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Skill: clarify checklist editing is out of scope, not DRI-specific Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * GUTDODP * Rename skill: shepherd-task-to-ready → shepherd-task-from-assignment-to-ready Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Working to prompt the second skill. On branch edburns/1810-java-tool-ergonomics-tool-as-lambda modified: 1810-java-tool-ergonomics-tool-as-lambda-remove-before-merge/20260630-prompts.md Signed-off-by: Ed Burns <edburns@microsoft.com> * Add shepherd-task-from-ready-to-merged-to-base skill Follow-up skill that takes a PR from Ready for Review through Copilot code review comment resolution (done locally) and merge to the specified base branch. Max 20 iterations. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Find the PR * feat(java): implement ToolDefinition.from* lambda overloads (Phase 4.2) (#1857) * Initial plan * feat(java): implement ToolDefinition.from* overloads for lambda-defined tools (Phase 4.2) Co-authored-by: edburns <75821+edburns@users.noreply.github.com> * Fix review comments: typed defaults, array items schema, primitive cast - buildSchemaFromParams: parse default values to declared type before placing in JSON schema (avoids String defaults for numeric/boolean) - schemaForClass: emit items schema for Java array types using getComponentType() for schema fidelity - coerceDefaultValue: use boxed valueOf() instead of type.cast() for primitive types to avoid ClassCastException Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Fix review round 3: ToolResultObject passthrough, Optional* schema - formatResult: pass ToolResultObject through directly instead of JSON-serializing, preserving structured result semantics - schemaForClass: add OptionalInt/OptionalLong/OptionalDouble support to match compile-time SchemaGenerator behavior Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Fix review round 4: Optional* coercion in coerceArg - Return OptionalInt.empty()/OptionalLong.empty()/OptionalDouble.empty() for missing non-required params instead of null - Construct Optional*.of(...) from Number when value is present - Avoids NPE and aligns with annotation-processor behavior Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Fix review round 5: null-future guards, Optional* cast safety, @SInCE tags - All async fromAsync*/fromAsyncWithToolInvocation handlers now check for null future and return failedFuture with clear NPE message - Optional* coercion catches ClassCastException for non-numeric values and throws IllegalArgumentException with diagnostic message - Fixed @SInCE 1.0.2 -> 1.0.6 on all new API entries (15 in ToolDefinition, 1 in Param) Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: edburns <75821+edburns@users.noreply.github.com> Co-authored-by: Ed Burns <edburns@microsoft.com> Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Skill: add GraphQL thread resolution to Step 8 Use resolveReviewThread mutation to programmatically mark review threads as resolved after replying, instead of requiring manual resolution in the UI. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * On branch edburns/1810-java-tool-ergonomics-tool-as-lambda modified: .github/skills/shepherd-task-from-ready-to-merged-to-base/SKILL.md Refine skill for resolving comments. modified: 1810-java-tool-ergonomics-tool-as-lambda-remove-before-merge/20260630-prompts.md - GUTDODP Signed-off-by: Ed Burns <edburns@microsoft.com> * On branch edburns/1810-java-tool-ergonomics-tool-as-lambda modified: .github/skills/shepherd-task-from-ready-to-merged-to-base/SKILL.md - Instruct the agent to close the issue after merging. modified: 1810-java-tool-ergonomics-tool-as-lambda-remove-before-merge/20260630-prompts.md - GUTDODP Signed-off-by: Ed Burns <edburns@microsoft.com> * Add shepherd-task skill: end-to-end orchestrator Invokes shepherd-task-from-assignment-to-ready then shepherd-task-from-ready-to-merged-to-base in sequence. Only proceeds to Phase 2 if Phase 1 succeeds. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * On branch edburns/1810-java-tool-ergonomics-tool-as-lambda modified: .github/skills/shepherd-task/SKILL.md - Tell the agent to mark the task as complete. modified: 1810-java-tool-ergonomics-tool-as-lambda-remove-before-merge/20260630-prompts.md - GUTDODP Signed-off-by: Ed Burns <edburns@microsoft.com> * Invoke skill result * fix: use --body-file to preserve markdown formatting when prepending to issue body The previous approach using inline --body with shell variable interpolation stripped newlines from the original issue body, causing the Copilot cloud agent to receive a wall of unformatted text and misinterpret the assignment. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * fix: add idempotency guard for base branch prepend in shepherd skill Skip the issue body prepend if it already starts with '**Base branch:**', preventing double-prepending on retries after partial failures. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * feat: add /compact between Phase 1 and Phase 2 in shepherd-task skill Reduces context window usage before entering the review-iteration-heavy Phase 2, retaining only essential state (PR number, branch, inputs). Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * feat(java): implement ParamSchema + ParamCoercion internals for Param (Phase 4.3) (#1877) * Initial plan * feat(java): implement schema + coercion internals for Param (Phase 4.3) Fixes #1841 Adds two package-private internal helper classes in com.github.copilot.tool: - ParamSchema: runtime JSON Schema generation from Param<?> metadata. buildSchema() validates duplicate names; forType() mirrors the compile-time SchemaGenerator using java.lang.reflect.Class instead of javax.lang.model. - ParamCoercion: runtime argument coercion using existing ObjectMapper policy. coerce() resolves args → default → empty-optional → required-error in order. coerceDefault() parses string defaults into declared Java types. emptyOptionalOrNull() returns empty Optional variants for optional primitives. Both classes are package-private per resolution 3.8 (no public internal helpers). coerce() takes Map<String,Object> to avoid a cyclic rpc dependency. Updates Phase 4.3 checkbox in plan file. Co-authored-by: edburns <75821+edburns@users.noreply.github.com> * fix(java): address Copilot code review comments on ParamSchema/ParamCoercion - Add null guard for varargs array in buildSchema() - Clarify ParamSchema class Javadoc: simplified counterpart, not full parity - Clarify forType() Javadoc: flat type mapping only, no generics resolution - Clarify coerceDefault() Javadoc: ObjectMapper fallback is safety net only Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * fix(java): correct ParamSchema Javadoc - arrays do produce items schema Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: edburns <75821+edburns@users.noreply.github.com> Co-authored-by: Ed Burns <edburns@microsoft.com> Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * fix(skills): improve PR discovery with multi-strategy polling The shepherd skill's PR polling only matched by title/branch regex, which fails when Copilot uses descriptive names without the issue number. Now uses three strategies per iteration: A) Issue timeline API for cross-referenced PRs (most reliable) B) PR body search for 'Fixes #N' references C) Title/branch regex match (original fallback) Also increased timeout from 10 to 15 minutes since Copilot can take 5-12 minutes to produce a PR. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * feat: add shepherd-task shell scripts (PowerShell + bash) Orchestrates two separate copilot --yolo sessions for Phase 1 and Phase 2, with gh CLI verification between phases. Avoids context window exhaustion by using independent copilot instances instead of /compact. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * [Java] Tool-as-lambda 4.4: Add unit tests for API behavior and validation (#1879) * Initial plan * Phase 4.4: Add ToolDefinitionLambdaTest – unit tests for lambda tool API behavior and validation Co-authored-by: edburns <75821+edburns@users.noreply.github.com> * Address Copilot review: tighten test assertions - requiredParam test: assert IllegalArgumentException directly (not generic Exception via .get()) and verify message mentions param name - resultFormatting test: parse result as JSON and assert specific fields instead of loose string contains check Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: edburns <75821+edburns@users.noreply.github.com> Co-authored-by: Ed Burns <edburns@microsoft.com> Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * fix: strip remote prefix when comparing merged base branch name GitHub's baseRefName API never includes the remote prefix (e.g., upstream/), so normalize BaseBranch before comparison to avoid false failures. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * On branch edburns/1810-java-tool-ergonomics-tool-as-lambda new file: 1810-java-tool-ergonomics-tool-as-lambda-remove-before-merge/20260701-prompts.md modified: 1810-java-tool-ergonomics-tool-as-lambda-remove-before-merge/1810-ignorance-reduction-for-implementation-plan.md - When writing the new E2E test, rely on the existing skill. Signed-off-by: Ed Burns <edburns@microsoft.com> * [Java] Add replay-proxy E2E coverage for inline lambda-defined tools (#1881) * Initial plan * Add Java lambda-based E2E tool definition coverage Co-authored-by: edburns <75821+edburns@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: edburns <75821+edburns@users.noreply.github.com> * Extract workflow approval into reusable sub-skill Extract Steps 4-5 (approve pending workflow runs and wait for completion) from shepherd-task-from-assignment-to-ready into a new standalone skill: shepherd-task-approve-workflows-and-wait-for-completion. The original skill now invokes the sub-skill by reference. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Add workflow approval calls to ready-to-merged skill Insert invocations of shepherd-task-approve-workflows-and-wait-for-completion before gathering review comments (Step 5), before re-requesting review (Step 11), and before final checks (Step 14). Renumber all steps sequentially (0-21). Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * On branch edburns/1810-java-tool-ergonomics-tool-as-lambda modified: 1810-java-tool-ergonomics-tool-as-lambda-remove-before-merge/20260701-prompts.md - GUTDODP Signed-off-by: Ed Burns <edburns@microsoft.com> * Use agent_assignment API to guarantee base branch on Copilot assignment Replace the body-prepend workaround with the REST API's agent_assignment.base_branch field when assigning issues to Copilot. This is the programmatic equivalent of selecting the branch in the GitHub UI and guarantees Copilot creates its topic branch from BASE_BRANCH instead of defaulting to main. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Revert "Use agent_assignment API to guarantee base branch on Copilot assignment" This reverts commit 2ab3cca9ca913b339147f4b9fdd614f311df65d6. * Strengthen base branch enforcement for Copilot assignment Add three layers of reinforcement to ensure Copilot uses the correct base branch: 1. More prominent body prepend using GitHub IMPORTANT callout syntax with explicit DO NOT instructions 2. Reinforcing comment posted immediately after assignment 3. Stronger fallback: if PR targets wrong base, fix it AND request Copilot rebase with a changes-requested review This is critical because issue descriptions reference plan files that only exist on the feature branch, not on main. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * On branch edburns/1810-java-tool-ergonomics-tool-as-lambda Your branch is up to date with 'upstream/edburns/1810-java-tool-ergonomics-tool-as-lambda'. Changes not staged for commit: (use "git add <file>..." to update what will be committed) (use "git restore <file>..." to discard changes in working directory) modified: 1810-java-tool-ergonomics-tool-as-lambda-remove-before-merge/20260701-prompts.md no changes added to commit (use "git add" and/or "git commit -a") Signed-off-by: Ed Burns <edburns@microsoft.com> * [Java] Align inline tool docs with final lambda API and ADR links (#1885) * Initial plan * [Java] Align inline tool docs with final lambda API and ADR links - README: Added inline lambda tool authoring section with ToolDefinition.from(...) examples - Documented Param.of(...) required/default behavior and fluent modifiers - ADR-006: Updated to reflect final API (Param.of vs Params.of/ParamDef) - ADR-006: Added ADR-005 cross-reference and README coverage note - Plan: Marked Phase 4.6 as complete Fixes #1884 Co-authored-by: edburns <75821+edburns@users.noreply.github.com> * fix: add CompletableFuture import to async snippet and remove invalid ToolDefer.ALWAYS - Added missing import statement to make the async handler code example self-contained and compilable. - Removed ALWAYS from ToolDefer values list; enum only has NONE, AUTO, NEVER. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: edburns <75821+edburns@users.noreply.github.com> Co-authored-by: Ed Burns <edburns@microsoft.com> Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Removing these files from this topic branch Skills: • `.github/skills/shepherd-task-approve-workflows-and-wait-for-completion/SKILL.md` • `.github/skills/shepherd-task-from-assignment-to-ready/SKILL.md` • `.github/skills/shepherd-task-from-ready-to-merged-to-base/SKILL.md` • `.github/skills/shepherd-task/SKILL.md` Scripts: • `1810-java-tool-ergonomics-tool-as-lambda-remove-before-merge/shepherd-task.ps1` • `1810-java-tool-ergonomics-tool-as-lambda-remove-before-merge/shepherd-task.sh` This work is now being tracked in **Feature** #1893. * Remove prompts before seeking review * java: delegate ToolDefinition schema/coercion to ParamSchema and ParamCoercion Move ParamSchema and ParamCoercion from com.github.copilot.tool to com.github.copilot.rpc (package-private). Update all from*/fromAsync*/ fromWithToolInvocation*/fromAsyncWithToolInvocation* overloads to delegate to these helpers instead of inline private methods. Remove dead private methods from ToolDefinition: buildSchemaFromParams, schemaForClass, coerceArg, coerceDefaultValue, emptyOptionalOrNull, requireNonNullParam, requireUniqueParamNames. Addresses PR reviewer feedback about unused classes and duplicated logic. * java: cache getConfiguredMapper() in local variable per invocation Capture the singleton ObjectMapper once at the top of each from* factory method instead of calling getConfiguredMapper() multiple times. The lambda closes over the local, making it explicit that schema building, coercion, and result formatting all use the same mapper instance. Addresses Copilot AI reviewer suggestion (discussion r3514594600). --------- Signed-off-by: Ed Burns <edburns@microsoft.com> Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: edburns <75821+edburns@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fixes: #1894
ADR-006: Inline tool definition with lambdas
Context and problem statement
ADR-005 introduced an ergonomic Java tools API based on
@CopilotToolmethod annotations,@CopilotToolParamparameter annotations, andToolDefinition.fromObject(...)for reflection-based tool registration. That model works well when teams define tools as methods on a class.The next ergonomics goal is an inline style comparable to C#
CopilotTool.DefineTool(...), where developers can define a tool at the call site without creating a separate tool container class.For this decision, we evaluated two alternatives:
ToolDefinition.from(tools::setCurrentPhase))ToolDefinition.from(..., phase -> ...))The key factor is metadata quality: tool name, description, parameter names, parameter descriptions, required/default semantics, and schema stability.
Considered options
Option 1: Method-reference API
Example:
In this model, metadata is sourced from existing method-level annotations (
@CopilotTool,@Param) on the referenced method.Advantages:
Drawbacks:
Option 2: Inline lambda API with explicit metadata
Example:
In this model, handler logic is inline, and metadata is provided explicitly through
Param.of(...)parameter definitions.Advantages:
-parametersDrawbacks:
Decision outcome
Chosen: Option 2 for ADR-006 scope — inline lambda API with explicit metadata.
Rationale:
ToolDefinitionconstruction and current invocation semantics.Option 1 remains valuable and can be added independently as a separate ergonomic layer. It is not blocked by this decision.
Design constraints and non-goals
Constraints for the inline lambda API:
RandCompletableFuture<R>).Stringpassthrough,voidmaps to"Success", non-string objects serialized to JSON).ToolDefinitionfields.Non-goals for this ADR:
@CopilotTool/fromObjectAPIs.Consequences
The SDK now provides an explicit inline path for developers who prefer to keep tool declarations at session creation while preserving high-quality schema metadata. Implemented API families include:
ToolDefinition.from(name, description, [params...], handler)— sync handlersToolDefinition.fromAsync(name, description, [params...], asyncHandler)— async handlers returningCompletableFuture<R>ToolDefinition.fromWithToolInvocation(...)— sync withToolInvocationcontext injectionToolDefinition.fromAsyncWithToolInvocation(...)— async withToolInvocationcontext injectionParameter metadata is defined using
Param.of(type, name, description)for required parameters andParam.of(type, name, description, required, defaultValue)for optional parameters with defaults.Fluent option modifiers (
.skipPermission(boolean),.defer(ToolDefer),.overridesBuiltInTool(boolean)) allow post-construction customization.The annotation-driven API from ADR-005 remains the recommended path for larger tool surfaces where co-locating metadata with method implementations improves maintainability. For usage examples and complete API coverage, see the Java SDK README.
Related work items
@CopilotToolergonomics #1792@CopilotToolergonomics: Define tool with lambda #1810