Skip to content

Commit d5e45da

Browse files
committed
improvement(naming): standardize file names to the folder and role conventions
Renames every check:file-names baseline entry (stutter and redundant-suffix) so the baseline is empty, plus the same rule's uncaught cases: grandparent-domain prefixes in application/ and orchestration/ folders, loose stores moved to stores/<feature>/store.ts, non-hook files dropped from the use-* prefix, and application context/errors outliers.
1 parent 0473322 commit d5e45da

1,368 files changed

Lines changed: 1989 additions & 2205 deletions

File tree

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

‎.agents/skills/add-block/SKILL.md‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -350,7 +350,7 @@ sourceIds.forEach((id) => delete params[id]) // subblock ids are deleted
350350
if (chosen !== undefined) params[group.canonicalId] = chosen
351351
```
352352

353-
So by the time `tools.config.params(inputs)` runs (`executor/handlers/generic/generic-handler.ts`),
353+
So by the time `tools.config.params(inputs)` runs (`executor/handlers/generic/handler.ts`),
354354
`params.uploadFile` and `params.fileRef` are **gone** and the value is under `params.file`. Reading a
355355
subblock id there yields `undefined` and silently sends no file.
356356

@@ -359,7 +359,7 @@ and the first non-empty advanced value in advanced mode, so a stale value in the
359359
never leak. `normalizeFileInput` then handles the JSON string that advanced-mode template resolution
360360
produces.
361361

362-
Note that `generic-handler` merges rather than replaces (`{ ...inputs, ...transformedParams }`), so
362+
Note that `generic/handler` merges rather than replaces (`{ ...inputs, ...transformedParams }`), so
363363
omitting a key from the returned object does not strip it from what the tool receives. Tools simply
364364
ignore params they do not declare.
365365

‎.agents/skills/add-integration/SKILL.md‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -657,8 +657,8 @@ File Downloads and Generated Files.
657657
| Helper | Location | Purpose |
658658
|--------|----------|---------|
659659
| `normalizeFileInput` | `@/blocks/utils` | Normalize file params in block config |
660-
| `processFilesToUserFiles` | `@/lib/uploads/utils/file-utils` | Convert raw inputs to UserFile[] |
661-
| `downloadFileFromStorage` | `@/lib/uploads/utils/file-utils.server` | Get file Buffer from UserFile |
660+
| `processFilesToUserFiles` | `@/lib/uploads/utils/file` | Convert raw inputs to UserFile[] |
661+
| `downloadFileFromStorage` | `@/lib/uploads/utils/file.server` | Get file Buffer from UserFile |
662662
| `FileToolProcessor` | `@/executor/utils/file-tool-processor` | Executor-side; stores declared file outputs (not called by tools) |
663663
| `isUserFile` | `@/lib/core/utils/user-file` | Type guard for UserFile objects |
664664
| `FileInputSchema` | `@/lib/uploads/utils/file-schemas` | Zod schema for file validation |

‎.agents/skills/migrate-application-operation/SKILL.md‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -47,7 +47,7 @@ Read these files completely before editing:
4747
- `apps/sim/lib/api/server/routes/internal-json-route.ts`
4848
- `apps/sim/lib/api/server/routes/v2-json-route.ts`
4949
- `apps/sim/lib/auth/internal-delegation.ts`
50-
- `apps/sim/lib/mothership/application/application-adapter.ts`
50+
- `apps/sim/lib/mothership/application/adapter.ts`
5151
- `apps/sim/lib/mothership/auth/application-delegation.ts`
5252

5353
Use the file domain only as a representative golden slice:

‎apps/sim/app/(auth)/login/login-form.tsx‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@ import { normalizeEmail } from '@sim/utils/string'
1515
import { useRouter, useSearchParams } from 'next/navigation'
1616
import { requestJson } from '@/lib/api/client/request'
1717
import { forgetPasswordContract } from '@/lib/api/contracts'
18-
import { client } from '@/lib/auth/auth-client'
18+
import { client } from '@/lib/auth/client'
1919
import { getEnv, isFalsy } from '@/lib/core/config/env'
2020
import { isSsoEnabled } from '@/lib/core/config/env-flags'
2121
import { validateCallbackUrl } from '@/lib/core/security/input-validation'

‎apps/sim/app/(auth)/oauth/consent/consent-view.test.tsx‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ import { QueryClient, QueryClientProvider } from '@tanstack/react-query'
77
import { createRoot, type Root } from 'react-dom/client'
88
import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest'
99

10-
vi.mock('@/lib/auth/auth-client', () => authClientMock)
10+
vi.mock('@/lib/auth/client', () => authClientMock)
1111

1212
import { OAuthConsentView } from '@/app/(auth)/oauth/consent/consent-view'
1313
import { oauthProviderKeys } from '@/hooks/queries/oauth-provider'

‎apps/sim/app/(auth)/signup/signup-form.tsx‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ import { useRouter, useSearchParams } from 'next/navigation'
77
import { usePostHog } from 'posthog-js/react'
88
import { trackFreebuffConversion } from '@/lib/analytics/freebuff'
99
import { trackGoogleEvent } from '@/lib/analytics/google'
10-
import { client, useSession } from '@/lib/auth/auth-client'
10+
import { client, useSession } from '@/lib/auth/client'
1111
import { useTrackingConsent } from '@/lib/consent/tracking-consent'
1212
import { getEnv, isFalsy } from '@/lib/core/config/env'
1313
import { isSsoEnabled } from '@/lib/core/config/env-flags'

‎apps/sim/app/(auth)/verify/use-verification.test.tsx‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ import { nextNavigationMock } from '@sim/testing/mocks/next-navigation.mock'
66
import { createRoot, type Root } from 'react-dom/client'
77
import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest'
88

9-
vi.mock('@/lib/auth/auth-client', () => authClientMock)
9+
vi.mock('@/lib/auth/client', () => authClientMock)
1010
vi.mock('next/navigation', () => nextNavigationMock)
1111

1212
import { useVerification } from '@/app/(auth)/verify/use-verification'

‎apps/sim/app/(auth)/verify/use-verification.ts‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ import { useEffect, useState } from 'react'
44
import { createLogger } from '@sim/logger'
55
import { normalizeEmail } from '@sim/utils/string'
66
import { useSearchParams } from 'next/navigation'
7-
import { client, useSession } from '@/lib/auth/auth-client'
7+
import { client, useSession } from '@/lib/auth/client'
88
import { SSO_REQUIRED_ERROR_CODE } from '@/lib/auth/constants'
99
import { validateCallbackUrl } from '@/lib/core/security/input-validation'
1010
import { DEFAULT_POST_AUTH_ROUTE, POST_AUTH_REDIRECT_STORAGE_KEY } from '@/app/(auth)/auth-redirect'

‎apps/sim/app/(interfaces)/chat/components/message/components/file-download.tsx‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ import { sleep } from '@sim/utils/helpers'
88
import { getDocumentIcon } from '@/components/icons/document-icons'
99
import { isSafeHttpUrl } from '@/lib/core/utils/urls'
1010
import { saveBlob } from '@/lib/uploads/client/download'
11-
import { tryInferContextFromKey } from '@/lib/uploads/utils/file-utils'
11+
import { tryInferContextFromKey } from '@/lib/uploads/utils/file'
1212
import type { ChatFile } from '@/app/(interfaces)/chat/components/message/message'
1313

1414
const logger = createLogger('ChatFileDownload')

‎apps/sim/app/(landing)/components/hero/components/hero-chat-loop/hero-tool-call-item.tsx‎

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,7 @@
11
import { Table } from '@sim/emcn/icons'
22
import { SlackIcon } from '@/components/icons'
33
import { ActivityStatus } from '@/components/ui/activity-status'
4-
import {
5-
getToolInProgressTitle,
6-
getToolStatusDisplayTitle,
7-
} from '@/lib/mothership/tools/tool-display'
4+
import { getToolInProgressTitle, getToolStatusDisplayTitle } from '@/lib/mothership/tools/display'
85
import type {
96
ToolActivityPresentation,
107
ToolCallItemProps,

0 commit comments

Comments
 (0)