Don't update license and readme based fields unless we have non-nil data

Author: finestructure

Sources/App/Commands/Ingestion.swift

@@ -333,27 +333,31 @@ enum Ingestion {
         repository.defaultBranch = repoMetadata.defaultBranch
         repository.forks = repoMetadata.forkCount
         repository.fundingLinks = repoMetadata.fundingLinks?.compactMap(FundingLink.init(from:)) ?? []
-        repository.hasSPIBadge = readmeInfo?.containsSPIBadge()
         repository.homepageUrl = repoMetadata.homepageUrl?.trimmed
         repository.isArchived = repoMetadata.isArchived
         repository.isInOrganization = repoMetadata.isInOrganization
         repository.keywords = Set(repoMetadata.topics.map { $0.lowercased() }).sorted()
         repository.lastIssueClosedAt = repoMetadata.lastIssueClosedAt
         repository.lastPullRequestClosedAt = repoMetadata.lastPullRequestClosedAt
         repository.license = .init(from: repoMetadata.licenseInfo)
-        repository.licenseUrl = licenseInfo?.htmlUrl
         repository.name = repoMetadata.repositoryName
         repository.openIssues = repoMetadata.openIssues.totalCount
         repository.openPullRequests = repoMetadata.openPullRequests.totalCount
         repository.owner = repoMetadata.repositoryOwner
         repository.ownerName = repoMetadata.owner.name
         repository.ownerAvatarUrl = repoMetadata.owner.avatarUrl
         repository.s3Readme = s3Readme
-        repository.readmeHtmlUrl = readmeInfo?.htmlUrl
         repository.releases = repoMetadata.releases.nodes.map(Release.init(from:))
         repository.stars = repoMetadata.stargazerCount
         repository.summary = repoMetadata.description
         repository.forkedFrom = fork
+        if let readmeInfo {
+            repository.hasSPIBadge = readmeInfo.containsSPIBadge()
+            repository.readmeHtmlUrl = readmeInfo.htmlUrl
+        }
+        if let licenseInfo {
+            repository.licenseUrl = licenseInfo.htmlUrl
+        }
 
         do {
             try await repository.save(on: database)

Tests/AppTests/IngestionTests.swift

@@ -213,6 +213,23 @@ extension AllTests.IngestionTests {
                 #expect(repo.stars == 2)
                 #expect(repo.summary == "package desc")
             }
+
+            // update again but without license and readme info
+            try await Ingestion.updateRepository(on: app.db,
+                                                 for: repo,
+                                                 metadata: md,
+                                                 licenseInfo: nil,
+                                                 readmeInfo: nil,
+                                                 s3Readme: .cached(s3ObjectUrl: "url", githubEtag: "etag"),
+                                                 fork: .parentURL("https://github.com/foo/bar.git"))
+
+            // validate that license and readme info stays unchanged
+            do {
+                #expect(repo.hasSPIBadge == true)
+                #expect(repo.license == .mit)
+                #expect(repo.licenseUrl == "license url")
+                #expect(repo.readmeHtmlUrl == "readme html url")
+            }
         }
     }