Skip to content

Conversation

@AtofStryker
Copy link
Contributor

@AtofStryker AtofStryker commented Nov 19, 2025

Reverts #32936. see #32936 (comment)


Note

Reverts mime to v3 across packages, adds @types/mime v3 types, updates a test to expect application/javascript, and cleans up changelog/lockfile.

  • Dependencies:
    • Downgrade mime to ^3.0.0 in packages/driver, packages/server, and packages/net-stubbing.
    • Add @types/mime@^3.0.1 in packages/driver, packages/server, and packages/net-stubbing.
    • Update yarn.lock to reflect mime@^3 and new @types/mime entries.
  • Tests:
    • Update MIME expectation for .js files in packages/driver/cypress/e2e/commands/actions/selectFile.cy.ts to application/javascript.
  • Docs:
    • Remove changelog entry claiming mime was upgraded to 4.1.0 in cli/CHANGELOG.md.

Written by Cursor Bugbot for commit e4b0960. This will update automatically on new commits. Configure here.

@AtofStryker AtofStryker force-pushed the revert-32936-issue-30144-mime-to-4.x branch from 5c5904f to e4b0960 Compare November 19, 2025 18:48
@AtofStryker AtofStryker merged commit 201e22e into develop Nov 19, 2025
6 of 7 checks passed
@AtofStryker AtofStryker deleted the revert-32936-issue-30144-mime-to-4.x branch November 19, 2025 18:49
expect($input.files?.[1].name).to.eq('app.js')
expect($input.files?.[0].type).to.eq('application/json')
expect($input.files?.[1].type).to.eq('text/javascript')
expect($input.files?.[1].type).to.eq('application/javascript')
Copy link

Choose a reason for hiding this comment

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

Bug: Test expects wrong MIME type after reverting mime

The test expects application/javascript as the MIME type for .js files, but after reverting the mime package from version 4.1.0 back to 3.0.0, the package will return text/javascript instead. The original test before the mime upgrade expected text/javascript, confirming that mime 3.0.0 returns this value. This test expectation was changed to match mime 4.x behavior but wasn't reverted along with the dependency downgrade, causing the test to fail.

Fix in Cursor Fix in Web

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.

3 participants