Skip to content

fix(vercel-sync): filter team shared env vars to sync-owned set#6144

Merged
victorvhs017 merged 2 commits into
mainfrom
fix/vercel-secret-sync-team-scope-error
Apr 24, 2026
Merged

fix(vercel-sync): filter team shared env vars to sync-owned set#6144
victorvhs017 merged 2 commits into
mainfrom
fix/vercel-secret-sync-team-scope-error

Conversation

@victorvhs017
Copy link
Copy Markdown
Contributor

Context

Vercel (team scope): Team-level shared environment variables are now limited to variables that match this sync’s configuration (targets, projects, and sensitivity-derived dev exclusion), so sync/get/remove no longer touches unrelated team shared vars from other syncs or manual Vercel setup.

Steps to verify the change

  1. Vercel team sync: Configure two team-scoped syncs (or one sync + manual vars) with different targets/projects; run sync/get/remove and confirm only vars “owned” by each sync’s destination config are read or updated.

Type

  • Fix
  • Feature
  • Improvement
  • Breaking
  • Docs
  • Chore

Checklist

  • Title follows the conventional commit format: type(scope): short description (scope is optional, e.g., fix: prevent crash on sync or fix(api): handle null response).
  • Tested locally
  • Updated docs (if needed)
  • Updated CLAUDE.md files (if needed)
  • Read the contributing guide

@maidul98
Copy link
Copy Markdown
Collaborator

maidul98 commented Apr 23, 2026

Snyk checks have passed. No issues have been found so far.

Status Scan Engine Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

Comment thread backend/src/services/secret-sync/vercel/vercel-sync-fns.ts
Comment thread backend/src/services/secret-sync/vercel/vercel-sync-fns.ts
@victorvhs017 victorvhs017 requested review from adilsitos and varonix0 and removed request for adilsitos April 24, 2026 01:07
@victorvhs017 victorvhs017 merged commit 86234b5 into main Apr 24, 2026
11 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