Fix weekly upstream sync agentic workflow prompt#50
Conversation
Rewrite the prompt from documentation-style to clear imperative instructions. The agent previously misinterpreted the documentation as instructions to perform the merge itself (invoking the agentic-merge-upstream skill), then failed because it couldn't push code from the sandbox. The new prompt: - Gives explicit step-by-step instructions - Provides bash commands for upstream change detection - Directs the agent to use safe-output tools (create_issue, close_issue, assign_to_agent, noop) - Explicitly prohibits invoking skills, editing files, or pushing code Only the markdown content changed; frontmatter is unchanged so no lock file recompilation is needed. Agent-Logs-Url: https://github.com/github/copilot-sdk-java/sessions/945e9454-9363-45bc-b281-0fe3d6c87019 Co-authored-by: brunoborges <129743+brunoborges@users.noreply.github.com>
There was a problem hiding this comment.
Pull request overview
Updates the agentic workflow prompt for weekly upstream sync checks so the automation agent reliably detects upstream changes and reports results via safe-output tools (without attempting merges).
Changes:
- Rewrites the prompt from descriptive documentation into imperative agent instructions.
- Adds explicit constraints to prevent invoking skills, editing files, or pushing code.
- Specifies concrete commands/steps for comparing
.lastmergeto upstream and creating/closing issues via safe outputs.
Show a summary per file
| File | Description |
|---|---|
| .github/workflows/weekly-upstream-sync.md | Reworks the workflow prompt into actionable instructions and adds guardrails for safe-output-only reporting. |
Copilot's findings
- Files reviewed: 1/1 changed files
- Comments generated: 2
| │ port changes → PR │ | ||
| └──────────────────────────┘ | ||
| ``` | ||
| Follow the [agentic-merge-upstream](.github/prompts/agentic-merge-upstream.prompt.md) prompt to port these changes to the Java SDK. |
There was a problem hiding this comment.
The link target (.github/prompts/agentic-merge-upstream.prompt.md) is relative to this file’s directory, so on GitHub it resolves to .github/workflows/.github/prompts/... and is broken. Use a correct relative path (e.g., ../prompts/agentic-merge-upstream.prompt.md) or an absolute GitHub URL so the referenced prompt can be opened reliably from the created issue body/instructions.
| Follow the [agentic-merge-upstream](.github/prompts/agentic-merge-upstream.prompt.md) prompt to port these changes to the Java SDK. | |
| Follow the [agentic-merge-upstream](../prompts/agentic-merge-upstream.prompt.md) prompt to port these changes to the Java SDK. |
| ``` | ||
| Follow the [agentic-merge-upstream](.github/prompts/agentic-merge-upstream.prompt.md) prompt to port these changes to the Java SDK. | ||
| ``` | ||
| 4. After creating the issue, use the `assign_to_agent` safe-output tool to assign Copilot to the newly created issue. |
There was a problem hiding this comment.
Step 3b instructs calling assign_to_agent but doesn’t specify which identifier to pass. The safe-output assign_to_agent tool requires one of issue_number or pull_number (no default in this workflow), so the instructions should explicitly say to use the created issue’s number (from create_issue output) or to set a temporary_id on create_issue and pass that to assign_to_agent.
| 4. After creating the issue, use the `assign_to_agent` safe-output tool to assign Copilot to the newly created issue. | |
| 4. After creating the issue, use the `assign_to_agent` safe-output tool to assign Copilot to the newly created issue, passing the created issue's number from the `create_issue` output as `issue_number`. |
|
Net effect @brunoborges you are clear to merge this. |
Resolves #41
Before the change?
weekly-upstream-sync.mdprompt was written as documentation ("This document describes the workflow..."), not as agent instructions. The agent misinterpreted it, invoked theagentic-merge-upstreamskill, attempted to actually merge upstream changes, failed to push from the AWF sandbox (no git credentials), and never called any safe-output tools — causing the workflow to fail with no output.After the change?
.lastmerge, use safe-output tools (create_issue,close_issue,assign_to_agent,noop) to report results.{{#runtime-import}}).Pull request checklist
mvn spotless:applyhas been run to format the codemvn clean verifypasses locallyDoes this introduce a breaking change?