Apply fixups from CodeRabbit review

[gustavderdrache]

Description

This PR applies fixups for issues identified during automated review of #199.

Checklist

• Tested changes against a test repository
• Added or updated relevant documentation (leave unchecked if not applicable)
• (If this PR is for a release) Updated README to point to the new tag (leave unchecked if not applicable)

Summary by CodeRabbit

• Bug Fixes
  • Improved Nix installer environment configuration to prevent setup failures in certain system setups.
• Chores
  • Renamed the Nix installation success tracking event to align with current analytics conventions.
  • Captures the workflow start time earlier to provide more accurate installation duration metrics.

[coderabbitai]

Walkthrough

Updates in src/index.ts: renamed the Nix install success event to "install_nix_success"; moved saving STATE_START_DATETIME to the start of main() using actionsCore.saveState; and corrected the Nix installer environment variable to use nixBuildUserBase for NIX_INSTALLER_NIX_BUILD_USER_ID_BASE.

Changes

Cohort / File(s)	Summary
Event and state handling updates src/index.ts	Renamed success event to "install_nix_success"; moved STATE_START_DATETIME save to start of main(); set NIX_INSTALLER_NIX_BUILD_USER_ID_BASE from nixBuildUserBase instead of nixBuildUserCount.

Sequence Diagram(s)

sequenceDiagram
  autonumber
  actor GH as GitHub Action
  participant Main as main()
  participant Core as actionsCore
  participant Installer as NixInstaller
  participant Events as EventEmitter/Telemetry

  GH->>Main: Start action
  rect rgba(220,240,255,0.5)
    note right of Main: Save start timestamp earlier (changed)
    Main->>Core: saveState(STATE_START_DATETIME, now)
  end

  Main->>Installer: run({ env: NIX_INSTALLER_NIX_BUILD_USER_ID_BASE = nixBuildUserBase }) 
  note right of Installer: Env source corrected (changed)

  Installer-->>Main: Success
  rect rgba(230,255,230,0.5)
    note right of Main: Emit renamed success event
    Main->>Events: emit("install_nix_success")
  end

  alt Installer fails
    Installer-->>Main: Error
    Main->>Events: emit("install_nix_failure")
  end

Loading

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Pre-merge checks (3 passed)

✅ Passed checks (3 passed)

Check name	Status	Explanation
Title Check	✅ Passed	The pull request title “Apply fixups from CodeRabbit review” correctly indicates that the changes address issues identified during an automated review and is related to the changeset. Although it refers to the overall corrective action rather than detailing specific changes, it remains accurate and relevant to the modifications made.
Description Check	✅ Passed	The pull request description clearly outlines that this PR applies fixups from the automated review of PR #199 and includes a checklist for testing and documentation updates. It directly relates to the changes in this branch and provides context for reviewers, satisfying the lenient description requirements.
Docstring Coverage	✅ Passed	No functions found in the changes. Docstring coverage check skipped.

Poem

I twitched my ears at dawn’s first byte,
Renamed success to set it right.
A timestamp saved at journey’s start,
Env vars aligned with engineer’s art.
Hop-hop—clean runs, no second guess,
Nix installs sing: success, success! 🐇✨

---

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between f161ab0 and cc68acb.

⛔ Files ignored due to path filters (1)

• dist/index.js is excluded by !**/dist/**

📒 Files selected for processing (1)

• src/index.ts (3 hunks)

🧰 Additional context used

🧬 Code graph analysis (1)

src/index.ts (1)

dist/index.js (2)

• executionEnv (100125-100125)
• executionEnv (100278-100278)

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (9)

• GitHub Check: Test: ubuntu-latest
• GitHub Check: Test: macos-14-xlarge
• GitHub Check: Test: macos-14-large with determinate
• GitHub Check: Test: macos-14-xlarge with determinate
• GitHub Check: Test: nscloud-ubuntu-22.04-amd64-4x16
• GitHub Check: Test: namespace-profile-default-arm64
• GitHub Check: Test: macos-14-large
• GitHub Check: Test: macos-13-large
• GitHub Check: Test: macos-13-large with determinate

🔇 Additional comments (3)

src/index.ts (3)

144-145: Saving STATE_START_DATETIME at the start of main() is the right move.

Ensures post-run reports/annotations have a stable lower bound even on early exits.

---

288-289: Correct env mapping verified. Input and docs naming are consistent; no drift detected.

---

23-23: Code-level update is correct; verify downstream consumers. No mismatches found in src/index.ts; please confirm any dashboards, analytics pipelines, or documentation expecting the old install_nix_success string are updated.

✨ Finishing Touches

• 📝 Generate Docstrings

🧪 Generate unit tests

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch gustavderdrache/push-wusxkxnkvqol

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}