Skip to content

Commit 9fec91e

Browse files
committed
improvement(audits): rebaseline ratchets on current staging, lint newly covered uploads files, anchor suppression detection to comments
1 parent e76cf6b commit 9fec91e

6 files changed

Lines changed: 192 additions & 40 deletions

File tree

‎apps/sim/lib/uploads/contexts/workspace/workspace-file-query.test.ts‎

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -152,9 +152,6 @@ describe('listWorkspaceFiles', () => {
152152
resetDbChainMock()
153153
})
154154

155-
const lastProjection = () =>
156-
Object.keys((dbChainMockFns.select.mock.calls.at(-1)?.[0] ?? {}) as Record<string, unknown>)
157-
158155
it('caps the rows read when the caller only needs to fit a budget', async () => {
159156
queueTableRows(schemaMock.workspaceFiles, [buildRow()])
160157

‎apps/sim/lib/uploads/providers/s3/client.test.ts‎

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,6 @@ import { afterAll, beforeEach, describe, expect, it, vi } from 'vitest'
77

88
const {
99
mockSend,
10-
mockS3Client,
1110
mockS3ClientConstructor,
1211
mockPutObjectCommand,
1312
mockGetObjectCommand,

‎apps/sim/lib/uploads/utils/file-utils.ts‎

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -10,14 +10,6 @@ import {
1010
import { isUuid } from '@/executor/constants'
1111
import type { UserFile } from '@/executor/types'
1212

13-
interface FileAttachment {
14-
id: string
15-
key: string
16-
filename: string
17-
media_type: string
18-
size: number
19-
}
20-
2113
export interface MessageContent {
2214
type: 'text' | 'image' | 'document' | 'audio' | 'video'
2315
text?: string

‎scripts/check-explicit-any.baseline.json‎

Lines changed: 19 additions & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@
55
"apps/realtime/src/handlers/variables.ts": 2,
66
"apps/sim/app/(auth)/login/login-form.tsx": 2,
77
"apps/sim/app/(auth)/verify/use-verification.ts": 1,
8-
"apps/sim/app/(interfaces)/resume/[workflowId]/[executionId]/resume-page-client.tsx": 17,
8+
"apps/sim/app/(interfaces)/resume/[workflowId]/[executionId]/resume-page-client.tsx": 16,
99
"apps/sim/app/api/auth/oauth/utils.test.ts": 2,
1010
"apps/sim/app/api/auth/sso/register/route.ts": 3,
1111
"apps/sim/app/api/chat/[identifier]/route.test.ts": 1,
@@ -33,7 +33,6 @@
3333
"apps/sim/app/api/workflows/route.ts": 1,
3434
"apps/sim/app/api/workflows/utils.ts": 1,
3535
"apps/sim/app/workspace/[workspaceId]/w/[workflowId]/components/chat/components/chat-message/chat-message.tsx": 1,
36-
"apps/sim/app/workspace/[workspaceId]/w/[workflowId]/components/panel/components/deploy/hooks/use-change-detection.ts": 2,
3736
"apps/sim/app/workspace/[workspaceId]/w/[workflowId]/components/panel/components/editor/components/connection-blocks/connection-blocks.tsx": 3,
3837
"apps/sim/app/workspace/[workspaceId]/w/[workflowId]/components/panel/components/editor/components/sub-block/components/checkbox-list/checkbox-list.tsx": 1,
3938
"apps/sim/app/workspace/[workspaceId]/w/[workflowId]/components/panel/components/editor/components/sub-block/components/condition-input/condition-input.tsx": 2,
@@ -79,15 +78,15 @@
7978
"apps/sim/app/workspace/providers/socket-provider.tsx": 10,
8079
"apps/sim/background/workflow-execution.ts": 2,
8180
"apps/sim/blocks/blocks/agent.test.ts": 1,
82-
"apps/sim/blocks/blocks/agent.ts": 7,
81+
"apps/sim/blocks/blocks/agent.ts": 6,
8382
"apps/sim/blocks/blocks/airtable.ts": 5,
8483
"apps/sim/blocks/blocks/apify.ts": 2,
8584
"apps/sim/blocks/blocks/browser_use.ts": 2,
8685
"apps/sim/blocks/blocks/context_dev.ts": 1,
8786
"apps/sim/blocks/blocks/deployments.ts": 2,
8887
"apps/sim/blocks/blocks/discord.ts": 1,
8988
"apps/sim/blocks/blocks/enrich.ts": 2,
90-
"apps/sim/blocks/blocks/evaluator.ts": 5,
89+
"apps/sim/blocks/blocks/evaluator.ts": 4,
9190
"apps/sim/blocks/blocks/firecrawl.ts": 1,
9291
"apps/sim/blocks/blocks/github.ts": 2,
9392
"apps/sim/blocks/blocks/gitlab.ts": 2,
@@ -109,7 +108,6 @@
109108
"apps/sim/blocks/blocks/mongodb.ts": 1,
110109
"apps/sim/blocks/blocks/mysql.ts": 1,
111110
"apps/sim/blocks/blocks/neo4j.ts": 1,
112-
"apps/sim/blocks/blocks/notion.ts": 1,
113111
"apps/sim/blocks/blocks/pinecone.ts": 3,
114112
"apps/sim/blocks/blocks/pipedrive.ts": 1,
115113
"apps/sim/blocks/blocks/polymarket.ts": 1,
@@ -175,7 +173,7 @@
175173
"apps/sim/executor/orchestrators/loop.ts": 2,
176174
"apps/sim/executor/orchestrators/parallel.test.ts": 2,
177175
"apps/sim/executor/orchestrators/parallel.ts": 5,
178-
"apps/sim/executor/types.ts": 24,
176+
"apps/sim/executor/types.ts": 18,
179177
"apps/sim/executor/utils.ts": 1,
180178
"apps/sim/executor/utils/errors.ts": 1,
181179
"apps/sim/executor/utils/json.ts": 1,
@@ -205,11 +203,10 @@
205203
"apps/sim/hooks/use-undo-redo.ts": 43,
206204
"apps/sim/lib/auth/auth.ts": 3,
207205
"apps/sim/lib/auth/connectors/providers.ts": 4,
208-
"apps/sim/lib/billing/client/types.ts": 2,
206+
"apps/sim/lib/billing/client/types.ts": 1,
209207
"apps/sim/lib/billing/client/upgrade.ts": 3,
210208
"apps/sim/lib/billing/core/subscription.ts": 1,
211209
"apps/sim/lib/billing/subscriptions/utils.ts": 7,
212-
"apps/sim/lib/billing/types/index.ts": 6,
213210
"apps/sim/lib/content/mdx.tsx": 15,
214211
"apps/sim/lib/content/registry-factory.ts": 3,
215212
"apps/sim/lib/core/config/redis.ts": 1,
@@ -257,7 +254,7 @@
257254
"apps/sim/lib/webhooks/providers/vercel.test.ts": 2,
258255
"apps/sim/lib/webhooks/providers/zoho-desk.test.ts": 8,
259256
"apps/sim/lib/workflows/blocks/block-outputs.ts": 8,
260-
"apps/sim/lib/workflows/comparison/compare.test.ts": 13,
257+
"apps/sim/lib/workflows/comparison/compare.test.ts": 15,
261258
"apps/sim/lib/workflows/comparison/format-description.test.ts": 1,
262259
"apps/sim/lib/workflows/custom-blocks/child-execution.test.ts": 1,
263260
"apps/sim/lib/workflows/custom-tools/operations.ts": 1,
@@ -386,7 +383,6 @@
386383
"apps/sim/tools/confluence/create_page.ts": 2,
387384
"apps/sim/tools/confluence/create_page_property.ts": 2,
388385
"apps/sim/tools/confluence/list_page_properties.ts": 1,
389-
"apps/sim/tools/confluence/types.ts": 1,
390386
"apps/sim/tools/confluence/update.ts": 1,
391387
"apps/sim/tools/confluence/upload_attachment.ts": 1,
392388
"apps/sim/tools/confluence/utils.ts": 1,
@@ -419,7 +415,7 @@
419415
"apps/sim/tools/discord/create_thread.ts": 1,
420416
"apps/sim/tools/discord/edit_message.ts": 1,
421417
"apps/sim/tools/discord/execute_webhook.ts": 1,
422-
"apps/sim/tools/discord/types.ts": 23,
418+
"apps/sim/tools/discord/types.ts": 22,
423419
"apps/sim/tools/discord/update_channel.ts": 1,
424420
"apps/sim/tools/discord/update_member.ts": 1,
425421
"apps/sim/tools/discord/update_role.ts": 1,
@@ -598,7 +594,6 @@
598594
"apps/sim/tools/google_slides/types.ts": 1,
599595
"apps/sim/tools/google_vault/create_matters_export.ts": 1,
600596
"apps/sim/tools/google_vault/create_matters_holds.ts": 2,
601-
"apps/sim/tools/google_vault/types.ts": 2,
602597
"apps/sim/tools/google_vault/update_matters_holds.ts": 2,
603598
"apps/sim/tools/grafana/create_dashboard.ts": 2,
604599
"apps/sim/tools/grain/create_hook.ts": 1,
@@ -658,7 +653,7 @@
658653
"apps/sim/tools/intercom/snooze_conversation.ts": 2,
659654
"apps/sim/tools/intercom/tag_contact.ts": 1,
660655
"apps/sim/tools/intercom/tag_conversation.ts": 1,
661-
"apps/sim/tools/intercom/types.ts": 2,
656+
"apps/sim/tools/intercom/types.ts": 1,
662657
"apps/sim/tools/intercom/untag_contact.ts": 1,
663658
"apps/sim/tools/intercom/update_contact.ts": 4,
664659
"apps/sim/tools/intercom/update_ticket.ts": 3,
@@ -688,7 +683,7 @@
688683
"apps/sim/tools/kalshi/amend_order.ts": 3,
689684
"apps/sim/tools/kalshi/create_order.ts": 2,
690685
"apps/sim/tools/kalshi/get_orders.ts": 1,
691-
"apps/sim/tools/kalshi/types.ts": 2,
686+
"apps/sim/tools/kalshi/types.ts": 1,
692687
"apps/sim/tools/knowledge/create_document.ts": 1,
693688
"apps/sim/tools/knowledge/delete_chunk.ts": 1,
694689
"apps/sim/tools/knowledge/delete_document.ts": 1,
@@ -746,7 +741,7 @@
746741
"apps/sim/tools/mailchimp/get_interests.ts": 1,
747742
"apps/sim/tools/mailchimp/get_merge_field.ts": 1,
748743
"apps/sim/tools/mailchimp/get_segments.ts": 1,
749-
"apps/sim/tools/memory/types.ts": 2,
744+
"apps/sim/tools/memory/types.ts": 1,
750745
"apps/sim/tools/microsoft_excel/table_add.ts": 3,
751746
"apps/sim/tools/microsoft_excel/utils.ts": 5,
752747
"apps/sim/tools/microsoft_excel/write.ts": 8,
@@ -795,7 +790,7 @@
795790
"apps/sim/tools/pinecone/list_indexes.ts": 1,
796791
"apps/sim/tools/pinecone/search_text.ts": 1,
797792
"apps/sim/tools/pinecone/search_vector.ts": 1,
798-
"apps/sim/tools/pinecone/types.ts": 14,
793+
"apps/sim/tools/pinecone/types.ts": 11,
799794
"apps/sim/tools/pipedrive/create_activity.ts": 1,
800795
"apps/sim/tools/pipedrive/create_deal.ts": 1,
801796
"apps/sim/tools/pipedrive/create_lead.ts": 1,
@@ -844,7 +839,6 @@
844839
"apps/sim/tools/posthog/list_session_recordings.ts": 1,
845840
"apps/sim/tools/posthog/list_surveys.ts": 3,
846841
"apps/sim/tools/posthog/query.ts": 3,
847-
"apps/sim/tools/posthog/types.ts": 28,
848842
"apps/sim/tools/posthog/update_cohort.ts": 4,
849843
"apps/sim/tools/posthog/update_event_definition.ts": 1,
850844
"apps/sim/tools/posthog/update_experiment.ts": 4,
@@ -876,7 +870,6 @@
876870
"apps/sim/tools/reddit/submit_post.ts": 2,
877871
"apps/sim/tools/reddit/types.ts": 1,
878872
"apps/sim/tools/request-transport.ts": 5,
879-
"apps/sim/tools/response/types.ts": 1,
880873
"apps/sim/tools/s3/utils.ts": 1,
881874
"apps/sim/tools/safe-assign.test.ts": 9,
882875
"apps/sim/tools/salesforce/create_account.ts": 1,
@@ -1080,7 +1073,6 @@
10801073
"apps/sim/triggers/circleback/utils.ts": 1,
10811074
"apps/sim/triggers/confluence/utils.ts": 6,
10821075
"apps/sim/triggers/googleforms/webhook.ts": 1,
1083-
"apps/sim/triggers/types.ts": 1,
10841076
"apps/sim/vitest.setup.ts": 2,
10851077
"packages/db/scripts/deregister-sso-provider.ts": 3,
10861078
"packages/db/scripts/migrate-block-api-keys-to-byok.ts": 2,
@@ -1107,7 +1099,7 @@
11071099
"scripts/check-fork-dependent-coverage.ts": 2,
11081100
"scripts/check-import-specifiers.ts": 1,
11091101
"scripts/create-single-release.ts": 2,
1110-
"scripts/generate-docs.ts": 43,
1102+
"scripts/generate-docs.ts": 41,
11111103
"scripts/generate-v2-cli-api.ts": 3
11121104
},
11131105
"nonNullAssertion": {
@@ -1215,7 +1207,7 @@
12151207
"apps/sim/app/workspace/[workspaceId]/files/components/file-viewer/rich-markdown-editor/rich-markdown-field.test.tsx": 2,
12161208
"apps/sim/app/workspace/[workspaceId]/files/components/file-viewer/xlsx-preview.tsx": 1,
12171209
"apps/sim/app/workspace/[workspaceId]/home/components/knowledge-search-results/knowledge-search-results.tsx": 1,
1218-
"apps/sim/app/workspace/[workspaceId]/home/components/knowledge-search-results/search-transitions.test.tsx": 14,
1210+
"apps/sim/app/workspace/[workspaceId]/home/components/knowledge-search-results/search-transitions.test.tsx": 13,
12191211
"apps/sim/app/workspace/[workspaceId]/home/components/message-content/components/chat-content/external-link.test.tsx": 1,
12201212
"apps/sim/app/workspace/[workspaceId]/home/components/message-content/components/source-chip/source-icon.test.tsx": 1,
12211213
"apps/sim/app/workspace/[workspaceId]/home/components/message-content/components/special-tags/special-tags.test.tsx": 2,
@@ -1264,6 +1256,7 @@
12641256
"apps/sim/app/workspace/[workspaceId]/w/[workflowId]/workflow.tsx": 2,
12651257
"apps/sim/app/workspace/[workspaceId]/w/components/preview/components/preview-editor/preview-editor.tsx": 1,
12661258
"apps/sim/app/workspace/[workspaceId]/w/components/sidebar/components/workspace-header/components/pending-invitations/view-invitations-modal.test.tsx": 1,
1259+
"apps/sim/app/workspace/[workspaceId]/w/components/workflow-diff/components/change-list/value-presentation.test.ts": 17,
12671260
"apps/sim/app/workspace/[workspaceId]/w/hooks/use-import-workflow.ts": 6,
12681261
"apps/sim/background/cleanup-soft-deletes.test.ts": 2,
12691262
"apps/sim/background/cleanup-table-row-ttl.integration.ts": 2,
@@ -1596,6 +1589,7 @@
15961589
"apps/sim/lib/mothership/agent-cli/resource-owner.test.ts": 1,
15971590
"apps/sim/lib/mothership/agent-cli/run-cli-files.test.ts": 1,
15981591
"apps/sim/lib/mothership/agent-cli/saved-run-read.live.test.ts": 18,
1592+
"apps/sim/lib/mothership/agent-cli/workbench-file-provenance.integration.ts": 4,
15991593
"apps/sim/lib/mothership/agent-cli/workbench-file-provenance.ts": 2,
16001594
"apps/sim/lib/mothership/assistant/tool-policy.test.ts": 2,
16011595
"apps/sim/lib/mothership/async-runs/orphaned-runs.integration.ts": 11,
@@ -1737,10 +1731,11 @@
17371731
"apps/sim/lib/webhooks/providers/twilio-voice.test.ts": 11,
17381732
"apps/sim/lib/webhooks/providers/twilio.test.ts": 9,
17391733
"apps/sim/lib/webhooks/providers/vercel.test.ts": 4,
1740-
"apps/sim/lib/webhooks/providers/whatsapp.test.ts": 7,
1734+
"apps/sim/lib/webhooks/providers/whatsapp.test.ts": 6,
17411735
"apps/sim/lib/webhooks/providers/zendesk.test.ts": 7,
17421736
"apps/sim/lib/webhooks/providers/zoom.test.ts": 1,
17431737
"apps/sim/lib/webhooks/slack-execution-stream.test.ts": 14,
1738+
"apps/sim/lib/workflows/application/compare-workflow-versions.integration.ts": 3,
17441739
"apps/sim/lib/workflows/application/inspect-workflow-tools.ts": 1,
17451740
"apps/sim/lib/workflows/application/mapped-import.ts": 1,
17461741
"apps/sim/lib/workflows/application/read-workflow-graph.test.ts": 1,
@@ -1750,6 +1745,7 @@
17501745
"apps/sim/lib/workflows/autolayout/utils.ts": 1,
17511746
"apps/sim/lib/workflows/blocks/block-path-calculator.ts": 1,
17521747
"apps/sim/lib/workflows/blocks/flatten-outputs.ts": 2,
1748+
"apps/sim/lib/workflows/comparison/redact.test.ts": 2,
17531749
"apps/sim/lib/workflows/credentials/credential-extractor.ts": 3,
17541750
"apps/sim/lib/workflows/diff/diff-engine.ts": 1,
17551751
"apps/sim/lib/workflows/editing/block-enablement.ts": 1,
@@ -1969,6 +1965,7 @@
19691965
"packages/emcn/src/components/toast/toast.tsx": 1,
19701966
"packages/sim-cli/src/contract/reference.ts": 6,
19711967
"packages/sim-cli/src/runtime/execute.test.ts": 3,
1968+
"packages/sim-cli/src/runtime/result.test.ts": 1,
19721969
"packages/sim-cli/src/transfer/embedded-files.test.ts": 1,
19731970
"packages/sim-setup/src/lifecycle.test.ts": 3,
19741971
"packages/ts-sdk/examples/basic-usage.ts": 4,

‎scripts/check-explicit-any.ts‎

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -79,7 +79,8 @@ function collect(): Baseline {
7979

8080
/** `biome-ignore` comments for either rule, which would hide a hit from Biome's count. */
8181
function suppressions(): string[] {
82-
const pattern = `biome-ignore(-all|-start)?[[:space:]]+(${Object.values(METRICS).join('|')})`
82+
// Anchored to a comment opener so the phrase inside a string literal (a test fixture) is not a hit.
83+
const pattern = `^[[:space:]]*(//|/\\*|\\{/\\*)[[:space:]]*biome-ignore(-all|-start)?[[:space:]]+(${Object.values(METRICS).join('|')})`
8384
const result = Bun.spawnSync(
8485
['git', 'grep', '-nE', '--untracked', pattern, '--', 'apps', 'packages', 'scripts'],
8586
{ cwd: ROOT, stdout: 'pipe', stderr: 'pipe' }

0 commit comments

Comments
 (0)