@ms-cloudpack/path-string-parsing 1.2.0 → 1.2.2

package/lib/index.d.ts

@@ -3,4 +3,5 @@ export { normalizeRelativePath } from './normalizeRelativePath.js';
 export { slash } from './slash.js';
 export { parseImportString, type ImportStringResult } from './parseImportString.js';
 export { parseNamedImports } from './parseNamedImports.js';
+export { makeUrl } from './makeUrl.js';
 //# sourceMappingURL=index.d.ts.map

package/lib/index.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,gBAAgB,EAAE,MAAM,uBAAuB,CAAC;AACzD,OAAO,EAAE,qBAAqB,EAAE,MAAM,4BAA4B,CAAC;AACnE,OAAO,EAAE,KAAK,EAAE,MAAM,YAAY,CAAC;AACnC,OAAO,EAAE,iBAAiB,EAAE,KAAK,kBAAkB,EAAE,MAAM,wBAAwB,CAAC;AACpF,OAAO,EAAE,iBAAiB,EAAE,MAAM,wBAAwB,CAAC"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,gBAAgB,EAAE,MAAM,uBAAuB,CAAC;AACzD,OAAO,EAAE,qBAAqB,EAAE,MAAM,4BAA4B,CAAC;AACnE,OAAO,EAAE,KAAK,EAAE,MAAM,YAAY,CAAC;AACnC,OAAO,EAAE,iBAAiB,EAAE,KAAK,kBAAkB,EAAE,MAAM,wBAAwB,CAAC;AACpF,OAAO,EAAE,iBAAiB,EAAE,MAAM,wBAAwB,CAAC;AAC3D,OAAO,EAAE,OAAO,EAAE,MAAM,cAAc,CAAC"}

package/lib/index.js

@@ -3,4 +3,5 @@ export { normalizeRelativePath } from './normalizeRelativePath.js';
 export { slash } from './slash.js';
 export { parseImportString } from './parseImportString.js';
 export { parseNamedImports } from './parseNamedImports.js';
+export { makeUrl } from './makeUrl.js';
 //# sourceMappingURL=index.js.map

package/lib/index.js.map

@@ -1 +1 @@
-{"version":3,"file":"index.js","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,gBAAgB,EAAE,MAAM,uBAAuB,CAAC;AACzD,OAAO,EAAE,qBAAqB,EAAE,MAAM,4BAA4B,CAAC;AACnE,OAAO,EAAE,KAAK,EAAE,MAAM,YAAY,CAAC;AACnC,OAAO,EAAE,iBAAiB,EAA2B,MAAM,wBAAwB,CAAC;AACpF,OAAO,EAAE,iBAAiB,EAAE,MAAM,wBAAwB,CAAC","sourcesContent":["export { safeRelativePath } from './safeRelativePath.js';\nexport { normalizeRelativePath } from './normalizeRelativePath.js';\nexport { slash } from './slash.js';\nexport { parseImportString, type ImportStringResult } from './parseImportString.js';\nexport { parseNamedImports } from './parseNamedImports.js';\n"]}
+{"version":3,"file":"index.js","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,gBAAgB,EAAE,MAAM,uBAAuB,CAAC;AACzD,OAAO,EAAE,qBAAqB,EAAE,MAAM,4BAA4B,CAAC;AACnE,OAAO,EAAE,KAAK,EAAE,MAAM,YAAY,CAAC;AACnC,OAAO,EAAE,iBAAiB,EAA2B,MAAM,wBAAwB,CAAC;AACpF,OAAO,EAAE,iBAAiB,EAAE,MAAM,wBAAwB,CAAC;AAC3D,OAAO,EAAE,OAAO,EAAE,MAAM,cAAc,CAAC","sourcesContent":["export { safeRelativePath } from './safeRelativePath.js';\nexport { normalizeRelativePath } from './normalizeRelativePath.js';\nexport { slash } from './slash.js';\nexport { parseImportString, type ImportStringResult } from './parseImportString.js';\nexport { parseNamedImports } from './parseNamedImports.js';\nexport { makeUrl } from './makeUrl.js';\n"]}

package/lib/makeUrl.d.ts

@@ -0,0 +1,6 @@
+/**
+ * Attempt to create a URL object from a string, and throw an informative error if it fails.
+ * (The URL constructor throws a very generic error.)
+ */
+export declare function makeUrl(input: string, base?: string | URL): URL;
+//# sourceMappingURL=makeUrl.d.ts.map

package/lib/makeUrl.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"makeUrl.d.ts","sourceRoot":"","sources":["../src/makeUrl.ts"],"names":[],"mappings":"AAAA;;;GAGG;AACH,wBAAgB,OAAO,CAAC,KAAK,EAAE,MAAM,EAAE,IAAI,CAAC,EAAE,MAAM,GAAG,GAAG,OAOzD"}

package/lib/makeUrl.js

@@ -0,0 +1,14 @@
+/**
+ * Attempt to create a URL object from a string, and throw an informative error if it fails.
+ * (The URL constructor throws a very generic error.)
+ */
+export function makeUrl(input, base) {
+    try {
+        // eslint-disable-next-line no-restricted-syntax -- this is the function definition
+        return new URL(input, base);
+    }
+    catch {
+        throw new Error(`Invalid URL: "${input}", base: "${base || ''}"`);
+    }
+}
+//# sourceMappingURL=makeUrl.js.map

package/lib/makeUrl.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"makeUrl.js","sourceRoot":"","sources":["../src/makeUrl.ts"],"names":[],"mappings":"AAAA;;;GAGG;AACH,MAAM,UAAU,OAAO,CAAC,KAAa,EAAE,IAAmB;IACxD,IAAI,CAAC;QACH,mFAAmF;QACnF,OAAO,IAAI,GAAG,CAAC,KAAK,EAAE,IAAI,CAAC,CAAC;IAC9B,CAAC;IAAC,MAAM,CAAC;QACP,MAAM,IAAI,KAAK,CAAC,iBAAiB,KAAK,aAAa,IAAI,IAAI,EAAE,GAAG,CAAC,CAAC;IACpE,CAAC;AACH,CAAC","sourcesContent":["/**\n * Attempt to create a URL object from a string, and throw an informative error if it fails.\n * (The URL constructor throws a very generic error.)\n */\nexport function makeUrl(input: string, base?: string | URL) {\n  try {\n    // eslint-disable-next-line no-restricted-syntax -- this is the function definition\n    return new URL(input, base);\n  } catch {\n    throw new Error(`Invalid URL: \"${input}\", base: \"${base || ''}\"`);\n  }\n}\n"]}

package/lib/normalizeRelativePath.d.ts

@@ -1,7 +1,10 @@
 /**
  * Normalize a path:
- * - If it's empty, undefined, or `'.'`, return `'.'`.
+ * - If it's empty, undefined, or `.`, return `.`.
  * - If it's relative, normalize slashes and ensure it starts with `./`.
+ * - If it starts with `/`, add a leading `.` (this is an absolute path but is sometimes treated
+ *   as package-relative)
+ * - If it's an absolute Windows path (starts with a drive letter), throw an error.
  * - If it's absolute, throw an error (since this likely indicates a bug in the calling code).
  *
  * Note that this will NOT remove a trailing slash (if present).

package/lib/normalizeRelativePath.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"normalizeRelativePath.d.ts","sourceRoot":"","sources":["../src/normalizeRelativePath.ts"],"names":[],"mappings":"AAEA;;;;;;;GAOG;AACH,wBAAgB,qBAAqB,CAAC,YAAY,EAAE,MAAM,GAAG,SAAS,GAAG,MAAM,CAgB9E"}
+{"version":3,"file":"normalizeRelativePath.d.ts","sourceRoot":"","sources":["../src/normalizeRelativePath.ts"],"names":[],"mappings":"AAEA;;;;;;;;;;GAUG;AACH,wBAAgB,qBAAqB,CAAC,YAAY,EAAE,MAAM,GAAG,SAAS,GAAG,MAAM,CA6B9E"}

package/lib/normalizeRelativePath.js

@@ -1,8 +1,11 @@
 import { slash } from './slash.js';
 /**
  * Normalize a path:
- * - If it's empty, undefined, or `'.'`, return `'.'`.
+ * - If it's empty, undefined, or `.`, return `.`.
  * - If it's relative, normalize slashes and ensure it starts with `./`.
+ * - If it starts with `/`, add a leading `.` (this is an absolute path but is sometimes treated
+ *   as package-relative)
+ * - If it's an absolute Windows path (starts with a drive letter), throw an error.
  * - If it's absolute, throw an error (since this likely indicates a bug in the calling code).
  *
  * Note that this will NOT remove a trailing slash (if present).
@@ -11,14 +14,24 @@ export function normalizeRelativePath(originalPath) {
     if (!originalPath || originalPath === '.') {
         return '.';
     }
-    // Throw if the path is absolute.
-    // NOTE: Don't use the `path` module because this function is used by `parseImportString`, which
-    // is used in the browser by the overlay.
-    if (originalPath[0] === '/' || /^[a-zA-Z]:[\\/]/.test(originalPath)) {
-        throw new Error(`Expected a relative path, but received an absolute path: ${originalPath}`);
+    // Throw on an absolute Windows path: this is definitely absolute regardless of the context it
+    // came from, and if the calling code attempts to join it to another path as if it was relative,
+    // that will have bad results. It's more likely on Windows that we could unintentionally end up
+    // with an absolute path since there's no way to do relative paths across drive letters
+    // (path.relative() will return an absolute path), so we throw here to make it clear that
+    // Cloudpack needs to have specific handling for any cases where that might be possible.
+    if (/^[a-zA-Z]:[\\/]/.test(originalPath)) {
+        throw new Error(`Received an absolute path where a relative path was expected: ${originalPath}.\n` +
+            'This indicates a bug in Cloudpack, likely due to calling path.relative() with two Windows paths ' +
+            'on different drives (which returns an absolute path) and then assuming the result is relative.');
     }
     // Ensure we have the right slashes.
-    originalPath = slash(originalPath);
-    return originalPath.startsWith('./') ? originalPath : `./${originalPath}`;
+    const normalizedPath = slash(originalPath);
+    if (normalizedPath[0] === '/') {
+        // Technically a path starting with / is absolute, but it's possible that package.json
+        // entry points like "main" could also use a leading slash to indicate the package root.
+        return '.' + normalizedPath;
+    }
+    return normalizedPath.startsWith('./') ? normalizedPath : `./${normalizedPath}`;
 }
 //# sourceMappingURL=normalizeRelativePath.js.map

package/lib/normalizeRelativePath.js.map

@@ -1 +1 @@
-{"version":3,"file":"normalizeRelativePath.js","sourceRoot":"","sources":["../src/normalizeRelativePath.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,KAAK,EAAE,MAAM,YAAY,CAAC;AAEnC;;;;;;;GAOG;AACH,MAAM,UAAU,qBAAqB,CAAC,YAAgC;IACpE,IAAI,CAAC,YAAY,IAAI,YAAY,KAAK,GAAG,EAAE,CAAC;QAC1C,OAAO,GAAG,CAAC;IACb,CAAC;IAED,iCAAiC;IACjC,gGAAgG;IAChG,yCAAyC;IACzC,IAAI,YAAY,CAAC,CAAC,CAAC,KAAK,GAAG,IAAI,iBAAiB,CAAC,IAAI,CAAC,YAAY,CAAC,EAAE,CAAC;QACpE,MAAM,IAAI,KAAK,CAAC,4DAA4D,YAAY,EAAE,CAAC,CAAC;IAC9F,CAAC;IAED,oCAAoC;IACpC,YAAY,GAAG,KAAK,CAAC,YAAY,CAAC,CAAC;IAEnC,OAAO,YAAY,CAAC,UAAU,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,KAAK,YAAY,EAAE,CAAC;AAC5E,CAAC","sourcesContent":["import { slash } from './slash.js';\n\n/**\n * Normalize a path:\n * - If it's empty, undefined, or `'.'`, return `'.'`.\n * - If it's relative, normalize slashes and ensure it starts with `./`.\n * - If it's absolute, throw an error (since this likely indicates a bug in the calling code).\n *\n * Note that this will NOT remove a trailing slash (if present).\n */\nexport function normalizeRelativePath(originalPath: string | undefined): string {\n  if (!originalPath || originalPath === '.') {\n    return '.';\n  }\n\n  // Throw if the path is absolute.\n  // NOTE: Don't use the `path` module because this function is used by `parseImportString`, which\n  // is used in the browser by the overlay.\n  if (originalPath[0] === '/' || /^[a-zA-Z]:[\\\\/]/.test(originalPath)) {\n    throw new Error(`Expected a relative path, but received an absolute path: ${originalPath}`);\n  }\n\n  // Ensure we have the right slashes.\n  originalPath = slash(originalPath);\n\n  return originalPath.startsWith('./') ? originalPath : `./${originalPath}`;\n}\n"]}
+{"version":3,"file":"normalizeRelativePath.js","sourceRoot":"","sources":["../src/normalizeRelativePath.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,KAAK,EAAE,MAAM,YAAY,CAAC;AAEnC;;;;;;;;;;GAUG;AACH,MAAM,UAAU,qBAAqB,CAAC,YAAgC;IACpE,IAAI,CAAC,YAAY,IAAI,YAAY,KAAK,GAAG,EAAE,CAAC;QAC1C,OAAO,GAAG,CAAC;IACb,CAAC;IAED,8FAA8F;IAC9F,gGAAgG;IAChG,+FAA+F;IAC/F,uFAAuF;IACvF,yFAAyF;IACzF,wFAAwF;IACxF,IAAI,iBAAiB,CAAC,IAAI,CAAC,YAAY,CAAC,EAAE,CAAC;QACzC,MAAM,IAAI,KAAK,CACb,iEAAiE,YAAY,KAAK;YAChF,kGAAkG;YAClG,gGAAgG,CACnG,CAAC;IACJ,CAAC;IAED,oCAAoC;IACpC,MAAM,cAAc,GAAG,KAAK,CAAC,YAAY,CAAC,CAAC;IAE3C,IAAI,cAAc,CAAC,CAAC,CAAC,KAAK,GAAG,EAAE,CAAC;QAC9B,sFAAsF;QACtF,wFAAwF;QACxF,OAAO,GAAG,GAAG,cAAc,CAAC;IAC9B,CAAC;IAED,OAAO,cAAc,CAAC,UAAU,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC,cAAc,CAAC,CAAC,CAAC,KAAK,cAAc,EAAE,CAAC;AAClF,CAAC","sourcesContent":["import { slash } from './slash.js';\n\n/**\n * Normalize a path:\n * - If it's empty, undefined, or `.`, return `.`.\n * - If it's relative, normalize slashes and ensure it starts with `./`.\n * - If it starts with `/`, add a leading `.` (this is an absolute path but is sometimes treated\n *   as package-relative)\n * - If it's an absolute Windows path (starts with a drive letter), throw an error.\n * - If it's absolute, throw an error (since this likely indicates a bug in the calling code).\n *\n * Note that this will NOT remove a trailing slash (if present).\n */\nexport function normalizeRelativePath(originalPath: string | undefined): string {\n  if (!originalPath || originalPath === '.') {\n    return '.';\n  }\n\n  // Throw on an absolute Windows path: this is definitely absolute regardless of the context it\n  // came from, and if the calling code attempts to join it to another path as if it was relative,\n  // that will have bad results. It's more likely on Windows that we could unintentionally end up\n  // with an absolute path since there's no way to do relative paths across drive letters\n  // (path.relative() will return an absolute path), so we throw here to make it clear that\n  // Cloudpack needs to have specific handling for any cases where that might be possible.\n  if (/^[a-zA-Z]:[\\\\/]/.test(originalPath)) {\n    throw new Error(\n      `Received an absolute path where a relative path was expected: ${originalPath}.\\n` +\n        'This indicates a bug in Cloudpack, likely due to calling path.relative() with two Windows paths ' +\n        'on different drives (which returns an absolute path) and then assuming the result is relative.',\n    );\n  }\n\n  // Ensure we have the right slashes.\n  const normalizedPath = slash(originalPath);\n\n  if (normalizedPath[0] === '/') {\n    // Technically a path starting with / is absolute, but it's possible that package.json\n    // entry points like \"main\" could also use a leading slash to indicate the package root.\n    return '.' + normalizedPath;\n  }\n\n  return normalizedPath.startsWith('./') ? normalizedPath : `./${normalizedPath}`;\n}\n"]}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ms-cloudpack/path-string-parsing",
-  "version": "1.2.0",
+  "version": "1.2.2",
   "description": "Common path-related string parsing utilities for Cloudpack",
   "license": "MIT",
   "type": "module",
@@ -24,7 +24,8 @@
     "test": "cloudpack-scripts test"
   },
   "devDependencies": {
-    "@ms-cloudpack/eslint-plugin-internal": "*"
+    "@ms-cloudpack/eslint-plugin-internal": "^0.0.1",
+    "@ms-cloudpack/scripts": "^0.0.1"
   },
   "files": [
     "lib/**/!(*.test.*)"