// Top matches · 7 engineers

Edaly Ahmed
@tomorro · Paris, France
96% MATCH
944 MERGED PRSDEVELOPER COMMUNICATION STYLE

WHY THEM  Writes about developer communication style in their own reviews.

“The orphaned document/S3 cleanup issue the bugbot flagged is real. Right now if `getPlaybookService.findMany` or the AI fetch fails after we've already done `documentRepository.save` + `sthree.uploadObject`, those resources just sit there. It would be nice if we either: (a) moved the document creation to after the AI call succeeds, or (b) added cleanup in a catch block. Option (a) is simpler — the document only needs to exist if selection succeeds, so we could defer the save until after `playbookId` is confirmed.”
Go deeper
95% MATCH
895 MERGED PRSCOMMUNICATION STYLE AND TONE

WHY THEM  Writes about communication style and tone in their own reviews.

“I have to say I still don't have a clear understanding of the nested behavior. There is an example below but it doesn't actually show what's going to happen. The way I'm reading it now, I assume that `ChildComponent` will be days-stale in the context of `ParentComponent` (due to the outer boundary, rule #1) but would be hours-stale if it was rendered on its own. Is that the expectation? This should ideally be spelled out just below the example — the code comment seems to contradict that.”
Go deeper
Load 3 more perspectives

People also asked