Add Turkish (tr) docs and update README#744
Conversation
Add a full set of Turkish documentation under docs/tr (agents, changelog, CLAUDE guide, contributing, code of conduct, and many agents/commands/skills/rules files). Update README to include a link to the Turkish docs and increment the supported language count from 5 to 6. This commit adds localized guidance and references to help Turkish-speaking contributors and users.
|
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
✅ Files skipped from review due to trivial changes (1)
📝 WalkthroughWalkthroughAdded Turkish localization: updated Changes
Sequence Diagram(s)(omitted) Estimated code review effort🎯 3 (Moderate) | ⏱️ ~25 minutes Possibly related PRs
Suggested labels
Suggested reviewers
Poem
✨ Finishing Touches🧪 Generate unit tests (beta)
|
Analysis Failed
Troubleshooting
Retry: |
There was a problem hiding this comment.
38 issues found across 139 files
Note: This PR contains a large number of files. cubic only reviews up to 75 files per PR, so some files may not have been reviewed.
Prompt for AI agents (unresolved issues)
Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.
<file name="docs/tr/agents/go-build-resolver.md">
<violation number="1" location="docs/tr/agents/go-build-resolver.md:57">
P2: `invalid type assertion` fix guidance is technically incorrect by restricting assertions to `interface{}` only; assertions are valid from any interface-typed expression.</violation>
</file>
<file name="docs/tr/agents/chief-of-staff.md">
<violation number="1" location="docs/tr/agents/chief-of-staff.md:85">
P2: Inconsistent relationship memory path (`private/relationships.md` vs `relationships.md`) in one workflow can cause read/write divergence.</violation>
</file>
<file name="docs/tr/agents/rust-reviewer.md">
<violation number="1" location="docs/tr/agents/rust-reviewer.md:14">
P2: CI/merge readiness is framed as an assumption instead of a mandatory precondition, conflicting with required "green checks + no conflicts before review" workflow.</violation>
</file>
<file name="docs/tr/commands/multi-plan.md">
<violation number="1" location="docs/tr/commands/multi-plan.md:127">
P2: Phase 1’de enhanced requirement zorunlu kılınırken Phase 2.1’de original requirement isteniyor; bu aynı komut akışında çelişkili input talimatı oluşturuyor.</violation>
</file>
<file name="docs/tr/agents/go-reviewer.md">
<violation number="1" location="docs/tr/agents/go-reviewer.md:14">
P2: Reviewer workflow omits required pre-review gates (green checks and conflict-free branch) and instructs immediate review.</violation>
</file>
<file name="docs/tr/agents/code-reviewer.md">
<violation number="1" location="docs/tr/agents/code-reviewer.md:14">
P2: Review process doc omits mandatory pre-review gates (green CI and no merge conflicts), leading to blocked/invalid review cycles.</violation>
</file>
<file name="docs/tr/commands/instinct-import.md">
<violation number="1" location="docs/tr/commands/instinct-import.md:29">
P2: HTTP import example uses a non-raw GitHub URL, which will not provide parseable instinct YAML and can make the documented command fail.</violation>
</file>
<file name="docs/tr/commands/go-review.md">
<violation number="1" location="docs/tr/commands/go-review.md:90">
P2: Nested fenced code blocks reuse triple backticks, which breaks Markdown rendering of the example block.</violation>
</file>
<file name="docs/tr/commands/tdd.md">
<violation number="1" location="docs/tr/commands/tdd.md:40">
P2: Nested triple-backtick fences in the example session break Markdown rendering; use a longer outer fence (e.g., four backticks) or restructure the block.</violation>
</file>
<file name="docs/tr/commands/e2e.md">
<violation number="1" location="docs/tr/commands/e2e.md:39">
P2: Nested fenced code in the example uses a triple-backtick outer fence; use a longer outer fence (e.g., ````) so inner ``` blocks do not break markdown rendering.</violation>
</file>
<file name="docs/tr/commands/checkpoint.md">
<violation number="1" location="docs/tr/commands/checkpoint.md:7">
P2: Usage syntax is inconsistent with documented arguments: `clear` is supported below but missing from the main command signature.</violation>
</file>
<file name="docs/tr/agents/java-build-resolver.md">
<violation number="1" location="docs/tr/agents/java-build-resolver.md:24">
P2: Diagnostic instructions conflict: agents are told to run Maven and Gradle commands sequentially, despite later requiring build-tool detection first.</violation>
</file>
<file name="docs/tr/SECURITY.md">
<violation number="1" location="docs/tr/SECURITY.md:46">
P2: Security policy adds a direct link to an external GitHub repo, conflicting with project guidance to avoid unvetted external code links in user-facing security docs.</violation>
</file>
<file name="docs/tr/commands/eval.md">
<violation number="1" location="docs/tr/commands/eval.md:7">
P2: Usage synopsis is inconsistent with documented subcommands/arguments (`clean` missing; optional feature arg appears to apply to subcommands that take none).</violation>
</file>
<file name="docs/tr/agents/cpp-reviewer.md">
<violation number="1" location="docs/tr/agents/cpp-reviewer.md:14">
P2: Reviewer workflow contradicts team policy by starting review immediately instead of gating on passing CI checks.</violation>
<violation number="2" location="docs/tr/agents/cpp-reviewer.md:44">
P2: Turkish translation corrupts the C++ style rule by reversing `typedef`/`using` semantics, changing reviewer guidance.</violation>
</file>
<file name="docs/tr/agents/java-reviewer.md">
<violation number="1" location="docs/tr/agents/java-reviewer.md:12">
P2: Java reviewer workflow starts detailed review without required pre-review gates for green checks and conflict-free merge state.</violation>
</file>
<file name="docs/tr/agents/kotlin-reviewer.md">
<violation number="1" location="docs/tr/agents/kotlin-reviewer.md:20">
P2: Workflow guidance omits required pre-review gates (green CI and resolved merge conflicts), causing process non-compliance and wasted reviewer effort.</violation>
</file>
<file name="docs/tr/AGENTS.md">
<violation number="1" location="docs/tr/AGENTS.md:3">
P2: Documentation inventory is internally inconsistent (agent/skill totals conflict with listed content), indicating stale translated guidance.</violation>
</file>
<file name="docs/tr/README.md">
<violation number="1" location="docs/tr/README.md:9">
P2: License badge link is relative to the nested docs folder and points to a non-existent `docs/tr/LICENSE` instead of the repository root license file.</violation>
<violation number="2" location="docs/tr/README.md:18">
P3: Language support count is outdated (`5`) and contradicts the six-language selector in the same README.</violation>
<violation number="3" location="docs/tr/README.md:394">
P2: FAQ contains broken in-page anchor links to non-existent sections, causing dead internal navigation.</violation>
</file>
<file name="docs/tr/agents/python-reviewer.md">
<violation number="1" location="docs/tr/agents/python-reviewer.md:14">
P2: Python reviewer workflow starts review immediately but omits required pre-review gates (passing checks and no merge conflicts), conflicting with established review policy.</violation>
<violation number="2" location="docs/tr/agents/python-reviewer.md:74">
P2: Localized severity token in structured review output breaks consistency with canonical markers used in other agent docs, risking parser/tooling incompatibility.</violation>
</file>
<file name="docs/tr/commands/verify.md">
<violation number="1" location="docs/tr/commands/verify.md:45">
P2: Verification workflow claims secrets/security checks in output and `pre-pr`, but the actual step-by-step instructions never execute any security scan.</violation>
</file>
<file name="docs/tr/commands/skill-create.md">
<violation number="1" location="docs/tr/commands/skill-create.md:58">
P2: Nested triple-backtick fences in the example output can prematurely close the outer code block and break Markdown rendering.</violation>
</file>
<file name="docs/tr/commands/go-build.md">
<violation number="1" location="docs/tr/commands/go-build.md:46">
P2: Nested fenced code blocks use the same triple-backtick delimiter, which can prematurely close the outer example block and break Markdown rendering.</violation>
</file>
<file name="docs/tr/TERMINOLOGY.md">
<violation number="1" location="docs/tr/TERMINOLOGY.md:42">
P2: The glossary’s status column uses undocumented value `-` even though the status legend defines only `Onaylandı` and `Beklemede`.</violation>
</file>
<file name="docs/tr/CONTRIBUTING.md">
<violation number="1" location="docs/tr/CONTRIBUTING.md:273">
P2: Hook command example is shell-invalid due to an unescaped apostrophe inside a single-quoted string.</violation>
</file>
<file name="docs/tr/commands/sessions.md">
<violation number="1" location="docs/tr/commands/sessions.md:11">
P2: Usage synopsis is incomplete: it excludes documented `aliases` and `unalias` commands.</violation>
</file>
<file name="docs/tr/examples/README.md">
<violation number="1" location="docs/tr/examples/README.md:68">
P2: Documentation instructs appending a full JSON object to `settings.json`, which can corrupt the file into invalid JSON.</violation>
<violation number="2" location="docs/tr/examples/README.md:81">
P2: Two newly added related-resource links target non-existent files in `docs/tr`, resulting in broken documentation navigation.</violation>
</file>
<file name="docs/tr/agents/e2e-runner.md">
<violation number="1" location="docs/tr/agents/e2e-runner.md:74">
P2: Dokümantasyon `page.click()` otomatik bekleme davranışını yanlış belirtiyor; bu Playwright kullanımını yanıltabilir.</violation>
</file>
<file name="docs/tr/CODE_OF_CONDUCT.md">
<violation number="1" location="docs/tr/CODE_OF_CONDUCT.md:98">
P2: New user-facing doc adds an external GitHub repository link that conflicts with the team’s unvetted-repo-link policy.</violation>
</file>
<file name="docs/tr/examples/statusline.json">
<violation number="1" location="docs/tr/examples/statusline.json:4">
P2: `todo_count` can become `0\n0` due to `grep -c` + `|| echo 0`, causing integer comparison errors in the statusline command.</violation>
</file>
<file name="docs/tr/TROUBLESHOOTING.md">
<violation number="1" location="docs/tr/TROUBLESHOOTING.md:373">
P2: Hardcoded marketplace cache path is inconsistent with actual nested cache layout, so the MODULE_NOT_FOUND recovery `cd` can fail on real installs.</violation>
<violation number="2" location="docs/tr/TROUBLESHOOTING.md:421">
P3: Relative link `./docs/` in `docs/tr/TROUBLESHOOTING.md` likely points to a non-existent `docs/tr/docs/` path, causing broken documentation navigation.</violation>
</file>
<file name="docs/tr/commands/multi-execute.md">
<violation number="1" location="docs/tr/commands/multi-execute.md:101">
P2: Timeout/proses yönetimi yönergeleri kendi içinde çelişiyor; bir yandan process'i asla öldürmeyi yasaklayıp diğer yandan öldürme seçeneğini kullanıcıya sormayı zorunlu kılıyor.</violation>
</file>
Since this is your first cubic review, here's how it works:
- cubic automatically reviews your code and comments on bugs and improvements
- Teach cubic by replying to its comments. cubic learns from your replies and gets better over time
- Add one-off context when rerunning by tagging
@cubic-dev-aiwith guidance or docs links (includingllms.txt) - Ask questions if you need clarification on any suggestion
Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.
Co-authored-by: cubic-dev-ai[bot] <191113872+cubic-dev-ai[bot]@users.noreply.github.com>
|
Analysis Failed
Troubleshooting
Retry: |
Co-authored-by: cubic-dev-ai[bot] <191113872+cubic-dev-ai[bot]@users.noreply.github.com>
|
Analysis Failed
Troubleshooting
Retry: |
There was a problem hiding this comment.
Actionable comments posted: 10
Note
Due to the large number of review comments, Critical, Major severity comments were prioritized as inline comments.
🟡 Minor comments (18)
docs/tr/skills/django-patterns/SKILL.md-483-483 (1)
483-483:⚠️ Potential issue | 🟡 MinorTürkçe ifade düzeltmesi gerekli.
"Sipariş ilgili iş mantığı..."yerine"Siparişle ilgili iş mantığı..."daha doğru ve akıcı.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@docs/tr/skills/django-patterns/SKILL.md` at line 483, Dokümantasyondaki Türkçe ifade hatası: sınıf/dosya içinde yer alan docstring veya açıklama satırı """Sipariş ilgili iş mantığı için service layer.""" ifadesini bulun (aranacak benzersiz sembol: "Sipariş ilgili iş mantığı için service layer.") ve metni "Siparişle ilgili iş mantığı için service layer." olarak düzeltin; yalnızca bu kelime birleşimini güncelleyin ve diğer metin yapısını değiştirmeyin.docs/tr/skills/django-patterns/SKILL.md-61-61 (1)
61-61:⚠️ Potential issue | 🟡 MinorAdd missing imports to code snippets
Two snippets lack required imports that would cause runtime errors if copied directly:
- Line 61:
env()call requiresimport environandenv = environ.Env()setup- Line 224:
slugify()call requiresfrom django.utils.text import slugifyFor a documentation focused on best practices, include these dependencies explicitly in each code block.
Example fixes
# config/settings/base.py from pathlib import Path +import environ + +env = environ.Env()from django.db import models from django.contrib.auth.models import AbstractUser from django.core.validators import MinValueValidator, MaxValueValidator +from django.utils.text import slugify🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@docs/tr/skills/django-patterns/SKILL.md` at line 61, The docs snippets are missing required imports: add the environ setup before using env() (e.g., import environ and initialize env = environ.Env() so SECRET_KEY = env('DJANGO_SECRET_KEY') works) and add the slugify import before using slugify() (e.g., from django.utils.text import slugify) — update the code blocks that contain SECRET_KEY/env() and the code that calls slugify() to include these imports and any minimal initialization.docs/tr/SPONSORS.md-59-59 (1)
59-59:⚠️ Potential issue | 🟡 MinorSabit tarih ifadesi hızlıca güncelliğini yitirir.
Son senkronizasyon: Şubat 2026metni kısa sürede stale olur. Bu alanı otomatik üretilen tarihle beslemeyi veya statik tarihi kaldırmayı düşünün.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@docs/tr/SPONSORS.md` at line 59, Satırdaki sabit metin "*Otomatik güncellenir. Son senkronizasyon: Şubat 2026*" çabuk güncelliğini yitirdiği için ya statik tarihi kaldırın ya da otomatik üretilen bir tarihle değiştirin; örneğin SPONSORS.md içindeki bu metni bir şablon değişkeni (ör. {{lastSync}} veya LAST_SYNC) ile değiştirin ve doküman üretim/builder sürecine (CI/build script) güncel tarih atayan adım ekleyin veya doğrudan metni tamamen kaldırarak yalnızca "Otomatik güncellenir" bırakın.docs/tr/TERMINOLOGY.md-42-127 (1)
42-127:⚠️ Potential issue | 🟡 MinorDurum sütununda
-kullanımı belirsizlik yaratıyor.Line 42 sonrası birçok satırda
Durumdeğeri-iken, doküman sadeceOnaylandı/Beklemededurumlarını tanımlıyor. Terim tablosunu tutarlı tutmak için-değerlerini açık bir statüyle değiştirin (örn.Beklemede).🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@docs/tr/TERMINOLOGY.md` around lines 42 - 127, The Durum column currently uses '-' for many rows (e.g., entries like "Supabase", "Redis", "Playwright", "Go/Golang", "Docker", etc.), which is ambiguous; replace all '-' values in the Durum column with the explicit status "Beklemede" so the table uses only the defined statuses (Onaylandı/Beklemede), and verify the status legend/description near the top of the document still matches these two values.docs/tr/CLAUDE.md-60-60 (1)
60-60:⚠️ Potential issue | 🟡 MinorDosya adlandırma kuralı skills için yanıltıcı olabilir.
Bu satır genel bir kural gibi yazılmış; ancak
skills/altında dosya adı standart olarakSKILL.mdolmalı, kebab-case ise klasör adına uygulanmalı. Burayı kapsamı netleştirerek güncelleyin.✍️ Önerilen metin düzeltmesi
-Dosya isimlendirme: tire ile küçük harfler (örn., `python-reviewer.md`, `tdd-workflow.md`) +Dosya isimlendirme: Agent/command dosyalarında tire ile küçük harfler kullanın (örn., `python-reviewer.md`). +`skills/` altında ise klasör adı tire ile küçük harfli olmalı, dosya adı standart olarak `SKILL.md` olmalıdır.Based on learnings: In the
skills/directory, the standard convention isSKILL.md(uppercase) inside a lowercase-hyphen folder.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@docs/tr/CLAUDE.md` at line 60, Güncelleme: mevcut tek satırlık kural "Dosya isimlendirme: tire ile küçük harfler (örn., `python-reviewer.md`, `tdd-workflow.md`)" yanıltıcı; değiştirirken açıkça belirtin ki genel içerik dosyaları için kebab-case ve küçük harf kullanılırken, skills dizini için hüküm farklıdır — klasör adı kebab-case (örn. `python-reviewer`) olmalı ve içindeki dosya standart olarak büyük harfli `SKILL.md` formatında olmalıdır; yani dokümanda mevcut cümleyi değiştirip hem genel kuralı (kebab-case, küçük harf) hem de skills/ dizini özel kuralını (`SKILL.md` inside kebab-case folder) ikili olarak netleştirin.docs/tr/commands/go-review.md-2-2 (1)
2-2:⚠️ Potential issue | 🟡 MinorTerminoloji yazımı tutarlılığını düzeltin (
İdiomatic/Static).Türkçe metinde daha tutarlı kullanım için “İdiyomatik” ve “Statik analiz” biçimlerini tercih edin.
Also applies to: 12-12
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@docs/tr/commands/go-review.md` at line 2, Replace inconsistent English-derived terms with the preferred Turkish terms: change occurrences of "İdiomatic" to "İdiyomatik" and "Static" to "Statik analiz" (or "Statik" if used in short form) throughout the document, including the sentence that currently reads "İdiomatic desenler, ..." and the other occurrence referenced (line 12); ensure casing and diacritics match the Turkish forms and update the phrase "go-reviewer agent'ını" if necessary for consistent Turkish phrasing.docs/tr/README.md-9-9 (1)
9-9:⚠️ Potential issue | 🟡 MinorFix stale language count and license link target.
Line 18 still says 5 language supports, but this PR introduces Turkish as the 6th.
Line 9 likely points to a non-existent localLICENSE; it should link to repository root.🛠️ Proposed doc fix
-[](LICENSE) +[](../../LICENSE) ... -> **50K+ yıldız** | **6K+ fork** | **30 katkıda bulunan** | **5 dil desteği** | **Anthropic Hackathon Kazananı** +> **50K+ yıldız** | **6K+ fork** | **30 katkıda bulunan** | **6 dil desteği** | **Anthropic Hackathon Kazananı**Also applies to: 18-18
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@docs/tr/README.md` at line 9, Update the README: change the license badge link target from the local "LICENSE" to the repository root LICENSE (so the badge points to the top-level LICENSE file instead of a non-existent local file) and increment the language support count from 5 to 6 where the language count is mentioned; use the badge markdown "[](LICENSE)" as the locator for the license link change and search for the language-count sentence ("5" language supports) to update it to "6".docs/tr/agents/loop-operator.md-1-7 (1)
1-7:⚠️ Potential issue | 🟡 MinorDescription field language inconsistency.
The
descriptionfield in the YAML frontmatter is in English while the rest of the document and other Turkish agent files (e.g.,planner.md) have Turkish descriptions. For consistency within the Turkish documentation set, consider translating the description:🌐 Proposed fix for consistency
--- name: loop-operator -description: Operate autonomous agent loops, monitor progress, and intervene safely when loops stall. +description: Otonom ajan döngülerini çalıştır, ilerlemeyi izle ve döngüler durduğunda güvenli bir şekilde müdahale et. tools: ["Read", "Grep", "Glob", "Bash", "Edit"] model: sonnet color: orange ---🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@docs/tr/agents/loop-operator.md` around lines 1 - 7, The YAML frontmatter's description for the loop-operator document is in English while the rest of the Turkish docs use Turkish; update the description field (the "description" key in the frontmatter for name: loop-operator) to a Turkish translation matching the tone of other files (e.g., similar to planner.md) so the frontmatter is fully Turkish and consistent across the docs.docs/tr/commands/checkpoint.md-7-7 (1)
7-7:⚠️ Potential issue | 🟡 MinorKullanım satırına
clearalt komutunu ekleyin.Argümanlarda
clearvar ama üstteki kullanım sözdiziminde görünmüyor; tek satır uyum düzeltmesi yeterli.🛠️ Önerilen küçük düzeltme
-`/checkpoint [create|verify|list] [isim]` +`/checkpoint [create|verify|list|clear] [isim]`Also applies to: 70-74
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@docs/tr/commands/checkpoint.md` at line 7, Update the usage line for the checkpoint command to include the missing "clear" subcommand so the syntax matches the documented arguments; specifically modify the `/checkpoint [create|verify|list] [isim]` line to `/checkpoint [create|verify|list|clear] [isim]` (and make the same change in the repeated usage block around lines 70-74) so the displayed usage matches the available `clear` argument.docs/tr/skills/frontend-patterns/SKILL.md-298-300 (1)
298-300:⚠️ Potential issue | 🟡 MinorAvoid mutating arrays in the
useMemosorting example.At Line 298,
markets.sort(...)mutates the original array, which can cause side effects ifmarketsis props/state. Prefer sorting a copied array.💡 Suggested fix
-const sortedMarkets = useMemo(() => { - return markets.sort((a, b) => b.volume - a.volume) -}, [markets]) +const sortedMarkets = useMemo(() => { + return [...markets].sort((a, b) => b.volume - a.volume) +}, [markets])🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@docs/tr/skills/frontend-patterns/SKILL.md` around lines 298 - 300, The useMemo block computing sortedMarkets currently calls markets.sort(...) which mutates the original markets array; change it to sort a shallow copy to avoid side effects (e.g., create a copy with [...markets] or Array.from(markets] and then call .sort) inside the useMemo, keeping the dependency on markets; update the expression around sortedMarkets/useMemo to use the copied array before sorting so the original markets prop/state is not mutated.docs/tr/examples/user-CLAUDE.md-35-35 (1)
35-35:⚠️ Potential issue | 🟡 MinorYazım hatası: “orkestrayonu” → “orkestrasyonu”.
Line 35’te küçük bir yazım düzeltmesi gerekli.
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@docs/tr/examples/user-CLAUDE.md` at line 35, Fix the spelling mistake in the table row string "agents.md | Agent orkestrayonu, hangi agent'ın ne zaman kullanılacağı" by replacing "orkestrayonu" with "orkestrasyonu" so the entry reads "agents.md | Agent orkestrasyonu, hangi agent'ın ne zaman kullanılacağı".docs/tr/agents/docs-lookup.md-8-8 (1)
8-8:⚠️ Potential issue | 🟡 MinorKüçük yazım düzeltmesi önerisi.
Line 8’deki “specialistisiniz” ifadesi Türkçe akışta yazım/dil tutarlılığı açısından zayıf kalıyor; “uzmanısınız” (veya “dokümantasyon uzmanısınız”) daha doğru olur.
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@docs/tr/agents/docs-lookup.md` at line 8, Replace the Turkish word "specialistisiniz" in the sentence on line 8 with a correct, natural alternative such as "uzmanısınız" or "dokümantasyon uzmanısınız" to improve spelling and language consistency in the phrase that currently reads "Bir dokümantasyon specialistisiniz."docs/tr/skills/eval-harness/SKILL.md-263-263 (1)
263-263:⚠️ Potential issue | 🟡 MinorYazım hatası: “kovalamken” → “kovalarken”.
Line 263’te küçük bir typo var; metin kalitesi için düzeltin.
✏️ Önerilen düzeltme
-- Geçme oranlarını kovalamken maliyet ve gecikme kaymasını görmezden gelmek +- Geçme oranlarını kovalarken maliyet ve gecikme kaymasını görmezden gelmek🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@docs/tr/skills/eval-harness/SKILL.md` at line 263, Metindeki yazım hatasını düzeltin: satırdaki "Geçme oranlarını kovalamken maliyet ve gecikme kaymasını görmezden gelmek" ifadesinde "kovalamken" kelimesini "kovalarken" olarak değiştirin (örnekteki cümleyi güncelleyin).docs/tr/commands/update-docs.md-45-55 (1)
45-55:⚠️ Potential issue | 🟡 MinorFix incorrect file path references in the update-docs command.
Line 45 references
docs/CONTRIBUTING.md, but the existing localized version isdocs/tr/CONTRIBUTING.md. Since this file is in a Turkish locale directory, use the corresponding locale-specific path for consistency.Line 54 references
docs/RUNBOOK.md, which does not exist in the repository at any location. Either provide the correct path if it exists elsewhere, or note that this file needs to be created.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@docs/tr/commands/update-docs.md` around lines 45 - 55, Update the incorrect documentation path references: replace the generic CONTRIBUTING.md reference with the repository's Turkish localized CONTRIBUTING.md file (use the localized CONTRIBUTING.md entry present in the repo) and fix the RUNBOOK.md reference by either pointing to the correct existing runbook file in the Turkish docs set or add/create a RUNBOOK.md in the docs so the "Runbook" section points to a valid file; update the two mentions in the update-docs text accordingly.docs/tr/commands/eval.md-7-7 (1)
7-7:⚠️ Potential issue | 🟡 MinorKullanım satırı
cleanalt komutunu eksik bırakıyor.Line 7’deki kullanım ifadesi
cleankomutunu içermiyor, ama Line 120’de belgelenmiş. Üstteki kullanım satırını eşleştirmenizi öneririm.🧩 Önerilen düzeltme
-`/eval [define|check|report|list] [feature-name]` +`/eval [define|check|report|list|clean] [feature-name]`🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@docs/tr/commands/eval.md` at line 7, Update the usage line string that currently reads "`/eval [define|check|report|list] [feature-name]`" to include the missing "clean" subcommand so it matches the detailed documentation (e.g., change the usage pattern to include "clean" among the subcommands); locate the usage line in the eval.md content (the string on Line 7) and add "clean" to the bracketed list of subcommands.docs/tr/agents/chief-of-staff.md-108-108 (1)
108-108:⚠️ Potential issue | 🟡 MinorTürkçe başlıkta yazım hatasını düzeltin.
Line 108’daki
Brifingıifadesi yazım olarak hatalı görünüyor;Brifingiolmalı.✏️ Önerilen düzeltme
-# Bugünün Brifingı — [Tarih] +# Bugünün Brifingi — [Tarih]🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@docs/tr/agents/chief-of-staff.md` at line 108, Başlık satırındaki yazım hatasını düzeltin: değiştirin "# Bugünün Brifingı — [Tarih]" ifadesini doğru yazımı olan "# Bugünün Brifingi — [Tarih]" ile güncelleyin (arama anahtarı: başlık satırı içeriği "Bugünün Brifingı").docs/tr/agents/kotlin-reviewer.md-39-39 (1)
39-39:⚠️ Potential issue | 🟡 MinorGüvenlik eskalasyon cümlesinde küçük dil hatası var.
Line 39’daki “
security-reviewer'a devreden” ifadesi “security-reviewer'a devredin” olarak düzeltilmeli.🔧 Önerilen düzeltme
-Eğer bir CRITICAL güvenlik sorunu bulursanız, daha fazla analiz yapmadan incelemeyi durdurun ve `security-reviewer`'a devreden. +Eğer bir CRITICAL güvenlik sorunu bulursanız, daha fazla analiz yapmadan incelemeyi durdurun ve `security-reviewer`'a devredin.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@docs/tr/agents/kotlin-reviewer.md` at line 39, Dosyadaki güvenlik eskalasyon cümlesindeki küçük dil hatasını düzeltin: "security-reviewer'а devreden" ifadesini bulun (satırdaki eskalasyon cümlesi/ifadesi) ve bunu "security-reviewer'a devredin" olarak değiştirin; ilgili metin tekil çekimde ve emir kipinde doğru okunacak şekilde güncellendiğinden başka değişiklik yapmanıza gerek yoktur.docs/tr/AGENTS.md-3-3 (1)
3-3:⚠️ Potential issue | 🟡 MinorSkill sayısı dosya içinde tutarsız.
Line 3’te
116 skill, Line 145’te115 iş akışı skillleriyazıyor. Tek bir doğru değere çekilmeli; aksi halde envanter güvenilirliği düşüyor.🔧 Önerilen düzeltme
-skills/ — 115 iş akışı skillleri ve alan bilgisi +skills/ — 116 iş akışı skillleri ve alan bilgisiAlso applies to: 145-145
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@docs/tr/AGENTS.md` at line 3, Dosyadaki tutarsızlık: metinde "116 skill" ve "115 iş akışı skillleri" ifadeleri çelişiyor; tek doğru değeri kullanmak için AGENTS.md içindeki her iki ifadeyi bulun (örneğin arama ile "116 skill" ve "115 iş akışı skillleri") ve geçerli envanter sayısına göre her iki ifadeyi aynı sayıyla eşitleyin; güncelleme sırasında ilgili başlık/bağlam (agent/skill listesi veya açıklama satırı) korunmalı ve yazım tutarlılığı (örn. "skill" vs "iş akışı skillleri") sağlanmalıdır.
🧹 Nitpick comments (7)
docs/tr/commands/plan.md (1)
1-3: Consider adding complete frontmatter fields for consistency.The frontmatter only includes
description, while other command documentation files (e.g.,instinct-status.md) includename,description, andcommand: true. For consistency across Turkish command documentation, consider adding these fields:📝 Suggested frontmatter enhancement
--- +name: plan description: Gereksinimleri yeniden ifade et, riskleri değerlendir ve adım adım uygulama planı oluştur. Herhangi bir koda dokunmadan önce kullanıcı ONAYINI BEKLE. +command: true ---🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@docs/tr/commands/plan.md` around lines 1 - 3, Add complete frontmatter fields to the top of the document: include a "name" key with the Turkish command title, keep the existing "description" value, and add "command: true" so the file matches other command docs (e.g., instinct-status.md). Preserve existing content and YAML delimiters (---) and ensure the new frontmatter keys are ordered and localized consistently with other Turkish command files.docs/tr/agents/build-error-resolver.md (1)
89-91: Lockfile silme adımını daha güvenli hale getirin.
package-lock.jsonsilmek gereksiz geniş değişiklik ve sürüm kayması yaratabilir; minimal-fix hedefiyle çelişiyor.🔧 Önerilen revizyon
-# Bağımlılıkları yeniden yükle -rm -rf node_modules package-lock.json && npm install +# Bağımlılıkları lockfile'ı koruyarak yeniden yükle +rm -rf node_modules && npm ci🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@docs/tr/agents/build-error-resolver.md` around lines 89 - 91, Dokümandaki "rm -rf node_modules package-lock.json && npm install" adımı lockfile'ı gereksiz yere siliyor; bunun yerine lockfile'ı koruyarak deterministik kurulum yapın: değiştirilecek komut/metin "rm -rf node_modules package-lock.json && npm install" olarak bulun ve bunu ya sadece node_modules'i kaldıracak şekilde "rm -rf node_modules && npm ci" ile ya da lockfile'ı koruyarak tekrar üretmeyecek bir alternatifle (ör. "rm -rf node_modules && npm install --prefer-offline" veya sadece "npm ci") güncelleyin; belgenin açıklamasında da lockfile'ı korumanın amacını kısa bir not olarak ekleyin.docs/tr/skills/continuous-learning/SKILL.md (1)
80-119: Add an explicit “Örnekler” section for skill format consistency.This skill includes usage and workflow, but it lacks a dedicated examples section, which makes it less consistent with existing skill-document conventions.
Based on learnings, skills should include clear sections for “When to use”, “How it works”, and “Examples”.
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@docs/tr/skills/continuous-learning/SKILL.md` around lines 80 - 119, Add an explicit "Örnekler" section to the SKILL.md to match other skill docs and include short example usages and expected outcomes; ensure the doc also contains the "Ne zaman kullanılır" (When to use) and "Nasıl çalışır" (How it works) headings if missing. Update the markdown under the existing "Neden Stop Hook?" and "İlgili" blocks by inserting a new "Örnekler" heading with 2–3 concise examples (input, brief action, expected result) and confirm the local headings match the project convention ("When to use" => "Ne zaman kullanılır", "How it works" => "Nasıl çalışır") so tooling and readers find them consistently.docs/tr/skills/backend-patterns/SKILL.md (1)
296-303: Consider referencing the structured logging example earlier.While
console.error()is used here for demonstration purposes, the coding guidelines emphasize using proper logging libraries instead of console statements. Since this file already includes a comprehensive structured logging example (lines 536-596), consider adding a comment here pointing readers to that pattern.Based on learnings: "No console.log statements in production code; use proper logging libraries instead" - applies to **/*.{ts,tsx,js,jsx}.
💡 Optional: Add reference to structured logging
// Beklenmeyen hataları logla - console.error('Unexpected error:', error) + // Production'da yapılandırılmış loglama kullanın (aşağıdaki Logger örneğine bakın) + console.error('Unexpected error:', error)🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@docs/tr/skills/backend-patterns/SKILL.md` around lines 296 - 303, Replace the demonstration console.error usage in the error handler (the block that logs Unexpected error: and returns NextResponse.json(...)) with a brief comment directing readers to the document's structured logging example and advising to use a proper logging library in production; reference the existing structured logging example in this document and mention the console.error call and the error-handling return (NextResponse.json) so reviewers can locate and update the error handler to point to the structured logging pattern.docs/tr/skills/e2e-testing/SKILL.md (1)
1-326: Comprehensive E2E testing patterns, but missing standard skill sections.The skill provides excellent Playwright testing examples with correct YAML frontmatter. However, it lacks the explicit "Ne Zaman Kullanmalı" (When to use) and "Nasıl Çalışır" (How it works) sections that are part of the standard skill format. The skill jumps directly into organizational patterns and examples.
Recommendation: Add these sections after line 9:
- Ne Zaman Kullanmalı (When to use) - e.g., "Critical user flows, regression prevention, cross-browser testing"
- Nasıl Çalışır (How it works) - Brief overview of Playwright's architecture and the Page Object Model approach
Based on learnings: Skills must be formatted as Markdown files with clear sections: 'When to use', 'How it works', and 'Examples'.
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@docs/tr/skills/e2e-testing/SKILL.md` around lines 1 - 326, The skill (name: e2e-testing) is missing the required "Ne Zaman Kullanmalı" and "Nasıl Çalışır" sections; add two new Markdown sections titled "## Ne Zaman Kullanmalı" and "## Nasıl Çalışır" near the top (after the short description/frontmatter) that briefly state when to use the patterns (e.g., critical user flows, regression prevention, cross-browser testing) and a short overview of Playwright architecture and the Page Object Model approach (mention ItemsPage as an example POM) so the document conforms to the standard skill format.docs/tr/commands/multi-plan.md (1)
40-40: Clarify the Gemini model flag substitution instruction.The instruction at line 40 is slightly unclear. It states:
{{GEMINI_MODEL_FLAG}}:--backend geminikullanırken,--gemini-model gemini-3-pro-previewile değiştir (trailing space not edin); codex için boş string kullanSuggestion: Clarify whether "trailing space not edin" means "don't include a trailing space" or if it's asking the user to note the trailing space. Consider rephrasing for clarity:
-**Model Parametre Notları**: -- `{{GEMINI_MODEL_FLAG}}`: `--backend gemini` kullanırken, `--gemini-model gemini-3-pro-preview` ile değiştir (trailing space not edin); codex için boş string kullan +**Model Parametre Notları**: +- `{{GEMINI_MODEL_FLAG}}`: `--backend gemini` kullanırken `--gemini-model gemini-2.0-flash-exp ` ile değiştir (sondaki boşluğa dikkat edin); codex için boş string kullanın🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@docs/tr/commands/multi-plan.md` at line 40, The phrase for {{GEMINI_MODEL_FLAG}} is ambiguous; replace the current line with a clearer instruction stating that when using --backend gemini you should substitute {{GEMINI_MODEL_FLAG}} with --gemini-model gemini-3-pro-preview (do not include a trailing space) and that for the codex backend the flag should be an empty string; update the wording to a concise Turkish sentence like: "`{{GEMINI_MODEL_FLAG}}`: --backend gemini kullanıyorsanız `--gemini-model gemini-3-pro-preview` ile değiştir (sonunda boşluk bırakmayın); codex için boş string kullanın."docs/tr/skills/docker-patterns/SKILL.md (1)
1-364: Well-structured Docker patterns documentation.The skill documentation provides comprehensive Docker and Docker Compose best practices with clear examples. The YAML frontmatter is correctly formatted, and code examples (YAML, Dockerfile, bash) are properly preserved.
Note: The skill uses a pattern-based organization (sections for specific Docker concepts) rather than the standard "When to use / How it works / Examples" structure mentioned in learnings. This organization works well for a reference-style skill, though you may consider adding a brief "Nasıl Çalışır" (How it works) overview section after line 18 if you want to align more closely with the standard skill format.
Based on learnings: Skills should be formatted with clear sections: 'When to use', 'How it works', and 'Examples'.
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@docs/tr/skills/docker-patterns/SKILL.md` around lines 1 - 364, The doc is missing a short "Nasıl Çalışır" (How it works) overview to match the recommended skill structure; add a brief paragraph-level section titled "## Nasıl Çalışır" summarizing core concepts (what Docker Compose does, multi-stage Dockerfile build flow, service discovery via Compose network names, and volume vs bind mount tradeoffs) and place it between the YAML frontmatter (containing name: docker-patterns) and the existing "## Ne Zaman Aktifleştirmeli" heading so readers get a quick conceptual primer before the usage patterns and examples.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@docs/tr/agents/e2e-runner.md`:
- Line 74: The current Turkish sentence wrongly claims page.click() does not
auto-wait; change it to state that both page.click() and page.locator().click()
perform Playwright's auto-wait/actionability checks, and clarify the real
difference: locator.click() (or page.locator().click()) is strict (will error on
multiple matches) and is the recommended modern approach for resilience—not
because it waits differently; update the wording to reflect this distinction
using the symbols page.click(), page.locator().click(), and locator.click().
In `@docs/tr/commands/build-fix.md`:
- Line 17: Replace the project-level validation command in the table cell that
currently shows `python -m py_compile` with `python -m compileall .` so the docs
recommend a recursive, project-wide bytecode compile (locate the table entry
containing the literal `python -m py_compile` and update it to `python -m
compileall .`). Ensure the surrounding formatting of the table row remains
intact.
In `@docs/tr/commands/code-review.md`:
- Around line 1-3: Bu dosyada eksik olan YAML frontmatter bloğunu ekleyin:
dosyanın en üstüne üç kısa çizgiyle başlanan/bitirilen frontmatter içine en
azından description alanını (ör. description: "kısa açıklama") ekleyin ve
gerekiyorsa title/metadatayı tamamlayarak commands/ komut dokümantasyon
şablonuyla uyumlu hale getirin; hedeflenen semboller: frontmatter bloğu ve
description alanı (description) — bu alanı docs/tr/commands/code-review.md
içeriğinin üstüne yerleştirin.
In `@docs/tr/commands/e2e.md`:
- Line 278: The workflow step currently uses the deprecated action reference
"uses: actions/upload-artifact@v3"; update that to a supported major version
(preferably "uses: actions/upload-artifact@v7", or at minimum "uses:
actions/upload-artifact@v4") in the workflow file, then validate the workflow
(run GitHub Actions workflow lint/syntax check or a dry run) and adjust any
inputs/fields if the newer action version changed parameter names or behavior.
In `@docs/tr/commands/verify.md`:
- Around line 1-3: Bu Markdown dokümanında eksik olan YAML frontmatter
(özellikle description alanı) eklenecek: dosyanın en başına üç çizgiyle ayrılmış
bir YAML bloğu ekleyin ve içine en azından description: "Kısa komut açıklaması"
olacak şekilde yerel dilde açıklama; gerekiyorsa title, tags veya locale gibi
CONTRIBUTING.md’de tanımlı diğer metadata alanlarını de dahil edin; frontmatter
bloğunu (--- ... ---) ekleyip kaydederek komut keşfi/işlenmesi için gereken
metadata’yı sağlamış olacaksınız.
In `@docs/tr/examples/README.md`:
- Around line 65-69: Update the “Status Line Konfigürasyonu” instructions to
remove the raw append command and instead tell the user to merge JSON safely:
explain that they should not use cat ... >> to append, and instruct them to
either manually merge the contents of statusline.json into their settings.json
maintaining valid JSON, or use a JSON-aware merge/cli tool (reference the
statusline.json file and the settings.json target in the README section header)
and replace the current two-line bash snippet with this safer guidance.
In `@docs/tr/examples/statusline.json`:
- Line 4: The todo_count assignment is producing "0\n0" when grep -c finds no
matches because grep exits non-zero and the subsequent || echo 0 appends another
zero; update the command that sets todo_count (the expression using todo_count,
transcript and grep -c '"type":"todo"') so that the grep's non-zero exit is
swallowed (e.g. wrap the grep invocation in a grouped command and append "||
true") before falling back to echo 0 when the transcript file is missing,
ensuring only a single numeric "0" is produced and [ "$todo_count" -gt 0 ] works
correctly.
In `@docs/tr/skills/coding-standards/SKILL.md`:
- Around line 399-401: sortedMarkets hesaplanırken useMemo içindeki
markets.sort(...) diziyi yerinde mutasyona uğratarak değişmezlik ilkesini ihlal
ediyor; bunu düzeltmek için useMemo içerisindeki markets.sort çağrısını kopya
bir dizi üzerinde çalıştıracak şekilde değiştir (örneğin markets.slice() veya
[...markets] ile yeni bir dizi oluşturup o diziyi sort et) ve sortedMarkets ile
useMemo kullanımını buna göre güncelle; ilgili yerleri bulmak için
sortedMarkets, useMemo ve markets.sort sembollerine bak.
In `@docs/tr/skills/golang-testing/SKILL.md`:
- Line 675: The fuzz command's target name doesn't match the actual test
function: update the example command to reference the real fuzz function name
FuzzParseJSON (or use a pattern that matches it, e.g. FuzzParse*) so the command
will run the intended test; locate the example command and replace
`-fuzz=FuzzParse` with `-fuzz=FuzzParseJSON` (or a matching pattern) where the
CLI example is shown.
In `@docs/tr/skills/kotlin-patterns/SKILL.md`:
- Around line 382-423: The dependency versions in the plugins block
(kotlin("jvm")/kotlin("plugin.serialization") "2.3.10", id("io.ktor.plugin")
"3.4.0", kotest/test deps, etc.) and the dependencies section are outdated;
either update these to current stable releases (e.g., Kotlin 2.3.20, Ktor 3.4.1,
Kotest 6.1.6 and matching library versions) by changing the version strings in
the plugins and dependencies entries, or add a clear “example versions (last
updated YYYY-MM-DD)” label above the block so readers know these are
illustrative and may be out of date; ensure you modify the plugins { ... } and
dependencies { ... } blocks referenced by kotlin("jvm"), id("io.ktor.plugin"),
and the Kotest/testImplementation lines accordingly.
---
Minor comments:
In `@docs/tr/AGENTS.md`:
- Line 3: Dosyadaki tutarsızlık: metinde "116 skill" ve "115 iş akışı skillleri"
ifadeleri çelişiyor; tek doğru değeri kullanmak için AGENTS.md içindeki her iki
ifadeyi bulun (örneğin arama ile "116 skill" ve "115 iş akışı skillleri") ve
geçerli envanter sayısına göre her iki ifadeyi aynı sayıyla eşitleyin;
güncelleme sırasında ilgili başlık/bağlam (agent/skill listesi veya açıklama
satırı) korunmalı ve yazım tutarlılığı (örn. "skill" vs "iş akışı skillleri")
sağlanmalıdır.
In `@docs/tr/agents/chief-of-staff.md`:
- Line 108: Başlık satırındaki yazım hatasını düzeltin: değiştirin "# Bugünün
Brifingı — [Tarih]" ifadesini doğru yazımı olan "# Bugünün Brifingi — [Tarih]"
ile güncelleyin (arama anahtarı: başlık satırı içeriği "Bugünün Brifingı").
In `@docs/tr/agents/docs-lookup.md`:
- Line 8: Replace the Turkish word "specialistisiniz" in the sentence on line 8
with a correct, natural alternative such as "uzmanısınız" or "dokümantasyon
uzmanısınız" to improve spelling and language consistency in the phrase that
currently reads "Bir dokümantasyon specialistisiniz."
In `@docs/tr/agents/kotlin-reviewer.md`:
- Line 39: Dosyadaki güvenlik eskalasyon cümlesindeki küçük dil hatasını
düzeltin: "security-reviewer'а devreden" ifadesini bulun (satırdaki eskalasyon
cümlesi/ifadesi) ve bunu "security-reviewer'a devredin" olarak değiştirin;
ilgili metin tekil çekimde ve emir kipinde doğru okunacak şekilde
güncellendiğinden başka değişiklik yapmanıza gerek yoktur.
In `@docs/tr/agents/loop-operator.md`:
- Around line 1-7: The YAML frontmatter's description for the loop-operator
document is in English while the rest of the Turkish docs use Turkish; update
the description field (the "description" key in the frontmatter for name:
loop-operator) to a Turkish translation matching the tone of other files (e.g.,
similar to planner.md) so the frontmatter is fully Turkish and consistent across
the docs.
In `@docs/tr/CLAUDE.md`:
- Line 60: Güncelleme: mevcut tek satırlık kural "Dosya isimlendirme: tire ile
küçük harfler (örn., `python-reviewer.md`, `tdd-workflow.md`)" yanıltıcı;
değiştirirken açıkça belirtin ki genel içerik dosyaları için kebab-case ve küçük
harf kullanılırken, skills dizini için hüküm farklıdır — klasör adı kebab-case
(örn. `python-reviewer`) olmalı ve içindeki dosya standart olarak büyük harfli
`SKILL.md` formatında olmalıdır; yani dokümanda mevcut cümleyi değiştirip hem
genel kuralı (kebab-case, küçük harf) hem de skills/ dizini özel kuralını
(`SKILL.md` inside kebab-case folder) ikili olarak netleştirin.
In `@docs/tr/commands/checkpoint.md`:
- Line 7: Update the usage line for the checkpoint command to include the
missing "clear" subcommand so the syntax matches the documented arguments;
specifically modify the `/checkpoint [create|verify|list] [isim]` line to
`/checkpoint [create|verify|list|clear] [isim]` (and make the same change in the
repeated usage block around lines 70-74) so the displayed usage matches the
available `clear` argument.
In `@docs/tr/commands/eval.md`:
- Line 7: Update the usage line string that currently reads "`/eval
[define|check|report|list] [feature-name]`" to include the missing "clean"
subcommand so it matches the detailed documentation (e.g., change the usage
pattern to include "clean" among the subcommands); locate the usage line in the
eval.md content (the string on Line 7) and add "clean" to the bracketed list of
subcommands.
In `@docs/tr/commands/go-review.md`:
- Line 2: Replace inconsistent English-derived terms with the preferred Turkish
terms: change occurrences of "İdiomatic" to "İdiyomatik" and "Static" to "Statik
analiz" (or "Statik" if used in short form) throughout the document, including
the sentence that currently reads "İdiomatic desenler, ..." and the other
occurrence referenced (line 12); ensure casing and diacritics match the Turkish
forms and update the phrase "go-reviewer agent'ını" if necessary for consistent
Turkish phrasing.
In `@docs/tr/commands/update-docs.md`:
- Around line 45-55: Update the incorrect documentation path references: replace
the generic CONTRIBUTING.md reference with the repository's Turkish localized
CONTRIBUTING.md file (use the localized CONTRIBUTING.md entry present in the
repo) and fix the RUNBOOK.md reference by either pointing to the correct
existing runbook file in the Turkish docs set or add/create a RUNBOOK.md in the
docs so the "Runbook" section points to a valid file; update the two mentions in
the update-docs text accordingly.
In `@docs/tr/examples/user-CLAUDE.md`:
- Line 35: Fix the spelling mistake in the table row string "agents.md | Agent
orkestrayonu, hangi agent'ın ne zaman kullanılacağı" by replacing "orkestrayonu"
with "orkestrasyonu" so the entry reads "agents.md | Agent orkestrasyonu, hangi
agent'ın ne zaman kullanılacağı".
In `@docs/tr/README.md`:
- Line 9: Update the README: change the license badge link target from the local
"LICENSE" to the repository root LICENSE (so the badge points to the top-level
LICENSE file instead of a non-existent local file) and increment the language
support count from 5 to 6 where the language count is mentioned; use the badge
markdown
"[](LICENSE)" as the
locator for the license link change and search for the language-count sentence
("5" language supports) to update it to "6".
In `@docs/tr/skills/django-patterns/SKILL.md`:
- Line 483: Dokümantasyondaki Türkçe ifade hatası: sınıf/dosya içinde yer alan
docstring veya açıklama satırı """Sipariş ilgili iş mantığı için service
layer.""" ifadesini bulun (aranacak benzersiz sembol: "Sipariş ilgili iş mantığı
için service layer.") ve metni "Siparişle ilgili iş mantığı için service layer."
olarak düzeltin; yalnızca bu kelime birleşimini güncelleyin ve diğer metin
yapısını değiştirmeyin.
- Line 61: The docs snippets are missing required imports: add the environ setup
before using env() (e.g., import environ and initialize env = environ.Env() so
SECRET_KEY = env('DJANGO_SECRET_KEY') works) and add the slugify import before
using slugify() (e.g., from django.utils.text import slugify) — update the code
blocks that contain SECRET_KEY/env() and the code that calls slugify() to
include these imports and any minimal initialization.
In `@docs/tr/skills/eval-harness/SKILL.md`:
- Line 263: Metindeki yazım hatasını düzeltin: satırdaki "Geçme oranlarını
kovalamken maliyet ve gecikme kaymasını görmezden gelmek" ifadesinde
"kovalamken" kelimesini "kovalarken" olarak değiştirin (örnekteki cümleyi
güncelleyin).
In `@docs/tr/skills/frontend-patterns/SKILL.md`:
- Around line 298-300: The useMemo block computing sortedMarkets currently calls
markets.sort(...) which mutates the original markets array; change it to sort a
shallow copy to avoid side effects (e.g., create a copy with [...markets] or
Array.from(markets] and then call .sort) inside the useMemo, keeping the
dependency on markets; update the expression around sortedMarkets/useMemo to use
the copied array before sorting so the original markets prop/state is not
mutated.
In `@docs/tr/SPONSORS.md`:
- Line 59: Satırdaki sabit metin "*Otomatik güncellenir. Son senkronizasyon:
Şubat 2026*" çabuk güncelliğini yitirdiği için ya statik tarihi kaldırın ya da
otomatik üretilen bir tarihle değiştirin; örneğin SPONSORS.md içindeki bu metni
bir şablon değişkeni (ör. {{lastSync}} veya LAST_SYNC) ile değiştirin ve doküman
üretim/builder sürecine (CI/build script) güncel tarih atayan adım ekleyin veya
doğrudan metni tamamen kaldırarak yalnızca "Otomatik güncellenir" bırakın.
In `@docs/tr/TERMINOLOGY.md`:
- Around line 42-127: The Durum column currently uses '-' for many rows (e.g.,
entries like "Supabase", "Redis", "Playwright", "Go/Golang", "Docker", etc.),
which is ambiguous; replace all '-' values in the Durum column with the explicit
status "Beklemede" so the table uses only the defined statuses
(Onaylandı/Beklemede), and verify the status legend/description near the top of
the document still matches these two values.
---
Nitpick comments:
In `@docs/tr/agents/build-error-resolver.md`:
- Around line 89-91: Dokümandaki "rm -rf node_modules package-lock.json && npm
install" adımı lockfile'ı gereksiz yere siliyor; bunun yerine lockfile'ı
koruyarak deterministik kurulum yapın: değiştirilecek komut/metin "rm -rf
node_modules package-lock.json && npm install" olarak bulun ve bunu ya sadece
node_modules'i kaldıracak şekilde "rm -rf node_modules && npm ci" ile ya da
lockfile'ı koruyarak tekrar üretmeyecek bir alternatifle (ör. "rm -rf
node_modules && npm install --prefer-offline" veya sadece "npm ci") güncelleyin;
belgenin açıklamasında da lockfile'ı korumanın amacını kısa bir not olarak
ekleyin.
In `@docs/tr/commands/multi-plan.md`:
- Line 40: The phrase for {{GEMINI_MODEL_FLAG}} is ambiguous; replace the
current line with a clearer instruction stating that when using --backend gemini
you should substitute {{GEMINI_MODEL_FLAG}} with --gemini-model
gemini-3-pro-preview (do not include a trailing space) and that for the codex
backend the flag should be an empty string; update the wording to a concise
Turkish sentence like: "`{{GEMINI_MODEL_FLAG}}`: --backend gemini
kullanıyorsanız `--gemini-model gemini-3-pro-preview` ile değiştir (sonunda
boşluk bırakmayın); codex için boş string kullanın."
In `@docs/tr/commands/plan.md`:
- Around line 1-3: Add complete frontmatter fields to the top of the document:
include a "name" key with the Turkish command title, keep the existing
"description" value, and add "command: true" so the file matches other command
docs (e.g., instinct-status.md). Preserve existing content and YAML delimiters
(---) and ensure the new frontmatter keys are ordered and localized consistently
with other Turkish command files.
In `@docs/tr/skills/backend-patterns/SKILL.md`:
- Around line 296-303: Replace the demonstration console.error usage in the
error handler (the block that logs Unexpected error: and returns
NextResponse.json(...)) with a brief comment directing readers to the document's
structured logging example and advising to use a proper logging library in
production; reference the existing structured logging example in this document
and mention the console.error call and the error-handling return
(NextResponse.json) so reviewers can locate and update the error handler to
point to the structured logging pattern.
In `@docs/tr/skills/continuous-learning/SKILL.md`:
- Around line 80-119: Add an explicit "Örnekler" section to the SKILL.md to
match other skill docs and include short example usages and expected outcomes;
ensure the doc also contains the "Ne zaman kullanılır" (When to use) and "Nasıl
çalışır" (How it works) headings if missing. Update the markdown under the
existing "Neden Stop Hook?" and "İlgili" blocks by inserting a new "Örnekler"
heading with 2–3 concise examples (input, brief action, expected result) and
confirm the local headings match the project convention ("When to use" => "Ne
zaman kullanılır", "How it works" => "Nasıl çalışır") so tooling and readers
find them consistently.
In `@docs/tr/skills/docker-patterns/SKILL.md`:
- Around line 1-364: The doc is missing a short "Nasıl Çalışır" (How it works)
overview to match the recommended skill structure; add a brief paragraph-level
section titled "## Nasıl Çalışır" summarizing core concepts (what Docker Compose
does, multi-stage Dockerfile build flow, service discovery via Compose network
names, and volume vs bind mount tradeoffs) and place it between the YAML
frontmatter (containing name: docker-patterns) and the existing "## Ne Zaman
Aktifleştirmeli" heading so readers get a quick conceptual primer before the
usage patterns and examples.
In `@docs/tr/skills/e2e-testing/SKILL.md`:
- Around line 1-326: The skill (name: e2e-testing) is missing the required "Ne
Zaman Kullanmalı" and "Nasıl Çalışır" sections; add two new Markdown sections
titled "## Ne Zaman Kullanmalı" and "## Nasıl Çalışır" near the top (after the
short description/frontmatter) that briefly state when to use the patterns
(e.g., critical user flows, regression prevention, cross-browser testing) and a
short overview of Playwright architecture and the Page Object Model approach
(mention ItemsPage as an example POM) so the document conforms to the standard
skill format.
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: e911f1ba-9c05-4dbd-aee7-9eeab3ce5541
📒 Files selected for processing (139)
README.mddocs/tr/AGENTS.mddocs/tr/CHANGELOG.mddocs/tr/CLAUDE.mddocs/tr/CODE_OF_CONDUCT.mddocs/tr/CONTRIBUTING.mddocs/tr/README.mddocs/tr/SECURITY.mddocs/tr/SPONSORING.mddocs/tr/SPONSORS.mddocs/tr/TERMINOLOGY.mddocs/tr/TROUBLESHOOTING.mddocs/tr/agents/architect.mddocs/tr/agents/build-error-resolver.mddocs/tr/agents/chief-of-staff.mddocs/tr/agents/code-reviewer.mddocs/tr/agents/cpp-build-resolver.mddocs/tr/agents/cpp-reviewer.mddocs/tr/agents/database-reviewer.mddocs/tr/agents/doc-updater.mddocs/tr/agents/docs-lookup.mddocs/tr/agents/e2e-runner.mddocs/tr/agents/flutter-reviewer.mddocs/tr/agents/go-build-resolver.mddocs/tr/agents/go-reviewer.mddocs/tr/agents/harness-optimizer.mddocs/tr/agents/java-build-resolver.mddocs/tr/agents/java-reviewer.mddocs/tr/agents/kotlin-build-resolver.mddocs/tr/agents/kotlin-reviewer.mddocs/tr/agents/loop-operator.mddocs/tr/agents/planner.mddocs/tr/agents/python-reviewer.mddocs/tr/agents/pytorch-build-resolver.mddocs/tr/agents/refactor-cleaner.mddocs/tr/agents/rust-build-resolver.mddocs/tr/agents/rust-reviewer.mddocs/tr/agents/security-reviewer.mddocs/tr/agents/tdd-guide.mddocs/tr/agents/typescript-reviewer.mddocs/tr/commands/build-fix.mddocs/tr/commands/checkpoint.mddocs/tr/commands/code-review.mddocs/tr/commands/e2e.mddocs/tr/commands/eval.mddocs/tr/commands/evolve.mddocs/tr/commands/go-build.mddocs/tr/commands/go-review.mddocs/tr/commands/go-test.mddocs/tr/commands/instinct-export.mddocs/tr/commands/instinct-import.mddocs/tr/commands/instinct-status.mddocs/tr/commands/learn-eval.mddocs/tr/commands/learn.mddocs/tr/commands/multi-backend.mddocs/tr/commands/multi-execute.mddocs/tr/commands/multi-frontend.mddocs/tr/commands/multi-plan.mddocs/tr/commands/multi-workflow.mddocs/tr/commands/orchestrate.mddocs/tr/commands/plan.mddocs/tr/commands/pm2.mddocs/tr/commands/refactor-clean.mddocs/tr/commands/sessions.mddocs/tr/commands/setup-pm.mddocs/tr/commands/skill-create.mddocs/tr/commands/tdd.mddocs/tr/commands/test-coverage.mddocs/tr/commands/update-docs.mddocs/tr/commands/verify.mddocs/tr/contexts/dev.mddocs/tr/contexts/research.mddocs/tr/contexts/review.mddocs/tr/examples/CLAUDE.mddocs/tr/examples/README.mddocs/tr/examples/statusline.jsondocs/tr/examples/user-CLAUDE.mddocs/tr/rules/README.mddocs/tr/rules/common/agents.mddocs/tr/rules/common/coding-style.mddocs/tr/rules/common/development-workflow.mddocs/tr/rules/common/git-workflow.mddocs/tr/rules/common/hooks.mddocs/tr/rules/common/patterns.mddocs/tr/rules/common/performance.mddocs/tr/rules/common/security.mddocs/tr/rules/common/testing.mddocs/tr/rules/golang/coding-style.mddocs/tr/rules/golang/hooks.mddocs/tr/rules/golang/patterns.mddocs/tr/rules/golang/security.mddocs/tr/rules/golang/testing.mddocs/tr/rules/python/coding-style.mddocs/tr/rules/python/hooks.mddocs/tr/rules/python/patterns.mddocs/tr/rules/python/security.mddocs/tr/rules/python/testing.mddocs/tr/rules/typescript/coding-style.mddocs/tr/rules/typescript/hooks.mddocs/tr/rules/typescript/patterns.mddocs/tr/rules/typescript/security.mddocs/tr/rules/typescript/testing.mddocs/tr/skills/api-design/SKILL.mddocs/tr/skills/backend-patterns/SKILL.mddocs/tr/skills/coding-standards/SKILL.mddocs/tr/skills/continuous-learning-v2/SKILL.mddocs/tr/skills/continuous-learning/SKILL.mddocs/tr/skills/database-migrations/SKILL.mddocs/tr/skills/deployment-patterns/SKILL.mddocs/tr/skills/django-patterns/SKILL.mddocs/tr/skills/docker-patterns/SKILL.mddocs/tr/skills/e2e-testing/SKILL.mddocs/tr/skills/eval-harness/SKILL.mddocs/tr/skills/frontend-patterns/SKILL.mddocs/tr/skills/golang-patterns/SKILL.mddocs/tr/skills/golang-testing/SKILL.mddocs/tr/skills/jpa-patterns/SKILL.mddocs/tr/skills/kotlin-patterns/SKILL.mddocs/tr/skills/kotlin-testing/SKILL.mddocs/tr/skills/laravel-patterns/SKILL.mddocs/tr/skills/laravel-security/SKILL.mddocs/tr/skills/laravel-tdd/SKILL.mddocs/tr/skills/laravel-verification/SKILL.mddocs/tr/skills/nextjs-turbopack/SKILL.mddocs/tr/skills/postgres-patterns/SKILL.mddocs/tr/skills/python-patterns/SKILL.mddocs/tr/skills/python-testing/SKILL.mddocs/tr/skills/rust-patterns/SKILL.mddocs/tr/skills/rust-testing/SKILL.mddocs/tr/skills/security-review/SKILL.mddocs/tr/skills/springboot-patterns/SKILL.mddocs/tr/skills/springboot-security/SKILL.mddocs/tr/skills/springboot-tdd/SKILL.mddocs/tr/skills/springboot-verification/SKILL.mddocs/tr/skills/tdd-workflow/SKILL.mddocs/tr/skills/verification-loop/SKILL.mddocs/tr/the-longform-guide.mddocs/tr/the-security-guide.mddocs/tr/the-shortform-guide.md
|
|
||
| - **Semantik locator'lar kullanın**: `[data-testid="..."]` > CSS seçiciler > XPath | ||
| - **Koşulları bekleyin, zamanı değil**: `waitForResponse()` > `waitForTimeout()` | ||
| - **Otomatik bekleme yerleşik**: `page.locator().click()` otomatik bekler; ham `page.click()` beklemez |
There was a problem hiding this comment.
🧩 Analysis chain
🌐 Web query:
In Playwright, does page.click() include auto-wait/actionability checks like locator.click()? Please cite official Playwright docs and clarify differences between page.click and locator.click.
💡 Result:
Yes, page.click includes auto-wait/actionability checks similar to locator.click. Both perform the same actionability checks: the element must be visible, stable, receive events, and be enabled before clicking. Official Playwright documentation confirms this in the Auto-waiting guide (Actionability checks table lists both locator.click and page.click as requiring Visible: Yes, Stable: Yes, Receives Events: Yes, Enabled: Yes). Examples throughout the docs use page.click(selector) with auto-waiting, such as "// Click will auto-wait for the element" in Navigations docs. Differences: - page.click(selector) internally creates a locator, selects the first matching element (non-strict by default; clicks first if multiple match, no error). - locator.click is strict: errors if multiple elements match the locator; supports chaining, filtering, and reuse for better test resilience. - page.click is discouraged in favor of locators for consistency, strictness, and best practices (e.g., Migrating from Puppeteer guide recommends replacing page.click with page.locator(selector).click; StackOverflow confirms page.click auto-waits but lacks locator benefits). Use locator.click for modern Playwright tests.
Citations:
- 1: https://playwright.dev/docs/actionability
- 2: https://stackoverflow.com/questions/73079690/playwright-how-to-wait-for-an-element-to-be-clickable
- 3: https://playwright.bootcss.com/python/docs/next/actionability
- 4: https://stackoverflow.com/questions/78635949/playwright-with-typescript-difference-between-locator-click-and-page-click
- 5: https://playwright.dev/docs/next/puppeteer
- 6: https://playwright.dev/docs/api/class-locator
- 7: https://playwright.dev/docs/locators
- 8: https://playwright.dev/docs/api/class-page
Correct the misleading Playwright auto-wait behavior statement (Line 74).
The Turkish text incorrectly states that page.click() "beklemez" (does not wait). Both page.click() and page.locator().click() include identical auto-wait/actionability checks per official Playwright documentation. The actual difference is that locator.click() is strict (errors on multiple matches) while page.click() is non-strict, and locator.click() is the recommended modern approach for test resilience—not due to waiting behavior. Rewrite this line to accurately reflect that both include auto-wait, but clarify the distinction that matters (strictness and best practices, not waiting behavior).
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@docs/tr/agents/e2e-runner.md` at line 74, The current Turkish sentence
wrongly claims page.click() does not auto-wait; change it to state that both
page.click() and page.locator().click() perform Playwright's
auto-wait/actionability checks, and clarify the real difference: locator.click()
(or page.locator().click()) is strict (will error on multiple matches) and is
the recommended modern approach for resilience—not because it waits differently;
update the wording to reflect this distinction using the symbols page.click(),
page.locator().click(), and locator.click().
| | `pom.xml` | `mvn compile` | | ||
| | `build.gradle` | `./gradlew compileJava` | | ||
| | `go.mod` | `go build ./...` | | ||
| | `pyproject.toml` | `python -m py_compile` veya `mypy .` | |
There was a problem hiding this comment.
🧩 Analysis chain
🌐 Web query:
Does python -m py_compilevalidate an entire project directory by itself, and ispython -m compileall . the correct recursive alternative?
💡 Result:
No, python -m py_compile does not validate an entire project directory by itself. It only compiles explicitly named Python source files provided as arguments and does not search or recurse into directories. Yes, python -m compileall . is the correct recursive alternative, as it traverses the current directory (.) and all subdirectories recursively, compiling all .py files found.
Citations:
- 1: https://docs.python.org/3/library/py_compile.html
- 2: http://docs.python.org/lib/module-compileall.html
- 3: https://docs.python.org/3/library/compileall.html
Replace python -m py_compile with python -m compileall . for project-level validation.
python -m py_compile compiles only explicitly named files and does not recurse into directories. For project-wide compilation checks, use python -m compileall . instead, which traverses the directory and all subdirectories.
Suggested fix
-| `pyproject.toml` | `python -m py_compile` veya `mypy .` |
+| `pyproject.toml` | `python -m compileall .` veya `mypy .` |📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| | `pyproject.toml` | `python -m py_compile` veya `mypy .` | | |
| | `pyproject.toml` | `python -m compileall .` veya `mypy .` | |
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@docs/tr/commands/build-fix.md` at line 17, Replace the project-level
validation command in the table cell that currently shows `python -m py_compile`
with `python -m compileall .` so the docs recommend a recursive, project-wide
bytecode compile (locate the table entry containing the literal `python -m
py_compile` and update it to `python -m compileall .`). Ensure the surrounding
formatting of the table row remains intact.
| # Code Review | ||
|
|
||
| Commit edilmemiş değişikliklerin kapsamlı güvenlik ve kalite incelemesi: |
There was a problem hiding this comment.
Komut dosyası için frontmatter eklenmeli.
commands/ altında bu dosyada description frontmatter’ı yok; komut dokümantasyon şablonuyla hizalamak için metadata bloğu ekleyin.
🧩 Önerilen minimal düzeltme
+---
+description: Commit edilmemiş değişikliklerin kapsamlı güvenlik ve kalite incelemesini yap.
+---
+
# Code Review
Commit edilmemiş değişikliklerin kapsamlı güvenlik ve kalite incelemesi:Based on learnings: Follow contributing guidelines from CONTRIBUTING.md for agents (Markdown with frontmatter), skills (clear sections), commands (Markdown with description), and hooks (JSON with matcher and hooks array).
📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| # Code Review | |
| Commit edilmemiş değişikliklerin kapsamlı güvenlik ve kalite incelemesi: | |
| --- | |
| description: Commit edilmemiş değişikliklerin kapsamlı güvenlik ve kalite incelemesini yap. | |
| --- | |
| # Code Review | |
| Commit edilmemiş değişikliklerin kapsamlı güvenlik ve kalite incelemesi: |
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@docs/tr/commands/code-review.md` around lines 1 - 3, Bu dosyada eksik olan
YAML frontmatter bloğunu ekleyin: dosyanın en üstüne üç kısa çizgiyle
başlanan/bitirilen frontmatter içine en azından description alanını (ör.
description: "kısa açıklama") ekleyin ve gerekiyorsa title/metadatayı
tamamlayarak commands/ komut dokümantasyon şablonuyla uyumlu hale getirin;
hedeflenen semboller: frontmatter bloğu ve description alanı (description) — bu
alanı docs/tr/commands/code-review.md içeriğinin üstüne yerleştirin.
|
|
||
| - name: Upload artifacts | ||
| if: always() | ||
| uses: actions/upload-artifact@v3 |
There was a problem hiding this comment.
🧩 Analysis chain
🌐 Web query:
What is the currently recommended/supported version of GitHub Action actions/upload-artifact, and is v3 deprecated?
💡 Result:
The currently recommended and supported version of the GitHub Actions actions/upload-artifact is v7.0.0, the latest release. Versions v6 and above run on Node.js 24 and are fully supported on github.com. v3 is deprecated: it was scheduled for deprecation on November 30, 2024 (past the current date of 2026-03-22), and v3.2.2 explicitly states it is deprecated on github.com with a recommendation to use Node 20 variant only for GHES. Users should migrate to v4+ (now up to v7) for performance improvements and new features like direct file uploads in v7.
Citations:
- 1: https://github.com/actions/upload-artifact/releases
- 2: https://github.com/actions/upload-artifact
- 3: https://github.com/marketplace/actions/upload-a-build-artifact
- 4: https://www.v3-partners.com
- 5: actions/upload-artifact@v6...v7
- 6: https://newreleases.io/project/github/actions/upload-artifact/release/v7.0.0
Update artifact upload action to v4 or later (Line 278).
actions/upload-artifact@v3 is deprecated as of November 30, 2024 and no longer supported on github.com. Upgrade to v7 (the current recommended version) or at minimum v4+.
- uses: actions/upload-artifact@v3
+ uses: actions/upload-artifact@v7📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| uses: actions/upload-artifact@v3 | |
| uses: actions/upload-artifact@v7 |
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@docs/tr/commands/e2e.md` at line 278, The workflow step currently uses the
deprecated action reference "uses: actions/upload-artifact@v3"; update that to a
supported major version (preferably "uses: actions/upload-artifact@v7", or at
minimum "uses: actions/upload-artifact@v4") in the workflow file, then validate
the workflow (run GitHub Actions workflow lint/syntax check or a dry run) and
adjust any inputs/fields if the newer action version changed parameter names or
behavior.
| # Verification Komutu | ||
|
|
||
| Mevcut kod tabanı durumu üzerinde kapsamlı doğrulama çalıştır. |
There was a problem hiding this comment.
Komut metadata frontmatter’ı eksik.
Bu dosyada YAML frontmatter (description) yok; komut dokümanlarında metadata beklentisi varsa bu dosya komut keşfi/işlenmesi dışında kalabilir.
🧩 Önerilen minimal düzeltme
+---
+description: Mevcut kod tabanı durumu üzerinde kapsamlı doğrulama çalıştır.
+---
+
# Verification Komutu
Mevcut kod tabanı durumu üzerinde kapsamlı doğrulama çalıştır.Based on learnings: Follow contributing guidelines from CONTRIBUTING.md for agents (Markdown with frontmatter), skills (clear sections), commands (Markdown with description), and hooks (JSON with matcher and hooks array).
📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| # Verification Komutu | |
| Mevcut kod tabanı durumu üzerinde kapsamlı doğrulama çalıştır. | |
| --- | |
| description: Mevcut kod tabanı durumu üzerinde kapsamlı doğrulama çalıştır. | |
| --- | |
| # Verification Komutu | |
| Mevcut kod tabanı durumu üzerinde kapsamlı doğrulama çalıştır. |
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@docs/tr/commands/verify.md` around lines 1 - 3, Bu Markdown dokümanında eksik
olan YAML frontmatter (özellikle description alanı) eklenecek: dosyanın en
başına üç çizgiyle ayrılmış bir YAML bloğu ekleyin ve içine en azından
description: "Kısa komut açıklaması" olacak şekilde yerel dilde açıklama;
gerekiyorsa title, tags veya locale gibi CONTRIBUTING.md’de tanımlı diğer
metadata alanlarını de dahil edin; frontmatter bloğunu (--- ... ---) ekleyip
kaydederek komut keşfi/işlenmesi için gereken metadata’yı sağlamış olacaksınız.
| ### Status Line Konfigürasyonu | ||
| ```bash | ||
| # settings.json dosyanıza ekleyin | ||
| cat docs/tr/examples/statusline.json >> ~/.claude/settings.json | ||
| ``` |
There was a problem hiding this comment.
settings.json için >> ile ekleme komutu dosyayı bozabilir.
cat ... >> ~/.claude/settings.json mevcut JSON sonuna ham içerik ekler; bu çoğu durumda geçersiz JSON üretir. Komutu “manual merge” veya güvenli JSON birleştirme adımıyla değiştirin.
💡 Önerilen dokümantasyon düzeltmesi
### Status Line Konfigürasyonu
```bash
-# settings.json dosyanıza ekleyin
-cat docs/tr/examples/statusline.json >> ~/.claude/settings.json
+# statusline.json içeriğini ~/.claude/settings.json içine JSON yapısını koruyarak birleştirin
+# (ham append yapmayın)
</details>
<details>
<summary>🤖 Prompt for AI Agents</summary>
Verify each finding against the current code and only fix it if needed.
In @docs/tr/examples/README.md around lines 65 - 69, Update the “Status Line
Konfigürasyonu” instructions to remove the raw append command and instead tell
the user to merge JSON safely: explain that they should not use cat ... >> to
append, and instruct them to either manually merge the contents of
statusline.json into their settings.json maintaining valid JSON, or use a
JSON-aware merge/cli tool (reference the statusline.json file and the
settings.json target in the README section header) and replace the current
two-line bash snippet with this safer guidance.
</details>
<!-- fingerprinting:phantom:triton:hawk -->
<!-- This is an auto-generated comment by CodeRabbit -->
| { | ||
| "statusLine": { | ||
| "type": "command", | ||
| "command": "input=$(cat); user=$(whoami); cwd=$(echo \"$input\" | jq -r '.workspace.current_dir' | sed \"s|$HOME|~|g\"); model=$(echo \"$input\" | jq -r '.model.display_name'); time=$(date +%H:%M); remaining=$(echo \"$input\" | jq -r '.context_window.remaining_percentage // empty'); transcript=$(echo \"$input\" | jq -r '.transcript_path'); todo_count=$([ -f \"$transcript\" ] && grep -c '\"type\":\"todo\"' \"$transcript\" 2>/dev/null || echo 0); cd \"$(echo \"$input\" | jq -r '.workspace.current_dir')\" 2>/dev/null; branch=$(git rev-parse --abbrev-ref HEAD 2>/dev/null || echo ''); status=''; [ -n \"$branch\" ] && { [ -n \"$(git status --porcelain 2>/dev/null)\" ] && status='*'; }; B='\\033[38;2;30;102;245m'; G='\\033[38;2;64;160;43m'; Y='\\033[38;2;223;142;29m'; M='\\033[38;2;136;57;239m'; C='\\033[38;2;23;146;153m'; R='\\033[0m'; T='\\033[38;2;76;79;105m'; printf \"${C}${user}${R}:${B}${cwd}${R}\"; [ -n \"$branch\" ] && printf \" ${G}${branch}${Y}${status}${R}\"; [ -n \"$remaining\" ] && printf \" ${M}ctx:${remaining}%%${R}\"; printf \" ${T}${model}${R} ${Y}${time}${R}\"; [ \"$todo_count\" -gt 0 ] && printf \" ${C}todos:${todo_count}${R}\"; echo", |
There was a problem hiding this comment.
🧩 Analysis chain
🏁 Script executed:
#!/bin/bash
# Repro: grep -c with no matches prints 0 but exits non-zero
tmp="$(mktemp)"
printf '{"type":"note"}\n' > "$tmp"
raw=$([ -f "$tmp" ] && grep -c '"type":"todo"' "$tmp" 2>/dev/null || echo 0)
printf 'raw todo_count value => [%s]\n' "$raw"
# Expected problematic output: [0
# 0]
fixed=0
if [ -f "$tmp" ]; then
fixed=$(grep -c '"type":"todo"' "$tmp" 2>/dev/null)
fi
printf 'fixed todo_count value => [%s]\n' "$fixed"
rm -f "$tmp"Repository: affaan-m/everything-claude-code
Length of output: 134
grep -c ile 0 eşleşme durumunda todo_count hatalı bir değer alıyor.
grep -c hiç eşleşme bulamazsa "0" yazdırıp sıfır olmayan çıkış kodu döndürür. Mevcut kod || echo 0 ile bu non-zero koşulunu yakalayıp ikinci bir "0" ekler, bu da değişkende "0\n0" (yeni satır içeren) sonucunu verir. Sonraki [ "$todo_count" -gt 0 ] karşılaştırması bu geçersiz değerle hata verir.
Düzeltme
- ... transcript=$(echo "$input" | jq -r '.transcript_path'); todo_count=$([ -f "$transcript" ] && grep -c '"type":"todo"' "$transcript" 2>/dev/null || echo 0); cd "$(echo "$input" | jq -r '.workspace.current_dir')" 2>/dev/null; ...
+ ... transcript=$(echo "$input" | jq -r '.transcript_path'); todo_count=0; if [ -f "$transcript" ]; then todo_count=$(grep -c '"type":"todo"' "$transcript" 2>/dev/null); fi; cd "$(echo "$input" | jq -r '.workspace.current_dir')" 2>/dev/null; ...🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@docs/tr/examples/statusline.json` at line 4, The todo_count assignment is
producing "0\n0" when grep -c finds no matches because grep exits non-zero and
the subsequent || echo 0 appends another zero; update the command that sets
todo_count (the expression using todo_count, transcript and grep -c
'"type":"todo"') so that the grep's non-zero exit is swallowed (e.g. wrap the
grep invocation in a grouped command and append "|| true") before falling back
to echo 0 when the transcript file is missing, ensuring only a single numeric
"0" is produced and [ "$todo_count" -gt 0 ] works correctly.
| const sortedMarkets = useMemo(() => { | ||
| return markets.sort((a, b) => b.volume - a.volume) | ||
| }, [markets]) |
There was a problem hiding this comment.
“İYİ” örnekte mutasyon var; immutable örneğe çevrilmeli.
Line 400’deki markets.sort(...) diziyi yerinde değiştirir. Bu, dokümanın “değişmezlik” ilkesine ters düşüyor ve React’ta yan etki üretebilir.
🔧 Önerilen düzeltme
-const sortedMarkets = useMemo(() => {
- return markets.sort((a, b) => b.volume - a.volume)
-}, [markets])
+const sortedMarkets = useMemo(() => {
+ return [...markets].sort((a, b) => b.volume - a.volume)
+}, [markets])📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| const sortedMarkets = useMemo(() => { | |
| return markets.sort((a, b) => b.volume - a.volume) | |
| }, [markets]) | |
| const sortedMarkets = useMemo(() => { | |
| return [...markets].sort((a, b) => b.volume - a.volume) | |
| }, [markets]) |
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@docs/tr/skills/coding-standards/SKILL.md` around lines 399 - 401,
sortedMarkets hesaplanırken useMemo içindeki markets.sort(...) diziyi yerinde
mutasyona uğratarak değişmezlik ilkesini ihlal ediyor; bunu düzeltmek için
useMemo içerisindeki markets.sort çağrısını kopya bir dizi üzerinde çalıştıracak
şekilde değiştir (örneğin markets.slice() veya [...markets] ile yeni bir dizi
oluşturup o diziyi sort et) ve sortedMarkets ile useMemo kullanımını buna göre
güncelle; ilgili yerleri bulmak için sortedMarkets, useMemo ve markets.sort
sembollerine bak.
| go test -bench=. -benchmem ./... | ||
|
|
||
| # Fuzzing çalıştır | ||
| go test -fuzz=FuzzParse -fuzztime=30s ./... |
There was a problem hiding this comment.
Fuzz komutu örnek fonksiyon adıyla uyuşmuyor.
Line 675’te -fuzz=FuzzParse yazıyor, ama dokümandaki fonksiyon FuzzParseJSON (Line 467). Komut doğrudan kopyalanınca hedef testi eşlemeyebilir.
🔧 Önerilen düzeltme
-go test -fuzz=FuzzParse -fuzztime=30s ./...
+go test -fuzz=FuzzParseJSON -fuzztime=30s ./...📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| go test -fuzz=FuzzParse -fuzztime=30s ./... | |
| go test -fuzz=FuzzParseJSON -fuzztime=30s ./... |
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@docs/tr/skills/golang-testing/SKILL.md` at line 675, The fuzz command's
target name doesn't match the actual test function: update the example command
to reference the real fuzz function name FuzzParseJSON (or use a pattern that
matches it, e.g. FuzzParse*) so the command will run the intended test; locate
the example command and replace `-fuzz=FuzzParse` with `-fuzz=FuzzParseJSON` (or
a matching pattern) where the CLI example is shown.
| // En son versiyonları kontrol et: https://kotlinlang.org/docs/releases.html | ||
| plugins { | ||
| kotlin("jvm") version "2.3.10" | ||
| kotlin("plugin.serialization") version "2.3.10" | ||
| id("io.ktor.plugin") version "3.4.0" | ||
| id("org.jetbrains.kotlinx.kover") version "0.9.7" | ||
| id("io.gitlab.arturbosch.detekt") version "1.23.8" | ||
| } | ||
|
|
||
| group = "com.example" | ||
| version = "1.0.0" | ||
|
|
||
| kotlin { | ||
| jvmToolchain(21) | ||
| } | ||
|
|
||
| dependencies { | ||
| // Ktor | ||
| implementation("io.ktor:ktor-server-core:3.4.0") | ||
| implementation("io.ktor:ktor-server-netty:3.4.0") | ||
| implementation("io.ktor:ktor-server-content-negotiation:3.4.0") | ||
| implementation("io.ktor:ktor-serialization-kotlinx-json:3.4.0") | ||
|
|
||
| // Exposed | ||
| implementation("org.jetbrains.exposed:exposed-core:1.0.0") | ||
| implementation("org.jetbrains.exposed:exposed-dao:1.0.0") | ||
| implementation("org.jetbrains.exposed:exposed-jdbc:1.0.0") | ||
| implementation("org.jetbrains.exposed:exposed-kotlin-datetime:1.0.0") | ||
|
|
||
| // Koin | ||
| implementation("io.insert-koin:koin-ktor:4.2.0") | ||
|
|
||
| // Coroutines | ||
| implementation("org.jetbrains.kotlinx:kotlinx-coroutines-core:1.10.2") | ||
|
|
||
| // Test | ||
| testImplementation("io.kotest:kotest-runner-junit5:6.1.4") | ||
| testImplementation("io.kotest:kotest-assertions-core:6.1.4") | ||
| testImplementation("io.kotest:kotest-property:6.1.4") | ||
| testImplementation("io.mockk:mockk:1.14.9") | ||
| testImplementation("io.ktor:ktor-server-test-host:3.4.0") | ||
| testImplementation("org.jetbrains.kotlinx:kotlinx-coroutines-test:1.10.2") |
There was a problem hiding this comment.
🧩 Analysis chain
🌐 Web query:
What are the latest stable versions for: Kotlin Gradle plugin, Ktor, Koin, Detekt, Kover, Kotest, and MockK? Please provide official release/documentation links.
💡 Result:
Kotlin Gradle plugin: 2.3.20 https://kotlinlang.org/docs/releases.html, https://plugins.gradle.org/plugin/org.jetbrains.kotlin.android Ktor: 3.4.1 https://github.com/ktorio/ktor/releases, https://ktor.io/docs/releases.html Koin: 4.2.0 https://github.com/InsertKoinIO/koin/releases Detekt: 1.23.8 https://github.com/detekt/detekt/releases Kover: 0.9.7 https://kotlin.github.io/kotlinx-kover/gradle-plugin/ Kotest: 6.1.6 https://github.com/kotest/kotest/releases MockK: 1.14.9 https://github.com/mockk/mockk/releases
Citations:
- 1: https://kotlinlang.org/docs/plugin-releases.html
- 2: https://kotlinlang.org/docs/releases.html
- 3: https://plugins.gradle.org/plugin/org.jetbrains.kotlin.android
- 4: https://github.com/ktorio/ktor/releases
- 5: https://ktor.io/docs/releases.html
- 6: https://github.com/InsertKoinIO/koin/releases
- 7: https://github.com/detekt/detekt/releases
- 8: https://kotlin.github.io/kotlinx-kover/gradle-plugin/
- 9: https://github.com/kotest/kotest/releases
- 10: https://github.com/mockk/mockk/releases
Update outdated dependency versions or add an "example versions" label with a last-updated date.
The dependency versions in lines 384-423 are already outdated. Kotlin is at 2.3.20 (snippet shows 2.3.10), Ktor at 3.4.1 (snippet shows 3.4.0), and Kotest at 6.1.6 (snippet shows 6.1.4). Since this is documentation that readers will likely copy-paste, either update the versions to current stable releases or clearly mark this section as "example versions as of [DATE]" to set expectations that versions may drift.
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@docs/tr/skills/kotlin-patterns/SKILL.md` around lines 382 - 423, The
dependency versions in the plugins block
(kotlin("jvm")/kotlin("plugin.serialization") "2.3.10", id("io.ktor.plugin")
"3.4.0", kotest/test deps, etc.) and the dependencies section are outdated;
either update these to current stable releases (e.g., Kotlin 2.3.20, Ktor 3.4.1,
Kotest 6.1.6 and matching library versions) by changing the version strings in
the plugins and dependencies entries, or add a clear “example versions (last
updated YYYY-MM-DD)” label above the block so readers know these are
illustrative and may be out of date; ensure you modify the plugins { ... } and
dependencies { ... } blocks referenced by kotlin("jvm"), id("io.ktor.plugin"),
and the Kotest/testImplementation lines accordingly.
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@docs/tr/README.md`:
- Line 404: In docs/tr/README.md replace the root-English CONTRIBUTING link
target ../../CONTRIBUTING.md with the local Turkish copy (e.g. CONTRIBUTING.md
or ./CONTRIBUTING.md) so the "[CONTRIBUTING.md]" link in the line containing
"'ye bakın." points to the localized docs/tr/CONTRIBUTING.md; apply the same
change for the other occurrence around the previously noted line (the one at the
other occurrence near line 439).
- Line 9: Update the license badge link in the README so it points to the
repository root LICENSE rather than a broken local path; replace the current
badge link target "(LICENSE)" in the line starting with "[![License]" with a
path to the repo root (for example "../../LICENSE" from docs/tr/README.md) so
the badge links correctly to the root LICENSE file.
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: 326b88f2-0c59-444b-8d7d-4b3f25c42cd4
📒 Files selected for processing (2)
docs/tr/README.mddocs/tr/TROUBLESHOOTING.md
| <details> | ||
| <summary><b>Yeni bir skill veya agent'a nasıl katkıda bulunurum?</b></summary> | ||
|
|
||
| [CONTRIBUTING.md](../../CONTRIBUTING.md)'ye bakın. Kısa versiyon: |
There was a problem hiding this comment.
Use Turkish CONTRIBUTING link for localized navigation consistency.
These links currently jump to root CONTRIBUTING.md (English). Since docs/tr/CONTRIBUTING.md exists in this PR scope, link to the Turkish version for a consistent localized flow.
Suggested fix
-[CONTRIBUTING.md](../../CONTRIBUTING.md)'ye bakın. Kısa versiyon:
+[CONTRIBUTING.md](./CONTRIBUTING.md)'ye bakın. Kısa versiyon:-Lütfen katkıda bulunun! Rehber için [CONTRIBUTING.md](../../CONTRIBUTING.md)'ye bakın.
+Lütfen katkıda bulunun! Rehber için [CONTRIBUTING.md](./CONTRIBUTING.md)'ye bakın.Also applies to: 439-439
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@docs/tr/README.md` at line 404, In docs/tr/README.md replace the root-English
CONTRIBUTING link target ../../CONTRIBUTING.md with the local Turkish copy (e.g.
CONTRIBUTING.md or ./CONTRIBUTING.md) so the "[CONTRIBUTING.md]" link in the
line containing "'ye bakın." points to the localized docs/tr/CONTRIBUTING.md;
apply the same change for the other occurrence around the previously noted line
(the one at the other occurrence near line 439).
Update Turkish docs: change license badge link to point to repository root (../../LICENSE), increment displayed language count from 5 to 6, and remove two outdated related links from docs/tr/examples/README.md to keep references accurate.
…ithub.com/berkcangumusisik/everything-claude-code into berkcangumusisik/feat/turkish-translation
|
Analysis Failed
Troubleshooting
Retry: |
Co-authored-by: cubic-dev-ai[bot] <191113872+cubic-dev-ai[bot]@users.noreply.github.com>
|
Analysis Failed
Troubleshooting
Retry: |
Co-authored-by: cubic-dev-ai[bot] <191113872+cubic-dev-ai[bot]@users.noreply.github.com>
|
Analysis Failed
Troubleshooting
Retry: |
There was a problem hiding this comment.
Actionable comments posted: 1
♻️ Duplicate comments (1)
docs/tr/README.md (1)
404-404:⚠️ Potential issue | 🟡 MinorUse Turkish CONTRIBUTING link for localized navigation.
These links still jump to root English
CONTRIBUTING.md. Since this localized file exists underdocs/tr/, point both to./CONTRIBUTING.mdfor a consistent Turkish flow.Suggested fix
-[CONTRIBUTING.md](../../CONTRIBUTING.md)'ye bakın. Kısa versiyon: +[CONTRIBUTING.md](./CONTRIBUTING.md)'ye bakın. Kısa versiyon:-Lütfen katkıda bulunun! Rehber için [CONTRIBUTING.md](../../CONTRIBUTING.md)'ye bakın. +Lütfen katkıda bulunun! Rehber için [CONTRIBUTING.md](./CONTRIBUTING.md)'ye bakın.Also applies to: 439-439
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@docs/tr/README.md` at line 404, Change the localized Turkish link target so it points to the local contributing file instead of the English root: update the markdown link labeled "[CONTRIBUTING.md]" in docs/tr/README.md (both occurrences cited) to target "./CONTRIBUTING.md" rather than "../../CONTRIBUTING.md" so the Turkish docs navigate to the localized CONTRIBUTING file.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@docs/tr/README.md`:
- Around line 394-395: The in-page anchors `#cursor-ide-desteği` and
`#-opencode-desteği` referenced in the lines mentioning "Cursor IDE Desteği" and
"OpenCode Desteği" are broken; fix by either adding matching headings (e.g., a
"## Cursor IDE Desteği" and a "## OpenCode Desteği" with content) so the anchors
resolve, or update the two links to point to the actual existing section IDs in
the document if those sections already exist under different slugs; ensure the
anchor text and heading text match exactly so `#cursor-ide-desteği` and
`#-opencode-desteği` become valid targets.
---
Duplicate comments:
In `@docs/tr/README.md`:
- Line 404: Change the localized Turkish link target so it points to the local
contributing file instead of the English root: update the markdown link labeled
"[CONTRIBUTING.md]" in docs/tr/README.md (both occurrences cited) to target
"./CONTRIBUTING.md" rather than "../../CONTRIBUTING.md" so the Turkish docs
navigate to the localized CONTRIBUTING file.
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: 33b8a3ec-252b-49c0-9c7a-537a8470e18f
📒 Files selected for processing (4)
docs/tr/README.mddocs/tr/commands/checkpoint.mddocs/tr/commands/instinct-import.mddocs/tr/examples/README.md
✅ Files skipped from review due to trivial changes (3)
- docs/tr/examples/README.md
- docs/tr/commands/checkpoint.md
- docs/tr/commands/instinct-import.md
| - **Cursor**: `.cursor/` içinde önceden çevrilmiş config'ler. [Cursor IDE Desteği](#cursor-ide-desteği) bölümüne bakın. | ||
| - **OpenCode**: `.opencode/` içinde tam plugin desteği. [OpenCode Desteği](#-opencode-desteği) bölümüne bakın. |
There was a problem hiding this comment.
Fix broken in-page anchors in FAQ compatibility section.
#cursor-ide-desteği and #-opencode-desteği don’t appear to have matching headings in this document, so these links likely break. Either add the target sections or replace these with valid links.
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@docs/tr/README.md` around lines 394 - 395, The in-page anchors
`#cursor-ide-desteği` and `#-opencode-desteği` referenced in the lines
mentioning "Cursor IDE Desteği" and "OpenCode Desteği" are broken; fix by either
adding matching headings (e.g., a "## Cursor IDE Desteği" and a "## OpenCode
Desteği" with content) so the anchors resolve, or update the two links to point
to the actual existing section IDs in the document if those sections already
exist under different slugs; ensure the anchor text and heading text match
exactly so `#cursor-ide-desteği` and `#-opencode-desteği` become valid targets.
There was a problem hiding this comment.
1 issue found across 2 files (changes from recent commits).
Prompt for AI agents (unresolved issues)
Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.
<file name="docs/tr/commands/instinct-import.md">
<violation number="1" location="docs/tr/commands/instinct-import.md:29">
P2: Usage example recommends importing from a mutable external `main` branch URL, creating avoidable supply-chain risk in behavioral config ingestion.</violation>
</file>
Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.
|
|
||
| ``` | ||
| /instinct-import team-instincts.yaml | ||
| /instinct-import https://raw.githubusercontent.com/org/repo/main/instincts.yaml |
There was a problem hiding this comment.
P2: Usage example recommends importing from a mutable external main branch URL, creating avoidable supply-chain risk in behavioral config ingestion.
Prompt for AI agents
Check if this issue is valid — if so, understand the root cause and fix it. At docs/tr/commands/instinct-import.md, line 29:
<comment>Usage example recommends importing from a mutable external `main` branch URL, creating avoidable supply-chain risk in behavioral config ingestion.</comment>
<file context>
@@ -26,7 +26,7 @@ Yerel dosya yollarından veya HTTP(S) URL'lerinden içgüdüleri içe aktar.
/instinct-import team-instincts.yaml
-/instinct-import https://github.com/org/repo/instincts.yaml
+/instinct-import https://raw.githubusercontent.com/org/repo/main/instincts.yaml
/instinct-import team-instincts.yaml --dry-run
/instinct-import team-instincts.yaml --scope global --force
</file context>
</details>
```suggestion
/instinct-import https://raw.githubusercontent.com/org/repo/<commit-sha>/instincts.yaml
|
Thanks for the Turkish translation. Will review. 🦞 |
affaan-m
left a comment
There was a problem hiding this comment.
Comprehensive Turkish translation with 100 files covering all major docs, agents, commands, examples, and rules. Clean docs-only contribution. Thanks @berkcangumusisik!
|
@berkcangumusisik this PR has merge conflicts with main. Please rebase your branch to resolve. The conflicts are likely in README or other files that were updated after you created this PR. Once rebased and all CI passes, I'll merge. Thanks for the contribution! |
|
Analysis Failed
Troubleshooting
Retry: |
|
@affaan-m conflict resolved |
* Add Turkish (tr) docs and update README Add a full set of Turkish documentation under docs/tr (agents, changelog, CLAUDE guide, contributing, code of conduct, and many agents/commands/skills/rules files). Update README to include a link to the Turkish docs and increment the supported language count from 5 to 6. This commit adds localized guidance and references to help Turkish-speaking contributors and users. * Update docs/tr/TROUBLESHOOTING.md Co-authored-by: cubic-dev-ai[bot] <191113872+cubic-dev-ai[bot]@users.noreply.github.com> * Update docs/tr/README.md Co-authored-by: cubic-dev-ai[bot] <191113872+cubic-dev-ai[bot]@users.noreply.github.com> * docs(tr): fix license link and update readmes Update Turkish docs: change license badge link to point to repository root (../../LICENSE), increment displayed language count from 5 to 6, and remove two outdated related links from docs/tr/examples/README.md to keep references accurate. * Update docs/tr/commands/instinct-import.md Co-authored-by: cubic-dev-ai[bot] <191113872+cubic-dev-ai[bot]@users.noreply.github.com> * Update docs/tr/commands/checkpoint.md Co-authored-by: cubic-dev-ai[bot] <191113872+cubic-dev-ai[bot]@users.noreply.github.com> --------- Co-authored-by: cubic-dev-ai[bot] <191113872+cubic-dev-ai[bot]@users.noreply.github.com>
* Add Turkish (tr) docs and update README Add a full set of Turkish documentation under docs/tr (agents, changelog, CLAUDE guide, contributing, code of conduct, and many agents/commands/skills/rules files). Update README to include a link to the Turkish docs and increment the supported language count from 5 to 6. This commit adds localized guidance and references to help Turkish-speaking contributors and users. * Update docs/tr/TROUBLESHOOTING.md Co-authored-by: cubic-dev-ai[bot] <191113872+cubic-dev-ai[bot]@users.noreply.github.com> * Update docs/tr/README.md Co-authored-by: cubic-dev-ai[bot] <191113872+cubic-dev-ai[bot]@users.noreply.github.com> * docs(tr): fix license link and update readmes Update Turkish docs: change license badge link to point to repository root (../../LICENSE), increment displayed language count from 5 to 6, and remove two outdated related links from docs/tr/examples/README.md to keep references accurate. * Update docs/tr/commands/instinct-import.md Co-authored-by: cubic-dev-ai[bot] <191113872+cubic-dev-ai[bot]@users.noreply.github.com> * Update docs/tr/commands/checkpoint.md Co-authored-by: cubic-dev-ai[bot] <191113872+cubic-dev-ai[bot]@users.noreply.github.com> --------- Co-authored-by: cubic-dev-ai[bot] <191113872+cubic-dev-ai[bot]@users.noreply.github.com>
feat(i18n): add comprehensive Turkish translation
Add a full set of Turkish documentation under docs/tr (agents, changelog, CLAUDE guide, contributing, code of conduct, and many agents/commands/skills/rules files). Update README to include a link to the Turkish docs and increment the supported language count from 5 to 6. This commit adds localized guidance and references to help Turkish-speaking contributors and users.
What Changed
Documentation Translation (137 files)
DevOps skills
Main README Updates
Why This Change
Turkish is a widely spoken language with a growing developer community. This translation:
Testing Done
node tests/run-all.js)Type of Change
fix:Bug fixfeat:New featurerefactor:Code refactoringdocs:Documentationtest:Testschore:Maintenance/toolingci:CI/CD changesSecurity & Quality Checklist
Documentation
Translation Methodology
All translations follow these principles:
Related Issues/PRs
This PR contributes Turkish language support similar to existing translations:
Branch:
berkcangumusisik/feat/turkish-translationReviewer Notes: This is a large PR (137 files) but all changes are localization only. No code or functionality changes. Files follow the established
pattern in
docs/zh-CN/,docs/ja-JP/, anddocs/ko-KR/.Summary by cubic
Adds comprehensive Turkish (
tr) documentation and updates the rootREADMEto include Türkçe, bringing supported languages to 7. Also polishes command docs (instinct-import,checkpoint) and fixes links.New Features
docs/tr(agents, commands, skills, rules, guides, examples).READMElanguage selector with Türkçe and “Dil”; supported languages now 7.Bug Fixes
../../LICENSE.docs/tr/examples/README.mdand updated displayed language counts in Turkish readmes.docs/tr/README.md,docs/tr/TROUBLESHOOTING.md, and command docs (docs/tr/commands/instinct-import.md,docs/tr/commands/checkpoint.md).Written for commit f441ad2. Summary will update on new commits.
Summary by CodeRabbit
New Features
Documentation