Skip to content

feat: add nix package - #103

Merged
tarik02 merged 4 commits into
masterfrom
feat/nix-package
Aug 16, 2026
Merged

feat: add nix package#103
tarik02 merged 4 commits into
masterfrom
feat/nix-package

Conversation

@tarik02

@tarik02 tarik02 commented Aug 15, 2026

Copy link
Copy Markdown
Collaborator

What changed

  • Added flake packages for x86_64 and ARM64 Linux and macOS, including shell completions.
  • Locked the upstream T3 Code source to the repository's submodule revision.
  • Added native CI builds for all four systems.
  • Added a pull request check that maintains one comment with proposed source-pin, lockfile, and pnpm hash changes.
  • Documented installation through nix profile.

Why

Keeping this derivation in a consumer repository allowed the CLI source, upstream source, and pnpm dependency hash to drift independently. The package now lives beside the lockfile and submodule that define its build. Pull requests receive the required Nix changes before merge instead of a follow-up repair PR.

Verification

  • nix build .#packages.x86_64-linux.default --builders ''
  • nix flake check --all-systems --no-build
  • t3cli --version
  • nixfmt --check flake.nix nix/package.nix
  • actionlint

@changeset-bot

changeset-bot Bot commented Aug 15, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 958c35b

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 8fe1f923fb

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread README.md Outdated
Comment on lines +13 to +16
On NixOS or another system with Nix installed:

```sh
nix profile install github:tarik02-org/t3code-cli

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Restrict the Nix installation claim to x86_64 Linux

On aarch64 Linux or either Darwin architecture, this documented command cannot resolve a default package because flake.nix defines packages.default only for x86_64-linux. Either expose the package for the additional supported Nix systems or state the architecture restriction here so users on other systems are not given a nonfunctional installation command.

Useful? React with 👍 / 👎.

@tarik02
tarik02 merged commit 4b3da0f into master Aug 16, 2026
7 of 8 checks passed
@github-actions

Copy link
Copy Markdown
Contributor

The Nix source pin or pnpm dependency hash is stale. Apply this change to the pull request:

diff --git a/flake.lock b/flake.lock
index b78a9e4..547b4a6 100644
--- a/flake.lock
+++ b/flake.lock
@@ -42,17 +42,17 @@
     "upstream-t3code": {
       "flake": false,
       "locked": {
-        "lastModified": 1786361740,
-        "narHash": "sha256-qZi9hMGzqpmnpqvvVtsQvkZIiVqTgOMWv1y15MiSAYg=",
+        "lastModified": 1786783355,
+        "narHash": "sha256-G6sttzF/SChyPCzR7nFIEMFnNjrBul0bP4pa7kS2M9M=",
         "owner": "pingdotgg",
         "repo": "t3code",
-        "rev": "3b72d17cbca691f0b64e6d4a10c9e349f42873a5",
+        "rev": "f0ebc628c6dd83fd0c7963078ad7778ce6028d0c",
         "type": "github"
       },
       "original": {
         "owner": "pingdotgg",
         "repo": "t3code",
-        "rev": "3b72d17cbca691f0b64e6d4a10c9e349f42873a5",
+        "rev": "f0ebc628c6dd83fd0c7963078ad7778ce6028d0c",
         "type": "github"
       }
     }
diff --git a/flake.nix b/flake.nix
index b15ce7a..3a080b6 100644
--- a/flake.nix
+++ b/flake.nix
@@ -6,7 +6,7 @@
     nixpkgs-darwin.url = "github:NixOS/nixpkgs/nixpkgs-26.05-darwin";
 
     upstream-t3code = {
-      url = "github:pingdotgg/t3code/3b72d17cbca691f0b64e6d4a10c9e349f42873a5";
+      url = "github:pingdotgg/t3code/f0ebc628c6dd83fd0c7963078ad7778ce6028d0c";
       flake = false;
     };
   };

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.

1 participant