Merge dev into main: security fixes#14
Merged
ApiliumDevTeam merged 2 commits intomainfrom Mar 5, 2026
Merged
Conversation
…ulnerable deps, add workflow permissions - Replace hardcoded base64 Google Antigravity OAuth credentials with env vars (MAYROS_ANTIGRAVITY_OAUTH_CLIENT_ID, MAYROS_ANTIGRAVITY_OAUTH_CLIENT_SECRET) - Remove hardcoded DEFAULT_PROJECT_ID fallback (rising-fact-p41fc) to avoid Google account bans from using their first-party project ID - Bump tar 7.5.9 → 7.5.10 (CVE hardlink path traversal) - Bump esbuild ^0.24.0 → ^0.25.0 (dev server request forgery) - Add least-privilege permissions (contents: read) to ci.yml and workflow-sanity.yml (resolves 11 code scanning alerts) Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
## Summary - Replace hardcoded base64 Google Antigravity OAuth credentials with env vars (`MAYROS_ANTIGRAVITY_OAUTH_CLIENT_ID`, `MAYROS_ANTIGRAVITY_OAUTH_CLIENT_SECRET`) - Remove hardcoded `DEFAULT_PROJECT_ID` fallback (`rising-fact-p41fc`) — Google's first-party project ID that was causing account bans - Bump `tar` 7.5.9 → 7.5.10 (CVE hardlink path traversal) - Bump `esbuild` ^0.24.0 → ^0.25.0 (dev server request forgery) - Add least-privilege `permissions: contents: read` to `ci.yml` and `workflow-sanity.yml` (resolves 11 code scanning alerts) ## Test plan - [x] `pnpm install` succeeds - [x] `pnpm build` compiles - [x] No base64 secrets remain in `extensions/google-antigravity-auth/` - [ ] GitHub re-scans and closes alerts after merge
ApiliumDevTeam
added a commit
that referenced
this pull request
Mar 9, 2026
## Summary - Merges `fix/security-alerts` changes from `dev` into `main` - Resolves 19 GitHub security alerts: hardcoded OAuth secrets, vulnerable deps (tar, esbuild), missing workflow permissions ## Changes included - Replace hardcoded base64 Google Antigravity OAuth credentials with env vars - Remove hardcoded `DEFAULT_PROJECT_ID` fallback (`rising-fact-p41fc`) - Bump `tar` 7.5.9 → 7.5.10, `esbuild` ^0.24.0 → ^0.25.0 - Add `permissions: contents: read` to CI workflows
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
1 participant
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.
Summary
fix/security-alertschanges fromdevintomainChanges included
DEFAULT_PROJECT_IDfallback (rising-fact-p41fc)tar7.5.9 → 7.5.10,esbuild^0.24.0 → ^0.25.0permissions: contents: readto CI workflows