Skip to content

Conversation

@ryoppippi
Copy link
Member

@ryoppippi ryoppippi commented Jan 11, 2026

Switch from mkShell to mkShellNoCC since this project doesn't require a C compiler toolchain. This reduces unnecessary dependencies and speeds up shell initialisation for Node.js/pnpm development.

Also updates flake.lock to latest inputs.


Summary by cubic

Use mkShellNoCC for the Nix dev shell to remove the C toolchain. This cuts unnecessary dependencies and speeds up shell startup for Node.js/pnpm.

Written for commit 6b1c7d3. Summary will update on new commits.

Copilot AI review requested due to automatic review settings January 11, 2026 13:24
@ryoppippi ryoppippi requested a review from a team as a code owner January 11, 2026 13:24
Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

No issues found across 2 files

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR optimizes the Nix development shell by switching from mkShell to mkShellNoCC, removing unnecessary C compiler toolchain dependencies for a Node.js/TypeScript SDK project. This change reduces dependencies and improves shell initialization speed.

Changes:

  • Replaced mkShell with mkShellNoCC in flake.nix
  • Updated flake.lock with latest versions of flake-parts and nixpkgs inputs

Reviewed changes

Copilot reviewed 1 out of 2 changed files in this pull request and generated no comments.

File Description
flake.nix Changed devShell from mkShell to mkShellNoCC to eliminate unnecessary C compiler dependencies
flake.lock Updated flake-parts and nixpkgs to their latest versions

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Switch from mkShell to mkShellNoCC since this project doesn't require
a C compiler toolchain. This reduces unnecessary dependencies and
speeds up shell initialisation for Node.js/pnpm development.
@ryoppippi ryoppippi force-pushed the chore/use-mkshellnocc branch from 4fb5421 to 6b1c7d3 Compare January 11, 2026 13:26
@pkg-pr-new
Copy link

pkg-pr-new bot commented Jan 11, 2026

Open in StackBlitz

npm i https://pkg.pr.new/StackOneHQ/stackone-ai-node/@stackone/ai@286

commit: 6b1c7d3

@ryoppippi ryoppippi merged commit 1a360dc into main Jan 11, 2026
19 checks passed
@ryoppippi ryoppippi deleted the chore/use-mkshellnocc branch January 11, 2026 13:30
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