Skip to content

chore: bump all deps#241

Merged
eddienubes merged 14 commits into
mainfrom
bump-all-deps
Mar 18, 2026
Merged

chore: bump all deps#241
eddienubes merged 14 commits into
mainfrom
bump-all-deps

Conversation

@eddienubes

Copy link
Copy Markdown
Owner

No description provided.

Copilot AI review requested due to automatic review settings February 24, 2026 19:23

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the repo to newer dependency versions (notably Fastify and Fastify plugins), and adjusts the library/tests/examples to match changed behavior (e.g., cookie headers and SameSite defaults), while removing a now-unneeded polyfill.

Changes:

  • Bump key dev dependencies (e.g., fastify to v5, @fastify/cookie to v11) and regenerate pnpm-lock.yaml.
  • Remove the global polyfill import and delete the polyfill implementation.
  • Update response/cookie-related tests and examples; tighten some typings (e.g., isBinary type guard, SageHttpResponse constructor typing).

Reviewed changes

Copilot reviewed 1 out of 2 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
package.json Dependency bumps (Fastify + plugins) and adds packageManager pin.
pnpm-lock.yaml Lockfile refresh reflecting dependency upgrades and new transitive deps/engines.
src/index.ts Removes polyfill side-effect import.
src/polyfill.ts Deletes the polyfill implementation.
test/setup.ts Removes polyfill import; keeps Node-version-based fs.openAsBlob shim.
src/utils.ts Refines isBinary into a TS type guard.
src/SageHttpResponse.ts Updates docs and loosens constructor input typing.
test/SageHttpResponse.spec.ts Removes as any casts now that constructor typing is looser.
src/Sage.ts Adjusts header expectation logic (as part of refactor/reorder).
test/request.spec.ts Updates cookie header expectations (SameSite=Lax appears).
examples/fastify.spec.ts Updates cookie options + expected set-cookie header output.
.vscode/settings.json Adds workspace TS SDK setting (currently uses a nonstandard key).
.gitignore Ignores additional local tool artifacts.
Comments suppressed due to low confidence (1)

package.json:80

  • Bumping fastify to ^5.7.4 raises the minimum supported Node.js version (Fastify v5 and some transitive deps require Node >= 20). The repo still declares engines.node as ">=18.0.0", so installs/tests on Node 18 will fail despite the declared support. Please align engines.node (and CI/tooling) with the new minimum Node version required by these dependencies.
    "express": "^4.21.2",
    "fastify": "^5.7.4",
    "form-data": "^4.0.4",

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread package.json Outdated
@codecov

codecov Bot commented Feb 24, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 86.59%. Comparing base (6484197) to head (895f8af).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #241      +/-   ##
==========================================
- Coverage   94.93%   86.59%   -8.34%     
==========================================
  Files          14       10       -4     
  Lines        2880      455    -2425     
  Branches      191      105      -86     
==========================================
- Hits         2734      394    -2340     
+ Misses        145       57      -88     
- Partials        1        4       +3     

☔ View full report in Codecov by Sentry.
📢 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.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@eddienubes eddienubes self-assigned this Feb 25, 2026
@eddienubes eddienubes merged commit 729863e into main Mar 18, 2026
7 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