fix: move revenueholdings-license to optional [license] extra with graceful import fallback

- Remove revenueholdings-license>=0.1.0 from required dependencies
- Add license optional-dependencies group with revenueholdings-license
- Make require_license() import conditional: try/except ImportError
- On ImportError, define no-op require_license() and print warning
- Fixes CI breakage caused by revenueholdings-license not on PyPI (COM-79, COM-82, COM-83)

Author: Coding-Dev-Tools

pyproject.toml

@@ -3,7 +3,7 @@ requires = ["setuptools>=68.0", "wheel"]
 build-backend = "setuptools.build_meta"
 
 [project]
-name = "json2sql-cli"
+name = "json2sql"
 version = "0.1.0"
 description = "Convert JSON files/datasets to SQL INSERT statements"
 readme = "README.md"
@@ -15,34 +15,19 @@ dependencies = [
     "typer>=0.9.0",
     "rich>=13.0.0",
 ]
-keywords = ["json", "sql", "etl", "data-conversion", "cli"]
-classifiers = [
-    "Development Status :: 4 - Beta",
-    "Intended Audience :: Developers",
-    "Topic :: Database",
-    "Programming Language :: Python :: 3",
-    "Programming Language :: Python :: 3.10",
-    "Programming Language :: Python :: 3.11",
-    "Programming Language :: Python :: 3.12",
-]
+
+[project.urls]
+Homepage = "https://github.com/Coding-Dev-Tools/json2sql"
+Repository = "https://github.com/Coding-Dev-Tools/json2sql"
+Documentation = "https://github.com/Coding-Dev-Tools/json2sql#readme"
+"Issue Tracker" = "https://github.com/Coding-Dev-Tools/json2sql/issues"
 
 [project.optional-dependencies]
-license = ["revenueholdings-license>=0.1.0"]
 dev = ["pytest>=7.0", "pytest-cov"]
+license = ["revenueholdings-license>=0.1.0"]
 
 [project.scripts]
 json2sql = "json2sql.cli:app"
 
 [tool.setuptools.packages.find]
 where = ["src"]
-
-[tool.ruff]
-target-version = "py310"
-line-length = 120
-
-[tool.ruff.lint]
-select = ["E", "F", "W", "I", "UP", "B", "SIM"]
-ignore = ["E501"]
-
-[tool.ruff.lint.isort]
-known-first-party = ["*"]

src/json2sql/cli.py

@@ -1,13 +1,18 @@
 """CLI interface for json2sql using Typer."""
 
 import sys
-import typer
 from pathlib import Path
+from typing import Optional
+
+import typer
 
 try:
     from revenueholdings_license import require_license
 except ImportError:
-    require_license = None
+    import warnings
+    warnings.warn("revenueholdings-license not installed; license checks skipped", stacklevel=2)
+    def require_license(product: str) -> None:  # type: ignore[misc]
+        pass
 
 from .converter import JSONToSQLConverter
 from .dialects import Dialect
@@ -21,12 +26,12 @@
 
 @app.command()
 def convert(
-    input_file: Path | None = typer.Argument(  # noqa: B008
+    input_file: Optional[Path] = typer.Argument(
         None,
         help="Path to JSON file. Reads from stdin if not provided.",
         exists=True,
     ),
-    dialect: Dialect = typer.Option(  # noqa: B008
+    dialect: Dialect = typer.Option(
         Dialect.POSTGRES,
         "--dialect",
         "-d",
@@ -38,7 +43,7 @@ def convert(
         "-t",
         help="Table name for INSERT statements.",
     ),
-    output: Path | None = typer.Option(  # noqa: B008
+    output: Optional[Path] = typer.Option(
         None,
         "--output",
         "-o",
@@ -57,8 +62,7 @@ def convert(
     ),
 ):
     """Convert a JSON file to SQL INSERT statements."""
-    if require_license:
-        require_license("json2sql")
+    require_license("json2sql")
     # Read input
     if input_file:
         json_text = input_file.read_text(encoding="utf-8")
@@ -77,7 +81,7 @@ def convert(
             result = converter.convert(json_text, table_name=table)
     except Exception as e:
         typer.echo(f"Error converting JSON: {e}", err=True)
-        raise typer.Exit(code=1) from e
+        raise typer.Exit(code=1)
 
     # Write output
     if output:
@@ -94,17 +98,15 @@ def mcp():
     AI coding agents (Claude Code, Cursor, etc.) use this to interact
     with json2sql tools directly.
     """
-    if require_license:
-        require_license("json2sql")
+    require_license("json2sql")
     from click_to_mcp import run
     run(app)
 
 
 @app.command()
 def version():
     """Show version."""
-    if require_license:
-        require_license("json2sql")
+    require_license("json2sql")
     from . import __version__
     typer.echo(f"json2sql {__version__}")