Skip to content

fix: simplify GSM SMS + add infra configs#544

Merged
ShaerWare merged 1 commit intomainfrom
fix/gsm-sms-simplify-and-infra-configs
Mar 5, 2026
Merged

fix: simplify GSM SMS + add infra configs#544
ShaerWare merged 1 commit intomainfrom
fix/gsm-sms-simplify-and-infra-configs

Conversation

@ShaerWare
Copy link
Owner

Summary

  • Removed complex PDU encoding/decoding from GSM SMS (~230 lines deleted)
  • SMS now uses simple text mode + AT+CSCS="UCS2" for Cyrillic — proven working with SIM7600E-H
  • Removed unused _encode_phone_pdu, _build_sms_pdu, _serial_send_sms_pdu, _decode_sms_pdu, _unpack_gsm7, _GSM7_DECODE_TABLE
  • Simplified read_sms and read_all_sms to text mode
  • Added ai-secretary.service — systemd unit for local dev
  • Added landing/nginx-landing.conf — nginx config for landing page

NEWS

🔧 Упрощение отправки SMS

Убрана сложная PDU-кодировка SMS в пользу текстового режима с UCS2.
Русские SMS теперь отправляются проще и стабильнее.
Также добавлены конфигурационные файлы для systemd и nginx.

Test plan

  • SMS send (Cyrillic) works via admin panel
  • SMS send (ASCII) works
  • SMS listing and reading works
  • systemd unit file syntax is valid
  • nginx config passes nginx -t

🤖 Generated with Claude Code

GSM SMS: removed complex PDU encoding/decoding (~230 lines).
SMS now uses text mode + AT+CSCS="UCS2" for Cyrillic —
simpler and proven working with SIM7600E-H.

Added: ai-secretary.service (systemd unit for local dev),
landing/nginx-landing.conf (nginx config for landing page).

## NEWS

🔧 **Упрощение отправки SMS**

Убрана сложная PDU-кодировка SMS в пользу текстового режима с UCS2.
Русские SMS теперь отправляются проще и стабильнее.
Также добавлены конфигурационные файлы для systemd и nginx.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@ShaerWare ShaerWare merged commit 7b7715b into main Mar 5, 2026
@ShaerWare ShaerWare deleted the fix/gsm-sms-simplify-and-infra-configs branch March 5, 2026 17:56
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