fix for settings on boot

[ljonesfl]

Note

Improves CLI boot configuration to support layered settings and secrets.

• Replaces direct Yaml source with SettingManagerFactory::create(...) to load base config, environment overlays, encrypted secrets, and .env fallbacks
• Changes config discovery to search for neuron.yaml under config/ or project root and initialize settings from that directory
• Keeps existing PHP version check and error handling intact

^{Written by Cursor Bugbot for commit b611418. This will update automatically on new commits. Configure here.}

Summary by CodeRabbit

Release Notes

• Improvements
  • Enhanced configuration discovery to support layered, environment-specific settings from multiple locations, providing greater flexibility for different deployment scenarios.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

📝 Walkthrough

Walkthrough

The config loading mechanism in bin/neuron is refactored to use SettingManagerFactory for layered configuration discovery instead of directly instantiating a single YAML source. The system now searches for configuration files in multiple locations (config/neuron.yaml and neuron.yaml at project root) and builds settings via the factory pattern with environment-aware layering support.

Changes

Cohort / File(s)	Summary
Configuration Loading Refactor bin/neuron	Replaced direct YAML settings instantiation with SettingManagerFactory-based multi-layer config loading. Expanded discovery paths to include config/neuron.yaml and root-level neuron.yaml. Integrated environment-aware configuration layering (base config, environment config, secrets, and .env fallback). Maintains existing error handling behavior.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Poem

🐰 A single file, a thoughtful shift,
From one path straight to layered lift,
Config now dances, layer by layer,
Environment-aware, a seasoned player,
Flexibility blooms in factory's care! ✨

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name	Status	Explanation	Resolution
Title check	❓ Inconclusive	The title 'fix for settings on boot' is vague and generic, using non-specific language that doesn't clearly convey what aspect of settings initialization is being fixed.	Provide a more specific title that describes the actual change, such as 'Replace single-source config loading with multi-layer SettingManagerFactory' or 'Implement layered configuration discovery for neuron.yaml'.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

---

---

📜 Recent review details

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 27fabc7 and b611418.

📒 Files selected for processing (1)

• bin/neuron

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)

• GitHub Check: Cursor Bugbot

🔇 Additional comments (1)

bin/neuron (1)

50-80: Configuration loading logic is correct. The refactoring from direct YAML loading to SettingManagerFactory::create() properly searches for config in config/ subdirectory first, then project root. The directory path is correctly passed to the factory after verifying neuron.yaml exists.

The silent exception handling (lines 75-78) is intentional design—it allows the application to gracefully degrade when config files are missing or unparseable, falling through to the next path or running with null settings if no config is found anywhere. If logging visibility into configuration failures is needed, that could be added, but the current behavior aligns with the framework's design.

_{✏️ Tip: You can disable this entire section by setting review_details to false in your review settings.}

---

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 and usage tips.}

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.

Flag	Coverage Δ	Complexity Δ
cli	77.40% <ø> (ø)	477.00 <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.