@quenty/nevermore-cli 4.18.0 → 4.19.0

package/src/utils/testing/parsers/roblox-path-resolver.test.ts

@@ -0,0 +1,129 @@
+import { describe, it, expect } from 'vitest';
+import { resolveRobloxTestPath } from './roblox-path-resolver.js';
+import { SourcemapResolver } from '../../sourcemap/index.js';
+import type { SourcemapNode } from '../../sourcemap/index.js';
+
+describe('resolveRobloxTestPath', () => {
+  it('resolves a standard ServerScriptService path', () => {
+    expect(
+      resolveRobloxTestPath(
+        'ServerScriptService.observablecollection.Shared.ObservableList.spec'
+      )
+    ).toBe('src/observablecollection/src/Shared/ObservableList.spec.lua');
+  });
+
+  it('resolves a nested subdirectory path', () => {
+    expect(
+      resolveRobloxTestPath(
+        'ServerScriptService.maid.Shared.Maid.spec'
+      )
+    ).toBe('src/maid/src/Shared/Maid.spec.lua');
+  });
+
+  it('resolves a deeply nested path', () => {
+    expect(
+      resolveRobloxTestPath(
+        'ServerScriptService.blend.Client.Blend.Spring.SpringObject.spec'
+      )
+    ).toBe('src/blend/src/Client/Blend/Spring/SpringObject.spec.lua');
+  });
+
+  it('handles missing ServerScriptService prefix as fallback', () => {
+    expect(
+      resolveRobloxTestPath('observablecollection.Shared.ObservableList.spec')
+    ).toBe('src/observablecollection/src/Shared/ObservableList.spec.lua');
+  });
+
+  it('strips :LINE suffix', () => {
+    expect(
+      resolveRobloxTestPath(
+        'ServerScriptService.observablecollection.Shared.ObservableList.spec:45'
+      )
+    ).toBe('src/observablecollection/src/Shared/ObservableList.spec.lua');
+  });
+
+  it('handles path without ServerScriptService prefix and with :LINE suffix', () => {
+    expect(
+      resolveRobloxTestPath('maid.Shared.Maid.spec:23')
+    ).toBe('src/maid/src/Shared/Maid.spec.lua');
+  });
+
+  it('handles a bare package slug', () => {
+    expect(
+      resolveRobloxTestPath('ServerScriptService.maid')
+    ).toBe('src/maid/src');
+  });
+
+  it('handles a single-level spec path', () => {
+    expect(
+      resolveRobloxTestPath('ServerScriptService.maid.Maid.spec')
+    ).toBe('src/maid/src/Maid.spec.lua');
+  });
+
+  describe('with sourcemap resolver', () => {
+    const repoRoot = '/repo';
+
+    const sourcemap: SourcemapNode = {
+      name: 'Nevermore',
+      className: 'DataModel',
+      children: [
+        {
+          name: 'mypkg',
+          className: 'Folder',
+          filePaths: [`${repoRoot}/src/mypkg/default.project.json`],
+          children: [
+            {
+              name: 'Shared',
+              className: 'Folder',
+              children: [
+                {
+                  name: 'MyModule',
+                  className: 'ModuleScript',
+                  filePaths: [`${repoRoot}/src/mypkg/src/Shared/MyModule.lua`],
+                  children: [
+                    {
+                      name: 'MyModule.spec',
+                      className: 'ModuleScript',
+                      filePaths: [
+                        `${repoRoot}/src/mypkg/src/Shared/MyModule.spec.lua`,
+                      ],
+                    },
+                  ],
+                },
+              ],
+            },
+          ],
+        },
+      ],
+    };
+
+    const resolver = SourcemapResolver.fromSourcemap(sourcemap, repoRoot);
+
+    it('uses sourcemap when path is found', () => {
+      expect(
+        resolveRobloxTestPath(
+          'ServerScriptService.mypkg.Shared.MyModule.MyModule.spec',
+          resolver
+        )
+      ).toBe('src/mypkg/src/Shared/MyModule.spec.lua');
+    });
+
+    it('falls back to heuristic when sourcemap has no mapping', () => {
+      expect(
+        resolveRobloxTestPath(
+          'ServerScriptService.unknownpkg.Shared.Foo.spec',
+          resolver
+        )
+      ).toBe('src/unknownpkg/src/Shared/Foo.spec.lua');
+    });
+
+    it('strips :LINE suffix with sourcemap', () => {
+      expect(
+        resolveRobloxTestPath(
+          'ServerScriptService.mypkg.Shared.MyModule.MyModule.spec:42',
+          resolver
+        )
+      ).toBe('src/mypkg/src/Shared/MyModule.spec.lua');
+    });
+  });
+});

package/src/utils/testing/parsers/roblox-path-resolver.ts

@@ -0,0 +1,85 @@
+/**
+ * Converts Roblox instance paths from Jest-lua output back to
+ * repo-relative local filesystem paths.
+ *
+ * Two strategies:
+ *
+ * 1. **Sourcemap** (preferred) — Uses the authoritative `sourcemap.json`
+ *    mapping from `rojo sourcemap --absolute`. Exact and handles all
+ *    project layouts.
+ *
+ * 2. **Heuristic** (fallback) — Assumes `src/{slug}/src/` layout and
+ *    hard-codes `.spec`/`.story` as known dotted suffixes. Works when no
+ *    sourcemap is available.
+ */
+
+import type { SourcemapResolver } from '../../sourcemap/index.js';
+
+const SSS_PREFIX = 'ServerScriptService.';
+
+/**
+ * Resolve a Roblox instance path (from Jest-lua or a stack trace) to a
+ * repo-relative filesystem path.
+ *
+ * When a `sourcemapResolver` is provided, it is tried first. Falls back to
+ * the heuristic if the resolver doesn't have a mapping.
+ */
+export function resolveRobloxTestPath(
+  instancePath: string,
+  sourcemapResolver?: SourcemapResolver
+): string {
+  if (sourcemapResolver) {
+    const resolved = sourcemapResolver.resolve(instancePath);
+    if (resolved) return resolved;
+  }
+
+  return _resolveHeuristic(instancePath);
+}
+
+/**
+ * Heuristic fallback: assumes `src/{slug}/src/` layout.
+ *
+ * Handles optional `:LINE` suffixes and missing `ServerScriptService.` prefix.
+ */
+function _resolveHeuristic(instancePath: string): string {
+  // Strip :LINE suffix if present
+  let path = instancePath.replace(/:\d+$/, '');
+
+  // Strip ServerScriptService. prefix
+  if (path.startsWith(SSS_PREFIX)) {
+    path = path.slice(SSS_PREFIX.length);
+  }
+
+  // First dot-separated segment is the package slug
+  const dotIndex = path.indexOf('.');
+  if (dotIndex === -1) {
+    // Just a package name with no sub-path — return the src dir
+    return `src/${path}/src`;
+  }
+
+  const packageSlug = path.slice(0, dotIndex);
+  const remaining = path.slice(dotIndex + 1);
+
+  // Split into segments. In Rojo, dots in filenames (e.g. "Maid.spec.lua"
+  // becomes instance "Maid.spec") are ambiguous with the hierarchy separator.
+  // Rejoin known filename suffixes like ".spec" and ".story" with a dot
+  // instead of a slash.
+  const segments = remaining.split('.');
+  const rejoined: string[] = [];
+
+  for (let i = 0; i < segments.length; i++) {
+    const seg = segments[i];
+    // If this segment is a known file suffix, merge it with the previous
+    if (
+      i > 0 &&
+      (seg === 'spec' || seg === 'story')
+    ) {
+      rejoined[rejoined.length - 1] += `.${seg}`;
+    } else {
+      rejoined.push(seg);
+    }
+  }
+
+  const subPath = rejoined.join('/');
+  return `src/${packageSlug}/src/${subPath}.lua`;
+}

package/src/utils/testing/test-log-parser.ts

@@ -1,3 +1,5 @@
+import { OutputHelper } from '@quenty/cli-output-helpers';
+
 export interface ParsedTestLogs {
   success: boolean;
   logs: string;
@@ -8,7 +10,7 @@ export interface ParsedTestLogs {
  * Shared by both Open Cloud log fetching and local run-in-roblox output.
  */
 export function parseTestLogs(rawOutput: string): ParsedTestLogs {
-  const cleanLogs = rawOutput.replace(/\x1b\[[0-9;]*[a-zA-Z]/g, '');
+  const cleanLogs = OutputHelper.stripAnsi(rawOutput);
 
   // Check for Jest-style test failures
   const failedSuites = cleanLogs.match(/Test Suites:\s*(\d+)\s+failed/);