feat: auto name lists for ai mode

[RohitR311]

What this PR does?

Auto names the list data captured for the AI mode.

Summary by CodeRabbit

• New Features
  • Workflow lists now automatically receive AI-generated contextual names based on detected data fields, improving organization and readability instead of generic default naming.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

Walkthrough

A new private static method generateListName is added to the WorkflowEnricher class that generates dynamic list names using LLM providers (Anthropic, OpenAI, or Ollama). The method integrates into buildWorkflowFromLLMDecision to replace hardcoded "List 1" naming when handling captureList decisions, with validation, normalization, and fallback error handling.

Changes

Cohort / File(s)	Summary
LLM-based list naming feature server/src/sdk/workflowEnricher.ts	Added private static method generateListName that selects an LLM provider, constructs prompts with field context, calls the appropriate API, validates and normalizes responses (trim, strip quotes, title case, max 50 chars). Integrated into buildWorkflowFromLLMDecision to dynamically name scrapeList actions instead of using "List 1". Includes comprehensive logging and fallback error handling.

Sequence Diagram

sequenceDiagram
    participant WE as WorkflowEnricher
    participant GN as generateListName
    participant LLM as LLM Provider<br/>(Anthropic/OpenAI/Ollama)
    participant Val as Validation & Normalization

    WE->>GN: captureList decision detected<br/>(prompt, url, fieldNames, llmConfig)
    activate GN
    
    GN->>GN: Select LLM provider<br/>based on config
    
    GN->>GN: Construct system prompt<br/>with field context
    
    GN->>LLM: Call API with prompts
    activate LLM
    LLM-->>GN: Response (raw list name)
    deactivate LLM
    
    GN->>Val: Validate & normalize<br/>(trim, strip quotes, title case)
    activate Val
    Val->>Val: Check length ≤ 50 chars
    Val-->>GN: Normalized name
    deactivate Val
    
    alt Success
        GN-->>WE: Generated list name
    else Error/Validation fails
        GN-->>WE: Fallback "List 1"
    end
    deactivate GN
    
    WE->>WE: Use returned name for<br/>scrapeList action

Loading

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

Poem

🐰 A prompt goes forth with field names bright,
LLM whispers back a name just right,
Trimmed and titled, validated true,
No more "List 1"—now lists have names anew!
When the models stumble, we gracefully fall,
To "List 1" we turn, it catches them all. 🎩✨

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title 'feat: auto name lists for ai mode' directly and clearly describes the main change: automatic naming of lists in AI mode, which aligns with the implementation of generateListName integration.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch lname-ai

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coderabbitai]

Actionable comments posted: 0

🧹 Nitpick comments (2)

server/src/sdk/workflowEnricher.ts (2)

1347-1375: Consider adding timeout for external API calls.

The axios calls to OpenAI (and similarly Anthropic SDK calls) don't specify a timeout. In production, this could cause the request to hang indefinitely if the API is unresponsive. Other similar methods in this file also lack timeouts, so this may be a broader pattern to address.

🔎 Proposed fix: Add timeout to axios call

         const response = await axios.post(`${openaiBaseUrl}/chat/completions`, {
           model: openaiModel,
           messages: [
             {
               role: 'system',
               content: systemPrompt
             },
             {
               role: 'user',
               content: userPrompt
             }
           ],
           max_tokens: 20,
           temperature: 0.1
         }, {
           headers: {
             'Authorization': `Bearer ${llmConfig?.apiKey || process.env.OPENAI_API_KEY}`,
             'Content-Type': 'application/json'
-          }
+          },
+          timeout: 30000 // 30 second timeout
         });

---

1392-1394: Title Case normalization may break acronyms.

The current normalization lowercases all characters after the first in each word, which could turn "API Products" into "Api Products" or "USA Jobs" into "Usa Jobs". This is minor given the LLM is instructed to use Title Case, but worth noting.

🔎 Alternative: Preserve all-caps words

-      listName = listName.split(' ')
-        .map((word: string) => word.charAt(0).toUpperCase() + word.slice(1).toLowerCase())
-        .join(' ');
+      listName = listName.split(' ')
+        .map((word: string) => {
+          // Preserve all-caps words (likely acronyms)
+          if (word === word.toUpperCase() && word.length > 1) {
+            return word;
+          }
+          return word.charAt(0).toUpperCase() + word.slice(1).toLowerCase();
+        })
+        .join(' ');

📜 Review details

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 33bfbb6 and 1999672.

📒 Files selected for processing (1)

• server/src/sdk/workflowEnricher.ts

🧰 Additional context used

🧬 Code graph analysis (1)

server/src/sdk/workflowEnricher.ts (2)

server/src/sdk/browserSide/pageAnalyzer.js (1)

• finalFields (999-999)

server/src/routes/index.ts (1)

• workflow (10-10)

🔇 Additional comments (3)

server/src/sdk/workflowEnricher.ts (3)

1243-1263: Method signature and setup look good.

The method follows the established patterns in this class for LLM-based operations. Limiting field context to 10 items is a reasonable choice to avoid token limits while still providing useful context.

---

1265-1292: Well-crafted prompts for list naming.

The system prompt is clear and provides good examples for domain adaptation. Explicitly requesting "ONLY the list name, nothing else" helps ensure consistent output format across providers.

---

1498-1510: Clean integration of list name generation.

The placement after finalFields is determined ensures the list name reflects the actual fields being extracted. Using Object.keys(finalFields) provides relevant context to the LLM for generating meaningful names.