Skip to content

Commit b49e5d8

Browse files
patnikoCopilot
andcommitted
fix: remove add-comment from runtime triage to prevent code leaks
The workflow was posting detailed analysis (including runtime code snippets) as comments on public SDK issues. Remove the add-comment safe-output and corresponding prompt instructions so the workflow only labels the source issue and creates issues/PRs in the runtime repo. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
1 parent c13dbba commit b49e5d8

File tree

2 files changed

+6
-51
lines changed

2 files changed

+6
-51
lines changed

.github/workflows/cross-repo-issue-analysis.lock.yml

Lines changed: 4 additions & 45 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

.github/workflows/cross-repo-issue-analysis.md

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -33,9 +33,6 @@ tools:
3333
safe-outputs:
3434
github-token: ${{ secrets.RUNTIME_TRIAGE_TOKEN }}
3535
allowed-github-references: ["repo", "github/copilot-agent-runtime"]
36-
add-comment:
37-
max: 1
38-
target: triggering
3936
add-labels:
4037
allowed: [runtime-fix-needed, sdk-fix-only, needs-investigation]
4138
max: 3
@@ -100,7 +97,7 @@ Common areas where runtime fixes are needed:
10097

10198
Classify the issue into one of these categories:
10299

103-
1. **SDK-fix-only**: The bug/feature is entirely in the SDK code. Label the issue `sdk-fix-only` and comment with your analysis.
100+
1. **SDK-fix-only**: The bug/feature is entirely in the SDK code. Label the issue `sdk-fix-only`.
104101

105102
2. **Runtime-fix-needed**: The root cause is in copilot-agent-runtime. Do ALL of the following:
106103
- Label the original issue `runtime-fix-needed`
@@ -113,9 +110,8 @@ Classify the issue into one of these categories:
113110
- Make the minimal, targeted code changes needed
114111
- Include a clear PR description linking back to both issues
115112
- If you're uncertain about the fix, still create the PR as a starting point for discussion
116-
- Comment on the original SDK issue summarizing your findings and linking to the new runtime issue and PR
117113

118-
3. **Needs-investigation**: You cannot confidently determine the root cause. Label the issue `needs-investigation` and comment explaining what you found and what needs further human review.
114+
3. **Needs-investigation**: You cannot confidently determine the root cause. Label the issue `needs-investigation`.
119115

120116
## Guidelines
121117

0 commit comments

Comments
 (0)