Maybe wrong formatted validation messages

[rvetere]

I'm integrating the snowplow-cli into our github workflow to automate most of our tasks and achieve a true "schema-as-code" approach. While developing our custom github action for that purpose, i'm using the ds validate command heavliy to display review comments right in the pull request UI:

Now what made me suspicous is this strange "path" in one of the messages (/). I think the first warning message solves it correctly with (/properties/testProperty). But the second warning does it probably wrong, it should be (/properties/platform) instead of properties platform (/), right?

Not really a big issue but i just wanted to report that before i try to parse it into the correct path myself :-)

[gleb-lobov]

Hi @rvetere

Looks funky, I'd like to take a closer look. Can you provide steps to replicate?
Like "1. add a new property to an existing schema, 2. run snowplow-cli ds validate --json-output"

Would help me to understand where the issue steams from

[rvetere]

Hi @gleb-lobov

I'm basically just removing a field from required but the field itself is still not nullable:

and then i run the command snowplow-cli data-structures validate --json-output -o *** -a *** -S *** ../../segments/snowplow/data-structures/ch.digitecgalaxus/gtmtest_action.json which will give me:

{"time":"2025-12-18T13:51:16.18553082Z","level":"INFO","msg":"validating from","paths":["../../segments/snowplow/data-structures/ch.digitecgalaxus/gtmtest_action.json"]}
{"time":"2025-12-18T13:51:16.41971742Z","level":"INFO","msg":"will patch","file":"../../segments/snowplow/data-structures/ch.digitecgalaxus/gtmtest_action.json","vendor":"ch.digitecgalaxus","name":"gtmtest_action","version":"10-0-1","local":"ecc4b5b46e46ff9b8323ae5d0b0a6b776fff8bfc23947190c03798126462e165","remote":"433ff176518b1c1f931c90cc74991acaf8993103329cb17db6e9bf35e5491156"}
{"time":"2025-12-18T13:51:17.956243263Z","level":"WARN","msg":"validation","file":"../../segments/snowplow/data-structures/ch.digitecgalaxus/gtmtest_action.json","messages":"Use \"type: null\" to indicate a field as optional for properties platform (/)"}
{"time":"2025-12-18T13:51:17.956271947Z","level":"ERROR","msg":"validation","file":"../../segments/snowplow/data-structures/ch.digitecgalaxus/gtmtest_action.json","destination":"bigquery","suggestedVersion":"10-0-2","messages":"Changing required property to nullable at /platform"}
{"time":"2025-12-18T13:51:17.956290121Z","level":"ERROR","msg":"validation failed: 1 validation failures"}

Thanks a lot to have a look into it ❤️