feat: Linux build support + dynamic runtime paths#24
Conversation
- Created localization files for MCP servers, overview, settings, skills, and slash commands in Chinese (zh-CN). - Implemented translation service to handle localization dynamically. - Added error messages and marketplace notifications in both English and Chinese. - Updated overview snapshot to reflect new localization structure.
|
Hi @scottzx, thank you for opening this. I really appreciate it, especially because this is one of the first external contributions to Skill Manager. Your Linux ARM validation is useful signal, and the direction is valuable. I did an initial review. There are a couple of ideas here that I want to keep:
I cannot merge this PR as-is because it also includes several unrelated changes, including i18n, a default port change, broad frontend rewrites, API error translation changes, and a snapshot/debug file. That makes the Linux support hard to review safely. To keep the project history clean, I’m going to create a smaller maintainer branch for the Linux support work and credit this PR / your contribution in the PR description. I’ll keep the scope narrow: Linux paths, artifact naming, CI/package smoke, and Settings path display. Thank you again. This PR is helpful as the starting point for Linux support, even if we don’t merge this exact branch. |
2 participants
Summary
linux/darwin) and produces correctly-named artifactsstorePath/marketplacePathfrom backend, frontend no longer hardcodes macOS-only paths~/.skill-manageraarch64for cross-platform consistencyChanges
scripts/build_release.pyskill_manager/paths.py~/.skill-managerrootskill_manager/application/settings/presenters.pystorePath/marketplacePathin settings APIfrontend/.../settings/api/types.tsstorePath/marketplacePathto interfacefrontend/.../screens/SettingsPage.tsxVerified
192.168.31.208:8000)🤖 Generated with Claude Code