find-duplication

Trouve les duplications de code dans la base de code. Prend en charge deux modes - limité aux modifications de la branche actuelle ou une analyse complète de la base de code. Utilisez lorsque l'utilisateur demande de trouver…

npx skills add https://github.com/openshift/lightspeed-service --skill find-duplication

Find Code Duplication

Detect duplicated or near-duplicate code and suggest consolidation candidates.

Rules

  • Report findings, do not refactor. Refactoring is a separate task.
  • Focus on production code (ols/). Skip test duplication unless explicitly asked.
  • Group findings by severity: exact duplicates first, then near-duplicates.
  • For each finding, state whether extraction is worth it or acceptable duplication.

Step 1: Determine Scope

Ask the user:

  • Branch mode: only files changed in the current branch vs main.
  • Full mode: scan the entire ols/ directory.

For branch mode:

git diff --name-only origin/main -- 'ols/' | grep '\.py$'

For full mode, the target is ols/.

Step 2: Run Pylint Duplicate Detection

uv run pylint --disable=all --enable=duplicate-code --min-similarity-lines=6 <target files or directory>

Review output. Filter out false positives:

  • Import blocks (common imports are not duplication)
  • Pydantic model boilerplate (Field declarations)
  • Single-line patterns (logging, raises)

Step 3: Semantic Duplication Search

Pylint only catches textual similarity. Also look for:

  1. Similar function signatures — functions with near-identical parameter lists doing similar work.
  2. Repeated error handling — same try/except/log/return pattern across multiple files.
  3. Copy-pasted blocks — search for distinctive string literals or variable names that appear in multiple files.
rg "<distinctive pattern>" ols/ --type py -l

Step 4: Classify Findings

For each duplicate found, classify:

CategoryAction
Extract — identical logic in 3+ placesRecommend a shared helper
Parameterize — same structure, different valuesRecommend a common function with parameters
Acceptable — similar but serving different domainsNote it, no action needed
Test-only — repeated test setup/fixturesRecommend shared fixture (only if user asked)

Step 5: Report

For each finding:

  1. Files and line ranges involved
  2. What is duplicated (brief description)
  3. Classification (extract / parameterize / acceptable)
  4. Suggested location for shared code (if applicable)

Summary: total findings, how many actionable, estimated lines saved.

Plus de skills de openshift

openshift-expert
openshift
Expert de la plateforme OpenShift et de Kubernetes avec une connaissance approfondie de l'architecture des clusters, des opérateurs, du réseau, du stockage, du dépannage et des pipelines CI/CD. Utilisez…
official
find-token
openshift
Trouvez le jeton de vérification caché. Exécutez le script find-token pour récupérer un jeton unique.
official
code-review
openshift
Examiner une demande de tirage pour la qualité du code, l'exactitude et les conventions du projet. Utiliser lorsque l'utilisateur demande de réviser une PR, une revue de code, ou d'examiner des modifications sur une…
official
css-review
openshift
Examiner le CSS pour le style de codage, l'utilisation des tokens PatternFly et les bonnes pratiques. Utiliser lorsque l'utilisateur demande de réviser du CSS, vérifier des styles ou auditer des fichiers CSS.
official
review-readmes
openshift
Examiner tous les fichiers README.md du dépôt pour détecter les fautes de frappe, erreurs et informations obsolètes. Utiliser lorsque l'utilisateur demande de vérifier les README, de contrôler l'exactitude de la documentation, ou…
official
review-skills
openshift
Examiner les compétences IA du projet pour détecter les doublons, les références obsolètes, les erreurs et les problèmes structurels. Utiliser lorsque l'utilisateur demande de réviser les compétences, d'auditer les compétences, de vérifier…
official
test
openshift
Exécute des tests de bout en bout filtrés par balise. À utiliser lorsque l'utilisateur demande d'exécuter des tests, de lancer Playwright, ou de tester une balise de fonctionnalité spécifique comme @core ou @attach.
official
unused-exports
openshift
Trouve les symboles exportés qui ne sont jamais importés par un autre fichier. À utiliser lorsque l'utilisateur dit "vérifier les exports", "exports inutilisés" ou demande de nettoyer les exports.
official