Ungenutzte Konstanten in constants.ts prüfen und aufräumen #125

New issue

Closed

opened 2026-01-24 09:33:02 +00:00 by jack · 1 comment

jack commented

2026-01-24 09:33:02 +00:00

Owner

Problem

In packages/shared/src/constants.ts sind mehrere Konstanten definiert, die nirgendwo im Code verwendet werden:

Konstante	Verwendet	Aktion
`HOOK_EXIT_CODES`	❌ Nein	Prüfen ob nötig, sonst entfernen
`PORTS`	❌ Nein	Prüfen ob nötig, sonst entfernen
`API_ENDPOINTS`	❌ Nein	Prüfen ob nötig, sonst entfernen
`TASK_QUEUE`	❌ Nein	Prüfen ob nötig, sonst entfernen

Optionen

Option A: Entfernen

Wenn diese Konstanten nicht benötigt werden, sollten sie entfernt werden um den Code sauber zu halten.

Option B: Verwenden

Falls die Konstanten sinnvoll sind, sollten sie an den entsprechenden Stellen verwendet werden:

HOOK_EXIT_CODES → in Hook-Handlern für konsistente Exit-Codes
PORTS → in Backend/Worker für Port-Konfiguration (statt hardcoded oder Settings)
API_ENDPOINTS → in API-Clients für konsistente Endpoint-Pfade
TASK_QUEUE → in TaskService/TaskDispatcher für Queue-Konfiguration

Betroffene Datei

packages/shared/src/constants.ts

## Problem In `packages/shared/src/constants.ts` sind mehrere Konstanten definiert, die nirgendwo im Code verwendet werden: | Konstante | Verwendet | Aktion | |-----------|-----------|--------| | `HOOK_EXIT_CODES` | ❌ Nein | Prüfen ob nötig, sonst entfernen | | `PORTS` | ❌ Nein | Prüfen ob nötig, sonst entfernen | | `API_ENDPOINTS` | ❌ Nein | Prüfen ob nötig, sonst entfernen | | `TASK_QUEUE` | ❌ Nein | Prüfen ob nötig, sonst entfernen | ## Optionen ### Option A: Entfernen Wenn diese Konstanten nicht benötigt werden, sollten sie entfernt werden um den Code sauber zu halten. ### Option B: Verwenden Falls die Konstanten sinnvoll sind, sollten sie an den entsprechenden Stellen verwendet werden: - `HOOK_EXIT_CODES` → in Hook-Handlern für konsistente Exit-Codes - `PORTS` → in Backend/Worker für Port-Konfiguration (statt hardcoded oder Settings) - `API_ENDPOINTS` → in API-Clients für konsistente Endpoint-Pfade - `TASK_QUEUE` → in TaskService/TaskDispatcher für Queue-Konfiguration ## Betroffene Datei `packages/shared/src/constants.ts`

jack added the

labels

2026-01-24 09:33:04 +00:00

jonas.hanisch referenced this issue from a commit

2026-01-24 09:40:53 +00:00

refactor(shared): remove unused constants from constants.ts

jonas.hanisch closed this issue

2026-01-24 09:40:53 +00:00

jack referenced this issue from a pull request that will close it,

2026-01-24 09:41:02 +00:00

refactor(shared): remove unused constants from constants.ts #130

jack added the

has-pr

label

2026-01-24 09:41:08 +00:00

jack commented

2026-01-24 09:42:47 +00:00

Author

Owner

⚠️ Wichtig: Nicht einfach löschen! Vor dem Entfernen sollte die komplette Implementierung geprüft werden. Möglicherweise fehlt nur die Verwendung an einer Stelle, wo diese Konstanten eigentlich hingehören.

⚠️ **Wichtig:** Nicht einfach löschen! Vor dem Entfernen sollte die komplette Implementierung geprüft werden. Möglicherweise fehlt nur die Verwendung an einer Stelle, wo diese Konstanten eigentlich hingehören.

jonas.hanisch referenced this issue from a commit

2026-01-24 10:39:47 +00:00

refactor(shared): remove unused constants from constants.ts

jonas.hanisch referenced this issue from a commit

2026-01-24 12:04:56 +00:00

refactor(shared): remove unused constants from constants.ts