Updates

Author: Sg312

apps/desktop/src/main/index.ts

@@ -412,19 +412,9 @@ function main(): void {
   })
 
   void app.whenReady().then(async () => {
-    // Unpackaged runs only: `electron .` has no bundle, so macOS shows the
-    // default Electron atom in the Dock unless we set the mark ourselves.
-    //
-    // A PACKAGED app must NOT do this. It already carries the per-channel
-    // build/icon*.icns, which Finder, the dmg, Launchpad, Cmd-Tab and the Dock
-    // all read — and the icns holds every representation up to 1024px, while
-    // these PNGs are a single 512px copy of the same artwork with no @2x. A
-    // lone PNG loads at scale factor 1, so macOS treated 512px as 512 POINTS
-    // and upscaled it 2x on Retina: the running app's Dock icon came out
-    // visibly softer than the identical icon shown for the same app when it
-    // was closed. Overriding a correct multi-resolution icon with a low-res
-    // copy of itself is all this ever did once packaged.
-    if (process.platform === 'darwin' && !app.isPackaged) {
+    // Use the same high-resolution source in packaged and unpackaged apps so
+    // macOS renders every environment marker consistently in the Dock.
+    if (process.platform === 'darwin') {
       const channel = channelForOrigin(config.getOrigin())
       app.dock?.setIcon(join(__dirname, '..', 'static', DOCK_ICON_FOR_CHANNEL[channel]))
     }

apps/sim/lib/copilot/vfs/operations.test.ts

@@ -67,6 +67,30 @@ describe('grep', () => {
     expect(matches[1]).toMatchObject({ path: 'a.txt', line: 3, content: 'hello' })
   })
 
+  it('truncates oversized matched lines so a minified single-line file cannot return whole', () => {
+    const giantLine = `{"status":"error"}${'x'.repeat(50_000)}`
+    const files = vfsFromEntries([['internal/tool-results/run.json', giantLine]])
+    const matches = grep(files, 'status', undefined, { outputMode: 'content' }) as Array<{
+      content: string
+    }>
+    expect(matches).toHaveLength(1)
+    expect(matches[0].content.length).toBeLessThan(2_200)
+    expect(matches[0].content).toContain('[line truncated: 50018 chars total]')
+  })
+
+  it('truncates oversized context lines around a match', () => {
+    const files = vfsFromEntries([['a.txt', `before${'y'.repeat(10_000)}\nneedle\nafter`]])
+    const matches = grep(files, 'needle', undefined, {
+      outputMode: 'content',
+      context: 1,
+    }) as Array<{
+      content: string
+    }>
+    expect(matches).toHaveLength(3)
+    expect(matches[0].content.length).toBeLessThan(2_200)
+    expect(matches[1].content).toBe('needle')
+  })
+
   it('strips CR before end-of-line matching on CRLF content', () => {
     const files = vfsFromEntries([['x.txt', 'foo\r\n']])
     const matches = grep(files, 'foo$', undefined, { outputMode: 'content' })

apps/sim/lib/copilot/vfs/operations.ts

@@ -1,4 +1,5 @@
 import { createLogger } from '@sim/logger'
+import { truncate } from '@sim/utils/string'
 import micromatch from 'micromatch'
 import {
   compileLinearRegex,
@@ -9,6 +10,23 @@ import {
 
 const logger = createLogger('VfsOperations')
 
+/**
+ * Maximum characters returned for one matched (or context) line in grep
+ * `content` mode. Minified single-line files (workflow JSON, persisted tool
+ * results) make one match the entire file otherwise — a single grep can then
+ * blow through the inline tool-result budget and the caller's context window.
+ */
+const GREP_MATCH_MAX_CHARS = 2_000
+
+/**
+ * Truncates one grep match line to {@link GREP_MATCH_MAX_CHARS}, noting the
+ * original length so the caller knows the line continues.
+ */
+function capGrepMatchContent(line: string): string {
+  if (line.length <= GREP_MATCH_MAX_CHARS) return line
+  return truncate(line, GREP_MATCH_MAX_CHARS, ` … [line truncated: ${line.length} chars total]`)
+}
+
 export interface GrepMatch {
   path: string
   line: number
@@ -221,14 +239,14 @@ export function grep(
             matches.push({
               path: filePath,
               line: showLineNumbers ? j + 1 : 0,
-              content: lines[j],
+              content: capGrepMatchContent(lines[j]),
             })
           }
         } else {
           matches.push({
             path: filePath,
             line: showLineNumbers ? i + 1 : 0,
-            content: lines[i],
+            content: capGrepMatchContent(lines[i]),
           })
         }
         if (matches.length >= maxResults) return matches