Test detsys-ts update

[grahamc]

Description

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

• Chores
  • Updated a project dependency to reference a specific commit/ref instead of an unqualified repository reference.
  • No changes to functionality, public APIs, or scripts; this is an internal dependency pinning with no user-visible behavior changes.

[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: 702b7b63-6b05-4b07-afff-a001421308c0

📥 Commits

Reviewing files that changed from the base of the PR and between 0a0f9c0 and a39d538.

⛔ Files ignored due to path filters (2)

• dist/index.js is excluded by !**/dist/**
• package-lock.json is excluded by !**/package-lock.json

📒 Files selected for processing (1)

• package.json

✅ Files skipped from review due to trivial changes (1)

• package.json

---

📝 Walkthrough

Walkthrough

The detsys-ts dependency in package.json was changed to reference a specific GitHub commit ref (#push-sqvzmxuokwlk) instead of the previous unqualified repository reference, pinning the dependency to that exact commit.

Changes

Cohort / File(s)	Summary
Dependency Update package.json	detsys-ts dependency updated to a specific GitHub commit ref (#push-sqvzmxuokwlk) replacing the unqualified repository reference.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A tiny hop through package land,
commits tied down by careful paw,
detsys-ts now knows its place,
fixed in time, no shifting race,
I twitch my nose and bless the law.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name	Status	Explanation	Resolution
Title check	❓ Inconclusive	The title 'Test detsys-ts update' is vague and does not clearly convey the specific nature of the change, using the generic term 'Test' which lacks meaningful information about what is being tested or updated.	Replace with a more specific title like 'Update detsys-ts dependency to specific GitHub commit' to clearly indicate the actual change being made.

✅ Passed checks (4 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.
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
• Commit unit tests in branch push-zutqoklymuso

---

_{Review rate limit: 0/5 reviews remaining, refill in 56 minutes and 10 seconds.}

_{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 the current code and only fix it if needed.

Inline comments:
In `@package.json`:
- Line 32: The detsys-ts dependency in package.json currently uses a mutable ref
"github:DeterminateSystems/detsys-ts#push-sqvzmxuokwlk"; replace that ref with
the immutable commit SHA so the dependency cannot move—update the detsys-ts
entry to
"github:DeterminateSystems/detsys-ts#ee36f3f49d08fa939d3ff3080226441dc5df366a"
(use the provided SHA ee36f3f49d08fa939d3ff3080226441dc5df366a) and save
package.json, then run your package manager install to ensure lockfile
consistency.

🪄 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: 0adbc524-d894-4321-8b55-511dd1bfd813

📥 Commits

Reviewing files that changed from the base of the PR and between 92148bb and 97f2ad7.

⛔ Files ignored due to path filters (2)

• dist/index.js is excluded by !**/dist/**
• package-lock.json is excluded by !**/package-lock.json

📒 Files selected for processing (1)

• package.json

[coderabbitai]

⚠️ Potential issue | 🟡 Minor

🧩 Analysis chain

🏁 Script executed:

#!/bin/bash
set -euo pipefail

REF="push-sqvzmxuokwlk"
API="https://api.github.com/repos/DeterminateSystems/detsys-ts/commits/${REF}"

echo "Resolving detsys-ts ref: ${REF}"
SHA="$(curl -fsSL "$API" | jq -r '.sha')"
echo "Resolved SHA: $SHA"

if [[ ! "$SHA" =~ ^[0-9a-f]{40}$ ]]; then
  echo "Ref did not resolve to a 40-char commit SHA."
  exit 1
fi

if [[ -f package-lock.json ]]; then
  echo "Found package-lock.json. Checking detsys-ts resolution entries:"
  jq -r '.. | .resolved? // empty' package-lock.json | rg -n 'detsys-ts|DeterminateSystems/detsys-ts' || true
else
  echo "No package-lock.json found in repository root."
fi

Repository: DeterminateSystems/nix-installer-action

Length of output: 348

---

🏁 Script executed:

cat -n package.json | sed -n '30,35p'

Repository: DeterminateSystems/nix-installer-action

Length of output: 337

---

Replace mutable ref with immutable commit SHA in detsys-ts dependency

Line 32 uses #push-sqvzmxuokwlk, a mutable branch/tag reference. While the lockfile pins the resolved immutable commit, using an immutable SHA directly in package.json is clearer and prevents accidental re-resolution to a different commit if the ref moves. The ref currently resolves to ee36f3f49d08fa939d3ff3080226441dc5df366a.

Suggested change

-    "detsys-ts": "github:DeterminateSystems/detsys-ts#push-sqvzmxuokwlk",
+    "detsys-ts": "github:DeterminateSystems/detsys-ts#ee36f3f49d08fa939d3ff3080226441dc5df366a",

📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change

	"detsys-ts": "github:DeterminateSystems/detsys-ts#push-sqvzmxuokwlk",
	"detsys-ts": "github:DeterminateSystems/detsys-ts#ee36f3f49d08fa939d3ff3080226441dc5df366a",

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@package.json` at line 32, The detsys-ts dependency in package.json currently
uses a mutable ref "github:DeterminateSystems/detsys-ts#push-sqvzmxuokwlk";
replace that ref with the immutable commit SHA so the dependency cannot
move—update the detsys-ts entry to
"github:DeterminateSystems/detsys-ts#ee36f3f49d08fa939d3ff3080226441dc5df366a"
(use the provided SHA ee36f3f49d08fa939d3ff3080226441dc5df366a) and save
package.json, then run your package manager install to ensure lockfile
consistency.