Remove redundant backward compatibility argument parsing

Co-authored-by: ekinnee <1707617+ekinnee@users.noreply.github.com>

Author: Copilot

AppleMusicXmlToLidarr.py

@@ -250,57 +250,41 @@ def albums_main(xml_path: str, output_json: str, not_found_json: str):
     import argparse
     import sys
 
-    # Check if the first argument is a subcommand
-    has_subcommand = len(sys.argv) > 1 and sys.argv[1] in ['tracks', 'albums']
+    # Set up argument parser with subcommands
+    parser = argparse.ArgumentParser(description="Convert Apple Music Library.xml to Lidarr JSON import format with not-found items exported.")
+    subparsers = parser.add_subparsers(dest='command', help='Available commands')
 
-    if has_subcommand:
-        # Use subcommands
-        parser = argparse.ArgumentParser(description="Convert Apple Music Library.xml to Lidarr JSON import format with not-found items exported.")
-        subparsers = parser.add_subparsers(dest='command', help='Available commands')
-        
-        # Tracks subcommand
-        tracks_parser = subparsers.add_parser('tracks', help='Process individual tracks (default)')
-        tracks_parser.add_argument("--recheck", action="store_true", 
-                            help="Recheck mode: process items from not_found_json instead of parsing XML")
-        tracks_parser.add_argument("xml_file", nargs="?", help="Path to Apple Music Library.xml (not needed in recheck mode)")
-        tracks_parser.add_argument("output_json", help="Output JSON file path for found items")
-        tracks_parser.add_argument("not_found_json", help="Output JSON file path for not found items")
-        
-        # Albums subcommand  
-        albums_parser = subparsers.add_parser('albums', help='Process unique albums')
-        albums_parser.add_argument("xml_file", help="Path to Apple Music Library.xml")
-        albums_parser.add_argument("output_json", help="Output JSON file path for found albums (default: albums.json)")
-        albums_parser.add_argument("not_found_json", help="Output JSON file path for not found albums (default: albums_notfound.json)")
-        
-        args = parser.parse_args()
-        
-        # Handle subcommands
-        if args.command == 'albums':
-            albums_main(args.xml_file, args.output_json, args.not_found_json)
-        elif args.command == 'tracks':
-            if args.recheck:
-                if args.xml_file:
-                    logging.warning("XML file argument ignored in recheck mode")
-                recheck_not_found(args.output_json, args.not_found_json)
-            else:
-                if not args.xml_file:
-                    tracks_parser.error("xml_file is required when not in recheck mode")
-                main(args.xml_file, args.output_json, args.not_found_json)
-    else:
-        # Backward compatibility: use original argument structure
-        parser = argparse.ArgumentParser(description="Convert Apple Music Library.xml to Lidarr JSON import format with not-found items exported.")
-        parser.add_argument("--recheck", action="store_true", 
-                            help="Recheck mode: process items from not_found_json instead of parsing XML")
-        parser.add_argument("xml_file", nargs="?", help="Path to Apple Music Library.xml (not needed in recheck mode)")
-        parser.add_argument("output_json", help="Output JSON file path for found items")
-        parser.add_argument("not_found_json", help="Output JSON file path for not found items")
-        args = parser.parse_args()
-        
+    # Tracks subcommand
+    tracks_parser = subparsers.add_parser('tracks', help='Process individual tracks (default)')
+    tracks_parser.add_argument("--recheck", action="store_true", 
+                        help="Recheck mode: process items from not_found_json instead of parsing XML")
+    tracks_parser.add_argument("xml_file", nargs="?", help="Path to Apple Music Library.xml (not needed in recheck mode)")
+    tracks_parser.add_argument("output_json", help="Output JSON file path for found items")
+    tracks_parser.add_argument("not_found_json", help="Output JSON file path for not found items")
+    
+    # Albums subcommand  
+    albums_parser = subparsers.add_parser('albums', help='Process unique albums')
+    albums_parser.add_argument("xml_file", help="Path to Apple Music Library.xml")
+    albums_parser.add_argument("output_json", help="Output JSON file path for found albums")
+    albums_parser.add_argument("not_found_json", help="Output JSON file path for not found albums")
+    
+    # Parse arguments, defaulting to 'tracks' if no subcommand provided
+    args = parser.parse_args()
+    
+    # If no subcommand was provided, show help
+    if args.command is None:
+        parser.print_help()
+        sys.exit(1)
+    
+    # Handle subcommands
+    if args.command == 'albums':
+        albums_main(args.xml_file, args.output_json, args.not_found_json)
+    elif args.command == 'tracks':
         if args.recheck:
             if args.xml_file:
                 logging.warning("XML file argument ignored in recheck mode")
             recheck_not_found(args.output_json, args.not_found_json)
         else:
             if not args.xml_file:
-                parser.error("xml_file is required when not in recheck mode")
+                tracks_parser.error("xml_file is required when not in recheck mode")
             main(args.xml_file, args.output_json, args.not_found_json)