[arnaud-moncel]

Definition of Done

Fixes PRD-1171

General

• Write an explicit title for the Pull Request, following Conventional Commits specification
• Test manually the implemented changes
• Validate the code quality (indentation, syntax, style, simplicity, readability)

Security

• Consider the security impact of the changes made

Note

Support nested condition trees and advertise canUseNestedConditionTree capability

• ConditionTreeParser.fromPlainObject now uses a recursive parse method so nested groups stay nested instead of being flattened. Empty root groups remain valid, but empty nested groups are rejected with a validation error.
• The parser's isLeaf and isBranch guards no longer treat null input or non-array conditions as valid shapes; these now hit the invalid-tree error path.
• FilterGenerator.computeMatch in filter.ts handles empty branches: an empty Or produces a match-none expression, and an empty And produces no restriction.
• The capabilities route reports canUseNestedConditionTree: true so clients can detect the new behavior.
• Adds unit tests for the parser and integration tests for MongoDB and Sequelize list filtering with nested and empty branches.
• Behavioral Change: filters that previously flattened or accepted empty nested groups now either throw a ValidationError (see parse in condition-tree-parser.ts) or match differently for empty Or/And branches.

Changes since #2029 opened

• Wrapped the ConditionTreeParser.fromPlainObject call in AuthorizationService.getScope in a try/catch block that throws an UnprocessableError when scope parsing fails. [e7d2070]

^{Macroscope summarized fcaa8a3.}

[qltysh]

2 new issues

Tool	Category	Rule	Count
qlty	Structure	Function with many returns (count = 4): fetchCapabilities	2

[qltysh]

Coverage Impact

This PR will not change total coverage.

Modified Files with Diff Coverage (2)

Rating	File	% Diff	Uncovered Line #s
	packages/agent/src/utils/condition-tree-parser.ts	100.0%
	packages/datasource-mongoose/src/utils/pipeline/filter.ts	100.0%
	Total	100.0%

🚦 See full report on Qlty Cloud »

🛟 Help

• Diff Coverage: Coverage for added or modified lines of code (excludes deleted files). Learn more.

• Total Coverage: Coverage for the whole repository, calculated as the sum of all File Coverage. Learn more.

• File Coverage: Covered Lines divided by Covered Lines plus Missed Lines. (Excludes non-executable lines including blank lines and comments.)

  • Indirect Changes: Changes to File Coverage for files that were not modified in this PR. Learn more.

[christophebrun-forest]

OK for me
Manual tests ok with server and front

[linear-code]

PRD-1171