Merge pull request #10508 from quarto-dev/bugfix/issue-10504

ignore range string error when attempting to detect a circular structure

src/core/lib/yaml-intelligence/annotated-yaml.ts

@@ -268,7 +268,24 @@ export function buildJsYamlAnnotation(mappedYaml: MappedString) {
     );
   }
 
-  JSON.stringify(results[0]); // this is here so that we throw on circular structures
+  // console.log(results[0]);
+  try {
+    JSON.stringify(results[0]); // this is here so that we throw on circular structures
+  } catch (e) {
+    if (e.message.match("invalid string length")) {
+      // https://github.com/quarto-dev/quarto-cli/issues/10504
+      // It seems to be relatively easy to hit string length limits in
+      // JSON.stringify. Since this call is only here to check for circular
+      // structures, we chose to ignore this error, even though it's not
+      // ideal
+    } else if (e.message.match(/circular structure/)) {
+      throw new InternalError(
+        `Circular structure detected in parsed yaml: ${e.message}`,
+      );
+    } else {
+
+    }
+  }
   return postProcessAnnotation(results[0]);
 }