Re: [PATCH v6 0/3] parser_json: support handle for rule positioning

[Phil Sutter <phil@nwl.cc>]

On Tue, Jan 20, 2026 at 08:53:00PM +0100, Alexandre Knecht wrote:
> This patch series enables handle-based rule positioning for JSON
> add/insert commands.
> 
> Changes since v5:
> - Merged nested if-conditionals, sorted expressions cheap to expensive
>   (Phil feedback)
> - Use Reverse Christmas Tree notation for variable declarations
>   (Phil feedback)
> - Test 0007: Use $DIFF to verify ruleset state after ADD operations
>   instead of just checking command success (Phil feedback)
> 
> Changes since v4:
> - CTX_F_EXPR_MASK now uses inverse mask (UINT32_MAX & ~(...)) as
>   suggested by Phil, for future-proof expression flag filtering
> - Removed nested block in json_parse_cmd(), variables declared at
>   function start per project style (Phil/Florian feedback)
> - Test 0007: Removed redundant insert position check (covered by 0008),
>   replaced grep|grep|awk with single sed call
> - Test 0008: Added test for insert without handle, added test for
>   multiple commands in single transaction, fixed error message typo
> - Added .json-nft dump files for both tests
> 
> All JSON tests pass. check-tree.sh shows no new errors.
> 
> Alexandre Knecht (3):
>   parser_json: support handle for rule positioning in explicit JSON
>     format
>   tests: shell: add JSON test for all object types
>   tests: shell: add JSON test for handle-based rule positioning

Series applied, thanks!

I added a small fix for tests/json_echo test suite which broke due to a
leftover handle in an add rule command.

Cheers, Phil