Improve ResultData candidate deduplication

[Copilot]

• Fix ResultData.init(candidates:) to prefer candidates with better kind (.phoneNumber or .link) over .text when deduplicating
• Rename existingIndex to index for concise naming
• Expand guard statement to use line breaks
• Add blank line after guard block and split multi-condition if onto separate lines
• Rename result to results (plural, since it's a collection)

---

🔒 GitHub Advanced Security automatically protects Copilot coding agent pull requests. You can protect all pull requests by enabling Advanced Security for your repositories. Learn more about Advanced Security.

[cyanzhong]

@copilot use a concise PR title.

[Copilot]

@copilot use a concise PR title.

I don't have the ability to update the PR title directly. A suggested concise title: "Improve ResultData candidate deduplication".

[Copilot]

Pull request overview

This PR refactors the ResultData deduplication logic to improve code organization and aims to prefer detected kinds (phoneNumber, link) over plain text when handling duplicate text values. The refactoring replaces a Set-based filter approach with an explicit loop that tracks seen text values using a dictionary mapping to indices.

Changes:

• Refactored deduplication logic from Set-based filtering to explicit loop with index tracking
• Changed data structure from Set<String> to [String: Int] dictionary for duplicate detection
• Renamed variables for clarity (existingIndex→index, result→results)

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.