Handle deviations (revertive) solely within DS

[steiler]

• Make the syncing of running not blindly overwrite but transfor the sync tree

[steiler]

We need to fix another conceptual flaw that exists atm with the non-revertive intents.
So now, if DS sees an intent (also non-revertive) for the first time, I will take it and apply it. Because this is a new intent basically, hence we need to apply. So far so good...
But when DS restarts and CS sends the intent again, I see the intent for the first time hence I assume that it is meant to be applied. So a restart of DS causes the first time intent apply to become revertive.

So we might need to add another flag for CS to set, which indicates if the intent is new or not.
What's the expected behaviour for the case where a config intent is altered?

[steiler]

Fixes #383
by considering only leaf entries as second highest if they are not marked for deletion.
https://github.com/sdcio/data-server/pull/379/changes#diff-3cda332246a4bf91747920508400cb96c741f5126758e42403547b035032a148R332

[henderiw]

#379 (comment)

Can we put these comments in the epic issue since they cover multiple entities
sdcio/.github#12

The way to solve this in my view is have recovery include the deviations.

[henderiw]

lgtm

[alexandernorth]

lgtm

[steiler]

Ran tests manually against actual main of config-server.
Only the non-revertive tests are failing which is expected at this point in time, since that functionality is not yet implemented in DS. This will be covered in a seperate PR.
https://github.com/sdcio/integration-tests/actions/runs/21937698324

Going to merge this now.

[codecov]

Codecov Report

❌ Patch coverage is 63.40289% with 228 lines in your changes missing coverage. Please review.

Files with missing lines	Patch %	Lines
pkg/tree/processor_remove_deleted.go	0.00%	45 Missing ⚠️
pkg/datastore/sync.go	43.63%	20 Missing and 11 partials ⚠️
pkg/tree/processor_importer.go	79.64%	13 Missing and 10 partials ⚠️
pkg/tree/types/import_stats.go	0.00%	21 Missing ⚠️
pkg/tree/leaf_variants.go	57.57%	11 Missing and 3 partials ⚠️
pkg/tree/importer/json/json_tree_importer.go	56.52%	10 Missing ⚠️
pkg/tree/processor_explicit_delete.go	82.35%	8 Missing and 1 partial ⚠️
pkg/datastore/types/transaction_intent.go	0.00%	8 Missing ⚠️
pkg/datastore/transaction_rpc.go	65.00%	7 Missing ⚠️
pkg/tree/processor_reset_flags.go	81.08%	4 Missing and 3 partials ⚠️
... and 15 more

📢 Thoughts on this report? Let us know!

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.