refactor(shared): remove unused functions from settings.ts #186

Merged

jack merged 1 commit from refactor/135-cleanup-settings into main

2026-01-24 16:52:05 +00:00

jack commented

2026-01-24 10:36:40 +00:00

Owner

Summary

Remove 6 unused functions/methods from settings.ts
Reduced file by 54 lines

Removed

getDefault() - method, never called
reload() - method, never called
resetSettings() - testing helper, never used
createSettingsManager() - factory function, never used
generateAuthToken() - only used by ensureWorkerAuthToken
ensureWorkerAuthToken() - never called

Closes #135

Test plan

TypeScript check passes
Verified via grep that removed functions are not called

## Summary - Remove 6 unused functions/methods from `settings.ts` - Reduced file by 54 lines ### Removed - `getDefault()` - method, never called - `reload()` - method, never called - `resetSettings()` - testing helper, never used - `createSettingsManager()` - factory function, never used - `generateAuthToken()` - only used by ensureWorkerAuthToken - `ensureWorkerAuthToken()` - never called Closes #135 ## Test plan - [x] TypeScript check passes - [x] Verified via grep that removed functions are not called

jack added 1 commit

2026-01-24 10:36:40 +00:00

refactor(shared): remove unused functions from settings.ts

CI / check (pull_request) Successful in 0s

Details

CI / build (pull_request) Successful in 42s

Details

CI / validate-plugin (pull_request) Successful in 35s

Details

1f219f817b

Remove 6 unused functions/methods:
- getDefault() method - never called
- reload() method - never called
- resetSettings() - testing helper, never used
- createSettingsManager() - factory function, never used
- generateAuthToken() - only used by ensureWorkerAuthToken
- ensureWorkerAuthToken() - never called

Closes #135

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