Telegram-Market/telegram-shop
1
1
Fork 0
Code Issues 40 Pull Requests Actions Packages Projects 1 Releases Wiki Activity

Сохранять userStates в SQLite вместо in-memory Map #59

New Issue
Closed
opened 2026-06-17 19:39:40 +00:00 by NW · 1 comment
NW commented 2026-06-17 19:39:40 +00:00
Owner
Copy Link

Проблема

src/context/userStates.js использует new Map() — все состояния теряются при рестарте. Нет TTL, нет лимита размера → утечка памяти.

Решение

  • Создать таблицу user_states в SQLite (user_id, state_key, state_data, updated_at)
  • Создать src/services/stateService.js с методами get/set/delete
  • Добавить TTL — удалять состояния старше 24ч
  • Заменить Map на stateService в index.js
  • Добавить cleanup-задачу (раз в час)

Приоритет: СРЕДНИЙ

## Проблема `src/context/userStates.js` использует `new Map()` — все состояния теряются при рестарте. Нет TTL, нет лимита размера → утечка памяти. ## Решение - [ ] Создать таблицу `user_states` в SQLite (user_id, state_key, state_data, updated_at) - [ ] Создать `src/services/stateService.js` с методами get/set/delete - [ ] Добавить TTL — удалять состояния старше 24ч - [ ] Заменить Map на stateService в index.js - [ ] Добавить cleanup-задачу (раз в час) ## Приоритет: СРЕДНИЙ
NW added this to the Phase 2: Architecture & Refactoring milestone 2026-06-17 19:39:40 +00:00
NW added the priority::mediumtype::featurephase::2 labels 2026-06-17 19:39:40 +00:00
NW commented 2026-06-22 09:03:13 +00:00
Author
Owner
Copy Link

✅ Реализовано

  • Создать таблицу user_states в SQLite (chat_id PK, state_data JSON, updated_at)
  • Создать src/services/stateService.js с методами get/set/delete/has
  • TTL 24 часа — expired states auto-deleted
  • Cleanup job раз в час (setInterval)
  • Заменить Map на async stateService proxy в src/context/userStates.js
  • Добавить await к 45 вызовам userStates.get/set/delete/has в 13 файлах
  • await initStates() в index.js при старте
  • Миграция 004_user_states

Коммит: a04e60d

## ✅ Реализовано - [x] Создать таблицу `user_states` в SQLite (chat_id PK, state_data JSON, updated_at) - [x] Создать `src/services/stateService.js` с методами get/set/delete/has - [x] TTL 24 часа — expired states auto-deleted - [x] Cleanup job раз в час (setInterval) - [x] Заменить Map на async stateService proxy в `src/context/userStates.js` - [x] Добавить `await` к 45 вызовам userStates.get/set/delete/has в 13 файлах - [x] `await initStates()` в index.js при старте - [x] Миграция 004_user_states **Коммит**: a04e60d
NW closed this issue 2026-06-22 09:03:13 +00:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
phase::1
Phase 1: Security
 phase::1
Phase 1
 phase::1
Phase 1
 phase::2
Phase 2: Architecture
 phase::2
Phase 2
 phase::3
Phase 3: Admin API
 phase::3
Phase 3
 phase::4
Phase 4: Web UI
 phase::4
Phase 4
 phase::5
Phase 5: Quality
 phase::5
Phase 5
 priority::critical
Critical priority
 priority::critical
Critical priority
 priority::high
High priority
 priority::high
High priority
 priority::medium
Medium priority
 priority::medium
Medium priority
 type::bug
Bug fix
 type::bug
Bug
 type::feature
New feature
 type::feature
Feature
 type::infra
Infrastructure
 type::infra
Infrastructure
 type::refactor
Code refactoring
 type::refactor
Refactoring
 type::security
Security issue
 type::security
Security
 type::testing
Testing
 type::testing
Testing
No Label phase::2 priority::medium type::feature
Milestone
No items
No Milestone Phase 2: Architecture & Refactoring
Projects
Clear projects
No project
Assignees
Clear assignees
NW
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Telegram-Market/telegram-shop#59
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?