easybib 0.2.0__tar.gz → 0.3.0__tar.gz

{easybib-0.2.0 → easybib-0.3.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: easybib
-Version: 0.2.0
+Version: 0.3.0
 Summary: Automatically fetch BibTeX entries from INSPIRE and ADS for LaTeX projects
 Author-email: Gregory Ashton <gregory.ashton@ligo.org>
 License-Expression: MIT
@@ -42,17 +42,18 @@ Pass a directory to scan all `.tex` files recursively, or a single `.tex` file.
 | Flag | Description |
 |------|-------------|
 | `-o`, `--output` | Output BibTeX file (default: `references.bib`) |
-| `-s`, `--source` | Preferred source: `ads` (default), `inspire`, or `auto` |
+| `-s`, `--preferred-source` | Preferred source: `ads` (default), `inspire`, or `auto` |
 | `-a`, `--max-authors` | Truncate author lists (default: 3, use 0 for no limit) |
 | `-l`, `--list-keys` | List found citation keys and exit (no fetching) |
 | `--fresh` | Ignore existing output file and start from scratch |
 | `--ads-api-key` | ADS API key (overrides `ADS_API_KEY` environment variable) |
+| `--config` | Path to config file (default: `~/.easybib.config`) |
 
 ### Examples
 
 ```bash
 # Scan a directory
-easybib ./paper -s inspire
+easybib ./paper --preferred-source inspire
 
 # Scan a single file
 easybib paper.tex
@@ -67,6 +68,34 @@ easybib ./paper -l
 easybib ./paper -a 0
 ```
 
+### Config file
+
+You can create a config file at `~/.easybib.config` to set persistent defaults, so you don't have to pass the same flags every time:
+
+```ini
+[easybib]
+output = references.bib
+max-authors = 3
+preferred-source = ads
+ads-api-key = your-key-here
+```
+
+All fields are optional. CLI flags override config file values, which override the built-in defaults.
+
+To use a config file at a different location:
+
+```bash
+easybib ./paper --config /path/to/my.config
+```
+
+### Source selection
+
+The `--preferred-source` flag controls where BibTeX entries are fetched from. The source determines which service provides the BibTeX data, regardless of the key format used in your `.tex` files.
+
+- **`ads`** (default) — Fetches BibTeX from ADS. If you use an INSPIRE-style key (e.g. `Author:2020abc`), easybib will cross-reference it via INSPIRE to find the corresponding ADS record, then pull the BibTeX from ADS. Falls back to INSPIRE if ADS lookup fails.
+- **`inspire`** — Fetches BibTeX from INSPIRE. Falls back to ADS if the INSPIRE lookup fails. Does not require an ADS API key unless the fallback is triggered.
+- **`auto`** — Chooses the source based on the key format: ADS bibcodes (e.g. `2016PhRvL.116f1102A`) are fetched from ADS, while INSPIRE-style keys are fetched from INSPIRE. Falls back to the other source if the preferred one fails.
+
 ### ADS API key
 
 When using ADS as the source (the default), provide your API key either via the command line:

{easybib-0.2.0 → easybib-0.3.0}/README.md

@@ -27,17 +27,18 @@ Pass a directory to scan all `.tex` files recursively, or a single `.tex` file.
 | Flag | Description |
 |------|-------------|
 | `-o`, `--output` | Output BibTeX file (default: `references.bib`) |
-| `-s`, `--source` | Preferred source: `ads` (default), `inspire`, or `auto` |
+| `-s`, `--preferred-source` | Preferred source: `ads` (default), `inspire`, or `auto` |
 | `-a`, `--max-authors` | Truncate author lists (default: 3, use 0 for no limit) |
 | `-l`, `--list-keys` | List found citation keys and exit (no fetching) |
 | `--fresh` | Ignore existing output file and start from scratch |
 | `--ads-api-key` | ADS API key (overrides `ADS_API_KEY` environment variable) |
+| `--config` | Path to config file (default: `~/.easybib.config`) |
 
 ### Examples
 
 ```bash
 # Scan a directory
-easybib ./paper -s inspire
+easybib ./paper --preferred-source inspire
 
 # Scan a single file
 easybib paper.tex
@@ -52,6 +53,34 @@ easybib ./paper -l
 easybib ./paper -a 0
 ```
 
+### Config file
+
+You can create a config file at `~/.easybib.config` to set persistent defaults, so you don't have to pass the same flags every time:
+
+```ini
+[easybib]
+output = references.bib
+max-authors = 3
+preferred-source = ads
+ads-api-key = your-key-here
+```
+
+All fields are optional. CLI flags override config file values, which override the built-in defaults.
+
+To use a config file at a different location:
+
+```bash
+easybib ./paper --config /path/to/my.config
+```
+
+### Source selection
+
+The `--preferred-source` flag controls where BibTeX entries are fetched from. The source determines which service provides the BibTeX data, regardless of the key format used in your `.tex` files.
+
+- **`ads`** (default) — Fetches BibTeX from ADS. If you use an INSPIRE-style key (e.g. `Author:2020abc`), easybib will cross-reference it via INSPIRE to find the corresponding ADS record, then pull the BibTeX from ADS. Falls back to INSPIRE if ADS lookup fails.
+- **`inspire`** — Fetches BibTeX from INSPIRE. Falls back to ADS if the INSPIRE lookup fails. Does not require an ADS API key unless the fallback is triggered.
+- **`auto`** — Chooses the source based on the key format: ADS bibcodes (e.g. `2016PhRvL.116f1102A`) are fetched from ADS, while INSPIRE-style keys are fetched from INSPIRE. Falls back to the other source if the preferred one fails.
+
 ### ADS API key
 
 When using ADS as the source (the default), provide your API key either via the command line:

{easybib-0.2.0 → easybib-0.3.0}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "easybib"
-version = "0.2.0"
+version = "0.3.0"
 description = "Automatically fetch BibTeX entries from INSPIRE and ADS for LaTeX projects"
 readme = "README.md"
 license = "MIT"

{easybib-0.2.0 → easybib-0.3.0}/src/easybib/cli.py

@@ -1,9 +1,11 @@
 """Command-line interface for easybib."""
 
+import configparser
 import os
 import argparse
 from pathlib import Path
 
+from easybib import __version__
 from easybib.core import (
     extract_cite_keys,
     extract_existing_bib_keys,
@@ -13,11 +15,52 @@ from easybib.core import (
 )
 
 
+def load_config(config_path):
+    """Read an INI config file and return a dict from the [easybib] section."""
+    path = Path(config_path).expanduser()
+    if not path.is_file():
+        return {}
+    config = configparser.ConfigParser()
+    config.read(path)
+    if "easybib" not in config:
+        return {}
+    return dict(config["easybib"])
+
+
 def main():
+    # First pass: extract --config so we know which config file to load
+    pre_parser = argparse.ArgumentParser(add_help=False)
+    pre_parser.add_argument(
+        "--config",
+        default="~/.easybib.config",
+        help="Path to config file (default: ~/.easybib.config)",
+    )
+    pre_args, _ = pre_parser.parse_known_args()
+
+    # Load config and build defaults
+    cfg = load_config(pre_args.config)
+    config_defaults = {}
+    if "output" in cfg:
+        config_defaults["output"] = cfg["output"]
+    if "max-authors" in cfg:
+        config_defaults["max_authors"] = int(cfg["max-authors"])
+    if "preferred-source" in cfg:
+        config_defaults["preferred_source"] = cfg["preferred-source"]
+    if "ads-api-key" in cfg:
+        config_defaults["ads_api_key"] = cfg["ads-api-key"]
+
     parser = argparse.ArgumentParser(
         description="Extract citations and download BibTeX from NASA/ADS"
     )
+    parser.add_argument(
+        "--version", action="version", version=f"%(prog)s {__version__}"
+    )
     parser.add_argument("path", help="LaTeX file or directory containing LaTeX files")
+    parser.add_argument(
+        "--config",
+        default="~/.easybib.config",
+        help="Path to config file (default: ~/.easybib.config)",
+    )
     parser.add_argument(
         "-o", "--output", default="references.bib", help="Output BibTeX file (existing entries are retained)"
     )
@@ -41,7 +84,7 @@ def main():
     )
     parser.add_argument(
         "-s",
-        "--source",
+        "--preferred-source",
         choices=["ads", "inspire", "auto"],
         default="ads",
         help="Preferred BibTeX source: 'ads' (default), 'inspire', or 'auto' (based on key format)",
@@ -50,6 +93,11 @@ def main():
         "--ads-api-key",
         help="ADS API key (overrides ADS_API_KEY environment variable)",
     )
+
+    # Apply config file defaults (CLI flags will still override)
+    if config_defaults:
+        parser.set_defaults(**config_defaults)
+
     args = parser.parse_args()
 
     # Collect all citation keys
@@ -80,12 +128,12 @@ def main():
             print(key)
         return 0
 
-    # Check for ADS API key (not required if using --source inspire)
+    # Check for ADS API key (not required if using --preferred-source inspire)
     api_key = args.ads_api_key or os.getenv("ADS_API_KEY")
-    if not api_key and args.source != "inspire":
+    if not api_key and args.preferred_source != "inspire":
         print("Error: ADS_API_KEY environment variable not set")
         print("Get your API key from: https://ui.adsabs.harvard.edu/user/settings/token")
-        print("(Or use --source inspire to fetch from INSPIRE without an ADS key)")
+        print("(Or use --preferred-source inspire to fetch from INSPIRE without an ADS key)")
         return 1
 
     # Check for existing bib file and determine which keys to fetch
@@ -108,7 +156,7 @@ def main():
     not_found = []
     for key in sorted(keys_to_fetch):
         print(f"Fetching {key}...", end=" ")
-        bibtex, source = fetch_bibtex(key, api_key, args.source)
+        bibtex, source = fetch_bibtex(key, api_key, args.preferred_source)
         if bibtex:
             bibtex = replace_bibtex_key(bibtex, key)
             bibtex = truncate_authors(bibtex, args.max_authors)

{easybib-0.2.0 → easybib-0.3.0}/src/easybib.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: easybib
-Version: 0.2.0
+Version: 0.3.0
 Summary: Automatically fetch BibTeX entries from INSPIRE and ADS for LaTeX projects
 Author-email: Gregory Ashton <gregory.ashton@ligo.org>
 License-Expression: MIT
@@ -42,17 +42,18 @@ Pass a directory to scan all `.tex` files recursively, or a single `.tex` file.
 | Flag | Description |
 |------|-------------|
 | `-o`, `--output` | Output BibTeX file (default: `references.bib`) |
-| `-s`, `--source` | Preferred source: `ads` (default), `inspire`, or `auto` |
+| `-s`, `--preferred-source` | Preferred source: `ads` (default), `inspire`, or `auto` |
 | `-a`, `--max-authors` | Truncate author lists (default: 3, use 0 for no limit) |
 | `-l`, `--list-keys` | List found citation keys and exit (no fetching) |
 | `--fresh` | Ignore existing output file and start from scratch |
 | `--ads-api-key` | ADS API key (overrides `ADS_API_KEY` environment variable) |
+| `--config` | Path to config file (default: `~/.easybib.config`) |
 
 ### Examples
 
 ```bash
 # Scan a directory
-easybib ./paper -s inspire
+easybib ./paper --preferred-source inspire
 
 # Scan a single file
 easybib paper.tex
@@ -67,6 +68,34 @@ easybib ./paper -l
 easybib ./paper -a 0
 ```
 
+### Config file
+
+You can create a config file at `~/.easybib.config` to set persistent defaults, so you don't have to pass the same flags every time:
+
+```ini
+[easybib]
+output = references.bib
+max-authors = 3
+preferred-source = ads
+ads-api-key = your-key-here
+```
+
+All fields are optional. CLI flags override config file values, which override the built-in defaults.
+
+To use a config file at a different location:
+
+```bash
+easybib ./paper --config /path/to/my.config
+```
+
+### Source selection
+
+The `--preferred-source` flag controls where BibTeX entries are fetched from. The source determines which service provides the BibTeX data, regardless of the key format used in your `.tex` files.
+
+- **`ads`** (default) — Fetches BibTeX from ADS. If you use an INSPIRE-style key (e.g. `Author:2020abc`), easybib will cross-reference it via INSPIRE to find the corresponding ADS record, then pull the BibTeX from ADS. Falls back to INSPIRE if ADS lookup fails.
+- **`inspire`** — Fetches BibTeX from INSPIRE. Falls back to ADS if the INSPIRE lookup fails. Does not require an ADS API key unless the fallback is triggered.
+- **`auto`** — Chooses the source based on the key format: ADS bibcodes (e.g. `2016PhRvL.116f1102A`) are fetched from ADS, while INSPIRE-style keys are fetched from INSPIRE. Falls back to the other source if the preferred one fails.
+
 ### ADS API key
 
 When using ADS as the source (the default), provide your API key either via the command line:

easybib-0.3.0/tests/test_cli.py

@@ -0,0 +1,274 @@
+"""Tests for easybib CLI."""
+
+import subprocess
+import sys
+from unittest.mock import patch
+
+from easybib.cli import main
+
+
+class TestListKeys:
+    def test_list_keys_single_file(self, tmp_path, capsys):
+        tex = tmp_path / "test.tex"
+        tex.write_text(r"\cite{Author:2020abc} and \citep{Other:2021xyz}")
+        with patch("sys.argv", ["easybib", str(tex), "--list-keys"]):
+            result = main()
+        assert result == 0
+        captured = capsys.readouterr()
+        assert "Author:2020abc" in captured.out
+        assert "Other:2021xyz" in captured.out
+
+    def test_list_keys_directory(self, tmp_path, capsys):
+        sub = tmp_path / "subdir"
+        sub.mkdir()
+        (tmp_path / "a.tex").write_text(r"\cite{A:2020abc}")
+        (sub / "b.tex").write_text(r"\cite{B:2021xyz}")
+        with patch("sys.argv", ["easybib", str(tmp_path), "--list-keys"]):
+            result = main()
+        assert result == 0
+        captured = capsys.readouterr()
+        assert "A:2020abc" in captured.out
+        assert "B:2021xyz" in captured.out
+
+
+class TestMissingApiKey:
+    def test_error_without_api_key(self, tmp_path, capsys):
+        tex = tmp_path / "test.tex"
+        tex.write_text(r"\cite{Author:2020abc}")
+        no_config = str(tmp_path / "nonexistent.config")
+        with (
+            patch("sys.argv", ["easybib", str(tex), "--config", no_config]),
+            patch.dict("os.environ", {}, clear=True),
+        ):
+            result = main()
+        assert result == 1
+        captured = capsys.readouterr()
+        assert "ADS_API_KEY" in captured.out
+
+    def test_inspire_source_no_api_key_ok(self, tmp_path, capsys):
+        """Using --preferred-source inspire should not require an ADS API key."""
+        tex = tmp_path / "test.tex"
+        tex.write_text(r"\cite{Author:2020abc}")
+        no_config = str(tmp_path / "nonexistent.config")
+        with (
+            patch("sys.argv", ["easybib", str(tex), "--preferred-source", "inspire", "--config", no_config]),
+            patch.dict("os.environ", {}, clear=True),
+            patch("easybib.cli.fetch_bibtex", return_value=(None, None)),
+        ):
+            result = main()
+        # Should not return 1 for missing API key
+        assert result is None
+
+
+class TestAdsApiKeyOverride:
+    def test_flag_overrides_env(self, tmp_path, capsys):
+        tex = tmp_path / "test.tex"
+        tex.write_text(r"\cite{Author:2020abc}")
+        with (
+            patch(
+                "sys.argv",
+                ["easybib", str(tex), "--ads-api-key", "flag-key", "-o", str(tmp_path / "out.bib")],
+            ),
+            patch.dict("os.environ", {"ADS_API_KEY": "env-key"}, clear=True),
+            patch("easybib.cli.fetch_bibtex") as mock_fetch,
+        ):
+            mock_fetch.return_value = (
+                "@article{Author:2020abc,\n  title={Test},\n  author={Doe, J.},\n}",
+                "ADS",
+            )
+            main()
+        # The flag value should be used, not the env var
+        call_args = mock_fetch.call_args
+        assert call_args[0][1] == "flag-key"
+
+
+class TestConfigFile:
+    def test_config_sets_defaults(self, tmp_path, capsys):
+        """Config file values are used when no CLI flags are given."""
+        tex = tmp_path / "test.tex"
+        tex.write_text(r"\cite{Author:2020abc}")
+        cfg = tmp_path / "test.config"
+        cfg.write_text(
+            "[easybib]\noutput = custom.bib\nmax-authors = 5\npreferred-source = inspire\nads-api-key = cfg-key\n"
+        )
+        with (
+            patch(
+                "sys.argv",
+                ["easybib", str(tex), "--config", str(cfg), "--list-keys"],
+            ),
+        ):
+            main()
+        captured = capsys.readouterr()
+        assert "Author:2020abc" in captured.out
+
+    def test_config_values_applied(self, tmp_path):
+        """Config file values feed into parsed args."""
+        tex = tmp_path / "test.tex"
+        tex.write_text(r"\cite{Author:2020abc}")
+        cfg = tmp_path / "test.config"
+        cfg.write_text(
+            "[easybib]\noutput = custom.bib\nmax-authors = 5\npreferred-source = inspire\nads-api-key = cfg-key\n"
+        )
+        with (
+            patch(
+                "sys.argv",
+                ["easybib", str(tex), "--config", str(cfg), "--list-keys"],
+            ),
+            patch("easybib.cli.extract_cite_keys", return_value=(set(), [])),
+        ):
+            # Access the parsed args by patching parse_args
+            import easybib.cli as cli_mod
+
+            original_parse = cli_mod.argparse.ArgumentParser.parse_args
+
+            captured_args = {}
+
+            def spy_parse(self_parser, *a, **kw):
+                result = original_parse(self_parser, *a, **kw)
+                captured_args.update(vars(result))
+                return result
+
+            with patch.object(
+                cli_mod.argparse.ArgumentParser, "parse_args", spy_parse
+            ):
+                main()
+
+            assert captured_args["output"] == "custom.bib"
+            assert captured_args["max_authors"] == 5
+            assert captured_args["preferred_source"] == "inspire"
+            assert captured_args["ads_api_key"] == "cfg-key"
+
+    def test_cli_flags_override_config(self, tmp_path):
+        """CLI flags take priority over config file values."""
+        tex = tmp_path / "test.tex"
+        tex.write_text(r"\cite{Author:2020abc}")
+        cfg = tmp_path / "test.config"
+        cfg.write_text(
+            "[easybib]\noutput = config.bib\nmax-authors = 5\npreferred-source = inspire\nads-api-key = cfg-key\n"
+        )
+        with (
+            patch(
+                "sys.argv",
+                [
+                    "easybib",
+                    str(tex),
+                    "--config",
+                    str(cfg),
+                    "--list-keys",
+                    "-o",
+                    "cli.bib",
+                    "--max-authors",
+                    "10",
+                    "--preferred-source",
+                    "ads",
+                    "--ads-api-key",
+                    "cli-key",
+                ],
+            ),
+            patch("easybib.cli.extract_cite_keys", return_value=(set(), [])),
+        ):
+            import easybib.cli as cli_mod
+
+            original_parse = cli_mod.argparse.ArgumentParser.parse_args
+            captured_args = {}
+
+            def spy_parse(self_parser, *a, **kw):
+                result = original_parse(self_parser, *a, **kw)
+                captured_args.update(vars(result))
+                return result
+
+            with patch.object(
+                cli_mod.argparse.ArgumentParser, "parse_args", spy_parse
+            ):
+                main()
+
+            assert captured_args["output"] == "cli.bib"
+            assert captured_args["max_authors"] == 10
+            assert captured_args["preferred_source"] == "ads"
+            assert captured_args["ads_api_key"] == "cli-key"
+
+    def test_missing_config_silently_ignored(self, tmp_path, capsys):
+        """A nonexistent config file does not cause an error."""
+        tex = tmp_path / "test.tex"
+        tex.write_text(r"\cite{Author:2020abc}")
+        with patch(
+            "sys.argv",
+            [
+                "easybib",
+                str(tex),
+                "--config",
+                str(tmp_path / "nonexistent.config"),
+                "--list-keys",
+            ],
+        ):
+            result = main()
+        assert result == 0
+        captured = capsys.readouterr()
+        assert "Author:2020abc" in captured.out
+
+    def test_custom_config_path(self, tmp_path, capsys):
+        """--config flag points to a custom path."""
+        tex = tmp_path / "test.tex"
+        tex.write_text(r"\cite{Author:2020abc}")
+        custom_dir = tmp_path / "custom"
+        custom_dir.mkdir()
+        cfg = custom_dir / "my.config"
+        cfg.write_text("[easybib]\npreferred-source = inspire\n")
+        with (
+            patch(
+                "sys.argv",
+                ["easybib", str(tex), "--config", str(cfg), "--list-keys"],
+            ),
+        ):
+            result = main()
+        assert result == 0
+
+    def test_config_ads_api_key_used_for_lookup(self, tmp_path):
+        """ads-api-key from config feeds into the API key chain."""
+        tex = tmp_path / "test.tex"
+        tex.write_text(r"\cite{Author:2020abc}")
+        cfg = tmp_path / "test.config"
+        cfg.write_text("[easybib]\nads-api-key = config-api-key\n")
+        with (
+            patch(
+                "sys.argv",
+                [
+                    "easybib",
+                    str(tex),
+                    "--config",
+                    str(cfg),
+                    "-o",
+                    str(tmp_path / "out.bib"),
+                ],
+            ),
+            patch.dict("os.environ", {}, clear=True),
+            patch("easybib.cli.fetch_bibtex") as mock_fetch,
+        ):
+            mock_fetch.return_value = (
+                "@article{Author:2020abc,\n  title={Test},\n  author={Doe, J.},\n}",
+                "ADS",
+            )
+            main()
+        call_args = mock_fetch.call_args
+        assert call_args[0][1] == "config-api-key"
+
+
+class TestFileVsDirectory:
+    def test_single_file(self, tmp_path, capsys):
+        tex = tmp_path / "paper.tex"
+        tex.write_text(r"\cite{A:2020abc}")
+        with patch("sys.argv", ["easybib", str(tex), "--list-keys"]):
+            main()
+        captured = capsys.readouterr()
+        assert "A:2020abc" in captured.out
+
+    def test_directory_recursive(self, tmp_path, capsys):
+        nested = tmp_path / "ch1"
+        nested.mkdir()
+        (nested / "intro.tex").write_text(r"\cite{A:2020abc}")
+        (tmp_path / "main.tex").write_text(r"\cite{B:2021xyz}")
+        with patch("sys.argv", ["easybib", str(tmp_path), "--list-keys"]):
+            main()
+        captured = capsys.readouterr()
+        assert "A:2020abc" in captured.out
+        assert "B:2021xyz" in captured.out

easybib-0.2.0/tests/test_cli.py

@@ -1,101 +0,0 @@
-"""Tests for easybib CLI."""
-
-import subprocess
-import sys
-from unittest.mock import patch
-
-from easybib.cli import main
-
-
-class TestListKeys:
-    def test_list_keys_single_file(self, tmp_path, capsys):
-        tex = tmp_path / "test.tex"
-        tex.write_text(r"\cite{Author:2020abc} and \citep{Other:2021xyz}")
-        with patch("sys.argv", ["easybib", str(tex), "--list-keys"]):
-            result = main()
-        assert result == 0
-        captured = capsys.readouterr()
-        assert "Author:2020abc" in captured.out
-        assert "Other:2021xyz" in captured.out
-
-    def test_list_keys_directory(self, tmp_path, capsys):
-        sub = tmp_path / "subdir"
-        sub.mkdir()
-        (tmp_path / "a.tex").write_text(r"\cite{A:2020abc}")
-        (sub / "b.tex").write_text(r"\cite{B:2021xyz}")
-        with patch("sys.argv", ["easybib", str(tmp_path), "--list-keys"]):
-            result = main()
-        assert result == 0
-        captured = capsys.readouterr()
-        assert "A:2020abc" in captured.out
-        assert "B:2021xyz" in captured.out
-
-
-class TestMissingApiKey:
-    def test_error_without_api_key(self, tmp_path, capsys):
-        tex = tmp_path / "test.tex"
-        tex.write_text(r"\cite{Author:2020abc}")
-        with (
-            patch("sys.argv", ["easybib", str(tex)]),
-            patch.dict("os.environ", {}, clear=True),
-        ):
-            result = main()
-        assert result == 1
-        captured = capsys.readouterr()
-        assert "ADS_API_KEY" in captured.out
-
-    def test_inspire_source_no_api_key_ok(self, tmp_path, capsys):
-        """Using --source inspire should not require an ADS API key."""
-        tex = tmp_path / "test.tex"
-        tex.write_text(r"\cite{Author:2020abc}")
-        with (
-            patch("sys.argv", ["easybib", str(tex), "--source", "inspire"]),
-            patch.dict("os.environ", {}, clear=True),
-            patch("easybib.cli.fetch_bibtex", return_value=(None, None)),
-        ):
-            result = main()
-        # Should not return 1 for missing API key
-        assert result is None
-
-
-class TestAdsApiKeyOverride:
-    def test_flag_overrides_env(self, tmp_path, capsys):
-        tex = tmp_path / "test.tex"
-        tex.write_text(r"\cite{Author:2020abc}")
-        with (
-            patch(
-                "sys.argv",
-                ["easybib", str(tex), "--ads-api-key", "flag-key"],
-            ),
-            patch.dict("os.environ", {"ADS_API_KEY": "env-key"}, clear=True),
-            patch("easybib.cli.fetch_bibtex") as mock_fetch,
-        ):
-            mock_fetch.return_value = (
-                "@article{Author:2020abc,\n  title={Test},\n  author={Doe, J.},\n}",
-                "ADS",
-            )
-            main()
-        # The flag value should be used, not the env var
-        call_args = mock_fetch.call_args
-        assert call_args[0][1] == "flag-key"
-
-
-class TestFileVsDirectory:
-    def test_single_file(self, tmp_path, capsys):
-        tex = tmp_path / "paper.tex"
-        tex.write_text(r"\cite{A:2020abc}")
-        with patch("sys.argv", ["easybib", str(tex), "--list-keys"]):
-            main()
-        captured = capsys.readouterr()
-        assert "A:2020abc" in captured.out
-
-    def test_directory_recursive(self, tmp_path, capsys):
-        nested = tmp_path / "ch1"
-        nested.mkdir()
-        (nested / "intro.tex").write_text(r"\cite{A:2020abc}")
-        (tmp_path / "main.tex").write_text(r"\cite{B:2021xyz}")
-        with patch("sys.argv", ["easybib", str(tmp_path), "--list-keys"]):
-            main()
-        captured = capsys.readouterr()
-        assert "A:2020abc" in captured.out
-        assert "B:2021xyz" in captured.out