@poe-platform/safe-fs 0.1.743 → 0.1.744

package/dist/safe-fs/contracts/path.d.ts

@@ -1,4 +1,4 @@
-import { posix } from "#safe-fs-platform";
+import { posix } from "#safe-fs-platform-path";
 export { assertPathWithin, isPathWithin, normalizePath, relativePath, resolvePath, validatePath } from "./virtual-path.js";
 export declare const posixPath: typeof posix;
 export declare const basename: typeof posix.basename;

package/dist/safe-fs/contracts/path.js

@@ -1,4 +1,4 @@
-import { posix } from "#safe-fs-platform";
+import { posix } from "#safe-fs-platform-path";
 export { assertPathWithin, isPathWithin, normalizePath, relativePath, resolvePath, validatePath } from "./virtual-path.js";
 export const posixPath = posix;
 export const basename = posix.basename;

package/dist/safe-fs/platform/browser-path.d.ts

@@ -0,0 +1 @@
+export { posixPath as posix } from "../contracts/portable-path.js";

package/dist/safe-fs/platform/browser-path.js

@@ -0,0 +1 @@
+export { posixPath as posix } from "../contracts/portable-path.js";

package/dist/safe-fs/platform/browser.d.ts

@@ -1,4 +1,3 @@
-export { posixPath as posix } from "../contracts/portable-path.js";
 import type { FsOptions } from "../contracts/filesystem.js";
 import type { ScopedTransportBudgetFrame } from "./transport-budget.js";
 export type PlatformErrno = number | undefined;

package/dist/safe-fs/platform/browser.js

@@ -1,5 +1,4 @@
 import { FsError } from "../contracts/errors.js";
-export { posixPath as posix } from "../contracts/portable-path.js";
 import { chargeTransportOptions } from "./transport-budget.js";
 export const platform = Object.freeze({
     nativeFileSystem: Object.freeze({ open: false, permissions: false, timestamps: false, conditionalChmod: false, trustedOwnedStaging: false, atomicRename: false }),

package/dist/safe-fs/platform/node-path.d.ts

@@ -0,0 +1 @@
+export { posix } from "node:path";

package/dist/safe-fs/platform/node-path.js

@@ -0,0 +1 @@
+export { posix } from "node:path";

package/dist/safe-fs/platform/node.d.ts

@@ -1,4 +1,3 @@
-export { posix } from "node:path";
 import type { FsOptions } from "../contracts/filesystem.js";
 import { type ScopedTransportBudgetFrame } from "./transport-budget.js";
 export type PlatformErrno = number;

package/dist/safe-fs/platform/node.js

@@ -1,5 +1,4 @@
 import { AsyncLocalStorage } from "node:async_hooks";
-export { posix } from "node:path";
 import { randomUUID } from "node:crypto";
 import * as util from "node:util";
 import { constants } from "node:os";

package/dist/safe-fs/xml.js

@@ -42,7 +42,9 @@ function namePart(point) {
         || (point >= 48 && point <= 57) || (point >= 0x300 && point <= 0x36f)
         || (point >= 0x203f && point <= 0x2040);
 }
-function qualifiedNameSync(name) {
+function qualifiedNameSync(name, cache) {
+    if (cache.name === name)
+        return cache.parts;
     let prefix = "";
     let start = 0;
     let first = true;
@@ -64,40 +66,13 @@ function qualifiedNameSync(name) {
     }
     if (first)
         invalid("invalid qualified name");
-    return [prefix, name.slice(start)];
-}
-function* qualifiedName(name) {
-    let prefix = "";
-    let start = 0;
-    let first = true;
-    let work = 0;
-    for (let offset = 0; offset < name.length;) {
-        const point = name.codePointAt(offset);
-        if (point === 58) {
-            if (start !== 0 || first)
-                invalid("invalid qualified name");
-            prefix = name.slice(0, offset);
-            start = offset + 1;
-            first = true;
-        }
-        else {
-            if (!(first ? nameStart(point) : namePart(point)))
-                invalid("invalid qualified name");
-            first = false;
-        }
-        const width = point > 0xffff ? 2 : 1;
-        offset += width;
-        work += width;
-        if (work >= 512) {
-            yield work;
-            work = 0;
-        }
+    const parts = [prefix, name.slice(start)];
+    // Retain only one short name; namespace resolution still happens per element.
+    if (name.length <= 512) {
+        cache.name = name;
+        cache.parts = parts;
     }
-    if (first)
-        invalid("invalid qualified name");
-    if (work)
-        yield work;
-    return [prefix, name.slice(start)];
+    return parts;
 }
 function* entities(text) {
     let result = "";
@@ -274,6 +249,7 @@ export function* parseXmlSteps(input, limits = {}) {
     let attributeCount = 0;
     let contentNodes = 0;
     let previousEmpty;
+    const qualifiedNames = { name: undefined, parts: undefined };
     const admitContent = () => {
         if (++contentNodes > maxContentNodes)
             throw new XmlLimitError("maxContentNodes", "XML content node limit exceeded");
@@ -298,7 +274,7 @@ export function* parseXmlSteps(input, limits = {}) {
             offset++;
         }
         const name = source.slice(start, offset);
-        const [prefix, localName] = qualifiedNameSync(name);
+        const [prefix, localName] = qualifiedNameSync(name, qualifiedNames);
         return [name, prefix, localName, (offset - start) * 2];
     };
     while (offset < source.length) {
@@ -475,12 +451,11 @@ export function* parseXmlSteps(input, limits = {}) {
                     yield 512;
                     pendingWork -= 512;
                 }
-                attributes ??= new Map();
-                if (attributes.has(attribute))
+                if (attributes?.has(attribute))
                     invalid("duplicate attribute");
                 if (++attributeCount > maxAttributes)
                     throw new XmlLimitError("maxAttributes", "XML attribute limit exceeded");
-                if (attributes.size >= maxAttributesPerElement)
+                if ((attributes?.size ?? 0) >= maxAttributesPerElement)
                     throw new XmlLimitError("maxAttributesPerElement", "XML attribute limit exceeded");
                 pendingWork += skipWhitespace();
                 if (source[offset++] !== "=")
@@ -513,7 +488,7 @@ export function* parseXmlSteps(input, limits = {}) {
                     pendingWork -= 512;
                 }
                 admitText(value);
-                attributes.set(attribute, value);
+                (attributes ??= new Map()).set(attribute, value);
                 attrMeta.push([attribute, attrPrefix, attrLocal]);
                 offset = end + 1;
                 if (attribute === "xmlns" || attribute.startsWith("xmlns:")) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@poe-platform/safe-fs",
-  "version": "0.1.743",
+  "version": "0.1.744",
   "description": "Composable filesystem with a portable core and explicit Node adapters",
   "type": "module",
   "license": "MIT",
@@ -144,6 +144,16 @@
       "browser": "./dist/safe-fs/platform/browser.js",
       "default": "./dist/safe-fs/platform/node.js"
     },
+    "#safe-fs-platform-path": {
+      "types": {
+        "workerd": "./dist/safe-fs/platform/browser-path.d.ts",
+        "browser": "./dist/safe-fs/platform/browser-path.d.ts",
+        "default": "./dist/safe-fs/platform/node-path.d.ts"
+      },
+      "workerd": "./dist/safe-fs/platform/browser-path.js",
+      "browser": "./dist/safe-fs/platform/browser-path.js",
+      "default": "./dist/safe-fs/platform/node-path.js"
+    },
     "#safe-fs-native-seek": {
       "types": "./dist/safe-fs/native/fs-seek/loader.d.ts",
       "workerd": "./dist/safe-fs/node/native-seek-unavailable.js",