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

Разбить database.js на миграции + подключение #57

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

Проблема

src/config/database.js — 284 строки. Смешивает подключение к БД и миграции схемы.

Решение

  • src/config/database.js (~30 строк) — подключение, экспорт db
  • src/migrations/001_initial_schema.js — CREATE TABLE
  • src/migrations/002_add_columns.js — ALTER TABLE
  • src/migrations/003_add_indexes.js — CREATE INDEX
  • src/migrations/runner.js — запуск миграций по порядку
  • Whitelist имён таблиц для PRAGMA
  • Хранить версию схемы в meta-таблице

Приоритет: ВЫСОКИЙ

## Проблема `src/config/database.js` — 284 строки. Смешивает подключение к БД и миграции схемы. ## Решение - [ ] `src/config/database.js` (~30 строк) — подключение, экспорт db - [ ] `src/migrations/001_initial_schema.js` — CREATE TABLE - [ ] `src/migrations/002_add_columns.js` — ALTER TABLE - [ ] `src/migrations/003_add_indexes.js` — CREATE INDEX - [ ] `src/migrations/runner.js` — запуск миграций по порядку - [ ] Whitelist имён таблиц для PRAGMA - [ ] Хранить версию схемы в meta-таблице ## Приоритет: ВЫСОКИЙ
NW added this to the Phase 2: Architecture & Refactoring milestone 2026-06-17 19:39:37 +00:00
NW added the priority::hightype::refactorphase::2 labels 2026-06-17 19:39:37 +00:00
NW commented 2026-06-17 21:28:28 +00:00
Author
Owner
Copy Link

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

  • src/config/database.js (42 строки) — подключение, экспорт db, PRAGMA, SIGINT
  • src/migrations/001_initial_schema.js (92 строки) — 7 CREATE TABLE в транзакции
  • src/migrations/002_add_columns.js (33 строки) — 5 ALTER TABLE с проверкой колонок
  • src/migrations/003_add_indexes.js (9 строк) — 6 CREATE INDEX
  • src/migrations/runner.js (56 строк) — запуск миграций по порядку, _meta таблица для версионности
  • ALLOWED_TABLES whitelist сохранён в runner.js
  • Schema version отслеживается в _meta таблице (idempotent re-runs)

Коммит: 4b8144a

## ✅ Реализовано - [x] `src/config/database.js` (42 строки) — подключение, экспорт db, PRAGMA, SIGINT - [x] `src/migrations/001_initial_schema.js` (92 строки) — 7 CREATE TABLE в транзакции - [x] `src/migrations/002_add_columns.js` (33 строки) — 5 ALTER TABLE с проверкой колонок - [x] `src/migrations/003_add_indexes.js` (9 строк) — 6 CREATE INDEX - [x] `src/migrations/runner.js` (56 строк) — запуск миграций по порядку, _meta таблица для версионности - [x] ALLOWED_TABLES whitelist сохранён в runner.js - [x] Schema version отслеживается в _meta таблице (idempotent re-runs) **Коммит**: 4b8144a
NW closed this issue 2026-06-17 21:28:32 +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::high type::refactor
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#57
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?