feat(copilot): build the docs vfs from a generated manifest, rescope search_docs by j15z · Pull Request #5948 · simstudioai/sim

j15z · 2026-07-25T00:37:39Z

Summary

Sim's public documentation becomes a read-only docs/ tree in the copilot VFS, built from a generated manifest and navigated with glob/read/grep. search_documentation is replaced by search_docs, scoped to docs/ VFS paths, with every result carrying the path to read next (see the companion PR for the mothership side).

Docs corpus (docs/): a generated manifest (docs-manifest.ts, synced from apps/docs/content/docs/en with a CI --check) defines every page; glob lists it for free, read fetches the page live from docs.sim.ai, grep is single-page only (each page is one fetch). Opt-in like uploads/: an unscoped glob/grep never sees it. Academy and API-reference are deliberately unmounted.
search_docs server tool replaces search_documentation: vector search over docs_embeddings with optional path scoping (page or section, handling both fumadocs on-disk layouts), topK clamping (default 5, max 25), and explicit shortfall reporting (note) so a filtered-empty result is never mistaken for "the docs lack this topic". Unscoped search excludes unmounted sections and the root homepage (indexed but deliberately absent from the manifest).
Retired outright — no shims: search_documentation has no registry alias and get_platform_actions's handler is deleted; both ids are out of the regenerated catalog/schemas. They stay in HIDDEN_TOOL_NAMES forever (the load_agent_skill precedent) so historical persisted chats replay without chips for retired tools. search-docs-dispatch.test.ts pins search_docs's own catalog -> route -> handler chain and the retired ids' gone-but-chip-hidden state.
Hardening from review: a trailing-slash docs/ glob resolves like docs instead of silently matching nothing; a docs page whose single line exceeds the inline cap fails with grep guidance instead of returning an over-cap payload as success; handler-level tests cover the docs routing, error surfacing, and truncation paths.
Lean search agent: the tool catalog is regenerated for the companion change — the search subagent's task description now tells callers to pass a fully self-contained task; it no longer inherits the conversation.
@Docs tagging is inert but documented as a seam: the 'docs' context kind stays in the wire schema and panel union with comments at all three layers saying a revival should be rebuilt on search_docs; the server drops the context (pinned by test). The old mention UI that emitted it was already unreachable.
UI chips: docs reads render as Section/page; search_docs renders as "Searching Sim docs" with the query in the title.

Type of Change

Testing

Unit suites cover the manifest scoping, path folding, search scoping/clamping/shortfalls, VFS docs routing and truncation, the search_docs dispatch chain, and tool display; tsc --noEmit and biome are clean. docs-manifest:check runs in CI so the manifest cannot drift from the docs tree. Reviewers should focus on docs-search.ts's source_document path mapping (en-relative mdx vs VFS/URL paths for section overviews) and the deploy-window behavior described below.

Checklist

Code follows project style guidelines
Self-reviewed my changes
Tests added/updated and passing
No new warnings introduced
I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

Screenshots/Videos

UI impact is chip text only: docs reads render as Section/page (e.g. "Read Workflows/agent") and docs searches as "Searching Sim docs for "..."".

Companion: simstudioai/mothership#371

Merge Sim first — the docs corpus VFS and the search_docs handler live here. There is deliberately no compatibility shim: whichever side lags during the deploy window sees recoverable tool-not-found results on docs lookups until the other side ships.

…ocs; serve openapi.json publicly - search_docs server tool: same vector search over docs_embeddings plus an optional docs/documentation/... VFS path prefix mapped onto a source_document scope (covers both <tail>.mdx and <tail>/... layouts); unscoped searches exclude academy/ and api-reference/ rows so the scope is exactly the Documentation tab - @docs chat context repointed to the new tool; display label updated - apps/docs now serves /openapi.json so the mothership can build its docs/api-reference/<tag>.json VFS views from the deployed spec - generated tool catalog/schemas regenerated from the mothership contract Companion: simstudioai/mothership feat/enhance-search-agent Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

…tool chips read("docs/documentation/workflows/index.mdx") now renders "Read Workflows/index" instead of the leaf-only fallback ("Read Index"). Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

"Searched docs" becomes 'Searched docs for "<query>"' (toolTitle/title preferred, query fallback, truncated at 60 chars). Also adds the missing browser_list_sessions display title the catalog regen surfaced. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

A 400 from the chat POST's ChatMessageSchema parse returned the zod issues to the client but logged nothing, making "message disappears on send" undiagnosable from server logs. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

A workspace_resource tag emitted with id:"" resolved through `??` chains (which keep empty strings), got persisted onto the chat's resources, and then every send attached it and failed the chat request's id min-length validation — the message vanished on send, forever. Resolve chip ids by first non-empty candidate and drop id-less resources when building request attachments. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

…search_docs Replaces the mothership's runtime docs corpus (llms.txt + llms-full.txt + openapi.json behind a 15m TTL cache) with a static manifest generated from the docs source, plus live per-page fetches. ~1,000 fewer lines of hand- written code and one repo instead of two. - scripts/sync-docs-manifest.ts walks apps/docs/content/docs/en and emits lib/copilot/generated/docs-manifest.ts. Each entry is simultaneously the docs/ VFS path and the docs.sim.ai URL path, so a read is a plain fetch. Section index pages fold onto their parent (fumadocs serves /workflows, not /workflows/index); academy/ and api-reference/ are excluded — they stay unmounted and unsearchable, reachable only via scrape_page. - docs-manifest:generate / :check, with a CI step so a page added, renamed, or deleted without regenerating fails the build. Content edits don't. - lib/copilot/docs/docs-corpus.ts + tools/handlers/vfs.ts: glob matches the manifest with no network, read fetches the page live, grep takes exactly ONE page (each is a fetch, so there is no corpus-wide grep). Opt-in like uploads/ — only an explicit docs/ prefix ever matches. - search_docs now scopes to the docs/ tree instead of docs/documentation/, validates its path against the manifest (a bad path errors instead of silently returning nothing), and returns the docs/ path with every chunk so search chains into read. Unscoped searches drop rows the agent could not then read: unmounted sections, and pages gone since the last index rebuild. - @docs tagging disabled: its query was the raw user message, a poor embedding query, and the mention UI it fed was already dead code. - Reverts the apps/docs /openapi.json route, added only for the old api-reference VFS views. Companion: simstudioai/mothership feat/enhance-search-agent Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>

Multi-agent review of the docs/ VFS change. Applied the behavior-preserving fixes plus two agent-facing bugs that made real pages unreadable. - docs read no longer hard-fails on oversized pages. Six+ live integration references exceed the inline cap (github.mdx is 354KB, sportmonks 513KB), so a plain read of them ALWAYS failed and cost a second fetch to recover. Truncate to the largest whole-line prefix that fits, keep the true totalLines, and tell the model how to page. An explicit offset/limit that still overflows is still an error — that one is a caller mistake. - classify docs fetch failures. Everything collapsed to null, so a permanent 404 was reported to the agent as "temporarily unavailable, retry shortly", inviting a retry loop on a page that will never exist. 4xx (except 429) is now permanent and says so; 5xx/429/network/timeout keep the retry wording. - register search_documentation as a transitional alias for search_docs. sim and mothership deploy independently and the rename deleted the old id on both sides, so BOTH deploy orders broke docs lookup for the window between them. Old params are a subset of the new. Remove once both ship. - extract the index-page fold (X/index.mdx <-> X.mdx) into docs-path.ts. It was re-derived in three places — the manifest generator, the source_document reverse mapping, and the search scope filter — which is the hand-synced-duplicate shape that has drifted in this repo before. - grepDocsPage now goes through grepReadResult, the primitive files/ and uploads/ grep already use, instead of calling grep directly. - couldMatchDocsScope delegates to isDocsPath; the bodies were identical. - drop the dead 'docs' member from AgentContextType. Tests: 404-vs-5xx-vs-429 classification, network failure, and a docs-path round-trip asserting one source candidate reproduces every manifest entry. Verified: tsc clean, 932 copilot tests, biome clean, docs-manifest:check, check:utils and check:api-validation:strict both pass. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>

The SQL LIMIT is applied before the similarity-threshold and liveness filters, so search_docs can return fewer hits than topK — or none, when every candidate was filtered. An empty array is indistinguishable from "the documentation does not cover this", which sends the agent off to guess instead of rephrasing or falling back to glob. searchDocs now returns the drop counts alongside the results, and the tool attaches a note when anything was dropped: how many candidates the index returned, why they went, and what to try next. Silent on the common path. This does not change which rows are returned or how many — the ordering issue behind the shortfall is a pre-existing bug the deleted search-documentation.ts had too, and pushing the threshold into SQL is its own change. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>

vercel · 2026-07-25T00:37:45Z

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment

Project	Deployment	Actions	Updated (UTC)
docs	Skipped		Jul 29, 2026 11:06pm

cursor · 2026-07-25T00:37:49Z

PR Summary

Medium Risk
Large copilot surface area (VFS, embeddings search, tool catalog) with a deliberate no-shim deploy window where docs tools can fail until mothership ships; runtime dependency on docs.sim.ai availability for reads.

Overview
Sim’s public docs move into a read-only docs/ VFS for the copilot: a generated manifest (sync-docs-manifest.ts, docs-manifest:check in CI) lists pages; glob/read/grep hit the corpus opt-in (only explicit docs/ patterns), with read/grep fetching live from docs.sim.ai and single-page grep to avoid hundreds of requests. Academy and api-reference stay unmounted; path folding for fumadocs index.mdx is centralized in docs-path.ts.

search_documentation → search_docs: vector search over docs_embeddings with optional path scoping aligned to VFS pages/sections, topK clamped (default 5, max 25), results carry docs/ paths for follow-up reads, and a note explains filtered/empty hits. get_platform_actions and the old search tool are removed from the catalog (no shims); retired ids remain chip-hidden for old transcripts.

@docs chat context is inert in processContextsServer (whole-message doc pre-search and sanitizeMessageForDocs removed). VFS handlers add docs routing, truncation for oversized pages, and UI labels for docs reads / search_docs queries. Catalog copy updates the search subagent task to be self-contained.

^{Reviewed by Cursor Bugbot for commit 6d75617. Bugbot is set up for automated code reviews on this repo. Configure here.}

github-actions · 2026-07-25T00:38:20Z

⚠️ Cross-repo companion check

One or more companion PRs aren't merged into staging yet. Merging this without them will leave copilot and sim out of sync — merge them in lockstep.

❌ simstudioai/mothership#371 — OPEN, not merged (targets staging) — feat(search): replace search_documentation with path-scoped search_docs

greptile-apps · 2026-07-25T00:45:46Z

Greptile Summary

The PR completes the documentation-tool migration and repairs the previously reported mixed-version dispatch path.

Retains search_documentation as a hidden, Sim-routed catalog alias backed by the search_docs server handler.
Adds coverage for catalog classification, server registration, legacy parameters, and hidden-tool behavior.
Introduces a manifest-backed read-only docs/ VFS with live page reads, single-page grep, and scoped semantic search.
Adds CI validation to keep the generated documentation manifest synchronized.

Confidence Score: 5/5

The PR appears safe to merge.

The previously reported transitional-alias dispatch failure is fully addressed, and no blocking failure remains.

Important Files Changed

Filename	Overview
apps/sim/lib/copilot/tools/server/router.ts	Registers both tool identifiers against the new documentation-search handler, completing the previously missing dispatch path.
apps/sim/lib/copilot/tools/server/docs/search-docs-alias.test.ts	Verifies that the legacy identifier remains known, Sim-routed, registered, hidden, and compatible with legacy parameters.
apps/sim/lib/copilot/generated/tool-catalog-v1.ts	Adds the new tool and preserves the old identifier as a hidden transitional catalog entry.
apps/sim/lib/copilot/docs/docs-corpus.ts	Implements the manifest-backed documentation VFS namespace, live reads, globbing, and single-page grep.
apps/sim/lib/copilot/docs/docs-search.ts	Implements manifest-aware semantic documentation search with path scoping and stale-result filtering.

_{Reviews (4): Last reviewed commit: "fix(copilot): make the search_docs topK ..." | Re-trigger Greptile}

The registry alias added for the rename never fired: executeTool gates on isKnownTool(toolId) — membership in the generated TOOL_CATALOG — before it consults baseServerToolRegistry, so an id absent from the catalog is rejected as unknown and routed to the app-tool path instead. The alias was dead code, and worse, it advertised a mixed-version safety net that did not exist. The companion PR restores search_documentation to the contract as a hidden, deprecated entry, so the id now passes the catalog gate and reaches this alias. Marks it hidden in the UI set too — a call only ever arrives from an older Mothership build and renders as the search_docs it maps onto. Adds a test pinning every link in the dispatch chain (catalog membership, sim route, registered handler, hidden, param shape) — the assertion that would have caught this. Remove all of it once search_docs is live in prod on both sides.

j15z · 2026-07-25T00:55:12Z

Confirmed — you're right, and this was a real hole rather than a nit.

executeTool gates on isKnownTool(toolId) (i.e. toolId in TOOL_CATALOG) before it consults baseServerToolRegistry:

const canUseRegisteredHandler =
  isKnownTool(toolId) && (isSimExecuted(toolId) || ...)
if (!canUseRegisteredHandler) return executeAppTool(toolId, appParams)  // → tool not found

So dropping search_documentation from the generated catalog made the registry alias unreachable, and the mixed-version window it was meant to cover was still open — worse than having no alias, because the PR claimed a safety net that didn't exist.

Fixed across both repos:

mothership#371 (aa46b879): registers search_documentation as a hidden, deprecated catalog entry routed to sim. It's in no agent's AllowedTools, so nothing is ever offered it — it exists only so an inbound call from an older build resolves.
this PR (06f3a3756f): regenerated catalog now contains the id, so it passes the gate and reaches the alias. Also added to the UI hidden set, since such a call renders as the search_docs it maps onto.
Added search-docs-alias.test.ts pinning every link in the chain — catalog membership, sim route, registered handler, hidden flag, param shape. That's the assertion that would have caught this.

One thing your flowchart implies that I want to state explicitly: this only fixes the sim-first direction. New Mothership emitting search_docs at an old Sim can't be fixed from either side, so deploy order remains sim first, then mothership. Noted on both PRs.

940 copilot tests, tsc, and all 11 CI audits pass.

j15z · 2026-07-25T00:55:13Z

@greptile

j15z · 2026-07-25T00:55:14Z

@cursor review

…g search A directory scope matched only `<section>/%`, which covers an overview stored as `<section>/index.mdx` but not one stored as a sibling `<section>.mdx`. Fumadocs accepts both layouts and page scope already handles both via docsSourceCandidates, so a scoped section search could silently omit the overview chunks — and the doc comment claimed it did not. Every section in the tree currently uses the index.mdx layout, so nothing is broken today; this closes the gap before someone adds a sibling overview and gets quietly incomplete results.

j15z · 2026-07-25T01:02:48Z

@greptile

j15z · 2026-07-25T01:02:49Z

@cursor review

cursor

✅ Bugbot reviewed your changes and found no new issues!

Comment @cursor review or bugbot run to trigger another review on this PR

^{Reviewed by Cursor Bugbot for commit e52205d. Configure here.}

The clamp guarded magnitude but not type: Math.min/Math.max propagate NaN, so a non-numeric topK reached the query as `.limit(NaN)`. The `?? DEFAULT` only caught undefined. Nothing enforced this but the generated Ajv schema, and searchDocs is also called directly, so it should not depend on that. Extract clampTopK, which falls back to the default for anything non-finite (NaN, Infinity, a string that slipped through) and clamps the rest to [1, 25]. The clamp was completely untested because the db mock's .limit() stub discarded its argument — the mock now records it. Covers default, cap, floor, truncation, and the non-finite fallback. Worth pinning: staging's search_documentation documented "max 10" and enforced nothing, so this bound is new behavior, not just a bigger number.

j15z · 2026-07-25T01:28:48Z

@greptile

Chips read "Searched docs" with no indication of what was searched. The query-aware title existed earlier on this branch and this commit's own predecessor dropped it: removing search_docs from the catalog deleted the display case and its test, and putting the tool back only restored the static map entry. The generic "every visible catalog tool has a title" assertion still passed, because it checks that a title exists, not that it is the useful one. Chips now read: Searching docs for "how to read workflow logs and view executions" -> Searched docs for "...". The gerund flip already preserves the suffix, so the completed state needs no extra handling — the test now pins that too, since it was the part most likely to regress silently.

…h default Two places decide what the docs/ corpus is: the manifest generator (what is readable) and the vector search's unscoped filter (what is findable). They each carried their own copy of the excluded-section list. If they drift, a hit in a section that is indexed but not mounted comes back as a chunk the agent cannot then read — dropped as stale, silently shrinking the result set. UNMOUNTED_DOCS_SECTIONS is now the one list both import. search_docs returns 5 chunks by default instead of 10; raise topK when a pass genuinely comes back thin. A truncated docs page now routes to one more fetch instead of two. grep and read cost the same single uncached fetch of the page, so grep is an alternative to a read here, never a step after one. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>

…erence tool Picks up get_platform_actions' hidden/retired description from mothership. The id stays in the catalog so isKnownTool keeps routing calls from an older build during a mixed deploy; the handler is unchanged. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>

Two TSDoc blocks sat back to back above DocsSearchOutcome; the first describes DocsSearchScopeError, which had no doc comment of its own. Moved it to the class. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>

…ocumentation The two retirement shims disagreed. search_documentation is in HIDDEN_TOOL_NAMES with no TOOL_TITLES entry, so a call from an older build renders nothing. get_platform_actions was only hidden:true in the generated catalog, which isToolHiddenInUi never consults, and it kept a live title — so the same situation surfaced a "Getting platform actions" chip for a capability that no longer exists. Hiding also drops the row from historical transcripts, since isToolHiddenInUi runs at transcript build time. That is accepted and is the existing behavior for search_documentation. Verified it cannot error: every consumer skips via continue or returns undefined, nothing indexes off the omitted entry, and neither id is a subagent trigger, so no tool group is orphaned. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>

…ex drops, oversized-line reads Review findings applied from the multi-agent pass on this branch: - glob("docs/") matched no key and silently returned empty; normalize now strips trailing slashes so it resolves like "docs" - unscoped search_docs no longer returns root-homepage chunks that would only be counted against topK and then dropped as stale (the manifest deliberately omits index.mdx) - a docs page whose single line exceeds the inline cap now fails with grep guidance instead of returning an over-cap payload as success - test coverage for the vfs docs routing (glob/read/grep dispatch, DocsCorpusError surfacing, truncation paths), the search_docs server tool's shortfall notes, the empty-embedding outcome, and the inert @docs context Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

The search subagent's task description now tells callers to pass a fully self-contained task — it no longer inherits the conversation (see the companion mothership change). Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

Resolution notes: - generated tool catalog/schemas regenerated from the merged mothership contract (staging's browser/terminal agents + this branch's search_docs and lean search task description) - hidden-tools comments merged (load_skill from staging + the two transitional shim notes from this branch) - docs manifest regenerated: staging added integrations/managed_agent, integrations/tiktok, and platform/enterprise/self-hosted pages Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

gitguardian · 2026-07-29T22:04:41Z

⚠️ GitGuardian has uncovered 1 secret following the scan of your pull request.

Please consider investigating the findings and remediating the incidents. Failure to do so may lead to compromising the associated services or software components.

🔎 Detected hardcoded secret in your pull request

GitGuardian id	GitGuardian status	Secret	Commit	Filename
35187658	Triggered	Username Password	`ac8eecc`	apps/desktop/src/main/browser-credentials/vault.test.ts	View secret

🛠 Guidelines to remediate hardcoded secrets

Understand the implications of revoking this secret by investigating where it is used in your code.
Replace and store your secret safely. Learn here the best practices.
Revoke and rotate this secret.
If possible, rewrite git history. Rewriting git history is not a trivial act. You might completely break other contributing developers' workflow and you risk accidentally deleting legitimate data.

To avoid such incidents in the future consider

following these best practices for managing and storing secrets including API keys and other credentials
install secret detection on pre-commit to catch secret before it leaves your machine and ease remediation.

^{🦉 GitGuardian detects secrets in your source code to help developers and security teams secure the modern development process. You are seeing this because you or someone else with access to this repository has authorized GitGuardian to scan your pull request.}

…tions outright, no shims The transitional apparatus is gone: no search_documentation registry alias, no get_platform_actions handler, and the ids are out of the regenerated catalog/schemas. During the deploy window an old Mothership build calling either id gets the recoverable tool-not-found result. The two ids stay in HIDDEN_TOOL_NAMES forever — like load_agent_skill, historical persisted chats contain their tool calls and must replay without rendering chips for retired tools. The alias test is replaced by a dispatch test pinning search_docs's own catalog -> route -> handler chain and the retired ids' gone-but-chip-hidden state. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

cursor

Cursor Bugbot has reviewed your changes using high effort and found 1 potential issue.

^{❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.}

^{Reviewed by Cursor Bugbot for commit a30ad25. Configure here.}

cursor · 2026-07-29T22:34:14Z

+      return { output: { content, totalLines: page.totalLines }, returnedLines: kept }
+    }
+    kept = Math.floor(kept / 2)
+  }


Truncation skips larger fitting prefixes

Medium Severity

truncateDocsPageToInlineCap is documented as returning the largest whole-line prefix that fits the inline budget, but it only halves kept and returns on the first fit. Pages just over the cap can come back at roughly half size even when a much larger prefix would fit, forcing extra read/grep round trips for large docs pages.

^{Reviewed by Cursor Bugbot for commit a30ad25. Configure here.}

…_docs revival The 'docs' ChatContext kind stays in the wire schema and the panel union but resolves to nothing server-side. Comments at all three layers now say why it is kept: a future @docs mention should be rebuilt on search_docs, not by reintroducing the kind or reviving the retired whole-message pre-search. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

…e case The static TOOL_TITLES entry is unreachable for search_docs — the dynamic switch case returns first so it can include the query — so the rename only takes effect there. Tests updated to the new wording. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

j15z and others added 8 commits July 24, 2026 17:13

improvement(copilot): label docs corpus reads as Section/filename in …

f5538e9

…tool chips read("docs/documentation/workflows/index.mdx") now renders "Read Workflows/index" instead of the leaf-only fallback ("Read Index"). Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

j15z requested a review from a team as a code owner July 25, 2026 00:37

github-actions Bot added the requires-mothership-merge Has a companion PR on the mothership/copilot side — merge in lockstep label Jul 25, 2026

greptile-apps Bot reviewed Jul 25, 2026

View reviewed changes

Comment thread apps/sim/lib/copilot/tools/server/router.ts Outdated

vercel Bot temporarily deployed to Preview July 25, 2026 00:55 Inactive

cursor Bot reviewed Jul 25, 2026

View reviewed changes

Comment thread apps/sim/lib/copilot/docs/docs-search.ts

vercel Bot temporarily deployed to Preview July 25, 2026 01:02 Inactive

cursor Bot reviewed Jul 25, 2026

View reviewed changes

vercel Bot temporarily deployed to Preview July 25, 2026 01:27 Inactive

j15z mentioned this pull request Jul 25, 2026

fix(copilot): stop the special-tag parser discarding text it cannot resolve #5952

Merged

10 tasks

vercel Bot temporarily deployed to Preview July 25, 2026 02:08 Inactive

j15z mentioned this pull request Jul 25, 2026

test(copilot): catch a tool silently dropping its query from the chip #5953

Draft

6 tasks

vercel Bot deployed to Preview July 28, 2026 22:38 View deployment

cursor Bot reviewed Jul 28, 2026

View reviewed changes

Comment thread apps/sim/lib/copilot/docs/docs-corpus.ts

Comment thread apps/sim/lib/copilot/docs/docs-search.ts

vercel Bot temporarily deployed to Preview July 28, 2026 23:54 Inactive

cursor Bot reviewed Jul 29, 2026

View reviewed changes

Comment thread apps/sim/lib/copilot/docs/docs-corpus.ts

j15z and others added 2 commits July 28, 2026 18:50

vercel Bot temporarily deployed to Preview July 29, 2026 02:33 Inactive

j15z and others added 2 commits July 29, 2026 09:53

vercel Bot temporarily deployed to Preview July 29, 2026 21:48 Inactive

vercel Bot deployed to Preview July 29, 2026 22:08 View deployment

vercel Bot temporarily deployed to Preview July 29, 2026 22:27 Inactive

cursor Bot reviewed Jul 29, 2026

View reviewed changes

vercel Bot temporarily deployed to Preview July 29, 2026 22:39 Inactive

changed search_docs tool title to Searching Sim docs

3b0c9fd

vercel Bot temporarily deployed to Preview July 29, 2026 22:43 Inactive

vercel Bot temporarily deployed to Preview July 29, 2026 23:06 Inactive

Uh oh!

Conversation

j15z commented Jul 25, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Type of Change

Testing

Checklist

Screenshots/Videos

Uh oh!

vercel Bot commented Jul 25, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

cursor Bot commented Jul 25, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

PR Summary

Uh oh!

github-actions Bot commented Jul 25, 2026

⚠️ Cross-repo companion check

Uh oh!

greptile-apps Bot commented Jul 25, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Greptile Summary

Confidence Score: 5/5

Important Files Changed

Uh oh!

Uh oh!

j15z commented Jul 25, 2026

Uh oh!

j15z commented Jul 25, 2026

Uh oh!

j15z commented Jul 25, 2026

Uh oh!

Uh oh!

j15z commented Jul 25, 2026

Uh oh!

j15z commented Jul 25, 2026

Uh oh!

cursor Bot left a comment

Choose a reason for hiding this comment

Uh oh!

j15z commented Jul 25, 2026

Uh oh!

Uh oh!

Uh oh!

Uh oh!

gitguardian Bot commented Jul 29, 2026

⚠️ GitGuardian has uncovered 1 secret following the scan of your pull request.

Uh oh!

cursor Bot left a comment

Choose a reason for hiding this comment

Uh oh!

cursor Bot Jul 29, 2026

Choose a reason for hiding this comment

Truncation skips larger fitting prefixes

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

j15z commented Jul 25, 2026 •

edited

Loading

vercel Bot commented Jul 25, 2026 •

edited

Loading

cursor Bot commented Jul 25, 2026 •

edited

Loading

greptile-apps Bot commented Jul 25, 2026 •

edited

Loading