core/cli - Lethean Network

core/cli

Fork 0

Commit graph

Author	SHA1	Message	Date
Snider	7f4682ab17	fix(i18n): address remaining code review issues - service.go: Simplify Default() to just call Init() (sync.Once handles idempotency) - service.go: Add nil check to SetDefault() with panic - service.go: Add documentation for ModeStrict panic behavior - loader.go: Add LanguagesErr() to expose directory scan errors - loader.go: Use path.Join instead of filepath.Join for fs.FS compatibility - transform.go: Add uint, uint8-64, int8, int16 support to type converters - grammar.go: Replace deprecated strings.Title with unicode-aware implementation - i18n_test.go: Add comprehensive concurrency tests with race detector Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>	2026-01-30 19:10:28 +00:00
Snider	285a4582b0	fix(i18n): address thread-safety issues from code review - hooks.go: Use atomic.Value for missingKeyHandler global - loader.go: Use sync.Once for FSLoader.Languages() caching - service.go: Use atomic.Pointer[Service] for defaultService All globals that could race between goroutines now use proper synchronization primitives. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>	2026-01-30 19:02:30 +00:00
Snider	67e7e552f3	refactor(i18n): implement extensible handler chain architecture Refactor the i18n package for extensibility without breaking changes: - Add KeyHandler interface for pluggable namespace handlers - Add Loader interface for format-agnostic translation loading - Add TranslationContext for translation disambiguation - Implement 6 built-in handlers (Label, Progress, Count, Done, Fail, Numeric) - Update T() to use handler chain instead of hardcoded logic - Add handler management methods (AddHandler, PrependHandler, ClearHandlers) File reorganisation: - types.go: all type definitions - loader.go: Loader interface + FSLoader (from mutate.go, checks.go) - handler.go: KeyHandler interface + built-in handlers - context.go: TranslationContext + C() builder - hooks.go: renamed from actions.go - service.go: merged interfaces.go content Deleted: interfaces.go, mode.go, mutate.go, checks.go Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>	2026-01-30 18:42:41 +00:00

Author

SHA1

Message

Date

Snider

7f4682ab17

fix(i18n): address remaining code review issues

- service.go: Simplify Default() to just call Init() (sync.Once handles idempotency)
- service.go: Add nil check to SetDefault() with panic
- service.go: Add documentation for ModeStrict panic behavior
- loader.go: Add LanguagesErr() to expose directory scan errors
- loader.go: Use path.Join instead of filepath.Join for fs.FS compatibility
- transform.go: Add uint, uint8-64, int8, int16 support to type converters
- grammar.go: Replace deprecated strings.Title with unicode-aware implementation
- i18n_test.go: Add comprehensive concurrency tests with race detector

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

2026-01-30 19:10:28 +00:00

Snider

285a4582b0

fix(i18n): address thread-safety issues from code review

- hooks.go: Use atomic.Value for missingKeyHandler global
- loader.go: Use sync.Once for FSLoader.Languages() caching
- service.go: Use atomic.Pointer[Service] for defaultService

All globals that could race between goroutines now use proper
synchronization primitives.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

2026-01-30 19:02:30 +00:00

Snider

67e7e552f3

refactor(i18n): implement extensible handler chain architecture

Refactor the i18n package for extensibility without breaking changes:

- Add KeyHandler interface for pluggable namespace handlers
- Add Loader interface for format-agnostic translation loading
- Add TranslationContext for translation disambiguation
- Implement 6 built-in handlers (Label, Progress, Count, Done, Fail, Numeric)
- Update T() to use handler chain instead of hardcoded logic
- Add handler management methods (AddHandler, PrependHandler, ClearHandlers)

File reorganisation:
- types.go: all type definitions
- loader.go: Loader interface + FSLoader (from mutate.go, checks.go)
- handler.go: KeyHandler interface + built-in handlers
- context.go: TranslationContext + C() builder
- hooks.go: renamed from actions.go
- service.go: merged interfaces.go content

Deleted: interfaces.go, mode.go, mutate.go, checks.go

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

2026-01-30 18:42:41 +00:00

3 commits