Chore: Gulp clean task migration (#33694)

Co-authored-by: Aliullov Vlad <91639107+GoodDayForSurf@users.noreply.github.com>
Co-authored-by: Aliullov Vlad <vladislav.aliullov@devexpress.com>

Author: ajivanyandev

.github/copilot-instructions.md

@@ -64,7 +64,7 @@ For the full executor catalogue, conventions, and refactoring guidance, see @pac
 
 ## Build Pipeline
 
-localization → component generation (Renovation) → transpile (`babel-transform` for JS, `build-typescript` for TS) → bundle (Webpack via `devextreme-nx-infra-plugin:bundle`, debug + prod targets) → TypeScript declarations → SCSS compile (`devextreme-scss`) → npm package preparation. Task orchestration goes through Nx; cross-package builds (`all:build-dev`, `all:build`) live in the `workflows` package. Pre-commit hook runs `lint-staged` (stylelint + eslint --quiet).
+clean (`devextreme-nx-infra-plugin:clean` preserving CSS and npm metadata) → localization → component generation (Renovation) → transpile (`babel-transform` for JS, `build-typescript` for TS) → bundle (Webpack via `devextreme-nx-infra-plugin:bundle`, debug + prod targets) → TypeScript declarations → SCSS compile (`devextreme-scss`) → npm package preparation. Task orchestration goes through Nx; cross-package builds (`all:build-dev`, `all:build`) live in the `workflows` package. The `gulpfile.js` clean task is a thin delegate to `pnpm nx clean:artifacts devextreme`. Pre-commit hook runs `lint-staged` (stylelint + eslint --quiet).
 
 ## Conventions
 

packages/devextreme/gulpfile.js

@@ -4,29 +4,11 @@
 const gulp = require('gulp');
 const multiProcess = require('gulp-multi-process');
 const env = require('./build/gulp/env-variables');
-const cache = require('gulp-cache');
 const shell = require('gulp-shell');
 const context = require('./build/gulp/context');
 const { REMOVE_NON_PRODUCTION_MODULE } = context;
 
-gulp.task('clean', function(callback) {
-    require('del').sync([
-        'artifacts/**',
-        '!artifacts',
-        '!artifacts/css',
-        '!artifacts/css/*',
-        '!artifacts/css/fonts',
-        '!artifacts/css/fonts/*',
-        '!artifacts/css/icons',
-        '!artifacts/css/icons/*',
-        '!artifacts/npm',
-        '!artifacts/npm/devextreme',
-        '!artifacts/npm/devextreme/*.json',
-        '!artifacts/npm/devextreme-dist',
-    ]);
-    cache.clearAll();
-    callback();
-});
+gulp.task('clean', shell.task('pnpm nx clean:artifacts devextreme'));
 
 require('./build/gulp/bundler-config');
 require('./build/gulp/transpile');

packages/devextreme/project.json

@@ -13,7 +13,8 @@
         "targetDirectory": "./artifacts",
         "excludePatterns": [
           "./artifacts/css",
-          "./artifacts/npm/devextreme/package.json"
+          "./artifacts/npm/devextreme/package.json",
+          "./artifacts/npm/devextreme-dist"
         ]
       }
     },

packages/nx-infra-plugin/AGENTS.md

@@ -62,3 +62,18 @@ Each behavior is owned by exactly ONE executor's canonical tests; consumers must
 3. Preserve exact functional parity. Verify with the executor's e2e spec before and after.
 4. Update consumer imports in one batch.
 5. Run the full validation pipeline.
+
+## Migrated gulp tasks
+
+Gulp tasks that have been migrated to Nx executor targets (the gulp task is now a thin `shell.task` delegate):
+
+| Gulp task | Nx target         | Notes                                                                                                                                                                                                                                                     |
+| --------- | ----------------- | --------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- |
+| `clean`   | `clean:artifacts` | Uses `devextreme-nx-infra-plugin:clean` with `excludePatterns` to preserve `artifacts/css`, `artifacts/npm/devextreme/package.json`, and `artifacts/npm/devextreme-dist`. The gulp task delegates via `shell.task('pnpm nx clean:artifacts devextreme')`. |
+
+When migrating additional gulp tasks, follow the same pattern:
+
+1. Ensure the Nx target fully replicates the gulp task's behavior (including exclusion lists, configurations, etc.)
+2. Replace the gulp task body with `shell.task('pnpm nx <target> <project>')`.
+3. Remove unused imports from the gulpfile.
+4. Test that both `gulp <task>` and `pnpm nx <target> <project>` produce identical results.