Skip to content

Commit 1a1ef13

Browse files
committed
Fix CodeRabbit review: lowercase negative pattern, add missing docstring
- "DockerClient" → "dockerclient" in docker skill's negative list (mixed-case never matched against lowered input) - Add docstring to main() to meet 80% coverage threshold
1 parent 53aa05a commit 1a1ef13

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed

.devcontainer/plugins/devs-marketplace/plugins/skill-engine/scripts/skill-suggester.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -188,7 +188,7 @@
188188
],
189189
"terms": ["dockerfile", "compose.yaml", "BuildKit"],
190190
"negative": ["docker-py", "docker py", "docker sdk", "docker from python",
191-
"aiodocker", "DockerClient"],
191+
"aiodocker", "dockerclient"],
192192
"context_guards": [
193193
"docker", "container", "compose", "image", "dockerfile",
194194
],
@@ -639,6 +639,7 @@ def match_skills(prompt: str) -> list[str]:
639639

640640

641641
def main() -> None:
642+
"""Read a hook event from stdin, score skills, and print suggestions to stdout."""
642643
raw = sys.stdin.read().strip()
643644
if not raw:
644645
return

0 commit comments

Comments
 (0)