fix: include commas in the profile size, handle failure case of profile more than 500kB in size

[yashasvibajpai]

Description

On debugging for the issue, we came across the two pointers addressed here.

1. commas in JSON were unaccounted for in the final size of payload.
2. we will throw an error in case a single profile's size itself crosses the 500 kb threshold set by Klaviyo themselves. Reference

Linear Ticket

Related to INT-2820, INT-2840

Security

• The code changed/added as part of this pull request won't create any security issues with how the software is being used.

[codecov]

Codecov Report

Attention: Patch coverage is 48.93617% with 96 lines in your changes missing coverage. Please review.

Project coverage is 73.58%. Comparing base (94aa254) to head (25fe1a5).

Files with missing lines	Patch %	Lines
...destinationmanager/klaviyobulkupload/apiService.go	12.76%	82 Missing ⚠️
...tionmanager/klaviyobulkupload/klaviyobulkupload.go	80.00%	8 Missing and 2 partials ⚠️
warehouse/internal/mocks/utils/mock_uploader.go	85.71%	4 Missing ⚠️

Additional details and impacted files

@@                Coverage Diff                 @@
##           release/1.38.x    #5241      +/-   ##
==================================================
+ Coverage           73.40%   73.58%   +0.18%     
==================================================
  Files                 424      425       +1     
  Lines               60243    59965     -278     
==================================================
- Hits                44221    44126      -95     
+ Misses              13578    13373     -205     
- Partials             2444     2466      +22     

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

[krishna2020]

Suggested change

	// if profileStructure length is more than 5 mb, throw an error
	// if profileStructure length is more than 500 kB, throw an error

[yashasvibajpai]

updated