@tinacms/graphql 0.0.0-202251418421 → 0.0.0-202251420313

package/CHANGELOG.md

@@ -1,13 +1,14 @@
 # tina-graphql
 
-## 0.0.0-202251418421
+## 0.0.0-202251420313
 
 ### Patch Changes
 
 - f6cb634c2: Added an optional config key to the schema that will be used for tina cloud media store
+- 8b81c3cf3: Added more context to error messages to help to user debug issues
 - Updated dependencies [f6cb634c2]
-  - @tinacms/schema-tools@0.0.0-202251418421
-  - @tinacms/datalayer@0.0.0-202251418421
+  - @tinacms/schema-tools@0.0.0-202251420313
+  - @tinacms/datalayer@0.0.0-202251420313
 
 ## 0.60.6
 

package/dist/index.d.ts

@@ -12,6 +12,7 @@ limitations under the License.
 */
 export { indexDB } from './build';
 export { resolve } from './resolve';
+export * from './resolver/error';
 export { createDatabase } from './database';
 export type { QueryOptions } from './database';
 import type { Database } from './database';

package/dist/index.js

@@ -10384,9 +10384,14 @@ var require_remark_mdx = __commonJS({
 
 // pnp:/home/runner/work/tinacms/tinacms/packages/@tinacms/graphql/src/index.ts
 __export(exports, {
+  TinaFetchError: () => TinaFetchError,
+  TinaGraphQLError: () => TinaGraphQLError,
+  TinaParseDocumentError: () => TinaParseDocumentError,
+  TinaQueryError: () => TinaQueryError,
   assertShape: () => assertShape,
   buildSchema: () => buildSchema,
   createDatabase: () => createDatabase,
+  handleFetchErrorError: () => handleFetchErrorError,
   indexDB: () => indexDB,
   parseFile: () => parseFile,
   resolve: () => resolve,
@@ -12830,15 +12835,52 @@ var package_default = {
 };
 
 // pnp:/home/runner/work/tinacms/tinacms/packages/@tinacms/graphql/src/resolver/error.ts
-var TinaError = class extends Error {
+var TinaGraphQLError = class extends Error {
   constructor(message, extensions) {
     super(message);
     if (!this.name) {
-      Object.defineProperty(this, "name", { value: "TinaError" });
+      Object.defineProperty(this, "name", { value: "TinaGraphQLError" });
     }
     this.extensions = __spreadValues({}, extensions);
   }
 };
+var TinaFetchError = class extends Error {
+  constructor(message, args) {
+    super(message);
+    this.name = "TinaFetchError";
+    this.collection = args.collection;
+    this.stack = args.stack;
+    this.file = args.file;
+    this.originalError = args.originalError;
+  }
+};
+var TinaQueryError = class extends TinaFetchError {
+  constructor(args) {
+    super(`Error querying file ${args.file} collection ${args.collection}. Please run "tinacms audit" or add the --verbose option for more info`, args);
+  }
+};
+var TinaParseDocumentError = class extends TinaFetchError {
+  constructor(args) {
+    super(`Error Parsing file ${args.file} collection ${args.collection}. Please run "tinacms audit" or add the --verbose option  for more info`, args);
+  }
+  toString() {
+    return super.toString() + "\n OriginalError: \n" + this.originalError.toString();
+  }
+};
+var handleFetchErrorError = (e, verbose) => {
+  if (e instanceof Error) {
+    if (e instanceof TinaFetchError) {
+      if (verbose) {
+        console.log(e.toString());
+        console.log(e);
+        console.log(e.stack);
+      }
+    }
+  } else {
+    console.error(e);
+  }
+  throw e;
+};
 
 // pnp:/home/runner/work/tinacms/tinacms/packages/@tinacms/graphql/src/schema/index.ts
 var createSchema = async ({
@@ -12951,7 +12993,7 @@ var TinaSchema = class {
           assertShape(data, (yup3) => yup3.object({ _template: yup3.string().required() }));
           const template = templateInfo.templates.find((template2) => template2.namespace[template2.namespace.length - 1] === data._template);
           if (!template) {
-            throw new TinaError(`Expected to find template named '${data._template}' for collection '${lastItem(collection.namespace)}'`, {
+            throw new TinaGraphQLError(`Expected to find template named '${data._template}' for collection '${lastItem(collection.namespace)}'`, {
               collection: lastItem(collection.namespace),
               possibleTemplates: templateInfo.templates.map((template2) => lastItem(template2.namespace)),
               data
@@ -22775,7 +22817,18 @@ var Resolver = class {
           _collection: rawData._collection,
           _template: rawData._template
         };
-        await sequential(template.fields, async (field) => this.resolveFieldData(field, rawData, data));
+        try {
+          await sequential(template.fields, async (field) => {
+            return this.resolveFieldData(field, rawData, data);
+          });
+        } catch (e) {
+          throw new TinaParseDocumentError({
+            originalError: e,
+            collection: collection.name,
+            file: relativePath,
+            stack: e.stack
+          });
+        }
         const titleField = template.fields.find((x) => {
           if (x.type === "string" && (x == null ? void 0 : x.isTitle)) {
             return true;
@@ -22801,8 +22854,8 @@ var Resolver = class {
           _values: data
         });
       } catch (e) {
-        if (e instanceof TinaError) {
-          throw new TinaError(e.message, __spreadValues({
+        if (e instanceof TinaGraphQLError) {
+          throw new TinaGraphQLError(e.message, __spreadValues({
             requestedDocument: fullPath
           }, e.extensions));
         }
@@ -23243,10 +23296,12 @@ var resolve = async ({
   query,
   variables,
   database,
-  silenceErrors
+  silenceErrors,
+  verbose
 }) => {
   var _a;
   try {
+    const verboseValue = verbose != null ? verbose : true;
     const graphQLSchemaAst = await database.getGraphQLSchema();
     const graphQLSchema = (0, import_graphql3.buildASTSchema)(graphQLSchemaAst);
     const tinaConfig = await database.getTinaSchema();
@@ -23274,141 +23329,152 @@ var resolve = async ({
         }
       },
       fieldResolver: async (source = {}, _args = {}, _context, info) => {
-        const args = JSON.parse(JSON.stringify(_args));
-        const returnType = (0, import_graphql3.getNamedType)(info.returnType).toString();
-        const lookup = await database.getLookup(returnType);
-        const isMutation = info.parentType.toString() === "Mutation";
-        const value = source[info.fieldName];
-        if (returnType === "Collection") {
-          if (value) {
-            return value;
-          }
-          if (info.fieldName === "collections") {
-            const collectionNode2 = info.fieldNodes.find((x) => x.name.value === "collections");
-            const hasDocuments2 = collectionNode2.selectionSet.selections.find((x) => {
-              var _a2;
-              return ((_a2 = x == null ? void 0 : x.name) == null ? void 0 : _a2.value) === "documents";
-            });
-            return tinaSchema.getCollections().map((collection) => {
-              return resolver2.resolveCollection(args, collection.name, Boolean(hasDocuments2));
-            });
-          }
-          const collectionNode = info.fieldNodes.find((x) => x.name.value === "collection");
-          const hasDocuments = collectionNode.selectionSet.selections.find((x) => {
-            var _a2;
-            return ((_a2 = x == null ? void 0 : x.name) == null ? void 0 : _a2.value) === "documents";
-          });
-          return resolver2.resolveCollection(args, args.collection, Boolean(hasDocuments));
-        }
-        if (info.fieldName === "getOptimizedQuery") {
-          try {
-            const [optimizedQuery] = (0, import_relay_operation_optimizer.optimizeDocuments)(info.schema, [(0, import_graphql3.parse)(args.queryString)], {
-              assumeValid: true,
-              includeFragments: false,
-              noLocation: true
-            });
-            return (0, import_graphql3.print)(optimizedQuery);
-          } catch (e) {
-            throw new Error(`Invalid query provided, Error message: ${e.message}`);
-          }
-        }
-        if (!lookup) {
-          return value;
-        }
-        const isCreation = lookup[info.fieldName] === "create";
-        switch (lookup.resolveType) {
-          case "nodeDocument":
-            assertShape(args, (yup3) => yup3.object({ id: yup3.string().required() }));
-            return resolver2.getDocument(args.id);
-          case "multiCollectionDocument":
-            if (typeof value === "string") {
-              return resolver2.getDocument(value);
+        try {
+          const args = JSON.parse(JSON.stringify(_args));
+          const returnType = (0, import_graphql3.getNamedType)(info.returnType).toString();
+          const lookup = await database.getLookup(returnType);
+          const isMutation = info.parentType.toString() === "Mutation";
+          const value = source[info.fieldName];
+          if (returnType === "Collection") {
+            if (value) {
+              return value;
             }
-            if (args && args.collection && info.fieldName === "addPendingDocument") {
-              return resolver2.resolveDocument({
-                args: __spreadProps(__spreadValues({}, args), { params: {} }),
-                collection: args.collection,
-                isMutation,
-                isCreation: true,
-                isAddPendingDocument: true
+            if (info.fieldName === "collections") {
+              const collectionNode2 = info.fieldNodes.find((x) => x.name.value === "collections");
+              const hasDocuments2 = collectionNode2.selectionSet.selections.find((x) => {
+                var _a2;
+                return ((_a2 = x == null ? void 0 : x.name) == null ? void 0 : _a2.value) === "documents";
               });
-            }
-            if ([
-              NAMER.documentQueryName(),
-              "createDocument",
-              "updateDocument",
-              "deleteDocument"
-            ].includes(info.fieldName)) {
-              const result2 = await resolver2.resolveDocument({
-                args,
-                collection: args.collection,
-                isMutation,
-                isCreation,
-                isDeletion: info.fieldName === "deleteDocument",
-                isAddPendingDocument: false,
-                isCollectionSpecific: false
+              return tinaSchema.getCollections().map((collection) => {
+                return resolver2.resolveCollection(args, collection.name, Boolean(hasDocuments2));
               });
-              return result2;
             }
-            return value;
-          case "multiCollectionDocumentList":
-            if (Array.isArray(value)) {
-              return {
-                totalCount: value.length,
-                edges: value.map((document3) => {
-                  return { node: document3 };
-                })
-              };
-            } else if (info.fieldName === "documents" && (value == null ? void 0 : value.collection) && (value == null ? void 0 : value.hasDocuments)) {
-              return resolver2.resolveCollectionConnection({
-                args,
-                collection: value.collection
+            const collectionNode = info.fieldNodes.find((x) => x.name.value === "collection");
+            const hasDocuments = collectionNode.selectionSet.selections.find((x) => {
+              var _a2;
+              return ((_a2 = x == null ? void 0 : x.name) == null ? void 0 : _a2.value) === "documents";
+            });
+            return resolver2.resolveCollection(args, args.collection, Boolean(hasDocuments));
+          }
+          if (info.fieldName === "getOptimizedQuery") {
+            try {
+              const [optimizedQuery] = (0, import_relay_operation_optimizer.optimizeDocuments)(info.schema, [(0, import_graphql3.parse)(args.queryString)], {
+                assumeValid: true,
+                includeFragments: false,
+                noLocation: true
               });
-            } else {
-              throw new Error(`Expected an array for result of ${info.fieldName} at ${info.path}`);
+              return (0, import_graphql3.print)(optimizedQuery);
+            } catch (e) {
+              throw new Error(`Invalid query provided, Error message: ${e.message}`);
             }
-          case "collectionDocument":
-            if (value) {
-              return value;
-            }
-            const result = value || await resolver2.resolveDocument({
-              args,
-              collection: lookup.collection,
-              isMutation,
-              isCreation,
-              isAddPendingDocument: false,
-              isCollectionSpecific: true
-            });
-            return result;
-          case "collectionDocumentList":
-            return resolver2.resolveCollectionConnection({
-              args,
-              collection: tinaSchema.getCollection(lookup.collection)
-            });
-          case "unionData":
-            if (!value) {
-              if (args.relativePath) {
+          }
+          if (!lookup) {
+            return value;
+          }
+          const isCreation = lookup[info.fieldName] === "create";
+          switch (lookup.resolveType) {
+            case "nodeDocument":
+              assertShape(args, (yup3) => yup3.object({ id: yup3.string().required() }));
+              return resolver2.getDocument(args.id);
+            case "multiCollectionDocument":
+              if (typeof value === "string") {
+                return resolver2.getDocument(value);
+              }
+              if (args && args.collection && info.fieldName === "addPendingDocument") {
+                return resolver2.resolveDocument({
+                  args: __spreadProps(__spreadValues({}, args), { params: {} }),
+                  collection: args.collection,
+                  isMutation,
+                  isCreation: true,
+                  isAddPendingDocument: true
+                });
+              }
+              if ([
+                NAMER.documentQueryName(),
+                "createDocument",
+                "updateDocument",
+                "deleteDocument"
+              ].includes(info.fieldName)) {
                 const result2 = await resolver2.resolveDocument({
                   args,
-                  collection: lookup.collection,
+                  collection: args.collection,
                   isMutation,
                   isCreation,
+                  isDeletion: info.fieldName === "deleteDocument",
                   isAddPendingDocument: false,
-                  isCollectionSpecific: true
+                  isCollectionSpecific: false
                 });
                 return result2;
               }
-            }
-            return value;
-          default:
-            console.error(lookup);
-            throw new Error(`Unexpected resolve type`);
+              return value;
+            case "multiCollectionDocumentList":
+              if (Array.isArray(value)) {
+                return {
+                  totalCount: value.length,
+                  edges: value.map((document3) => {
+                    return { node: document3 };
+                  })
+                };
+              } else if (info.fieldName === "documents" && (value == null ? void 0 : value.collection) && (value == null ? void 0 : value.hasDocuments)) {
+                return resolver2.resolveCollectionConnection({
+                  args,
+                  collection: value.collection
+                });
+              } else {
+                throw new Error(`Expected an array for result of ${info.fieldName} at ${info.path}`);
+              }
+            case "collectionDocument":
+              if (value) {
+                return value;
+              }
+              const result = value || await resolver2.resolveDocument({
+                args,
+                collection: lookup.collection,
+                isMutation,
+                isCreation,
+                isAddPendingDocument: false,
+                isCollectionSpecific: true
+              });
+              return result;
+            case "collectionDocumentList":
+              return resolver2.resolveCollectionConnection({
+                args,
+                collection: tinaSchema.getCollection(lookup.collection)
+              });
+            case "unionData":
+              if (!value) {
+                if (args.relativePath) {
+                  const result2 = await resolver2.resolveDocument({
+                    args,
+                    collection: lookup.collection,
+                    isMutation,
+                    isCreation,
+                    isAddPendingDocument: false,
+                    isCollectionSpecific: true
+                  });
+                  return result2;
+                }
+              }
+              return value;
+            default:
+              console.error(lookup);
+              throw new Error(`Unexpected resolve type`);
+          }
+        } catch (e) {
+          handleFetchErrorError(e, verboseValue);
         }
       }
     });
     if (res.errors) {
       if (!silenceErrors) {
-        console.error(res.errors);
+        res.errors.map((e) => {
+          console.error(e.toString());
+          if (verboseValue) {
+            console.error("More error context below");
+            console.error(e.message);
+            console.error(e);
+          }
+        });
       }
     }
     return res;
@@ -23561,25 +23627,34 @@ var Database = class {
       });
     };
     this.put = async (filepath, data, collection) => {
-      if (SYSTEM_FILES.includes(filepath)) {
-        throw new Error(`Unexpected put for config file ${filepath}`);
-      } else {
-        let collectionIndexDefinitions;
-        if (collection) {
-          const indexDefinitions = await this.getIndexDefinitions();
-          collectionIndexDefinitions = indexDefinitions == null ? void 0 : indexDefinitions[collection];
-        }
-        const { stringifiedFile, payload, keepTemplateKey } = await this.stringifyFile(filepath, data);
-        if (this.store.supportsSeeding()) {
-          await this.bridge.put(normalizePath(filepath), stringifiedFile);
+      try {
+        if (SYSTEM_FILES.includes(filepath)) {
+          throw new Error(`Unexpected put for config file ${filepath}`);
+        } else {
+          let collectionIndexDefinitions;
+          if (collection) {
+            const indexDefinitions = await this.getIndexDefinitions();
+            collectionIndexDefinitions = indexDefinitions == null ? void 0 : indexDefinitions[collection];
+          }
+          const { stringifiedFile, payload, keepTemplateKey } = await this.stringifyFile(filepath, data);
+          if (this.store.supportsSeeding()) {
+            await this.bridge.put(normalizePath(filepath), stringifiedFile);
+          }
+          await this.store.put(normalizePath(filepath), payload, {
+            keepTemplateKey,
+            collection,
+            indexDefinitions: collectionIndexDefinitions
+          });
         }
-        await this.store.put(normalizePath(filepath), payload, {
-          keepTemplateKey,
+        return true;
+      } catch (error) {
+        throw new TinaFetchError(`Error in PUT for ${filepath}`, {
+          originalError: error,
+          file: filepath,
           collection,
-          indexDefinitions: collectionIndexDefinitions
+          stack: error.stack
         });
       }
-      return true;
     };
     this.stringifyFile = async (filepath, data) => {
       if (SYSTEM_FILES.includes(filepath)) {
@@ -23755,11 +23830,24 @@ var Database = class {
       } = await this.store.query(storeQueryOptions);
       return {
         edges: await sequential(edges, async (edge) => {
-          const node = await hydrator(edge.path);
-          return {
-            node,
-            cursor: (0, import_datalayer2.btoa)(edge.cursor)
-          };
+          try {
+            const node = await hydrator(edge.path);
+            return {
+              node,
+              cursor: (0, import_datalayer2.btoa)(edge.cursor)
+            };
+          } catch (error) {
+            if (error instanceof Error) {
+              throw new TinaQueryError({
+                originalError: error,
+                file: edge.path,
+                collection,
+                stack: error.stack
+              });
+            } else {
+              throw error;
+            }
+          }
         }),
         pageInfo: {
           hasPreviousPage,
@@ -23906,10 +23994,19 @@ var _indexContent = async (database, documentPaths, collection) => {
     };
   }
   await sequential(documentPaths, async (filepath) => {
-    const dataString = await database.bridge.get(normalizePath(filepath));
-    const data = parseFile(dataString, import_path4.default.extname(filepath), (yup3) => yup3.object({}));
-    if (database.store.supportsSeeding()) {
-      await database.store.seed(normalizePath(filepath), data, seedOptions);
+    try {
+      const dataString = await database.bridge.get(normalizePath(filepath));
+      const data = parseFile(dataString, import_path4.default.extname(filepath), (yup3) => yup3.object({}));
+      if (database.store.supportsSeeding()) {
+        await database.store.seed(normalizePath(filepath), data, seedOptions);
+      }
+    } catch (error) {
+      throw new TinaFetchError(`Unable to seed ${filepath}`, {
+        originalError: error,
+        file: filepath,
+        collection: collection.name,
+        stack: error.stack
+      });
     }
   });
 };
@@ -23942,9 +24039,14 @@ var buildSchema = async (rootPath, database, flags) => {
 };
 // Annotate the CommonJS export names for ESM import in node:
 0 && (module.exports = {
+  TinaFetchError,
+  TinaGraphQLError,
+  TinaParseDocumentError,
+  TinaQueryError,
   assertShape,
   buildSchema,
   createDatabase,
+  handleFetchErrorError,
   indexDB,
   parseFile,
   resolve,

package/dist/resolve.d.ts

@@ -12,12 +12,13 @@ limitations under the License.
 */
 import type { Database } from './database';
 import type { GraphQLConfig } from './types';
-export declare const resolve: ({ config, query, variables, database, silenceErrors, }: {
+export declare const resolve: ({ config, query, variables, database, silenceErrors, verbose, }: {
     config?: GraphQLConfig;
     query: string;
     variables: object;
     database: Database;
     silenceErrors?: boolean;
+    verbose?: boolean;
 }) => Promise<import("graphql").ExecutionResult<{
     [key: string]: any;
 }, {

package/dist/resolver/error.d.ts

@@ -11,7 +11,7 @@ See the License for the specific language governing permissions and
 limitations under the License.
 */
 import { ASTNode, GraphQLError, Source, SourceLocation } from 'graphql';
-export declare class TinaError extends Error implements GraphQLError {
+export declare class TinaGraphQLError extends Error implements GraphQLError {
     extensions: Record<string, any>;
     readonly name: string;
     readonly locations: ReadonlyArray<SourceLocation> | undefined;
@@ -23,3 +23,32 @@ export declare class TinaError extends Error implements GraphQLError {
     [key: string]: any;
     constructor(message: string, extensions?: Record<string, any>);
 }
+export declare type TypeFetchErrorArgs = {
+    stack?: string;
+    file?: string;
+    originalError: Error;
+    collection?: string;
+};
+export declare class TinaFetchError extends Error {
+    stack?: string;
+    collection?: string;
+    file?: string;
+    originalError: Error;
+    constructor(message: string, args: TypeFetchErrorArgs);
+}
+export declare class TinaQueryError extends TinaFetchError {
+    stack?: string;
+    collection?: string;
+    file?: string;
+    originalError: Error;
+    constructor(args: TypeFetchErrorArgs);
+}
+export declare class TinaParseDocumentError extends TinaFetchError {
+    stack?: string;
+    collection?: string;
+    file?: string;
+    originalError: Error;
+    constructor(args: TypeFetchErrorArgs);
+    toString(): string;
+}
+export declare const handleFetchErrorError: (e: unknown, verbose: any) => never;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tinacms/graphql",
-  "version": "0.0.0-202251418421",
+  "version": "0.0.0-202251420313",
   "main": "dist/index.js",
   "typings": "dist/index.d.ts",
   "files": [
@@ -47,7 +47,7 @@
   },
   "dependencies": {
     "@graphql-tools/relay-operation-optimizer": "^6.4.1",
-    "@tinacms/datalayer": "0.0.0-202251418421",
+    "@tinacms/datalayer": "0.0.0-202251420313",
     "body-parser": "^1.19.0",
     "cors": "^2.8.5",
     "dataloader": "^2.0.0",
@@ -97,9 +97,9 @@
     "directory": "packages/tina-graphql"
   },
   "devDependencies": {
-    "@tinacms/datalayer": "0.0.0-202251418421",
-    "@tinacms/schema-tools": "0.0.0-202251418421",
-    "@tinacms/scripts": "0.0.0-202251418421",
+    "@tinacms/datalayer": "0.0.0-202251420313",
+    "@tinacms/schema-tools": "0.0.0-202251420313",
+    "@tinacms/scripts": "0.0.0-202251420313",
     "@types/cors": "^2.8.7",
     "@types/estree": "^0.0.50",
     "@types/express": "^4.17.8",