automaker

ros/automaker

Fork 0

mirror of https://github.com/AutoMaker-Org/automaker.git synced 2026-01-30 14:22:02 +00:00

Commit Graph

Author	SHA1	Message	Date
SuperComboGamer	584f5a3426	Merge main into massive-terminal-upgrade Resolves merge conflicts: - apps/server/src/routes/terminal/common.ts: Keep randomBytes import, use @automaker/utils for createLogger - apps/ui/eslint.config.mjs: Use main's explicit globals list with XMLHttpRequest and MediaQueryListEvent additions - apps/ui/src/components/views/terminal-view.tsx: Keep our terminal improvements (killAllSessions, beforeunload, better error handling) - apps/ui/src/config/terminal-themes.ts: Keep our search highlight colors for all themes - apps/ui/src/store/app-store.ts: Keep our terminal settings persistence improvements (merge function) 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>	2025-12-21 20:27:44 -05:00
Kacper	49a5a7448c	fix: Address PR review feedback for shared packages This commit addresses all "Should Fix" items from the PR review: 1. Security Documentation (platform package) - Added comprehensive inline documentation in security.ts explaining why path validation is disabled - Added Security Model section to platform README.md - Documented rationale, implications, and future re-enabling steps 2. Model Resolver Tests - Created comprehensive test suite (34 tests, 100% coverage) - Added vitest configuration with strict coverage thresholds - Tests cover: alias resolution, full model strings, priority handling, edge cases, and integration scenarios - Updated package.json with test scripts and vitest dependency 3. Feature Loader Logging Migration - Replaced all console.log/warn/error calls with @automaker/utils logger - Consistent with rest of codebase logging pattern - Updated corresponding tests to match new logger format 4. Module Format Consistency - Verified all packages use consistent module formats (ESM) - No changes needed All tests passing (632 tests across 31 test files). 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>	2025-12-21 00:05:42 +01:00

Author

SHA1

Message

Date

SuperComboGamer

584f5a3426

Merge main into massive-terminal-upgrade

Resolves merge conflicts:
- apps/server/src/routes/terminal/common.ts: Keep randomBytes import, use @automaker/utils for createLogger
- apps/ui/eslint.config.mjs: Use main's explicit globals list with XMLHttpRequest and MediaQueryListEvent additions
- apps/ui/src/components/views/terminal-view.tsx: Keep our terminal improvements (killAllSessions, beforeunload, better error handling)
- apps/ui/src/config/terminal-themes.ts: Keep our search highlight colors for all themes
- apps/ui/src/store/app-store.ts: Keep our terminal settings persistence improvements (merge function)

🤖 Generated with [Claude Code](https://claude.com/claude-code)

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

2025-12-21 20:27:44 -05:00

Kacper

49a5a7448c

fix: Address PR review feedback for shared packages

This commit addresses all "Should Fix" items from the PR review:

1. Security Documentation (platform package)
   - Added comprehensive inline documentation in security.ts explaining
     why path validation is disabled
   - Added Security Model section to platform README.md
   - Documented rationale, implications, and future re-enabling steps

2. Model Resolver Tests
   - Created comprehensive test suite (34 tests, 100% coverage)
   - Added vitest configuration with strict coverage thresholds
   - Tests cover: alias resolution, full model strings, priority handling,
     edge cases, and integration scenarios
   - Updated package.json with test scripts and vitest dependency

3. Feature Loader Logging Migration
   - Replaced all console.log/warn/error calls with @automaker/utils logger
   - Consistent with rest of codebase logging pattern
   - Updated corresponding tests to match new logger format

4. Module Format Consistency
   - Verified all packages use consistent module formats (ESM)
   - No changes needed

All tests passing (632 tests across 31 test files).

🤖 Generated with [Claude Code](https://claude.com/claude-code)

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

2025-12-21 00:05:42 +01:00

2 Commits