Skip to content

Harden gardener investigation workflow#1449

Merged
byrichardpowell merged 1 commit into
mainfrom
harden-gardener-investigate-report-only
Jun 15, 2026
Merged

Harden gardener investigation workflow#1449
byrichardpowell merged 1 commit into
mainfrom
harden-gardener-investigate-report-only

Conversation

@byrichardpowell

Copy link
Copy Markdown
Contributor

Summary

  • Make the Gardener issue investigation workflow read-only/report-only.
  • Reduce GITHUB_TOKEN permissions from write-capable scopes to read-only scopes.
  • Remove write-capable Claude tools (Edit, Write, git push, git commit, gh pr create, etc.).
  • Update the investigation skill to always return a report and describe proposed fixes instead of implementing them.

Test plan

  • git diff --check
  • Parsed .github/workflows/gardener-investigate-issue.yml with yq

@github-actions github-actions Bot added the devtools-gardener Post the issue or PR to Slack for the gardener label Jun 15, 2026

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR hardens the “Gardener - Investigate Issue” GitHub Actions workflow and the associated investigating-github-issues skill to be strictly report-only and read-only, reducing the risk of unintended repository modifications during automated investigations.

Changes:

  • Reduced workflow GITHUB_TOKEN permissions from write scopes to read-only.
  • Removed write-capable Claude tools (edit/write, commit/push/PR creation) from the workflow and skill allowlists.
  • Updated the investigation skill and workflow prompt/schema to always return an investigation report and only describe proposed fixes.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
.github/workflows/gardener-investigate-issue.yml Drops write permissions and restricts Claude tools/prompt to report-only behavior.
.claude/skills/investigating-github-issues/SKILL.md Reframes the skill as read-only and removes “fix PR” behavior in favor of proposed-fix reporting.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines 15 to +18
permissions:
contents: write
contents: read
issues: read
pull-requests: write
pull-requests: read
Comment on lines 96 to +97
gh release list --limit 10
git tag -l 'v*' | sort -V | tail -10
git tag -l 'v*'
@byrichardpowell byrichardpowell merged commit 9d5f4b6 into main Jun 15, 2026
9 checks passed
@byrichardpowell byrichardpowell deleted the harden-gardener-investigate-report-only branch June 15, 2026 15:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

devtools-gardener Post the issue or PR to Slack for the gardener

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants