Refresh secret schemas and shared test mocks

Author: Sg312

apps/sim/lib/copilot/generated/tool-catalog-v1.ts

@@ -5265,10 +5265,19 @@ export const SetEnvironmentVariables: ToolCatalogEntry = {
         items: {
           type: 'object',
           properties: {
+            description: {
+              type: 'string',
+              description:
+                'What the variable is for, in one short phrase — aim for under 80 characters, like "Stripe live key for the billing workflow". Not a sentence, and never a restatement of the name. Workspace scope only; sending it with scope personal is rejected. Omit it on an existing variable to leave its current description untouched; send an empty string to clear one. You may send it alone, without a value, to describe a secret that already exists.',
+            },
             name: { type: 'string', description: 'Variable name' },
-            value: { type: 'string', description: 'Variable value' },
+            value: {
+              type: 'string',
+              description:
+                "Variable value. Omit it to leave an existing variable's value untouched and change only its description — never invent or guess a value you were not given, which would overwrite the real secret.",
+            },
           },
-          required: ['name', 'value'],
+          required: ['name'],
         },
       },
     },

apps/sim/lib/copilot/generated/tool-schemas-v1.ts

@@ -5168,16 +5168,22 @@ export const TOOL_RUNTIME_SCHEMAS: Record<string, ToolRuntimeSchemaEntry> = {
           items: {
             type: 'object',
             properties: {
+              description: {
+                type: 'string',
+                description:
+                  'What the variable is for, in one short phrase — aim for under 80 characters, like "Stripe live key for the billing workflow". Not a sentence, and never a restatement of the name. Workspace scope only; sending it with scope personal is rejected. Omit it on an existing variable to leave its current description untouched; send an empty string to clear one. You may send it alone, without a value, to describe a secret that already exists.',
+              },
               name: {
                 type: 'string',
                 description: 'Variable name',
               },
               value: {
                 type: 'string',
-                description: 'Variable value',
+                description:
+                  "Variable value. Omit it to leave an existing variable's value untouched and change only its description — never invent or guess a value you were not given, which would overwrite the real secret.",
               },
             },
-            required: ['name', 'value'],
+            required: ['name'],
           },
         },
       },

packages/testing/src/mocks/logger.mock.ts

@@ -39,6 +39,7 @@ export const loggerMock = {
   logger: createMockLogger(),
   runWithRequestContext: vi.fn(<T>(_ctx: unknown, fn: () => T): T => fn()),
   getRequestContext: vi.fn(() => undefined),
+  setRequestTraceId: vi.fn(),
 }
 
 /**

packages/testing/src/mocks/permissions.mock.ts

@@ -45,4 +45,9 @@ export const permissionsMock = {
   getWorkspaceMemberProfiles: permissionsMockFns.mockGetWorkspaceMemberProfiles,
   hasWorkspaceAdminAccess: permissionsMockFns.mockHasWorkspaceAdminAccess,
   getManageableWorkspaces: permissionsMockFns.mockGetManageableWorkspaces,
+  // Faithful to the real type guard without importing the class the mock
+  // replaces: the constructor stamps name = 'WorkspaceAccessDeniedError', so a
+  // test exercising the 403 projection throws an Error carrying that name.
+  isWorkspaceAccessDeniedError: (error: unknown): boolean =>
+    error instanceof Error && error.name === 'WorkspaceAccessDeniedError',
 }