Disable progress bars when not in a TTY

Author: peterjdolan

beets/ui/__init__.py

@@ -1337,6 +1337,8 @@ def add_all_common_options(self):
 
 
 M = library.Album | library.Item | Any
+
+
 def iprogress_bar(sequence: Sequence[M], **kwargs) -> Generator[M, None, None]:
     """Construct and manage an `enlighten.Counter` progress bar while iterating.
 
@@ -1371,14 +1373,18 @@ def iprogress_bar(sequence: Sequence[M], **kwargs) -> Generator[M, None, None]:
     if "total" not in kwargs and hasattr(sequence, "__len__"):
         kwargs["total"] = len(sequence)
 
-    # Disabled in windows environments. See above for details
-    with enlighten.Manager(enabled=not is_windows) as manager:
+    # Disabled in windows environments and when not attached to a TTY. See method docs
+    # for details.
+    with enlighten.Manager(
+        enabled=not is_windows
+        and (hasattr(sys.stdout, "isatty") and sys.stdout.isatty())
+    ) as manager:
         with manager.counter(**kwargs) as counter:
             change_counter = counter.add_subcounter("blue")
 
             for item in sequence:
                 revision = None
-                if hasattr(item, '_revision'):
+                if hasattr(item, "_revision"):
                     revision = item._revision
 
                 # Yield the item, allowing it to be modified, or not.

beetsplug/permissions.py

@@ -103,9 +103,8 @@ def set_permissions(self, files=[], dirs=[]):
 
         # Change permissions for the files.
         for path in ui.iprogress_bar(
-            files,
-            desc="Setting permissions",
-            unit="files"):
+            files, desc="Setting permissions", unit="files"
+        ):
             # Changing permissions on the destination file.
             self._log.debug(
                 "setting file permissions on {}",
@@ -119,9 +118,8 @@ def set_permissions(self, files=[], dirs=[]):
 
         # Change permissions for the directories.
         for dir in ui.iprogress_bar(
-            dirs,
-            desc="Setting permissions",
-            unit="directories"):
+            dirs, desc="Setting permissions", unit="directories"
+        ):
             # Changing permissions on the destination directory.
             self._log.debug(
                 "setting directory permissions on {}",