docs: correct comments that name symbols which no longer exist

Each of these points a reader at an identifier that is not in the repo:

- table/import-data.ts, table/service.ts — `acquireTablePositionLock` and
  `nextAutoPosition` were removed with the service.ts split; the surviving lock
  is `acquireRowOrderLock`, which import-data.ts already imports and calls.
  service.ts's mention is load-bearing: it exists to tell the reader which
  other lock this one mirrors, for lock-ordering.
- resources/orchestration/restore-resource.ts — named `performRestoreFolder`
  (the callee is `restoreFolder`) and described a `'workflow'` default it
  falls back to. There is no such default: resourceType is required and the
  config lookup is a bare index. Describing a fiction is how a future reader
  talks themselves into relaxing the total Record to a Partial.
- knowledge/search/queries.ts — cited apps/docs/app/api/chat/route.ts, deleted
  with Ask AI. The k=60 it pins against now lives in the docs search route.
- rate-limiter/hosted-key/queue.ts — documented a `waitForHead` method the
  class does not have; the queue exposes `checkHead` and the polling loop is
  private to the consumer.
- logs/log-views.ts — a "Level 1.5 / 2 / 3" scheme that appears nowhere else;
  the real contract is the five named views. Dropped, and the three banner
  rules with it (CLAUDE.md bans banner separators).

Comment-only apart from the log-views banners.

Author: waleedlatif1

apps/sim/lib/core/rate-limiter/hosted-key/queue.ts

@@ -69,7 +69,7 @@ export interface EnqueueResult {
 /**
  * Per-workspace+provider FIFO queue for hosted-key acquisitions.
  *
- * Callers `enqueue` to claim a position, then `waitForHead` until they're at
+ * Callers `enqueue` to claim a position, then poll `checkHead` until they're at
  * the head, then attempt to consume from the token bucket. On success or cap
  * exceeded, they `dequeue` to make room for the next caller.
  *

apps/sim/lib/knowledge/search/queries.ts

@@ -314,7 +314,7 @@ const FTS_CONFIG = 'english'
 
 /**
  * Reciprocal-rank-fusion damping constant. 60 is the value from the original RRF
- * paper and matches the docs Ask-AI retriever (`apps/docs/app/api/chat/route.ts`).
+ * paper and matches the docs search retriever (`apps/docs/app/api/search/route.ts`).
  */
 export const RRF_K = 60
 

apps/sim/lib/logs/log-views.ts

@@ -45,8 +45,7 @@ const DEFAULT_MATCH_TIME_BUDGET_MS = 5_000
  */
 const DEFAULT_MAX_SCANNED_CHARS = 64 * 1024 * 1024
 
-// Overview (Level 2): block tree with timing + cost, NO input/output.
-
+/** Block tree with timing and cost, without input/output. */
 export interface OverviewSpan {
   id: string
   blockId?: string
@@ -75,10 +74,7 @@ export function toOverview(spans: TraceSpan[]): OverviewSpan[] {
   })
 }
 
-// ---------------------------------------------------------------------------
-// Trace (Level 1.5): condensed per-block digest — names, statuses, counts.
-// ---------------------------------------------------------------------------
-
+/** Condensed per-block digest: names, statuses, counts. */
 export interface TraceDigestEntry {
   /** Block id when the spans carry one; the drill-in key for `full` blockIds. */
   blockId?: string
@@ -125,9 +121,7 @@ export function toTrace(spans: TraceSpan[]): TraceDigestEntry[] {
   return Array.from(byKey.values())
 }
 
-// ---------------------------------------------------------------------------
-// Full (Level 3): block tree WITH materialized input/output.
-
+/** Block tree with materialized input/output. */
 export interface FullSpan extends OverviewSpan {
   startTime?: string
   endTime?: string

apps/sim/lib/resources/orchestration/restore-resource.ts

@@ -48,8 +48,9 @@ type RestorableFolderType = 'folder' | 'knowledge_folder' | 'table_folder'
 /**
  * Deliberately a total `Record` over the folder types, not a `Partial` one: adding a tree to
  * `RestorableFolderType` without a mapping here has to fail the build. With a partial map the
- * lookup would yield `undefined`, `performRestoreFolder` would fall back to its `'workflow'`
- * default, and the restore would silently target the wrong tree.
+ * lookup would yield `undefined`, which `restoreFolder` types as a required
+ * `FolderResourceType` — so the failure would surface as an undefined folder config deep in
+ * the cascade rather than at the call site.
  */
 const FOLDER_RESOURCE_TYPE_BY_RESTORABLE: Record<RestorableFolderType, FolderResourceType> = {
   folder: 'workflow',

apps/sim/lib/table/import-data.ts

@@ -58,7 +58,7 @@ export interface BulkImportBatch {
  * Inserts one batch of rows for an async import in a single committed statement.
  *
  * Differs from {@link batchInsertRowsWithTx} for the bulk-load case: caller-supplied
- * contiguous positions (no `acquireTablePositionLock` / `nextAutoPosition` scan — an
+ * contiguous order keys (no `acquireRowOrderLock` scan — an
  * import owns its hidden table as the sole writer), no `RETURNING`, and **no
  * `fireTableTrigger` / `runWorkflowColumn`** (a 1M-row import must not dispatch a
  * workflow run per row). `row_count` is maintained set-based by the statement-level

apps/sim/lib/table/service.ts

@@ -122,7 +122,7 @@ function readLocks(row: {
  * Uses an advisory lock (not `SELECT ... FOR UPDATE` on the definition row) so
  * it adds no edges to the row-lock graph — the row-count trigger (migration
  * 0198) locks the definition row from `insertRow`/`deleteRow`, and a FOR UPDATE
- * here would invert that order. Mirrors `acquireTablePositionLock`. The lock and
+ * here would invert that order. Mirrors `acquireRowOrderLock`. The lock and
  * the read both release at COMMIT/ROLLBACK; the wait is bounded by the
  * `statement_timeout` set in `setTableTxTimeouts`.
  */