refactor(yaml): add sanitizeInput() function

[timreichen]

• moves code into a dedicated sanitizeInput() function

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 95.10%. Comparing base (832be24) to head (44820fd).

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #5248      +/-   ##
==========================================
- Coverage   95.47%   95.10%   -0.37%     
==========================================
  Files         461      461              
  Lines       38002    38006       +4     
  Branches     5535     5504      -31     
==========================================
- Hits        36281    36146     -135     
- Misses       1679     1818     +139     
  Partials       42       42              

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

[kt3k]

sanitizeInput is only used once. I don't see this change reasonable.

[iuioiua]

I don't see the purpose of this change either.

[timreichen]

sanitizeInput is only used once. I don't see this change reasonable.

I will create a second PR that uses generator functions for documents in loader.ts. Therefore load() and loadDocuments() will need to call sanitizeInput() individually. The gain will be to throw early if there are more than one document in load() instead of parsing all documents and then throw.

[kt3k]

LGTM