docs(tests): restore the heavy-graph, global-mock and permanent-implementation rules and small doc pointers

Author: waleedlatif1

.claude/rules/sim-testing.md

@@ -125,6 +125,10 @@ Global (installed by `vitest.setup.ts` — knobs only, no `vi.mock`):
 | `@sim/platform-authz/workflow` | `workflowAuthzMockFns` (`workflow-authz.mock`) |
 | `@/lib/environment/utils` | `environmentUtilsMockFns` (`environment-utils.mock`) |
 
+Also global, with no knobs — never re-mock: the console/terminal/execution stores,
+`@/blocks/registry`, `@/tools/registry`, `@/tools/metadata`, `@/tools/metadata-outputs`,
+`@trigger.dev/core/v3`. `apps/sim/vitest.setup.ts` is the authoritative list.
+
 Helpers — use these instead of redefining them:
 
 | Need | Helper |
@@ -147,6 +151,11 @@ module scope or in `beforeAll` is undone before the first test. Integration mode
 fixtures (restore/unstub off). Node is the default environment — add
 `/** @vitest-environment jsdom */` only when the test needs the DOM.
 
+Those resets clear call history and undo spies, but not an implementation you install on a central
+mock's `vi.fn`: `xMockFns.mockFoo.mockReturnValue(...)` carries into later tests in the file. Prefer
+`*Once`; a test that installs a permanent one calls `mockFoo.mockReset()` (which restores the
+default) in `beforeEach`.
+
 ### Performance rules
 
 The suite's wall time is bound by the single Vite server thread that serves every module fetch and
@@ -155,8 +164,10 @@ The suite's wall time is bound by the single Vite server thread that serves ever
 1. `vi.hoisted()` + `vi.mock()` + static imports. Never `vi.resetModules()` + `vi.doMock()` +
    dynamic `import()`, except for a module that caches a singleton at module scope.
 2. Never `vi.importActual()`/`importOriginal` to build a partial mock — use the central mock.
-3. No real timers: `vi.useFakeTimers()`, `flushMicrotasks()`, or `flushMacrotask()`.
-4. Absolute imports only.
+3. Mock heavy graphs a test does not need and the setup does not already mock: `@/blocks`,
+   `@/triggers/registry`, `@/tools/generated/*`.
+4. No real timers: `vi.useFakeTimers()`, `flushMicrotasks()`, or `flushMacrotask()`.
+5. Absolute imports only.
 
 ### Running
 

.cursor/rules/sim-testing.mdc

@@ -123,6 +123,10 @@ Global (installed by `vitest.setup.ts` — knobs only, no `vi.mock`):
 | `@sim/platform-authz/workflow` | `workflowAuthzMockFns` (`workflow-authz.mock`) |
 | `@/lib/environment/utils` | `environmentUtilsMockFns` (`environment-utils.mock`) |
 
+Also global, with no knobs — never re-mock: the console/terminal/execution stores,
+`@/blocks/registry`, `@/tools/registry`, `@/tools/metadata`, `@/tools/metadata-outputs`,
+`@trigger.dev/core/v3`. `apps/sim/vitest.setup.ts` is the authoritative list.
+
 Helpers — use these instead of redefining them:
 
 | Need | Helper |
@@ -145,6 +149,11 @@ module scope or in `beforeAll` is undone before the first test. Integration mode
 fixtures (restore/unstub off). Node is the default environment — add
 `/** @vitest-environment jsdom */` only when the test needs the DOM.
 
+Those resets clear call history and undo spies, but not an implementation you install on a central
+mock's `vi.fn`: `xMockFns.mockFoo.mockReturnValue(...)` carries into later tests in the file. Prefer
+`*Once`; a test that installs a permanent one calls `mockFoo.mockReset()` (which restores the
+default) in `beforeEach`.
+
 ### Performance rules
 
 The suite's wall time is bound by the single Vite server thread that serves every module fetch and
@@ -153,8 +162,10 @@ The suite's wall time is bound by the single Vite server thread that serves ever
 1. `vi.hoisted()` + `vi.mock()` + static imports. Never `vi.resetModules()` + `vi.doMock()` +
    dynamic `import()`, except for a module that caches a singleton at module scope.
 2. Never `vi.importActual()`/`importOriginal` to build a partial mock — use the central mock.
-3. No real timers: `vi.useFakeTimers()`, `flushMicrotasks()`, or `flushMacrotask()`.
-4. Absolute imports only.
+3. Mock heavy graphs a test does not need and the setup does not already mock: `@/blocks`,
+   `@/triggers/registry`, `@/tools/generated/*`.
+4. No real timers: `vi.useFakeTimers()`, `flushMicrotasks()`, or `flushMacrotask()`.
+5. Absolute imports only.
 
 ### Running
 

CLAUDE.md

@@ -2,7 +2,7 @@
 
 You are a professional software engineer. All code must follow best practices: accurate, readable, clean, and efficient.
 
-This file (also `AGENTS.md`) holds the repo-wide rules. Area detail lives in `.claude/rules/*.md`: Claude loads each one by path, and any other agent reads the file a section points to before editing in that area. Skills live in `.agents/skills/`.
+This file (also `AGENTS.md`) holds the repo-wide rules. Area detail lives in `.claude/rules/*.md` (indexed in `apps/sim/AGENTS.md`): Claude loads each one by path, and any other agent reads the file a section points to before editing in that area. Skills live in `.agents/skills/`.
 
 ## Global Standards
 
@@ -24,7 +24,7 @@ This file (also `AGENTS.md`) holds the repo-wide rules. Area detail lives in `.c
   - `compareStrings(left, right)` from `@sim/utils/string` — code-unit ordering for hashes, fingerprints, and cross-process comparisons; never `localeCompare` there
   - `backoffWithJitter(attempt, retryAfterMs, options?)` / `parseRetryAfter(header)` from `@sim/utils/retry` — never reimplement exponential backoff inline
 - **Deployment flags in the browser**: client code inside a workspace, organization, or standalone settings surface reads `hosted`, `billingEnabled`, `chatEnabled`, and the enterprise feature set through `useDeploymentShape()` (components) or `getDeploymentShape()` (block conditions, stores, helpers) from `@/lib/core/config/deployment-shape`, never `isHosted`/`isBillingEnabled`/... from `env-flags`. Those constants freeze at module init from the root layout's `NEXT_PUBLIC_*` transport, which Next's bare 404 shell and `global-error` never emit, so a recovered tab would render Sim Cloud as self-hosted; the reader is seeded from the server-resolved workspace host context, organization layout, or standalone settings layout instead. Server code keeps reading `env-flags`.
-- **Type-checking**: `bun run type-check` (per workspace) or `bunx turbo run type-check` (all). Never remove the `@typescript/native` alias from the root `devDependencies`. Nothing imports it; it exists so a bare `tsc` resolves to the native TypeScript 7 compiler. `apps/sim` needs `@typescript/typescript6`, whose `@typescript/old` dependency (an alias of `typescript@6`) ships its own `tsc` bin, and bin winners are picked by lexical sort, so without the alias `tsc` silently becomes the ~10x slower JavaScript compiler. `bun run check:native-typecheck` enforces this ([microsoft/typescript-go#4567](https://github.com/microsoft/typescript-go/issues/4567)).
+- **Type-checking**: `bun run type-check` (per workspace) or `bunx turbo run type-check` (all). Never remove the `@typescript/native` alias from the root `devDependencies`. Nothing imports it; it exists so a bare `tsc` resolves to the native TypeScript 7 compiler. `apps/sim` needs `@typescript/typescript6`, whose `@typescript/old` dependency (an alias of `typescript@6`) ships its own `tsc` bin, and bin winners are picked by lexical sort, so without the alias `tsc` silently becomes the ~10x slower JavaScript compiler. `bun run check:native-typecheck` enforces this, and also fails when a newly added dependency that sorts ahead of `@typescript/native` ships a `tsc` bin ([microsoft/typescript-go#4567](https://github.com/microsoft/typescript-go/issues/4567)).
 - **Checks**: `bun run lint` autofixes formatting; `bun run check:audits` runs every `check:*` audit CI enforces.
 
 ## Architecture

apps/sim/AGENTS.md

@@ -10,6 +10,7 @@ Applies to `apps/sim/**` on top of the root [AGENTS.md](/AGENTS.md), which holds
 - URL state, list and menu order, caching: `sim-url-state.md`, `sim-list-ordering.md`, `sim-caching.md`
 - Tools, blocks, triggers: `sim-integrations.md`; isolated-vm sandbox worker: `sim-sandbox.md`
 - Tests: `sim-testing.md` and the `test-audit` skill
+- Landing pages: `app/(landing)/CLAUDE.md`, `landing-seo-geo.md`, `constitution.md` (product language)
 
 <!-- BEGIN:nextjs-agent-rules -->