fix: script pull abort and async generator cleanup (#212)

## Problems

**1. `new_event_loop().run_until_complete()` skips async generator
cleanup**

As the Python docs note:
> "consider using the higher-level `asyncio.run()` function, instead of
using these lower level functions to manually create and close an event
loop." — [asyncio event
loop](https://docs.python.org/3/library/asyncio-eventloop.html)

`asyncio.run()` additionally calls `loop.shutdown_asyncgens()` before
closing, which `run_until_complete()` does not.
Async generators (e.g. `TreeModule.list`) that are not fully exhausted
have their `aclose()` scheduled but never awaited, producing a
`RuntimeWarning`.

**2. `click.Abort` during overwrite prompt aborts the entire pull**

`click.confirm()` raises `click.exceptions.Abort` when it receives EOF
(non-interactive stdin, e.g. scripts or some IDEs) or the user presses
Ctrl+C.
Previously this propagated uncaught out of `process_entry`, aborting the
whole pull and leaving all remaining files unpulled.

## Fixes

- Replace all `asyncio.new_event_loop().run_until_complete()` calls with
`asyncio.run()` (`get_modules`, `pull`, `push`, `run`)
- Catch `click.exceptions.Abort` per file in `script pull` and skip to
the next entry instead of aborting

Author: sveneberth

src/viur_cli/scriptor/cli.py

@@ -23,7 +23,7 @@ def get_modules():
     global _modules
     if _modules is None:
         _modules = Modules(scriptor_config["base_url"], cookies=scriptor_config["cookies"])
-        asyncio.new_event_loop().run_until_complete(_modules.init())
+        asyncio.run(_modules.init())
 
     return _modules
 
@@ -146,9 +146,12 @@ def create_file():
                         with open(_path, "r") as f:
                             if hashlib.sha256(entry["script"].encode()).digest() \
                                     != hashlib.sha256(f.read().encode()).digest():
-                                if click.confirm(f"There is a difference with {entry['path']}. Overwrite?"):
-                                    os.remove(_path)
-                                    create_file()
+                                try:
+                                    if click.confirm(f"There is a difference with {entry['path']}. Overwrite?"):
+                                        os.remove(_path)
+                                        create_file()
+                                except click.exceptions.Abort:
+                                    click.echo("\nSkipping...")
 
                 else:
                     create_file()
@@ -161,7 +164,7 @@ def create_file():
         async for leaf in tree.list(skel_type="leaf"):
             await process_entry(leaf, False)
 
-    asyncio.new_event_loop().run_until_complete(main())
+    asyncio.run(main())
 
 
 @script.command()
@@ -313,7 +316,7 @@ def on_modified(event):
                     elif os.path.getmtime(event.src_path) == modified_files[event.src_path]:
                         return
                     modified_files[event.src_path] = os.path.getmtime(event.src_path)
-                    asyncio.new_event_loop().run_until_complete(main(event.src_path))
+                    asyncio.run(main(event.src_path))
                 except Exception as e:
                     import traceback
                     print(f"Error: on file {event.src_path} {e}")
@@ -341,9 +344,9 @@ def on_modified(event):
                 observer.stop()
                 observer.join()
 
-        asyncio.new_event_loop().run_until_complete(watch_loop())
+        watch_loop()
         return
-    asyncio.new_event_loop().run_until_complete(main())
+    asyncio.run(main())
 
 
 @script.command()
@@ -376,4 +379,4 @@ async def main():
 
         await module.main()
 
-    asyncio.new_event_loop().run_until_complete(main())
+    asyncio.run(main())