Add CSV validation warnings and example CSV files

[Pranav-0440]

Fixes - #134

[Copilot]

Pull request overview

Adds a warning-collecting CSV parsing path and documents example CSV inputs to help users spot “wrong-but-parseable” CSV values (Issue #134).

Changes:

• Extend parseCsv to return { data, warnings } and add basic validations for type and modbus:entity.
• Surface CSV warnings in the CSV import flow (CreateTd) and add example valid.csv / invalid.csv files plus README.
• Add/adjust tests for the new parse result shape and validation behavior.

Reviewed changes

Copilot reviewed 8 out of 8 changed files in this pull request and generated 9 comments.

Show a summary per file

File	Description
src/utils/parser.ts	Returns parsed CSV data plus validation warnings for select fields.
src/utils/parser.test.ts	Updates tests for the new return shape and adds warning assertions.
src/components/App/CreateTd.tsx	Displays CSV warnings after import (currently via error state/dialog).
src/components/Dialogs/ConvertTmDialog.tsx	Large refactor unrelated to CSV import; currently introduces a TM→TD conversion regression and misleading copy.
test/csv-examples.test.ts	Adds a “test” file for example CSVs, but currently contains no actual Vitest tests/assertions.
doc/examples/csv/valid.csv	Adds a “known-good” CSV example.
doc/examples/csv/invalid.csv	Adds an “intentionally invalid” CSV example to trigger warnings.
doc/examples/csv/README.md	Documents the examples and warning expectations (currently inconsistent).

---

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

[netlify]

✅ Deploy Preview for editdor ready!

Name	Link
🔨 Latest commit	2afb413
🔍 Latest deploy log	https://app.netlify.com/projects/editdor/deploys/69a1e717588efe0008dc98a4
😎 Deploy Preview	https://deploy-preview-195--editdor.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify project configuration.

[Pranav-0440]

I’ve cleaned up the PR to reduce unrelated diffs by resetting to origin/master and cherry-picking only the CSV validation commit. Please let me know if any further adjustments are needed.

[TejInaco]

I’ve cleaned up the PR to reduce unrelated diffs by resetting to origin/master and cherry-picking only the CSV validation commit. Please let me know if any further adjustments are needed.

Yes, you did. But you didn't implemented the changes requested by the code reviews. Change to draft until changes are committed.

[Pranav-0440]

Thanks @TejInaco for marking it draft.
I’ve now implemented all requested review changes:

• Wired hasHeaders correctly into Papa.parse
• Removed duplicate transform option
• Precomputed the lowercase Modbus entity list outside the loop
• Added separate case-mismatch warning logic for modbus:entity
• Fixed warning rendering in CreateTd
• Removed unused warnings state from ConvertTmDialog
• Converted csv-examples.test.ts into proper Vitest tests
• Updated README to clarify case-insensitive validation with casing warnings

Marking the PR ready for review again. Please let me know if anything else needs adjustment.

[TejInaco]

You still have pending reviews @Pranav-0440 to be done. Return to draft.

[Pranav-0440]

@TejInaco - All review threads have now been addressed and resolved.
Please let me know if anything further needs adjustment.

[Pranav-0440]

@TejInaco Thanks for the feedback.

I’ve cleaned the branch and removed all unrelated changes (external files, documentation, and example CSV files).
The PR now only includes the CSV validation changes in:

• src/utils/parser.ts
• src/utils/parser.test.ts
• src/components/App/CreateTd.tsx

Please let me know if any further adjustments are needed.

[TejInaco]

I didn't say to remove those files. Do a test folder inside utils to have your example of valid csv and invalid csv. Add more test to read from these files and try to find more edge cases on unit test. Mark the pending reviews as soon as you read them

[TejInaco]

Copyright is not to be deleted. Revert these changes

[Pranav-0440]

@TejInaco I restored the original license header and reverted unrelated changes.
The CSV fixtures are now placed under src/utils/test/fixtures and the tests have been updated to read from them. Additional edge cases were added and all tests are passing locally. Please let me know if further adjustments are needed.

[Pranav-0440]

Manual UI verification:

1. valid.csv loads successfully without warnings.

2. invalid.csv displays aggregated validation warnings (invalid type, invalid modbus entity, and casing issues).

3. TD properties are still created correctly.