Merge origin/staging into improvement/resource-views-final

Conflict resolutions of substance:

- files serve + public-share content routes: kept the branch's byte-range
  media streaming and added staging's `ServeOptions` / `preview=1` HEIC
  derivative path. The two are disjoint (`isMediaContentType` is audio/video
  only), so both branches now coexist ahead of the buffered compile path.
- `hooks/use-file-content-source.tsx` is deleted on this branch; staging's
  additions to it (the `preview` URL flag and the `ImageDimensionsSource`
  capability) moved to `resources/file-source.ts`, and the workspace adapter
  now resolves from the mounted source rather than a second React context.
- `UnsupportedPreview`: the branch's download-carrying variant replaces the
  moved copy in `preview-shared`, so the image preview's new decode-failure
  fallback lands on the version that works on chrome-less hosts.
- logs: staging's workflow deep-link (#6275) ported into `LogView` through a
  new `resources/log-source.ts`, so the unit reads the href off its source
  instead of hand-building a workspace path.
- `public-file-view`: kept the branch's `Navbar` + `h-dvh` scroll port, which
  supersedes staging's header layout fix on the header it replaced.
- `test-build.yml`: took staging's single `check:audits` step; `run-audits.ts`
  derives the list from package.json, so `check:resources` is picked up.
- API route baseline raised to the merged tree's real count (1008).

Author: waleedlatif1

.agents/skills/add-column-type/SKILL.md

@@ -15,7 +15,7 @@ This was not always true: adding `currency` originally took ~40 edits across 32
 Do **not** hunt for places to edit. Add your type to the `ColumnType` union first and let `tsc` produce the list:
 
 ```bash
-cd apps/sim && bunx tsc --noEmit -p tsconfig.json
+cd apps/sim && bun run type-check
 ```
 
 You will get two errors, naming `column-types/registry.ts` and `column-types/registry.server.ts`. Register in both.
@@ -153,7 +153,7 @@ Registering the *type* is compiler-enforced. Registering its *metadata* is not,
 
 ## Final Validation (Required)
 
-1. **`cd apps/sim && bunx tsc --noEmit -p tsconfig.json`** — must be clean. If any file *outside* `column-types/` errors, that file has a hardcoded type list; fix it to read the registry.
+1. **`cd apps/sim && bun run type-check`** — must be clean. If any file *outside* `column-types/` errors, that file has a hardcoded type list; fix it to read the registry.
 2. **Grep for leaks** — `grep -rnE "(===|!==) '{id}'|case '{id}':" apps/sim --include='*.ts' --include='*.tsx' | grep -v column-types/`. (All three forms: a plain `!==` and a `case` are how half of `currency`'s real branches are written.) Hits are expected; judge each. A hit is fine when it mounts a specific React component or encodes a genuinely one-off behavior (`json`'s mono textarea, `date`'s timezone-aware parsing). A hit is a **leak** when it restates something the registry could answer — an icon, a label, a colour, an operator set, a cast, a coercion. Leaks get a registry field, not a new branch.
 3. **Run the suite** — `bunx vitest run lib/table 'app/workspace/[workspaceId]/tables' lib/api app/api/table app/api/v1 lib/copilot/tools/server/table`. Existing tests must pass **unchanged**; needing to edit one means you changed behavior for the other types.
 4. **`bun run lint:check`, `bun run check:api-validation`, `bun run check:client-boundary`** from the repo root.

.agents/skills/add-enrichment/SKILL.md

@@ -128,7 +128,7 @@ export const ENRICHMENT_REGISTRY: EnrichmentRegistry = {
 
 ## Step 5: Verify
 
-1. `bunx tsc --noEmit` (from `apps/sim`, `NODE_OPTIONS=--max-old-space-size=8192`) and `bunx biome check` on the changed files.
+1. `bun run type-check` (from `apps/sim`) and `bunx biome check` on the changed files.
 2. In a table → **+ New column → Enrichments** → pick the new enrichment, map its inputs to columns, name the output column(s), Save. Confirm it appears in the catalog with its icon/description.
 3. With hosted keys (or a workspace BYOK key) configured for each provider's service, run a row and confirm the cell fills; the dev-server log shows `Enrichment hit { provider }`. A row whose providers all miss completes blank; a row where every provider errored shows an error cell.
 

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

@@ -1,6 +1,6 @@
 ---
 name: add-integration
-description: Add a complete Sim integration from API docs, covering tools, block, icon, optional triggers, registrations, and integration conventions. Use when introducing a new service under `apps/sim/tools`, `apps/sim/blocks`, and `apps/sim/triggers`.
+description: Add a complete Sim integration from API docs, covering tools, block, icon, optional triggers, registrations, resolved-secret/model-input safety, and integration conventions. Use when introducing a new service under `apps/sim/tools`, `apps/sim/blocks`, and `apps/sim/triggers`.
 argument-hint: <service-name> [api-docs-url]
 ---
 
@@ -122,6 +122,64 @@ export const {service}{Action}Tool: ToolConfig<Params, Response> = {
 - When using `type: 'json'` and you know the object shape, define `properties` with the inner fields so downstream consumers know the structure. Only use bare `type: 'json'` when the shape is truly dynamic
 - If you do not know the response JSON shape from docs or verified examples, you MUST tell the user and stop. Never guess outputs or response mappings.
 
+### Resolved Secrets at Model and Persistence Boundaries
+
+Classify every request field before implementing the tool:
+
+This is opt-in, not a blanket integration migration. Add a model-input declaration only when the
+service's official documentation or an unambiguous local execution path proves that the exact
+field is consumed by an AI model. If that cannot be established, preserve existing tool behavior
+and leave the field unannotated.
+
+- **Ordinary provider/API input:** leave it unchanged. Do not add blanket result sanitization.
+- **Text or structured content consumed by an AI model:** declare `request.modelInput` with
+  `mode: 'project'` and select only the exact model-visible fields. The shared executor replaces
+  activated Sim secrets with canonical `{{NAME}}` labels before request formatting. For nested or
+  JSON-string fields, use a small shared selector plus `applyProjected`; verify that selecting the
+  rebuilt params reproduces the projected selection.
+- **Opaque model input sent directly to an external provider** such as a model-read URL or image
+  payload: declare `request.opaqueModelInput` with `mode: 'reject-resolved-secrets'` and select only
+  the exact effective value. The shared `executeTool` preflight rejects incomplete or secret-bearing
+  committed provenance before URL/body formatting or network I/O, preserves safe request bytes,
+  and sends no provenance metadata to the provider.
+- **Opaque model input owned by an authenticated internal route** such as uploaded audio, image,
+  video, file bytes, or signed URLs: add `privateProvenance` to a projected request, or use
+  `mode: 'private-provenance'` when there is no textual projection. The route must call
+  `validateOpaqueModelInputProvenance` before downloading or sending content to the model and must
+  apply the workspace-file provenance guard before reading a persisted workspace file.
+- **Sim-owned durable storage or internal execution handoff** that can later enter a workflow/model
+  (table cells, Agent memory, knowledge documents/chunks, workspace-file contents, or child-workflow
+  input): transport encrypted field-scoped provenance with `request.secretProvenance`. The
+  authenticated receiver validates the exact selection and scope, strips the private envelope, and
+  persists, imports, or propagates it at the owning boundary. Preserve shared legacy behavior for
+  headerless internal calls and rows/files whose provenance marker is `NULL`; never invent a
+  tool-local migration rule.
+
+Hard rules:
+
+- Never substitute secret plaintext into source or serialize plaintext provenance.
+- Never hand-roll private provenance headers/envelopes; the shared `executeTool` boundary owns
+  transport and strips private metadata from functional results.
+- Never attach private provenance to an external URL or to `directExecution`. Use the centralized
+  `opaqueModelInput` rejection mode for external/direct opaque model inputs, or an authenticated
+  internal route when encrypted provenance must cross the boundary.
+- Never sanitize arbitrary third-party tool results. Projection applies only to secrets activated
+  by Sim's resolved-secret provenance for that execution/tool call.
+- Do not add provenance merely because a value is persisted, returned by a tool, or appears in a
+  filename. Require a concrete Sim `{{...}}` resolution path and a later model/log boundary. If an
+  unsupported field can resolve a secret but does not justify durable tracking (for example a
+  `file_write` path), reject it at that exact ingress.
+- At diagnostic boundaries, project only values carrying execution-scoped provenance. Ordinary
+  provider responses, filenames, URLs, and errors remain unchanged when Sim did not resolve a
+  secret into them.
+
+Add focused tests covering named projection, ordinary identical text without provenance, nested
+shape preservation, malformed/incomplete private metadata failing closed, centralized external
+opaque rejection before formatting/I/O without byte changes or metadata transport, headerless
+legacy requests, and absence of private metadata in the public tool result. For durable sinks, also
+cover legacy `NULL` markers, exact-empty new writes, tracked secret writes, stale/missing sidecars,
+and scope isolation.
+
 ## Step 3: Create Block
 
 ### File Location
@@ -535,6 +593,11 @@ If creating V2 versions (API-aligned outputs):
 - [ ] Created `index.ts` barrel export
 - [ ] Registered all tools in `tools/registry.ts`
 - [ ] Ran `bun run tool-metadata:generate` and committed the regenerated artifacts
+- [ ] Classified every model-visible, opaque, Sim-durable, and internal-execution request field
+- [ ] Added shared model-input projection, centralized opaque rejection, or private provenance only
+      where required
+- [ ] Confirmed ordinary third-party tool results are not generically sanitized
+- [ ] Added provenance compatibility and fail-closed boundary tests where applicable
 
 ### Block
 - [ ] Created `blocks/blocks/{service}.ts`

.agents/skills/add-managed-cli/SKILL.md

@@ -0,0 +1,149 @@
+---
+name: add-managed-cli
+description: Add or upgrade a curated, immutable managed CLI for Sim Function sandboxes, including client-safe catalog metadata, a pinned server-only installation recipe, checksum and executable verification, provider compatibility, PATH propagation, content-addressed image identity, and tests. Use when adding a CLI to the Sandbox managed-CLI selector or changing an existing managed CLI version or recipe.
+---
+
+# Add a Managed CLI
+
+Add CLIs through the curated registry. Never turn this surface into arbitrary commands or package names: system packages already cover validated Debian/APT coordinates, while managed CLIs require immutable artifacts and reproducible recipes.
+
+## Read First
+
+Read these live sources before editing; do not copy their current entries into this skill:
+
+1. `apps/sim/lib/execution/remote-sandbox/cli-tools.ts` — persisted IDs and client-safe metadata.
+2. `apps/sim/lib/execution/remote-sandbox/cli-tools.server.ts` — server-only recipes and recipe helpers.
+3. `apps/sim/lib/execution/remote-sandbox/cli-tools.test.ts` — catalog and supply-chain invariants.
+4. `apps/sim/lib/execution/remote-sandbox/cli-tools-boundary.test.ts` — client/server import boundary.
+5. `apps/sim/lib/execution/remote-sandbox/sandbox-spec.ts` — content-addressed hash inputs.
+
+Read `resolve.ts` and `e2b.ts` only when changing provisioning mechanics. A normal catalog addition should not require UI, API, database, resolver, or provider edits; those paths derive from the registries.
+
+Do not modify the dedicated Function base image or the separate Mothership Shell template for a normal managed CLI addition. Managed recipes layer on the Function base image. Do not change `MAX_SANDBOX_CLI_TOOLS` from ten unless the user separately requests a product-limit change.
+
+## 1. Verify the Upstream Release
+
+Use primary upstream release documentation and official artifacts. Establish all of the following before writing code:
+
+- Exact version and stable Linux x86-64 artifact URL. Never use `latest`, mutable redirects, or an unversioned installer.
+- SHA-256 for the exact artifact. Prefer a publisher-signed checksum; otherwise download the official artifact and compute it independently.
+- Archive layout and the exact executable paths to install.
+- Noninteractive, credential-free verification commands for every advertised executable, normally version commands.
+- Required PATH entries under `/opt/sim-cli`.
+- E2B and Daytona compatibility. Default to both only when the same Linux recipe works on both.
+
+When the user does not name a version, select the current stable upstream release from primary sources and state the exact version chosen. Do not silently choose a prerelease or infer a version from an unverified secondary source.
+
+Reject curl-to-shell installers, `npm install`, `pip install`, distro package repositories, arbitrary user commands, and artifacts from unofficial mirrors. Never place credentials, tokens, login commands, or account configuration in an image recipe or build log; authentication is runtime-only.
+
+## 2. Choose an Immutable ID
+
+Use `<tool>@<upstream-version>-r<recipe-revision>`.
+
+- New upstream version: append a new ID ending in `-r1`.
+- Recipe-only change for the same upstream version: append `-r2`, `-r3`, and so on.
+- Never mutate or delete an existing ID or recipe. Persisted sandboxes must continue resolving to the bytes and behavior they selected.
+- On upgrade, retain the old ID and recipe and set its metadata to `selectable: false`. Only the newest version keeps the public label selectable.
+
+Before shipping the first upgrade for a tool family, verify that editing a sandbox cannot leave both the retired and replacement IDs selected. If the generic selector and API validation do not already replace or reject colliding versions, address that once at the generic registry boundary with focused UI and contract tests; never special-case the individual CLI or silently install two versions that expose the same executable.
+
+Recipe identity includes the ID, revision, and SHA-256 in the sandbox image hash. Keeping old entries is what makes that identity reproducible rather than merely cache-busting.
+
+## 3. Add Client-Safe Metadata
+
+In `cli-tools.ts`:
+
+1. Append the ID to `SANDBOX_CLI_TOOL_IDS` in the same order used by the metadata and recipe registries.
+2. Add a `SANDBOX_CLI_TOOLS` entry whose key and `id` exactly match.
+3. Provide a unique selectable `label`, concise `description`, existing `category`, and useful executable/vendor aliases in `searchTerms`.
+4. Add a category only when no existing category is accurate, then ensure it has at least one selectable entry.
+
+Keep this file safe for client bundles. It must not contain artifact URLs, checksums, install commands, verification commands, PATH recipes, provider SDKs, or imports from `cli-tools.server.ts`.
+
+The API enum and searchable grouped selector derive from this registry. Do not add parallel option arrays or route-local wire types.
+
+## 4. Add the Server-Only Recipe
+
+In `cli-tools.server.ts`, use the narrowest existing helper:
+
+- `defineBinaryRecipe` for one downloaded binary.
+- `defineTarGzipRecipe` or `defineZipRecipe` for archives containing binaries.
+- `defineVerifiedRecipe` for a vendor archive or installer layout that needs explicit commands.
+- A direct typed entry only when the helpers cannot faithfully model the release.
+
+Provide every field the recipe contract requires:
+
+- Exact `version`, `artifactUrl`, `artifactName`, and lowercase 64-character `sha256`.
+- Every installed `executable` and a corresponding `verificationCommands` entry.
+- Deterministic extraction/install commands into `/opt/sim-cli`; quote fixed paths and clean temporary artifacts.
+- `pathEntries` when the executable is not installed into the helper's default `bin` directory.
+- `supportedProviders` only when it differs from the E2B-and-Daytona default.
+- `revision` when it differs from `1`; it must agree with the ID suffix.
+
+Verification must prove the command is discoverable through `sandboxCliEnvironment`, not authenticate or contact a user account. Recipe commands run as root during both prebuilt image creation and runtime provisioning.
+
+If the artifact host is new, add only the exact official hostname to the `officialHosts` allowlist in `cli-tools.test.ts`. Treat that as a supply-chain review, not a way to silence the test.
+
+## 5. Preserve Generic Behavior
+
+Confirm the existing generic paths remain sufficient:
+
+- `sandboxCliToolRecipes` canonicalizes and resolves the recipe.
+- `sandboxCliEnvironment` propagates PATH to Python subprocesses, JavaScript subprocesses, and Shell.
+- E2B bakes the recipe into the custom image; runtime-strategy providers install it within the Function timeout.
+- CLI-only sandboxes remain buildable even with no language packages.
+- `hashSandboxSpec` includes recipe ID, revision, and checksum while preserving the legacy hash for an empty CLI list.
+- The settings selector derives groups and search aliases from client-safe metadata.
+
+Do not special-case a CLI in those layers unless the registry contract cannot express a genuine provider requirement. Extend the registry contract generically when multiple CLIs need the same new behavior.
+
+## 6. Test the Addition
+
+Extend tests when the new entry introduces behavior not already covered:
+
+- For every upgrade, add a regression proving the old ID and recipe remain resolvable but non-selectable, while the replacement ID is selectable.
+- Add important executable aliases to the table-driven search assertion.
+- Add a focused assertion for a multi-executable recipe, custom PATH, or restricted provider.
+- Add an opt-in credentialed smoke test only when installation plus a real minimal command cannot be validated without authentication. Read credentials from test-only environment variables, skip by default, create them only at runtime, and always tear down the sandbox.
+
+Never commit downloaded artifacts or credentials.
+
+## Required Validation
+
+From `apps/sim`:
+
+```bash
+bunx vitest run \
+  lib/execution/remote-sandbox/cli-tools.test.ts \
+  lib/execution/remote-sandbox/cli-tools-boundary.test.ts \
+  lib/execution/remote-sandbox/sandbox-spec.test.ts \
+  lib/execution/remote-sandbox/resolve.test.ts \
+  lib/api/contracts/sandboxes.test.ts \
+  'app/workspace/[workspaceId]/settings/components/sandboxes/utils.test.ts' \
+  'app/workspace/[workspaceId]/settings/components/sandboxes/components/sandbox-editor.test.tsx'
+```
+
+From the repository root:
+
+```bash
+bun run type-check
+bun run check:api-validation
+bunx biome check \
+  apps/sim/lib/execution/remote-sandbox/cli-tools.ts \
+  apps/sim/lib/execution/remote-sandbox/cli-tools.server.ts \
+  apps/sim/lib/execution/remote-sandbox/cli-tools.test.ts
+git diff --check
+```
+
+For a new recipe, also exercise its install and every verification command in an actual E2B or Daytona sandbox when credentials and network access are available. Report clearly when only registry/unit validation ran.
+
+## Completion Checklist
+
+- [ ] Official immutable Linux x86-64 artifact and SHA-256 verified.
+- [ ] Versioned ID appended; old IDs and recipes retained.
+- [ ] Client metadata is searchable, categorized, unique, and recipe-free.
+- [ ] Server recipe is pinned, integrity-checked, noninteractive, and credential-free.
+- [ ] Every advertised executable has an offline verification command and PATH entry.
+- [ ] Provider compatibility is explicit and accurate.
+- [ ] Catalog, boundary, hash, resolver, type, API-validation, format, and diff checks pass.
+- [ ] Real provider installation was tested, or the missing live verification is disclosed.

.agents/skills/add-managed-cli/agents/openai.yaml

@@ -0,0 +1,4 @@
+interface:
+  display_name: "Add Managed CLI"
+  short_description: "Add a verified CLI to sandbox images"
+  default_prompt: "Use $add-managed-cli to add a pinned, verified CLI to Sim sandbox images."