feat(log): Add wrapper function for standardized logging #15037
ci.yml
on: pull_request
Detect Version
Test and Build
/
Test and Build
1m 36s
Check Docs Changes
Matrix: Build Dev ECR
Matrix: Build ARM64 (GHCR Only)
Matrix: Build AMD64
Migrate Dev DB
/
Apply Database Migrations
Matrix: Create GHCR Manifests
Process Docs
/
Process Documentation Embeddings
Create GitHub Release
0s
Annotations
10 errors and 7 warnings
|
app/api/folders/route.test.ts > Folders API Route > POST /api/folders > should create subfolder with parent reference:
apps/sim/lib/core/utils/with-route-handler.ts#L30
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:30:12
❯ app/api/folders/route.test.ts:350:30
|
|
app/api/folders/route.test.ts > Folders API Route > POST /api/folders > should create folder with correct sort order:
apps/sim/lib/core/utils/with-route-handler.ts#L30
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:30:12
❯ app/api/folders/route.test.ts:322:30
|
|
app/api/folders/route.test.ts > Folders API Route > POST /api/folders > should create a new folder successfully:
apps/sim/lib/core/utils/with-route-handler.ts#L30
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:30:12
❯ app/api/folders/route.test.ts:291:30
|
|
app/api/folders/route.test.ts > Folders API Route > GET /api/folders > should handle database errors gracefully:
apps/sim/lib/core/utils/with-route-handler.ts#L30
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:30:12
❯ app/api/folders/route.test.ts:262:30
|
|
app/api/folders/route.test.ts > Folders API Route > GET /api/folders > should return 403 when user has only read permissions:
apps/sim/lib/core/utils/with-route-handler.ts#L30
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:30:12
❯ app/api/folders/route.test.ts:242:30
|
|
app/api/folders/route.test.ts > Folders API Route > GET /api/folders > should return 403 when user has no workspace permissions:
apps/sim/lib/core/utils/with-route-handler.ts#L30
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:30:12
❯ app/api/folders/route.test.ts:225:30
|
|
app/api/folders/route.test.ts > Folders API Route > GET /api/folders > should return 400 when workspaceId is missing:
apps/sim/lib/core/utils/with-route-handler.ts#L30
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:30:12
❯ app/api/folders/route.test.ts:208:30
|
|
app/api/folders/route.test.ts > Folders API Route > GET /api/folders > should return 401 for unauthenticated requests:
apps/sim/lib/core/utils/with-route-handler.ts#L30
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:30:12
❯ app/api/folders/route.test.ts:192:30
|
|
app/api/folders/route.test.ts > Folders API Route > GET /api/folders > should return folders for a valid workspace:
apps/sim/lib/core/utils/with-route-handler.ts#L30
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:30:12
❯ app/api/folders/route.test.ts:170:30
|
|
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
|