Skip to content

[WIP] Refactor repeated safe-output handler scaffolding across JS handlers#25483

Closed
Copilot wants to merge 1 commit intomainfrom
copilot/refactor-duplicate-output-handlers
Closed

[WIP] Refactor repeated safe-output handler scaffolding across JS handlers#25483
Copilot wants to merge 1 commit intomainfrom
copilot/refactor-duplicate-output-handlers

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 9, 2026

  • Analyze all 35 count-gated handlers and categorize by increment position
  • Create actions/setup/js/handler_factory.cjs with createCountGatedHandler utility
  • Create actions/setup/js/handler_factory.test.cjs with focused max-limit tests (7 tests)
  • Migrate cohort 1: add_labels, remove_labels
  • Migrate cohort 2: assign_to_user, unassign_from_user
  • Migrate remaining 20 handlers: close_issue, close_pull_request, close_discussion, add_reviewer, dispatch_workflow, hide_comment, call_workflow, add_comment, create_pull_request, autofix_code_scanning_alert, assign_milestone, create_project, create_project_status_update, create_discussion, link_sub_issue, mark_pull_request_as_ready_for_review, resolve_pr_review_thread, set_issue_type, push_to_pull_request_branch, missing_data
  • Run make fmt-cjs to format
  • Verify all existing handler tests still pass (773+ tests across all migrated handlers)
  • Fix 3 stale comments flagged by code review (non-blocking)

@pelikhan
Copy link
Copy Markdown
Collaborator

pelikhan commented Apr 9, 2026

@copilot finish work and push changes

Copilot AI requested a review from pelikhan April 9, 2026 15:05
@pelikhan pelikhan closed this Apr 9, 2026
Copilot stopped work on behalf of pelikhan due to an error April 9, 2026 15:28
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.

Duplicate code: repeated safe-output handler scaffolding across JS handlers

2 participants