More fixes

These include resolving a problem with interactive edits on mac not
working. This was likely due to a quirk with the NamedTemporaryFile
implementation. I insteda switched to manually creating a temporary file
and that worked.

It also includes an early escape for when a user's inventory is empty.

broker/commands.py

@@ -326,6 +326,9 @@ def inventory(details, _list, sync, filter):
         )
         for host in inventory
     ]
+    if not curated_host_info:
+        CONSOLE.print("No hosts found in inventory.")
+        return
     table = helpers.dictlist_to_table(curated_host_info, "Host Inventory", _id=True)
     if _list:
         table.title = None

broker/config_manager.py

@@ -6,7 +6,6 @@
 from pathlib import Path
 import pkgutil
 import sys
-from tempfile import NamedTemporaryFile
 
 import click
 from logzero import logger
@@ -56,11 +55,11 @@ def __init__(self, settings_path=None):
 
     def _interactive_edit(self, chunk):
         """Write the chunk data to a temporary file and open it in an editor."""
-        with NamedTemporaryFile(mode="w+", suffix=".yaml") as tmp:
-            yaml.dump(chunk, tmp)
-            click.edit(filename=tmp.name)
-            tmp.seek(0)
-            new_data = tmp.read()
+        temp_file = Path("temp_settings.yaml")
+        yaml.dump(chunk, temp_file)
+        click.edit(filename=str(temp_file))
+        new_data = temp_file.read_text()
+        temp_file.unlink()
         # first try to load it as yaml
         try:
             return yaml.load(new_data)