Skip to content

feat(log): Add wrapper function for standardized logging #15034

feat(log): Add wrapper function for standardized logging

feat(log): Add wrapper function for standardized logging #15034

Triggered via pull request April 21, 2026 04:14
Status Failure
Total duration 1m 23s
Artifacts

ci.yml

on: pull_request
Detect Version
0s
Detect Version
Test and Build  /  Test and Build
1m 14s
Test and Build / Test and Build
Check Docs Changes
0s
Check Docs Changes
Matrix: Build Dev ECR
Matrix: Build ARM64 (GHCR Only)
Matrix: Build AMD64
Migrate Dev DB  /  Apply Database Migrations
Migrate Dev DB / Apply Database Migrations
Matrix: Create GHCR Manifests
Process Docs  /  Process Documentation Embeddings
Process Docs / Process Documentation Embeddings
Create GitHub Release
Create GitHub Release
Fit to window
Zoom out
Zoom in

Annotations

10 errors and 7 warnings
app/api/chat/route.test.ts > Chat API Route > POST > should reject if identifier already exists: apps/sim/lib/core/utils/with-route-handler.ts#L29
Error: [vitest] No "runWithRequestContext" export is defined on the "@sim/logger" mock. Did you forget to return it from "vi.mock"? If you need to partially mock a module, you can use "importOriginal" helper inside: vi.mock(import("@sim/logger"), async (importOriginal) => { const actual = await importOriginal() return { ...actual, // your mocked methods } }) ❯ Module.<anonymous> lib/core/utils/with-route-handler.ts:29:12 ❯ app/api/chat/route.test.ts:165:30
app/api/chat/route.test.ts > Chat API Route > POST > should validate request data: apps/sim/lib/core/utils/with-route-handler.ts#L29
Error: [vitest] No "runWithRequestContext" export is defined on the "@sim/logger" mock. Did you forget to return it from "vi.mock"? If you need to partially mock a module, you can use "importOriginal" helper inside: vi.mock(import("@sim/logger"), async (importOriginal) => { const actual = await importOriginal() return { ...actual, // your mocked methods } }) ❯ Module.<anonymous> lib/core/utils/with-route-handler.ts:29:12 ❯ app/api/chat/route.test.ts:138:30
app/api/chat/route.test.ts > Chat API Route > POST > should return 401 when user is not authenticated: apps/sim/lib/core/utils/with-route-handler.ts#L29
Error: [vitest] No "runWithRequestContext" export is defined on the "@sim/logger" mock. Did you forget to return it from "vi.mock"? If you need to partially mock a module, you can use "importOriginal" helper inside: vi.mock(import("@sim/logger"), async (importOriginal) => { const actual = await importOriginal() return { ...actual, // your mocked methods } }) ❯ Module.<anonymous> lib/core/utils/with-route-handler.ts:29:12 ❯ app/api/chat/route.test.ts:121:30
app/api/chat/route.test.ts > Chat API Route > GET > should handle errors when fetching deployments: apps/sim/lib/core/utils/with-route-handler.ts#L29
Error: [vitest] No "runWithRequestContext" export is defined on the "@sim/logger" mock. Did you forget to return it from "vi.mock"? If you need to partially mock a module, you can use "importOriginal" helper inside: vi.mock(import("@sim/logger"), async (importOriginal) => { const actual = await importOriginal() return { ...actual, // your mocked methods } }) ❯ Module.<anonymous> lib/core/utils/with-route-handler.ts:29:12 ❯ app/api/chat/route.test.ts:106:30
app/api/chat/route.test.ts > Chat API Route > GET > should return chat deployments for authenticated user: apps/sim/lib/core/utils/with-route-handler.ts#L29
Error: [vitest] No "runWithRequestContext" export is defined on the "@sim/logger" mock. Did you forget to return it from "vi.mock"? If you need to partially mock a module, you can use "importOriginal" helper inside: vi.mock(import("@sim/logger"), async (importOriginal) => { const actual = await importOriginal() return { ...actual, // your mocked methods } }) ❯ Module.<anonymous> lib/core/utils/with-route-handler.ts:29:12 ❯ app/api/chat/route.test.ts:91:30
app/api/chat/route.test.ts > Chat API Route > GET > should return 401 when user is not authenticated: apps/sim/lib/core/utils/with-route-handler.ts#L29
Error: [vitest] No "runWithRequestContext" export is defined on the "@sim/logger" mock. Did you forget to return it from "vi.mock"? If you need to partially mock a module, you can use "importOriginal" helper inside: vi.mock(import("@sim/logger"), async (importOriginal) => { const actual = await importOriginal() return { ...actual, // your mocked methods } }) ❯ Module.<anonymous> lib/core/utils/with-route-handler.ts:29:12 ❯ app/api/chat/route.test.ts:76:30
background/async-preprocessing-correlation.test.ts > async preprocessing correlation threading > passes schedule correlation into preprocessing: apps/sim/background/schedule-execution.ts#L331
Error: [vitest] No "runWithRequestContext" export is defined on the "@sim/logger" mock. Did you forget to return it from "vi.mock"? If you need to partially mock a module, you can use "importOriginal" helper inside: vi.mock(import("@sim/logger"), async (importOriginal) => { const actual = await importOriginal() return { ...actual, // your mocked methods } }) ❯ Module.executeScheduleJob background/schedule-execution.ts:331:10 ❯ background/async-preprocessing-correlation.test.ts:232:11
background/async-preprocessing-correlation.test.ts > async preprocessing correlation threading > does not pre-start schedule logging before core execution: apps/sim/background/schedule-execution.ts#L331
Error: [vitest] No "runWithRequestContext" export is defined on the "@sim/logger" mock. Did you forget to return it from "vi.mock"? If you need to partially mock a module, you can use "importOriginal" helper inside: vi.mock(import("@sim/logger"), async (importOriginal) => { const actual = await importOriginal() return { ...actual, // your mocked methods } }) ❯ Module.executeScheduleJob background/schedule-execution.ts:331:10 ❯ background/async-preprocessing-correlation.test.ts:176:11
background/async-preprocessing-correlation.test.ts > async preprocessing correlation threading > does not pre-start workflow logging before core execution: apps/sim/background/workflow-execution.ts#L62
Error: [vitest] No "runWithRequestContext" export is defined on the "@sim/logger" mock. Did you forget to return it from "vi.mock"? If you need to partially mock a module, you can use "importOriginal" helper inside: vi.mock(import("@sim/logger"), async (importOriginal) => { const actual = await importOriginal() return { ...actual, // your mocked methods } }) ❯ Module.executeWorkflowJob background/workflow-execution.ts:62:10 ❯ background/async-preprocessing-correlation.test.ts:139:11
Test and Build / Test and Build
sim#test: command (/home/runner/_work/sim/sim/apps/sim) /home/runner/.bun/bin/bun run test exited (1)
Test and Build / Test and Build
Node.js 20 actions are deprecated. The following actions are running on Node.js 20 and may not work as expected: actions/cache@v4, actions/setup-node@v4. Actions will be forced to run with Node.js 24 by default starting June 2nd, 2026. Node.js 20 will be removed from the runner on September 16th, 2026. Please check if updated versions of these actions are available that support Node.js 24. To opt into Node.js 24 now, set the FORCE_JAVASCRIPT_ACTIONS_TO_NODE24=true environment variable on the runner or in your workflow file. Once Node.js 24 becomes the default, you can temporarily opt out by setting ACTIONS_ALLOW_USE_UNSECURE_NODE_VERSION=true. For more information see: https://github.blog/changelog/2025-09-19-deprecation-of-node-20-on-github-actions-runners/
Test and Build / Test and Build
Skipping sticky disk commit due to previous step failures
Test and Build / Test and Build
- Step: unknown (failed)
Test and Build / Test and Build
Found 2 failed/cancelled steps in previous workflow steps
Test and Build / Test and Build
Skipping sticky disk commit due to previous step failures
Test and Build / Test and Build
- Step: unknown (failed)
Test and Build / Test and Build
Found 2 failed/cancelled steps in previous workflow steps