Skip to content

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

Merged
grahamc merged 2 commits intoDeterminateSystems:mainfrom
detsys-pr-bot:detsys-ts-update-4b33fd06d268e376dd34c17d6ec0d4ebf01d02a9
Jan 7, 2026
Merged

Update detsys-ts: nix store trust: accept Determinate Nix 3.15 and newer, which uses a bool instead of int (#127)#163
grahamc merged 2 commits intoDeterminateSystems:mainfrom
detsys-pr-bot:detsys-ts-update-4b33fd06d268e376dd34c17d6ec0d4ebf01d02a9

Conversation

@detsys-pr-bot
Copy link
Copy Markdown
Contributor

@detsys-pr-bot detsys-pr-bot commented Jan 7, 2026

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

Summary by CodeRabbit

  • Chores
    • Updated CI/CD runner environment configuration.
    • Updated development dependencies to latest patch versions.

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

… and newer, which uses a bool instead of int (DeterminateSystems#127)` (`4b33fd06d268e376dd34c17d6ec0d4ebf01d02a9`)
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Jan 7, 2026

📝 Walkthrough

Walkthrough

Updated GitHub Actions CI runner configuration from macos-13 to macos-15-intel for x86_64-darwin builds, and bumped three package.json dependencies: got (^14.6.6), @types/node (^20.19.27), and prettier (^3.7.4).

Changes

Cohort / File(s) Summary
CI Configuration
.github/workflows/ci.yml
Updated macOS runner matrix entry from macos-13 to macos-15-intel for x86_64-darwin target
Dependency Updates
package.json
Bumped three dependencies: got (^14.6.5 → ^14.6.6), @types/node (^20.19.25 → ^20.19.27), prettier (^3.7.3 → ^3.7.4)

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Suggested reviewers

  • lucperkins

Poem

🐰 Hops of joy, I must say,
With newer runners and deps on their way,
Macos fifteen hops so fine,
Got, Node types, Prettier align,
All bumped up—hooray, hooray!

Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 inconclusive)
Check name Status Explanation Resolution
Title check ❓ Inconclusive The PR title mentions updating detsys-ts for Determinate Nix 3.15, but the changeset primarily shows dependency version bumps in package.json and a macOS runner update in CI—not the specific Nix store trust bool/int handling change described. Verify whether the title accurately represents the main change. If the primary focus is dependency updates and CI changes, consider updating the title to reflect that. If the title is meant to convey the upstream detsys-ts change, clarify why the changeset doesn't directly show implementation of that 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.


📜 Recent review details

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between b827652 and ff5feef.

⛔ 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 (2)
  • .github/workflows/ci.yml
  • 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
🔇 Additional comments (2)
.github/workflows/ci.yml (1)

94-94: The macos-15-intel runner is available and stable for use.

GitHub's hosted macOS 15 Intel runner is officially available with support through August 2027, confirming this update aligns with the PR objectives and provides a solid, long-term-supported migration path.

package.json (1)

33-33: All dependency versions verified and no known vulnerabilities found.

The versions got@14.6.6, @types/node@20.19.27, and prettier@3.7.4 all exist in the npm registry and have no reported security vulnerabilities. These are routine patch-level updates.


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

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

@grahamc grahamc added this pull request to the merge queue Jan 7, 2026
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Jan 7, 2026
@grahamc grahamc enabled auto-merge January 7, 2026 04:35
@grahamc grahamc added this pull request to the merge queue Jan 7, 2026
Merged via the queue into DeterminateSystems:main with commit 3da28ac Jan 7, 2026
6 checks passed
@coderabbitai coderabbitai Bot mentioned this pull request Mar 25, 2026
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants