community[patch]: Better errors for neo4j vector (#5501)

* Better errors for neo4j vector

* Lint

---------

Co-authored-by: jacoblee93 <[email protected]>

Author: tomasonjo

libs/langchain-community/.eslintrc.cjs

@@ -60,6 +60,7 @@ module.exports = {
     "no-else-return": 0,
     "func-names": 0,
     "no-lonely-if": 0,
+    "prefer-template": 0,
     "prefer-rest-params": 0,
     "new-cap": ["error", { properties: false, capIsNew: false }],
     "arrow-body-style": 0,

libs/langchain-community/src/vectorstores/neo4j_vector.ts

@@ -720,6 +720,22 @@ export class Neo4jVectorStore extends VectorStore {
     const results = await this.query(readQuery, parameters);
 
     if (results) {
+      if (results.some((result) => result.text == null)) {
+        if (!this.retrievalQuery) {
+          throw new Error(
+            "Make sure that none of the '" +
+              this.textNodeProperty +
+              "' properties on nodes with label '" +
+              this.nodeLabel +
+              "' are missing or empty"
+          );
+        } else {
+          throw new Error(
+            "Inspect the 'retrievalQuery' and ensure it doesn't return null for the 'text' column"
+          );
+        }
+      }
+
       const docs: [Document, number][] = results.map((result: Any) => [
         new Document({
           pageContent: result.text,
@@ -1026,7 +1042,7 @@ function handleFieldFilter(
     return [querySnippet, queryParam];
   } else if (["$in", "$nin", "$like", "$ilike"].includes(operator)) {
     if (["$in", "$nin"].includes(operator)) {
-      filterValue.forEach((val: any) => {
+      filterValue.forEach((val: Any) => {
         if (
           typeof val !== "string" &&
           typeof val !== "number" &&