Skip to content

[review] Migrate set_issue_type safe output from GraphQL to single REST issues.update call#41284

Merged
pelikhan merged 12 commits into
mainfrom
copilot/migrate-set-issue-type-handler-review-1782325463724
Jun 24, 2026
Merged

[review] Migrate set_issue_type safe output from GraphQL to single REST issues.update call#41284
pelikhan merged 12 commits into
mainfrom
copilot/migrate-set-issue-type-handler-review-1782325463724

Conversation

@github-actions

@github-actions github-actions Bot commented Jun 24, 2026

Copy link
Copy Markdown
Contributor

Migrate set_issue_type safe output handler: GraphQL → REST issues.update

Summary

Migrates the set_issue_type safe-output handler from a GraphQL mutation to a single REST call (issues.update), preserving all existing validation, filtering, and error-mapping behaviour.

The review branch introduces one new file (a patch-level changeset entry); the full handler migration is carried in the branch commit history.


What changed

Area Detail
Handler set_issue_type now calls the REST issues.update endpoint instead of the GraphQL mutation
Error mapping REST error data parsing refactored; status type checks tightened to ensure canonical error types are returned
Type canonicalisation Typing stabilised so the handler output shape is identical to the previous GraphQL path
Tests Test suite updated to exercise the REST-based code path; naming polished for clarity
Changeset patch-migrate-set-issue-type-rest.md added to record the release-level change

Behaviour preserved

  • Input validation and filtering are unchanged
  • Error messages visible to callers remain the same
  • Output shape / safe-output contract is unmodified

Changeset

.changeset/patch-migrate-set-issue-type-rest.md  (added — patch)

Type: patch (non-breaking refactor)
Breaking: no


Commit sequence

a5bb69df fix: migrate set_issue_type handler to REST issues.update
81049b82 test: update set_issue_type tests for REST issue type updates
d90d57f7 Fix set_issue_type REST error mapping and type canonicalization
d12e2776 Address remaining set_issue_type review follow-ups
0d8de742 Stabilize set_issue_type typing and error parsing
2990c7ef Refactor set_issue_type REST error data parsing
e90bb429 Tighten status type checks in set_issue_type
5ad89841 Polish set_issue_type naming and tests
ac755f5a Add changeset

Generated by PR Description Updater for #41284 · 50.4 AIC · ⌖ 9.7 AIC · ⊞ 4.6K ·

Copilot AI and others added 11 commits June 24, 2026 15:24
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Base automatically changed from copilot/migrate-set-issue-type-handler to main June 24, 2026 18:32
@github-actions

Copy link
Copy Markdown
Contributor Author

Rebase onto main, then rerun the safe-output checks.

Generated by 👨‍🍳 PR Sous Chef · 66.5 AIC · ⌖ 1.12 AIC · ⊞ 17.6K ·

@pelikhan pelikhan merged commit 273587e into main Jun 24, 2026
@pelikhan pelikhan deleted the copilot/migrate-set-issue-type-handler-review-1782325463724 branch June 24, 2026 19:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants