Accept null sections in config_items.json; fix test:config

[MitchBradley]

This follows up #51. That PR was merged at 3ca31cf, before this fix reached its branch.

Problem

• The real config_items.json is always rejected. The shape check added in c53b17c rejects any section that isn't an object. Field-less types are null in the real file: axes.<letter>.motorN.null_motor, NoSpindle, kinematics.midtbot and kinematics.Cartesian. So FigUI never uses the file and always falls back to its built-in field tables. The test passed only because its fixture had no null section.
• npm run test:config fails on main. Load FluidNC's config_items.json instead of the retired schema file #51 made ConfigStudio import the store, and the store's module-level browser code (localStorage.getItem("theme")) fails when the test loads ConfigStudio under Node.

Fix

• parseConfig() now allows null sections. test:config covers this with a NoSpindle: null fixture, and no longer asserts that axes: null is invalid.
• ConfigStudio no longer imports the store. CodeEditor passes espInfo.version down as a firmwareVersion prop.

Testing

• tsc --noEmit is clean, and npm run test:config passes.
• The real FluidNC/docs/config_items.json (from Emit config_items.json beside config_items.yaml bdring/FluidNC#1901) now loads and builds a schema.

🤖 Generated with Claude Code

[coderabbitai]

Warning

Review limit reached

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Next included review available in 42 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

• Configuration used: Repository: figamore/FigUI/.coderabbit.yaml
• Review profile: CHILL
• Plan: Advanced
• Run ID: b2b99d9d-6d2c-4cc8-8558-d576e84982bb

📥 Commits

Reviewing files that changed from the base of the PR and between 2390db8 and e87aafb.

📒 Files selected for processing (4)

• scripts/test-config-compatibility.mjs
• src/components/CodeEditor.tsx
• src/components/ConfigStudio.tsx
• src/lib/fluidSchema.ts

• Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[Copilot]

🟢 Approval recommended

The changes correctly address both reported regressions and include focused test coverage.

0 open findings

What changed in this PR

Allows FluidNC field-less configuration sections and removes ConfigStudio’s direct store dependency.

Changes:

• Accepts null configuration sections and adds regression coverage.
• Passes firmware version from CodeEditor into ConfigStudio.
• Restores Node compatibility for configuration tests.

File	Description
src/​lib/​fluidSchema.ts	Accepts null field-less sections.
src/​components/​ConfigStudio.tsx	Receives firmware version via props.
src/​components/​CodeEditor.tsx	Supplies the connected firmware version.
scripts/​test-config-compatibility.mjs	Tests null-section parsing.

🧠 Review effort: Balanced

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.