Fix resolveBreakpointPath treating absolute local paths as server paths

Absolute local paths like /Users/.../app_mysite/cartridge/controllers/Loyalty.js
start with / and were returned as-is (treated as server paths) before
the source mapper had a chance to map them. Now the source mapper runs
first — if it matches, the local path is correctly converted to a
server path. The / passthrough only applies if the mapper doesn't match.

Author: clavery

packages/b2c-tooling-sdk/src/operations/debug/resolve-path.ts

@@ -12,9 +12,9 @@ import type {CartridgeMapping} from '../code/cartridges.js';
  * Resolve a user-provided file path to an SDAPI script_path.
  *
  * Accepts:
- *   - Server path: `/app_storefront/cartridge/controllers/Cart.js` (passed through)
  *   - Absolute local path: `/Users/.../app_storefront/cartridge/controllers/Cart.js`
  *   - Relative local path: `./cartridges/app_storefront/cartridge/controllers/Cart.js`
+ *   - Server path: `/app_storefront/cartridge/controllers/Cart.js` (passed through if not a local match)
  *   - Cartridge-name-prefixed: `app_storefront/cartridge/controllers/Cart.js` (prefixed with /)
  *
  * Returns the SDAPI script_path (e.g. `/app_storefront/cartridge/controllers/Cart.js`).
@@ -25,18 +25,21 @@ export function resolveBreakpointPath(
   sourceMapper: SourceMapper,
   cartridges: CartridgeMapping[],
 ): string {
-  if (input.startsWith('/')) {
-    return input;
-  }
-
-  // Try direct source mapper (handles absolute and resolvable relative paths)
+  // Always try source mapper first — handles both absolute and relative local paths
   const mapped = sourceMapper.toServerPath(input);
   if (mapped) return mapped;
 
-  // Try resolving relative to cwd
+  // Try resolving relative to cwd (for paths like ./cartridges/app_mysite/...)
   const resolved = path.resolve(input);
-  const mappedResolved = sourceMapper.toServerPath(resolved);
-  if (mappedResolved) return mappedResolved;
+  if (resolved !== input) {
+    const mappedResolved = sourceMapper.toServerPath(resolved);
+    if (mappedResolved) return mappedResolved;
+  }
+
+  // If it starts with / and the source mapper didn't match, treat as server path
+  if (input.startsWith('/')) {
+    return input;
+  }
 
   // Check if the input starts with a known cartridge name — treat as server path missing leading /
   const normalized = input.split(path.sep).join('/');
@@ -46,14 +49,15 @@ export function resolveBreakpointPath(
   }
 
   const cartridgeNames = cartridges.map((c) => c.name).join(', ');
-  const hint = cartridges.length > 0
-    ? `Known cartridges: ${cartridgeNames}\n` +
-      `Accepted forms:\n` +
-      `  /cartridge_name/cartridge/path/to/file.js  (server path)\n` +
-      `  cartridge_name/cartridge/path/to/file.js    (server path without leading /)\n` +
-      `  ./path/to/cartridge_name/cartridge/file.js  (relative local path)\n` +
-      `  /absolute/path/to/cartridge_name/file.js    (absolute local path)`
-    : 'No cartridges discovered. Use a server path: /cartridge_name/cartridge/path/file.js';
+  const hint =
+    cartridges.length > 0
+      ? `Known cartridges: ${cartridgeNames}\n` +
+        `Accepted forms:\n` +
+        `  /cartridge_name/cartridge/path/to/file.js  (server path)\n` +
+        `  cartridge_name/cartridge/path/to/file.js    (server path without leading /)\n` +
+        `  ./path/to/cartridge_name/cartridge/file.js  (relative local path)\n` +
+        `  /absolute/path/to/cartridge_name/file.js    (absolute local path)`
+      : 'No cartridges discovered. Use a server path: /cartridge_name/cartridge/path/file.js';
 
   throw new Error(`Cannot resolve "${input}" to a server script path.\n${hint}`);
 }