fix: /clear memory güvenliği + agent tetikleme iyileştirmeleri#16
Merged
kiliczsh merged 3 commits intokomunite:mainfrom Apr 27, 2026
Merged
Conversation
…yileştirmeleri - CLAUDE.md: /clear çalıştırmadan önce memory.md ve günlük not yazma zorunluluğu eklendi - CLAUDE.md: Agent delegasyon kuralları tablosu eklendi - Tüm agent description'ları 3 parçalı yapıya çevrildi (ne yapar / ne zaman / ne zaman değil) - Eksik "Ne Zaman Çağrılırsın" bölümleri eklendi (6 agent) - Agent'lara <example> blokları eklendi (LLM routing doğruluğu için) - Agent'lar arası sınırlar netleştirildi (error-whisperer / unsticker / rubber-duck) - Türkçe belirsizlikler ve yazım hataları düzeltildi
📝 WalkthroughWalkthroughDocumentation and operational updates across Claude agent specs and core ops: agent descriptions, invocation boundaries, routing guidance, and trigger criteria clarified; CLAUDE.md adds agent delegation rules and stricter /clear preconditions; Changes
Sequence Diagram(s)sequenceDiagram
participant User as "User"
participant CLI as "CLAUDE `/clear`"
participant Hook as ".claude/hooks/pre-clear-check.sh"
participant FS as "Filesystem (.claude/workspace/DailyNotes)"
User->>CLI: invoke `/clear`
CLI->>Hook: run pre-clear-check.sh
Hook->>FS: check for `MMDDYY.md` existence
alt file missing
Hook-->>CLI: exit 1 + message (block)
CLI-->>User: abort `/clear` with blocking message
else file exists
Hook->>FS: grep for `## Oturum Devri`
alt header missing
Hook-->>CLI: exit 1 + message (block)
CLI-->>User: abort `/clear` with blocking message
else header present
Hook-->>CLI: exit 0 (ok)
CLI->>CLI: proceed with clear workflow (Adım 0b -> cleanup)
CLI-->>User: `/clear` completed
end
end
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
.claude/agents/coach.md (1)
13-13:⚠️ Potential issue | 🟠 MajorRemove unimplemented tool restrictions or document why they exist.
The
Bash(date:*,wc:*,find:*)declaration in coach.md is not enforced. According to CLAUDE.md (line 105), agent definitions must contain only definitive, complete instructions with no undocumented features. The syntax is:
- Not parsed or validated anywhere in the codebase
- Not enforced by
.claude/settings.json(global guard-bash.sh has no agent awareness)- Not documented in any guide
The coach agent can execute any Bash command that passes the global
guard-bash.shhook, making this declaration misleading. Either remove the syntax or implement + document agent-level tool restrictions.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In @.claude/agents/coach.md at line 13, Remove or implement the misleading "Bash(date:*,wc:*,find:*)" agent restriction in .claude/agents/coach.md: either delete the Bash(...) line so coach.md contains only accurate, enforceable directives, or implement enforcement and documentation by (a) adding agent-level validation logic that parses coach.md's tool restrictions and integrates them with the existing guard-bash.sh hook (or a new agent-aware wrapper) and (b) documenting the syntax and enforcement behavior in CLAUDE.md; reference the DSL string "Bash(date:*,wc:*,find:*)" and the guard script "guard-bash.sh" when making the change so reviewers can locate and verify the implementation.
🧹 Nitpick comments (1)
CLAUDE.md (1)
120-122: Turkish grammar: question particle spacing.Static analysis flagged that Turkish question particles should attach to the preceding word: "güncellendimi", "yazıldımı", "gösterildimi" instead of "güncellendi mi?", "yazıldı mı?", "gösterildi mi?".
This is a minor stylistic issue and doesn't affect functionality.
📝 Proposed Turkish grammar fix
-1. **memory.md güncellendi mi?** → Oturumda yeni karar, öncelik veya bağlam değişikliği varsa memory.md'ye yaz. Değişiklik yoksa bu adımı atla ama bunu bilinçli bir karar olarak al. -2. **Günlük nota devir yazıldı mı?** → `.claude/workspace/DailyNotes/AAGGYY.md` dosyasına oturum devri yazılmalı. Bu adım ASLA atlanamaz. -3. **Öğrenimler aday gösterildi mi?** → Oturumda keşfedilen kurallar/gerçekler varsa `knowledge-nominations.md` veya `knowledge-base.md`'ye yazılmalı. +1. **memory.md güncellendimi?** → Oturumda yeni karar, öncelik veya bağlam değişikliği varsa memory.md'ye yaz. Değişiklik yoksa bu adımı atla ama bunu bilinçli bir karar olarak al. +2. **Günlük nota devir yazıldımı?** → `.claude/workspace/DailyNotes/AAGGYY.md` dosyasına oturum devri yazılmalı. Bu adım ASLA atlanamaz. +3. **Öğrenimler aday gösterildimi?** → Oturumda keşfedilen kurallar/gerçekler varsa `knowledge-nominations.md` veya `knowledge-base.md`'ye yazılmalı.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@CLAUDE.md` around lines 120 - 122, The Turkish question particle in three checklist items is incorrectly spaced; update the phrases in CLAUDE.md by removing the space before the question particle so the strings "güncellendi mi?", "yazıldı mı?", "gösterildi mi?" become "güncellendimi?", "yazıldı mı?" -> "yazıldı mı?" actually should be "yazıldımı?", and "gösterildi mi?" -> "gösterildimi?"; locate and replace those exact substrings in the file (search for "güncellendi mi?", "yazıldı mı?", "gösterildi mi?") and ensure the punctuation remains (keep the trailing question mark).
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@CLAUDE.md`:
- Around line 116-126: The documentation requires that /clear be blocked until
mandatory writes complete, but no hook enforces this; add a pre-tool-use hook
(.claude/hooks/pre-tool-use-clear.sh) that checks the DAILY_NOTE file (e.g.,
.claude/workspace/DailyNotes/$(date +%d%m%y).md) for the "## Oturum Devri"
section and exit non‑zero to prevent /clear if missing, or alternatively update
.claude/hooks/log-stop-verdict.sh to perform the same check and return a
blocking status when the handoff is absent; ensure the new/modified hook returns
a failure code to stop session termination and is discoverable by the hook
runner.
---
Outside diff comments:
In @.claude/agents/coach.md:
- Line 13: Remove or implement the misleading "Bash(date:*,wc:*,find:*)" agent
restriction in .claude/agents/coach.md: either delete the Bash(...) line so
coach.md contains only accurate, enforceable directives, or implement
enforcement and documentation by (a) adding agent-level validation logic that
parses coach.md's tool restrictions and integrates them with the existing
guard-bash.sh hook (or a new agent-aware wrapper) and (b) documenting the syntax
and enforcement behavior in CLAUDE.md; reference the DSL string
"Bash(date:*,wc:*,find:*)" and the guard script "guard-bash.sh" when making the
change so reviewers can locate and verify the implementation.
---
Nitpick comments:
In `@CLAUDE.md`:
- Around line 120-122: The Turkish question particle in three checklist items is
incorrectly spaced; update the phrases in CLAUDE.md by removing the space before
the question particle so the strings "güncellendi mi?", "yazıldı mı?",
"gösterildi mi?" become "güncellendimi?", "yazıldı mı?" -> "yazıldı mı?"
actually should be "yazıldımı?", and "gösterildi mi?" -> "gösterildimi?"; locate
and replace those exact substrings in the file (search for "güncellendi mi?",
"yazıldı mı?", "gösterildi mi?") and ensure the punctuation remains (keep the
trailing question mark).
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: 6473f173-fe0b-43f5-8804-c4c45b53fe4a
📒 Files selected for processing (11)
.claude/agents/archaeologist.md.claude/agents/auditor.md.claude/agents/coach.md.claude/agents/debt-collector.md.claude/agents/error-whisperer.md.claude/agents/onboarding-sherpa.md.claude/agents/pr-ghostwriter.md.claude/agents/rubber-duck.md.claude/agents/unsticker.md.claude/agents/yak-shave-detector.mdCLAUDE.md
Comment on lines
+116
to
+126
| **`/clear` öncesi hafıza yazma garantisi (KESİN KURALLAR):** | ||
|
|
||
| `/clear` çalıştırmadan önce aşağıdaki adımlar TAMAMLANMALIDIR. Bu kurallar isteğe bağlı değildir: | ||
|
|
||
| 1. **memory.md güncellendi mi?** → Oturumda yeni karar, öncelik veya bağlam değişikliği varsa memory.md'ye yaz. Değişiklik yoksa bu adımı atla ama bunu bilinçli bir karar olarak al. | ||
| 2. **Günlük nota devir yazıldı mı?** → `.claude/workspace/DailyNotes/AAGGYY.md` dosyasına oturum devri yazılmalı. Bu adım ASLA atlanamaz. | ||
| 3. **Öğrenimler aday gösterildi mi?** → Oturumda keşfedilen kurallar/gerçekler varsa `knowledge-nominations.md` veya `knowledge-base.md`'ye yazılmalı. | ||
| 4. **Doğrulama** → Yukarıdaki yazma işlemleri tamamlanmadan `/clear` komutu çalıştırılMAZ. Acil durum modunda bile en az günlük nota devir notu yazılmalıdır. | ||
|
|
||
| ⚠️ Bu adımları atlayarak `/clear` çalıştırmak bağlam kaybına neden olur ve oturum sürekliliğini bozar. "/clear çalıştırmam gerekiyor" düşüncesi hiçbir zaman hafıza yazma adımlarını atlamak için geçerli bir neden değildir. | ||
|
|
There was a problem hiding this comment.
Enforcement gap: guarantee without mechanism.
The new rules state that /clear CANNOT run until mandatory writes complete (line 123), with DailyNotes handoff NEVER skippable (line 121). However, there is no enforcement mechanism:
.claude/hooks/session-reset.shonly creates directories, doesn't validate writes occurred.claude/hooks/log-stop-verdict.shlogs verdicts but doesn't block stop if handoff is missing- No PreToolUse hook prevents
/clearfrom running when preconditions aren't met
This creates a gap: the documented guarantee can be silently violated if Claude skips the writes or a session ends prematurely.
🛡️ Recommended enforcement approaches
Option 1: PreToolUse hook for /clear command
Create .claude/hooks/pre-tool-use-clear.sh that checks:
#!/bin/bash
# Block /clear if DailyNotes handoff is missing
DAILY_NOTE=".claude/workspace/DailyNotes/$(date +%d%m%y).md"
if ! grep -q "^## Oturum Devri" "$DAILY_NOTE" 2>/dev/null; then
echo "BLOCK: /clear çalıştırılamaz - günlük nota oturum devri yazılmadı"
exit 1
fiOption 2: Stop hook enforcement
Modify .claude/hooks/log-stop-verdict.sh to block session termination if handoff is missing:
# Check for required handoff before allowing stop
DAILY_NOTE=".claude/workspace/DailyNotes/$(date +%d%m%y).md"
if ! grep -q "^## Oturum Devri" "$DAILY_NOTE" 2>/dev/null; then
echo "- \`$TIMESTAMP\` | VERDICT | BLOCK | Oturum devri yazılmadan kapatılamaz" >> "$INCIDENT_LOG"
exit 1
fiBased on learnings: /clear should be blocked when preconditions aren't met to prevent context loss.
🧰 Tools
🪛 LanguageTool
[grammar] ~120-~120: Sual edatları ögdeki söznen beraber yazılmalı: "güncellendimi"
Context: ... isteğe bağlı değildir: 1. memory.md güncellendi mi? → Oturumda yeni karar, öncelik veya ...
(QUESTION_PARTICLE_SEPARATION)
[grammar] ~121-~121: Sual edatları ögdeki söznen beraber yazılmalı: "yazıldımı"
Context: ...karar olarak al. 2. Günlük nota devir yazıldı mı? → `.claude/workspace/DailyNotes/AAGG...
(QUESTION_PARTICLE_SEPARATION)
[grammar] ~122-~122: Sual edatları ögdeki söznen beraber yazılmalı: "gösterildimi"
Context: ...ım ASLA atlanamaz. 3. Öğrenimler aday gösterildi mi? → Oturumda keşfedilen kurallar/gerçe...
(QUESTION_PARTICLE_SEPARATION)
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@CLAUDE.md` around lines 116 - 126, The documentation requires that /clear be
blocked until mandatory writes complete, but no hook enforces this; add a
pre-tool-use hook (.claude/hooks/pre-tool-use-clear.sh) that checks the
DAILY_NOTE file (e.g., .claude/workspace/DailyNotes/$(date +%d%m%y).md) for the
"## Oturum Devri" section and exit non‑zero to prevent /clear if missing, or
alternatively update .claude/hooks/log-stop-verdict.sh to perform the same check
and return a blocking status when the handoff is absent; ensure the new/modified
hook returns a failure code to stop session termination and is discoverable by
the hook runner.
… tool syntax - Add pre-clear-check.sh hook to enforce DailyNotes handoff before /clear - Remove invalid Bash(pattern) syntax from coach.md and clear.md - Document enforcement mechanism in CLAUDE.md Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
There was a problem hiding this comment.
Actionable comments posted: 1
♻️ Duplicate comments (1)
CLAUDE.md (1)
123-125:⚠️ Potential issue | 🟠 Major“Tüm adımlar tamamlanmadan /clear çalışmaz” iddiası, mevcut enforcement ile tam örtüşmüyor.
Line 123-125 tam bloklama garantisi veriyor; fakat belirtilen mekanizma (
pre-clear-check.sh) yalnızca günlük devir notunu doğruluyor. Bu, kural metni ile teknik doğrulama arasında boşluk bırakıyor.Önerilen düzeltme (metni mekanizmayla hizalama)
- 4. **Doğrulama** → Yukarıdaki yazma işlemleri tamamlanmadan `/clear` komutu çalıştırılMAZ. Acil durum modunda bile en az günlük nota devir notu yazılmalıdır. + 4. **Doğrulama** → `/clear` öncesi en az günlük nota devir notu zorunludur ve teknik olarak doğrulanır. + `memory.md` ve öğrenim yazımı ise oturum sorumluluğudur; mümkünse kontrol edilebilir bir işaretle (örn. devir notunda checklist) kayıt altına alınmalıdır. - **Uygulama mekanizması:** `/clear` komutu (`.claude/commands/clear.md`) Adım 0'da `pre-clear-check.sh` hook'unu çalıştırarak günlük nota oturum devrinin yazılmış olduğunu doğrular. Kontrol başarısız olursa `/clear` ilerlemez. + **Uygulama mekanizması:** `/clear` komutu (`.claude/commands/clear.md`) Adım 0'da `pre-clear-check.sh` hook'unu çalıştırır. Mevcut kontrol günlük not + `## Oturum Devri` bölümünü doğrular; kapsam genişletilecekse hook buna göre güncellenmelidir.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@CLAUDE.md` around lines 123 - 125, The claim that "/clear cannot run until all steps complete" is too strong; update the CLAUDE.md wording to align with the actual enforcement: state that the /clear command (defined in .claude/commands/clear.md) will not proceed unless the pre-clear-check.sh hook verifies the daily handover note has been written, rather than asserting a blanket block on all write operations; explicitly reference pre-clear-check.sh and the verification it performs so the text matches the implemented mechanism.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In @.claude/commands/clear.md:
- Around line 19-27: Metin "Adım 0: Ön doğrulama — oturum devri kontrolü"
altındaki “Acil durum modunda bu adım atlanabilir” ifadesi pre-clear kontrolünü
fiilen baypas ediyor; değiştirerek pre-clear scripti
"$CLAUDE_PROJECT_DIR/.claude/hooks/pre-clear-check.sh" çalıştırılmadan /clear
işlemine izin verilmeyeceğini açıkça belirt (acil durumda da ön kontrol
zorunlu), veya eğer acil durumda anlık /clear’e izin verilecekse bunu sadece
geçici istisna olarak tanımla ve mutlaka hemen sonrasında Adım 3’ü (devir notunu
günlük nota yaz) çalıştırma ve pre-clear-check.sh çıktısını/nedenini kaydetme
gerekliliğini ekle; böylece hem "devir zorunlu" hedefi korunur hem acil durum
akışında zorunlu takip adımları netleşir.
---
Duplicate comments:
In `@CLAUDE.md`:
- Around line 123-125: The claim that "/clear cannot run until all steps
complete" is too strong; update the CLAUDE.md wording to align with the actual
enforcement: state that the /clear command (defined in
.claude/commands/clear.md) will not proceed unless the pre-clear-check.sh hook
verifies the daily handover note has been written, rather than asserting a
blanket block on all write operations; explicitly reference pre-clear-check.sh
and the verification it performs so the text matches the implemented mechanism.
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: f9d96e5b-d371-4203-bae1-5ee8c2a6e1f1
📒 Files selected for processing (4)
.claude/agents/coach.md.claude/commands/clear.md.claude/hooks/pre-clear-check.shCLAUDE.md
✅ Files skipped from review due to trivial changes (1)
- .claude/hooks/pre-clear-check.sh
🚧 Files skipped from review as they are similar to previous changes (1)
- .claude/agents/coach.md
Acil durum modunda da devir notu zorunlu — bypass yolu kaldırıldı. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
There was a problem hiding this comment.
🧹 Nitpick comments (1)
.claude/commands/clear.md (1)
19-29: Acil durum akışını tek cümleyle daha da netleştirin.Bu bölüm çok daha iyi; ancak yorum farklarını önlemek için “acilde Adım 3’ten sonra Adım 0 doğrulaması tekrar zorunlu” ifadesini açıkça eklemenizi öneririm.
Önerilen metin güncellemesi
- Günlük nota oturum devrinin yazılmış olduğunu doğrula. Acil durumda önce kısa bir devir notu yaz, sonra doğrulamayı çalıştır: + Günlük nota oturum devrinin yazılmış olduğunu doğrula. Acil durumda önce kısa bir devir notu yaz; ardından Adım 0 doğrulamasını çalıştırmadan `/clear` ile devam etme:- **Acil durum modu** (Sıkıştırma/prompt-too-long): Okumaları atla, yalnızca bağlam içi hafızadan özetle, Adım 3'e git. + **Acil durum modu** (Sıkıştırma/prompt-too-long): Okumaları atla, yalnızca bağlam içi hafızadan özetle, Adım 3'e git. Not: Adım 3 sonrası Adım 0 doğrulaması yine zorunludur.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In @.claude/commands/clear.md around lines 19 - 29, Add a single clear sentence to the "Acil durum" flow in the "Adım 0" section stating that if the pre-clear check fails and the user completes Adım 3 (writing the transfer note), they must re-run and pass the Adım 0 pre-validation before invoking /clear; reference the existing pre-check command "$CLAUDE_PROJECT_DIR/.claude/hooks/pre-clear-check.sh", mention "Adım 3" and "/clear" by name, and place the sentence immediately after the line that says to stop when the command exits non‑zero so readers see the required re-validation step.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Nitpick comments:
In @.claude/commands/clear.md:
- Around line 19-29: Add a single clear sentence to the "Acil durum" flow in the
"Adım 0" section stating that if the pre-clear check fails and the user
completes Adım 3 (writing the transfer note), they must re-run and pass the Adım
0 pre-validation before invoking /clear; reference the existing pre-check
command "$CLAUDE_PROJECT_DIR/.claude/hooks/pre-clear-check.sh", mention "Adım 3"
and "/clear" by name, and place the sentence immediately after the line that
says to stop when the command exits non‑zero so readers see the required
re-validation step.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Bu PR, topluluk geri bildirimine dayanarak üç temel iyileştirme yapar:
1.
/clearöncesi memory yazma garantisi/clearçalıştırmadan önce memory.md ve günlük not yazılması zorunluluğu eklendi/clearöncesi memory yazma adımlarını atlayabiliyor ve bu bağlam kaybına neden oluyor2. Agent description'ları güçlendirildi
<example>blokları eklendi (LLM routing doğruluğu için)3. Türkçe netlik ve yazım düzeltmeleri
Değişen dosyalar
CLAUDE.md— /clear güvenliği + delegasyon tablosu.claude/agents/*.md— 10 agent dosyasıTest plan
/clearkomutunu çalıştırıp memory.md ve günlük notun yazıldığını doğrulaSummary by CodeRabbit