Skip to content

Commit 22d7ae2

Browse files
committed
fix: reserve TRIGGER_PREVIEW_BRANCH from external sync
1 parent 77086df commit 22d7ae2

1 file changed

Lines changed: 4 additions & 9 deletions

File tree

apps/webapp/app/v3/environmentVariableRules.server.ts

Lines changed: 4 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -10,15 +10,10 @@ const blacklistedVariables: VariableRule[] = [
1010
{ type: "exact", key: "TRIGGER_API_URL" },
1111
];
1212

13-
// Deploy-managed keys that are not blacklisted (so users may set them manually),
14-
// but should never be pulled in from an external integration like Vercel because
15-
// the platform owns their value.
16-
const additionalExternalSyncReservedKeys = ["TRIGGER_VERSION"];
17-
18-
/**
19-
* Returns true if a key would be stripped by `removeBlacklistedVariables`, i.e. the
20-
* environment variable repository will refuse to store it.
21-
*/
13+
// Platform-managed keys that aren't blacklisted but should never be synced from an
14+
// external integration. Keep in sync with the cloud repo's EXCLUDED_ENV_VAR_KEYS.
15+
const additionalExternalSyncReservedKeys = ["TRIGGER_VERSION", "TRIGGER_PREVIEW_BRANCH"];
16+
2217
export function isBlacklistedVariable(key: string): boolean {
2318
const whitelisted = blacklistedVariables.find((bv) => bv.type === "whitelist" && bv.key === key);
2419
if (whitelisted) {

0 commit comments

Comments
 (0)