Skip to content

build: remove unused redis dependency from apps/api - #98

Closed
agentHits wants to merge 1 commit into
AgentHits-Devfrom
build/remove-redis-api
Closed

agentHits wants to merge 1 commit into
AgentHits-Devfrom
build/remove-redis-api

Conversation

@agentHits

Copy link
Copy Markdown
Collaborator

Зачем

Пакет redis (4.7.1) был объявлен в зависимостях apps/api, но нигде не используется: в apps/ и packages/ нет ни одного import/require из redis или @redis/*, в конфигурации сборки apps/api и в Dockerfile.server он тоже не упоминается. Слово «redis» в коде встречается только как тип сервиса базы данных и в комментарии. Поэтому вместо обновления 4 → 6 пакет просто удалён.

apps/api — облачный сервис upstream (собирается из Dockerfile.server), в образ панели форка (./Dockerfile) он не входит. Работа панели не меняется.

Что изменилось

  • apps/api/package.json: удалена строка "redis": "4.7.1".
  • bun.lock: удалены redis@4.7.1, @redis/bloom, @redis/client, @redis/graph, @redis/json, @redis/search, @redis/time-series, generic-pool и вложенные копии @redis/client/cluster-key-slot, @redis/client/yallist (−21 строка).
  • ioredis, bullmq, cluster-key-slot, redis-errors, redis-parser остаются: их использует apps/schedules.

Проверки

  • bun install --frozen-lockfile после изменения — no changes.
  • bun run typecheck — @dokploy/server, @dokploy/schedules, @dokploy/api, dokploy: код выхода 0.
  • bun run server:build, затем bun run --cwd apps/api build (rimraf dist && tsc) — без ошибок. После bun run server:script packages/server/package.json без изменений; сгенерированные dist удалены.
  • bunx biome check apps/api/package.json — без замечаний.
  • bun audit --audit-level=moderate — No vulnerabilities found (1516 пакетов).
  • Сборку apps/dokploy (next build) проверит CI этого PR.

Что проверить вручную на dev

Ничего: изменение не затрагивает образ панели.

🤖 Generated with Claude Code

Что:
- apps/api/package.json: удалена зависимость redis 4.7.1.
- bun.lock: удалены redis@4.7.1, @redis/bloom, @redis/client, @redis/graph, @redis/json, @redis/search, @redis/time-series, generic-pool и вложенные копии @redis/client/cluster-key-slot и @redis/client/yallist. Пакеты ioredis, bullmq, cluster-key-slot, redis-errors и redis-parser остаются: их использует apps/schedules.
Зачем:
- Пакет нигде не импортируется: в apps/ и packages/ нет ни одного import или require из redis или @redis/*; в конфигурации сборки apps/api и Dockerfile.server его тоже нет. Удалить проще, чем обновлять 4 → 6.
- apps/api — облачный сервис upstream, в образ панели форка (./Dockerfile) он не входит.
Риски:
- Минимальные: код, который мог бы загрузить пакет, отсутствует. Если upstream позже добавит импорт redis в apps/api, это проявится при слиянии с canary как ошибка typecheck.
Проверки:
- bun install --frozen-lockfile после изменения - без изменений.
- bun run typecheck - все 4 workspace без ошибок.
- bun run server:build, затем сборка apps/api (tsc) - без ошибок; после bun run server:script packages/server/package.json без изменений, сборочные dist удалены.
- bunx biome check apps/api/package.json - без замечаний.
- bun audit --audit-level=moderate - No vulnerabilities found (1516 пакетов).

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@agentHits

Copy link
Copy Markdown
Collaborator Author

Влито в составе #101 одним PR, чтобы вся первая волна вышла одной версией. Этот PR закрываю без слияния, его коммит есть в #101.

@agentHits agentHits closed this Oct 27, 2026
@agentHits
agentHits deleted the build/remove-redis-api branch October 27, 2026 16:12
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