PhyberApex/kuroshiro

refactor(api): ScreensService.add complexity and duplicated screen lookup

Aperta

#852 aperta il 22 ago 2026

Vedi su GitHub
 (0 commenti) (0 reazioni) (0 assegnatari)TypeScript (0 fork)auto 404
fallowgood first issueready-for-agent

Metriche repository

Star
 (10 stelle)
Metriche merge PR
 (Merge medio 3g 2h) (92 PR mergiate in 30 g)

Descrizione

Problem Statement

packages/api/src/screens/screens.service.ts:

  • add (line 34): cyclomatic 16, cognitive 18, 71 lines — high (fallow health)
  • Clone group: delete (116-121) and updateExternalScreen (180-185) both do "find screen with device relation, warn + NotFoundException('Screen not found')".

Solution

  • Extract private async findScreenWithDevice(id): Promise<Screen> that throws the NotFoundException, and use it from delete, updateExternalScreen and any other method doing the same lookup.
  • Split add into the branches it handles (upload vs external link vs plugin/mashup source) as private methods so the public method is a dispatcher.
  • screens/__test__ specs stay green.

Verification

  • pnpm lint && pnpm type-check && pnpm test pass
  • Run pnpm fallow:baseline in the same PR so the finding disappears from .fallow-baselines/, then confirm pnpm fallow:ci passes. Never add new findings to a baseline; see docs/agents/fallow.md.

Found by the initial fallow scan (pnpm fallow) on 2026-08-22 and baselined so CI stays green until this is fixed.

Guida contributor

Direzione di ricerca
Esplora la classe ScreensService in packages/api/src/screens/screens.service.ts. Comprendi come funzionano add, delete e updateExternalScreen e nota la ricerca duplicata dello schermo. Esamina i test esistenti in screens/ test per vedere il comportamento atteso. Quindi pianifica come estrarre un metodo privato findScreenWithDevice e suddividere add in metodi più piccoli.
Tech stack
nodejstypescript
Dominio
backend
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
GitTypeScript
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
80