Make ARTIFACTS_OBJECT_STORE_BUCKET optional

Avoid confusion with self-hosters.

Author: myftija

apps/webapp/app/env.server.ts

@@ -346,7 +346,7 @@ const EnvironmentSchema = z
     OBJECT_STORE_REGION: z.string().optional(),
     OBJECT_STORE_SERVICE: z.string().default("s3"),
 
-    ARTIFACTS_OBJECT_STORE_BUCKET: z.string(),
+    ARTIFACTS_OBJECT_STORE_BUCKET: z.string().optional(),
     ARTIFACTS_OBJECT_STORE_BASE_URL: z.string().optional(),
     ARTIFACTS_OBJECT_STORE_ACCESS_KEY_ID: z.string().optional(),
     ARTIFACTS_OBJECT_STORE_SECRET_ACCESS_KEY: z.string().optional(),

apps/webapp/app/routes/api.v1.artifacts.ts

@@ -63,10 +63,15 @@ export async function action({ request }: ActionFunctionArgs) {
             logger.error("Failed to create presigned POST", { error });
             return json({ error: "Failed to generate artifact upload URL" }, { status: 500 });
           }
-          default:
+          case "artifacts_bucket_not_configured": {
+            logger.error("Artifacts bucket not configured", { error });
+            return json({ error: "Internal server error" }, { status: 500 });
+          }
+          default: {
             error satisfies never;
             logger.error("Failed creating artifact", { error });
             return json({ error: "Internal server error" }, { status: 500 });
+          }
         }
       }
     );

apps/webapp/app/v3/services/artifacts.server.ts

@@ -61,6 +61,12 @@ export class ArtifactsService extends BaseService {
   }
 
   private createPresignedPost(key: string, sizeLimit: number, contentLength?: number) {
+    if (!this.bucket) {
+      return errAsync({
+        type: "artifacts_bucket_not_configured" as const,
+      });
+    }
+
     const ttlSeconds = 300; // 5 minutes
     const expiresAt = new Date(Date.now() + ttlSeconds * 1000);