feat: redesign i18n system with static patching and SKILL-based insta…#13
Merged
glauberlima merged 1 commit intomainfrom Feb 4, 2026
Merged
feat: redesign i18n system with static patching and SKILL-based insta…#13glauberlima merged 1 commit intomainfrom
glauberlima merged 1 commit intomainfrom
Conversation
…llation Complete redesign of internationalization architecture eliminating runtime overhead and simplifying installation workflow. Changes: 1. Static Message Patching System - Add patch-statusline.sh for build-time message compilation - Replace runtime config loading with @CONFIG_START/@CONFIG_END blocks - Replace runtime message loading with @MESSAGES_START/@MESSAGES_END blocks - Hardcode SHOW_MESSAGES/SHOW_COST flags as readonly constants - Refactor get_context_message() to use bash arrays (not pipe-delimited strings) - Performance: Eliminates 3-5ms i18n overhead (~100ms → ~95ms) 2. Simplify Message Format - Migrate from bash (.sh) to JSON format - Remove metadata and nested .tiers structure - Flatten to simple tier-to-array mapping - Remove backward compatibility code (~200 lines) - Breaking change: JSON-only format (no migration) 3. SKILL-based Installation - Add skills/install-statusline/SKILL.md (257 lines) - Remove install.sh (798 lines) and install.ps1 (548 lines) - Unified installation workflow via Claude Code SKILL system 4. Cross-Platform Fixes - Strip CRLF line endings from jq output (Windows Git Bash) - Force C locale for printf decimal formatting (LC_NUMERIC=C) - Preserve executable permissions during patching (chmod --reference) - Fix CI test environment compatibility 5. Test Suite Improvements - Consolidate redundant tests (76 unit + 24 integration = 90 total) - Remove circular tests (functions only in test file) - Add fixture-based integration test - Achieve 100% pass rate - Update for static patching (remove runtime loading tests) 6. Documentation Updates - CLAUDE.md: Reflect static patching architecture - README.md: Add patch-statusline.sh usage examples - messages/README.md: Document simplified JSON format Net changes: -1217 lines deleted, +830 lines added Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
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
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.
…llation
Complete redesign of internationalization architecture eliminating runtime overhead and simplifying installation workflow.
Changes:
Static Message Patching System
Simplify Message Format
SKILL-based Installation
Cross-Platform Fixes
Test Suite Improvements
Documentation Updates
Net changes: -1217 lines deleted, +830 lines added