Add configurable clang-tidy Include Cleaner provider mappings

[Sean McManus (sean-mcmanus)]

Summary

Add the opt-in, resource-scoped C_Cpp.codeAnalysis.clangTidy.includeCleanerMappings setting for mapping exact Include Cleaner diagnostic symbol names to public include spellings.

This PR was investigated and created by GitHub Copilot (in VS Code). Any message starting with ✨Copilot: was sent by Copilot.

The setting defaults to an empty object and is forwarded with workspace-folder configuration. Its schema and runtime validation reject empty or control-containing symbol names and malformed header spellings while preserving valid entries, Unicode, and literal operator names. Mapping keys and values are excluded from settings telemetry.

Example:

"C_Cpp.codeAnalysis.clangTidy.includeCleanerMappings": {
    "example::widget": "<widget.h>"
}

Validation

Unit tests cover resource scope, schema/runtime agreement, invalid input, exact names, Unicode and prototype-sensitive keys, and telemetry redaction. TypeScript compilation, lint, and all 223 unit tests pass.

Note

Theoretically this could also be implemented on the LLVM/clang-tidy side later on...not sure why they haven't done this yet and/or if they are likely to accept such a change and/or how much work would be involved.

[Copilot]

Copilot review overview

🟢 Approval recommended

The setting is consistently defined, validated, forwarded, redacted, and thoroughly tested.

Review effort: Balanced
Findings: None

What changed in this PR

Adds an opt-in, resource-scoped mapping from Include Cleaner symbols to public headers.

Changes:

• Adds schema, localization, validation, and workspace forwarding.
• Redacts mapping contents from telemetry.
• Adds comprehensive unit coverage.

File	Description
Extension/​package.json	Defines the setting schema.
Extension/​package.nls.json	Documents the setting.
Extension/​src/​LanguageServer/​settings.ts	Validates mappings.
Extension/​src/​LanguageServer/​client.ts	Forwards mappings to the language server.
Extension/​src/​LanguageServer/​settingsTracker.ts	Redacts telemetry values.
Extension/​test/​unit/​settings.test.ts	Tests schema, validation, scope, and redaction.

---

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

[Sean McManus (sean-mcmanus)]

The cpptools side needs approval still...