Ungenutzte Funktionen in settings.ts prüfen und aufräumen #128

New issue

Closed

opened 2026-01-24 09:39:13 +00:00 by jack · 1 comment

jack commented

2026-01-24 09:39:13 +00:00

Owner

Problem

In packages/shared/src/settings.ts sind mehrere Funktionen definiert und exportiert, die nirgendwo im Code verwendet werden.

Ungenutzte Funktionen

Funktion	Zeile	Verwendet
`getDefault`	?	❌ Nein
`reload` (SettingsManager)	?	❌ Nein
`resetSettings`	437	❌ Nein
`createSettingsManager`	444	❌ Nein
`ensureWorkerAuthToken`	486	❌ Nein

Analyse

getDefault - Möglicherweise für UI Settings-Editor gedacht?
reload - Teil des SettingsManager, aber nirgends aufgerufen
resetSettings - Nützlich für "Reset to Defaults" Feature, aber nicht implementiert
createSettingsManager - Factory-Funktion, aber nirgends verwendet
ensureWorkerAuthToken - Sollte vermutlich beim Worker-Start verwendet werden

Optionen

Option A: Entfernen

Ungenutzte Funktionen entfernen um den Code sauber zu halten.

Option B: Verwenden

Falls die Funktionen sinnvoll sind:

resetSettings → In UI Settings-Page für "Reset" Button
ensureWorkerAuthToken → Beim Worker-Spawn für sichere Kommunikation
createSettingsManager → Für custom Settings-Pfade (Tests, Multi-Instance)

Betroffene Datei

packages/shared/src/settings.ts

## Problem In `packages/shared/src/settings.ts` sind mehrere Funktionen definiert und exportiert, die nirgendwo im Code verwendet werden. ### Ungenutzte Funktionen | Funktion | Zeile | Verwendet | |----------|-------|-----------| | `getDefault` | ? | ❌ Nein | | `reload` (SettingsManager) | ? | ❌ Nein | | `resetSettings` | 437 | ❌ Nein | | `createSettingsManager` | 444 | ❌ Nein | | `ensureWorkerAuthToken` | 486 | ❌ Nein | ## Analyse - `getDefault` - Möglicherweise für UI Settings-Editor gedacht? - `reload` - Teil des SettingsManager, aber nirgends aufgerufen - `resetSettings` - Nützlich für "Reset to Defaults" Feature, aber nicht implementiert - `createSettingsManager` - Factory-Funktion, aber nirgends verwendet - `ensureWorkerAuthToken` - Sollte vermutlich beim Worker-Start verwendet werden ## Optionen ### Option A: Entfernen Ungenutzte Funktionen entfernen um den Code sauber zu halten. ### Option B: Verwenden Falls die Funktionen sinnvoll sind: - `resetSettings` → In UI Settings-Page für "Reset" Button - `ensureWorkerAuthToken` → Beim Worker-Spawn für sichere Kommunikation - `createSettingsManager` → Für custom Settings-Pfade (Tests, Multi-Instance) ## Betroffene Datei `packages/shared/src/settings.ts`

jack added the

labels

2026-01-24 09:39:15 +00:00

jack commented

2026-01-24 09:42:50 +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 Funktionen 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 Funktionen eigentlich hingehören.