Skip to content

chore(devextreme): replace gulp js-bundles-debug with native nx targets#32712

Open
chaosmirage wants to merge 3 commits into26_1from
chore/migrate-devextreme-to-nx-part-5
Open

chore(devextreme): replace gulp js-bundles-debug with native nx targets#32712
chaosmirage wants to merge 3 commits into26_1from
chore/migrate-devextreme-to-nx-part-5

Conversation

@chaosmirage
Copy link
Contributor

No description provided.

@chaosmirage chaosmirage force-pushed the chore/migrate-devextreme-to-nx-part-5 branch from a2590e7 to cd74a5d Compare February 27, 2026 10:46
@chaosmirage chaosmirage requested a review from a team February 27, 2026 11:15
@chaosmirage chaosmirage changed the title Chore/migrate devextreme to nx part 5 chore(nx-infra-plugin): add bundle executor for webpack bundling Feb 27, 2026
@chaosmirage chaosmirage changed the title chore(nx-infra-plugin): add bundle executor for webpack bundling chore(devextreme): replace gulp js-bundles-debug with native nx targets Feb 27, 2026
@chaosmirage chaosmirage marked this pull request as ready for review February 27, 2026 11:26
@chaosmirage chaosmirage requested a review from a team as a code owner February 27, 2026 11:26
Copilot AI review requested due to automatic review settings February 27, 2026 11:26
Copy link
Contributor

Copilot AI left a comment

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 replaces the Gulp-based js-bundles-debug task with native Nx targets using a new custom bundle executor in the devextreme-nx-infra-plugin package. This continues the ongoing migration from Gulp to Nx for build orchestration in the DevExtreme monorepo.

Changes:

  • Adds a new bundle executor to the nx-infra-plugin that wraps webpack bundling with support for debug and production modes
  • Splits the bundle:debug target into two sub-targets (bundle:debug:build and bundle:debug:headers) for better caching and modularity
  • Updates the bundle:debug target to use native Nx executors instead of calling Gulp directly

Reviewed changes

Copilot reviewed 8 out of 9 changed files in this pull request and generated no comments.

Show a summary per file
File Description
packages/nx-infra-plugin/src/executors/bundle/schema.ts TypeScript schema defining the bundle executor options including entries, source/output directories, mode, and webpack config path
packages/nx-infra-plugin/src/executors/bundle/schema.json JSON Schema validation for the bundle executor options
packages/nx-infra-plugin/src/executors/bundle/executor.ts Main executor implementation that loads webpack dynamically, builds entry maps, creates webpack configuration with debug/production modes, and handles BUILD_INTERNAL_PACKAGE environment variable
packages/nx-infra-plugin/src/executors/bundle/executor.e2e.spec.ts E2E test verifying the executor can bundle multiple entries in debug mode and produces the expected output files
packages/nx-infra-plugin/package.json Adds webpack as an optional peer dependency and dev dependency to support the new bundle executor
packages/nx-infra-plugin/executors.json Registers the new bundle executor in the plugin's executor manifest
packages/devextreme/project.json Replaces gulp js-bundles-debug command with new bundle:debug:build and bundle:debug:headers targets, updates inputs/outputs to match new build structure
pnpm-lock.yaml Updates webpack and related dependencies, primarily upgrading enhanced-resolve, es-module-lexer, and watchpack versions
.github/copilot-instructions.md Updates documentation to reflect that bundle creation now uses the devextreme-nx-infra-plugin:bundle executor
Files not reviewed (1)
  • pnpm-lock.yaml: Language not supported

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants