CI Doctors: use correct file for configuring plugin hooks

[pmtk]

Summary by CodeRabbit

• New Features

  • Added validation for MicroShift CI RCA output when analysis completes.
  • Improved plugin hook path handling for reliable RCA validation.

• Chores

  • Updated the lvms-ci plugin to version 1.3.2.
  • Updated the microshift-ci plugin to version 1.5.4.

[openshift-ci]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: pmtk

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details

Needs approval from an approver in each of these files:

• OWNERS [pmtk]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[coderabbitai]

Walkthrough

The PR updates LVMS and MicroShift CI plugin versions. It adjusts the LVMS RCA validation path and adds a timed MicroShift SubagentStop validation hook.

Changes

CI plugin updates

Layer / File(s)	Summary
Plugin version metadata .claude-plugin/marketplace.json, plugins/lvms-ci/.claude-plugin/plugin.json, plugins/microshift-ci/.claude-plugin/plugin.json	The LVMS version changes to 1.3.2. The MicroShift version changes to 1.5.4.
RCA validation hooks plugins/lvms-ci/hooks/hooks.json, plugins/microshift-ci/hooks/hooks.json, plugins/microshift-ci/.claude/settings.json	The LVMS command braces CLAUDE_PLUGIN_ROOT. MicroShift adds the Prow analyzer SubagentStop validation hook and removes the previous settings-based hook configuration.

Estimated code review effort: 2 (Simple) | ~10 minutes

Possibly related PRs

• openshift-eng/edge-tooling#244: Updates the same plugin versions and RCA validation hook configurations.

Suggested reviewers: kasturinarra, ggiguash

🚥 Pre-merge checks | ✅ 11

✅ Passed checks (11 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.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.
No-Weak-Crypto	✅ Passed	The PR changes only plugin metadata and hook configuration; patch and changed-file scans found no weak crypto, custom crypto, or secret-comparison code.
Container-Privileges	✅ Passed	The PR changes only plugin versions and hook JSON paths. No changed file or added line contains privileged, host namespace, SYS_ADMIN, or allowPrivilegeEscalation settings.
No-Sensitive-Data-In-Logs	✅ Passed	The diff adds only generic hook commands, status text, and version values; scans found no logging statements or sensitive-data literals.
No-Hardcoded-Secrets	✅ Passed	The PR adds only version values and hook configuration; scans found no API keys, tokens, passwords, private keys, credential URLs, or base64 strings over 32 characters.
No-Injection-Vectors	✅ Passed	The added hook commands use quoted CLAUDE_PLUGIN_ROOT paths, and no changed lines contain SQL concatenation, shell=True, eval/exec, pickle.loads, yaml.load, os.system, or dangerouslySetInnerHTML.
Ai-Attribution	✅ Passed	The PR contains one commit with no AI-tool mention and no attribution trailers, so no Assisted-by or Generated-by trailer is required and no AI Co-Authored-By trailer is present.
Title check	✅ Passed	The title clearly describes the main change: correcting the file used to configure plugin hooks.

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

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

[ggiguash]

/lgtm