Update detsys-ts: nix store trust: accept Determinate Nix 3.15 and newer, which uses a bool instead of int (#127)

[detsys-pr-bot]

Bump detsys-ts to the latest HEAD and regenerate application bundle.

• Source ref: 4b33fd06d268e376dd34c17d6ec0d4ebf01d02a9
• Source pull request: DeterminateSystems/detsys-ts#127
• Source diff: f335f0d0939f2b8488174d3cd5f818d1cec23ab6..4b33fd06d268e376dd34c17d6ec0d4ebf01d02a9
• Diff file: 127.diff

Summary by CodeRabbit

• Chores
  • Updated development dependencies to maintain compatibility and receive latest patches.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

📝 Walkthrough

Walkthrough

This change bumps the @types/node devDependency from version ^20.19.26 to ^20.19.27 in package.json, representing a patch-level update with no runtime behavior changes.

Changes

Cohort / File(s)	Summary
Dependency Version Bump package.json	Patch version update for @types/node devDependency: ^20.19.26 → ^20.19.27

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

• Update detsys-ts: Merge pull request #109 from DeterminateSystems/tsup-to-tsdown #86: Modifies the same @types/node devDependency version constraint in package.json
• Update detsys-ts: Merge pull request #116 from DeterminateSystems/dependabot/github_actions/actions-deps-76468cb07f #88: Updates devDependencies in package.json with a specific focus on @types/node version changes

Suggested reviewers

• lucperkins

Poem

🐰 A whisker-thin bump, a patch so precise,
Types flow more sweetly, oh my, how nice!
From point twenty-six to twenty-seven we hop,
Our node types refined, we'll never stop! ✨

Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Title check	⚠️ Warning	The PR title references 'detsys-ts' and Nix store trust changes, but the actual changeset only shows a minor @types/node version bump in package.json with no observable behavioral changes.	The title describes updates to detsys-ts dependency handling Nix 3.15 changes, but the raw summary shows only a patch-level devDependency update. Clarify if this is a detsys-ts dependency bump or update the title to reflect the actual package.json change.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

---

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: 0

🧹 Nitpick comments (1)

package.json (1)

29-29: Consider pinning detsys-ts to a specific commit for reproducibility.

The detsys-ts dependency uses an unpinned GitHub URL reference. While the lockfile will resolve this to a specific commit, explicitly pinning it in package.json improves clarity and ensures intentional updates.

🔎 Proposed fix to pin detsys-ts to specific commit

-    "detsys-ts": "github:DeterminateSystems/detsys-ts"
+    "detsys-ts": "github:DeterminateSystems/detsys-ts#4b33fd06d268e376dd34c17d6ec0d4ebf01d02a9"

This makes the dependency version explicit and aligns with the PR objectives.

📜 Review details

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 8f0e778 and 49f26ec.

⛔ Files ignored due to path filters (2)

• dist/index.js is excluded by !**/dist/**
• pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml

📒 Files selected for processing (1)

• package.json

⏰ 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). (1)

• GitHub Check: Build and check the TypeScript Action

🔇 Additional comments (2)

package.json (2)

33-33: LGTM: Routine patch update.

The @types/node bump from ^20.19.26 to ^20.19.27 is a routine patch-level update with no breaking changes expected.

---

1-45: Lockfile correctly updated with expected detsys-ts commit.

The pnpm-lock.yaml file contains the correct detsys-ts resolution: github.com/DeterminateSystems/detsys-ts/4b33fd06d268e376dd34c17d6ec0d4ebf01d02a9. The unpinned GitHub URL in package.json is the correct approach for pnpm; dependency resolution to the specific commit happens via the lockfile, which has been properly updated. The PR successfully implements the intended detsys-ts update for Nix store trust changes (bool vs int in Determinate Nix 3.15+).

Likely an incorrect or invalid review comment.