@yuneta/gobj-js 7.28.0 → 7.29.0

package/dist/gobj-js.umd.js

@@ -430,7 +430,7 @@
 		}
 	};
 	function tab() {
-		return " ".repeat(Math.max(1, __inside__ * 2));
+		return " ".repeat(Math.max(1, __inside__ * 4));
 	}
 	var __trace_machine_format__ = 1;
 	function gobj_set_trace_machine_format(format) {
@@ -7547,28 +7547,43 @@
 	*  Build a URL/path from segments, ensuring exactly one '/'
 	*  between each part.
 	*
-	*  - The first segment keeps its leading '/' (absolute) or
-	*    not (relative).
-	*  - Trailing '/' is stripped from every segment.
-	*  - Leading '/' is stripped from every segment except the first.
+	*  - The first segment is the ROOT: it keeps its leading '/'
+	*    (absolute) or not (relative), and its trailing '/' are
+	*    stripped -- but a root made only of '/' stays "/".
+	*  - Leading and trailing '/' are stripped from every later
+	*    segment, and duplicate '/' inside it are collapsed.
+	*  - '.' and '..' of the later segments are resolved, and '..'
+	*    never rises above the root: ("/store", "../../etc") is
+	*    "/store/etc".
 	*  - Empty/null/undefined segments are skipped.
 	*
-	*  Mirrors the C helper build_path() from gobj-c/src/helpers.c.
+	*  Mirrors the C helper build_path() from gobj-c/src/helpers.c
+	*  (tests/build_path.test.js pins the same cases). Up to 7.28.1
+	*  a root "/" was lost (("/", "tmp") gave "tmp") and '.'/'..'
+	*  were kept as they came.
 	************************************************************/
 	function build_path(...segments) {
-		let result = "";
-		let i = 0;
+		let root = null;
+		let tail = [];
 		for (let segment of segments) {
 			if (segment == null || segment === "") continue;
 			segment = String(segment);
-			if (i === 0) segment = segment.replace(/\/+$/, "");
-			else segment = segment.replace(/^\/+/, "").replace(/\/+$/, "");
-			if (segment === "") continue;
-			if (i > 0) result += "/";
-			result += segment;
-			i++;
+			if (root === null) {
+				root = segment.replace(/(.)\/+$/, "$1");
+				continue;
+			}
+			for (let component of segment.split("/")) {
+				if (component === "" || component === ".") continue;
+				if (component === "..") {
+					tail.pop();
+					continue;
+				}
+				tail.push(component);
+			}
 		}
-		return result;
+		if (root === null) return "";
+		if (tail.length === 0) return root;
+		return root + (root.endsWith("/") ? "" : "/") + tail.join("/");
 	}
 	/***************************************
 	*      Flat json (id -> value)