Skip to content

More dogfood#11

Merged
cooper (czxtm) merged 13 commits intomainfrom
jp/more-dogfood
Mar 31, 2026
Merged

More dogfood#11
cooper (czxtm) merged 13 commits intomainfrom
jp/more-dogfood

Conversation

@arximboldi
Copy link
Copy Markdown
Contributor

Woof woof.

More small usability issues found along the way of trying the repo in nixmac.

func writeCanonicalJSON(path string, doc map[string]any, mode string) (bool, error) {
data, err := json.MarshalIndent(doc, "", " ")
if err != nil {
var buf bytes.Buffer
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

omg thank you i was wondering why

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Yeah apparently MarshalIndent does escape all strings!

mkCheckDerivation = appName: appCfg: cmdName: cmdCfg: let
appPath = appCfg.path or ".";
repoRoot = ../../../..;
repoRoot = self.outPath;
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

interesting - what does this do?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

So ../../../.. is relative to the current codebase, which means, refers to the stackpanel codebase itself. However we want to look into the codebase of the project where we are using stackpanel. self is the derivation that refers to the flake itself (basiocally it is the copy of the flake sources into the /nix/store) and outPath, as per usual in the a derivation, refers the actual path (somewhat equivalent to doing "${self}").

# Runtime PATH for the generated wrapper script.
# nodejs is always included so `node` is available when startScript uses it.
runtimePath = lib.makeBinPath ([pkgs.nodejs] ++ bunCfg.runtimeInputs);
runtimePath = lib.makeBinPath ([pkgs.nodejs pkgs.bun] ++ bunCfg.runtimeInputs);
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

ah, by any chance is this why it wasnt installing on entry?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Sadly no. For that simply there is no code doing that in our codebase. You need to enable languages.bun.install from devenv to get that feature.

@czxtm cooper (czxtm) merged commit cec7c98 into main Mar 31, 2026
3 of 5 checks passed
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