chore(deps): bump knip to 6.44.1

knip 6.44.1 tracks member use inside import().then() callbacks and follows
Tailwind @reference and bun spawn arguments, so check:unused-exports now sees:

- six exports only used in their own file (previously kept alive by an
  import('...').then(...) of their module): drop the export keyword and the
  matching keys in the storage-service test mock and the shared telemetry mock
- @sim/auth-ui's stylesheet uses @reference/@apply from tailwindcss without
  listing it: declare it as a peer dependency (consumers compile it)
- packages/db integration tests spawn bun scripts with an explicit cwd that
  knip cannot see: ignore those three specifiers with a reason

Author: waleedlatif1

apps/sim/app/workspace/[workspaceId]/w/[workflowId]/components/note-block/note-markdown-editor.tsx

@@ -5,15 +5,14 @@ import type { NoteContentEditorProps } from '@sim/workflow-renderer'
 import { RichMarkdownField } from '@/app/workspace/[workspaceId]/files/components/file-viewer/rich-markdown-editor/rich-markdown-field'
 import { useNoteImageUpload } from '@/app/workspace/[workspaceId]/w/[workflowId]/components/note-block/use-note-image-upload'
 
-export const NOTE_EDITOR_PROSE_CLASS_NAME = [
+const NOTE_EDITOR_PROSE_CLASS_NAME = [
   'min-h-full w-full text-current',
   /* The height the shared node chrome centres a checklist's checkbox against —
      the note's own line box, not the file editor's taller default. */
   '[--rich-markdown-line-height:1.25rem]',
   /* Mirrors NOTE_MARKDOWN_FLOW, the rhythm Streamdown paints in the read view.
      Tailwind's JIT only sees literal class strings, so this cannot be composed
-     from that constant — the note view exports it, and the parity test pins the
-     two together. Without it the editor falls back to the per-element margins
+     from that constant, so keep the two in step by hand. Without it the editor falls back to the per-element margins
      and every block after the first sits ~12px higher than when viewing. */
   '[&_.ProseMirror]:space-y-4 [&_.ProseMirror>*:first-child]:mt-0 [&_.ProseMirror>*:last-child]:mb-0',
   '[&_.ProseMirror]:min-h-full [&_.ProseMirror]:break-words [&_.ProseMirror]:pt-0.5 [&_.ProseMirror]:pb-2 [&_.ProseMirror]:outline-hidden',

apps/sim/lib/core/telemetry.ts

@@ -212,7 +212,7 @@ const BLOCK_TYPE_MAPPING: Record<
  * Convert a TraceSpan to an OpenTelemetry span
  * Creates a proper OTel span with all the metadata from the trace span
  */
-export function createOTelSpanFromTraceSpan(traceSpan: TraceSpan, parentSpan?: Span): Span | null {
+function createOTelSpanFromTraceSpan(traceSpan: TraceSpan, parentSpan?: Span): Span | null {
   try {
     const tracer = getTracer()
 
@@ -377,7 +377,7 @@ export function createOTelSpansForWorkflowExecution(params: {
 /**
  * Track platform events (workflow creation, knowledge base operations, etc.)
  */
-export function trackPlatformEvent(
+function trackPlatformEvent(
   eventName: string,
   attributes: Record<string, string | number | boolean | string[]>
 ): void {

apps/sim/lib/uploads/core/storage-service.test.ts

@@ -20,7 +20,6 @@ const {
   mockGetSignedUrl,
   mockHeadS3Object,
   mockPutObjectCommand,
-  mockS3Client,
   partBodies,
 } = vi.hoisted(() => ({
   mockInitiate: vi.fn(),
@@ -33,7 +32,6 @@ const {
   mockGetSignedUrl: vi.fn(),
   mockHeadS3Object: vi.fn(),
   mockPutObjectCommand: vi.fn().mockImplementation(class {}),
-  mockS3Client: {},
   partBodies: [] as Buffer[],
 }))
 
@@ -54,7 +52,6 @@ vi.mock('@/lib/uploads/providers/s3/client', () => ({
   abortS3MultipartUpload: mockAbort,
   uploadToS3: mockUploadToS3,
   deleteFromS3: mockDeleteFromS3,
-  getS3Client: () => mockS3Client,
   headS3Object: mockHeadS3Object,
 }))
 

apps/sim/lib/uploads/providers/google-cloud-storage/client.ts

@@ -52,7 +52,7 @@ interface GcsInlineCredentials {
  * Returns null when the variable is unset (Application Default Credentials).
  * @throws Error when the variable is set but not valid service-account JSON
  */
-export function parseGcsCredentials(): GcsInlineCredentials | null {
+function parseGcsCredentials(): GcsInlineCredentials | null {
   if (!env.GCS_CREDENTIALS_JSON) return null
 
   let parsed: unknown

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

@@ -48,7 +48,7 @@ export function resetS3ClientForTesting(): void {
   _s3Client = null
 }
 
-export function getS3Client(): S3Client {
+function getS3Client(): S3Client {
   if (_s3Client) return _s3Client
 
   const { region } = S3_CONFIG

bun.lock

@@ -151,6 +151,13 @@
         "script-migrations/*.ts",
         "**/*.integration.ts",
         "vitest.integration.setup.ts"
+      ],
+      // Integration tests spawn `bun <script>` with an explicit cwd (this package, or apps/sim
+      // for backfill-projects.ts); knip resolves the path from the test file's own directory.
+      "ignoreUnresolved": [
+        "scripts/push.ts",
+        "./scripts/purge-search-projection.ts",
+        "scripts/backfill-projects.ts"
       ]
     },
     // The contract audit reads this snapshot by filename.

knip.jsonc

@@ -200,7 +200,7 @@
     "gray-matter": "4.0.3",
     "husky": "9.1.7",
     "json-schema-to-typescript": "15.0.4",
-    "knip": "6.41.0",
+    "knip": "6.44.1",
     "lint-staged": "16.4.0",
     "opentype.js": "1.3.4",
     "pdf-lib": "1.17.1",

package.json

@@ -25,7 +25,8 @@
   },
   "peerDependencies": {
     "react": "^19",
-    "react-dom": "^19"
+    "react-dom": "^19",
+    "tailwindcss": "^4"
   },
   "devDependencies": {
     "@sim/tsconfig": "workspace:*",

packages/auth-ui/package.json

@@ -621,7 +621,7 @@ export function createLogger(module: string, config?: LoggerConfig): Logger {
   return new Logger(module, config)
 }
 
-export type { RequestAuth, RequestContext, SetRequestAuthOptions } from './request-context'
+export type { RequestAuth, RequestContext } from './request-context'
 export {
   getRequestContext,
   runWithRequestContext,