Skip to content

fix: localize agent skill options#1290

Open
ziwon wants to merge 1 commit into
Tencent:mainfrom
ziwon:codex/localize-agent-skill-options
Open

fix: localize agent skill options#1290
ziwon wants to merge 1 commit into
Tencent:mainfrom
ziwon:codex/localize-agent-skill-options

Conversation

@ziwon
Copy link
Copy Markdown

@ziwon ziwon commented May 12, 2026

Description

Localizes the built-in Agent Skills displayed in the agent editor.

The backend returns Skill metadata directly from SKILL.md, so the skill cards could appear in Chinese even when the UI locale is Korean or English. This PR adds display-only localized names and descriptions for known preloaded skills while preserving the original skill identifiers used by selected_skills and backend execution.

It also keeps the Skills menu gated by sandbox availability and defaults the loading state to visible until the backend explicitly reports skills_available: false, avoiding a brief disappearance during dependency loading.

Type of Change

  • 🐛 Bug fix
  • ✨ New feature
  • 💥 Breaking change
  • 📚 Documentation update
  • 🎨 Refactor
  • ⚡ Performance improvement
  • 🧪 Test
  • 🔧 Configuration / Build / CI

Related Issue

N/A

Testing

  • Ran npm ci in frontend/
  • Ran npm run build-only in frontend/
  • Verified the change is limited to display text: checkbox values still use the original Skill name

Checklist

  • make fmt && make lint && make test pass locally
  • Self-reviewed the code
  • Added/updated tests covering the change
  • Updated related documentation (README, docs/, Swagger annotations, etc.)
  • Breaking changes are clearly called out in the description above

Screenshots / Recordings

Not attached. This is a small text-only UI localization fix for the Agent Skills selection list.

@ziwon ziwon marked this pull request as ready for review May 12, 2026 11:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant