@codeyam/codeyam-cli 0.1.0-staging.a77070e → 0.1.0-staging.aa28063

package/codeyam-cli/templates/seed-adapters/supabase.ts

@@ -10,27 +10,39 @@
  *   "tableName": [{ "column": "value", ... }, ...],
  *   "_auth": {                          // optional
  *     "email": "alice@example.com",
- *     "password": "test123"
+ *     "password": "test123"             // optional — default used if omitted
  *   }
  * }
  *
  * When _auth is present, the adapter:
  * 1. Creates the user if they don't exist (auto-confirms email)
- * 2. Signs in with signInWithPassword to get a valid session
- * 3. Writes session cookies to .codeyam/tmp/seed-session.json
+ * 2. Builds a synthetic JWT with far-future expiry (no real sign-in needed)
+ * 3. Writes session cookies + auth mocks to .codeyam/tmp/seed-session.json
  *    so the CodeYam proxy can inject them into the browser
  *
+ * The password field is optional — if omitted, a default dev password is used.
+ * No real JWTs are stored; the preload module intercepts all auth API calls.
+ *
+ * Data operations use direct PostgreSQL (pg) for speed — TRUNCATE CASCADE
+ * and batched INSERT are ~2x faster than individual PostgREST HTTP calls.
+ *
  * Requirements:
+ * - DATABASE_URL env var with a PostgreSQL connection string (session pooler recommended)
  * - A Supabase project URL (https://<ref>.supabase.co) in any env var
  * - A secret key (sb_secret_... or legacy eyJ... service_role JWT) in any env var
  *
- * The adapter scans ALL env vars by value pattern — no specific naming required.
+ * The adapter scans ALL env vars by value pattern for Supabase URL and secret key —
+ * no specific naming required. DATABASE_URL must be set explicitly.
  */
 
 import { createClient } from '@supabase/supabase-js';
+import { Prisma } from '@prisma/client';
+import pg from 'pg';
 import * as fs from 'fs';
 import * as path from 'path';
 
+const { Client } = pg;
+
 /**
  * Scan all env vars for values matching a pattern.
  * Returns ALL unique matches (not just the first) so callers can disambiguate.
@@ -65,7 +77,6 @@ const supabaseUrl = findAllEnvByPattern(
 
 // New-format keys are unambiguous by prefix
 const newSecretKey = findAllEnvByPattern(/^sb_secret_/)[0];
-const newAnonKey = findAllEnvByPattern(/^sb_publishable_/)[0];
 
 // Legacy JWT keys — disambiguate by decoding the role claim
 const legacyJwts = findAllEnvByPattern(
@@ -74,10 +85,8 @@ const legacyJwts = findAllEnvByPattern(
 const legacyServiceRole = legacyJwts.find(
   (jwt) => getJwtRole(jwt) === 'service_role',
 );
-const legacyAnon = legacyJwts.find((jwt) => getJwtRole(jwt) === 'anon');
 
 const secretKey = newSecretKey || legacyServiceRole;
-const anonKey = newAnonKey || legacyAnon;
 
 if (!supabaseUrl || !secretKey) {
   console.error(
@@ -92,7 +101,17 @@ if (!supabaseUrl || !secretKey) {
   process.exit(1);
 }
 
-// Admin client for user management (uses secret key to bypass RLS)
+// DATABASE_URL for direct PostgreSQL access (session pooler recommended for speed)
+const databaseUrl = process.env.DATABASE_URL;
+if (!databaseUrl || !databaseUrl.startsWith('postgresql://')) {
+  console.error('DATABASE_URL must be set to a PostgreSQL connection string.');
+  console.error(
+    'Use the session pooler connection string (port 5432) from your Supabase dashboard.',
+  );
+  process.exit(1);
+}
+
+// Admin client for user management only (uses secret key to bypass RLS)
 const supabase = createClient(supabaseUrl, secretKey, {
   auth: { autoRefreshToken: false, persistSession: false },
 });
@@ -108,53 +127,213 @@ function getProjectRef(): string {
   }
 }
 
+/**
+ * Build a mapping from lowercased seed-data keys to actual PostgreSQL table names.
+ * Prisma creates tables with the model name (PascalCase) unless @@map is used.
+ * PostgreSQL requires the exact table name, so we need this translation.
+ */
+function buildTableNameMap(): Record<string, string> {
+  const map: Record<string, string> = {};
+  for (const model of Prisma.dmmf.datamodel.models) {
+    // dbName is set when @@map is used, otherwise null → use model name
+    const dbName = (model as any).dbName || model.name;
+    // Map lowercased model name → actual table name
+    const lowered = model.name.charAt(0).toLowerCase() + model.name.slice(1);
+    map[lowered] = dbName;
+    map[model.name] = dbName; // Also map PascalCase for safety
+  }
+  return map;
+}
+
+/**
+ * Build column name mapping from Prisma field names to database column names.
+ * Prisma uses camelCase field names but generates snake_case columns unless @map is used.
+ */
+function buildColumnMap(tableName: string): Record<string, string> {
+  const map: Record<string, string> = {};
+  for (const model of Prisma.dmmf.datamodel.models) {
+    const dbName = (model as any).dbName || model.name;
+    if (dbName !== tableName) continue;
+    for (const field of model.fields) {
+      if (field.kind === 'object') continue; // Skip relation fields
+      const colName = (field as any).dbName || field.name;
+      map[field.name] = colName;
+    }
+    break;
+  }
+  return map;
+}
+
 async function seedTables(seed: Record<string, unknown[]>) {
-  const tableNames = Object.keys(seed);
-  if (tableNames.length === 0) return;
-
-  console.log(`Seeding tables: ${tableNames.join(', ')}`);
-
-  // Wipe tables in reverse order (to respect foreign keys)
-  for (const table of [...tableNames].reverse()) {
-    const { error } = await supabase.from(table).delete().gte('id', 0);
-    if (error) {
-      const { error: error2 } = await supabase
-        .from(table)
-        .delete()
-        .not('id', 'is', null);
-      if (error2) {
-        console.warn(`  Could not clear ${table}: ${error2.message}`);
-      } else {
-        console.log(`  Cleared ${table}`);
+  if (Object.keys(seed).length === 0) return;
+
+  const tableMap = buildTableNameMap();
+
+  // Discover ALL models from the Prisma schema — not just the tables in the seed data.
+  // This ensures FK-dependent tables are cleared even when the seed only contains
+  // the parent table's data.
+  const allTables = Prisma.dmmf.datamodel.models.map(
+    (m) => (m as any).dbName || m.name,
+  );
+
+  console.log(
+    `Clearing ${allTables.length} tables, seeding: ${Object.keys(seed).join(', ')}`,
+  );
+
+  const client = new Client({
+    connectionString: databaseUrl,
+    ssl: { rejectUnauthorized: false },
+  });
+  await client.connect();
+
+  try {
+    // TRUNCATE all tables in one statement — CASCADE handles FK dependencies
+    // automatically, so table order doesn't matter. Much faster than
+    // individual DELETE FROM queries via PostgREST.
+    const quoted = allTables.map((t) => `"${t}"`).join(', ');
+    await client.query(`TRUNCATE ${quoted} CASCADE`);
+    console.log(`  Cleared ${allTables.length} tables`);
+
+    // Reset auto-increment sequences so IDs start from 1 on each scenario switch.
+    // Without this, IDs keep climbing and hardcoded URLs like /drinks/1 break.
+    for (const table of allTables) {
+      try {
+        await client.query(
+          `SELECT setval(pg_get_serial_sequence('"${table}"', 'id'), 1, false)`,
+        );
+      } catch {
+        // Table may not have an 'id' serial column — skip
       }
-    } else {
-      console.log(`  Cleared ${table}`);
     }
+
+    // Insert seed data using batched INSERT for speed
+    for (const [seedKey, rows] of Object.entries(seed)) {
+      if (!Array.isArray(rows) || rows.length === 0) continue;
+      const table = tableMap[seedKey] || seedKey;
+      const columnMap = buildColumnMap(table);
+
+      // Get column names from the first row's keys, mapped to DB column names
+      const fieldNames = Object.keys(rows[0] as Record<string, unknown>);
+      const dbColumns = fieldNames.map((f) => columnMap[f] || f);
+      const quotedCols = dbColumns.map((c) => `"${c}"`).join(', ');
+
+      // Build parameterized VALUES clause for all rows at once
+      const params: unknown[] = [];
+      const valueClauses: string[] = [];
+      for (let i = 0; i < rows.length; i++) {
+        const row = rows[i] as Record<string, unknown>;
+        const placeholders: string[] = [];
+        for (const field of fieldNames) {
+          params.push(row[field]);
+          placeholders.push(`$${params.length}`);
+        }
+        valueClauses.push(`(${placeholders.join(', ')})`);
+      }
+
+      await client.query(
+        `INSERT INTO "${table}" (${quotedCols}) VALUES ${valueClauses.join(', ')}`,
+        params,
+      );
+      console.log(`  Seeded ${rows.length} rows into ${table}`);
+    }
+  } finally {
+    await client.end();
   }
+}
 
-  // Insert seed data
-  for (const [table, rows] of Object.entries(seed)) {
-    if (!Array.isArray(rows) || rows.length === 0) continue;
-    const { error } = await supabase.from(table).insert(rows);
-    if (error) {
-      console.error(`  Failed to seed ${table}: ${error.message}`);
-      process.exit(1);
+/**
+ * Export mode: dump current database state to a JSON file.
+ * Used by CodeYam to save interactive changes back to scenario seed data.
+ *
+ * Usage: npx tsx .codeyam/seed-adapter.ts --export <output-path.json>
+ */
+async function exportData(outputPath: string) {
+  const tableMap = buildTableNameMap();
+  const modelNames = Prisma.dmmf.datamodel.models.map((m) => m.name);
+
+  const client = new Client({
+    connectionString: databaseUrl,
+    ssl: { rejectUnauthorized: false },
+  });
+  await client.connect();
+
+  try {
+    const seed: Record<string, unknown[]> = {};
+    for (const model of modelNames) {
+      const camelCase = model.charAt(0).toLowerCase() + model.slice(1);
+      const table = tableMap[camelCase] || model;
+      try {
+        const result = await client.query(`SELECT * FROM "${table}"`);
+        if (result.rows.length > 0) {
+          seed[camelCase] = result.rows;
+        }
+      } catch {
+        // Skip tables that can't be queried
+      }
     }
-    console.log(`  Seeded ${rows.length} rows into ${table}`);
+
+    fs.writeFileSync(outputPath, JSON.stringify(seed, null, 2));
+    console.log(`Exported ${Object.keys(seed).length} tables`);
+  } finally {
+    await client.end();
   }
 }
 
 /**
- * Handle auth: create user, sign in, write session cookies.
+ * Build a minimal JWT with a far-future expiry.
+ * The signature is fake — it doesn't matter because the CodeYam preload module
+ * intercepts the Supabase auth validation endpoint and returns mock data.
+ */
+function buildFakeJwt(userId: string, email: string): string {
+  const header = { alg: 'HS256', typ: 'JWT' };
+  const payload = {
+    sub: userId,
+    email,
+    role: 'authenticated',
+    aud: 'authenticated',
+    exp: 9999999999,
+    iat: Math.floor(Date.now() / 1000),
+  };
+  const encode = (obj: object) =>
+    Buffer.from(JSON.stringify(obj)).toString('base64url');
+  return `${encode(header)}.${encode(payload)}.codeyam-mock-signature`;
+}
+
+/**
+ * Build a Supabase user response object matching the /auth/v1/user endpoint format.
+ */
+function buildUserResponse(user: { id: string; email?: string }) {
+  return {
+    id: user.id,
+    aud: 'authenticated',
+    role: 'authenticated',
+    email: user.email || '',
+    email_confirmed_at: new Date().toISOString(),
+    app_metadata: { provider: 'email', providers: ['email'] },
+    user_metadata: {},
+    created_at: new Date().toISOString(),
+  };
+}
+
+const DEFAULT_AUTH_PASSWORD = 'codeyam-dev-password-123!';
+
+/**
+ * Handle auth: create user, build synthetic session, write session cookies.
  * Returns the user ID so callers can replace __AUTH_USER_ID__ placeholders in seed data.
+ *
+ * No real sign-in is performed — the session cookie uses a synthetic JWT with
+ * far-future expiry, and externalApis mocks intercept all Supabase auth API calls.
+ * This avoids storing real tokens (security risk + expiration) in scenario files.
  */
 async function handleAuth(auth: {
   email: string;
-  password: string;
+  password?: string;
 }): Promise<string> {
-  const { email, password } = auth;
+  const email = auth.email;
+  const password = auth.password || DEFAULT_AUTH_PASSWORD;
 
-  // Create user if they don't exist (auto-confirm email so sign-in works)
+  // Create user if they don't exist (auto-confirm email)
+  // We still need the auth.users row for PostgREST FK relationships.
   const { data: createData, error: createError } =
     await supabase.auth.admin.createUser({
       email,
@@ -162,72 +341,79 @@ async function handleAuth(auth: {
       email_confirm: true,
     });
 
+  let userId: string;
+
   if (createError) {
     if (createError.message.includes('already been registered')) {
-      // User exists — update their password so sign-in works even if the
-      // scenario specifies a different password than a previous run.
+      // User exists — look up their ID
       const { data: listData } = await supabase.auth.admin.listUsers();
       const existingUser = listData?.users?.find((u) => u.email === email);
       if (existingUser) {
-        await supabase.auth.admin.updateUserById(existingUser.id, {
-          password,
-        });
-        console.log(`  Updated password for existing user ${email}`);
+        userId = existingUser.id;
+        console.log(`  Found existing user ${email} (user: ${userId})`);
+      } else {
+        console.error(
+          `  User ${email} reported as registered but not found in listUsers`,
+        );
+        process.exit(1);
       }
     } else {
       console.error(`  Failed to create auth user: ${createError.message}`);
       process.exit(1);
     }
+  } else {
+    userId = createData!.user.id;
+    console.log(`  Created user ${email} (user: ${userId})`);
   }
 
-  // Sign in to get a valid session
-  // Use a separate client with the publishable/anon key for sign-in (if available)
-  const signInClient =
-    anonKey && anonKey !== secretKey
-      ? createClient(supabaseUrl!, anonKey, {
-          auth: { autoRefreshToken: false, persistSession: false },
-        })
-      : supabase;
-
-  const { data: signInData, error: signInError } =
-    await signInClient.auth.signInWithPassword({ email, password });
-
-  if (signInError || !signInData.session) {
-    console.error(
-      `  Failed to sign in as ${email}: ${signInError?.message || 'no session returned'}`,
-    );
-    process.exit(1);
-  }
-
-  const userId = signInData.user.id;
-  console.log(`  Signed in as ${email} (user: ${userId})`);
+  // Build synthetic JWT and mock responses — no real sign-in needed.
+  // The preload module intercepts server-side getUser() calls so Next.js
+  // middleware returns the mock user without hitting real Supabase.
+  const fakeJwt = buildFakeJwt(userId!, email);
+  const userResponse = buildUserResponse({ id: userId!, email });
 
-  // Write session cookies for the proxy to inject
   const projectRef = getProjectRef();
   const sessionOutput = {
     cookies: [
       {
         name: `sb-${projectRef}-auth-token`,
         value: JSON.stringify({
-          access_token: signInData.session.access_token,
-          refresh_token: signInData.session.refresh_token,
+          access_token: fakeJwt,
+          refresh_token: 'codeyam-mock-refresh-token',
           token_type: 'bearer',
-          expires_in: signInData.session.expires_in,
-          expires_at: signInData.session.expires_at,
+          expires_in: 315360000,
+          expires_at: 9999999999,
         }),
         path: '/',
         sameSite: 'Lax' as const,
       },
     ],
+    externalApis: {
+      [`${supabaseUrl}/auth/v1/user`]: {
+        body: userResponse,
+        status: 200,
+      },
+      [`${supabaseUrl}/auth/v1/token`]: {
+        body: {
+          access_token: fakeJwt,
+          token_type: 'bearer',
+          expires_in: 315360000,
+          expires_at: 9999999999,
+          refresh_token: 'codeyam-mock-refresh-token',
+          user: userResponse,
+        },
+        status: 200,
+      },
+    },
   };
 
   const outputDir = path.join(process.cwd(), '.codeyam', 'tmp');
   fs.mkdirSync(outputDir, { recursive: true });
   const outputPath = path.join(outputDir, 'seed-session.json');
   fs.writeFileSync(outputPath, JSON.stringify(sessionOutput, null, 2));
-  console.log(`  Session cookies written to ${outputPath}`);
+  console.log(`  Session cookies + auth mocks written to ${outputPath}`);
 
-  return userId;
+  return userId!;
 }
 
 /**
@@ -266,7 +452,7 @@ async function main() {
   // in seed data (e.g. for user_id foreign key columns with Supabase RLS)
   if (auth) {
     const userId = await handleAuth(
-      auth as { email: string; password: string },
+      auth as { email: string; password?: string },
     );
     seed = replaceAuthPlaceholders(seed, userId);
   }
@@ -276,7 +462,14 @@ async function main() {
   console.log('Seed complete');
 }
 
-main().catch((e) => {
-  console.error('Seed adapter error:', e);
-  process.exit(1);
-});
+if (process.argv[2] === '--export') {
+  exportData(process.argv[3]).catch((e) => {
+    console.error('Seed adapter export error:', e);
+    process.exit(1);
+  });
+} else {
+  main().catch((e) => {
+    console.error('Seed adapter error:', e);
+    process.exit(1);
+  });
+}

package/codeyam-cli/templates/skills/codeyam-editor/SKILL.md

@@ -16,6 +16,15 @@ You MUST follow a step-by-step workflow driven by `codeyam editor` commands. Eac
 
 **The rule:** After completing what a command tells you to do, run the NEXT command it specifies. The commands are your instructions — follow them one at a time.
 
+## TASK Directive
+
+Some `codeyam editor N` commands output a `━━━ TASK ━━━` section with an `EXACT_TASK_TITLE:` line. When you see one:
+
+1. Delete ALL existing tasks: run TaskList, then run TaskUpdate with status "deleted" for every task returned
+2. Run TaskCreate with the EXACT title from the `EXACT_TASK_TITLE:` line — copy it character-for-character, do NOT paraphrase
+
+This ensures only one task is ever visible. Do NOT create tasks on your own. Only create them when a `━━━ TASK ━━━` directive tells you to.
+
 ## Migration Mode
 
 When `codeyam editor steps` shows **"Project Migration"** instead of "Feature Cycle", the project is being migrated from an existing codebase. Follow the `codeyam editor migrate` commands instead of the normal 13-step cycle.
@@ -117,6 +126,9 @@ This command gives you the post-change checklist (re-register scenarios, re-run
 ## Key Rules
 
 - **Run the commands** — they ARE your instructions, not suggestions
+- **One step at a time** — run each `codeyam editor N` command, read its FULL output, complete every checklist item, then advance. The CLI enforces a minimum time per step.
+- **NEVER batch-run steps** — `for step in 5 6 7 8; do codeyam editor $step; done` or piping to `head` defeats the entire workflow. Each step has unique instructions you must read and follow.
+- **NEVER delegate multiple steps to a subagent** — each step must be run, read, and completed in the main conversation. You MAY use subagents for parallelizable work _within_ a single step (e.g. extracting components + writing tests in step 5).
 - **Every feature gets scenarios** — this is the core value of CodeYam
 - **Always scaffold with a database** (Prisma + SQLite)
 - **Build real API routes** — the proxy handles scenario data transparently
@@ -163,6 +175,21 @@ Isolation routes are committed to git (not gitignored). They are protected by a
 
 When a collaborator clones the repo and runs `codeyam editor`, scenarios are auto-imported from `scenarios-manifest.json`. Run `codeyam editor sync` to manually re-sync after pulling new changes.
 
+## Expo / React Native Projects
+
+When working with an Expo project (tech stack `expo-react-native`):
+
+- Use `<View>`, `<Text>`, `<ScrollView>`, `<Pressable>` from `react-native` — **never use HTML elements** (`<div>`, `<span>`, `<h1>`, etc.)
+- Do NOT use `'use client'` — this is a Next.js directive that has no meaning in Expo
+- Do NOT import from `next/navigation`, `next/router`, or any `next/*` package
+- Use `expo-router` for navigation: `useRouter()`, `useLocalSearchParams()`, `<Link>`
+- Use `@/lib/theme` for all design tokens — not CSS custom properties (`var(--token)` does not work in React Native)
+- AsyncStorage maps to localStorage on web — CodeYam's `localStorage` injection in scenarios works automatically
+- Test with `npx jest`, not `npx vitest`
+- Isolation routes use Expo Router: `app/isolated-components/ComponentName.tsx` (not a `/page.tsx` subdirectory)
+- Capture wrappers use `nativeID="codeyam-capture"` (not `id="codeyam-capture"`) — `nativeID` maps to `id` on web
+- The preview renders via react-native-web in a browser — some visual differences from native devices are expected (fonts, SafeAreaView, shadows). See `MOBILE_SETUP.md` for details.
+
 ## Quick Reference
 
 ```bash
@@ -176,7 +203,13 @@ codeyam editor register '{"name":"Full Catalog","type":"application","url":"/","
 # Register app scenario with localStorage (for apps using client-side storage instead of a database)
 codeyam editor register '{"name":"Full Library","type":"application","url":"/","dimensions":["<name from screenSizes>"],"localStorage":{"articles":[...],"collections":[...]}}'
 
-# For large seed/localStorage data, write JSON to a temp file and use @ prefix:
+# BULK REGISTRATION (preferred — register all scenarios at once):
+# Write an array of scenarios to a temp file, then register with @ prefix.
+# This is faster and avoids repeated screenshot capture overhead.
+# File format: [{"name":"...","type":"...","url":"...","seed":{...}}, ...]
+codeyam editor register @.codeyam/tmp/scenarios.json
+
+# Single scenario from file (for large seed/localStorage data):
 codeyam editor register @/tmp/scenario-data.json
 
 # Journal entry (one per feature, references scenario names)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@codeyam/codeyam-cli",
-  "version": "0.1.0-staging.a77070e",
+  "version": "0.1.0-staging.aa28063",
   "description": "Local development CLI for CodeYam analysis",
   "type": "module",
   "bin": {

package/packages/ai/src/lib/astScopes/methodSemantics.js

@@ -616,6 +616,97 @@ export class UseStateSemantics {
         return true;
     }
 }
+// ─── Set/Map method semantics ──────────────────────────────────────────
+//
+// Set/Map membership operations (.has, .delete, .add, .get, .set, .clear)
+// don't create data flow equivalencies — they're membership checks or
+// mutations, not transformations that propagate schema structure.
+// Without these semantics, code like `filterRatings.has(articleId)` creates
+// spurious equivalencies (332 entries in the Margo LibraryPage case).
+/**
+ * Set.has() / Map.has() — membership check, returns boolean
+ */
+export class CollectionHasSemantics {
+    getReturnType() {
+        return 'boolean';
+    }
+    addEquivalences(methodCallPath, _sourcePath, context) {
+        context.addType(methodCallPath, 'boolean');
+    }
+    isComplete() {
+        return true;
+    }
+}
+/**
+ * Set.delete() / Map.delete() — removal, returns boolean
+ */
+export class CollectionDeleteSemantics {
+    getReturnType() {
+        return 'boolean';
+    }
+    addEquivalences(methodCallPath, _sourcePath, context) {
+        context.addType(methodCallPath, 'boolean');
+    }
+    isComplete() {
+        return true;
+    }
+}
+/**
+ * Set.add() — adds an element, returns the Set (for chaining)
+ */
+export class SetAddSemantics {
+    getReturnType() {
+        return 'object'; // Returns the Set itself
+    }
+    addEquivalences(_methodCallPath, _sourcePath, _context) {
+        // No equivalencies — add() doesn't create data flow
+    }
+    isComplete() {
+        return true;
+    }
+}
+/**
+ * Set.clear() / Map.clear() — removes all elements, returns void
+ */
+export class CollectionClearSemantics {
+    getReturnType() {
+        return 'void';
+    }
+    addEquivalences() {
+        // No equivalencies
+    }
+    isComplete() {
+        return true;
+    }
+}
+/**
+ * Map.get() — retrieves a value by key, returns unknown (value type)
+ */
+export class MapGetSemantics {
+    getReturnType() {
+        return 'unknown';
+    }
+    addEquivalences(_methodCallPath, _sourcePath, _context) {
+        // No equivalencies — the return value type depends on the map contents
+    }
+    isComplete() {
+        return true;
+    }
+}
+/**
+ * Map.set() — sets a key-value pair, returns the Map (for chaining)
+ */
+export class MapSetSemantics {
+    getReturnType() {
+        return 'object'; // Returns the Map itself
+    }
+    addEquivalences() {
+        // No equivalencies
+    }
+    isComplete() {
+        return true;
+    }
+}
 /**
  * Create and populate the registry with method semantics
  */
@@ -645,6 +736,14 @@ export function createMethodRegistry() {
     // Register React hooks
     registry.register('useState', new UseStateSemantics(), 'React');
     registry.register('useMemo', new UseMemoSemantics(), 'React');
+    // Register Set/Map collection methods.
+    // Only .has() and .delete() are safe to register by method name — they always
+    // return boolean regardless of the collection type (Set, Map, WeakSet, WeakMap,
+    // FormData, URLSearchParams, etc.).
+    // .add(), .clear(), .get(), .set() are too ambiguous across different types
+    // and can't be registered until the lookup code supports class-aware resolution.
+    registry.register('has', new CollectionHasSemantics());
+    registry.register('delete', new CollectionDeleteSemantics());
     return registry;
 }
 // Create a singleton instance of the registry for use throughout the codebase