src/index.ts (3)

50-50: Please document the new fact

Add a short note in the README/docs explaining the new nix_version fact (format, when it’s set, possible value "unknown").

I can draft the doc blurb if you want.

---

517-519: Move the second nix-version probe to finally to capture error paths

If main() throws, the post-main() probe won’t run. Move it to finally (guarded by isMain) so shutdown/exception events also have the updated version.

Apply this removal near the end of the main block:

-        // Run the preflight of the nix version a second time so our "shutdown" events have updated version info.
-        await this.preflightNixVersion();

Then insert before await this.complete(); in the finally block:

      // Ensure shutdown/exception events have the latest nix version too
      if (this.isMain) {
        try {
          await this.preflightNixVersion();
        } catch {
          // ignore
        }
      }

Also applies to: 524-526

---

1113-1130: Normalize the captured version to just the numeric component

nix --version often returns strings like “nix (Nix) 2.24.0”. Storing only the version number makes downstream analytics/filters simpler while keeping compatibility when the command output changes slightly.

-    this.addFact(FACT_NIX_VERSION, output);
+    const match = output.match(/\b\d+(?:\.\d+)+\b/);
+    this.addFact(FACT_NIX_VERSION, match?.[0] ?? output);

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

💡 Knowledge Base configuration:

• MCP integration is disabled by default for public repositories
• Jira integration is disabled by default for public repositories
• Linear integration is disabled by default for public repositories

You can enable these sources in your CodeRabbit configuration.

src/index.ts (2)

524-526: Run the “final” nix_version capture even on failure.

Right now, if main() throws, the second capture won’t run. Prefer moving this to finally so shutdown/exception events always include the latest version.

Apply this diff:

@@
       if (this.isMain) {
         await this.main();
-
-        // Run the preflight of the nix version a second time so our "shutdown" events have updated version info.
-        await this.preflightNixVersion();
       } else if (this.isPost) {
         await this.post();
       }
@@
     } finally {
+      // Ensure shutdown/exception events carry the latest Nix version.
+      await this.preflightNixVersion();
       if (this.isPost) {
         await this.collectBacktraces();
       }
 
       await this.complete();
     }

---

1113-1131: Also record a parsed/normalized version for easier analytics.

Keep nix_version as-is, but additionally extract a semver-like token when present. This avoids downstream parsing in analytics queries.

Apply these diffs:

@@
-const FACT_NIX_VERSION = "nix_version";
+const FACT_NIX_VERSION = "nix_version";
+const FACT_NIX_VERSION_SEMVER = "nix_version_semver";

   private async preflightNixVersion(): Promise<void> {
     let output = "unknown";
 
     try {
       ({ stdout: output } = await actionsExec.getExecOutput(
         "nix",
         ["--version"],
         {
           silent: true,
         },
       ));
       output = output.trim() || "unknown";
     } catch {
       // That's fine.
     }
 
     this.addFact(FACT_NIX_VERSION, output);
+    // Try to pull out a semver-ish token, e.g., "2.24.2" from "nix (Nix) 2.24.2".
+    const m = output.match(/\b\d+\.\d+\.\d+(?:[0-9A-Za-z.-]*)?\b/);
+    if (m?.[0]) {
+      this.addFact(FACT_NIX_VERSION_SEMVER, m[0]);
+    }
   }

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

💡 Knowledge Base configuration:

• MCP integration is disabled by default for public repositories
• Jira integration is disabled by default for public repositories
• Linear integration is disabled by default for public repositories

You can enable these sources in your CodeRabbit configuration.