Skip to content

fix: misc fixes — deprecated logRetention, Korean strings, gitignore cleanup#18

Open
tobigs wants to merge 4 commits intoserithemage:mainfrom
tobigs:pr/misc-fixes
Open

fix: misc fixes — deprecated logRetention, Korean strings, gitignore cleanup#18
tobigs wants to merge 4 commits intoserithemage:mainfrom
tobigs:pr/misc-fixes

Conversation

@tobigs
Copy link
Copy Markdown

@tobigs tobigs commented Mar 26, 2026

Summary

  • fix: Replace deprecated logRetention with explicit logGroup + retention in ApiStack (CDK deprecated the logRetention prop on HttpApi/WebSocketApi in favor of explicit LogGroup constructs)
  • fix: Translate hardcoded Korean strings to English across container, gateway, and web packages
  • chore: Expand .gitignore with IDE files (.idea/, .vscode/, *.code-workspace) and common build artifacts
  • chore: Remove packages/shared/tsconfig.tsbuildinfo from tracking (already covered by .gitignore)

Test plan

  • npm run build — compiles clean
  • npm run lint — no lint errors
  • npm run test — all unit tests pass
  • npm run test:e2e — CDK synth E2E tests pass (36 tests)

🤖 Generated with Claude Code

Tobias added 4 commits March 26, 2026 17:38
The Telegram bot system prompt in bridge.ts was in Korean, causing the
AI to respond in Korean. Also translated all other user-facing Korean
strings across gateway handlers, identity service, container startup
notifications, monitoring dashboard descriptions, web UI, and tests.
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