Fix T201 in release_tools and scripts: replace print() with sys.stdout/stderr.write()

Agent-Logs-Url: https://github.com/GitHubSecurityLab/seclab-taskflow-agent/sessions/8acf69e6-94a8-40d4-a66c-2b90b143f143

Co-authored-by: kevinbackhouse <4358136+kevinbackhouse@users.noreply.github.com>

Author: Copilot

release_tools/copy_files.py

@@ -29,7 +29,7 @@ def copy_files(file_list, dest_dir):
         abs_dest = os.path.abspath(os.path.join(dest_dir, rel_path))
         os.makedirs(os.path.dirname(abs_dest), exist_ok=True)
         shutil.copy2(abs_src, abs_dest)
-        print(f"Copied {abs_src} -> {abs_dest}")
+        sys.stdout.write(f"Copied {abs_src} -> {abs_dest}\n")
 
 
 def ensure_git_repo(dest_dir):
@@ -41,22 +41,22 @@ def ensure_git_repo(dest_dir):
     if not os.path.isdir(git_dir):
         try:
             subprocess.run(["git", "init", "-b", "main"], cwd=dest_dir, check=True)
-            print(f"Initialized new git repository in {dest_dir} with 'main' as the default branch")
+            sys.stdout.write(f"Initialized new git repository in {dest_dir} with 'main' as the default branch\n")
         except subprocess.CalledProcessError as e:
-            print(f"Failed to initialize git repository in {dest_dir}: {e}")
+            sys.stderr.write(f"Failed to initialize git repository in {dest_dir}: {e}\n")
             sys.exit(1)
     else:
         # Ensure main branch exists and is checked out
         try:
             branches = subprocess.check_output(["git", "branch"], cwd=dest_dir, text=True)
             if "main" not in branches:
                 subprocess.run(["git", "checkout", "-b", "main"], cwd=dest_dir, check=True)
-                print("Created and switched to 'main' branch.")
+                sys.stdout.write("Created and switched to 'main' branch.\n")
             else:
                 subprocess.run(["git", "checkout", "main"], cwd=dest_dir, check=True)
-                print("Switched to 'main' branch.")
+                sys.stdout.write("Switched to 'main' branch.\n")
         except subprocess.CalledProcessError as e:
-            print(f"Failed to ensure 'main' branch in {dest_dir}: {e}")
+            sys.stderr.write(f"Failed to ensure 'main' branch in {dest_dir}: {e}\n")
             sys.exit(1)
 
 
@@ -70,16 +70,16 @@ def git_add_files(file_list, dest_dir):
         for rel_path in file_list:
             try:
                 subprocess.run(["git", "add", "-f", rel_path], check=True)
-                print(f"git add {rel_path}")
+                sys.stdout.write(f"git add {rel_path}\n")
             except subprocess.CalledProcessError as e:
-                print(f"Failed to git add {rel_path}: {e}")
+                sys.stderr.write(f"Failed to git add {rel_path}: {e}\n")
     finally:
         os.chdir(cwd)
 
 
 if __name__ == "__main__":
     if len(sys.argv) != 3:
-        print("Usage: python copy_files.py <file_list.txt> <dest_dir>")
+        sys.stderr.write("Usage: python copy_files.py <file_list.txt> <dest_dir>\n")
         sys.exit(1)
     file_list_path = sys.argv[1]
     dest_dir = sys.argv[2]

release_tools/publish_docker.py

@@ -25,17 +25,17 @@ def build_and_push_image(dest_dir, image_name, tag):
     )
     # Push
     subprocess.run(["docker", "push", f"{image_name}:{tag}"], check=True)
-    print(f"Pushed {image_name}:{tag}")
+    sys.stdout.write(f"Pushed {image_name}:{tag}\n")
     digest = get_image_digest(image_name, tag)
-    print(f"Image digest: {digest}")
+    sys.stdout.write(f"Image digest: {digest}\n")
     with open("/tmp/digest.txt", "w") as f:
         f.write(digest)
 
 
 if __name__ == "__main__":
     if len(sys.argv) != 3:
-        print("Usage: python build_and_publish_docker.py <ghcr_username/repo> <tag>")
-        print("Example: python build_and_publish_docker.py ghcr.io/anticomputer/my-python-app latest")
+        sys.stderr.write("Usage: python build_and_publish_docker.py <ghcr_username/repo> <tag>\n")
+        sys.stderr.write("Example: python build_and_publish_docker.py ghcr.io/anticomputer/my-python-app latest\n")
         sys.exit(1)
 
     image_name = sys.argv[1]

scripts/migrate_to_jinja2.py

@@ -91,7 +91,7 @@ def migrate_file(self, file_path: Path) -> bool:
             True if file was modified, False otherwise
         """
         if file_path.suffix != '.yaml':
-            print(f"Skipping non-YAML file: {file_path}")
+            sys.stdout.write(f"Skipping non-YAML file: {file_path}\n")
             return False
 
         try:
@@ -101,25 +101,25 @@ def migrate_file(self, file_path: Path) -> bool:
             migrated_content = self.migrate_content(original_content)
 
             if migrated_content == original_content:
-                print(f"No changes needed: {file_path}")
+                sys.stdout.write(f"No changes needed: {file_path}\n")
                 return False
 
             if self.dry_run:
-                print(f"\n{'='*60}")
-                print(f"Would modify: {file_path}")
-                print(f"{'='*60}")
+                sys.stdout.write(f"\n{'='*60}\n")
+                sys.stdout.write(f"Would modify: {file_path}\n")
+                sys.stdout.write(f"{'='*60}\n")
                 self._show_diff(original_content, migrated_content)
                 return True
 
             # Write migrated content
             with open(file_path, 'w') as f:
                 f.write(migrated_content)
 
-            print(f"Migrated: {file_path}")
+            sys.stdout.write(f"Migrated: {file_path}\n")
             return True
 
         except Exception as e:
-            print(f"Error migrating {file_path}: {e}", file=sys.stderr)
+            sys.stderr.write(f"Error migrating {file_path}: {e}\n")
             return False
 
     def _show_diff(self, original: str, migrated: str):
@@ -129,9 +129,9 @@ def _show_diff(self, original: str, migrated: str):
 
         for i, (orig, mig) in enumerate(zip(orig_lines, mig_lines, strict=False), 1):
             if orig != mig:
-                print(f"Line {i}:")
-                print(f"  - {orig}")
-                print(f"  + {mig}")
+                sys.stdout.write(f"Line {i}:\n")
+                sys.stdout.write(f"  - {orig}\n")
+                sys.stdout.write(f"  + {mig}\n")
 
     def migrate_directory(self, directory: Path, recursive: bool = True) -> int:
         """Migrate all YAML files in directory.
@@ -143,10 +143,10 @@ def migrate_directory(self, directory: Path, recursive: bool = True) -> int:
         yaml_files = list(directory.glob(pattern))
 
         if not yaml_files:
-            print(f"No YAML files found in {directory}")
+            sys.stdout.write(f"No YAML files found in {directory}\n")
             return 0
 
-        print(f"Found {len(yaml_files)} YAML files")
+        sys.stdout.write(f"Found {len(yaml_files)} YAML files\n")
 
         modified_count = 0
         for yaml_file in yaml_files:
@@ -184,7 +184,7 @@ def main():
     total_modified = 0
     for path in args.paths:
         if not path.exists():
-            print(f"Path not found: {path}", file=sys.stderr)
+            sys.stderr.write(f"Path not found: {path}\n")
             continue
 
         if path.is_file():
@@ -197,14 +197,14 @@ def main():
             )
             total_modified += modified
         else:
-            print(f"Invalid path: {path}", file=sys.stderr)
+            sys.stderr.write(f"Invalid path: {path}\n")
 
-    print(f"\n{'='*60}")
+    sys.stdout.write(f"\n{'='*60}\n")
     if args.dry_run:
-        print(f"Dry run complete. {total_modified} files would be modified.")
+        sys.stdout.write(f"Dry run complete. {total_modified} files would be modified.\n")
     else:
-        print(f"Migration complete. {total_modified} files modified.")
-    print(f"{'='*60}")
+        sys.stdout.write(f"Migration complete. {total_modified} files modified.\n")
+    sys.stdout.write(f"{'='*60}\n")
 
 
 if __name__ == '__main__':