fix(function): report code-placeholder compiler invariants as server failures

Overlapping source edits and exhausted parser-sentinel/Python-marker allocation are broken compiler invariants, not problems in the user's code, but they threw CodePlaceholderCompileError and the Function route answered them with 422 like a user syntax error. They now throw a sibling CodePlaceholderInvariantError, which the route's instanceof check does not match, so it falls through to the existing 500 path. User placeholder and syntax errors keep their 422.

Author: waleedlatif1

apps/sim/lib/execution/code-placeholders/compiler.test.ts

@@ -6,6 +6,11 @@ import {
   type CodePlaceholderRuntimeBinding,
   compileCodePlaceholders,
 } from '@/lib/execution/code-placeholders'
+import {
+  applySourceEdits,
+  CodePlaceholderCompileError,
+  CodePlaceholderInvariantError,
+} from '@/lib/execution/code-placeholders/shared'
 import { CodeLanguage } from '@/lib/execution/languages'
 
 const installedGlobals = new Set<string>()
@@ -1543,3 +1548,21 @@ describe('python true positives survive the dot guard', () => {
     expect(await directReadNames(code, CodeLanguage.Python)).toEqual(['API_KEY'])
   })
 })
+
+describe('compiler invariant failures', () => {
+  it('reports overlapping source edits as a compiler invariant, not a user compile error', () => {
+    const thrown = (() => {
+      try {
+        applySourceEdits('return value', [
+          { start: 0, end: 6, text: 'yield' },
+          { start: 3, end: 9, text: 'x' },
+        ])
+      } catch (error) {
+        return error
+      }
+    })()
+
+    expect(thrown).toBeInstanceOf(CodePlaceholderInvariantError)
+    expect(thrown).not.toBeInstanceOf(CodePlaceholderCompileError)
+  })
+})

apps/sim/lib/execution/code-placeholders/javascript.ts

@@ -2,6 +2,7 @@ import ts from '@typescript/typescript6'
 import {
   applySourceEdits,
   CodePlaceholderCompileError,
+  CodePlaceholderInvariantError,
   createCodePlaceholderCompilationContext,
   isOffsetInRanges,
   type SourceEdit,
@@ -261,7 +262,7 @@ function createSentinel(
       return sentinel
     }
   }
-  throw new CodePlaceholderCompileError('Unable to allocate a collision-free parser sentinel')
+  throw new CodePlaceholderInvariantError('Unable to allocate a collision-free parser sentinel')
 }
 
 function createSentinelSource(

apps/sim/lib/execution/code-placeholders/python.ts

@@ -2,6 +2,7 @@ import { sha256Hex } from '@sim/security/hash'
 import {
   applySourceEdits,
   CodePlaceholderCompileError,
+  CodePlaceholderInvariantError,
   createCodePlaceholderCompilationContext,
   isOffsetInRanges,
   type SourceEdit,
@@ -243,7 +244,7 @@ function createSentinel(code: string): string {
     const sentinel = `__sim_placeholder_${digest}__`
     if (!code.includes(sentinel)) return sentinel
   }
-  throw new CodePlaceholderCompileError('Unable to allocate a collision-free Python marker')
+  throw new CodePlaceholderInvariantError('Unable to allocate a collision-free Python marker')
 }
 
 function pythonTriviaGap(

apps/sim/lib/execution/code-placeholders/shared.ts

@@ -27,6 +27,18 @@ export class CodePlaceholderCompileError extends Error {
   }
 }
 
+/**
+ * A broken compiler invariant: Sim's own compilation failed, not the user's code.
+ * Deliberately not a {@link CodePlaceholderCompileError}, so callers that answer a
+ * user compile error with a client status treat this as a server failure.
+ */
+export class CodePlaceholderInvariantError extends Error {
+  constructor(message: string) {
+    super(message)
+    this.name = 'CodePlaceholderInvariantError'
+  }
+}
+
 /**
  * Scans `{{name}}` placeholders, accepting exactly what `/\{\{([^}]+)\}\}/g` accepts —
  * a name may contain `{`, because parameter keys are arbitrary strings rather than
@@ -270,7 +282,7 @@ export function applySourceEdits(code: string, edits: SourceEdit[]): string {
   let output = ''
   for (const edit of sorted) {
     if (edit.start < cursor || edit.end < edit.start || edit.end > code.length) {
-      throw new CodePlaceholderCompileError('Overlapping code placeholder transformations')
+      throw new CodePlaceholderInvariantError('Overlapping code placeholder transformations')
     }
     output += code.slice(cursor, edit.start)
     output += edit.text