fix(platform): normalize xlsx column headers for reliable import

[yannickmonney]

Summary

• Normalize all Excel record keys to lowercase+trimmed in parseExcelFile — single point of normalization
• Simplify all three excel mappers (customer, vendor, product) to only check lowercase keys
• Added 23 tests covering excel mapper happy paths, missing fields, and defaults

Fixes #1040, fixes #1054

Summary by CodeRabbit

Release Notes

• New Features

  • Enhanced Excel file parsing with automatic column header normalization for improved compatibility.

• Bug Fixes

  • Fixed field mapping inconsistencies in product, customer, and vendor imports.
  • Improved validation of required fields during file uploads.

• Tests

  • Added comprehensive test coverage for Excel import scenarios.

[greptile-apps]

Your free trial has ended. If you'd like to continue receiving code reviews, you can add a payment method here.

[coderabbitai]

Caution

Review failed

Pull request was closed or merged during review

📝 Walkthrough

Walkthrough

This pull request normalizes Excel column header handling during file imports. The file parsing utility now trims whitespace and converts all column headers to lowercase when reading Excel files. The import mappers for vendors, customers, and products are updated to work with these normalized keys, removing fallbacks for alternate key casings. A comprehensive test suite validates that the Excel parsers correctly handle lowercase keys, apply appropriate defaults, enforce required fields, and normalize specific field names like imageurl to imageUrl.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 50.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately summarizes the main change: normalizing XLSX column headers for reliable import.
Linked Issues check	✅ Passed	The PR addresses both linked issues by implementing case-insensitive field matching and normalizing column headers to resolve product and customer import failures.
Out of Scope Changes check	✅ Passed	All changes are directly scoped to fixing XLSX import issues; test additions, mapper simplifications, and header normalization align with addressing the linked issues.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch fix/xlsx-import-field-matching

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}