Skip to content

chore: add workflow to close stale issues needing more info#882

Merged
pedrobonamin merged 3 commits into
mainfrom
chore-add-needs-more-info-workflow
May 13, 2026
Merged

chore: add workflow to close stale issues needing more info#882
pedrobonamin merged 3 commits into
mainfrom
chore-add-needs-more-info-workflow

Conversation

@pedrobonamin

Copy link
Copy Markdown
Contributor

Description

Duplicates the needs more info workflows from sanity monorepo sanity-io/sanity#12015

What to review

Testing

@pedrobonamin pedrobonamin requested a review from jordanl17 May 13, 2026 07:43
@pedrobonamin pedrobonamin requested a review from a team as a code owner May 13, 2026 07:43
Copilot AI review requested due to automatic review settings May 13, 2026 07:43
@vercel

vercel Bot commented May 13, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
plugins-studio Ready Ready Preview, Comment May 13, 2026 7:56am

Request Review

@changeset-bot

changeset-bot Bot commented May 13, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 4cd5087

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@socket-security

socket-security Bot commented May 13, 2026

Copy link
Copy Markdown

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Addedgithub/​actions/​stale@​3a9db7e6a41a89f618792c92c0e97cc736e1b13f10010010010080

View full report

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

Adds an automated GitHub Issues follow-up process for issues labeled needs-more-info, including documentation and issue templates to set expectations for reporters.

Changes:

  • Introduces a scheduled actions/stale workflow to warn (after 10 days) and close (after 14 days total) issues needing more info.
  • Adds an issue_comment workflow to remove needs-more-info / stale-warned when the original author responds and post an acknowledgement.
  • Updates contributor docs and adds bug/feature issue templates to describe the follow-up process.

Reviewed changes

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

Show a summary per file
File Description
CONTRIBUTING.md Documents the “needs more info” follow-up and closure timeline.
.github/workflows/stale-issues.yml Scheduled workflow to label/close stale needs-more-info issues and emit a step summary.
.github/workflows/remove-needs-info-on-response.yml Automation to remove labels on author response and post a thank-you comment.
.github/ISSUE_TEMPLATE/feature_request.md Feature request template including follow-up expectations.
.github/ISSUE_TEMPLATE/bug_report.md Bug report template including follow-up expectations and security guidance.
Comments suppressed due to low confidence (1)

.github/workflows/remove-needs-info-on-response.yml:40

  • This removes the needs-more-info label and posts a message implying the requested details were provided on any comment by the issue author. That conflicts with the documented “Need more time?” flow (where the author may be acknowledging they need time but hasn't provided details yet) and may prematurely take the issue out of the needs-more-info queue. Consider either (a) only removing stale-warned on any author response and keeping needs-more-info until a maintainer removes it, or (b) requiring an explicit command/keyword to remove needs-more-info, and tailoring the auto-reply text accordingly.
        run: |
          gh issue edit "$ISSUE_NUMBER" --remove-label "needs-more-info" --repo "$REPO"

          # Remove stale-warned if present (user may have responded after receiving the warning)
          gh issue edit "$ISSUE_NUMBER" --remove-label "stale-warned" --repo "$REPO" 2>/dev/null || true

      - name: Post thank you comment
        run: |
          gh issue comment "$ISSUE_NUMBER" --repo "$REPO" --body \
            "Hi there, thanks for providing the additional information! We've received your update and will review it shortly. We appreciate your help in making Sanity plugins better!"

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

Comment thread .github/workflows/remove-needs-info-on-response.yml Outdated
Comment thread CONTRIBUTING.md
Comment thread .github/workflows/stale-issues.yml
@pedrobonamin pedrobonamin enabled auto-merge (squash) May 13, 2026 08:35
@pedrobonamin pedrobonamin merged commit 3562698 into main May 13, 2026
10 checks passed
@pedrobonamin pedrobonamin deleted the chore-add-needs-more-info-workflow branch May 13, 2026 08:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants