Skip to content

refactor: Comprehensive API refactoring and type improvements#3645

Merged
jdalrymple merged 14 commits intomainfrom
bug/3580
Apr 25, 2026
Merged

refactor: Comprehensive API refactoring and type improvements#3645
jdalrymple merged 14 commits intomainfrom
bug/3580

Conversation

@jdalrymple
Copy link
Copy Markdown
Owner

@jdalrymple jdalrymple commented Oct 20, 2024

Summary

This is a complete rework of the entire library, making the calls to the various request helpers more explicit to support a wider ranger of flows such as the one where we need to pass raw data to a caller without assuming its structure. Additionally, this simplifies other types of body's we were sending, such as FormData 🙌

Additional fixes:

fixes #3580, #3807, #3829

@codecov
Copy link
Copy Markdown

codecov Bot commented Oct 27, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 82.58%. Comparing base (efa3645) to head (4cb3e05).
⚠️ Report is 9 commits behind head on main.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3645      +/-   ##
==========================================
- Coverage   83.12%   82.58%   -0.55%     
==========================================
  Files         234        3     -231     
  Lines        2578      155    -2423     
  Branches      273       28     -245     
==========================================
- Hits         2143      128    -2015     
+ Misses        393       14     -379     
+ Partials       42       13      -29     
Components Coverage Δ
Requester Utils ∅ <ø> (∅)
Core ∅ <ø> (∅)
REST 83.63% <ø> (ø)
CLI 82.00% <ø> (ø)
see 231 files with indirect coverage changes

@jdalrymple
Copy link
Copy Markdown
Owner Author

Back on this, haha doing it well is a pain 🫠

@jdalrymple jdalrymple added the breaking Changes are not backwards compatible label Nov 17, 2025
@jdalrymple jdalrymple force-pushed the bug/3580 branch 2 times, most recently from 3cd3b5d to 47f50d5 Compare February 24, 2026 22:10
@jdalrymple jdalrymple force-pushed the bug/3580 branch 2 times, most recently from 7f36469 to 389f039 Compare April 2, 2026 14:06
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 5, 2026

Bundle Report

Changes will increase total bundle size by 144.41kB (9.41%) ⬆️⚠️, exceeding the configured threshold of 5%.

Bundle name Size Change
core 1.55MB 143.36kB (10.16%) ⬆️⚠️
requester-utils 27.37kB 978 bytes (3.71%) ⬆️
rest 91.29kB 68 bytes (0.07%) ⬆️

Affected Assets, Files, and Routes:

view changes for bundle: rest

Assets Changed:

Asset Name Size Change Total Size Change (%)
index.cjs 34 bytes 19.55kB 0.17%
index.mjs 34 bytes 13.15kB 0.26%
view changes for bundle: requester-utils

Assets Changed:

Asset Name Size Change Total Size Change (%)
index.cjs 190 bytes 8.08kB 2.41%
index.mjs 190 bytes 6.61kB 2.96%
index.d.cts 299 bytes 6.34kB 4.95%
index.d.mts 299 bytes 6.34kB 4.95%
view changes for bundle: core

Assets Changed:

Asset Name Size Change Total Size Change (%)
index.d.cts 4.13kB 481.84kB 0.86%
index.d.mts 4.13kB 481.84kB 0.86%
index.cjs 67.47kB 263.07kB 34.5% ⚠️
index.mjs 67.47kB 253.8kB 36.21% ⚠️
map.json 153 bytes 73.53kB 0.21%

@jdalrymple jdalrymple changed the title WIP: Fixing default raw data handling refactor: Comprehensive API refactoring and type improvements Apr 6, 2026
@jdalrymple jdalrymple merged commit 19ce619 into main Apr 25, 2026
22 checks passed
@jdalrymple jdalrymple deleted the bug/3580 branch April 25, 2026 19:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

breaking Changes are not backwards compatible type:bug Changes fix a minor bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

PackageRegistry.publish not properly uploading files from html file input

1 participant