fix: CI test failures, undefined prefix bug, and duplicate docs entries

- Fix objectStore test failures: replace manual PrismaClient(process.env.DATABASE_URL)
  with proper postgresAndMinioTest fixture (new combined testcontainer)
- Fix undefined prefix in uploadDataToObjectStore: pathname now conditional
- Remove duplicate MAXIMUM_DEV_QUEUE_SIZE/MAXIMUM_DEPLOYED_QUEUE_SIZE rows from docs

Author: matt-aitken

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

@@ -293,7 +293,7 @@ export async function uploadDataToObjectStore(
     });
 
     const url = new URL(config.baseUrl);
-    url.pathname = `${prefix}/${filename}`;
+    url.pathname = prefix ? `/${prefix}/${filename}` : `/${filename}`;
 
     logger.debug("Uploading to object store", { url: url.href, protocol: protocol || "default" });
 

apps/webapp/test/objectStore.test.ts

@@ -1,4 +1,4 @@
-import { minioTest } from "@internal/testcontainers";
+import { postgresAndMinioTest } from "@internal/testcontainers";
 import { type IOPacket } from "@trigger.dev/core/v3";
 import { PrismaClient } from "@trigger.dev/database";
 import { afterAll, describe, expect, it, vi } from "vitest";
@@ -10,7 +10,7 @@ import {
 } from "~/v3/objectStore.server";
 
 // Extend the timeout for container tests
-vi.setConfig({ testTimeout: 30_000 });
+vi.setConfig({ testTimeout: 60_000 });
 
 // Helper to create a test environment
 async function createTestEnvironment(prisma: PrismaClient) {
@@ -54,21 +54,6 @@ async function createTestEnvironment(prisma: PrismaClient) {
 // Mock env module for testing
 const originalEnv = process.env;
 
-// Create extended test with Prisma client
-const minioWithPrismaTest = minioTest.extend<{ prisma: PrismaClient }>({
-  prisma: async ({}, use) => {
-    const prisma = new PrismaClient({
-      datasources: {
-        db: {
-          url: process.env.DATABASE_URL,
-        },
-      },
-    });
-    await use(prisma);
-    await prisma.$disconnect();
-  },
-});
-
 describe("Object Storage", () => {
   describe("URI parsing functions", () => {
     it("should parse URI with protocol", () => {
@@ -106,7 +91,7 @@ describe("Object Storage", () => {
     });
   });
 
-  minioWithPrismaTest(
+  postgresAndMinioTest(
     "should upload and download data without protocol (legacy)",
     async ({ minioConfig, prisma }) => {
       // Set up env for default provider
@@ -151,7 +136,7 @@ describe("Object Storage", () => {
     }
   );
 
-  minioWithPrismaTest(
+  postgresAndMinioTest(
     "should upload and download data with protocol prefix",
     async ({ minioConfig, prisma }) => {
       // Set up env for named provider
@@ -197,7 +182,7 @@ describe("Object Storage", () => {
     }
   );
 
-  minioWithPrismaTest(
+  postgresAndMinioTest(
     "should support migration from default provider to named provider",
     async ({ minioConfig, prisma }) => {
       const environment = await createTestEnvironment(prisma);

docs/self-hosting/env/webapp.mdx

@@ -143,8 +143,6 @@ mode: "wide"
 | `RUN_ENGINE_RATE_LIMIT_REJECTION_LOGS_ENABLED`   | No       | 1                     | Run engine rate limit rejection logs.                                                                              |
 | `RUN_ENGINE_RATE_LIMIT_LIMITER_LOGS_ENABLED`     | No       | 0                     | Run engine rate limit limiter logs.                                                                                |
 | `RUN_ENGINE_DEFAULT_MAX_TTL`                     | No       | —                     | Maximum TTL for all runs (e.g. "14d"). Runs without a TTL use this as default; runs with a larger TTL are clamped. |
-| `MAXIMUM_DEV_QUEUE_SIZE`                         | No       | —                     | Maximum queued runs per queue in development environments.                                                         |
-| `MAXIMUM_DEPLOYED_QUEUE_SIZE`                    | No       | —                     | Maximum queued runs per queue in deployed (staging/prod) environments.                                             |
 | **Misc**                                         |          |                       |                                                                                                                    |
 | `TRIGGER_TELEMETRY_DISABLED`                     | No       | —                     | Disable telemetry.                                                                                                 |
 | `NODE_MAX_OLD_SPACE_SIZE`                        | No       | 8192                  | Maximum memory allocation for Node.js heap in MiB (e.g. "4096" for 4GB).                                           |

internal-packages/testcontainers/src/index.ts

@@ -292,3 +292,13 @@ export const minioTest = test.extend<MinIOContext>({
   minioContainer,
   minioConfig,
 });
+
+type PostgresAndMinIOContext = NetworkContext & PostgresContext & MinIOContext;
+
+export const postgresAndMinioTest = test.extend<PostgresAndMinIOContext>({
+  network,
+  postgresContainer,
+  prisma,
+  minioContainer,
+  minioConfig,
+});