find-duplication

tarafından openshift

Kod tabanında kod tekrarını bulur. Mevcut dal değişiklikleriyle sınırlı veya tüm kod tabanı taraması olmak üzere iki modu destekler. Kullanıcı bulma talebinde bulunduğunda kullanılır…

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.

openshift tarafından daha fazla skill

openshift-expert
openshift
OpenShift platform ve Kubernetes uzmanı olup, küme mimarisi, operatörler, ağ oluşturma, depolama, sorun giderme ve CI/CD boru hatları konularında derin bilgiye sahiptir. Kullan…
official
find-token
openshift
Gizli doğrulama token'ını bulun. Benzersiz bir token almak için find-token betiğini çalıştırın.
official
code-review
openshift
Bir çekme isteğini kod kalitesi, doğruluk ve proje kuralları açısından inceleyin. Kullanıcı bir PR'ı incelemeyi, kod incelemesi yapmayı veya bir... üzerindeki değişiklikleri incelemeyi istediğinde kullanın.
official
css-review
openshift
CSS kod stilini, PatternFly token kullanımını ve en iyi uygulamaları gözden geçir. Kullanıcı CSS incelemesi, stil kontrolü veya CSS dosyalarını denetleme talebinde bulunduğunda kullan.
official
review-readmes
openshift
Depodaki tüm README.md dosyalarını yazım hataları, hatalar ve güncel olmayan bilgiler açısından gözden geçir. Kullanıcı README'leri gözden geçirmeyi, dokümantasyon doğruluğunu kontrol etmeyi istediğinde veya…
official
review-skills
openshift
Proje yapay zeka becerilerini tekrarlar, güncelliğini yitirmiş referanslar, hatalar ve yapısal sorunlar açısından gözden geçir. Kullanıcı becerileri gözden geçirmek, denetlemek, kontrol etmek istediğinde kullan…
official
test
openshift
Etiketle filtrelenmiş uçtan uca testleri çalıştırın. Kullanıcı test çalıştırmak, Playwright çalıştırmak veya @core ya da @attach gibi belirli bir özellik etiketini test etmek istediğinde kullanın.
official
unused-exports
openshift
Başka bir dosya tarafından hiç içe aktarılmayan dışa aktarılmış sembolleri bulur. Kullanıcı "dışa aktarımları kontrol et", "kullanılmayan dışa aktarımlar" dediğinde veya dışa aktarımları temizlemeyi istediğinde kullanın.
official