only try to sync parent env vars for preview deployments

Author: nicktrn

apps/webapp/app/routes/api.v1.projects.$projectRef.envvars.$slug.import.ts

@@ -47,6 +47,7 @@ export async function action({ params, request }: ActionFunctionArgs) {
     })),
   });
 
+  // Only sync parent variables if this is a branch environment
   if (environment.parentEnvironmentId && body.parentVariables) {
     const parentResult = await repository.create(environment.project.id, {
       override: typeof body.override === "boolean" ? body.override : false,

packages/cli-v3/src/commands/deploy.ts

@@ -195,6 +195,7 @@ async function _deployCommand(dir: string, options: DeployCommandOptions) {
 
   const branch =
     options.env === "preview" ? getBranch({ specified: options.branch, gitMeta }) : undefined;
+
   if (options.env === "preview" && !branch) {
     throw new Error(
       "Didn't auto-detect preview branch, so you need to specify one. Pass --branch <branch>."
@@ -318,10 +319,9 @@ async function _deployCommand(dir: string, options: DeployCommandOptions) {
   }
 
   const hasVarsToSync =
-    buildManifest.deploy.sync &&
-    ((buildManifest.deploy.sync.env && Object.keys(buildManifest.deploy.sync.env).length > 0) ||
-      (buildManifest.deploy.sync.parentEnv &&
-        Object.keys(buildManifest.deploy.sync.parentEnv).length > 0));
+    Object.keys(buildManifest.deploy.sync?.env || {}).length > 0 ||
+    // Only sync parent variables if this is a branch environment
+    (branch && Object.keys(buildManifest.deploy.sync?.parentEnv || {}).length > 0);
 
   if (hasVarsToSync) {
     const childVars = buildManifest.deploy.sync?.env ?? {};
@@ -333,21 +333,22 @@ async function _deployCommand(dir: string, options: DeployCommandOptions) {
     if (!options.skipSyncEnvVars) {
       const $spinner = spinner();
       $spinner.start(`Syncing ${numberOfEnvVars} env ${vars} with the server`);
-      const success = await syncEnvVarsWithServer(
+
+      const uploadResult = await syncEnvVarsWithServer(
         projectClient.client,
         resolvedConfig.project,
         options.env,
         childVars,
         parentVars
       );
 
-      if (!success) {
+      if (!uploadResult.success) {
         await failDeploy(
           projectClient.client,
           deployment,
           {
             name: "SyncEnvVarsError",
-            message: `Failed to sync ${numberOfEnvVars} env ${vars} with the server`,
+            message: `Failed to sync ${numberOfEnvVars} env ${vars} with the server: ${uploadResult.error}`,
           },
           "",
           $spinner
@@ -587,13 +588,11 @@ export async function syncEnvVarsWithServer(
   envVars: Record<string, string>,
   parentEnvVars?: Record<string, string>
 ) {
-  const uploadResult = await apiClient.importEnvVars(projectRef, environmentSlug, {
+  return await apiClient.importEnvVars(projectRef, environmentSlug, {
     variables: envVars,
     parentVariables: parentEnvVars,
     override: true,
   });
-
-  return uploadResult.success;
 }
 
 async function failDeploy(
@@ -637,7 +636,7 @@ async function failDeploy(
         console.log(lastFewLines);
       }
     } else {
-      outro(`${chalkError(`${prefix}:`)} ${error.message}.`);
+      outro(`${chalkError(`${prefix}:`)} ${error.message}`);
     }
   };