Skip to content

Commit fa5c3aa

Browse files
TheodoreSpeaksTheodoreSpeaks
committed
Merge remote-tracking branch 'origin/staging' into feat/auto-redaction
2 parents adb8f71 + b5674d9 commit fa5c3aa

785 files changed

Lines changed: 5290 additions & 4636 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.

‎.claude/rules/global.md‎

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@ Use TSDoc for documentation. No `====` separators. No non-TSDoc comments.
1010
Never update global styles. Keep all styling local to components.
1111

1212
## ID Generation
13-
Never use `crypto.randomUUID()`, `nanoid`, or the `uuid` package directly. Use the utilities from `@/lib/core/utils/uuid`:
13+
Never use `crypto.randomUUID()`, `nanoid`, or the `uuid` package directly. Use the utilities from `@sim/utils/id`:
1414

1515
- `generateId()` — UUID v4, use by default
1616
- `generateShortId(size?)` — short URL-safe ID (default 21 chars), for compact identifiers
@@ -24,14 +24,14 @@ import { v4 as uuidv4 } from 'uuid'
2424
const id = crypto.randomUUID()
2525

2626
// ✓ Good
27-
import { generateId, generateShortId } from '@/lib/core/utils/uuid'
27+
import { generateId, generateShortId } from '@sim/utils/id'
2828
const uuid = generateId()
2929
const shortId = generateShortId()
3030
const tiny = generateShortId(8)
3131
```
3232

3333
## Common Utilities
34-
Use shared helpers from `@/lib/core/utils/helpers` instead of writing inline implementations:
34+
Use shared helpers from `@sim/utils` instead of writing inline implementations:
3535

3636
- `sleep(ms)` — async delay. Never write `new Promise(resolve => setTimeout(resolve, ms))`
3737
- `toError(value)` — normalize unknown caught values to `Error`. Never write `e instanceof Error ? e : new Error(String(e))`
@@ -44,7 +44,8 @@ const msg = error instanceof Error ? error.message : String(error)
4444
const err = error instanceof Error ? error : new Error(String(error))
4545

4646
// ✓ Good
47-
import { sleep, toError } from '@/lib/core/utils/helpers'
47+
import { sleep } from '@sim/utils/helpers'
48+
import { toError } from '@sim/utils/errors'
4849
await sleep(1000)
4950
const msg = toError(error).message
5051
const err = toError(error)

‎.claude/rules/sim-testing.md‎

Lines changed: 52 additions & 31 deletions
Original file line numberDiff line numberDiff line change
@@ -102,10 +102,6 @@ vi.mock('@/lib/workspaces/utils', () => ({
102102
}))
103103
```
104104

105-
### NEVER use `mockAuth()`, `mockConsoleLogger()`, or `setupCommonApiMocks()` from `@sim/testing`
106-
107-
These helpers internally use `vi.doMock()` which is slow. Use direct `vi.hoisted()` + `vi.mock()` instead.
108-
109105
### Mock heavy transitive dependencies
110106

111107
If a module under test imports `@/blocks` (200+ files), `@/tools/registry`, or other heavy modules, mock them:
@@ -135,38 +131,61 @@ await new Promise(r => setTimeout(r, 1))
135131
vi.useFakeTimers()
136132
```
137133

138-
## Mock Pattern Reference
134+
## Centralized Mocks (prefer over local declarations)
135+
136+
`@sim/testing` exports ready-to-use mock modules for common dependencies. Import and pass directly to `vi.mock()` — no `vi.hoisted()` boilerplate needed. Each paired `*MockFns` object exposes the underlying `vi.fn()`s for per-test overrides.
137+
138+
| Module mocked | Import | Factory form |
139+
|---|---|---|
140+
| `@/app/api/auth/oauth/utils` | `authOAuthUtilsMock`, `authOAuthUtilsMockFns` | `vi.mock('@/app/api/auth/oauth/utils', () => authOAuthUtilsMock)` |
141+
| `@/app/api/knowledge/utils` | `knowledgeApiUtilsMock`, `knowledgeApiUtilsMockFns` | `vi.mock('@/app/api/knowledge/utils', () => knowledgeApiUtilsMock)` |
142+
| `@/app/api/workflows/utils` | `workflowsApiUtilsMock`, `workflowsApiUtilsMockFns` | `vi.mock('@/app/api/workflows/utils', () => workflowsApiUtilsMock)` |
143+
| `@/lib/audit/log` | `auditMock`, `auditMockFns` | `vi.mock('@/lib/audit/log', () => auditMock)` |
144+
| `@/lib/auth` | `authMock`, `authMockFns` | `vi.mock('@/lib/auth', () => authMock)` |
145+
| `@/lib/auth/hybrid` | `hybridAuthMock`, `hybridAuthMockFns` | `vi.mock('@/lib/auth/hybrid', () => hybridAuthMock)` |
146+
| `@/lib/copilot/request/http` | `copilotHttpMock`, `copilotHttpMockFns` | `vi.mock('@/lib/copilot/request/http', () => copilotHttpMock)` |
147+
| `@/lib/core/config/env` | `envMock`, `createEnvMock(overrides)` | `vi.mock('@/lib/core/config/env', () => envMock)` |
148+
| `@/lib/core/config/feature-flags` | `featureFlagsMock` | `vi.mock('@/lib/core/config/feature-flags', () => featureFlagsMock)` |
149+
| `@/lib/core/config/redis` | `redisConfigMock`, `redisConfigMockFns` | `vi.mock('@/lib/core/config/redis', () => redisConfigMock)` |
150+
| `@/lib/core/security/encryption` | `encryptionMock`, `encryptionMockFns` | `vi.mock('@/lib/core/security/encryption', () => encryptionMock)` |
151+
| `@/lib/core/security/input-validation.server` | `inputValidationMock`, `inputValidationMockFns` | `vi.mock('@/lib/core/security/input-validation.server', () => inputValidationMock)` |
152+
| `@/lib/core/utils/request` | `requestUtilsMock`, `requestUtilsMockFns` | `vi.mock('@/lib/core/utils/request', () => requestUtilsMock)` |
153+
| `@/lib/core/utils/urls` | `urlsMock`, `urlsMockFns` | `vi.mock('@/lib/core/utils/urls', () => urlsMock)` |
154+
| `@/lib/execution/preprocessing` | `executionPreprocessingMock`, `executionPreprocessingMockFns` | `vi.mock('@/lib/execution/preprocessing', () => executionPreprocessingMock)` |
155+
| `@/lib/logs/execution/logging-session` | `loggingSessionMock`, `loggingSessionMockFns`, `LoggingSessionMock` | `vi.mock('@/lib/logs/execution/logging-session', () => loggingSessionMock)` |
156+
| `@/lib/workflows/orchestration` | `workflowsOrchestrationMock`, `workflowsOrchestrationMockFns` | `vi.mock('@/lib/workflows/orchestration', () => workflowsOrchestrationMock)` |
157+
| `@/lib/workflows/persistence/utils` | `workflowsPersistenceUtilsMock`, `workflowsPersistenceUtilsMockFns` | `vi.mock('@/lib/workflows/persistence/utils', () => workflowsPersistenceUtilsMock)` |
158+
| `@/lib/workflows/utils` | `workflowsUtilsMock`, `workflowsUtilsMockFns` | `vi.mock('@/lib/workflows/utils', () => workflowsUtilsMock)` |
159+
| `@/lib/workspaces/permissions/utils` | `permissionsMock`, `permissionsMockFns` | `vi.mock('@/lib/workspaces/permissions/utils', () => permissionsMock)` |
160+
| `@sim/db/schema` | `schemaMock` | `vi.mock('@sim/db/schema', () => schemaMock)` |
139161

140162
### Auth mocking (API routes)
141163

142164
```typescript
143-
const { mockGetSession } = vi.hoisted(() => ({
144-
mockGetSession: vi.fn(),
145-
}))
165+
import { authMock, authMockFns } from '@sim/testing'
166+
import { beforeEach, describe, expect, it, vi } from 'vitest'
146167

147-
vi.mock('@/lib/auth', () => ({
148-
auth: { api: { getSession: vi.fn() } },
149-
getSession: mockGetSession,
150-
}))
168+
vi.mock('@/lib/auth', () => authMock)
151169

152-
// In tests:
153-
mockGetSession.mockResolvedValue({ user: { id: 'user-1', email: 'test@example.com' } })
154-
mockGetSession.mockResolvedValue(null) // unauthenticated
170+
import { GET } from '@/app/api/my-route/route'
171+
172+
beforeEach(() => {
173+
vi.clearAllMocks()
174+
authMockFns.mockGetSession.mockResolvedValue({ user: { id: 'user-1' } })
175+
})
155176
```
156177

178+
Only define a local `vi.mock('@/lib/auth', ...)` if the module under test consumes exports outside the centralized shape (e.g., `auth.api.verifyOneTimeToken`, `auth.api.resetPassword`).
179+
157180
### Hybrid auth mocking
158181

159182
```typescript
160-
const { mockCheckSessionOrInternalAuth } = vi.hoisted(() => ({
161-
mockCheckSessionOrInternalAuth: vi.fn(),
162-
}))
183+
import { hybridAuthMock, hybridAuthMockFns } from '@sim/testing'
163184

164-
vi.mock('@/lib/auth/hybrid', () => ({
165-
checkSessionOrInternalAuth: mockCheckSessionOrInternalAuth,
166-
}))
185+
vi.mock('@/lib/auth/hybrid', () => hybridAuthMock)
167186

168187
// In tests:
169-
mockCheckSessionOrInternalAuth.mockResolvedValue({
188+
hybridAuthMockFns.mockCheckSessionOrInternalAuth.mockResolvedValue({
170189
success: true, userId: 'user-1', authType: 'session',
171190
})
172191
```
@@ -197,21 +216,23 @@ Always prefer over local test data.
197216

198217
| Category | Utilities |
199218
|----------|-----------|
200-
| **Mocks** | `loggerMock`, `databaseMock`, `drizzleOrmMock`, `setupGlobalFetchMock()` |
219+
| **Module mocks** | See "Centralized Mocks" table above |
220+
| **Logger helpers** | `loggerMock`, `createMockLogger()`, `getLoggerCalls()`, `clearLoggerMocks()` |
221+
| **Database helpers** | `databaseMock`, `drizzleOrmMock`, `createMockDb()`, `createMockSql()`, `createMockSqlOperators()` |
222+
| **Fetch helpers** | `setupGlobalFetchMock()`, `createMockFetch()`, `createMockResponse()`, `mockFetchError()` |
201223
| **Factories** | `createSession()`, `createWorkflowRecord()`, `createBlock()`, `createExecutionContext()` |
202224
| **Builders** | `WorkflowBuilder`, `ExecutionContextBuilder` |
203225
| **Assertions** | `expectWorkflowAccessGranted()`, `expectBlockExecuted()` |
204-
| **Requests** | `createMockRequest()`, `createEnvMock()` |
226+
| **Requests** | `createMockRequest()`, `createMockFormDataRequest()` |
205227

206228
## Rules Summary
207229

208230
1. `@vitest-environment node` unless DOM is required
209-
2. `vi.hoisted()` + `vi.mock()` + static imports — never `vi.resetModules()` + `vi.doMock()` + dynamic imports
210-
3. `vi.mock()` calls before importing mocked modules
211-
4. `@sim/testing` utilities over local mocks
231+
2. Prefer centralized mocks from `@sim/testing` (see table above) over local `vi.hoisted()` + `vi.mock()` boilerplate
232+
3. `vi.hoisted()` + `vi.mock()` + static imports — never `vi.resetModules()` + `vi.doMock()` + dynamic imports
233+
4. `vi.mock()` calls before importing mocked modules
212234
5. `beforeEach(() => vi.clearAllMocks())` to reset state — no redundant `afterEach`
213235
6. No `vi.importActual()` — mock everything explicitly
214-
7. No `mockAuth()`, `mockConsoleLogger()`, `setupCommonApiMocks()` — use direct mocks
215-
8. Mock heavy deps (`@/blocks`, `@/tools/registry`, `@/triggers`) in tests that don't need them
216-
9. Use absolute imports in test files
217-
10. Avoid real timers — use 1ms delays or `vi.useFakeTimers()`
236+
7. Mock heavy deps (`@/blocks`, `@/tools/registry`, `@/triggers`) in tests that don't need them
237+
8. Use absolute imports in test files
238+
9. Avoid real timers — use 1ms delays or `vi.useFakeTimers()`

‎.cursor/rules/global.mdc‎

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ Use TSDoc for documentation. No `====` separators. No non-TSDoc comments.
1717
Never update global styles. Keep all styling local to components.
1818

1919
## ID Generation
20-
Never use `crypto.randomUUID()`, `nanoid`, or the `uuid` package directly. Use the utilities from `@/lib/core/utils/uuid`:
20+
Never use `crypto.randomUUID()`, `nanoid`, or the `uuid` package directly. Use the utilities from `@sim/utils/id`:
2121

2222
- `generateId()` — UUID v4, use by default
2323
- `generateShortId(size?)` — short URL-safe ID (default 21 chars), for compact identifiers
@@ -31,14 +31,14 @@ import { v4 as uuidv4 } from 'uuid'
3131
const id = crypto.randomUUID()
3232

3333
// ✓ Good
34-
import { generateId, generateShortId } from '@/lib/core/utils/uuid'
34+
import { generateId, generateShortId } from '@sim/utils/id'
3535
const uuid = generateId()
3636
const shortId = generateShortId()
3737
const tiny = generateShortId(8)
3838
```
3939

4040
## Common Utilities
41-
Use shared helpers from `@/lib/core/utils/helpers` instead of writing inline implementations:
41+
Use shared helpers from `@sim/utils` instead of writing inline implementations:
4242

4343
- `sleep(ms)` — async delay. Never write `new Promise(resolve => setTimeout(resolve, ms))`
4444
- `toError(value)` — normalize unknown caught values to `Error`. Never write `e instanceof Error ? e : new Error(String(e))`
@@ -51,7 +51,8 @@ const msg = error instanceof Error ? error.message : String(error)
5151
const err = error instanceof Error ? error : new Error(String(error))
5252

5353
// ✓ Good
54-
import { sleep, toError } from '@/lib/core/utils/helpers'
54+
import { sleep } from '@sim/utils/helpers'
55+
import { toError } from '@sim/utils/errors'
5556
await sleep(1000)
5657
const msg = toError(error).message
5758
const err = toError(error)

‎.cursor/rules/sim-testing.mdc‎

Lines changed: 53 additions & 31 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,7 @@ description: Testing patterns with Vitest and @sim/testing
33
globs: ["apps/sim/**/*.test.ts", "apps/sim/**/*.test.tsx"]
44
---
55

6+
67
# Testing Patterns
78

89
Use Vitest. Test files: `feature.ts` → `feature.test.ts`
@@ -101,10 +102,6 @@ vi.mock('@/lib/workspaces/utils', () => ({
101102
}))
102103
```
103104

104-
### NEVER use `mockAuth()`, `mockConsoleLogger()`, or `setupCommonApiMocks()` from `@sim/testing`
105-
106-
These helpers internally use `vi.doMock()` which is slow. Use direct `vi.hoisted()` + `vi.mock()` instead.
107-
108105
### Mock heavy transitive dependencies
109106

110107
If a module under test imports `@/blocks` (200+ files), `@/tools/registry`, or other heavy modules, mock them:
@@ -134,38 +131,61 @@ await new Promise(r => setTimeout(r, 1))
134131
vi.useFakeTimers()
135132
```
136133

137-
## Mock Pattern Reference
134+
## Centralized Mocks (prefer over local declarations)
135+
136+
`@sim/testing` exports ready-to-use mock modules for common dependencies. Import and pass directly to `vi.mock()` — no `vi.hoisted()` boilerplate needed. Each paired `*MockFns` object exposes the underlying `vi.fn()`s for per-test overrides.
137+
138+
| Module mocked | Import | Factory form |
139+
|---|---|---|
140+
| `@/app/api/auth/oauth/utils` | `authOAuthUtilsMock`, `authOAuthUtilsMockFns` | `vi.mock('@/app/api/auth/oauth/utils', () => authOAuthUtilsMock)` |
141+
| `@/app/api/knowledge/utils` | `knowledgeApiUtilsMock`, `knowledgeApiUtilsMockFns` | `vi.mock('@/app/api/knowledge/utils', () => knowledgeApiUtilsMock)` |
142+
| `@/app/api/workflows/utils` | `workflowsApiUtilsMock`, `workflowsApiUtilsMockFns` | `vi.mock('@/app/api/workflows/utils', () => workflowsApiUtilsMock)` |
143+
| `@/lib/audit/log` | `auditMock`, `auditMockFns` | `vi.mock('@/lib/audit/log', () => auditMock)` |
144+
| `@/lib/auth` | `authMock`, `authMockFns` | `vi.mock('@/lib/auth', () => authMock)` |
145+
| `@/lib/auth/hybrid` | `hybridAuthMock`, `hybridAuthMockFns` | `vi.mock('@/lib/auth/hybrid', () => hybridAuthMock)` |
146+
| `@/lib/copilot/request/http` | `copilotHttpMock`, `copilotHttpMockFns` | `vi.mock('@/lib/copilot/request/http', () => copilotHttpMock)` |
147+
| `@/lib/core/config/env` | `envMock`, `createEnvMock(overrides)` | `vi.mock('@/lib/core/config/env', () => envMock)` |
148+
| `@/lib/core/config/feature-flags` | `featureFlagsMock` | `vi.mock('@/lib/core/config/feature-flags', () => featureFlagsMock)` |
149+
| `@/lib/core/config/redis` | `redisConfigMock`, `redisConfigMockFns` | `vi.mock('@/lib/core/config/redis', () => redisConfigMock)` |
150+
| `@/lib/core/security/encryption` | `encryptionMock`, `encryptionMockFns` | `vi.mock('@/lib/core/security/encryption', () => encryptionMock)` |
151+
| `@/lib/core/security/input-validation.server` | `inputValidationMock`, `inputValidationMockFns` | `vi.mock('@/lib/core/security/input-validation.server', () => inputValidationMock)` |
152+
| `@/lib/core/utils/request` | `requestUtilsMock`, `requestUtilsMockFns` | `vi.mock('@/lib/core/utils/request', () => requestUtilsMock)` |
153+
| `@/lib/core/utils/urls` | `urlsMock`, `urlsMockFns` | `vi.mock('@/lib/core/utils/urls', () => urlsMock)` |
154+
| `@/lib/execution/preprocessing` | `executionPreprocessingMock`, `executionPreprocessingMockFns` | `vi.mock('@/lib/execution/preprocessing', () => executionPreprocessingMock)` |
155+
| `@/lib/logs/execution/logging-session` | `loggingSessionMock`, `loggingSessionMockFns`, `LoggingSessionMock` | `vi.mock('@/lib/logs/execution/logging-session', () => loggingSessionMock)` |
156+
| `@/lib/workflows/orchestration` | `workflowsOrchestrationMock`, `workflowsOrchestrationMockFns` | `vi.mock('@/lib/workflows/orchestration', () => workflowsOrchestrationMock)` |
157+
| `@/lib/workflows/persistence/utils` | `workflowsPersistenceUtilsMock`, `workflowsPersistenceUtilsMockFns` | `vi.mock('@/lib/workflows/persistence/utils', () => workflowsPersistenceUtilsMock)` |
158+
| `@/lib/workflows/utils` | `workflowsUtilsMock`, `workflowsUtilsMockFns` | `vi.mock('@/lib/workflows/utils', () => workflowsUtilsMock)` |
159+
| `@/lib/workspaces/permissions/utils` | `permissionsMock`, `permissionsMockFns` | `vi.mock('@/lib/workspaces/permissions/utils', () => permissionsMock)` |
160+
| `@sim/db/schema` | `schemaMock` | `vi.mock('@sim/db/schema', () => schemaMock)` |
138161

139162
### Auth mocking (API routes)
140163

141164
```typescript
142-
const { mockGetSession } = vi.hoisted(() => ({
143-
mockGetSession: vi.fn(),
144-
}))
165+
import { authMock, authMockFns } from '@sim/testing'
166+
import { beforeEach, describe, expect, it, vi } from 'vitest'
145167

146-
vi.mock('@/lib/auth', () => ({
147-
auth: { api: { getSession: vi.fn() } },
148-
getSession: mockGetSession,
149-
}))
168+
vi.mock('@/lib/auth', () => authMock)
150169

151-
// In tests:
152-
mockGetSession.mockResolvedValue({ user: { id: 'user-1', email: 'test@example.com' } })
153-
mockGetSession.mockResolvedValue(null) // unauthenticated
170+
import { GET } from '@/app/api/my-route/route'
171+
172+
beforeEach(() => {
173+
vi.clearAllMocks()
174+
authMockFns.mockGetSession.mockResolvedValue({ user: { id: 'user-1' } })
175+
})
154176
```
155177

178+
Only define a local `vi.mock('@/lib/auth', ...)` if the module under test consumes exports outside the centralized shape (e.g., `auth.api.verifyOneTimeToken`, `auth.api.resetPassword`).
179+
156180
### Hybrid auth mocking
157181

158182
```typescript
159-
const { mockCheckSessionOrInternalAuth } = vi.hoisted(() => ({
160-
mockCheckSessionOrInternalAuth: vi.fn(),
161-
}))
183+
import { hybridAuthMock, hybridAuthMockFns } from '@sim/testing'
162184

163-
vi.mock('@/lib/auth/hybrid', () => ({
164-
checkSessionOrInternalAuth: mockCheckSessionOrInternalAuth,
165-
}))
185+
vi.mock('@/lib/auth/hybrid', () => hybridAuthMock)
166186

167187
// In tests:
168-
mockCheckSessionOrInternalAuth.mockResolvedValue({
188+
hybridAuthMockFns.mockCheckSessionOrInternalAuth.mockResolvedValue({
169189
success: true, userId: 'user-1', authType: 'session',
170190
})
171191
```
@@ -196,21 +216,23 @@ Always prefer over local test data.
196216

197217
| Category | Utilities |
198218
|----------|-----------|
199-
| **Mocks** | `loggerMock`, `databaseMock`, `drizzleOrmMock`, `setupGlobalFetchMock()` |
219+
| **Module mocks** | See "Centralized Mocks" table above |
220+
| **Logger helpers** | `loggerMock`, `createMockLogger()`, `getLoggerCalls()`, `clearLoggerMocks()` |
221+
| **Database helpers** | `databaseMock`, `drizzleOrmMock`, `createMockDb()`, `createMockSql()`, `createMockSqlOperators()` |
222+
| **Fetch helpers** | `setupGlobalFetchMock()`, `createMockFetch()`, `createMockResponse()`, `mockFetchError()` |
200223
| **Factories** | `createSession()`, `createWorkflowRecord()`, `createBlock()`, `createExecutionContext()` |
201224
| **Builders** | `WorkflowBuilder`, `ExecutionContextBuilder` |
202225
| **Assertions** | `expectWorkflowAccessGranted()`, `expectBlockExecuted()` |
203-
| **Requests** | `createMockRequest()`, `createEnvMock()` |
226+
| **Requests** | `createMockRequest()`, `createMockFormDataRequest()` |
204227

205228
## Rules Summary
206229

207230
1. `@vitest-environment node` unless DOM is required
208-
2. `vi.hoisted()` + `vi.mock()` + static imports — never `vi.resetModules()` + `vi.doMock()` + dynamic imports
209-
3. `vi.mock()` calls before importing mocked modules
210-
4. `@sim/testing` utilities over local mocks
231+
2. Prefer centralized mocks from `@sim/testing` (see table above) over local `vi.hoisted()` + `vi.mock()` boilerplate
232+
3. `vi.hoisted()` + `vi.mock()` + static imports — never `vi.resetModules()` + `vi.doMock()` + dynamic imports
233+
4. `vi.mock()` calls before importing mocked modules
211234
5. `beforeEach(() => vi.clearAllMocks())` to reset state — no redundant `afterEach`
212235
6. No `vi.importActual()` — mock everything explicitly
213-
7. No `mockAuth()`, `mockConsoleLogger()`, `setupCommonApiMocks()` — use direct mocks
214-
8. Mock heavy deps (`@/blocks`, `@/tools/registry`, `@/triggers`) in tests that don't need them
215-
9. Use absolute imports in test files
216-
10. Avoid real timers — use 1ms delays or `vi.useFakeTimers()`
236+
7. Mock heavy deps (`@/blocks`, `@/tools/registry`, `@/triggers`) in tests that don't need them
237+
8. Use absolute imports in test files
238+
9. Avoid real timers — use 1ms delays or `vi.useFakeTimers()`

‎AGENTS.md‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ You are a professional software engineer. All code must follow best practices: a
77
- **Logging**: Import `createLogger` from `@sim/logger`. Use `logger.info`, `logger.warn`, `logger.error` instead of `console.log`
88
- **Comments**: Use TSDoc for documentation. No `====` separators. No non-TSDoc comments
99
- **Styling**: Never update global styles. Keep all styling local to components
10-
- **ID Generation**: Never use `crypto.randomUUID()`, `nanoid`, or `uuid` package. Use `generateId()` (UUID v4) or `generateShortId()` (compact) from `@/lib/core/utils/uuid`
10+
- **ID Generation**: Never use `crypto.randomUUID()`, `nanoid`, or `uuid` package. Use `generateId()` (UUID v4) or `generateShortId()` (compact) from `@sim/utils/id`
1111
- **Package Manager**: Use `bun` and `bunx`, not `npm` and `npx`
1212

1313
## Architecture

0 commit comments

Comments
 (0)