[WIP] SDCICD-1801 pdhelper claude skill

[ritmun]

• see pdhelper help/pdhelper how to list all available commands
• document team ownership mapping with 8 teams and their components
• you should have local clone of app-interface.

🤖 Generated with https://claude.com/claude-code

Co-Authored-By: Claude noreply@anthropic.com

Example

pdhelper team rocket

[openshift-ci-robot]

There are test jobs defined for this repository which are not configured to run automatically. Comment /test ? to see a list of all defined jobs. Review these jobs and use /test <job> to manually trigger jobs most likely to be impacted by the proposed changes.Comment /pipeline required to trigger all required & necessary jobs.

[coderabbitai]

Important

Review skipped

Auto reviews are limited based on label configuration.

🚫 Review skipped — only excluded labels are configured. (1)

• do-not-merge/work-in-progress

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository YAML (base), Organization UI (inherited)

Review profile: CHILL

Plan: Pro

Run ID: f12434d0-6bca-46e6-aab9-60beb20c8afe

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

✨ 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 and usage tips.}

[openshift-ci]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: ritmun

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 [ritmun]

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

[coderabbitai]

Actionable comments posted: 4

🧹 Nitpick comments (1)

.claude/skills/pdhelper.md (1)

25-33: Add language identifier to code block.

The example permission request block lacks a language identifier. While this is documentation rather than executable code, adding an identifier improves clarity.

📝 Proposed fix

-```
+```text
 To help you track commit deployments, I need permission for this session to:

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In @.claude/skills/pdhelper.md around lines 25 - 33, The fenced code block
starting with ``` that contains the permission request should include a language
identifier for clarity; update the opening fence from ``` to ```text (i.e.,
change the first triple-backtick that precedes "To help you track commit
deployments, I need permission for this session to:" to ```text) so the block is
explicitly marked as plain text.

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Inline comments:
In @.claude/skills.json:
- Line 9: Add a trailing newline at the end of the file .claude/skills.json so
the file ends with a newline character (i.e., ensure the final character before
EOF is '\n'); simply update the file's end-of-file to include the missing
newline.

In @.claude/skills/pdhelper.md:
- Line 633: The file ending line "App file:
`services/{service-name}/{operator-name}/app.yml`" is missing a trailing
newline; update the file `.claude/skills/pdhelper.md` to ensure the last
character is a newline (add a single trailing newline at EOF) so the file ends
with a newline character per convention.
- Line 55: The "Available teams: `rocket`, `aurora`, `thor`, `hulk`, `orange`,
`security`, `rosa`" entry is missing two teams (`fedramp`, `lpsre`) that are
referenced elsewhere; update that string to include `fedramp` and `lpsre` so it
reads e.g. "Available teams: `rocket`, `aurora`, `thor`, `hulk`, `orange`,
`security`, `rosa`, `fedramp`, `lpsre`" and scan the document for other
occurrences of the team list to make sure all references are consistent.
- Around line 585-589: The ownership list in .claude/skills/pdhelper.md
currently ends with an ambiguous note "(and all other team's repos)"; replace
that ambiguity by either enumerating every ROSA-owned component or by replacing
the phrase with an explicit sentence stating that Team ROSA owns all remaining
repos and why. Specifically update the list entries (e.g., clusterimagesets,
osd-gcp-marketplace-config, rosa-marketplace-release-generator) to include all
remaining component names OR change the trailing line to a clear statement like
"Team ROSA owns all other unlisted repositories for X reason" so queries against
the ROSA ownership list are deterministic.

---

Nitpick comments:
In @.claude/skills/pdhelper.md:
- Around line 25-33: The fenced code block starting with ``` that contains the
permission request should include a language identifier for clarity; update the
opening fence from ``` to ```text (i.e., change the first triple-backtick that
precedes "To help you track commit deployments, I need permission for this
session to:" to ```text) so the block is explicitly marked as plain text.

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Repository YAML (base), Organization UI (inherited)

Review profile: CHILL

Plan: Pro

Run ID: 1b72a979-d31d-433d-805c-83bb4d1ffaad

📥 Commits

Reviewing files that changed from the base of the PR and between 2aedfe3 and 8be0dc0.

📒 Files selected for processing (2)

• .claude/skills.json
• .claude/skills/pdhelper.md

[openshift-ci]

@ritmun: all tests passed!

Full PR test history. Your PR dashboard.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.