Radish alpha
r
radicle-desktop
Radicle desktop app
Radicle
Git (anonymous pull)
Log in to clone via SSH
Commits ›
Optimize: Collect into sorted collection
✓ CI success Matthias Beyer committed 1 month ago
commit db6130f827bc520fdd1230715a8cb017008d81eb
parent 052839ac7cc10b5e53b9688099df52ce8a3492d9
1 passed (1 total) View logs
1 file changed +18 -17
modified crates/radicle-types/src/traits/repo.rs
@@ -77,26 +77,27 @@ pub trait Repo: Profile {
    fn list_repos_summary(&self) -> Result<Vec<repo::RepoSummary>, Error> {

        let profile = self.profile();

        let storage = &profile.storage;
- 
        let repos = storage.repositories()?;
- 
        let mut entries = Vec::new();
- 
        for RepositoryInfo { rid, doc, .. } in repos {
- 
            let Some(data) = doc
- 
                .payload()
- 
                .get(&doc::PayloadId::project())
- 
                .and_then(|payload| repo::ProjectPayloadData::try_from((*payload).clone()).ok())
- 
            else {
- 
                continue;
- 
            };
- 
            entries.push(repo::RepoSummary {
- 
                rid,
- 
                name: data.name,
- 
            });
- 
        }
+ 
        let entries = storage
+ 
            .repositories()?
+ 
            .into_iter()
+ 
            .filter_map(|RepositoryInfo { rid, doc, .. }| {
+ 
                doc.payload()
+ 
                    .get(&doc::PayloadId::project())
+ 
                    .and_then(|payload| repo::ProjectPayloadData::try_from((*payload).clone()).ok())
+ 
                    .map(|data| (rid, data))
+ 
            })
+ 
            .map(|(rid, data)| {
+ 
                let summary = repo::RepoSummary {
+ 
                    rid,
+ 
                    name: data.name,
+ 
                };
- 
        entries.sort_by_key(|r| r.name.to_lowercase());
+ 
                (summary.name.to_lowercase(), summary)
+ 
            })
+ 
            .collect::<BTreeMap<_, _>>();
- 
        Ok::<_, Error>(entries)
+ 
        Ok::<_, Error>(entries.values().collect())

    }



    fn repo_count(&self) -> Result<repo::RepoCount, Error> {