Use XDG_RUNTIME_DIR instead of tmp

Author: sonnyp

src/Previewer/DBusPreviewer.js

@@ -1,13 +1,15 @@
 import Gio from "gi://Gio";
 
 import previewer_xml from "./previewer.xml" with { type: "string" };
+import { buildRuntimePath } from "../util.js";
 
 const nodeInfo = Gio.DBusNodeInfo.new_for_xml(previewer_xml);
 const interface_info = nodeInfo.interfaces[0];
 
 const guid = Gio.dbus_generate_guid();
+const path = buildRuntimePath(`workbench_preview_dbus_socket_${Date.now()}`);
 const server = Gio.DBusServer.new_sync(
-  "unix:path=/var/tmp/workbench_preview_dbus_socket",
+  `unix:path=${path}`,
   Gio.DBusServerFlags.AUTHENTICATION_REQUIRE_SAME_USER,
   guid,
   null,

src/TermConsole.js

@@ -3,6 +3,7 @@ import GLib from "gi://GLib";
 import Gio from "gi://Gio";
 import Gdk from "gi://Gdk";
 import Adw from "gi://Adw";
+import { buildRuntimePath } from "./util.js";
 
 // https://gist.github.com/fnky/458719343aabd01cfb17a3a4f7296797
 const ERASE_ENTIRE_SCREEN = "\u001b[2J";
@@ -24,10 +25,13 @@ export default function TermConsole({
   // terminal.set_cursor_blink_mode(Vte.CursorBlinkMode.ON);
   // terminal.set_input_enabled(true);
 
+  // See src/workbench
+  const path = buildRuntimePath("typescript");
+
   terminal.spawn_async(
     Vte.PtyFlags.DEFAULT, // pty_flags
     null, // working_directory
-    ["/bin/tail", "--line=0", "--follow", "/var/tmp/workbench"], // argv
+    ["/bin/tail", "--line=0", "--follow", path], // argv
     [], // envv
     GLib.SpawnFlags.DEFAULT, // spawn_flags
     null,

src/util.js

@@ -179,3 +179,11 @@ export function getNowForFilename() {
 export function makeDropdownFlat(dropdown) {
   dropdown.get_first_child().add_css_class("flat");
 }
+
+export function buildRuntimePath(...args) {
+  return GLib.build_filenamev([
+    GLib.getenv("XDG_RUNTIME_DIR"),
+    GLib.getenv("FLATPAK_ID"),
+    ...args,
+  ]);
+}

src/window.js

@@ -6,7 +6,7 @@ import Adw from "gi://Adw";
 import Vte from "gi://Vte";
 
 import * as xml from "./langs/xml/xml.js";
-import { languages } from "./util.js";
+import { buildRuntimePath, languages } from "./util.js";
 import Document from "./Document.js";
 import PanelUI from "./PanelUI.js";
 import PanelCode from "./PanelCode.js";
@@ -317,7 +317,8 @@ export default function Window({ application, session }) {
       // because gjs doesn't appear to use etag for module caching
       // ?foo=Date.now() also does not work as expected
       // TODO: File a bug
-      const [file_javascript] = Gio.File.new_tmp("workbench-XXXXXX.js");
+      const path = buildRuntimePath(`workbench-${Date.now()}`);
+      const file_javascript = Gio.File.new_for_path(path);
       await file_javascript.replace_contents_async(
         new GLib.Bytes(text),
         null,

src/workbench

@@ -16,4 +16,5 @@ export CARGO_TARGET_X86_64_UNKNOWN_LINUX_GNU_RUSTFLAGS="-C link-arg=-fuse-ld=/us
 export CARGO_TARGET_AARCH64_UNKNOWN_LINUX_GNU_LINKER=clang
 export CARGO_TARGET_AARCH64_UNKNOWN_LINUX_GNU_RUSTFLAGS="-C link-arg=-fuse-ld=/usr/lib/sdk/rust-stable/bin/mold"
 
-SHELL=/bin/sh script --flush --quiet --return /var/tmp/workbench --command "@app_id@ $@"
+mkdir -p $XDG_RUNTIME_DIR/$FLATPAK_ID
+SHELL=/bin/sh script --flush --quiet --return $XDG_RUNTIME_DIR/$FLATPAK_ID/typescript --command "@app_id@ $@"