Add determinateNixd.telemetry.sentry.endpoint

[cpick]

Add the missing telemetry Determinate Nixd
option.

Summary by CodeRabbit

• New Features
  • Added a configurable Sentry endpoint for crash-reporting telemetry. Default is "unset"; set a custom endpoint to enable reporting or set to null to disable. When configured, the endpoint is included in the generated Determinate Nixd configuration.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 0b547279-739d-4f59-83f3-1bd7228ab1f6

📥 Commits

Reviewing files that changed from the base of the PR and between 9184cc7 and 4e85b3e.

📒 Files selected for processing (1)

• modules/nix-darwin/default.nix

---

📝 Walkthrough

Walkthrough

The PR adds a new telemetry.sentry.endpoint configuration option to the nix-darwin Determinate Nixd module. The option declaration provides a nullable string type with an "unset" default, and the config generation logic conditionally includes the endpoint in the serialized JSON only when explicitly configured.

Changes

Sentry Telemetry Configuration

Layer / File(s)	Summary
Telemetry endpoint option declaration modules/nix-darwin/default.nix	New telemetry.sentry.endpoint option added to the submodule with type nullOr str and default "unset" for configuring Sentry crash reporting endpoints.
Conditional Sentry endpoint serialization modules/nix-darwin/default.nix	Config.json generation logic updated to merge telemetry.sentry.endpoint into the output JSON only when the value is not "unset".

---

🎯 2 (Simple) | ⏱️ ~10 minutes

---

🐰 A Sentry endpoint now hops in place,
Conditional and neat in this Nix-Darwin space,
Telemetry configured with grace,
When not "unset"—it finds its place! 🎯

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly and specifically describes the main change: adding a new determinateNixd.telemetry.sentry.endpoint option to the nix-darwin module.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

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.}

[coderabbitai]

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@modules/nix-darwin/default.nix`:
- Around line 789-795: The current mkIf checks explicitlySetAttrs != {} so a
config containing only dnixd.telemetry.sentry.endpoint is never written; instead
build the final attribute set first (e.g. finalAttrs = explicitlySetAttrs //
lib.optionalAttrs (dnixd.telemetry.sentry.endpoint != "unset") {
telemetry.sentry.endpoint = dnixd.telemetry.sentry.endpoint; }) and then use
lib.mkIf (finalAttrs != { }) and set text = builtins.toJSON finalAttrs (update
the block that defines text/builtins.toJSON and the mkIf condition to reference
finalAttrs rather than explicitlySetAttrs).

🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

• Push a commit to this branch (recommended)
• Create a new PR with the fixes

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 22f800c0-80b8-412a-8bef-832dd7985d9b

📥 Commits

Reviewing files that changed from the base of the PR and between 8a9c57e and 9184cc7.

📒 Files selected for processing (1)

• modules/nix-darwin/default.nix

[lucperkins]

Awesome, thanks a lot @cpick!