squally-reporter 0.1.0 → 0.2.0

package/dist/index.js

@@ -53,6 +53,39 @@ function detectCommitSha() {
         return null;
     }
 }
+// GitLab CI exposes the full commit message directly (CI_COMMIT_MESSAGE).
+// GitHub Actions has no equivalent env var, so it falls through to the git
+// fallback below same as local runs - actions/checkout always leaves at
+// least the HEAD commit available, even at fetch depth 1.
+function detectCommitMessage() {
+    if (process.env.CI_COMMIT_MESSAGE)
+        return process.env.CI_COMMIT_MESSAGE;
+    try {
+        return (0, node_child_process_1.execSync)("git log -1 --pretty=%B").toString().trim();
+    }
+    catch {
+        return null;
+    }
+}
+// GitHub Actions' GITHUB_ACTOR is the user who triggered the workflow run
+// (a username, not necessarily the commit author, but the closest thing
+// GitHub exposes without shelling out). GitLab's CI_COMMIT_AUTHOR is
+// "Name <email>" - only the name part is kept. Falls back to the actual
+// git commit author for everything else (other CI providers, local runs).
+function detectAuthorName() {
+    if (process.env.GITHUB_ACTOR)
+        return process.env.GITHUB_ACTOR;
+    if (process.env.CI_COMMIT_AUTHOR) {
+        const match = process.env.CI_COMMIT_AUTHOR.match(/^(.*?)\s*<[^>]*>\s*$/);
+        return match ? match[1] : process.env.CI_COMMIT_AUTHOR;
+    }
+    try {
+        return (0, node_child_process_1.execSync)("git log -1 --pretty=%an").toString().trim();
+    }
+    catch {
+        return null;
+    }
+}
 // --- Flakiness Engine v2 (Phase A): tested application revision ---
 //
 // commitSha above is the test-automation repo's own commit. For teams whose
@@ -102,6 +135,30 @@ function detectEventType() {
         return process.env.CI_PIPELINE_SOURCE;
     return null;
 }
+// path.relative() joins with path.sep, which is "\" on Windows and "/" on
+// POSIX. Since testName+filePath is Squally's test identity, an
+// un-normalized filePath would make the same test look like two different
+// tests depending on which OS the run happened on - always normalize to
+// forward slashes regardless of host platform.
+function toPosixPath(p) {
+    return p.split(node_path_1.default.sep).join("/");
+}
+// Playwright captures stdout/stderr as two separate arrays (each internally
+// chronological, but not merged with each other), so a true byte-accurate
+// interleaving between the two streams isn't available here - stdout lines
+// come first, then stderr, each tagged with its source. No [warn] prefix:
+// Node itself writes console.warn to stderr right alongside console.error,
+// so that distinction isn't recoverable from this API either.
+function formatConsoleOutput(result) {
+    const lines = [];
+    for (const chunk of result.stdout) {
+        lines.push(`[log] ${chunk.toString().replace(/\s+$/, "")}`);
+    }
+    for (const chunk of result.stderr) {
+        lines.push(`[error] ${chunk.toString().replace(/\s+$/, "")}`);
+    }
+    return lines.length > 0 ? lines.join("\n") : null;
+}
 class SquallyReporter {
     constructor(options = {}) {
         this.startedAt = new Date();
@@ -122,12 +179,13 @@ class SquallyReporter {
         const error = result.errors[0];
         this.results.push({
             testName: test.titlePath().slice(1).join(" > "),
-            filePath: node_path_1.default.relative(process.cwd(), test.location.file),
+            filePath: toPosixPath(node_path_1.default.relative(process.cwd(), test.location.file)),
             status: mapStatus(result.status),
             durationMs: result.duration,
             retryCount: result.retry,
             errorMessage: error?.message ?? null,
             errorStack: error?.stack ?? null,
+            consoleOutput: formatConsoleOutput(result),
         });
     }
     async onEnd(result) {
@@ -135,6 +193,8 @@ class SquallyReporter {
         const payload = {
             branch: detectBranch(),
             commitSha: detectCommitSha(),
+            commitMessage: detectCommitMessage(),
+            authorName: detectAuthorName(),
             ciProvider: detectCiProvider(),
             status: result.status === "passed" ? "passed" : "failed",
             startedAt: this.startedAt.toISOString(),

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "squally-reporter",
-  "version": "0.1.0",
+  "version": "0.2.0",
   "description": "Playwright reporter that sends test results to your Squally dashboard.",
   "author": "Squally <squally.dev@gmail.com>",
   "keywords": [