fix: remove requirement for responses to be in Korean in review workflows

[ysm-dev]

No description provided.

[claude]

Claude finished @ysm-dev's task —— View job

---

Review Checklist

• Read and review changed workflow files for correctness/impact
• Read and review new CLAUDE.md for clarity and accuracy
• Read and review entrypoints/content.tsx for code quality and issues
• Summarize findings and recommendations

---

Inline review:

Workflow files

• .github/workflows/claude-review.yml / .github/workflows/claude.yml: ✅
  • Language restriction for Korean responses has been removed cleanly.
  • Only the prompt/comment content for Claude is affected—no risk to build/review automation; keeps the reviews language-agnostic.
  • All other workflow logic/permissions remain unchanged and minimal; triggers, job steps, and secrets are appropriately scoped.

CLAUDE.md

• Provides clear, detailed guidelines on project structure, build/dev/test commands, architectural overview, workflow, and conventions.
• Emphasizes not to create documentation files unless explicitly requested (repeat/reminder is good for bot contributors).
• Linting/formatting/dev processes are well explained.
• Section order and level of detail are strong; refers to all main architectural/utility choices.
• Minor: A couple typos ("Deffudle" should likely be “Defuddle” in useDeffudle).

entrypoints/content.tsx

• All changes present are additions/enhancements for handling notification and conversion logic—no regressions visible.
• All imported utilities are clean and explicit; strong separation of content modes (Defuddle, Readability, Turndown) and YouTube handling.
• Robust error handling is in place for async/copy operations and Defuddle fallback, with console logging for error diagnosis.
• Uses DOM APIs safely (clipboard/textarea/selection for fallback) with proper cleanup.
• Token counting and feedback are well wired for UX.
• Security: Does not expose secrets or unsafe browser APIs, validates inputs, and is safe for extension context.
• Testability: Logic is modular; message listeners and mode branching could be unit tested if not already.
• Suggestion: Minor code cleanup possible by extracting copyAndNotify or content processing logic for even clearer separation; not required for merge.

Documentation/Test needs:

• CLAUDE.md covers architecture and developer workflow—no gaps detected.
• Source file self-explains via code, variable naming, and fallback comments.
• If more tests exist, coverage for error/fallback/copy flows encouraged but not required for this change.

---

Summary & Recommendations

• The PR accomplishes its goal. No language restriction remains in the review prompt; everything else is low-risk or improved (with the new CLAUDE.md!).
• Code and workflow are safe, high quality; error and edge handling are solid.
• Gain: Improved flexibility for future reviews and clear project onboarding.
• Consider minor typo correction and, optionally, add tests for error/copy fallback logic over time.
• No action required before merge.

fix-13 branch