feat: add uv support and modernize CLI

README.md

@@ -9,15 +9,24 @@ A library for reading and writing Garmin FIT files.
 Installation
 ==================
 
+### Using uv (recommended)
+
+```bash
+uv add fit-tool
 ```
+
+### Using pip
+
+```bash
 python3 -m pip install --upgrade pip
-python3 -m pip install --upgrade fit_tool
+python3 -m pip install --upgrade fit-tool
 ```
 
 Command line interface
 =======================
+
 ```console
-usage: fittool [-h] [-v] [-o OUTPUT] [-l LOG] [-t TYPE] FILE
+usage: fit-tool [-h] [-v] [-o OUTPUT] [-l LOG] [-t TYPE] FILE
 
 Tool for managing FIT files.
 
@@ -34,8 +43,13 @@ optional arguments:
 ```
 
 ### Convert file to CSV
-```console
-fittool oldstage.fit 
+
+```bash
+# Using uv
+uv run fit-tool oldstage.fit
+
+# Or after installation
+fit-tool oldstage.fit
 ```
 
 Library Usage

fit_tool/cli.py

@@ -0,0 +1,79 @@
+"""
+Command line interface for fit_tool.
+
+Copyright (c) 2017 Stages Cycling. All rights reserved.
+"""
+import argparse
+import logging
+import os
+
+from fit_tool.fit_file import FitFile
+from fit_tool.utils.logging import logger
+
+
+def parse_args():
+    """Parse command line arguments."""
+    parser = argparse.ArgumentParser(
+        description="Tool for managing FIT files."
+    )
+    parser.add_argument(
+        'fitfile',
+        metavar='FILE',
+        help='FIT file to process'
+    )
+    parser.add_argument(
+        '-v', '--verbose',
+        action='store_true',
+        help='specify verbose output'
+    )
+    parser.add_argument("-o", "--output", help="Output filename.")
+    parser.add_argument("-l", "--log", help="Log filename.")
+    parser.add_argument(
+        "-t", "--type",
+        help="Output format type. Options: csv, fit."
+    )
+
+    return parser.parse_args()
+
+
+def main():
+    """Main entry point."""
+    args = parse_args()
+
+    formatter = logging.Formatter(fmt="%(asctime)s %(levelname)s %(message)s")
+
+    if args.log:
+        handler = logging.FileHandler(args.log)
+        handler.setFormatter(formatter)
+        logger.addHandler(handler)
+
+    if args.verbose:
+        handler = logging.StreamHandler()
+        handler.setFormatter(formatter)
+        logger.addHandler(handler)
+        logger.setLevel(logging.DEBUG)
+    logger.info(f'Loading fit file {args.fitfile}...')
+
+    fit_file = FitFile.from_file(args.fitfile)
+
+    if args.type:
+        format_type = args.type
+    elif args.output:
+        _, out_ext = os.path.splitext(os.path.basename(args.output))
+        format_type = out_ext.lstrip('.')
+    else:
+        format_type = 'csv'
+
+    basename_noext, _ = os.path.splitext(os.path.basename(args.fitfile))
+    output_filename = args.output or f'{basename_noext}.{format_type}'
+
+    logger.info(f'Exporting fit file to {output_filename} as format {format_type}...')
+
+    if format_type == 'csv':
+        fit_file.to_csv(output_filename)
+    elif format_type == 'fit':
+        fit_file.to_file(output_filename)
+
+
+if __name__ == "__main__":
+    main()

fit_tool/tests/test_cli.py

@@ -0,0 +1,92 @@
+"""Tests for the command line interface."""
+
+import os
+import tempfile
+import unittest
+from unittest.mock import patch
+
+from fit_tool.cli import main, parse_args
+
+
+class TestParseArgs(unittest.TestCase):
+
+    def test_parse_args_minimal(self):
+        with patch('sys.argv', ['fit-tool', 'test.fit']):
+            args = parse_args()
+            self.assertEqual(args.fitfile, 'test.fit')
+            self.assertFalse(args.verbose)
+            self.assertIsNone(args.output)
+            self.assertIsNone(args.log)
+            self.assertIsNone(args.type)
+
+    def test_parse_args_with_options(self):
+        with patch('sys.argv', ['fit-tool', 'test.fit', '-v', '-o', 'out.csv', '-l', 'log.txt', '-t', 'csv']):
+            args = parse_args()
+            self.assertEqual(args.fitfile, 'test.fit')
+            self.assertTrue(args.verbose)
+            self.assertEqual(args.output, 'out.csv')
+            self.assertEqual(args.log, 'log.txt')
+            self.assertEqual(args.type, 'csv')
+
+
+class TestMain(unittest.TestCase):
+
+    def setUp(self):
+        self.test_dir = tempfile.mkdtemp()
+        self.test_fit_file = os.path.join(
+            os.path.dirname(__file__),
+            'data',
+            'sdk',
+            'Activity.fit'
+        )
+
+    def tearDown(self):
+        for f in os.listdir(self.test_dir):
+            os.remove(os.path.join(self.test_dir, f))
+        os.rmdir(self.test_dir)
+
+    def test_main_convert_to_csv(self):
+        output_file = os.path.join(self.test_dir, 'output.csv')
+        with patch('sys.argv', ['fit-tool', self.test_fit_file, '-o', output_file]):
+            main()
+        self.assertTrue(os.path.exists(output_file))
+
+    def test_main_convert_to_fit(self):
+        output_file = os.path.join(self.test_dir, 'output.fit')
+        with patch('sys.argv', ['fit-tool', self.test_fit_file, '-o', output_file, '-t', 'fit']):
+            main()
+        self.assertTrue(os.path.exists(output_file))
+
+    def test_main_with_verbose(self):
+        output_file = os.path.join(self.test_dir, 'output.csv')
+        with patch('sys.argv', ['fit-tool', self.test_fit_file, '-v', '-o', output_file]):
+            main()
+        self.assertTrue(os.path.exists(output_file))
+
+    def test_main_with_log_file(self):
+        output_file = os.path.join(self.test_dir, 'output.csv')
+        log_file = os.path.join(self.test_dir, 'test.log')
+        with patch('sys.argv', ['fit-tool', self.test_fit_file, '-o', output_file, '-l', log_file]):
+            main()
+        self.assertTrue(os.path.exists(output_file))
+        self.assertTrue(os.path.exists(log_file))
+
+    def test_main_default_output_filename(self):
+        original_cwd = os.getcwd()
+        try:
+            os.chdir(self.test_dir)
+            with patch('sys.argv', ['fit-tool', self.test_fit_file]):
+                main()
+            self.assertTrue(os.path.exists('Activity.csv'))
+        finally:
+            os.chdir(original_cwd)
+
+    def test_main_infer_format_from_output(self):
+        output_file = os.path.join(self.test_dir, 'output.fit')
+        with patch('sys.argv', ['fit-tool', self.test_fit_file, '-o', output_file]):
+            main()
+        self.assertTrue(os.path.exists(output_file))
+
+
+if __name__ == '__main__':
+    unittest.main()

pyproject.toml

@@ -40,7 +40,7 @@ Homepage = "https://github.com/shaonianche/python_fit_tool"
 Repository = "https://github.com/shaonianche/python_fit_tool.git"
 
 [project.scripts]
-fittool = "fit_tool.cli:main"
+fit-tool = "fit_tool.cli:main"
 
 [tool.setuptools.packages.find]
 include = ["fit_tool*"]