Unused methods in SessionService: getSessionSummaries, getWorkingDirectories #159

New issue

Closed

opened 2026-01-24 10:09:53 +00:00 by jack · 1 comment

jack commented

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

Owner

Beschreibung

In packages/backend/src/services/session-service.ts werden folgende Methoden nicht verwendet:

1. `getSessionSummaries` (Zeile 271-273)

async getSessionSummaries(memorySessionId: string): Promise<SessionSummaryRecord[]> {
  return this.summaries.getBySessionId(memorySessionId);
}

2. `getWorkingDirectories` (Zeile 281-320)

async getWorkingDirectories(contentSessionId: string): Promise<string[]> {
  const session = await this.sessions.findByContentSessionId(contentSessionId);
  if (!session?.memory_session_id || !session.working_directory) {
    return [];
  }

  // Get all observations for this session
  const observations = await this.observations.getBySessionId(session.memory_session_id);

  // Extract unique cwd values
  const cwdSet = new Set<string>();
  for (const obs of observations) {
    if (obs.cwd) {
      cwdSet.add(obs.cwd);
    }
  }
  // ... filters for subdirectories with existing CLAUDE.md files
}

Analyse

getSessionSummaries: Einfacher Wrapper um Repository-Methode
getWorkingDirectories: Komplexere Logik für Subdirectory-Erkennung mit CLAUDE.md-Prüfung

Zu prüfen

Sind diese Methoden für API-Endpunkte vorgesehen?
- Möglicherweise geplant aber nicht angebunden
Wird getWorkingDirectories für CLAUDE.md-Generierung benötigt?
- Die Logik passt zum Subdirectory-Feature
Gibt es externe Consumers?
- Andere Services, Routes, oder UI-Komponenten

Betroffene Datei

packages/backend/src/services/session-service.ts

⚠️ Hinweis

Nicht einfach löschen! Zuerst prüfen:

Ob diese Methoden für geplante Features benötigt werden
Ob die Implementierung an anderer Stelle fehlt (z.B. Route-Handler)
Insbesondere getWorkingDirectories enthält nicht-triviale Logik für Subdirectory-Handling

## Beschreibung In `packages/backend/src/services/session-service.ts` werden folgende Methoden nicht verwendet: ### 1. `getSessionSummaries` (Zeile 271-273) ```typescript async getSessionSummaries(memorySessionId: string): Promise<SessionSummaryRecord[]> { return this.summaries.getBySessionId(memorySessionId); } ``` ### 2. `getWorkingDirectories` (Zeile 281-320) ```typescript async getWorkingDirectories(contentSessionId: string): Promise<string[]> { const session = await this.sessions.findByContentSessionId(contentSessionId); if (!session?.memory_session_id || !session.working_directory) { return []; } // Get all observations for this session const observations = await this.observations.getBySessionId(session.memory_session_id); // Extract unique cwd values const cwdSet = new Set<string>(); for (const obs of observations) { if (obs.cwd) { cwdSet.add(obs.cwd); } } // ... filters for subdirectories with existing CLAUDE.md files } ``` ## Analyse - `getSessionSummaries`: Einfacher Wrapper um Repository-Methode - `getWorkingDirectories`: Komplexere Logik für Subdirectory-Erkennung mit CLAUDE.md-Prüfung ## Zu prüfen 1. **Sind diese Methoden für API-Endpunkte vorgesehen?** - Möglicherweise geplant aber nicht angebunden 2. **Wird `getWorkingDirectories` für CLAUDE.md-Generierung benötigt?** - Die Logik passt zum Subdirectory-Feature 3. **Gibt es externe Consumers?** - Andere Services, Routes, oder UI-Komponenten ## Betroffene Datei - `packages/backend/src/services/session-service.ts` ## ⚠️ Hinweis Nicht einfach löschen! Zuerst prüfen: - Ob diese Methoden für geplante Features benötigt werden - Ob die Implementierung an anderer Stelle fehlt (z.B. Route-Handler) - Insbesondere `getWorkingDirectories` enthält nicht-triviale Logik für Subdirectory-Handling

jack added the

labels

2026-01-24 10:09:56 +00:00

jack commented

2026-01-24 10:11:08 +00:00

Author

Owner

Korrektur: getSessionSummaries wird tatsächlich verwendet:

Route-Handler in data.ts:59
API-Client in ui/src/api/client.ts:217
Sessions-View in ui/src/views/Sessions.tsx:211,379

Nur getWorkingDirectories ist tatsächlich ungenutzt und kann entfernt werden.

**Korrektur:** `getSessionSummaries` wird tatsächlich verwendet: - Route-Handler in `data.ts:59` - API-Client in `ui/src/api/client.ts:217` - Sessions-View in `ui/src/views/Sessions.tsx:211,379` Nur `getWorkingDirectories` ist tatsächlich ungenutzt und kann entfernt werden.