Nova emits a
broken-string-indentation warning at
flake.nix:740
The Radar replacement string starts with sidebar = ''{. Because the opening
brace is on the first line, it participates in indentation calculation and
prevents the intended stripping of indentation. This is nonfatal, but it adds
warnings to downstream package and Home Manager evaluations.
Reproduction
run:
nix-instantiate --parse flake.nix > /dev/null
Lix reports that text on the first line of the multiline string disables
indentation stripping and points to the opening brace.
Proposed fix
Start the content on the next line and indent the block consistently:
sidebar = ''
{
plugin location="radar"
}
'';
Keep the existing pkgs.replaceVars call and Radar layout structure. Update
the repository's LOC scorecard for the two additional source lines.
Validation and acceptance
A local patch against 352f1f77d97a82cd4e0769b90a02d0b26a571fc5 passed Nix
parsing. Evaluating the corrected string produced no warning, and substituting
it into layout.swap.kdl preserved the existing non-whitespace token sequence.
This verifies parsing and the intended whitespace-only change; it is not a
full package build or interactive layout test.
Acceptance: package evaluation no longer emits this indentation warning, and
the existing layout check still accepts both open and collapsed Radar layouts.
The previously merged #624 addressed
a different indented string in the old Home Manager module.
Nova emits a
broken-string-indentationwarning atflake.nix:740The Radar replacement string starts with
sidebar = ''{. Because the openingbrace is on the first line, it participates in indentation calculation and
prevents the intended stripping of indentation. This is nonfatal, but it adds
warnings to downstream package and Home Manager evaluations.
Reproduction
run:
nix-instantiate --parse flake.nix > /dev/nullLix reports that text on the first line of the multiline string disables
indentation stripping and points to the opening brace.
Proposed fix
Start the content on the next line and indent the block consistently:
Keep the existing
pkgs.replaceVarscall and Radar layout structure. Updatethe repository's LOC scorecard for the two additional source lines.
Validation and acceptance
A local patch against
352f1f77d97a82cd4e0769b90a02d0b26a571fc5passed Nixparsing. Evaluating the corrected string produced no warning, and substituting
it into
layout.swap.kdlpreserved the existing non-whitespace token sequence.This verifies parsing and the intended whitespace-only change; it is not a
full package build or interactive layout test.
Acceptance: package evaluation no longer emits this indentation warning, and
the existing layout check still accepts both open and collapsed Radar layouts.
The previously merged #624 addressed
a different indented string in the old Home Manager module.