Skip to content

[WIP] Upgrade go-github to v 0.86#2452

Draft
iulia-b wants to merge 7 commits into
github:mainfrom
iulia-b:iunia/update-go-github-version
Draft

[WIP] Upgrade go-github to v 0.86#2452
iulia-b wants to merge 7 commits into
github:mainfrom
iulia-b:iunia/update-go-github-version

Conversation

@iulia-b
Copy link
Copy Markdown
Contributor

@iulia-b iulia-b commented May 11, 2026

Summary

Update google/go-github from v82 to v86, fixing all breaking API changes across the four major versions.

Why

Fixes #
Keeping dependencies up to date for bug fixes, security patches, and new API support. v83–v86 include security improvements (cross-host redirect
rejection, URL path traversal protection, webhook payload size limits) and new GitHub API coverage.

What changed

  • Updated google/go-github from v82 to v86 across all Go files and go.mod
  • pkg/raw/raw.go: Pass context.Context to NewRequest (v86 refactor), remove redundant WithContext
  • pkg/github/actions.go: Handle new *WorkflowDispatchRunDetails return value from CreateWorkflowDispatchEventByID/ByFileName (v84)
  • pkg/github/issues.go: Replace removed IssueListOptions with ListOptions for SubIssue.ListByIssue (v84)
  • pkg/github/notifications.go: MarkThreadDone now takes string instead of int64 (v85) — removed unnecessary strconv.ParseInt conversion and unused
    strconv import
  • pkg/github/notifications_test.go: Removed invalid threadID format test case (no longer applicable)
  • pkg/github/projects.go: ListProjectsPaginationOptions fields changed from pointers to values (v83) — updated Query, After, Before, PerPage usage;
    removed unused queryPtr variable
  • e2e/e2e_test.go: Removed nil opts arg from ListReviewers call (v83)

MCP impact

  • No tool or API changes
  • Tool schema or behavior changed
  • New tool added

Prompts tested (tool changes only)

Security / limits

  • No security or limits impact
  • Auth / permissions considered
  • Data exposure, filtering, or token/size limits considered

Tool renaming

  • I am renaming tools as part of this PR (e.g. a part of a consolidation effort)
    • I have added the new tool aliases in deprecated_tool_aliases.go
  • I am not renaming tools as part of this PR

Note: if you're renaming tools, you must add the tool aliases. For more information on how to do so, please refer to the official docs.

Lint & tests

  • Linted locally with ./script/lint
  • Tested locally with ./script/test

Docs

  • Not needed
  • Updated (README / docs / examples)

@iulia-b iulia-b changed the title upgrade to v 0.86 [WIP] Upgrade go-github to v 0.86 May 11, 2026
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.

2 participants