Fix no inspectable targets issue with Chrome Browser

news/changelog-1.3.md

@@ -200,6 +200,7 @@
 - Fix typo `thumnail-image` -> `thumbnail-image` in listing template ([#4602](//github.com/quarto-dev/quarto-cli/pull/4602)) (Thank you, @mattspence!).
 - Add support for targeting the `#refs` divs with citations when using `natbib` or `biblatex` to generate a bibliography.
 - Warn users about Chromium installation issues in WSL ([#4596](https://github.com/quarto-dev/quarto-cli/issues/4586)).
+- Fix issue with "No inspectable targets" with Chrome Browser ([#4653](https://github.com/quarto-dev/quarto-cli/issues/4653))
 - Add `title` attribute for callouts (can be used rather than heading for defining the title)
 
 ## Pandoc filter changes

src/core/cri/cri.ts

@@ -114,6 +114,7 @@ export async function criClient(appPath?: string, port?: number) {
       for (let i = 0; i < maxTries; ++i) {
         try {
           client = await cdp({ port });
+          break;
         } catch (e) {
           if (i === maxTries - 1) {
             throw e;

src/core/cri/deno-cri/chrome.js

@@ -61,20 +61,30 @@ export default class Chrome extends EventEmitter {
   constructor(options, notifier) {
     super();
     // options
-    const defaultTarget = (targets) => {
-      // prefer type = 'page' inspectable targets as they represents
-      // browser tabs (fall back to the first inspectable target
-      // otherwise)
-      let backup;
-      let target = targets.find((target) => {
-        if (target.webSocketDebuggerUrl) {
-          backup = backup || target;
-          return target.type === "page";
-        } else {
-          return false;
+    const defaultTarget = async (targets) => {
+      let target;
+      // If no targets available in browser, create a new one
+      // Related to https://github.com/quarto-dev/quarto-cli/issues/4653
+      if (!targets.length) {
+        target = await devtools.New(options);
+        if (!target.id) {
+          throw new Error("No inspectable targets and unable to create one");
         }
-      });
-      target = target || backup;
+      } else {
+        // prefer type = 'page' inspectable targets as they represents
+        // browser tabs (fall back to the first inspectable target
+        // otherwise)
+        let backup;
+        target = targets.find((target) => {
+          if (target.webSocketDebuggerUrl) {
+            backup = backup || target;
+            return target.type === "page";
+          } else {
+            return false;
+          }
+        });
+        target = target || backup;
+      }
       if (target) {
         return target;
       } else {
@@ -228,7 +238,7 @@ export default class Chrome extends EventEmitter {
       case "function": {
         const func = userTarget;
         const targets = await devtools.List(options);
-        const result = func(targets);
+        const result = await func(targets);
         const object = typeof result === "number" ? targets[result] : result;
         return object.webSocketDebuggerUrl;
       }