@tinacms/graphql 0.59.6 → 0.59.7

package/CHANGELOG.md

@@ -1,5 +1,12 @@
 # tina-graphql
 
+## 0.59.7
+
+### Patch Changes
+
+- c730fa1dd: fix: #1452: update indexDocument to handle adding new docs
+- cd0f6f022: Do not resolve all documents in `getCollection` if it is not needed
+
 ## 0.59.6
 
 ### Patch Changes

package/dist/index.js

@@ -12370,7 +12370,7 @@ var validateField = async (field) => {
 
 // pnp:/home/runner/work/tinacms/tinacms/packages/@tinacms/graphql/package.json
 var name = "@tinacms/graphql";
-var version = "0.59.6";
+var version = "0.59.7";
 var main = "dist/index.js";
 var typings = "dist/index.d.ts";
 var files = [
@@ -21995,7 +21995,7 @@ var stringify = (node, field) => {
         spread: false,
         check: null,
         children: [
-          stringify(p, field),
+          ...stringifyChildren([p], field),
           ...stringifyChildren(extraChildren, field)
         ]
       };
@@ -22019,12 +22019,13 @@ var stringify = (node, field) => {
       return {
         type: "link",
         url: node.url,
-        children: node.children.map((child) => stringify(child, field))
+        title: node.title,
+        children: stringifyChildren(node.children, field)
       };
     case plateElements.ELEMENT_BLOCKQUOTE:
       return {
         type: "blockquote",
-        children: node.children.map((child) => stringify(child, field))
+        children: stringifyChildren(node.children, field)
       };
     case "mdxJsxTextElement":
     case "mdxJsxFlowElement":
@@ -22189,12 +22190,7 @@ var stringify = (node, field) => {
                 }
                 break;
               case "rich-text":
-                const tree = value.children.map((item) => stringify(item, field2)).filter((item) => {
-                  if (item.type === "text" && !item.text) {
-                    return false;
-                  }
-                  return true;
-                });
+                const tree = stringifyChildren(value.children, field2);
                 if (field2.name === "children") {
                   children2 = tree;
                 } else {
@@ -22269,10 +22265,13 @@ var createResolver2 = (args) => {
 var Resolver = class {
   constructor(init) {
     this.init = init;
-    this.resolveCollection = async (collectionName) => {
+    this.resolveCollection = async (collectionName, hasDocuments) => {
       const collection = this.tinaSchema.getCollection(collectionName);
       const extraFields = {};
-      const documents = await this.getDocumentsForCollection(collectionName);
+      let documents = {};
+      if (hasDocuments) {
+        documents = await this.getDocumentsForCollection(collectionName);
+      }
       return __spreadValues(__spreadValues({
         documents
       }, collection), extraFields);
@@ -22948,11 +22947,21 @@ var resolve = async ({
             return value;
           }
           if (info.fieldName === "getCollections") {
+            const getCollectionNode2 = info.fieldNodes.find((x) => x.name.value === "getCollections");
+            const hasDocuments2 = getCollectionNode2.selectionSet.selections.find((x) => {
+              var _a;
+              return ((_a = x == null ? void 0 : x.name) == null ? void 0 : _a.value) === "documents";
+            });
             return tinaSchema.getCollections().map((collection) => {
-              return resolver2.resolveCollection(collection.name);
+              return resolver2.resolveCollection(collection.name, Boolean(hasDocuments2));
             });
           }
-          return resolver2.resolveCollection(args.collection);
+          const getCollectionNode = info.fieldNodes.find((x) => x.name.value === "getCollection");
+          const hasDocuments = getCollectionNode.selectionSet.selections.find((x) => {
+            var _a;
+            return ((_a = x == null ? void 0 : x.name) == null ? void 0 : _a.value) === "documents";
+          });
+          return resolver2.resolveCollection(args.collection, Boolean(hasDocuments));
         }
         if (info.fieldName === "getDocumentFields") {
           return resolver2.getDocumentFields();
@@ -23583,21 +23592,31 @@ var indexDocument = async ({
   data,
   database
 }) => {
+  var _a;
   const schema = await database.getSchema();
   const collection = await schema.getCollectionByFullPath(filepath);
-  const existingData = await database.get(filepath);
-  const attributesToFilterOut = await _indexCollectable({
-    record: filepath,
-    value: existingData,
-    field: collection,
-    prefix: collection.name,
-    database
-  });
-  await sequential(attributesToFilterOut, async (attribute) => {
-    const records = await database.store.get(attribute) || [];
-    await database.store.put(attribute, records.filter((item) => item !== filepath));
-    return true;
-  });
+  let existingData;
+  try {
+    existingData = await database.get(filepath);
+  } catch (err) {
+    if (((_a = err.extensions) == null ? void 0 : _a["status"]) !== 404) {
+      throw err;
+    }
+  }
+  if (existingData) {
+    const attributesToFilterOut = await _indexCollectable({
+      record: filepath,
+      value: existingData,
+      field: collection,
+      prefix: collection.name,
+      database
+    });
+    await sequential(attributesToFilterOut, async (attribute) => {
+      const records = await database.store.get(attribute) || [];
+      await database.store.put(attribute, records.filter((item) => item !== filepath));
+      return true;
+    });
+  }
   const attributes = await _indexCollectable({
     record: filepath,
     field: collection,

package/dist/resolver/index.d.ts

@@ -27,7 +27,7 @@ export declare class Resolver {
     database: Database;
     tinaSchema: TinaSchema;
     constructor(init: ResolverConfig);
-    resolveCollection: (collectionName: string) => Promise<{
+    resolveCollection: (collectionName: string, hasDocuments?: boolean) => Promise<{
         fields: string | import("../types").TinaFieldInner<true>[];
         templates?: undefined;
         references?: import("../types").ReferenceTypeWithNamespace[];
@@ -37,7 +37,7 @@ export declare class Resolver {
         path: string;
         format?: "json" | "md" | "markdown" | "mdx";
         match?: string;
-        documents: string[];
+        documents: {};
     } | {
         templates: (string | {
             label: string;
@@ -54,7 +54,7 @@ export declare class Resolver {
         path: string;
         format?: "json" | "md" | "markdown" | "mdx";
         match?: string;
-        documents: string[];
+        documents: {};
     }>;
     getDocument: (fullPath: unknown) => Promise<{
         __typename: string;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tinacms/graphql",
-  "version": "0.59.6",
+  "version": "0.59.7",
   "main": "dist/index.js",
   "typings": "dist/index.d.ts",
   "files": [
@@ -95,7 +95,7 @@
   },
   "devDependencies": {
     "@tinacms/datalayer": "0.0.2",
-    "@tinacms/scripts": "0.50.5",
+    "@tinacms/scripts": "0.50.6",
     "@types/cors": "^2.8.7",
     "@types/estree": "^0.0.50",
     "@types/express": "^4.17.8",