chore: remove internal patch reference comments

Author: Alp

packages/zql/src/ivm/memory-source.ts

@@ -84,8 +84,13 @@ export type Connection = {
     | undefined;
   readonly debug?: DebugDelegate | undefined;
   lastPushedEpoch: number;
+  // Cached result of primaryKeyConstraintFromFilters(), computed once at
+  // connect() time. Avoids recomputing from filters on every #fetch() call.
   pkConstraint: Constraint | undefined;
+  // Per-connection cache of #getOrCreateIndex results, keyed by constraint
+  // shape + sort. Avoids repeated JSON.stringify + Map lookups on every fetch.
   indexCache: Map<string, Index>;
+  // Pre-computed sort key for index cache lookups, avoids recomputing per fetch.
   requestedSortKey: string;
 };
 
@@ -101,6 +106,8 @@ export class MemorySource implements Source {
   readonly #columns: Record<string, SchemaValue>;
   readonly #primaryKey: PrimaryKey;
   readonly #primaryIndexSort: Ordering;
+  // Cached JSON.stringify of #primaryIndexSort, used in #getPrimaryIndex()
+  // and constructor to avoid repeated serialization.
   readonly #primaryIndexKey: string;
   readonly #indexes: Map<string, Index> = new Map();
   readonly #connections: Connection[] = [];
@@ -270,7 +277,6 @@ export class MemorySource implements Source {
     const connectionComparator = (r1: Row, r2: Row) =>
       compareRows(r1, r2) * (req.reverse ? -1 : 1);
 
-    // Patch 7: Use cached pkConstraint from connection instead of recomputing
     const pkConstraint = conn.pkConstraint;
     // The primary key constraint will be more limiting than the constraint
     // so swap out to that if it exists.
@@ -295,7 +301,6 @@ export class MemorySource implements Source {
       indexSort.push(...requestedSort);
     }
 
-    // Patch 4: Cache index lookup per connection
     let constraintShapeKey = '';
     if (fetchOrPkConstraint) {
       for (const key of Object.keys(fetchOrPkConstraint)) {