fix(safari): address review comments on #581

Author: moonD4rk

browser/safari/profiles.go

@@ -116,6 +116,9 @@ func readNamedProfilesFromDB(container string) ([]profileContext, error) {
 		}
 		out = append(out, newNamedProfile(externalUUID.String, title.String))
 	}
+	if err := rows.Err(); err != nil {
+		return nil, fmt.Errorf("iterate SafariTabs.db rows: %w", err)
+	}
 	return out, nil
 }
 

browser/safari/safari_test.go

@@ -130,15 +130,16 @@ func TestResolveSourcePaths(t *testing.T) {
 	dir := t.TempDir()
 	mkFile(t, dir, "History.db")
 
-	sources := buildSources(profileContext{legacyHome: dir})
+	sources := buildSources(profileContext{legacyHome: dir, container: deriveContainerRoot(dir)})
 	resolved := resolveSourcePaths(sources)
 	assert.Contains(t, resolved, types.History)
 	assert.Equal(t, filepath.Join(dir, "History.db"), resolved[types.History].absPath)
 	assert.False(t, resolved[types.History].isDir)
 }
 
 func TestResolveSourcePaths_Empty(t *testing.T) {
-	sources := buildSources(profileContext{legacyHome: t.TempDir()})
+	dir := t.TempDir()
+	sources := buildSources(profileContext{legacyHome: dir, container: deriveContainerRoot(dir)})
 	assert.Empty(t, resolveSourcePaths(sources))
 }
 

browser/safari/source.go

@@ -41,7 +41,9 @@ func defaultSources(p profileContext) map[types.Category][]sourcePath {
 	}
 }
 
-// namedSources omits shared categories (Bookmark, Download) — those are attributed to the default profile.
+// namedSources omits Bookmark (shared plist with no per-entry profile tag, so attributed to default).
+// Download is included because Downloads.plist carries DownloadEntryProfileUUIDStringKey per entry;
+// extractDownloads filters by owner UUID so default and named profiles each see their own downloads.
 //
 // LocalStorage slot for a follow-up PR:
 //