chore(4317): Xlsx to exceljs

[drernie]

Description

"Fixes issues #4317

Remove xlsx, which has been abandoned and has security issues.
Replace with exceljs.

TODO

• [X ] Unit tests
• Changelog entry (skip if change is not significant to end users, e.g. docs only)

[drernie]

It fails a "no error" test because the function rejects the invalid type. Do we need to change the type signature?

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 91.44%. Comparing base (7488a87) to head (5f6b42e).

Additional details and impacted files

@@             Coverage Diff             @@
##           master    #4325       +/-   ##
===========================================
+ Coverage   39.10%   91.44%   +52.34%     
===========================================
  Files         787      108      -679     
  Lines       34827     9974    -24853     
  Branches     5525        0     -5525     
===========================================
- Hits        13618     9121     -4497     
+ Misses      20666      853    -19813     
+ Partials      543        0      -543     

Flag	Coverage Δ
api-python	91.41% <ø> (ø)
catalog	?
lambda	91.53% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ 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.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[greptile-apps]

PR Summary

This PR replaces the deprecated 'xlsx' library with 'exceljs' for spreadsheet handling in the Quilt catalog, improving security and maintainability while preserving existing functionality.

• Updated /catalog/app/utils/spreadsheets/spreadsheets.ts to use ExcelJS's eachRow method for parsing spreadsheets
• Added new helper functions readXlsx and readCSVString in /catalog/app/utils/spreadsheets/spreadsheets.spec.ts for testing
• Implemented proper async file reading with error handling in readSpreadsheet function
• Maintained comprehensive test coverage across multiple file formats (xlsx, xlsm, ods, fods, csv)
• Upgraded to ExcelJS 4.4.0 in package.json for enhanced security and stability

_{3 file(s) reviewed, 4 comment(s)
Edit PR Review Bot Settings | Greptile}

[greptile-apps]

style: test case uses invalid input type but relies on TypeScript compilation - should test runtime behavior