read_csv_id_file: Remove unnecessary contains_key

[alexdewar]

I noticed this while reviewing your PR. It's not actually that big of a deal, but I felt the itch to fix it anyway.

[Copilot]

Pull Request Overview

This PR refines the duplicate check when reading CSV records by removing an explicit contains_key check and instead relying on the return value of the insert operation.

• Simplifies duplicate key detection by using map.insert return value
• Removes redundant contains_key check

Comments suppressed due to low confidence (1)

src/input.rs:156

• [nitpick] The variable name 'existing' may be ambiguous regarding its purpose; consider renaming it to something more descriptive such as 'duplicateFound' to enhance clarity.

let existing = map.insert(Rc::clone(&id), record).is_some();

[codecov]

Codecov Report

Attention: Patch coverage is 66.66667% with 1 line in your changes missing coverage. Please review.

Project coverage is 95.13%. Comparing base (441197f) to head (cfb422c).
Report is 3 commits behind head on main.

Files with missing lines	Patch %	Lines
src/input.rs	66.66%	0 Missing and 1 partial ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #480      +/-   ##
==========================================
- Coverage   95.13%   95.13%   -0.01%     
==========================================
  Files          36       36              
  Lines        4931     4930       -1     
  Branches     4931     4930       -1     
==========================================
- Hits         4691     4690       -1     
  Misses        120      120              
  Partials      120      120              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.