ADORSYS-GIS / ADORSYS-GIS/lightbridge-governance

[Ticket]: split FileLock out of cache.rs — 719 lines after three baseline raises

Aperta
#235 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub
ticket
Lingua principale
Rust
Stelle
1
Fork
2
Merge medio
14h 36m
PR unite (30g)
107

Descrizione

## Summary

`app/governance-auth/src/cache.rs` is now **719 lines** against a 200-line ceiling, having been baselined three times in a single branch (#228): 602 → 665 → 719. It is the largest file in the crate and wants splitting before the next feature touches it.

## Why it grew

Legitimately, each time — the lock ceiling for the hung-collector fix and the empty-lock grace both belong next to `FileLock`, and a second stale-lock implementation elsewhere would have been strictly worse. An independent review agreed the placement was right and that the raises were consistent with the gate's own remediation text and eight merged precedents.

That does not make three raises in one branch a good steady state.

## Suggested split

Extract `FileLock` (and its stale-lock reasoning, PID liveness, the empty-lock grace and the acquisition ceiling) into its own module. That is the cohesive unit: the session cache and the lock are separate concerns that happen to share a file.

## Related, separate

An observation from the same review, worth its own decision rather than folding in here: the LoC gate's stated rule is that grandfathered files "may be touched but **must not grow**", yet **every** commit that has touched one since the gate landed has raised its baseline — 8 of them, including one whose entire purpose was the raise and one that raised `otel.rs` by 166 lines. The clause has never bound anyone. Either it means something or it should be reworded.

## AI Usage Declaration

AI-assisted. Line counts and the precedent history are measured from the repository, not estimated. A human owns intent, verification and consequences.

Guida per i contributori

Apri la guida per i contributori

Direzione di ricerca

Start by reading app/governance-auth/src/cache.rs and mapping FileLock's references within the crate. Keep the stale-lock reasoning, PID liveness, empty-lock grace, and acquisition ceiling together during the extraction. Done means FileLock has its own module, cache responsibilities remain separate, and behavior is unchanged.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Stack tecnologico
rust
Ambito
backend
Tipo di issue
Refactoring
Difficoltà
3/5
Tempo stimato
1-2 giorni
Stato di attività
Attiva
Chiarezza
Specificata chiaramente
Idoneità per principianti
72/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.