Fix GitHub App configuration detection and dynamic installation ID lookup - #119
Closed
ishaileshpant wants to merge 1 commit into
Closed
ishaileshpant wants to merge 1 commit into
ishaileshpant wants to merge 1 commit into
Conversation
…okup - Remove requirement for GITHUB_APP_INSTALLATION_ID environment variable - Add dynamic installation ID lookup for prepguides organization - Improve GitHub App initialization with better error handling - Resolves FUNCTION_INVOCATION_FAILED error by properly detecting GitHub App The bot API was failing because it required GITHUB_APP_INSTALLATION_ID as an environment variable, but GitHub Apps can dynamically find their installation ID. This fix allows the bot to work with just GITHUB_APP_ID and GITHUB_APP_PRIVATE_KEY, then automatically finds the installation ID for the prepguides organization. Changes: 1. Removed GITHUB_APP_INSTALLATION_ID requirement from env check 2. Added dynamic installation ID lookup using GitHub API 3. Improved error handling and logging for GitHub App initialization 4. Maintains backward compatibility with explicit installation ID if provided
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Collaborator
Author
|
Closing this PR as it's no longer needed. PR #118 was merged which includes the fallback mechanism, making this GitHub App configuration fix unnecessary. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
🔧 Fix GitHub App Configuration Detection
Problem
The bot API was failing with "FUNCTION_INVOCATION_FAILED" error because it required
GITHUB_APP_INSTALLATION_IDas an environment variable, but the GitHub App is installed and configured in Vercel.Root Cause
The bot API was checking for
GITHUB_APP_INSTALLATION_IDas a required environment variable, but GitHub Apps can dynamically find their installation ID using the GitHub API.Solution
GITHUB_APP_INSTALLATION_IDenvironment variableTechnical Changes
How It Works
Files Changed
api/github/create-content-pr.js- Fixed GitHub App configuration detection and initializationImpact
Testing