Skip to content

Remove more unused dev deps after grunt/karma removal#1304

Merged
matux merged 1 commit into
masterfrom
matux/remove-unusued-devdeps
Sep 8, 2025
Merged

Remove more unused dev deps after grunt/karma removal#1304
matux merged 1 commit into
masterfrom
matux/remove-unusued-devdeps

Conversation

@matux

@matux matux commented Sep 8, 2025

Copy link
Copy Markdown
Contributor

Description of the change

This PR completes the test infrastructure modernization by removing 12 additional unused devDependencies. This reduces the project's dependency footprint by 37 packages total (274 packages removed from node_modules), when counting both Karma and Grunt dev dependencies.

Each removed dependency and its previous purpose:

  • babel-eslint: Legacy ESLint parser, superseded by @babel/eslint-parser already in use
  • coverage-istanbul-loader: Webpack loader for code coverage, was only used by Karma
  • jade: Template engine, only referenced in server error parsing for legacy jade template
    errors
  • jasmine-core: Jasmine test framework, project uses Mocha/Chai instead
  • natives: Node.js native module list, no usage found in codebase
  • requirejs: AMD module loader, only used in one Karma test that's been migrated
  • script-loader: Webpack loader for legacy scripts, was only configured in Karma
  • @types/glob: TypeScript types for glob, not needed without TypeScript
  • bluebird: Promise library, project uses native Promises
  • chalk: Terminal string styling, not used in our scripts
  • jquery-mockjax: jQuery AJAX mocking, tests use Sinon instead
  • stackframe: Stack trace parsing, project uses error-stack-parser instead

Verification

  • ✅ All browser tests pass with Web Test Runner
  • ✅ All server tests pass with Mocha
  • ✅ Build process works without Grunt
  • ✅ All distribution files generated correctly

Related issues

SDK-493/replace-karma-with-webtest-runner-for-modern-performant-browser

@matux matux self-assigned this Sep 8, 2025
@matux matux requested a review from Copilot September 8, 2025 21:50

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 completes the test infrastructure modernization by removing 12 unused devDependencies that were previously required for the now-removed Karma and Grunt build systems. The cleanup reduces the project's dependency footprint significantly while maintaining all existing functionality.

  • Removes legacy testing and build tool dependencies no longer needed after Karma/Grunt removal
  • Eliminates redundant packages that have been superseded by modern alternatives already in use
  • Cleans up unused utility libraries that have no remaining references in the codebase

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@matux matux merged commit 2c62d89 into master Sep 8, 2025
4 checks passed
@matux matux deleted the matux/remove-unusued-devdeps branch September 8, 2025 22:03
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