Skip to content

Conversation

@hanna-skryl
Copy link
Collaborator

@hanna-skryl hanna-skryl commented Aug 14, 2025

This PR implements GitHub App installation authentication for the Code PushUp GitHub Action with automatic fallback to standard token authentication.

  • Add an authenticate() function that retrieves installation tokens from the Cloud Run service
  • Fall back to the standard token when the app is unavailable
  • Add unit tests with fetch mocking
  • Update main.ts to use the new authentication flow
Screenshot 2025-08-18 at 8 58 45 AM

@hanna-skryl hanna-skryl self-assigned this Aug 14, 2025
@github-actions
Copy link

github-actions bot commented Aug 14, 2025

Code PushUp

🥳 Code PushUp report has improved – compared current commit 5bb3fdd with previous commit 5b22a49.

🕵️ See full comparison in Code PushUp portal 🔍

🏷️ Categories

🏷️ Category ⭐ Previous score ⭐ Current score 🔄 Score change
Code coverage 🟡 59 🟡 62 ↑ +3.5
Bug prevention 🟢 100 🟢 100
Code style 🟢 100 🟢 100
Security 🔴 15 🔴 15
Updates 🟢 94 🟢 94
👍 1 group improved, 👍 3 audits improved

🗃️ Groups

🔌 Plugin 🗃️ Group ⭐ Previous score ⭐ Current score 🔄 Score change
Code coverage Code coverage metrics 🟡 59 🟡 62 ↑ +3.5

11 other groups are unchanged.

🛡️ Audits

🔌 Plugin 🛡️ Audit 📏 Previous value 📏 Current value 🔄 Value change
Code coverage Branch coverage 🟥 38 % 🟥 42.9 % ↑ +12.9 %
Code coverage Line coverage 🟨 57.5 % 🟨 60.4 % ↑ +5.1 %
Code coverage Function coverage 🟨 69 % 🟨 71.9 % ↑ +4.2 %

169 other audits are unchanged.

@hanna-skryl hanna-skryl marked this pull request as ready for review August 18, 2025 13:01
Copy link
Contributor

@matejchalk matejchalk left a comment

Choose a reason for hiding this comment

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

Well done 👏

@hanna-skryl hanna-skryl force-pushed the github-app-auth branch 2 times, most recently from 1872fec to 03537e4 Compare August 18, 2025 14:23
@hanna-skryl hanna-skryl merged commit 0c0009a into main Aug 18, 2025
19 checks passed
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.

3 participants