fix: filter file watcher duplicate events

[Valentin271]

This fixes #198

As discussed in the issue, I refactored the file watcher to use notify_debouncer_full.

I did so with as few modifications as possible to avoid breaking anything. The most interesting part is the DebounceEventHandler impl in which I basically filtered unwanted and duplicate events.
The debouncer timeout of 10ms seems good for up to 100 ~ 150Kb files on my machine.

All unit tests passed locally. I've also tested with the files I was having issues with.

I also added logs I found interesting for debugging and changed the log format to show milliseconds since the file_watcher is always faster than a second.

[CosmicHorrorDev]

Seems to be failing on mac, but all the file watching logic is notorious for platform specific quirks. I'll go ahead and re-run the failed jobs, but I doubt they'll pass this time (more so for my sanity). I won't be able to take too close of a look till after holidays, but I'll still be around to hit the CI button or for reviews (I also think CI l still runs in your fork when you push to it for whenever I'm not around)

[Valentin271]

Alright no problem. I had a feeling it would only work on my machine, it's good you have a CI to check that. I'll fix them.

It does run on my fork after enabling it.

[CosmicHorrorDev]

Great! And do note that the watcher tests are a bit weird. They're flaky when using short delays, but longer delays suck for dev-feedback loops, so it starts with shorter delays and retries the tests a few times with increased delays on failure

Just in case the logs for the failed tests look confusing

[Valentin271]

Alright good to know, I'm currently looking at the tests and I'm suspecting the delay is too short.

[CosmicHorrorDev]

Found this issue thread. Looks like it could be relevant (specifically that write events may be getting lost when using the debouncer on macos)

notify-rs/notify#150 (comment)

[CosmicHorrorDev]

I just pushed a commit that should fix the sanity file watching tests since it looks like we were ignoring Create events and that was the only one getting sent

Aside: trace logs are currently getting filtered out from tests, but I'm starting to think that it'd be a good idea to retain them still

[CosmicHorrorDev]

Eh went ahead and changed that too. We really need all the info we can get when it comes to CI-driven development 🙃

[CosmicHorrorDev]

CI is green. Hope you didn't mind all the commits 🥳

I'm happy to merge this as-is. Most of the issues seem to stem from MacOS sending create events instead of modify events when using the debouncer from some reason

[Valentin271]

We really need all the info we can get when it comes to CI-driven development

Yep, I agree.

Hope you didn't mind all the commits

No problem go ahead.

---

Most of the issues seem to stem from MacOS sending create events instead of modify

From notify_debouncer_full docs:

Doesn’t emit Modify events after a Create event

I'm happy to merge this as-is

Can confirm it still works, happy to see this merged then :)

[CosmicHorrorDev]

I want to offer a huge thanks for all your help on this :)

[Valentin271]

You're welcome, I'm glad I could help. This is a great tool that I really love and want to help with and see grow (or at least not abandoned).

[CosmicHorrorDev]

There are plenty of things to work on to say the least. I can tag you in some issues that may be easier to start out with

[Valentin271]

I've seen a few already, but I wouldn't mind if you tagged the good first issues.

[CosmicHorrorDev]

Keep in mind that features don't have to be perfect to open a PR. I'm more than happy to help with draft PRs. From looking over the issues I see:

• Some keybindings stuff if you're comfortable with recursion
  • extra keybindings should have precedence over base #188
• Adding history to our navigation
  • Feature: a way to go back #162
  • [Suggestion] History and Navigation #53
• Adding an --edit-config flag
  • Add an --edit-config flag #131
• Adding double/triple click although that may be complicated
  • Support double and triple click for highlighting text #43

I picked those because they're either well tested or generally well isolated

I also have a lot of other things in an obsidian notebook, so I can start opening issues for those over the next few days. Feel free to reach out in whichever issues you want to take on so that we can talk about the general design/approach

[CosmicHorrorDev]

Opened some new issues that include a lot of potentially good starter issues along with actually labeling easy(ish) issues

https://github.com/trimental/inlyne/issues?q=is%3Aopen+is%3Aissue+label%3AE-easy