Apply code review

Author: d2dyno1

src/Core/SecureFolderFS.Core.FileSystem/Helpers/Health/HealthHelpers.Sidecar.cs

@@ -28,7 +28,7 @@ public static async Task DetectOrphanSidecarsAsync(IFolder folder, IProgress<IRe
             var shortenedNames = new HashSet<string>(StringComparer.OrdinalIgnoreCase);
             var sidecars = new List<IStorableChild>();
 
-            await foreach (var item in folder.GetItemsAsync(StorableType.File, cancellationToken))
+            await foreach (var item in folder.GetItemsAsync(StorableType.All, cancellationToken))
             {
                 if (AbstractPathHelpers.IsSidecarName(item.Name))
                     sidecars.Add(item);

src/Core/SecureFolderFS.Core.FileSystem/Validators/BaseFileSystemValidator.cs

@@ -60,6 +60,11 @@ protected async Task<IResult> ValidateNameResultAsync(IStorableChild storable, C
             if (!string.IsNullOrEmpty(decryptedName))
                 return decryptedName;
 
+            // A shortened file (.sffsn) that couldn't be decrypted means its sidecar is missing.
+            // Report this as an invalid name so the health system can offer to generate a new one.
+            if (storable.Name.EndsWith(Constants.Names.SHORTENED_FILE_EXTENSION, StringComparison.OrdinalIgnoreCase))
+                return null;
+
             // We want to suppress failures that might be raised when the Directory ID file is not found.
             // This case should be already handled in the folder validator