release-please 17.4.1 → 17.5.1

package/build/src/github.js

@@ -13,23 +13,17 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.sleepInMs = exports.GitHub = exports.GH_GRAPHQL_URL = exports.GH_API_URL = void 0;
-const code_suggester_1 = require("code-suggester");
-const rest_1 = require("@octokit/rest");
-const request_1 = require("@octokit/request");
-const graphql_1 = require("@octokit/graphql");
+exports.sleepInMs = exports.GitHub = void 0;
 const request_error_1 = require("@octokit/request-error");
+const code_suggester_1 = require("code-suggester");
 const errors_1 = require("./errors");
 const MAX_ISSUE_BODY_SIZE = 65536;
 const MAX_SLEEP_SECONDS = 20;
-exports.GH_API_URL = 'https://api.github.com';
-exports.GH_GRAPHQL_URL = 'https://api.github.com';
 const logger_1 = require("./util/logger");
 const manifest_1 = require("./manifest");
+const github_api_1 = require("./github-api");
 const signoff_commit_message_1 = require("./util/signoff-commit-message");
 const git_file_utils_1 = require("@google-automations/git-file-utils");
-const https_proxy_agent_1 = require("https-proxy-agent");
-const http_proxy_agent_1 = require("http-proxy-agent");
 const composite_1 = require("./updaters/composite");
 class GitHub {
     constructor(options) {
@@ -133,130 +127,6 @@ class GitHub {
             this.logger.debug(`finding files by glob: ${glob}, ref: ${ref}, prefix: ${prefix}`);
             return await this.fileCache.findFilesByGlob(glob, ref, prefix);
         });
-        /**
-         * Open a pull request
-         *
-         * @param {PullRequest} pullRequest Pull request data to update
-         * @param {string} targetBranch The base branch of the pull request
-         * @param {string} message The commit message for the commit
-         * @param {Update[]} updates The files to update
-         * @param {CreatePullRequestOptions} options The pull request options
-         * @throws {GitHubAPIError} on an API error
-         */
-        this.createPullRequest = wrapAsync(async (pullRequest, targetBranch, message, updates, options) => {
-            //  Update the files for the release if not already supplied
-            const changes = await this.buildChangeSet(updates, targetBranch);
-            const prNumber = await (0, code_suggester_1.createPullRequest)(this.octokit, changes, {
-                upstreamOwner: this.repository.owner,
-                upstreamRepo: this.repository.repo,
-                title: pullRequest.title,
-                branch: pullRequest.headBranchName,
-                description: pullRequest.body,
-                primary: targetBranch,
-                force: true,
-                fork: !!(options === null || options === void 0 ? void 0 : options.fork),
-                message,
-                logger: this.logger,
-                draft: !!(options === null || options === void 0 ? void 0 : options.draft),
-                labels: pullRequest.labels,
-            });
-            if (prNumber === 0) {
-                this.logger.warn('no code changes detected, skipping pull request creation');
-                return {
-                    headBranchName: pullRequest.headBranchName,
-                    baseBranchName: targetBranch,
-                    number: 0,
-                    title: pullRequest.title,
-                    body: pullRequest.body,
-                    labels: pullRequest.labels,
-                    files: [],
-                };
-            }
-            return await this.getPullRequest(prNumber);
-        });
-        /**
-         * Fetch a pull request given the pull number
-         * @param {number} number The pull request number
-         * @returns {PullRequest}
-         */
-        this.getPullRequest = wrapAsync(async (number) => {
-            const response = await this.octokit.pulls.get({
-                owner: this.repository.owner,
-                repo: this.repository.repo,
-                pull_number: number,
-            });
-            return {
-                headBranchName: response.data.head.ref,
-                baseBranchName: response.data.base.ref,
-                number: response.data.number,
-                title: response.data.title,
-                body: response.data.body || '',
-                files: [],
-                labels: response.data.labels
-                    .map(label => label.name)
-                    .filter(name => !!name),
-            };
-        });
-        /**
-         * Update a pull request's title and body.
-         * @param {number} number The pull request number
-         * @param {ReleasePullRequest} releasePullRequest Pull request data to update
-         * @param {string} targetBranch The target branch of the pull request
-         * @param {string} options.signoffUser Optional. Commit signoff message
-         * @param {boolean} options.fork Optional. Whether to open the pull request from
-         *   a fork or not. Defaults to `false`
-         * @param {PullRequestOverflowHandler} options.pullRequestOverflowHandler Optional.
-         *   Handles extra large pull request body messages.
-         */
-        this.updatePullRequest = wrapAsync(async (number, releasePullRequest, targetBranch, options) => {
-            //  Update the files for the release if not already supplied
-            const changes = await this.buildChangeSet(releasePullRequest.updates, targetBranch);
-            let message = releasePullRequest.title.toString();
-            if (options === null || options === void 0 ? void 0 : options.signoffUser) {
-                message = (0, signoff_commit_message_1.signoffCommitMessage)(message, options.signoffUser);
-            }
-            const title = releasePullRequest.title.toString();
-            const body = ((options === null || options === void 0 ? void 0 : options.pullRequestOverflowHandler)
-                ? await options.pullRequestOverflowHandler.handleOverflow(releasePullRequest)
-                : releasePullRequest.body)
-                .toString()
-                .slice(0, MAX_ISSUE_BODY_SIZE);
-            const prNumber = await (0, code_suggester_1.createPullRequest)(this.octokit, changes, {
-                upstreamOwner: this.repository.owner,
-                upstreamRepo: this.repository.repo,
-                title,
-                branch: releasePullRequest.headRefName,
-                description: body,
-                primary: targetBranch,
-                force: true,
-                fork: (options === null || options === void 0 ? void 0 : options.fork) === false ? false : true,
-                message,
-                logger: this.logger,
-                draft: releasePullRequest.draft,
-            });
-            if (prNumber !== number) {
-                this.logger.warn(`updated code for ${prNumber}, but update requested for ${number}`);
-            }
-            const response = await this.octokit.pulls.update({
-                owner: this.repository.owner,
-                repo: this.repository.repo,
-                pull_number: number,
-                title: releasePullRequest.title.toString(),
-                body,
-                state: 'open',
-            });
-            return {
-                headBranchName: response.data.head.ref,
-                baseBranchName: response.data.base.ref,
-                number: response.data.number,
-                title: response.data.title,
-                body: response.data.body || '',
-                files: [],
-                labels: response.data.labels
-                    .map(label => label.name)
-                    .filter(name => !!name),
-            };
-        });
         /**
          * Returns a list of paths to all files with a given file
          * extension.
@@ -277,213 +147,27 @@ class GitHub {
             }
             return this.fileCache.findFilesByExtension(extension, ref, prefix);
         });
-        /**
-         * Create a GitHub release
-         *
-         * @param {Release} release Release parameters
-         * @param {ReleaseOptions} options Release option parameters
-         * @throws {DuplicateReleaseError} if the release tag already exists
-         * @throws {GitHubAPIError} on other API errors
-         */
-        this.createRelease = wrapAsync(async (release, options = {}) => {
-            if (options.forceTag) {
-                try {
-                    await this.octokit.git.createRef({
-                        owner: this.repository.owner,
-                        repo: this.repository.repo,
-                        ref: `refs/tags/${release.tag.toString()}`,
-                        sha: release.sha,
-                    });
-                }
-                catch (err) {
-                    // ignore if tag already exists
-                    if (err.status === 422) {
-                        this.logger.debug(`Tag ${release.tag.toString()} already exists, skipping tag creation`);
-                    }
-                    else {
-                        throw err;
-                    }
-                }
-            }
-            const resp = await this.octokit.repos.createRelease({
-                name: release.name,
-                owner: this.repository.owner,
-                repo: this.repository.repo,
-                tag_name: release.tag.toString(),
-                body: release.notes,
-                draft: !!options.draft,
-                prerelease: !!options.prerelease,
-                target_commitish: release.sha,
-            });
-            return {
-                id: resp.data.id,
-                name: resp.data.name || undefined,
-                tagName: resp.data.tag_name,
-                sha: resp.data.target_commitish,
-                notes: resp.data.body_text ||
-                    resp.data.body ||
-                    resp.data.body_html ||
-                    undefined,
-                url: resp.data.html_url,
-                draft: resp.data.draft,
-                uploadUrl: resp.data.upload_url,
-            };
-        }, e => {
-            if (e instanceof request_error_1.RequestError) {
-                if (e.status === 422 &&
-                    errors_1.GitHubAPIError.parseErrors(e).some(error => {
-                        return error.code === 'already_exists';
-                    })) {
-                    throw new errors_1.DuplicateReleaseError(e, 'tagName');
-                }
-            }
-        });
-        /**
-         * Makes a comment on a issue/pull request.
-         *
-         * @param {string} comment - The body of the comment to post.
-         * @param {number} number - The issue or pull request number.
-         * @throws {GitHubAPIError} on an API error
-         */
-        this.commentOnIssue = wrapAsync(async (comment, number) => {
-            this.logger.debug(`adding comment to https://github.com/${this.repository.owner}/${this.repository.repo}/issues/${number}`);
-            const resp = await this.octokit.issues.createComment({
-                owner: this.repository.owner,
-                repo: this.repository.repo,
-                issue_number: number,
-                body: comment,
-            });
-            return resp.data.html_url;
-        });
-        /**
-         * Removes labels from an issue/pull request.
-         *
-         * @param {string[]} labels The labels to remove.
-         * @param {number} number The issue/pull request number.
-         */
-        this.removeIssueLabels = wrapAsync(async (labels, number) => {
-            if (labels.length === 0) {
-                return;
-            }
-            this.logger.debug(`removing labels: ${labels} from issue/pull ${number}`);
-            await Promise.all(labels.map(label => this.octokit.issues.removeLabel({
-                owner: this.repository.owner,
-                repo: this.repository.repo,
-                issue_number: number,
-                name: label,
-            })));
-        });
-        /**
-         * Adds label to an issue/pull request.
-         *
-         * @param {string[]} labels The labels to add.
-         * @param {number} number The issue/pull request number.
-         */
-        this.addIssueLabels = wrapAsync(async (labels, number) => {
-            if (labels.length === 0) {
-                return;
-            }
-            this.logger.debug(`adding labels: ${labels} from issue/pull ${number}`);
-            await this.octokit.issues.addLabels({
-                owner: this.repository.owner,
-                repo: this.repository.repo,
-                issue_number: number,
-                labels,
-            });
-        });
         this.repository = options.repository;
         this.octokit = options.octokitAPIs.octokit;
-        this.request = options.octokitAPIs.request;
         this.graphql = options.octokitAPIs.graphql;
         this.fileCache = new git_file_utils_1.RepositoryFileCache(this.octokit, this.repository);
         this.logger = (_a = options.logger) !== null && _a !== void 0 ? _a : logger_1.logger;
+        this.gitHubApi = new github_api_1.GitHubApi({
+            repository: this.repository,
+            octokitAPIs: options.octokitAPIs,
+            logger: this.logger,
+        });
     }
-    static createDefaultAgent(baseUrl, defaultProxy) {
-        if (!defaultProxy) {
-            return undefined;
-        }
-        const { host, port } = defaultProxy;
-        if (new URL(baseUrl).protocol.replace(':', '') === 'http') {
-            return new http_proxy_agent_1.HttpProxyAgent(`http://${host}:${port}`);
-        }
-        else {
-            return new https_proxy_agent_1.HttpsProxyAgent(`https://${host}:${port}`);
-        }
+    getGitHubApi() {
+        return this.gitHubApi;
     }
-    /**
-     * Build a new GitHub client with auto-detected default branch.
-     *
-     * @param {GitHubCreateOptions} options Configuration options
-     * @param {string} options.owner The repository owner.
-     * @param {string} options.repo The repository name.
-     * @param {string} options.defaultBranch Optional. The repository's default branch.
-     *   Defaults to the value fetched via the API.
-     * @param {string} options.apiUrl Optional. The base url of the GitHub API.
-     * @param {string} options.graphqlUrl Optional. The base url of the GraphQL API.
-     * @param {OctokitAPISs} options.octokitAPIs Optional. Override the internal
-     *   client instances with a pre-authenticated instance.
-     * @param {string} token Optional. A GitHub API token used for authentication.
-     */
     static async create(options) {
-        var _a, _b, _c, _d;
-        const apiUrl = (_a = options.apiUrl) !== null && _a !== void 0 ? _a : exports.GH_API_URL;
-        const graphqlUrl = (_b = options.graphqlUrl) !== null && _b !== void 0 ? _b : exports.GH_GRAPHQL_URL;
-        const releasePleaseVersion = require('../../package.json').version;
-        const apis = (_c = options.octokitAPIs) !== null && _c !== void 0 ? _c : {
-            octokit: new rest_1.Octokit({
-                baseUrl: apiUrl,
-                auth: options.token,
-                request: {
-                    agent: this.createDefaultAgent(apiUrl, options.proxy),
-                    fetch: options.fetch,
-                },
-            }),
-            request: request_1.request.defaults({
-                baseUrl: apiUrl,
-                headers: {
-                    'user-agent': `release-please/${releasePleaseVersion}`,
-                    Authorization: `token ${options.token}`,
-                },
-                fetch: options.fetch,
-            }),
-            graphql: graphql_1.graphql.defaults({
-                baseUrl: graphqlUrl,
-                request: {
-                    agent: this.createDefaultAgent(graphqlUrl, options.proxy),
-                    fetch: options.fetch,
-                },
-                headers: {
-                    'user-agent': `release-please/${releasePleaseVersion}`,
-                    Authorization: `token ${options.token}`,
-                    'content-type': 'application/vnd.github.v3+json',
-                },
-            }),
-        };
-        const opts = {
-            repository: {
-                owner: options.owner,
-                repo: options.repo,
-                defaultBranch: (_d = options.defaultBranch) !== null && _d !== void 0 ? _d : (await GitHub.defaultBranch(options.owner, options.repo, apis.octokit)),
-            },
-            octokitAPIs: apis,
+        const gitHubApi = await github_api_1.GitHubApi.create(options);
+        return new GitHub({
+            repository: gitHubApi.repository,
+            octokitAPIs: gitHubApi.octokitAPIs,
             logger: options.logger,
-        };
-        return new GitHub(opts);
-    }
-    /**
-     * Returns the default branch for a given repository.
-     *
-     * @param {string} owner The GitHub repository owner
-     * @param {string} repo The GitHub repository name
-     * @param {OctokitType} octokit An authenticated octokit instance
-     * @returns {string} Name of the default branch
-     */
-    static async defaultBranch(owner, repo, octokit) {
-        const { data } = await octokit.repos.get({
-            repo,
-            owner,
         });
-        return data.default_branch;
     }
     /**
      * Returns the list of commits to the default branch after the provided filter
@@ -545,8 +229,8 @@ class GitHub {
         }
     }
     async mergeCommitsGraphQL(targetBranch, cursor, options = {}) {
-        var _a, _b, _c, _d, _e, _f, _g, _h, _j;
-        var _k;
+        var _a, _b, _c, _d, _e, _f, _g, _h, _j, _k;
+        var _l;
         this.logger.debug(`Fetching merge commits on branch ${targetBranch} with cursor: ${cursor}`);
         const query = `query pullRequestsSince($owner: String!, $repo: String!, $num: Int!, $maxFilesChanged: Int, $targetBranch: String!, $cursor: String) {
       repository(owner: $owner, name: $repo) {
@@ -583,6 +267,13 @@ class GitHub {
                   }
                   sha: oid
                   message
+                  author {
+                    name
+                    email
+                    user {
+                      login
+                    }
+                  }
                 }
                 pageInfo {
                   hasNextPage
@@ -624,7 +315,7 @@ class GitHub {
         for (const commit of commits) {
             for (const pr of commit.associatedPullRequests.nodes) {
                 if ((_c = pr.mergeCommit) === null || _c === void 0 ? void 0 : _c.oid) {
-                    (_d = mergeCommitCount[_k = pr.mergeCommit.oid]) !== null && _d !== void 0 ? _d : (mergeCommitCount[_k] = 0);
+                    (_d = mergeCommitCount[_l = pr.mergeCommit.oid]) !== null && _d !== void 0 ? _d : (mergeCommitCount[_l] = 0);
                     mergeCommitCount[pr.mergeCommit.oid]++;
                 }
             }
@@ -634,6 +325,13 @@ class GitHub {
             const commit = {
                 sha: graphCommit.sha,
                 message: graphCommit.message,
+                author: graphCommit.author
+                    ? {
+                        name: graphCommit.author.name || 'Unknown',
+                        email: graphCommit.author.email,
+                        username: (_e = graphCommit.author.user) === null || _e === void 0 ? void 0 : _e.login,
+                    }
+                    : undefined,
             };
             const mergePullRequest = graphCommit.associatedPullRequests.nodes.find(pr => {
                 return (
@@ -654,15 +352,15 @@ class GitHub {
                     number: pullRequest.number,
                     baseBranchName: pullRequest.baseRefName,
                     headBranchName: pullRequest.headRefName,
-                    mergeCommitOid: (_e = pullRequest.mergeCommit) === null || _e === void 0 ? void 0 : _e.oid,
+                    mergeCommitOid: (_f = pullRequest.mergeCommit) === null || _f === void 0 ? void 0 : _f.oid,
                     title: pullRequest.title,
                     body: pullRequest.body,
                     labels: pullRequest.labels.nodes.map(node => node.name),
-                    files: (((_f = pullRequest.files) === null || _f === void 0 ? void 0 : _f.nodes) || []).map(node => node.path),
+                    files: (((_g = pullRequest.files) === null || _g === void 0 ? void 0 : _g.nodes) || []).map(node => node.path),
                 };
             }
             if (mergePullRequest) {
-                if (((_h = (_g = mergePullRequest.files) === null || _g === void 0 ? void 0 : _g.pageInfo) === null || _h === void 0 ? void 0 : _h.hasNextPage) &&
+                if (((_j = (_h = mergePullRequest.files) === null || _h === void 0 ? void 0 : _h.pageInfo) === null || _j === void 0 ? void 0 : _j.hasNextPage) &&
                     options.backfillFiles) {
                     this.logger.info(`PR #${mergePullRequest.number} has many files, backfilling`);
                     commit.files = await this.getCommitFiles(graphCommit.sha);
@@ -670,7 +368,7 @@ class GitHub {
                 else {
                     // We cannot directly fetch files on commits via graphql, only provide file
                     // information for commits with associated pull requests
-                    commit.files = (((_j = mergePullRequest.files) === null || _j === void 0 ? void 0 : _j.nodes) || []).map(node => node.path);
+                    commit.files = (((_k = mergePullRequest.files) === null || _k === void 0 ? void 0 : _k.nodes) || []).map(node => node.path);
                 }
             }
             else if (options.backfillFiles) {
@@ -700,167 +398,7 @@ class GitHub {
      * @throws {GitHubAPIError} on an API error
      */
     async *pullRequestIterator(targetBranch, status = 'MERGED', maxResults = Number.MAX_SAFE_INTEGER, includeFiles = true) {
-        const generator = includeFiles
-            ? this.pullRequestIteratorWithFiles(targetBranch, status, maxResults)
-            : this.pullRequestIteratorWithoutFiles(targetBranch, status, maxResults);
-        for await (const pullRequest of generator) {
-            yield pullRequest;
-        }
-    }
-    /**
-     * Helper implementation of pullRequestIterator that includes files via
-     * the graphQL API.
-     *
-     * @param {string} targetBranch The base branch of the pull request
-     * @param {string} status The status of the pull request
-     * @param {number} maxResults Limit the number of results searched
-     */
-    async *pullRequestIteratorWithFiles(targetBranch, status = 'MERGED', maxResults = Number.MAX_SAFE_INTEGER) {
-        let cursor = undefined;
-        let results = 0;
-        while (results < maxResults) {
-            const response = await this.pullRequestsGraphQL(targetBranch, status, cursor);
-            // no response usually means we ran out of results
-            if (!response) {
-                break;
-            }
-            for (let i = 0; i < response.data.length; i++) {
-                results += 1;
-                yield response.data[i];
-            }
-            if (!response.pageInfo.hasNextPage) {
-                break;
-            }
-            cursor = response.pageInfo.endCursor;
-        }
-    }
-    /**
-     * Helper implementation of pullRequestIterator that excludes files
-     * via the REST API.
-     *
-     * @param {string} targetBranch The base branch of the pull request
-     * @param {string} status The status of the pull request
-     * @param {number} maxResults Limit the number of results searched
-     */
-    async *pullRequestIteratorWithoutFiles(targetBranch, status = 'MERGED', maxResults = Number.MAX_SAFE_INTEGER) {
-        const statusMap = {
-            OPEN: 'open',
-            CLOSED: 'closed',
-            MERGED: 'closed',
-        };
-        let results = 0;
-        for await (const { data: pulls } of this.octokit.paginate.iterator('GET /repos/{owner}/{repo}/pulls', {
-            state: statusMap[status],
-            owner: this.repository.owner,
-            repo: this.repository.repo,
-            base: targetBranch,
-            sort: 'updated',
-            direction: 'desc',
-        })) {
-            for (const pull of pulls) {
-                // The REST API does not have an option for "merged"
-                // pull requests - they are closed with a `merged_at` timestamp
-                if (status !== 'MERGED' || pull.merged_at) {
-                    results += 1;
-                    yield {
-                        headBranchName: pull.head.ref,
-                        baseBranchName: pull.base.ref,
-                        number: pull.number,
-                        title: pull.title,
-                        body: pull.body || '',
-                        labels: pull.labels.map(label => label.name),
-                        files: [],
-                        sha: pull.merge_commit_sha || undefined,
-                    };
-                    if (results >= maxResults) {
-                        break;
-                    }
-                }
-            }
-            if (results >= maxResults) {
-                break;
-            }
-        }
-    }
-    /**
-     * Return a list of merged pull requests. The list is not guaranteed to be sorted
-     * by merged_at, but is generally most recent first.
-     *
-     * @param {string} targetBranch - Base branch of the pull request. Defaults to
-     *   the configured default branch.
-     * @param {number} page - Page of results. Defaults to 1.
-     * @param {number} perPage - Number of results per page. Defaults to 100.
-     * @returns {PullRequestHistory | null} - List of merged pull requests
-     * @throws {GitHubAPIError} on an API error
-     */
-    async pullRequestsGraphQL(targetBranch, states = 'MERGED', cursor) {
-        var _a;
-        this.logger.debug(`Fetching ${states} pull requests on branch ${targetBranch} with cursor ${cursor}`);
-        const response = await this.graphqlRequest({
-            query: `query mergedPullRequests($owner: String!, $repo: String!, $num: Int!, $maxFilesChanged: Int, $targetBranch: String!, $states: [PullRequestState!], $cursor: String) {
-        repository(owner: $owner, name: $repo) {
-          pullRequests(first: $num, after: $cursor, baseRefName: $targetBranch, states: $states, orderBy: {field: CREATED_AT, direction: DESC}) {
-            nodes {
-              number
-              title
-              baseRefName
-              headRefName
-              labels(first: 10) {
-                nodes {
-                  name
-                }
-              }
-              body
-              mergeCommit {
-                oid
-              }
-              files(first: $maxFilesChanged) {
-                nodes {
-                  path
-                }
-                pageInfo {
-                  endCursor
-                  hasNextPage
-                }
-              }
-            }
-            pageInfo {
-              endCursor
-              hasNextPage
-            }
-          }
-        }
-      }`,
-            cursor,
-            owner: this.repository.owner,
-            repo: this.repository.repo,
-            num: 25,
-            targetBranch,
-            states,
-            maxFilesChanged: 64,
-        });
-        if (!((_a = response === null || response === void 0 ? void 0 : response.repository) === null || _a === void 0 ? void 0 : _a.pullRequests)) {
-            this.logger.warn(`Could not find merged pull requests for branch ${targetBranch} - it likely does not exist.`);
-            return null;
-        }
-        const pullRequests = (response.repository.pullRequests.nodes ||
-            []);
-        return {
-            pageInfo: response.repository.pullRequests.pageInfo,
-            data: pullRequests.map(pullRequest => {
-                var _a, _b, _c;
-                return {
-                    sha: (_a = pullRequest.mergeCommit) === null || _a === void 0 ? void 0 : _a.oid,
-                    number: pullRequest.number,
-                    baseBranchName: pullRequest.baseRefName,
-                    headBranchName: pullRequest.headRefName,
-                    labels: (((_b = pullRequest.labels) === null || _b === void 0 ? void 0 : _b.nodes) || []).map(l => l.name),
-                    title: pullRequest.title,
-                    body: pullRequest.body + '',
-                    files: (((_c = pullRequest.files) === null || _c === void 0 ? void 0 : _c.nodes) || []).map(node => node.path),
-                };
-            }),
-        };
+        yield* this.gitHubApi.pullRequestIterator(targetBranch, status, maxResults, includeFiles);
     }
     /**
      * Iterate through releases with a max number of results scanned.
@@ -872,80 +410,7 @@ class GitHub {
      * @throws {GitHubAPIError} on an API error
      */
     async *releaseIterator(options = {}) {
-        var _a;
-        const maxResults = (_a = options.maxResults) !== null && _a !== void 0 ? _a : Number.MAX_SAFE_INTEGER;
-        let results = 0;
-        let cursor = undefined;
-        while (true) {
-            const response = await this.releaseGraphQL(cursor);
-            if (!response) {
-                break;
-            }
-            for (let i = 0; i < response.data.length; i++) {
-                if ((results += 1) > maxResults) {
-                    break;
-                }
-                yield response.data[i];
-            }
-            if (results > maxResults || !response.pageInfo.hasNextPage) {
-                break;
-            }
-            cursor = response.pageInfo.endCursor;
-        }
-    }
-    async releaseGraphQL(cursor) {
-        this.logger.debug(`Fetching releases with cursor ${cursor}`);
-        const response = await this.graphqlRequest({
-            query: `query releases($owner: String!, $repo: String!, $num: Int!, $cursor: String) {
-        repository(owner: $owner, name: $repo) {
-          releases(first: $num, after: $cursor, orderBy: {field: CREATED_AT, direction: DESC}) {
-            nodes {
-              name
-              tag {
-                name
-              }
-              tagCommit {
-                oid
-              }
-              url
-              description
-              isDraft
-            }
-            pageInfo {
-              endCursor
-              hasNextPage
-            }
-          }
-        }
-      }`,
-            cursor,
-            owner: this.repository.owner,
-            repo: this.repository.repo,
-            num: 25,
-        });
-        if (!response.repository.releases.nodes.length) {
-            this.logger.warn('Could not find releases.');
-            return null;
-        }
-        const releases = response.repository.releases.nodes;
-        return {
-            pageInfo: response.repository.releases.pageInfo,
-            data: releases
-                .filter(release => !!release.tagCommit)
-                .map(release => {
-                if (!release.tag || !release.tagCommit) {
-                    this.logger.debug(release);
-                }
-                return {
-                    name: release.name || undefined,
-                    tagName: release.tag ? release.tag.name : 'unknown',
-                    sha: release.tagCommit.oid,
-                    notes: release.description,
-                    url: release.url,
-                    draft: release.isDraft,
-                };
-            }),
-        };
+        yield* this.gitHubApi.releaseIterator(options);
     }
     /**
      * Iterate through tags with a max number of results scanned.
@@ -1042,33 +507,91 @@ class GitHub {
     /**
      * Open a pull request
      *
-     * @deprecated This logic is handled by the Manifest class now as it
-     *   can be more complicated if the release notes are too big
-     * @param {ReleasePullRequest} releasePullRequest Pull request data to update
+     * @param {PullRequest} pullRequest Pull request data to update
      * @param {string} targetBranch The base branch of the pull request
-     * @param {GitHubPR} options The pull request options
+     * @param {string} message The commit message for the commit
+     * @param {Update[]} updates The files to update
+     * @param {CreatePullRequestOptions} options The pull request options
      * @throws {GitHubAPIError} on an API error
      */
-    async createReleasePullRequest(releasePullRequest, targetBranch, options) {
+    async createPullRequest(pullRequest, targetBranch, message, updates, options) {
+        const changes = await this.buildChangeSet(updates, targetBranch);
+        const prNumber = await (0, code_suggester_1.createPullRequest)(this.octokit, changes, {
+            upstreamOwner: this.repository.owner,
+            upstreamRepo: this.repository.repo,
+            title: pullRequest.title,
+            branch: pullRequest.headBranchName,
+            description: pullRequest.body,
+            primary: targetBranch,
+            force: true,
+            fork: !!(options === null || options === void 0 ? void 0 : options.fork),
+            message,
+            logger: this.logger,
+            draft: !!(options === null || options === void 0 ? void 0 : options.draft),
+            labels: pullRequest.labels,
+        });
+        if (prNumber === 0) {
+            this.logger.warn('no code changes detected, skipping pull request creation');
+            return {
+                headBranchName: pullRequest.headBranchName,
+                baseBranchName: targetBranch,
+                number: 0,
+                title: pullRequest.title,
+                body: pullRequest.body,
+                labels: pullRequest.labels,
+                files: [],
+            };
+        }
+        return await this.getPullRequest(prNumber);
+    }
+    /**
+     * Fetch a pull request given the pull number
+     * @param {number} number The pull request number
+     * @returns {PullRequest}
+     */
+    async getPullRequest(number) {
+        return await this.gitHubApi.getPullRequest(number);
+    }
+    /**
+     * Update a pull request's title and body.
+     * @param {number} number The pull request number
+     * @param {ReleasePullRequest} releasePullRequest Pull request data to update
+     * @param {string} targetBranch The target branch of the pull request
+     * @param {string} options.signoffUser Optional. Commit signoff message
+     * @param {boolean} options.fork Optional. Whether to open the pull request from
+     *   a fork or not. Defaults to `false`
+     * @param {PullRequestOverflowHandler} options.pullRequestOverflowHandler Optional.
+     *   Handles extra large pull request body messages.
+     */
+    async updatePullRequest(number, releasePullRequest, targetBranch, options) {
+        const changes = await this.buildChangeSet(releasePullRequest.updates, targetBranch);
         let message = releasePullRequest.title.toString();
         if (options === null || options === void 0 ? void 0 : options.signoffUser) {
             message = (0, signoff_commit_message_1.signoffCommitMessage)(message, options.signoffUser);
         }
-        const pullRequestLabels = (options === null || options === void 0 ? void 0 : options.skipLabeling)
-            ? []
-            : releasePullRequest.labels;
-        return await this.createPullRequest({
-            headBranchName: releasePullRequest.headRefName,
-            baseBranchName: targetBranch,
-            number: -1,
-            title: releasePullRequest.title.toString(),
-            body: releasePullRequest.body.toString().slice(0, MAX_ISSUE_BODY_SIZE),
-            labels: pullRequestLabels,
-            files: [],
-        }, targetBranch, message, releasePullRequest.updates, {
-            fork: options === null || options === void 0 ? void 0 : options.fork,
+        const title = releasePullRequest.title.toString();
+        const body = ((options === null || options === void 0 ? void 0 : options.pullRequestOverflowHandler)
+            ? await options.pullRequestOverflowHandler.handleOverflow(releasePullRequest)
+            : releasePullRequest.body)
+            .toString()
+            .slice(0, MAX_ISSUE_BODY_SIZE);
+        const prNumber = await (0, code_suggester_1.createPullRequest)(this.octokit, changes, {
+            upstreamOwner: this.repository.owner,
+            upstreamRepo: this.repository.repo,
+            title,
+            branch: releasePullRequest.headRefName,
+            description: body,
+            primary: targetBranch,
+            force: true,
+            fork: (options === null || options === void 0 ? void 0 : options.fork) === false ? false : true,
+            message,
+            logger: this.logger,
             draft: releasePullRequest.draft,
         });
+        if (prNumber !== number) {
+            this.logger.warn(`updated code for ${prNumber}, but update requested for ${number}`);
+        }
+        return this.gitHubApi.updatePullRequest(number, title, body);
     }
     /**
      * Given a set of proposed updates, build a changeset to suggest.
@@ -1129,6 +652,45 @@ class GitHub {
     async findFilesByExtension(extension, prefix) {
         return this.findFilesByExtensionAndRef(extension, this.repository.defaultBranch, prefix);
     }
+    /**
+     * Create a GitHub release
+     *
+     * @param {Release} release Release parameters
+     * @param {ReleaseOptions} options Release option parameters
+     * @throws {DuplicateReleaseError} if the release tag already exists
+     * @throws {GitHubAPIError} on other API errors
+     */
+    async createRelease(release, options = {}) {
+        return await this.gitHubApi.createRelease(release, options);
+    }
+    /**
+     * Makes a comment on a issue/pull request.
+     *
+     * @param {string} comment - The body of the comment to post.
+     * @param {number} number - The issue or pull request number.
+     * @throws {GitHubAPIError} on an API error
+     */
+    async commentOnIssue(comment, number) {
+        return await this.gitHubApi.commentOnIssue(comment, number);
+    }
+    /**
+     * Removes labels from an issue/pull request.
+     *
+     * @param {string[]} labels The labels to remove.
+     * @param {number} number The issue/pull request number.
+     */
+    async removeIssueLabels(labels, number) {
+        return await this.gitHubApi.removeIssueLabels(labels, number);
+    }
+    /**
+     * Adds label to an issue/pull request.
+     *
+     * @param {string[]} labels The labels to add.
+     * @param {number} number The issue/pull request number.
+     */
+    async addIssueLabels(labels, number) {
+        return await this.gitHubApi.addIssueLabels(labels, number);
+    }
     /**
      * Generate release notes from GitHub at tag
      * @param {string} tagName Name of new release tag
@@ -1136,14 +698,7 @@ class GitHub {
      * @param {string} previousTag Optional. Name of previous tag to analyze commits since
      */
     async generateReleaseNotes(tagName, targetCommitish, previousTag) {
-        const resp = await this.octokit.repos.generateReleaseNotes({
-            owner: this.repository.owner,
-            repo: this.repository.repo,
-            tag_name: tagName,
-            previous_tag_name: previousTag,
-            target_commitish: targetCommitish,
-        });
-        return resp.data.body;
+        return await this.gitHubApi.generateReleaseNotes(tagName, targetCommitish, previousTag);
     }
     /**
      * Create a single file on a new branch based on an existing
@@ -1156,107 +711,7 @@ class GitHub {
      * @returns {string} HTML URL of the new file
      */
     async createFileOnNewBranch(filename, contents, newBranchName, baseBranchName) {
-        // create or update new branch to match base branch
-        await this.forkBranch(newBranchName, baseBranchName);
-        // use the single file upload API
-        const { data: { content }, } = await this.octokit.repos.createOrUpdateFileContents({
-            owner: this.repository.owner,
-            repo: this.repository.repo,
-            path: filename,
-            // contents need to be base64 encoded
-            content: Buffer.from(contents, 'binary').toString('base64'),
-            message: 'Saving release notes',
-            branch: newBranchName,
-        });
-        if (!(content === null || content === void 0 ? void 0 : content.html_url)) {
-            throw new Error(`Failed to write to file: ${filename} on branch: ${newBranchName}`);
-        }
-        return content.html_url;
-    }
-    /**
-     * Helper to fetch the SHA of a branch
-     * @param {string} branchName The name of the branch
-     * @return {string | undefined} Returns the SHA of the branch
-     *   or undefined if it can't be found.
-     */
-    async getBranchSha(branchName) {
-        this.logger.debug(`Looking up SHA for branch: ${branchName}`);
-        try {
-            const { data: { object: { sha }, }, } = await this.octokit.git.getRef({
-                owner: this.repository.owner,
-                repo: this.repository.repo,
-                ref: `heads/${branchName}`,
-            });
-            this.logger.debug(`SHA for branch: ${sha}`);
-            return sha;
-        }
-        catch (e) {
-            if (e instanceof request_error_1.RequestError && e.status === 404) {
-                this.logger.debug(`Branch: ${branchName} does not exist`);
-                return undefined;
-            }
-            throw e;
-        }
-    }
-    /**
-     * Helper to fork a branch from an existing branch. Uses `force` so
-     * it will overwrite the contents of `targetBranchName` to match
-     * the current contents of `baseBranchName`.
-     *
-     * @param {string} targetBranchName The name of the new forked branch
-     * @param {string} baseBranchName The base branch from which to fork.
-     * @returns {string} The branch SHA
-     * @throws {ConfigurationError} if the base branch cannot be found.
-     */
-    async forkBranch(targetBranchName, baseBranchName) {
-        const baseBranchSha = await this.getBranchSha(baseBranchName);
-        if (!baseBranchSha) {
-            // this is highly unlikely to be thrown as we will have
-            // already attempted to read from the branch
-            throw new errors_1.ConfigurationError(`Unable to find base branch: ${baseBranchName}`, 'core', `${this.repository.owner}/${this.repository.repo}`);
-        }
-        // see if newBranchName exists
-        if (await this.getBranchSha(targetBranchName)) {
-            // branch already exists, update it to the match the base branch
-            const branchSha = await this.updateBranchSha(targetBranchName, baseBranchSha);
-            this.logger.debug(`Updated ${targetBranchName} to match ${baseBranchName} at ${branchSha}`);
-            return branchSha;
-        }
-        else {
-            // branch does not exist, create a new branch from the base branch
-            const branchSha = await this.createNewBranch(targetBranchName, baseBranchSha);
-            this.logger.debug(`Forked ${targetBranchName} from ${baseBranchName} at ${branchSha}`);
-            return branchSha;
-        }
-    }
-    /**
-     * Helper to create a new branch from a given SHA.
-     * @param {string} branchName The new branch name
-     * @param {string} branchSha The SHA of the branch
-     * @returns {string} The SHA of the new branch
-     */
-    async createNewBranch(branchName, branchSha) {
-        this.logger.debug(`Creating new branch: ${branchName} at ${branchSha}`);
-        const { data: { object: { sha }, }, } = await this.octokit.git.createRef({
-            owner: this.repository.owner,
-            repo: this.repository.repo,
-            ref: `refs/heads/${branchName}`,
-            sha: branchSha,
-        });
-        this.logger.debug(`New branch: ${branchName} at ${sha}`);
-        return sha;
-    }
-    async updateBranchSha(branchName, branchSha) {
-        this.logger.debug(`Updating branch ${branchName} to ${branchSha}`);
-        const { data: { object: { sha }, }, } = await this.octokit.git.updateRef({
-            owner: this.repository.owner,
-            repo: this.repository.repo,
-            ref: `heads/${branchName}`,
-            sha: branchSha,
-            force: true,
-        });
-        this.logger.debug(`Updated branch: ${branchName} to ${sha}`);
-        return sha;
+        return await this.gitHubApi.createFileOnNewBranch(filename, contents, newBranchName, baseBranchName);
     }
 }
 exports.GitHub = GitHub;