Improve 'json-prune' — match key with specific value, remove array or object if it contains specific item

[slavaleleka]

there are few issues:

1 current json-prune does not match values if we need specify pruning, only keys — obligatoryProps — are checked

2 if specific array item has to be pruned and we can target such items by their extra fields

#%#//scriptlet('json-prune', 'recs_group.[].tiles.[].[].*', 'recs_group.[].tiles.[].[].ad_origin')

but whole recs_group.[].tiles.[].[] array items will be pruned, not just ones with ad_origin.
check on fontanka.ru
https://uploads.adguard.com/slbdzdrv7cfxo.png

[ameshkov]

@slavaleleka what do you think about implementing a scriptlet that supports jq-like syntax for that? At least a subset of it.
https://stedolan.github.io/jq/tutorial/

Alternatively, there is this: https://jmespath.org/

I do not suggest supporting it fully, but we could take that syntax as a base and build our limited subset of it. This way it'd be easier for the scriptlet users to understand the capabilities.

PS: we'll probably need to reassign this task to a later milestone.

[ngorskikh]

No description provided.

[ameshkov]

@ngorskikh the spec should've been posted here I think: AdguardTeam/CoreLibs#1447

We'll also need a filter expression for checking whether the value contains the specified string: ?(key-contains <key> <value>)

[piquark6046]

Currently, $jsonprune modifier can modify only response of a request.
However, during maintaining List-KR filter, I often need its scriptlet version.

Example:

Gmarket.co.kr

URL: https://gmarket.co.kr/
Description: Extended CSS can hide the elements having AD label. However, their network requests cannot be blocked with the previous modifiers and URL rules (including a RegExp)
Related Issue: List-KR/List-KR#563
Targeted array items: $props.pageProps.serverData.topWideBannerSection.banners[?(@.isAd == true)]

[ameshkov]

Could you please explain what you're trying to do with that request?

Do you need to block a request that contains such a JSON path?

[piquark6046]

#183 (comment)
Oh, I forgot to add what i want.
What i want is a feature removing a property with a JSON path from result of calling JSON.parse.

[ameshkov]

So you'd like a scriptlet alternative to $jsonprune?

[piquark6046]

Yes.

[ameshkov]

Makes sense, $jsonprune syntax is more powerful and the end goal was indeed to provide a scriptlet version of it.

Moved it to Scriptlets v1.9 for now. @slavaleleka @stanislav-atr what do you think about it? To understand better what's required please check out this issue in CL: AdguardTeam/CoreLibs#1447

[slavaleleka]

we shall return to the issue later

[Alex-302]

Later is now, @slavaleleka :)

[slavaleleka]

the same is needed in scriptlets as well:
AdguardTeam/CoreLibs#1717

UPD: jsonp cannot be handled by the json-prune scriptlet

[slavaleleka]

second issue in first issue post cannot be fixed in current syntax implementation — obligatoryProps is checked for the whole json and there is no way to check exact inner array item child (which is object) and conditionally prune it while other array items remains unchanged

[ameshkov]

@slavaleleka not sure what you mean, aren't filter expressions supposed to solve this?

[maximtop]

I couldn't figure out how we can handle JSONP requests, apart from one method. For example, let's say there's a JSONP callback like this:

...
<script>
    function handleResponse(data) {
        console.log(JSON.stringify(data));
    }
</script>
<script src="http://server.com/jsonp?callback=handleResponse"></script>
...

We can find the matching callback function from the page and proxify the call to it.

[ameshkov]

Probably jsonp shouldn't be in scope of this issue since there's no clear solution.

[piquark6046]

Please take a look for json-prune scriptlet version: gorhill/uBlock@3152896 and gorhill/uBlock@14d60ac.

[Alex-302]

Another case
AdguardTeam/AdguardFilters#195194
@maximtop