fallback uri processing

Author: RubenHalman

README.md

@@ -368,33 +368,40 @@ npm install @flow-scanner/lightning-flow-scanner-core
    npm run test
    ```
 
-5. Deploy Demo Flows (Optional):
+5. Testing the module locally(Optional):
 
-```bash
-cd assets/example-flows && sf project deploy start &&
-```
+   To link the module, run:
 
-Navigate to the [Demo Readme](assets\example-flows\README.md) for full details
+   ```bash
+   npm run link
+   ```
 
-6. Test as local dependency(Optional):
-   a. run:
+   a. Ad-Hoc Testing with node:
 
    ```bash
    npm run link
    ```
 
-   b. Go to the dependent project (e.g. VSX or CLI) and use:
+   b. Test in a dependent project (e.g. VSX or CLI):
 
    ```bash
    npm link @flow-scanner/lightning-flow-scanner-core
    ```
 
    Your local module will now replace any installed version and update on rebuild.
 
+6. Deploy Demo Flows (Optional):
+
+   ```bash
+   cd assets/example-flows && sf project deploy start &&
+   ```
+
+   Navigate to the [Demo Readme](assets\example-flows\README.md) for full details
+
 7. Create a standalone UMD Module(Optional):
 
-```bash
-  npm run vite:dist // creates UMD at`dist/lightning-flow-scanner-core.umd.js`.
-```
+   ```bash
+     npm run vite:dist // creates UMD at`dist/lightning-flow-scanner-core.umd.js`.
+   ```
 
 <p><strong>Want to help improve Lightning Flow Scanner? See our <a href="https://github.com/Flow-Scanner/lightning-flow-scanner-core?tab=contributing-ov-file">Contributing Guidelines</a></strong></p>

src/main/libs/exportAsSarif.ts

@@ -1,7 +1,6 @@
 import { Flow } from "../models/Flow";
 import { ScanResult } from "../models/ScanResult";
 import { Violation } from "../models/Violation";
-import * as p from "path";  // For relative path fallback
 
 export function exportSarif(results: ScanResult[]): string {
   const runs = results.map((result) => {
@@ -53,15 +52,20 @@ export function exportSarif(results: ScanResult[]): string {
 }
 
 function getUri(flow: Flow): string {
-  // Prioritize uri (virtual/relative from input, e.g., for browser)
+  // Prefer uri (works in both browser and Node)
   if (flow.uri) {
     return flow.uri.replace(/\\/g, "/");
   }
-  // Fallback to relative fsPath for Node
+  
+  // Node only: fsPath is only set in Node environments
   if (flow.fsPath) {
-    return p.relative(process.cwd(), flow.fsPath).replace(/\\/g, "/");
+    const match = flow.fsPath.match(/(?:force-app|src)\/.+$/);
+    if (match) {
+      return match[0].replace(/\\/g, "/");
+    }
+    return flow.fsPath.replace(/\\/g, "/");
   }
-  // Final generic fallback
+  
   return `flows/${flow.name}.flow-meta.xml`;
 }
 

src/main/models/Flow.ts

@@ -69,7 +69,13 @@ export class Flow {
   constructor(path?: string, data?: unknown) {
     if (path) {
       this.uri = path;  // Always set general URI from input (file path or virtual)
-      this.fsPath = p.resolve(path);  // Resolve for Node; test can override to undefined for browser sim
+      
+      // Only resolve fsPath in Node.js environments
+      // In browser with polyfills, fsPath stays undefined
+      if (typeof process !== 'undefined' && process.cwd) {
+        this.fsPath = p.resolve(path);
+      }
+      
       let flowName = p.basename(p.basename(path), p.extname(path));
       if (flowName.includes(".")) {
         flowName = flowName.split(".")[0];