@silverbulletmd/silverbullet 2.5.3 → 2.6.1

package/client/space_lua/stdlib/format.ts

@@ -58,7 +58,8 @@ function parseSpec(
 
   // Parse width
   let width = 0;
-  if (i < len && fmt.charCodeAt(i) === 42) { // '*'
+  if (i < len && fmt.charCodeAt(i) === 42) {
+    // '*'
     width = -1;
     i++;
   } else {
@@ -71,10 +72,12 @@ function parseSpec(
   // Parse precision
   let hasPrec = false;
   let prec = 0;
-  if (i < len && fmt.charCodeAt(i) === 46) { // '.'
+  if (i < len && fmt.charCodeAt(i) === 46) {
+    // '.'
     hasPrec = true;
     i++;
-    if (i < len && fmt.charCodeAt(i) === 42) { // '*'
+    if (i < len && fmt.charCodeAt(i) === 42) {
+      // '*'
       prec = -1;
       i++;
     } else {
@@ -109,9 +112,10 @@ function parseSpec(
 function pad(s: string, width: number, flags: number, numPad: boolean): string {
   if (width <= 0 || s.length >= width) return s;
   const n = width - s.length;
-  if (numPad && (flags & FLAG_ZERO) && !(flags & FLAG_MINUS)) {
+  if (numPad && flags & FLAG_ZERO && !(flags & FLAG_MINUS)) {
     let signLen = 0;
-    if (s.charCodeAt(0) === 45 || s.charCodeAt(0) === 43) { // '-' or '+'
+    if (s.charCodeAt(0) === 45 || s.charCodeAt(0) === 43) {
+      // '-' or '+'
       signLen = 1;
     } else if (
       s.charCodeAt(0) === 48 &&
@@ -235,14 +239,17 @@ function formatFloat(n: number, spec: FormatSpec): string {
 
   let body: string;
 
-  if (lower === 102) { // 'f'
+  if (lower === 102) {
+    // 'f'
     body = abs.toFixed(prec);
-  } else if (lower === 101) { // 'e'
+  } else if (lower === 101) {
+    // 'e'
     body = abs.toExponential(prec);
     // Ensure exponent has at least 2 digits
     body = ensureExpTwoDigits(body);
-  } else { // 'g'
-    const gPrec = (prec === 0) ? 1 : prec;
+  } else {
+    // 'g'
+    const gPrec = prec === 0 ? 1 : prec;
     if (abs === 0) {
       body = "0";
     } else {
@@ -268,7 +275,7 @@ function formatFloat(n: number, spec: FormatSpec): string {
   }
 
   // Alt flag for 'f'/'e': ensure decimal point exists
-  if ((spec.flags & FLAG_HASH) && lower !== 103) {
+  if (spec.flags & FLAG_HASH && lower !== 103) {
     if (body.indexOf(".") === -1) {
       // Insert dot before 'e' if present, else append
       const eIdx = body.indexOf("e");
@@ -283,7 +290,7 @@ function formatFloat(n: number, spec: FormatSpec): string {
   }
 
   // Alt flag for 'g': keep trailing zeros but ensure decimal point
-  if ((spec.flags & FLAG_HASH) && lower === 103) {
+  if (spec.flags & FLAG_HASH && lower === 103) {
     if (body.indexOf(".") === -1) {
       const expIdx = findExpIndex(body);
       if (expIdx !== -1) {
@@ -337,7 +344,8 @@ function stripTrailingZerosG(s: string): string {
   if (dotIdx === -1) return s; // nothing to strip
 
   let end = mantissa.length;
-  while (end > dotIdx + 1 && mantissa.charCodeAt(end - 1) === 48) { // '0'
+  while (end > dotIdx + 1 && mantissa.charCodeAt(end - 1) === 48) {
+    // '0'
     end--;
   }
   // Remove dot if nothing after it
@@ -389,7 +397,8 @@ function formatHexFloat(n: number, spec: FormatSpec): string {
     if (pIdx !== -1) {
       let hasDot = false;
       for (let k = 0; k < pIdx; k++) {
-        if (body.charCodeAt(k) === 46) { // '.'
+        if (body.charCodeAt(k) === 46) {
+          // '.'
           hasDot = true;
           break;
         }
@@ -436,8 +445,8 @@ function hexFloatBody(abs: number, spec: FormatSpec): string {
   const view = new DataView(buf.buffer);
   view.setFloat64(0, abs);
   const bits = view.getBigUint64(0);
-  const biasedExp = Number((bits >> 52n) & 0x7FFn);
-  const frac = bits & 0xFFFFFFFFFFFFFn;
+  const biasedExp = Number((bits >> 52n) & 0x7ffn);
+  const frac = bits & 0xfffffffffffffn;
 
   let exponent: number;
   let mantBits: bigint;
@@ -557,7 +566,8 @@ function padHexRight(s: string, len: number): string {
 
 function stripHexTrailingZeros(s: string): string {
   let end = s.length;
-  while (end > 0 && s.charCodeAt(end - 1) === 48) { // '0'
+  while (end > 0 && s.charCodeAt(end - 1) === 48) {
+    // '0'
     end--;
   }
   if (end === s.length) return s;
@@ -669,7 +679,8 @@ export function luaFormat(fmt: string, ...args: any[]): string {
 
   while (i < len) {
     const c = fmt.charCodeAt(i);
-    if (c !== 37) { // not '%'
+    if (c !== 37) {
+      // not '%'
       // Fast path: scan for next '%' or end
       let j = i + 1;
       while (j < len && fmt.charCodeAt(j) !== 37) j++;
@@ -743,15 +754,18 @@ export function luaFormat(fmt: string, ...args: any[]): string {
           false,
         );
         break;
-      case 112: { // 'p'
+      case 112: {
+        // 'p'
         out += formatPointer(args[ai++], spec);
         break;
       }
-      case 113: { // 'q'
+      case 113: {
+        // 'q'
         out += formatQ(args[ai++]);
         break;
       }
-      case 115: { // 's'
+      case 115: {
+        // 's'
         let s = String(args[ai++]);
         if (spec.hasPrec && s.length > spec.prec) {
           s = s.slice(0, spec.prec);

package/client/space_lua/stdlib/js.ts

@@ -12,13 +12,9 @@ export const jsApi = new LuaTable({
    * @param args - The arguments to pass to the constructor.
    * @returns The new instance.
    */
-  new: new LuaBuiltinFunction(
-    (sf, constructorFn: any, ...args) => {
-      return new constructorFn(
-        ...args.map((v) => luaValueToJS(v, sf)),
-      );
-    },
-  ),
+  new: new LuaBuiltinFunction((sf, constructorFn: any, ...args) => {
+    return new constructorFn(...args.map((v) => luaValueToJS(v, sf)));
+  }),
   /**
    * Imports a JavaScript module.
    * @param url - The URL of the module to import.

package/client/space_lua/stdlib/load.ts

@@ -18,9 +18,7 @@ export function luaLoad(code: LuaValue, sf: LuaStackFrame): LuaValue {
 
     // Be vocal when no _GLOBAL is set
     if (!globalEnvMaybe) {
-      console.warn(
-        "load() called without _GLOBAL in thread-local environment",
-      );
+      console.warn("load() called without _GLOBAL in thread-local environment");
       return new LuaMultiRes([null, "Global environment not set"]);
     }
 

package/client/space_lua/stdlib/math.ts

@@ -60,7 +60,8 @@ export const mathApi = new LuaTable({
     }
     if (typeof x === "string") {
       const n = untagNumber(x); // Number(x) coerces the string
-      if (Number.isNaN(n) || !Number.isFinite(n) || !Number.isInteger(n)) return null;
+      if (Number.isNaN(n) || !Number.isFinite(n) || !Number.isInteger(n))
+        return null;
       return n;
     }
     return null;
@@ -100,15 +101,15 @@ export const mathApi = new LuaTable({
   ceil: new LuaBuiltinFunction((_sf, x: number) => Math.ceil(untagNumber(x))),
   floor: new LuaBuiltinFunction((_sf, x: number) => Math.floor(untagNumber(x))),
   max: new LuaBuiltinFunction((_sf, ...args: number[]) =>
-    Math.max(...args.map(untagNumber))
+    Math.max(...args.map(untagNumber)),
   ),
   min: new LuaBuiltinFunction((_sf, ...args: number[]) =>
-    Math.min(...args.map(untagNumber))
+    Math.min(...args.map(untagNumber)),
   ),
 
   // Rounding and remainder
-  fmod: new LuaBuiltinFunction((_sf, x: number, y: number) =>
-    untagNumber(x) % untagNumber(y)
+  fmod: new LuaBuiltinFunction(
+    (_sf, x: number, y: number) => untagNumber(x) % untagNumber(y),
   ),
   modf: new LuaBuiltinFunction((_sf, x: number) => {
     const xn = untagNumber(x);
@@ -141,8 +142,8 @@ export const mathApi = new LuaTable({
   }),
 
   // Returns m * 2^e (the inverse of frexp).  Mirrors C99/Lua.
-  ldexp: new LuaBuiltinFunction((_sf, m: number, e: number) =>
-    untagNumber(m) * 2 ** untagNumber(e)
+  ldexp: new LuaBuiltinFunction(
+    (_sf, m: number, e: number) => untagNumber(m) * 2 ** untagNumber(e),
   ),
 
   // Power and logarithms
@@ -154,8 +155,8 @@ export const mathApi = new LuaTable({
     return Math.log(untagNumber(x)) / Math.log(untagNumber(base));
   }),
   // Power function (deprecated in Lua 5.4 but retained for compatibility)
-  pow: new LuaBuiltinFunction((_sf, x: number, y: number) =>
-    untagNumber(x) ** untagNumber(y)
+  pow: new LuaBuiltinFunction(
+    (_sf, x: number, y: number) => untagNumber(x) ** untagNumber(y),
   ),
   sqrt: new LuaBuiltinFunction((_sf, x: number) => Math.sqrt(untagNumber(x))),
 
@@ -178,14 +179,14 @@ export const mathApi = new LuaTable({
   tanh: new LuaBuiltinFunction((_sf, x: number) => Math.tanh(untagNumber(x))),
 
   // Additional utility
-  deg: new LuaBuiltinFunction((_sf, x: number) =>
-    untagNumber(x) * 180 / Math.PI
+  deg: new LuaBuiltinFunction(
+    (_sf, x: number) => (untagNumber(x) * 180) / Math.PI,
   ),
-  rad: new LuaBuiltinFunction((_sf, x: number) =>
-    untagNumber(x) * Math.PI / 180
+  rad: new LuaBuiltinFunction(
+    (_sf, x: number) => (untagNumber(x) * Math.PI) / 180,
   ),
   ult: new LuaBuiltinFunction((_sf, m: number, n: number) => {
-    return (untagNumber(m) >>> 0) < (untagNumber(n) >>> 0);
+    return untagNumber(m) >>> 0 < untagNumber(n) >>> 0;
   }),
 
   // Keep the cosineSimilarity utility function

package/client/space_lua/stdlib/net.ts

@@ -14,17 +14,18 @@ export const netApi = new LuaTable({
     ): Promise<ProxyFetchResponse> => {
       // JSONify any non-serializable body
       if (
-        options?.body && typeof options.body !== "string" &&
+        options?.body &&
+        typeof options.body !== "string" &&
         !(options.body instanceof Uint8Array)
       ) {
         options.body = JSON.stringify(options.body);
       }
       const fetchOptions = options
         ? {
-          method: options.method,
-          headers: {} as Record<string, string>,
-          body: options.body,
-        }
+            method: options.method,
+            headers: {} as Record<string, string>,
+            body: options.body,
+          }
         : {};
       fetchOptions.headers = buildProxyHeaders(options.headers);
       const resp = await client.httpSpacePrimitives.authenticatedFetch(
@@ -40,19 +41,27 @@ export const netApi = new LuaTable({
         };
       }
       // Do sensible things with the body based on the content type
+      // Read as ArrayBuffer first to safely handle empty responses (e.g.
+      // PUT/DELETE returning 204 with Content-Type: application/json).
+      // resp.arrayBuffer() never throws on an empty body, whereas
+      // resp.json() would throw a SyntaxError.
+      const rawBytes = new Uint8Array(await resp.arrayBuffer());
       let body: any;
-      const contentTypeHeader = options.responseEncoding ||
+      const contentTypeHeader =
+        options.responseEncoding ||
         resp.headers.get("x-proxy-header-content-type");
       const statusCode = +(resp.headers.get("x-proxy-status-code") || "200");
-      if (contentTypeHeader?.startsWith("application/json")) {
-        body = await resp.json();
+      if (rawBytes.length === 0) {
+        body = null;
+      } else if (contentTypeHeader?.startsWith("application/json")) {
+        body = JSON.parse(new TextDecoder().decode(rawBytes));
       } else if (
         contentTypeHeader?.startsWith("application/xml") ||
         contentTypeHeader?.startsWith("text/")
       ) {
-        body = await resp.text();
+        body = new TextDecoder().decode(rawBytes);
       } else {
-        body = new Uint8Array(await resp.arrayBuffer());
+        body = rawBytes;
       }
       return {
         ok: resp.ok,
@@ -85,8 +94,11 @@ export const netApi = new LuaTable({
 function buildProxyUrl(client: Client, url: string) {
   url = url.replace(/^https?:\/\//, "");
   // Strip off the /.fs and replace with /.proxy
-  return client.httpSpacePrimitives.url.slice(0, -fsEndpoint.length) +
-    "/.proxy/" + url;
+  return (
+    client.httpSpacePrimitives.url.slice(0, -fsEndpoint.length) +
+    "/.proxy/" +
+    url
+  );
 }
 
 function buildProxyHeaders(headers?: Record<string, any>): Record<string, any> {
@@ -100,9 +112,7 @@ function buildProxyHeaders(headers?: Record<string, any>): Record<string, any> {
   return newHeaders;
 }
 
-function extractProxyHeaders(
-  headers: Headers,
-): Record<string, any> {
+function extractProxyHeaders(headers: Headers): Record<string, any> {
   const newHeaders: Record<string, any> = {};
   for (const [key, value] of headers.entries()) {
     if (key.toLowerCase().startsWith("x-proxy-header-")) {

package/client/space_lua/stdlib/os.ts

@@ -16,9 +16,7 @@ function weekNumber(
 
   if (iso) {
     const target = new Date(date);
-    target.setUTCDate(
-      target.getUTCDate() + 3 - ((target.getUTCDay() + 6) % 7),
-    );
+    target.setUTCDate(target.getUTCDate() + 3 - ((target.getUTCDay() + 6) % 7));
 
     const yearStart = new Date(Date.UTC(target.getUTCFullYear(), 0, 4));
     const weekStart = new Date(yearStart);
@@ -34,11 +32,10 @@ function weekNumber(
   const offset = (7 + (startDay - weekStartDay)) % 7;
   const firstWeekStart = new Date(yearStart);
 
-  firstWeekStart.setUTCDate(yearStart.getUTCDate() + (7 - offset) % 7);
+  firstWeekStart.setUTCDate(yearStart.getUTCDate() + ((7 - offset) % 7));
 
   if (date < firstWeekStart) return 0;
-  return 1 +
-    Math.floor((date.getTime() - firstWeekStart.getTime()) / ONE_WEEK);
+  return 1 + Math.floor((date.getTime() - firstWeekStart.getTime()) / ONE_WEEK);
 }
 
 function dayOfYear(d: Date, utc: boolean): number {
@@ -47,25 +44,25 @@ function dayOfYear(d: Date, utc: boolean): number {
 
   const current = utc
     ? new Date(
-      Date.UTC(
-        year,
-        d.getUTCMonth(),
-        d.getUTCDate(),
-        d.getUTCHours(),
-        d.getUTCMinutes(),
-        d.getUTCSeconds(),
-      ),
-    )
+        Date.UTC(
+          year,
+          d.getUTCMonth(),
+          d.getUTCDate(),
+          d.getUTCHours(),
+          d.getUTCMinutes(),
+          d.getUTCSeconds(),
+        ),
+      )
     : new Date(
-      Date.UTC(
-        year,
-        d.getMonth(),
-        d.getDate(),
-        d.getHours(),
-        d.getMinutes(),
-        d.getSeconds(),
-      ),
-    );
+        Date.UTC(
+          year,
+          d.getMonth(),
+          d.getDate(),
+          d.getHours(),
+          d.getMinutes(),
+          d.getSeconds(),
+        ),
+      );
 
   return Math.floor((current.getTime() - start.getTime()) / ONE_DAY);
 }
@@ -76,9 +73,7 @@ function isoWeekYear(d: Date, utc: boolean): number {
   const day = utc ? d.getUTCDate() : d.getDate();
   const target = new Date(Date.UTC(year, month, day));
 
-  target.setUTCDate(
-    target.getUTCDate() + 3 - ((target.getUTCDay() + 6) % 7),
-  );
+  target.setUTCDate(target.getUTCDate() + 3 - ((target.getUTCDay() + 6) % 7));
 
   return target.getUTCFullYear();
 }
@@ -138,7 +133,7 @@ function dateTable(d: Date, utc: boolean): LuaTable {
   });
 
   if (!utc) {
-    tbl.rawSet("isdst", isDST(d));
+    void tbl.rawSet("isdst", isDST(d));
   }
 
   return tbl;
@@ -146,107 +141,104 @@ function dateTable(d: Date, utc: boolean): LuaTable {
 
 // Build the specifier map for a given `Date` and `utc` flag.
 // Returns a record mapping single-char specifier to its output string.
-function buildSpecMap(
-  d: Date,
-  utc: boolean,
-): Record<string, () => string> {
+function buildSpecMap(d: Date, utc: boolean): Record<string, () => string> {
   const h = () => hr(d, utc);
   const h12 = () => h() % 12 || 12;
   const dow = () => wd(d, utc);
 
   return {
     // Date
-    "Y": () => yr(d, utc).toString(),
-    "y": () => pad2(yr(d, utc) % 100),
-    "C": () => pad2(Math.floor(yr(d, utc) / 100)),
-    "m": () => pad2(mo(d, utc) + 1),
-    "d": () => pad2(da(d, utc)),
-    "e": () => da(d, utc).toString().padStart(2, " "),
-    "j": () => pad3(dayOfYear(d, utc)),
+    Y: () => yr(d, utc).toString(),
+    y: () => pad2(yr(d, utc) % 100),
+    C: () => pad2(Math.floor(yr(d, utc) / 100)),
+    m: () => pad2(mo(d, utc) + 1),
+    d: () => pad2(da(d, utc)),
+    e: () => da(d, utc).toString().padStart(2, " "),
+    j: () => pad3(dayOfYear(d, utc)),
 
     // Time
-    "H": () => pad2(h()),
-    "I": () => pad2(h12()),
-    "M": () => pad2(mi(d, utc)),
-    "S": () => pad2(sc(d, utc)),
-    "p": () => h() >= 12 ? "PM" : "AM",
+    H: () => pad2(h()),
+    I: () => pad2(h12()),
+    M: () => pad2(mi(d, utc)),
+    S: () => pad2(sc(d, utc)),
+    p: () => (h() >= 12 ? "PM" : "AM"),
 
     // Weekday
-    "A": () =>
+    A: () =>
       d.toLocaleString("en-US", {
         weekday: "long",
         ...(utc ? { timeZone: "UTC" } : {}),
       }),
-    "a": () =>
+    a: () =>
       d.toLocaleString("en-US", {
         weekday: "short",
         ...(utc ? { timeZone: "UTC" } : {}),
       }),
-    "w": () => dow().toString(),
-    "u": () => (dow() === 0 ? 7 : dow()).toString(),
+    w: () => dow().toString(),
+    u: () => (dow() === 0 ? 7 : dow()).toString(),
 
     // Month name
-    "b": () =>
+    b: () =>
       d.toLocaleString("en-US", {
         month: "short",
         ...(utc ? { timeZone: "UTC" } : {}),
       }),
-    "h": () =>
+    h: () =>
       d.toLocaleString("en-US", {
         month: "short",
         ...(utc ? { timeZone: "UTC" } : {}),
       }),
-    "B": () =>
+    B: () =>
       d.toLocaleString("en-US", {
         month: "long",
         ...(utc ? { timeZone: "UTC" } : {}),
       }),
 
     // Week number
-    "U": () => pad2(weekNumber(d, utc, 0, false)),
-    "W": () => pad2(weekNumber(d, utc, 1, false)),
-    "V": () => pad2(weekNumber(d, utc, 1, true)),
-    "G": () => isoWeekYear(d, utc).toString(),
-    "g": () => pad2(isoWeekYear(d, utc) % 100),
+    U: () => pad2(weekNumber(d, utc, 0, false)),
+    W: () => pad2(weekNumber(d, utc, 1, false)),
+    V: () => pad2(weekNumber(d, utc, 1, true)),
+    G: () => isoWeekYear(d, utc).toString(),
+    g: () => pad2(isoWeekYear(d, utc) % 100),
 
     // Composite specifiers
-    "c": () =>
+    c: () =>
       d.toLocaleString("en-US", {
         ...(utc ? { timeZone: "UTC" } : {}),
       }),
-    "x": () =>
+    x: () =>
       d.toLocaleDateString("en-US", {
         ...(utc ? { timeZone: "UTC" } : {}),
       }),
-    "X": () =>
+    X: () =>
       d.toLocaleTimeString("en-US", {
         ...(utc ? { timeZone: "UTC" } : {}),
       }),
-    "D": () =>
+    D: () =>
       `${pad2(mo(d, utc) + 1)}/${pad2(da(d, utc))}/${pad2(yr(d, utc) % 100)}`,
-    "F": () =>
+    F: () =>
       `${yr(d, utc).toString()}-${pad2(mo(d, utc) + 1)}-${pad2(da(d, utc))}`,
-    "R": () => `${pad2(h())}:${pad2(mi(d, utc))}`,
-    "T": () => `${pad2(h())}:${pad2(mi(d, utc))}:${pad2(sc(d, utc))}`,
-    "r": () =>
+    R: () => `${pad2(h())}:${pad2(mi(d, utc))}`,
+    T: () => `${pad2(h())}:${pad2(mi(d, utc))}:${pad2(sc(d, utc))}`,
+    r: () =>
       `${pad2(h12())}:${pad2(mi(d, utc))}:${pad2(sc(d, utc))} ${
         h() >= 12 ? "PM" : "AM"
       }`,
 
     // Epoch
-    "s": () => Math.floor(d.getTime() / 1000).toString(),
+    s: () => Math.floor(d.getTime() / 1000).toString(),
 
     // Whitespace
-    "n": () => "\n",
-    "t": () => "\t",
+    n: () => "\n",
+    t: () => "\t",
 
     // Timezone
-    "Z": () => {
+    Z: () => {
       if (utc) return "UTC";
       const match = d.toTimeString().match(/\((.*)\)/);
       return match ? match[1] : "";
     },
-    "z": () => {
+    z: () => {
       if (utc) return "+0000";
       const offset = -d.getTimezoneOffset();
       const sign = offset >= 0 ? "+" : "-";
@@ -344,27 +336,26 @@ export const osApi = new LuaTable({
   // Otherwise, format specifiers follow ISO C `strftime`.
   //
   // If format is absent, it defaults to `%c`.
-  date: new LuaBuiltinFunction(
-    (_sf, format?: string, timestamp?: number) => {
-      let fmt = format ?? "%c";
-      let utc = false;
-
-      if (fmt.startsWith("!")) {
-        utc = true;
-        fmt = fmt.slice(1);
-      }
+  date: new LuaBuiltinFunction((_sf, format?: string, timestamp?: number) => {
+    let fmt = format ?? "%c";
+    let utc = false;
 
-      const d = timestamp !== undefined && timestamp !== null
+    if (fmt.startsWith("!")) {
+      utc = true;
+      fmt = fmt.slice(1);
+    }
+
+    const d =
+      timestamp !== undefined && timestamp !== null
         ? new Date(timestamp * 1000)
         : new Date();
 
-      if (fmt === "*t") {
-        return dateTable(d, utc);
-      }
+    if (fmt === "*t") {
+      return dateTable(d, utc);
+    }
 
-      return luaFormatTime(fmt, d, utc);
-    },
-  ),
+    return luaFormatTime(fmt, d, utc);
+  }),
 
   // Returns an approximation of CPU time used by the program in seconds.
   clock: new LuaBuiltinFunction((_sf): number => {