Skip to content

Commit de66619

Browse files
committed
fix(analytics): tolerate get-only header readers, gate CLI notice on stderr, re-read telemetry state before send
1 parent 019cdad commit de66619

30 files changed

Lines changed: 86 additions & 27 deletions

File tree

‎apps/sim/app/_shell/providers/posthog-provider.test.tsx‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,7 @@ const { consent, mockCapture, mockInit, mockOptIn, mockOptOut, mockPostHog, mock
1212
capture: vi.fn(),
1313
init: vi.fn(),
1414
opt_in_capturing: vi.fn(),
15+
register: vi.fn(),
1516
opt_out_capturing: vi.fn(),
1617
}
1718
posthog.init.mockImplementation(() => {

‎apps/sim/app/api/auth/forget-password/route.test.ts‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -61,6 +61,7 @@ vi.mock('@sim/logger', () => ({
6161
createLogger: vi.fn().mockReturnValue(mockLogger),
6262
runWithRequestContext: <T>(_ctx: unknown, fn: () => T): T => fn(),
6363
getRequestContext: () => undefined,
64+
setRequestAuth: vi.fn(),
6465
}))
6566

6667
import { POST } from '@/app/api/auth/forget-password/route'

‎apps/sim/app/api/auth/reset-password/route.test.ts‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -43,6 +43,7 @@ vi.mock('@sim/logger', () => ({
4343
createLogger: vi.fn().mockReturnValue(mockLogger),
4444
runWithRequestContext: <T>(_ctx: unknown, fn: () => T): T => fn(),
4545
getRequestContext: () => undefined,
46+
setRequestAuth: vi.fn(),
4647
}))
4748

4849
import { POST } from '@/app/api/auth/reset-password/route'

‎apps/sim/app/api/files/serve/[...path]/route.test.ts‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,7 @@ vi.mock('@sim/logger', () => ({
1313
logger: serveLogger,
1414
runWithRequestContext: vi.fn(<T>(_ctx: unknown, fn: () => T): T => fn()),
1515
getRequestContext: vi.fn(() => undefined),
16+
setRequestAuth: vi.fn(),
1617
}))
1718

1819
const {

‎apps/sim/app/api/folders/[id]/duplicate/route.test.ts‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -51,6 +51,7 @@ vi.mock('@sim/logger', () => ({
5151
createLogger: vi.fn().mockReturnValue(mockLogger),
5252
runWithRequestContext: <T>(_ctx: unknown, fn: () => T): T => fn(),
5353
getRequestContext: () => undefined,
54+
setRequestAuth: vi.fn(),
5455
}))
5556
vi.mock('@/lib/workspaces/permissions/utils', () => permissionsMock)
5657
vi.mock('@/lib/folders/orchestration', () => ({ nextFolderSortOrder: mockNextFolderSortOrder }))

‎apps/sim/app/api/folders/[id]/route.test.ts‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -47,6 +47,7 @@ vi.mock('@sim/logger', () => ({
4747
createLogger: vi.fn().mockReturnValue(mockLogger),
4848
runWithRequestContext: <T>(_ctx: unknown, fn: () => T): T => fn(),
4949
getRequestContext: () => undefined,
50+
setRequestAuth: vi.fn(),
5051
}))
5152
vi.mock('@/lib/workspaces/permissions/utils', () => permissionsMock)
5253
vi.mock('@/lib/folders/orchestration', () => foldersOrchestrationMock)

‎apps/sim/app/api/folders/route.test.ts‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -35,6 +35,7 @@ vi.mock('@sim/logger', () => ({
3535
createLogger: vi.fn().mockReturnValue(mockLogger),
3636
runWithRequestContext: <T>(_ctx: unknown, fn: () => T): T => fn(),
3737
getRequestContext: () => undefined,
38+
setRequestAuth: vi.fn(),
3839
}))
3940
vi.mock('@/lib/workspaces/permissions/utils', () => permissionsMock)
4041

‎apps/sim/app/api/pinned-items/[resourceType]/[resourceId]/route.test.ts‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -23,6 +23,7 @@ vi.mock('@sim/logger', () => ({
2323
createLogger: vi.fn().mockReturnValue(mockLogger),
2424
runWithRequestContext: <T>(_ctx: unknown, fn: () => T): T => fn(),
2525
getRequestContext: () => undefined,
26+
setRequestAuth: vi.fn(),
2627
}))
2728
vi.mock('@sim/db', () => ({ db: mockDb, ...schemaMock }))
2829

‎apps/sim/app/api/pinned-items/route.test.ts‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -31,6 +31,7 @@ vi.mock('@sim/logger', () => ({
3131
createLogger: vi.fn().mockReturnValue(mockLogger),
3232
runWithRequestContext: <T>(_ctx: unknown, fn: () => T): T => fn(),
3333
getRequestContext: () => undefined,
34+
setRequestAuth: vi.fn(),
3435
}))
3536
vi.mock('@/lib/workspaces/permissions/utils', () => permissionsMock)
3637
// The route and `lib/pinned-items/resources` import both the db client AND the table

‎apps/sim/app/api/providers/ollama/models/route.test.ts‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -23,6 +23,7 @@ vi.mock('@sim/logger', () => ({
2323
logger: ollamaLogger,
2424
runWithRequestContext: vi.fn(<T>(_ctx: unknown, fn: () => T): T => fn()),
2525
getRequestContext: vi.fn(() => undefined),
26+
setRequestAuth: vi.fn(),
2627
}))
2728

2829
vi.mock('@/providers/utils', () => ({

0 commit comments

Comments
 (0)