GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — Upgrade @actions/github to v9 and @octokit/core to v7+ by Copilot · Pull Request #708 · actions/github-script

Conversation

Copilot AI commented Mar 23, 2026 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Contributor

Upgrades @actions/github from v6 to v9 and @octokit/core from v5 to v7, which are ESM-only packages requiring TypeScript module resolution changes.

Dependency versions

Package Old New
@actions/github ^6.0.0 ^9.0.0
@octokit/core ^5.0.1 ^7.0.0
@octokit/plugin-request-log ^4.0.0 ^6.0.0
@octokit/plugin-retry ^6.0.1 ^8.0.0

TypeScript config (tsconfig.json)

  • moduleResolution: "bundler" — required for ESM exports map resolution in @octokit/core v7 and @actions/github v9
  • module: "es2022" — required by bundler resolution
  • target: "es2022" — needed for ErrorOptions type used in @octokit/request-error
  • ts-jest configured with module: "commonjs" override to preserve CJS test execution

Import path fixes

  • src/retry-options.ts: @octokit/core/dist-types/types → @octokit/core/types (v7 subpath export)
  • src/async-function.ts: @actions/github/lib/context is no longer in the v9 exports map. Replaced with typeof context derived from @actions/github/lib/utils:
// Before
import {Context} from '@actions/github/lib/context'
import {GitHub} from '@actions/github/lib/utils'
// ...
context: Context

// After
import {context as _ghContext, GitHub} from '@actions/github/lib/utils'
// ...
context: typeof _ghContext

Build script

Updated build:types to pass --target es2022 --module es2022 --moduleResolution bundler since it specifies files directly (bypasses tsconfig.json).


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions
Initial plan
179b645
Copilot AI assigned Copilot and angel-jiakou Mar 23, 2026
Copilot started work on behalf of angel-jiakou March 23, 2026 20:17
View session
Copilot AI linked an issue Mar 23, 2026 that may be closed by this pull request
Upgrade @actions/github to v9 and @octokit/core to v7+ #707
Closed
@angel-jiakou
Upgrade @actions/github to v9, @octokit/core to v7, and related plugins
4bcf639
- @actions/github: ^6.0.0 → ^9.0.0
- @octokit/core: ^5.0.1 → ^7.0.0
- @octokit/plugin-request-log: ^4.0.0 → ^6.0.0
- @octokit/plugin-retry: ^6.0.1 → ^8.0.0
- Update tsconfig.json to use moduleResolution: "bundler" for ESM exports map support
- Update import paths for new package structures
- Update build:types script for compatible compiler options

Co-authored-by: angel-jiakou <[email protected]>
Agent-Logs-Url: https://github.com/actions/github-script/sessions/17de5ca1-8bdc-41e4-a06d-ab2d8c2e6e8c
Copilot AI changed the title [WIP] Upgrade @actions/github to v9 and @octokit/core to v7 Upgrade @actions/github to v9 and @octokit/core to v7+ Mar 23, 2026
Copilot AI requested a review from angel-jiakou March 23, 2026 20:29
Copilot finished work on behalf of angel-jiakou March 23, 2026 20:29
Copilot AI temporarily deployed to debug-integration-test March 23, 2026 20:33 Inactive
@github-actions

github-actions Bot commented Mar 23, 2026 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown

Hello from actions/github-script! (2f6961d)

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@angel-kou
Fix user-agent integration test for orchestration ID
4b873e4
@angel-jiakou
angel-jiakou temporarily deployed to debug-integration-test March 23, 2026 23:08 — with GitHub Actions Inactive
@angel-kou
Update license cache for upgraded dependencies
fa89b8b
@angel-jiakou
angel-jiakou temporarily deployed to debug-integration-test March 23, 2026 23:30 — with GitHub Actions Inactive
@angel-jiakou
angel-jiakou marked this pull request as ready for review March 23, 2026 23:33
@angel-jiakou
angel-jiakou requested a review from a team as a code owner March 23, 2026 23:33
Copilot AI review requested due to automatic review settings March 23, 2026 23:33
Copilot started reviewing on behalf of angel-jiakou March 23, 2026 23:33
View session
angel-jiakou
angel-jiakou approved these changes Mar 23, 2026
View reviewed changes
Copilot AI reviewed Mar 23, 2026
View reviewed changes

Copilot AI left a comment

Copy link
Copy Markdown

There was a problem hiding this comment.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Upgrades the Action’s GitHub/Octokit dependencies to ESM-only major versions and updates the TypeScript + test/build configuration and import paths to remain compatible.

Changes:

  • Upgrade @actions/github to v9 and @octokit/* dependencies to their v6/v7/v8 major versions.
  • Update TypeScript compilation settings for ESM (target/module es2022, moduleResolution: bundler) and adjust ts-jest to keep tests running in CJS.
  • Fix type/import paths impacted by the new packages’ exports maps and relax integration workflow user-agent assertions.

Reviewed changes

Copilot reviewed 28 out of 34 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
types/async-function.d.ts Updates exported argument types to avoid removed @actions/github/lib/context path.
tsconfig.json Switches project TS target/module to ES2022 and module resolution to bundler.
src/retry-options.ts Updates Octokit type import to the new v7 exported subpath.
src/async-function.ts Adjusts context typing approach to match @actions/github v9 export map changes.
package.json Bumps deps, updates build:types, and adds ts-jest TS overrides.
package-lock.json Lockfile updates for the new dependency graph and transitive packages.
.licenses/npm/wrappy.dep.yml Removes cached license entry (dependency no longer in licensed set).
.licenses/npm/universal-user-agent.dep.yml Updates cached license metadata version for transitive bump.
.licenses/npm/undici-6.24.1.dep.yml Corrects cached license metadata to match [email protected].
.licenses/npm/undici-5.28.5.dep.yml Adds cached license metadata for [email protected].
.licenses/npm/once.dep.yml Removes cached license entry (dependency no longer in licensed set).
.licenses/npm/json-with-bigint.dep.yml Adds cached license metadata for new transitive dependency.
.licenses/npm/fast-content-type-parse.dep.yml Adds cached license metadata for new transitive dependency.
.licenses/npm/deprecation.dep.yml Removes cached license entry (dependency removed from graph).
.licenses/npm/before-after-hook.dep.yml Updates cached license metadata version for transitive bump.
.licenses/npm/@octokit/types.dep.yml Updates cached license metadata version for transitive bump.
.licenses/npm/@octokit/types-12.0.0.dep.yml Removes cached license entry for superseded version.
.licenses/npm/@octokit/request.dep.yml Updates cached license metadata version for transitive bump.
.licenses/npm/@octokit/request-error.dep.yml Updates cached license metadata version for transitive bump.
.licenses/npm/@octokit/plugin-retry.dep.yml Updates cached license metadata version for transitive bump.
.licenses/npm/@octokit/plugin-rest-endpoint-methods.dep.yml Updates cached license metadata version for transitive bump.
.licenses/npm/@octokit/plugin-request-log.dep.yml Updates cached license metadata version for transitive bump.
.licenses/npm/@octokit/plugin-paginate-rest.dep.yml Updates cached license metadata version for transitive bump.
.licenses/npm/@octokit/openapi-types.dep.yml Updates cached license metadata version for transitive bump.
.licenses/npm/@octokit/openapi-types-19.0.0.dep.yml Removes cached license entry for superseded version.
.licenses/npm/@octokit/graphql.dep.yml Updates cached license metadata version for transitive bump.
.licenses/npm/@octokit/endpoint.dep.yml Updates cached license metadata version for transitive bump.
.licenses/npm/@octokit/core.dep.yml Updates cached license metadata version for direct dependency bump.
.licenses/npm/@octokit/auth-token.dep.yml Updates cached license metadata version for transitive bump.
.licenses/npm/@actions/http-client-3.0.2.dep.yml Adds cached license metadata for new transitive version.
.licenses/npm/@actions/http-client-2.2.0.dep.yml Adds cached license metadata for existing transitive version.
.licenses/npm/@actions/github.dep.yml Updates cached license metadata version for direct dependency bump.
.github/workflows/integration.yml Adjusts user-agent integration assertions to allow the new user-agent format.

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

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions
Comment thread src/async-function.ts Outdated
Comment on lines 3 to 4
import {context as _ghContext, GitHub} from '@actions/github/lib/utils'
import * as glob from '@actions/glob'

Copilot AI Mar 23, 2026

Copy link

There was a problem hiding this comment.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@actions/github/lib/utils is a deep import that isn’t part of the stable public API surface and has already changed once (lib/context disappeared in v9). Since this import is only used to derive types, prefer using public exports from @actions/github (e.g., typeof context for context and ReturnType<typeof getOctokit> for github/octokit) and make the import type-only so it can’t accidentally become a runtime dependency.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions
Copilot uses AI. Check for mistakes.

@angel-jiakou angel-jiakou Mar 26, 2026

Copy link
Copy Markdown

There was a problem hiding this comment.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this comment is valid so I'm gonna make these public exports in case this import changes again.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions

@angel-jiakou angel-jiakou Mar 26, 2026

Copy link
Copy Markdown

There was a problem hiding this comment.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@heavymachinery

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions
@angel-kou
Address comment
c1a9577
@angel-jiakou
angel-jiakou temporarily deployed to debug-integration-test March 26, 2026 22:05 — with GitHub Actions Inactive
@angel-jiakou
angel-jiakou requested a review from danwkennedy March 26, 2026 22:08
danwkennedy
danwkennedy reviewed Mar 27, 2026
View reviewed changes

@danwkennedy danwkennedy left a comment

Copy link
Copy Markdown

There was a problem hiding this comment.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks fine so far but I'd upgrade all the Actions packages instead of just a few.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions
Comment thread package.json
@@ -40,12 +45,12 @@
"dependencies": {
"@actions/core": "^1.10.1",

@danwkennedy danwkennedy Mar 27, 2026

Copy link
Copy Markdown

There was a problem hiding this comment.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All the @actions/* packages have been upgraded, I would pick up those new versions as well.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions
ericsciple
ericsciple requested changes Apr 2, 2026
View reviewed changes

@ericsciple ericsciple left a comment •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown

There was a problem hiding this comment.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hold — requires changes in main first to add a wrapper for getOctokit

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions
@salmanmkc salmanmkc mentioned this pull request Apr 7, 2026
feat(github): add orchestration ID to user-agent in getOctokitOptions actions/toolkit#2364
Merged
salmanmkc added a commit that referenced this pull request Apr 9, 2026
@salmanmkc
merge: combine v9 upgrade (PR #708) with createOctokit feature
c769f41
Merges the @actions/github v9 upgrade branch into the createOctokit
feature branch, combining ADR steps 1 and 3 into a single major release.

Changes from v9 upgrade:
- @actions/github 6.x → 9.0.0 (ESM-only)
- @octokit/core v5→v7, plugin-request-log v4→v6, plugin-retry v6→v8
- tsconfig: moduleResolution bundler, module/target es2022
- ts-jest: module commonjs override for CJS test execution
- License file updates for new dependency versions

Conflict resolutions:
- src/async-function.ts: v9 import style + createOctokit type
- types/async-function.d.ts: aligned with .ts file
- integration.yml: v9 UA checks + createOctokit test job
- dist/index.js: rebuilt via ncc

Additional fix:
- getoctokit-integration.test.ts: use mock instead of direct
  @actions/github import (ESM-only v9 incompatible with Jest CJS)

All 32 tests passing, TypeScript clean, lint clean.
@salmanmkc salmanmkc closed this Apr 10, 2026
@salmanmkc

salmanmkc commented Apr 10, 2026

Copy link
Copy Markdown
Contributor

Closing in favour of #700

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@coderabbitai coderabbitai Bot mentioned this pull request Jul 30, 2026
ci: redesign CI authorization and build pipeline for master openshift-service-mesh/proxy#323
Merged

This branch was previously deployed

1 inactive deployment
debug-integration-test — c1a95771 Deployed Mar 26, 2026 by angel-jiakou via Integration test: debug option (runner.debug mode enabled) #602
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@angel-jiakou angel-jiakou angel-jiakou approved these changes
Copilot code review Copilot
Copilot review effort
Applies to this pull request for everyone.Learn more about Copilot code review.
Copilot left review comments
@ericsciple ericsciple ericsciple requested changes
+1 more reviewer
@danwkennedy danwkennedy danwkennedy left review comments
Reviewers whose approvals may not affect merge requirements

Uh oh!

There was an error while loading. Please reload this page.

Assignees

@angel-jiakou angel-jiakou

Copilot code review Copilot

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Upgrade @actions/github to v9 and @octokit/core to v7+

Uh oh!

There was an error while loading. Please reload this page.

7 participants

@salmanmkc @danwkennedy @ericsciple @angel-jiakou @angel-kou
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.