fix(dep): upgrade corslite#425
Conversation
There was a problem hiding this comment.
Pull request overview
This PR updates the project’s CORS request helper dependency by switching from the deprecated corslite package to the @mapbox/corslite scoped package, aligning with the upstream module’s maintained namespace.
Changes:
- Replace
corslitewith@mapbox/corsliteinpackage.jsondependencies. - Regenerate
package-lock.jsonto reflect the new dependency and updated resolved tree.
Reviewed changes
Copilot reviewed 1 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| package.json | Switch dependency from corslite to @mapbox/corslite. |
| package-lock.json | Update lockfile for the new dependency (and other transitive resolution changes). |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| }, | ||
| "dependencies": { | ||
| "corslite": "^0.0.6", | ||
| "@mapbox/corslite": "^0.0.7", |
There was a problem hiding this comment.
corslite was replaced with @mapbox/corslite in dependencies, but the codebase still imports/mocks the old module name (require('corslite') in src/shortlink.js and jest.mock('corslite') in test/shortlink.test.js). As-is, installs/tests will fail with "Cannot find module 'corslite'". Either update those imports/mocks to the new package name, or add an npm alias so require('corslite') continues to resolve to @mapbox/corslite.
| "@mapbox/corslite": "^0.0.7", | |
| "@mapbox/corslite": "^0.0.7", | |
| "corslite": "npm:@mapbox/corslite@^0.0.7", |
No description provided.