Merge pull request #13541 from quarto-dev/revealjs/change-multiplex-url

Author: cderv

.github/workflows/test-smokes.yml

@@ -83,6 +83,15 @@ jobs:
         with:
           node-version: 20
 
+      - name: Cache multiplex server node_modules
+        if: ${{ runner.os != 'Windows' || github.event_name == 'schedule' }}
+        uses: actions/cache@v4
+        with:
+          path: tests/integration/playwright/multiplex-server/node_modules
+          key: ${{ runner.os }}-multiplex-server-${{ hashFiles('tests/integration/playwright/multiplex-server/package.json') }}
+          restore-keys: |
+            ${{ runner.os }}-multiplex-server-
+
       - name: Install node dependencies
         if: ${{ runner.os != 'Windows' || github.event_name == 'schedule' }}
         run: yarn

news/changelog-1.9.md

@@ -4,6 +4,7 @@ All changes included in 1.9:
 
 - ([#13396](https://github.com/quarto-dev/quarto-cli/issues/13396)): Fix `quarto publish connect` regression.
 - ([#13441](https://github.com/quarto-dev/quarto-cli/pull/13441)): Catch `undefined` exceptions in Pandoc failure to avoid spurious error message.
+- ([#13046](https://github.com/quarto-dev/quarto-cli/issues/13046)): Use new url for multiplex socket.io server <https://multiplex.up.railway.app/> as default for `format: revealjs` and `revealjs.multiplex: true`.
 
 ## Dependencies
 

src/format/reveal/format-reveal-multiplex.ts

@@ -104,7 +104,7 @@ interface RevealMultiplexToken {
   url: string;
 }
 
-const kDefaultMultiplexUrl = "https://reveal-multiplex.glitch.me/";
+const kDefaultMultiplexUrl = "https://multiplex.up.railway.app/";
 
 async function revealMultiplexToken(
   format: Format,

src/resources/editor/tools/vs-code.mjs

@@ -19300,7 +19300,7 @@ var require_yaml_intelligence_resources = __commonJS({
                   properties: {
                     url: {
                       string: {
-                        default: "https://reveal-multiplex.glitch.me/",
+                        default: "https://multiplex.up.railway.app/",
                         description: "Multiplex token server (defaults to Reveal-hosted server)\n"
                       }
                     },

src/resources/editor/tools/yaml/web-worker.js

@@ -12272,7 +12272,7 @@
               "properties": {
                 "url": {
                   "string": {
-                    "default": "https://reveal-multiplex.glitch.me/",
+                    "default": "https://multiplex.up.railway.app/",
                     "description": "Multiplex token server (defaults to Reveal-hosted server)\n"
                   }
                 },

src/resources/editor/tools/yaml/yaml-intelligence-resources.json

@@ -5,4 +5,4 @@ config:
   multiplex:
     secret: null
     id: null
-    url: "https://reveal-multiplex.glitch.me/"
+    url: "https://multiplex.up.railway.app/"

src/resources/formats/revealjs/plugins/multiplex/plugin.yml

@@ -87,7 +87,7 @@
           properties:
             url:
               string:
-                default: https://reveal-multiplex.glitch.me/
+                default: https://multiplex.up.railway.app/
                 description: |
                   Multiplex token server (defaults to Reveal-hosted server)
             id:

src/resources/schema/document-reveal-tools.yml

@@ -0,0 +1,34 @@
+---
+title: "Multiplex Test Presentation"
+format:
+  revealjs:
+    multiplex:
+      url: "http://127.0.0.1:1948/"
+      # setting secret and id is required to avoid having multiplex server running at render time
+      secret: "c04998070a4ec17940ab3c52101daefd"
+      id: "52d3aedefbff55dbe54e1fa5229df99b849a33951c7c09afb6bcf7e6f33233c7"
+---
+
+## Slide 1 {#slide-1}
+
+This is the first slide.
+
+## Slide 2 {#slide-2}
+
+This is the second slide with some content.
+
+## Slide 3 {#slide-3}
+
+### Fragment Test
+
+::: {.fragment}
+Fragment 1
+:::
+
+::: {.fragment}
+Fragment 2
+:::
+
+## Slide 4 {#slide-4}
+
+Final slide with more content.

tests/docs/playwright/revealjs/multiplex.qmd

@@ -16,6 +16,8 @@ import { execProcess } from "../../src/core/process.ts";
 import { quartoDevCmd } from "../utils.ts";
 import { fail } from "testing/asserts";
 import { isWindows } from "../../src/deno_ral/platform.ts";
+import { join } from "../../src/deno_ral/path.ts";
+import { existsSync } from "../../src/deno_ral/fs.ts";
 
 async function fullInit() {
   await initYamlIntelligenceResourcesFromFilesystem();
@@ -30,6 +32,19 @@ const globOutput = Deno.args.length
 setInitializer(fullInit);
 await initState();
 
+// Install multiplex server dependencies if needed
+const multiplexServerPath = "integration/playwright/multiplex-server";
+const multiplexNodeModules = join(multiplexServerPath, "node_modules");
+if (!existsSync(multiplexNodeModules)) {
+  console.log("Installing multiplex server dependencies...");
+  await execProcess({
+    cmd: isWindows ? "npm.cmd" : "npm",
+    args: ["install", "--loglevel=error"],
+    cwd: multiplexServerPath,
+  });
+  console.log("Multiplex server dependencies installed.");
+}
+
 // const promises = [];
 const fileNames: string[] = [];
 const extraOpts = [
@@ -52,7 +67,7 @@ for (const { path: fileName } of globOutput) {
   // mediabag inspection if we don't wait all renders
   // individually. This is very slow..
   await execProcess({
-    cmd: quartoDevCmd(), 
+    cmd: quartoDevCmd(),
     args: ["render", input, ...options],
   });
   fileNames.push(fileName);