Skip to content

ci: automatically close new empty issues - #407

Merged
bboozzoo merged 3 commits into
mainfrom
ci/close-empty-issues
Sep 15, 2026
Merged

bboozzoo merged 3 commits into
mainfrom
ci/close-empty-issues

Conversation

@raimundo-henriques

Copy link
Copy Markdown
Contributor

No description provided.

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.

🟡 Changes recommended

Remove the trailing + so the workflow script parses successfully.

Get a fresh assessment by requesting another Copilot review.

Pull request overview

Adds a GitHub Actions workflow to automatically close newly opened empty issues.

Changes:

  • Detects untouched default issue submissions.
  • Closes matching issues and posts an explanatory comment.
File summaries
File Summary
.github/workflows/automatic-issue-checks.yml Adds empty-issue automation. Critical finding (3 votes): a trailing + makes the embedded JavaScript invalid.
Review details
  • Files reviewed: 1/1 changed files
  • Comments generated: 1
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread .github/workflows/automatic-issue-checks.yml
Comment thread .github/workflows/automatic-issue-checks.yml
Comment thread .github/workflows/automatic-issue-checks.yml

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.

🔵 Needs a closer look

Approval readiness is not unanimous; final human review is warranted.

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

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.

🔵 Needs a closer look

Fetch the current issue before evaluating and closing it to avoid closing issues populated after creation.

Review details

Suppressed comments (1)

.github/workflows/automatic-issue-checks.yml:18

  • Because this workflow is triggered by issues.opened, context.payload.issue is only the event-time snapshot. If a reporter fills in the body before the job starts, this predicate still sees the stale empty template and closes the now-populated issue. Fetch the current issue with github.rest.issues.get before applying the predicate (and closing it).
            const issue = context.payload.issue;
  • Files reviewed: 1/1 changed files
  • Comments generated: 0 new
  • Review effort level: Lite

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.

🟢 Approval recommended

The reviewed workflow changes have no unresolved blocking issues.

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

@bboozzoo bboozzoo 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.

Thanks

@bboozzoo
bboozzoo merged commit 2039285 into main Sep 15, 2026
2 checks passed
@bboozzoo
bboozzoo deleted the ci/close-empty-issues branch September 15, 2026 14:26
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