add branch name to payload

[FilipHarald]

A second try at adding branch name to payload #19 . This time with 5-second cache to reduce calls to systemlist, hopefully resolving #26 .

[FilipHarald]

Still learning vimscript. @ErikBjare @johan-bjareholt , let me know if I can improve anything.

[johan-bjareholt]

This call is still synchronous. So while it's an improvement, there's still a risk of VIM freezing when doing s blocking operation like systemlist. It would be optimal if we did something similar as in HTTPPostJson with jobstart.

[FilipHarald]

@johan-bjareholt I think this will work. I update the branch async, but the branch is still sent with the rest of the data. I think this is preferred over sending it in a separate heartbeat?

[FilipHarald]

@johan-bjareholt , do you want to have something added to the README about how to access the data through the query-api?
Like so:

afk_events = query_bucket(find_bucket("aw-watcher-afk_"));
editor_events = query_bucket(find_bucket("aw-watcher-vim_"));
editor_events = filter_period_intersect(editor_events, filter_keyvals(afk_events, "status", ["not-afk"]));
merged_events = merge_events_by_keys(editor_events, ["branch"]);
RETURN = sort_by_duration(merged_events);

[ErikBjare]

do you want to have something added to the README about how to access the data through the query-api?

@FilipHarald That would be nice :)

[ErikBjare]

LGTM! Would like a review from @johan-bjareholt too though.

[johan-bjareholt]

Like I said before, systemlist is a synchronous operation and needs to be replaced with something asynchronous such as jobstart/job_start.

[varac]

@FilipHarald This is pretty stale already but I'd love to get this merged, any chance you would continue with this ?

[FilipHarald]

@varac it helps a little bit now that I know that someone needs it :) I will see what I can do. No promises though!

[FilipHarald]

@johan-bjareholt I made a second try now.

Also, instead of doing N/A for when not a git-repo, I'm just splitting the tag completely.