Перенос в Main by intervisionlord · Pull Request #25 · intervisionlord/ShoppingBotNC

intervisionlord · 2025-11-23T16:04:59Z

No description provided.

… wirkflow

build(requirements): Понижение версии pydantic для совместимости с workflow

Создан pylint.yml для workflow github

- Хендлер получения данных по доскам - Модель деки - Роут для командыд получения деки - Обновлены настройки (подключение к NextCloud) - Обновлен .env.example #10 #12

feat(NC): Интеграция с nextcloud

- Докстринги в хендлере - Докстринги в логировании #15

#12

- Запросы через handler_requests - Улучшено ответное сообщение со списком - Добавлены докстринги #12 #15

…просов) #12

#12

…еделенной доске #12

- Разделение логики обработки команд - Подключение роутов в единый основной роут - Добавление модели стека - Отключен Isort на стороне pre-commit #12

#15

Feature/deck show cards

#15

#12

- Выборка осуществляется только по указанной доске и стеку - Отображение деталей карточки - Добавление покупок в карточку - Удаление покупок из карточки - Отметка о выполненной покупке - Разбиение логики на отдельные суброуты - Упрощение моделей и отказ от лишних моделей - Отказ от лишних хендлеров #12 #13 #9

- Оптимизация кода, чистка, упрощение - Убран flake

- Добавление элементов в текущую открытую карточку - Отслеживание нахождения в конкретной карточке

Feature/deck show cards

- Оптимизация обработки условий - Изменение строки с версией для sonar gate

style(Bot): Полировка кода

sonarqubecloud · 2025-11-23T16:05:28Z

Quality Gate passed

Issues
3 New issues
3 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

PavStepan · 2025-11-23T16:13:00Z

.scripts/project_tree.py

+    :param prefix: Префикс для отступов
+    :return: Генератор строк с древовидной структурой
    """
    ignore_dirs = [] if ignore_dirs is None else ignore_dirs


лучше используй
ignore_dirs = ignore_dirs or []

PavStepan · 2025-11-23T16:16:52Z

config/settings.py

-    LOG_LEVEL: str = Field("INFO", description="Уровень логирования")
+    LOG_LEVEL: str = "INFO"

    class Config:


Pydantic рекомендует использовать ConfigDict

PavStepan · 2025-11-23T16:17:41Z

config/settings.py

+    VERSION: str = "v.:0.0.3.0"

    # Токен бота Telegram
    BOT_TOKEN: Optional[str] = Field(None, description="Токен бота от @BotFather")


Токен бота обязательный аргумент, так что он не должен быть None

С python 3.10 рекомендуется использовать str | None вместо Optional

PavStepan · 2025-11-23T16:23:47Z

config/settings.py



-# Создаем глобальный экземпляр настроек
 settings = Settings()


У обязательных полей не должно быть значение None.
У тебя приложение не должно запускаться без правильных настрое

PavStepan · 2025-11-23T16:27:29Z

models/model_card.py

+            for line in lines:
+                line = line.strip()
+                if not line:
+                    continue


Старайся избегать оператора continue
Вместо этого лучше используй filter на итерируемом объекте

for line in filter(lambda r: r, lines):
логика цикла

PavStepan · 2025-11-23T17:26:50Z

handlers/bot_routes/route_view_card.py

+ELLIPSIS_LENGTH = 3
+
+# Простой кэш карточек на время сессии
+_card_cache = {}


Посмотри в строну базы данных

так твои данные будут структурированные.

А кэш используй потом, когда будет необходим быстрый доступ к данных

PavStepan · 2025-11-23T17:27:02Z

handlers/bot_routes/route_view_card.py

+
+view_router = Router()
+
+MAX_ITEM_PREVIEW_LENGTH = 50


вынеси в settings

PavStepan · 2025-11-23T17:27:26Z

handlers/bot_routes/route_list_cards.py

+
+list_router = Router()
+
+MAX_TITLE_LENGTH = 30


вынести в settings

PavStepan · 2025-11-23T17:29:19Z

handlers/bot_routes/route_list_cards.py

+    :type callback_data: CardCallback
+    """
+    try:
+        del callback_data


Зачем ты удаляешь callback_data?
Если она тебе не нужна, то просто не передавай ее в функцию.

чтобы линтер не ругался

PavStepan · 2025-11-23T17:43:24Z

main.py

@@ -1,3 +1,5 @@
+"""Точка входа приложения"""
+


Тут опишу общие рекомендации по структуре проекта

весь исполняемы код, который относится к проекту помести в одну директорию (я предпочитаю называть ее "internal")

раздели внутри все в логические модули. Если это router то в них только точки входа в приложение. Какая то бизнес логика в отдельном пакете, dto тоже в отдельном

Структура которую использую я:

├── internal -- храним весь используемый код │ ├── app │ ├── bot -- все что связно с ботом в проекте │ │ ├── keyboard │ │ ├── message │ │ ├── middleware │ │ └── scenes │ ├── config │ │ └── settings -- настройки api │ ├── controller -- точки входа в приложение (роутеры) │ │ ├── bot -- точки входа для бота │ │ │ └── v1 │ │ └── http -- точки входа для fastapi │ │ └── v1 │ ├── dto -- объекты с данными │ ├── models -- модели базы данных │ ├── service -- бизнес логика приложения │ └── utils -- прочие универсальные инструменты └── main -- точка запуска приложения

Не дублируй название файлов и директории. Ты для этого и создаешь папочную структура, чтобы логически разделить проект

у тебя routers -> routers_webhook
называй routers -> webhooks

intervisionlord and others added 30 commits October 27, 2025 15:22

chore(CHANGELOG): Обновлен конфиг cliff

7d28a61

chore: Обновлен gitignore

d5fcd98

docs(CHANGELOG): Обновлен changelog по новому формату

e11c068

Создан pylint.yml для workflow github

4af2d1a

build(requirements): Понижение версии pydantic для совместимости с CI…

cfa108d

… wirkflow

Merge pull request #20 from intervisionlord/dev

76af99d

build(requirements): Понижение версии pydantic для совместимости с workflow

build(requirements): Понижение pydantic до 2.11.4

3e8a4d5

Merge pull request #19 from intervisionlord/chore-workflow-pylint

028ed92

Создан pylint.yml для workflow github

docs(README): Обновление readme

18d5c55

build(requirements): Добавлен httpx

a8ccca6

Merge pull request #21 from intervisionlord/feature/nc_auth

8cd4c96

feat(NC): Интеграция с nextcloud

chore(maintenance): В обвязку добавлена очистка локальных веток

902dcf4

chore(maintenance): Добавлен pre-commit

453d3dc

docs(Bot): Докстринги и аннотации

9a5afd7

- Докстринги в хендлере - Докстринги в логировании #15

feat(NC): Вынесение отправки запросов в отдельный хендлер

e229481

#12

refactor(Bot): Логика запросов через отдельный хендлер

194c504

- Запросы через handler_requests - Улучшено ответное сообщение со списком - Добавлены докстринги #12 #15

refactor(NC): Сокращен обработчик получения досок (вынесена логика за…

78ae945

…просов) #12

docs(README): Обновлен readme

e952603

docs(CHANGELOG): Обновлен changelog

0416a96

fix(settings): Актуализирована версия

775f07d

docs(README): Обновлен readme

7ce0c89

feat(NC): Добавлено получение стеков из доски

235d418

#12

feat(NC): Добавлена модель для стека

c8f076d

#12

feat(Bot): Добавлен обработчик команды выбора получения стеков по опр…

b493998

…еделенной доске #12

fix(NC): Исправление endpoint для досок и стеков

42968c3

docs(README): Обновлен readme

88703c9

chore(maintenance): Обновлен project-tree

6157912

docs(CHANGELOG): Обновлен changelog

17e410e

intervisionlord added 13 commits October 28, 2025 17:38

docs(linter): Докстринги и исправления ошибок линтера

fd48397

#15

Merge pull request #22 from intervisionlord/feature/deck_show_cards

e3d04c3

Feature/deck show cards

docs(APP): Докстринги для оставшихся файлов

747964b

#15

docs(APP): Исправление докстрингов

720bca0

#15

docs(maintenance): Дописывание докстрингов в служебном скрипте

c034cb4

feat(Bot): Пересмотр логики. Выборка только из указанного стека

7d7982b

#12

refactor(Bot): Глобальный рефакторинг (итерация 1)

f0e07e7

- Оптимизация кода, чистка, упрощение - Убран flake

fix(Bot): Работа с несколькими карточками

b48cd77

- Добавление элементов в текущую открытую карточку - Отслеживание нахождения в конкретной карточке

Merge pull request #23 from intervisionlord/feature/deck_show_cards

e2c1090

Feature/deck show cards

refactor(Bot): Замена if .. else на match .. case

e4d3730

- Оптимизация обработки условий - Изменение строки с версией для sonar gate

style(Bot): Полировка кода

5ecce9b

Merge pull request #24 from intervisionlord/refactor/code_polish

cad5b3f

style(Bot): Полировка кода

intervisionlord self-assigned this Nov 23, 2025

intervisionlord added this to ShoppingBotNC Nov 23, 2025

github-project-automation bot moved this to В очереди in ShoppingBotNC Nov 23, 2025

intervisionlord added Новый функционал Разное labels Nov 23, 2025

PavStepan approved these changes Nov 23, 2025

View reviewed changes



		# Создаем глобальный экземпляр настроек
		settings = Settings()

		@@ -1,3 +1,5 @@
		"""Точка входа приложения"""


		view_router = Router()

		MAX_ITEM_PREVIEW_LENGTH = 50


		list_router = Router()

		MAX_TITLE_LENGTH = 30

Conversation

intervisionlord commented Nov 23, 2025

Uh oh!

sonarqubecloud bot commented Nov 23, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Quality Gate passed

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

sonarqubecloud bot commented Nov 23, 2025 •

edited

Loading