Skip to content

add: base mini app, tests; refactor controller; update ui + use smiles - #10

Merged
nais2008 merged 19 commits into
devfrom
features/tg-mini-app
Mar 19, 2026
Merged

nais2008 merged 19 commits into
devfrom
features/tg-mini-app

Conversation

@nais2008

Copy link
Copy Markdown
Owner

добавлено:

  • тг мини апп, было проверенно, работает, но пока только через костыли, на верселе нет
  • деконструкция контроллера

чекни пж app/store/bot/worker.py, я чето так посмотрел и не увидел в нем смысла, когда у нас сейчас отдельный сервис

еще, на счет конфигов, сейчас какая-то путаница с ними, я думаю что по хорошему надо перейти на env, но как будто бы это будет немного больно

• Коротко: в проекте основной источник настроек — YAML, а env используется как подстановка и для части runtime-параметров.

  - Все сервисы стартуют с пути etc/config.yml через default_config_path():11, затем загружают конфиг в BaseService:21.
  - Чтение конфига делает load_config():80: файл YAML читается и прогоняется через os.path.expandvars(...) (строка 82:82).
    Это значит ${BOT_TOKEN} и т.п. в YAML берутся из переменных окружения процесса.
  - В Docker Compose каждый сервис получает /app/etc/config.yml из ./etc/config.docker.yml:36 (аналогично на 50:50, 68:68, и т.д.), то есть в
    контейнерах тоже источник YAML.
  - .env Python-кодом напрямую не читается (нет dotenv), но используется самим docker compose для подстановки ${...} и environment (например
    BOT_TOKEN) в docker-compose.yml:48.
  - Дополнительно есть env-переменные, читаемые напрямую кодом mini-app через os.getenv (например MINI_APP_CORS_ORIGINS, MINI_APP_SESSION_SECRET) в
    services/mini_app/main.py:24.
  - Отдельно: Alembic читает YAML напрямую в alembic/env.py:31 через yaml.safe_load (37:37) без expandvars.

  Итого: не “env или yml”, а “yml + env-подстановка” (плюс несколько env-only параметров для mini-app).

@nais2008
nais2008 merged commit 696a0e8 into dev Mar 19, 2026
1 check passed
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