Skip to content

Update axios ro 1.16.0 and mocha to 11.7.5#1494

Merged
w666 merged 1 commit into
masterfrom
feature-update-deps-2026-05-14
May 13, 2026
Merged

Update axios ro 1.16.0 and mocha to 11.7.5#1494
w666 merged 1 commit into
masterfrom
feature-update-deps-2026-05-14

Conversation

@w666

@w666 w666 commented May 13, 2026

Copy link
Copy Markdown
Collaborator

Summary by CodeRabbit

  • Bug Fixes

    • Improved handling of multipart HTTP responses to robustly detect and parse content types.
  • Chores

    • Updated axios dependency to version 1.16.0
    • Updated mocha testing framework dependency to version 11.7.5

Review Change Stack

@coderabbitai

coderabbitai Bot commented May 13, 2026

Copy link
Copy Markdown

Caution

Review failed

Pull request was closed or merged during review

📝 Walkthrough

Walkthrough

Bumps axios (^1.15.0 → ^1.16.0) and mocha (^11.7.4 → ^11.7.5) in package.json, and normalizes Axios content-type header shapes in HttpClient.request() before creating MIMEType and parsing multipart/related responses.

Changes

Dependency Updates

Layer / File(s) Summary
Dependency version bumps
package.json
axios updated to ^1.16.0 and mocha updated to ^11.7.5.

HTTP multipart parsing

Layer / File(s) Summary
Normalize content-type and MIMEType initialization
src/http.ts
HttpClient.request() normalizes response.headers['content-type'] (handles string or array), constructs MIMEType from the normalized value, then continues multipart/related boundary parsing and SOAP envelope extraction.

Estimated Code Review Effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Poem

🐰 I nibbled headers, smoothed the lines,
content-types tidy, parsing aligns,
axios and mocha took a gentle leap,
MIME assembled, multipart runs deep,
code hops onward — steady, not steep.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main changes: dependency version bumps for axios and mocha, which are the primary alterations in this changeset.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feature-update-deps-2026-05-14

Warning

Review ran into problems

🔥 Problems

Stopped waiting for pipeline failures after 30000ms. One of your pipelines takes longer than our 30000ms fetch window to run, so review may not consider pipeline-failure results for inline comments if any failures occurred after the fetch window. Increase the timeout if you want to wait longer or run a @coderabbit review after the pipeline has finished.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@w666 w666 force-pushed the feature-update-deps-2026-05-14 branch from 1156a4a to b4dad50 Compare May 13, 2026 20:58
 - Fix parsing headers after axios update
@w666 w666 force-pushed the feature-update-deps-2026-05-14 branch from b4dad50 to 1d890fc Compare May 13, 2026 21:23
@w666 w666 merged commit bdace03 into master May 13, 2026
3 of 4 checks passed
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.

2 participants