fix: Add ACTIONS_BOT_TOKEN and VBOT_GITHUB_API_TOKEN to token check

[JakeSCahill]

The getGitHubToken() helper was missing ACTIONS_BOT_TOKEN (used in CI) and VBOT_GITHUB_API_TOKEN (used in legacy code), causing authentication to fail and hitting unauthenticated rate limits (60 req/hr instead of 5000 req/hr).

This was causing 403/429 errors when checking cloud-docs for 471 connectors.

Updated token priority order:

1. REDPANDA_GITHUB_TOKEN
2. ACTIONS_BOT_TOKEN (CI)
3. GITHUB_TOKEN (GitHub Actions default)
4. VBOT_GITHUB_API_TOKEN (legacy)
5. GH_TOKEN (GitHub CLI)

[coderabbitai]

Important

Review skipped

Auto incremental reviews are disabled on this repository.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

📝 Walkthrough

Walkthrough

The PR modifies the environment variable priority chain in the getGitHubToken() function within cli-utils/github-token.js. The updated token selection order is: ACTIONS_BOT_TOKEN, GITHUB_TOKEN, VBOT_GITHUB_API_TOKEN, and GH_TOKEN. The function's public API remains unchanged, and the modification updates only the comment documentation and the conditional evaluation order for token selection.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~5 minutes

Possibly related PRs

• Fix/array of objects and cloud docs check #166: Adds support for VBOT_GITHUB_API_TOKEN and GITHUB_TOKEN in Octokit configuration, aligning with the token priority changes introduced in this PR.

Suggested reviewers

• andrewhsu
• paulohtb6

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title accurately describes the main change: adding two new GitHub tokens (ACTIONS_BOT_TOKEN and VBOT_GITHUB_API_TOKEN) to the token check logic.
Description check	✅ Passed	The description is directly related to the changeset, explaining the problem (missing token support), the impact (rate limiting issues), and the solution (adding tokens with priority order).
Docstring Coverage	✅ Passed	Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[netlify]

✅ Deploy Preview for docs-extensions-and-macros ready!

Name	Link
🔨 Latest commit	71755f7
🔍 Latest deploy log	https://app.netlify.com/projects/docs-extensions-and-macros/deploys/69709d99796649000866843d
😎 Deploy Preview	https://deploy-preview-167--docs-extensions-and-macros.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify project configuration.