[WIP] Investigate review process for commit approval

[Copilot]

Investigation Plan: Fix PR Review Bypass Issue

The problem statement indicates that a commit "Fix redundant std::move by making savedBroker movable" was able to bypass PR review when it shouldn't have. The issue is in ksMqttConfigProvider.cpp where savedBroker is a const reference but std::move() is being used on it (which is a no-op).

Root Cause Analysis

• Analyze the current code issue (redundant std::move on const reference)
• Identify why there's no static analysis to catch this
• Check CI/CD pipeline for gaps in code quality checks

Solution Plan

• Add clang-tidy configuration to detect redundant moves and const reference issues
• Create a GitHub workflow for static analysis on PRs
• Add CODEOWNERS file to require review
• Fix the actual code issue in ksMqttConfigProvider.cpp
• Test the solution with examples
• Document the new review process

Original prompt

Check why commit Fix redundant std::move by making savedBroker movable was able to pass (shouldnt due to pull request review)

---

💬 Share your feedback on Copilot coding agent for the chance to win a $200 gift card! Click here to start the survey.

[cziter15]

@copilot just modify validate-commits.yml