random: set CONFIG_MAX_REPOS_NO_TOKEN default to max int, effectively disabling this feature by default (since this is only useful in certain scenarios like cloud)

Author: brendan-kellam

packages/web/src/actions.ts

@@ -1433,6 +1433,15 @@ const parseConnectionConfig = (connectionType: string, config: string) => {
         } satisfies ServiceError;
     }
 
+    const isValidConfig = ajv.validate(schema, parsedConfig);
+    if (!isValidConfig) {
+        return {
+            statusCode: StatusCodes.BAD_REQUEST,
+            errorCode: ErrorCode.INVALID_REQUEST_BODY,
+            message: `config schema validation failed with errors: ${ajv.errorsText(ajv.errors)}`,
+        } satisfies ServiceError;
+    }
+
     const { numRepos, hasToken } = (() => {
         switch (connectionType) {
             case "github": {
@@ -1479,15 +1488,6 @@ const parseConnectionConfig = (connectionType: string, config: string) => {
         } satisfies ServiceError;
     }
 
-    const isValidConfig = ajv.validate(schema, parsedConfig);
-    if (!isValidConfig) {
-        return {
-            statusCode: StatusCodes.BAD_REQUEST,
-            errorCode: ErrorCode.INVALID_REQUEST_BODY,
-            message: `config schema validation failed with errors: ${ajv.errorsText(ajv.errors)}`,
-        } satisfies ServiceError;
-    }
-
     return parsedConfig;
 }
 

packages/web/src/env.mjs

@@ -37,7 +37,7 @@ export const env = createEnv({
         STRIPE_ENABLE_TEST_CLOCKS: booleanSchema.default('false'),
 
         // Misc
-        CONFIG_MAX_REPOS_NO_TOKEN: numberSchema.default(500),
+        CONFIG_MAX_REPOS_NO_TOKEN: numberSchema.default(Number.MAX_SAFE_INTEGER),
         SOURCEBOT_ROOT_DOMAIN: z.string().default("localhost:3000"),
         NODE_ENV: z.enum(["development", "test", "production"]),
         SOURCEBOT_TELEMETRY_DISABLED: booleanSchema.default('false'),