Version checking in CLI isn't working (yet)

[josephjclark]

Things we probably need to do:

• When checking out, track the version we forked from: we can work out local version history from there
• this local version needs to be saved to openfn.yaml probably
• If you make a change and then check out another branch - those changes and history will be lost. We can warn, but there's not a lot else we can do. This is when project export comes in handy, as I can dump my changes when checking out. We do not automatically "write" local changes back to project.yaml because that's just a view of the app.
• Does the app need to track forked-from too? No, because the first history item IS the version it was forked from

Versions are compatible if the remote project has no versions since the forked-from version.

After deploying, presumably we need to update forked from. The local project doesn't get removed or anything, so we need to keep maintaining the state

[josephjclark]

been though this in some depth this afternoon

The key is: when asking "has the local working version diverged from the app"? we need to check the app's history and make sure it hasn't moved. When we do this check, we don't care what the current cli version is - of course that's not in the history. What we care about is that the version we implicitly forked from is still the head.

So openfn.yaml under project must keep a forked_from version number. And when deploying, we just look for that version in the remote head. After deploy, we update the forked_from version so that the app can safely compact histories.

Also note that on fetch, we basically ignore the local history and just write the latest app history to it.

Everything should track beautifully then.

[josephjclark]

Note that if the app changes and the forked_From version is no longer valid, well, that's OK! It proves that the app version changed and the versions are incomatible.

[josephjclark]

ah dang - a complication here is that forked_from is obviously on each workflow, not on the project. I keep forgetting this. That makes tracking harder and noisier