Skip to content

Fix contributor check gh api marker argument error#2151

Merged
aaronpowell merged 4 commits into
mainfrom
aaronpowell-fix-reputation-comment-updates
Jun 29, 2026
Merged

Fix contributor check gh api marker argument error#2151
aaronpowell merged 4 commits into
mainfrom
aaronpowell-fix-reputation-comment-updates

Conversation

@aaronpowell

Copy link
Copy Markdown
Contributor

Pull Request Checklist

  • I have read and followed the CONTRIBUTING.md guidelines.
  • I have read and followed the Guidance for submissions involving paid services.
  • My contribution adds a new instruction, prompt, agent, skill, workflow, or canvas extension file in the correct directory.
  • The file follows the required naming convention.
  • The content is clearly structured and follows the example format.
  • I have tested my instructions, prompt, agent, skill, workflow, or canvas extension with GitHub Copilot.
  • I have run npm start and verified that README.md is up to date.
  • I am targeting the main branch for this pull request.

Description

The contributor reputation check workflow failed in Sync risk comment because gh api does not support --arg. That caused the step to exit before it could update or dedupe the marker comment.

This update moves marker parameterization to jq (jq -r --arg marker ...) while keeping the same comment matching logic.


Type of Contribution

  • New instruction file.
  • New prompt file.
  • New agent file.
  • New plugin.
  • New skill file.
  • New agentic workflow.
  • New canvas extension.
  • Update to existing instruction, prompt, agent, plugin, skill, workflow, or canvas extension.
  • Other (please specify):

Additional Notes

N/A


By submitting this pull request, I confirm that my contribution abides by the Code of Conduct and will be licensed under the MIT License.

aaronpowell and others added 2 commits June 29, 2026 10:11
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings June 29, 2026 02:13
@github-actions github-actions Bot added the workflow PR touches workflow automation label Jun 29, 2026
GitHub Advanced Security started work on behalf of aaronpowell June 29, 2026 02:14 View session
GitHub Advanced Security finished work on behalf of aaronpowell June 29, 2026 02:14

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Fixes the “Sync risk comment” step in the contributor reputation check workflow by removing the unsupported gh api --arg usage and moving marker parameterization into jq.

Changes:

  • Replace gh api --jq ... --arg ... with a jq --arg ... pipeline to avoid gh api marker-argument failures.
  • Add duplicate-marker cleanup logic to delete extra matching comments and keep one canonical thread.
Show a summary per file
File Description
.github/workflows/contributor-check.yml Updates the risk comment sync logic to use jq --arg and adds duplicate comment cleanup.

Review details

  • Files reviewed: 1/1 changed files
  • Comments generated: 1
  • Review effort level: Low

Comment thread .github/workflows/contributor-check.yml Outdated
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings June 29, 2026 02:22
GitHub Advanced Security started work on behalf of aaronpowell June 29, 2026 02:22 View session
GitHub Advanced Security finished work on behalf of aaronpowell June 29, 2026 02:23
@aaronpowell aaronpowell merged commit 7f19be7 into main Jun 29, 2026
11 of 12 checks passed
@aaronpowell aaronpowell deleted the aaronpowell-fix-reputation-comment-updates branch June 29, 2026 02:23
GitHub Advanced Security started work on behalf of aaronpowell June 29, 2026 02:23 View session
GitHub Advanced Security finished work on behalf of aaronpowell June 29, 2026 02:23

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Review details

Comments suppressed due to low confidence (1)

.github/workflows/contributor-check.yml:204

  • The comment here says it is keeping one canonical comment thread, but the loop below deletes all matching marker comments when risk drops below MEDIUM. This is a documentation mismatch that could confuse future maintenance.
              # Keep one canonical comment thread by removing all matching comments
              # when risk drops below MEDIUM.
  • Files reviewed: 1/1 changed files
  • Comments generated: 0 new
  • Review effort level: Low

Copilot AI review requested due to automatic review settings June 29, 2026 02:24

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Review details

  • Files reviewed: 1/1 changed files
  • Comments generated: 0 new
  • Review effort level: Low

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

Labels

workflow PR touches workflow automation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants