Add debugging skills and repo-specific patterns for container services - #329
Conversation
|
Looking for one thing? Review this PR in Change Stack to search files, summaries, diffs, and code without losing your place. No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
WalkthroughAdds a comprehensive debugging skill document for container service troubleshooting in the homelab infrastructure. The skill defines a seven-step workflow: fetching resolved Docker Compose configuration, reading container logs, checking mounted volume logs, searching documentation, triaging GitHub issues, analyzing release notes, and performing last-resort source inspection and patching with explicit confirmation. ChangesContainer Service Debugging Skill
🎯 2 (Simple) | ⏱️ ~12 minutes 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
There was a problem hiding this comment.
Actionable comments posted: 4
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In @.claude/skills/debug-container-service/references/repo-specifics.md:
- Around line 22-24: The two fenced code blocks containing the path
"/mnt/local/storage/docker-volumes/<service-name>/" and the command
"mcp__infra-mcp__container-service-recreate(service_name=\"category/service-name\")"
are missing language identifiers; add a language tag (e.g., ```text) to each
opening fence so they become "```text" to satisfy markdownlint MD040 and ensure
both blocks (the one with /mnt/local/storage/docker-volumes/<service-name>/ and
the one with mcp__infra-mcp__container-service-recreate(...)) are updated.
- Line 13: Update the documentation line that describes Docker Compose container
naming: replace the legacy pattern reference `<project>_<service>` with the
current Docker Compose v2 default pattern `<project>-<service>-1` and mention
that this hyphenated format is used when `container_name:` is not set so users
see the correct, DNS-safe naming behavior.
In @.claude/skills/debug-container-service/SKILL.md:
- Around line 30-32: Update the fenced code blocks in
.claude/skills/debug-container-service/SKILL.md to include an explicit language
tag (e.g., use ```text or ```bash) so markdownlint MD040 is satisfied;
specifically add the language tag to the block containing
mcp__infra-mcp__container-service-config(service_name="category/service-name")
and to the block with mcp__context7__resolve-library-id(...) and
mcp__context7__query-docs(...), and apply the same change to the other similar
fence around the same snippets referenced in the comment.
- Around line 73-78: The example commands in Step 3 hardcode the volume root
(/mnt/local/storage/docker-volumes/...), which contradicts the earlier
instruction to use the compose-derived DOCKER_VOLUMES path; update the two find
examples to use the variable placeholder (e.g. ${DOCKER_VOLUMES}/<service>/) or
reference the compose-derived path instead of the literal path so the examples
align with the compose config guidance in Step 1 and work on hosts with
different volume roots.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
Run ID: 61443a85-d63c-4b40-a9ea-1629e4cc0277
📒 Files selected for processing (2)
.claude/skills/debug-container-service/SKILL.md.claude/skills/debug-container-service/references/repo-specifics.md
8430072 to
ff21321
Compare
Summary by CodeRabbit