LCORE-1404: up to date configuration documentation

[tisnik]

Description

LCORE-1404: up to date configuration documentation

Type of change

• Refactor
• New feature
• Bug fix
• CVE fix
• Optimization
• Documentation Update
• Configuration Update
• Bump-up service version
• Bump-up dependent library
• Bump-up library or tool used for development (does not change the final image)
• CI configuration change
• Konflux configuration change
• Unit tests improvement
• Integration tests improvement
• End to end tests improvement
• Benchmarks improvement

Tools used to create PR

• Assisted-by: N/A
• Generated by: N/A

Related Tickets & Documents

• Related Issue #LCORE-1404

Summary by CodeRabbit

Release Notes

• New Features
  • Added support for disabling shield ID overrides in customization settings.
  • Extended Model Context Protocol Server configuration with custom header support.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: bb8b6006-4de9-4c68-b37a-d76ba643e884

📥 Commits

Reviewing files that changed from the base of the PR and between 528738d and 9fc16a3.

⛔ Files ignored due to path filters (2)

• docs/config.png is excluded by !**/*.png
• docs/config.svg is excluded by !**/*.svg

📒 Files selected for processing (1)

• docs/config.puml

---

Walkthrough

Three new attributes are added to configuration models: a boolean field disable_shield_ids_override in the Customization class and a headers field plus validate_headers method in the ModelContextProtocolServer class. No existing functionality is modified.

Changes

Cohort / File(s)	Summary
Configuration Model Enhancements src/models/config.py	Added disable_shield_ids_override field to Customization class; added headers field and validate_headers() validation method to ModelContextProtocolServer class.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Possibly related PRs

• LCORE-1170: Up-to-date-configuration documentation after a2a patch #1023: Adds related ModelContextProtocolServer configuration fields (authorization_headers and timeout) to the same class being extended in this PR.

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Title check	⚠️ Warning	The title references updating configuration documentation, but the actual changes add new fields and a validation method to the config model, which represents code changes beyond documentation updates.	Update the title to reflect the actual changes, such as 'Add validation and shield ID override support to config models' or clarify if the primary intent is documentation-only.

✅ 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.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

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.}