change collect bounds flags and add a delete option

Author: ramSeraph

README.md

@@ -174,12 +174,13 @@ This project provides several command-line tools to work with topographic maps.
 Collects individual GeoJSONL bound files into a single GeoJSON FeatureCollection.
 
 ```bash
-collect-bounds --bounds-dir <directory> --output-file <file> [--preexisting-file <file>]
+collect-bounds -b <directory> -o <file> [-u] [-d <id1> <id2> ...]
 ```
 
--   `--bounds-dir`: Directory containing GeoJSONL files. (Required)
--   `--output-file`: Output GeoJSON file path. (Required)
--   `--preexisting-file`: Pre existing GeoJSON file path. (Optional)
+-   `-b`, `--bounds-dir`: Directory containing GeoJSONL files. (Required)
+-   `-o`, `--output-file`: Output GeoJSON file path. (Required)
+-   `-u`, `--update`: Update the output file in place. (Optional)
+-   `-d`, `--delete`: List of ids to delete from the output file (only available in update mode). (Optional)
 
 #### `retile`
 

topo_map_processor/tools/collect_bounds.py

@@ -5,23 +5,32 @@
 
 def cli():
     parser = argparse.ArgumentParser(description='Collect GeoJSONL files into a single GeoJSON file.')
-    parser.add_argument('--bounds-dir', required=True, help='Directory containing GeoJSONL files')
-    parser.add_argument('--output-file', required=True, help='Output GeoJSON file path')
-    parser.add_argument('--preexisting-file', help='preexisting GeoJSON file to base things on', default=None)
+    parser.add_argument('-b', '--bounds-dir', required=True, help='Directory containing GeoJSONL files')
+    parser.add_argument('-o', '--output-file', required=True, help='Output GeoJSON file path')
+    parser.add_argument('-u', '--update', action='store_true', help='Update the output file in place')
+    parser.add_argument('-d', '--delete', nargs='+', help='List of ids to delete from the output file (only in update mode)')
 
     args = parser.parse_args()
 
+    if args.delete and not args.update:
+        raise parser.error("--delete can only be used with --update")
+
     feat_map = {}
-    if args.preexisting_file:
-        preexisting_file = Path(args.preexisting_file)
-        if not preexisting_file.exists():
-            raise FileNotFoundError(f"Preexisting file does not exist: {preexisting_file}")
+    if args.update:
+        output_file = Path(args.output_file)
+        if not output_file.exists():
+            raise FileNotFoundError(f"Output file does not exist for updating: {output_file}")
 
-        existing_data = json.loads(preexisting_file.read_text())
+        existing_data = json.loads(output_file.read_text())
         for feature in existing_data.get('features', []):
             if 'properties' in feature and 'id' in feature['properties']:
                 sheet_name = feature['properties']['id']
                 feat_map[sheet_name] = feature
+    
+    if args.delete:
+        for sheet_id in args.delete:
+            if sheet_id in feat_map:
+                del feat_map[sheet_id]
 
     bounds_dir = Path(args.bounds_dir)