AI Assistant: sanitize null values for optional args

[anna-shakhova]

No description provided.

[Copilot]

Pull request overview

This PR updates the DataGrid AI Assistant command validation pipeline so that Zod transforms are applied during validation, enabling null values for optional command arguments to be sanitized into undefined (treating null as “absent”) before command execution.

Changes:

• Added optionalNullish() Zod helper to accept null | undefined and transform null → undefined for optional fields.
• Changed GridCommands.validate() to return parsed/transformed actions (or null), and updated the AI Assistant controller + tests accordingly.
• Updated selected command schemas (e.g., column pinning, summary) and added/extended Jest tests to cover null-to-undefined sanitization.

Reviewed changes

Copilot reviewed 13 out of 13 changed files in this pull request and generated 1 comment.

Show a summary per file

File	Description
packages/devextreme/testing/helpers/stubs/zodStub.js	Extends the QUnit/SystemJS Zod stub to support nullish() and transform() chaining.
packages/devextreme/js/__internal/grids/grid_core/ai_assistant/grid_commands.ts	Returns parsed/transformed actions from validation and applies schema parsing output to executed args.
packages/devextreme/js/__internal/grids/grid_core/ai_assistant/commands/utils.ts	Introduces optionalNullish() helper to treat null like an omitted optional value.
packages/devextreme/js/__internal/grids/grid_core/ai_assistant/commands/index.ts	Modifies the core command registry (currently comments out filterValueCommand).
packages/devextreme/js/__internal/grids/grid_core/ai_assistant/commands/columns.ts	Uses optionalNullish() for fixedPosition to sanitize null values.
packages/devextreme/js/__internal/grids/grid_core/ai_assistant/commands/tests/utils.test.ts	Adds unit tests for optionalNullish().
packages/devextreme/js/__internal/grids/grid_core/ai_assistant/commands/tests/columns.test.ts	Adds coverage for fixedPosition: null → undefined.
packages/devextreme/js/__internal/grids/grid_core/ai_assistant/ai_assistant_controller.ts	Consumes parsed actions from validation before execution.
packages/devextreme/js/__internal/grids/grid_core/ai_assistant/tests/grid_commands.test.ts	Updates validation tests to assert returned parsed actions and transform application.
packages/devextreme/js/__internal/grids/grid_core/ai_assistant/tests/error_handling.integration.test.ts	Updates validate spy behavior to return actions (new API).
packages/devextreme/js/__internal/grids/grid_core/ai_assistant/tests/ai_assistant_view_controller.integration.test.ts	Updates validate spy behavior to return actions (new API).
packages/devextreme/js/__internal/grids/data_grid/ai_assistant/commands/summary.ts	Uses optionalNullish() for optional summary item fields.
packages/devextreme/js/__internal/grids/data_grid/ai_assistant/commands/tests/summary.test.ts	Adds coverage for null → undefined sanitization in summary command args.

[Copilot]

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

[Copilot]

Pull request overview

Copilot reviewed 13 out of 13 changed files in this pull request and generated 4 comments.