JSON parsing changes from new product query

[beojan]

This eliminates need for default constructor.

See #267

[beojan]

@phlexbot format

[github-actions]

No automatic markdownlint fixes were necessary.

[github-actions]

No automatic cmake-format fixes were necessary.

[github-actions]

Automatic clang-format fixes pushed (commit 4312595).
⚠️ Note: Some issues may require manual review and fixing.

[knoepfel]

Changes look reasonable, @beojan. Can you first rebase the branch on top of the latest commits to main? That should make all of the tests succeed.

[beojan]

@phlexbot format

[github-actions]

No automatic markdownlint fixes were necessary.

[github-actions]

No automatic cmake-format fixes were necessary.

[github-actions]

Automatic clang-format fixes pushed (commit aaebf2f).
⚠️ Note: Some issues may require manual review and fixing.

[codecov]

Codecov Report

❌ Patch coverage is 14.28571% with 36 lines in your changes missing coverage. Please review.

Files with missing lines	Patch %	Lines
phlex/configuration.cpp	7.69%	36 Missing ⚠️

❌ Your patch status has failed because the patch coverage (14.28%) is below the target coverage (80.00%). You can increase the patch coverage or adjust the target coverage.
❌ Your project status has failed because the head coverage (74.28%) is below the target coverage (80.00%). You can increase the head coverage or adjust the target coverage.

@@            Coverage Diff             @@
##             main     #282      +/-   ##
==========================================
- Coverage   75.20%   74.28%   -0.93%     
==========================================
  Files         124      124              
  Lines        2916     2955      +39     
  Branches      513      513              
==========================================
+ Hits         2193     2195       +2     
- Misses        503      540      +37     
  Partials      220      220              

Flag	Coverage Δ
unittests	74.28% <14.28%> (-0.93%)	⬇️

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

Files with missing lines	Coverage Δ
phlex/configuration.hpp	95.23% <100.00%> (+0.79%)	⬆️
phlex/core/product_query.cpp	80.00% <ø> (-1.82%)	⬇️
phlex/core/product_query.hpp	90.00% <ø> (ø)
phlex/configuration.cpp	23.40% <7.69%> (-76.60%)	⬇️

... and 1 file with indirect coverage changes

---

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8fe4a39...aaebf2f. Read the comment docs.

🚀 New features to boost your workflow:

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

[knoepfel]

LGTM. We should make sure the value_if_exists function is tested when it is eventually used. It will not hold up this PR, though