Verifiche per convenzioni di nomenclatura, design delle funzioni, principio DRY e SSOT.

[ ] Nomi unit seguono WvdS.<Categoria>.<Componente>.pas
[ ] Rispecchia la struttura Microsoft .NET con prefisso WvdS.
[ ] Esempio: WvdS.UI.Controls.Button, WvdS.System.Logging

[ ] Base astratta usa prefisso TWvdSCustom*
[ ] Base pubblica usa prefisso TWvdSBase*
[ ] Classi concrete usano prefisso TWvdS*
[ ] Pattern: TWvdSCustomXxx -> TWvdSBaseXxx -> TUserXxx

[ ] Nomi descrittivi (nessuna singola lettera tranne cicli)
[ ] Nessuna abbreviazione tranne quelle conosciute (URL, HTTP, JSON)
[ ] Interfacce hanno GUID per container DI
[ ] Campi privati: prefisso F* (FValue, FCount)
[ ] Parametri: prefisso A* (AIndex, APath)

[ ] 12-24 righe ideale
[ ] Massimo 40 righe (oltre dividere)
[ ] Profondità massima annidamento: 3

[ ] Massimo 5-7 parametri
[ ] Con più parametri: usare Record/Classe
[ ] Parametri opzionali alla fine

[ ] Ogni funzione ha UN compito
[ ] Il nome funzione descrive completamente il compito
[ ] Se serve "e" nel nome -> dividere

[ ] Get* - Recuperare valore
[ ] Set* - Impostare valore
[ ] Calculate* - Eseguire calcolo
[ ] Process* - Eseguire elaborazione
[ ] Validate* - Eseguire validazione
[ ] Create* - Creare oggetto
[ ] Parse* - Analizzare input
[ ] Format* - Formattare output

[ ] Nessun codice duplicato (più di 3 righe identiche -> estrarre)
[ ] Nessun pattern copia-incolla
[ ] Logica comune in funzioni helper
[ ] Costanti invece di letterali ripetuti

(* VIETATO - Codice duplicato *)
procedure ProcessUserA(const AUser: TUser);
begin
  if not ValidateEmail(AUser.Email) then
    raise EValidationError.Create('Invalid email');
  if Length(AUser.Name) > 64 then
    raise EValidationError.Create('Name too long');
  (* ... ulteriore elaborazione *)
end;
 
procedure ProcessUserB(const AUser: TUser);
begin
  if not ValidateEmail(AUser.Email) then
    raise EValidationError.Create('Invalid email');
  if Length(AUser.Name) > 64 then
    raise EValidationError.Create('Name too long');
  (* ... altra elaborazione *)
end;
 
(* CORRETTO - Logica comune estratta *)
procedure ValidateUser(const AUser: TUser);
begin
  if not ValidateEmail(AUser.Email) then
    raise EValidationError.Create(rsInvalidEmail);
  if Length(AUser.Name) > MAX_NAME_LENGTH then
    raise EValidationError.Create(rsNameTooLong);
end;
 
procedure ProcessUserA(const AUser: TUser);
begin
  ValidateUser(AUser);
  (* ... ulteriore elaborazione *)
end;

[ ] API NodeJS via ~/sources/common/web/nodejs/
    - NodeJS.FS, NodeJS.Path, NodeJS.ChildProcess
[ ] API VSCode via ~/sources/common/web/vscode/
    - VSCode.Window, VSCode.Commands, VSCode.Workspace
[ ] WvdS Core via ~/sources/common/core/
    - WvdS.System.Logging, WvdS.VSCode.Security

[ ] Nessuna chiamata require() diretta nelle Extension
[ ] Nessun duplicato di unit Common nelle Extension
[ ] Nessuna implementazione wrapper locale

(* VIETATO - require() locale *)
function FileExists(const APath: string): Boolean;
begin
  asm
    var fs = require('fs');  (* Violazione SSOT! *)
    Result = fs.existsSync(APath);
  end;
end;
 
(* CORRETTO - Libreria Common *)
uses
  NodeJS.FS;
 
function FileExists(const APath: string): Boolean;
begin
  Result := ExistsSync(APath);  (* Da NodeJS.FS *)
end;

[ ] Logica UI solo in strato UI (extension_main, *Dialog)
[ ] Logica business solo in strato Service (*Service)
[ ] Strutture dati solo in strato Model (*Models)
[ ] Services non chiamano API VSCode direttamente
[ ] Models non contengono logica

+-------------------------------------------------+
|  UI Layer (extension_main.pas, *Dialog.pas)     |
|  -> API VSCode, Interazione Utente              |
+-------------------------------------------------+
|  Service Layer (*Service.pas)                   |
|  -> Logica Business, Validazione                |
+-------------------------------------------------+
|  Model Layer (*Models.pas)                      |
|  -> Record, Tipi, Enum                          |
+-------------------------------------------------+
|  Infrastructure (NodeJS.*, VSCode.*, WvdS.*)    |
|  -> API Esterne, Astrazione Piattaforma         |
+-------------------------------------------------+

Checklist Review (Qualità Codice):

Naming:
- [ ] Nomi unit: WvdS.<Categoria>.<Componente>.pas
- [ ] Classi: prefisso TWvdS*
- [ ] Interfacce: prefisso IWvdS*
- [ ] Nomi descrittivi (nessuna abbreviazione)

Funzioni:
- [ ] 12-24 righe ideale, max 40
- [ ] Max 5-7 parametri
- [ ] Single Responsibility
- [ ] Naming Verb-First

DRY:
- [ ] Nessun codice duplicato
- [ ] Logica comune estratta
- [ ] Costanti invece di letterali

SSOT:
- [ ] Librerie Common usate
- [ ] Nessuna chiamata require() locale
- [ ] Nessun duplicato di unit Common

SoC:
- [ ] UI solo in strato UI
- [ ] Logica solo in strato Service
- [ ] Services senza chiamate VSCode dirette

• Panoramica Garanzia Qualità
• Convenzioni di Codice
• Checklist Core
• Sviluppo Extension