Skip to content

fix: 🐛 String codable decoding upgrade - #26

Merged
Mithos5r merged 14 commits into
mainfrom
feat/fix_json_string_codable_decoding
Oct 6, 2026
Merged

Mithos5r merged 14 commits into
mainfrom
feat/fix_json_string_codable_decoding

Conversation

@Mithos5r

@Mithos5r Mithos5r commented Oct 5, 2026

Copy link
Copy Markdown
Contributor

📝 Description

Fix non-ascii decoding in JsonStringCodable and add a decoding benchmark

✅ Type of Change

Please check the box that applies to your pull request.

  • 🐛 Bug fix (a non-breaking change that fixes an issue)
  • ✨ New feature (a non-breaking change that adds functionality)
  • 💥 Breaking change (a fix or feature that would cause existing functionality to not work as expected)
  • ♻️ Refactor (a code change that neither fixes a bug nor adds a feature)
  • 📚 Documentation update
  • ⚙️ Chore (a change to the build process or auxiliary tools)

✔️ Checklist

Please make sure you have completed the following tasks before submitting your pull request:

  • I have read the CONTRIBUTING.md file.
  • My code follows the style guidelines of this project.
  • I have performed a self-review of my own code.
  • I have added tests that prove my fix is effective or that my feature works.
  • All existing tests pass with my changes.
  • I have updated the documentation accordingly.

@Mithos5r Mithos5r self-assigned this Oct 5, 2026
Base automatically changed from update to main October 5, 2026 19:14
@codecov

codecov Bot commented Oct 5, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 97.11%. Comparing base (2ae8c55) to head (12d3250).
⚠️ Report is 5 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff             @@
##             main      #26       +/-   ##
===========================================
+ Coverage   82.29%   97.11%   +14.81%     
===========================================
  Files          13       13               
  Lines         209      208        -1     
===========================================
+ Hits          172      202       +30     
+ Misses         37        6       -31     

☔ View full report in Codecov by Harness.
📢 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.

@Mithos5r Mithos5r changed the title feat: 🐛 String codable decoding upgrade fix: 🐛 String codable decoding upgrade Oct 6, 2026
Use type patterns in a new _jsonKind helper so the argument message is
stable in obfuscated and minified release builds, where runtimeType of
user-defined types is not contractually preserved. Add bool and double
scalar cases to the bytes deserialization tests.
@Mithos5r
Mithos5r merged commit dd1fbde into main Oct 6, 2026
4 checks passed
@Mithos5r
Mithos5r deleted the feat/fix_json_string_codable_decoding branch October 6, 2026 11:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant