Skip to content

Nix Dev Shell Beautification: Zsh + highlighting + autocomplete#746

Closed
jeff-hykin wants to merge 6 commits intodimensionalOS:mainfrom
jeff-hykin:jeff_riced_dev_shell
Closed

Nix Dev Shell Beautification: Zsh + highlighting + autocomplete#746
jeff-hykin wants to merge 6 commits intodimensionalOS:mainfrom
jeff-hykin:jeff_riced_dev_shell

Conversation

@jeff-hykin
Copy link
Member

@jeff-hykin jeff-hykin commented Nov 14, 2025

Current bash dev shell (ouch my eyes)

Screen Shot 2025-11-14 at 3 01 59 PM

Dev shell after this PR:

Screen Shot 2025-11-14 at 3 01 29 PM

Its zsh with starship terminal and custom ▣ prompt for dimos.




Aesthetics aside, last summer I made Xome that takes the full power of Nix home manager and crams in into a dev shell. So anything (not just zsh/starship) is just a couple lines away with this PR. On top of that Xome makes dev shells more reliable by cutting off access to the real home folder, e.g. no more "well it works on my machine". Python, mujoco, matplotlib can all store stuff in home and have pretty different behavior because of it.

Understandible if you don't want to merge this big of a change, or if you want a completely different home manager config, but I figured basically anything is an improvement over vanilla bash.

Copy link

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

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.

ℹ️ About Codex in GitHub

Codex has been enabled to automatically 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 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

Comment on lines +3 to +7
"core": {
"locked": {
"lastModified": 1,
"narHash": "sha256-lWX5DUltOFcS57I8wHH0Sz3J++zMORxHf+CXoZZLQzU=",
"path": "./helpers/builtins",

Choose a reason for hiding this comment

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

P1 Badge Remove dangling local flake input

The new core node in flake.lock points to "path": "./helpers/builtins", but the repository contains no helpers/builtins directory. Because Nix resolves path inputs relative to the checkout, any nix develop/nix flake show invocation now fails immediately with an error like getting status of '/workspace/dimos/helpers/builtins': No such file or directory. This breaks the dev shell entirely. Please either commit the missing helpers tree or update the lock file to refer to an actual source (e.g., a git input) before landing the change.

Useful? React with 👍 / 👎.

@jeff-hykin jeff-hykin closed this Dec 29, 2025
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