From 356bc76a0fc41ea518bde1e9d7b552a03139479b Mon Sep 17 00:00:00 2001
From: jlin53882 <jlin53882@gmail.com>
Date: Fri, 17 Apr 2026 01:24:38 +0800
Subject: [PATCH] fix(extraction): change cases abstract to descriptive format
 (Issue #640)

- Changed abstract from imperative to descriptive format
- Old: 'LanceDB BigInt error -> Use Number() coercion before arithmetic'
- New: 'LanceDB BigInt numeric handling issue'
- Added unit test to verify prompt format detection
---
 src/extraction-prompts.ts             |  2 +-
 test/issue-640-bigint-prompt.test.mjs | 68 +++++++++++++++++++++++++++
 2 files changed, 69 insertions(+), 1 deletion(-)
 create mode 100644 test/issue-640-bigint-prompt.test.mjs

diff --git a/src/extraction-prompts.ts b/src/extraction-prompts.ts
index 6fe16180..520a87a2 100644
--- a/src/extraction-prompts.ts
+++ b/src/extraction-prompts.ts
@@ -102,7 +102,7 @@ Each memory contains three levels:
 \`\`\`json
 {
   "category": "cases",
-  "abstract": "LanceDB BigInt error -> Use Number() coercion before arithmetic",
+  "abstract": "LanceDB BigInt numeric handling issue",
   "overview": "## Problem\\nLanceDB 0.26+ returns BigInt for numeric columns\\n\\n## Solution\\nCoerce values with Number(...) before arithmetic",
   "content": "When LanceDB returns BigInt values, wrap them with Number() before doing arithmetic operations."
 }
diff --git a/test/issue-640-bigint-prompt.test.mjs b/test/issue-640-bigint-prompt.test.mjs
new file mode 100644
index 00000000..b3144d2a
--- /dev/null
+++ b/test/issue-640-bigint-prompt.test.mjs
@@ -0,0 +1,68 @@
+/**
+ * Issue #640 Test: cases category prompt should be descriptive, not imperative
+ * 
+ * Test verifies that the abstract format change prevents LLM from skipping
+ * [cases] category memories.
+ * 
+ * Run: npx tsx test/issue-640-bigint-prompt.test.mjs
+ */
+
+// Helper to check if prompt is misleading
+function isPromptMisleading(abstract) {
+  const misleadingPatterns = [
+    "-> use",
+    "error ->",
+    "solution:",
+    "use number()",
+    "coercion",
+    "before arithmetic",
+  ];
+  const lower = abstract.toLowerCase();
+  for (const pattern of misleadingPatterns) {
+    if (lower.includes(pattern.toLowerCase())) {
+      return true;
+    }
+  }
+  return false;
+}
+
+// Test cases
+const testCases = [
+  {
+    abstract: "LanceDB BigInt error -> Use Number() coercion before arithmetic",
+    expectedMisleading: true,
+    description: "Old format (buggy) - should be detected as misleading",
+  },
+  {
+    abstract: "LanceDB BigInt numeric handling issue",
+    expectedMisleading: false,
+    description: "New format (fixed) - should NOT be misleading",
+  },
+];
+
+console.log("=== Issue #640: BigInt Prompt Format Test ===\n");
+
+let passed = 0;
+let failed = 0;
+
+for (const tc of testCases) {
+  const isMisleading = isPromptMisleading(tc.abstract);
+  const ok = isMisleading === tc.expectedMisleading;
+  
+  console.log(`[${tc.description}]`);
+  console.log(`  Abstract: "${tc.abstract}"`);
+  console.log(`  Misleading: ${isMisleading} (expected: ${tc.expectedMisleading})`);
+  console.log(`  Result: ${ok ? "✅ PASS" : "❌ FAIL"}`);
+  console.log("");
+  
+  if (ok) passed++;
+  else failed++;
+}
+
+console.log("----------------------------------------");
+console.log(`Total: ${passed} passed, ${failed} failed`);
+console.log("----------------------------------------");
+
+if (failed > 0) {
+  process.exit(1);
+}
\ No newline at end of file