Import commands now require existing db (use farchive init first)

eliask · eliask · commit 5dd0e4daaa9f · 2026-04-05T13:13:42.000+03:00
- import-files and import-manifest no longer auto-create db
- Update tests to use farchive init before import commands
- Consistent behavior: all write commands require init first
diff --git a/src/farchive/_cli.py b/src/farchive/_cli.py
@@ -966,9 +966,7 @@ def _cmd_observe(args: argparse.Namespace) -> None:
 
 def _cmd_import_files(args: argparse.Namespace) -> None:
     """Import files from a directory into the archive."""
-    new_db = _maybe_create_db(args)
-    if new_db:
-        print("Created new archive.", file=sys.stderr)
+    _ensure_db(args)
     root = Path(args.root).resolve()
     if not root.is_dir():
         print(f"Not a directory: {args.root}", file=sys.stderr)
@@ -1075,9 +1073,7 @@ def _cmd_import_files(args: argparse.Namespace) -> None:
 
 def _cmd_import_manifest(args: argparse.Namespace) -> None:
     """Import from a manifest file (JSONL or TSV)."""
-    new_db = _maybe_create_db(args)
-    if new_db:
-        print("Created new archive.", file=sys.stderr)
+    _ensure_db(args)
     manifest_path = Path(args.manifest)
     if not manifest_path.is_file():
         print(f"Manifest not found: {args.manifest}", file=sys.stderr)
diff --git a/tests/test_cli_import.py b/tests/test_cli_import.py
@@ -130,6 +130,9 @@ class TestImportFiles:
 
     def test_import_files_basic(self, tmp_path):
         db = tmp_path / "test.db"
+        subprocess.run(
+            [sys.executable, "-m", "farchive._cli", "init", str(db)], check=True
+        )
         root = tmp_path / "files"
         root.mkdir()
         (root / "a.txt").write_bytes(b"content a")
@@ -152,6 +155,9 @@ def test_import_files_basic(self, tmp_path):
 
     def test_import_files_dry_run(self, tmp_path):
         db = tmp_path / "test.db"
+        subprocess.run(
+            [sys.executable, "-m", "farchive._cli", "init", str(db)], check=True
+        )
         root = tmp_path / "files"
         root.mkdir()
         (root / "a.txt").write_bytes(b"content")
@@ -172,6 +178,9 @@ def test_import_files_dry_run(self, tmp_path):
 
     def test_import_files_recursive(self, tmp_path):
         db = tmp_path / "test.db"
+        subprocess.run(
+            [sys.executable, "-m", "farchive._cli", "init", str(db)], check=True
+        )
         root = tmp_path / "files"
         root.mkdir()
         (root / "sub").mkdir()
@@ -194,6 +203,9 @@ def test_import_files_recursive(self, tmp_path):
 
     def test_import_files_storage_class_by_ext(self, tmp_path):
         db = tmp_path / "test.db"
+        subprocess.run(
+            [sys.executable, "-m", "farchive._cli", "init", str(db)], check=True
+        )
         root = tmp_path / "files"
         root.mkdir()
         (root / "page.html").write_bytes(b"<html></html>")
@@ -230,6 +242,9 @@ class TestImportManifest:
 
     def test_import_manifest_jsonl(self, tmp_path):
         db = tmp_path / "test.db"
+        subprocess.run(
+            [sys.executable, "-m", "farchive._cli", "init", str(db)], check=True
+        )
         root = tmp_path / "files"
         root.mkdir()
         (root / "a.txt").write_bytes(b"content a")
@@ -252,6 +267,9 @@ def test_import_manifest_jsonl(self, tmp_path):
 
     def test_import_manifest_dry_run(self, tmp_path):
         db = tmp_path / "test.db"
+        subprocess.run(
+            [sys.executable, "-m", "farchive._cli", "init", str(db)], check=True
+        )
         root = tmp_path / "files"
         root.mkdir()
         (root / "a.txt").write_bytes(b"content")
