Fix pyproject.toml section ordering and test compatibility

- Move [project.urls] after dependencies to fix TOML parsing error
- Remove License classifier (superseded by license expression per PEP 639)
- Fix crawler tests to patch crawl4ai.AsyncWebCrawler (lazy import)
- Fix MCP server tests to use FunctionTool.fn (FastMCP 2.x API)

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: hanselhansel

pyproject.toml

@@ -17,7 +17,6 @@ classifiers = [
     "Development Status :: 4 - Beta",
     "Environment :: Console",
     "Intended Audience :: Developers",
-    "License :: OSI Approved :: MIT License",
     "Programming Language :: Python :: 3",
     "Programming Language :: Python :: 3.10",
     "Programming Language :: Python :: 3.11",
@@ -27,10 +26,6 @@ classifiers = [
     "Typing :: Typed",
 ]
 
-[project.urls]
-Homepage = "https://github.com/hanselhansel/aeo-cli"
-Repository = "https://github.com/hanselhansel/aeo-cli"
-Issues = "https://github.com/hanselhansel/aeo-cli/issues"
 dependencies = [
     "typer>=0.9",
     "rich>=13.0",
@@ -41,6 +36,11 @@ dependencies = [
     "fastmcp>=2.0",
 ]
 
+[project.urls]
+Homepage = "https://github.com/hanselhansel/aeo-cli"
+Repository = "https://github.com/hanselhansel/aeo-cli"
+Issues = "https://github.com/hanselhansel/aeo-cli/issues"
+
 [project.optional-dependencies]
 dev = [
     "pytest>=8.0",

tests/test_crawler_errors.py

@@ -13,21 +13,16 @@
 async def test_crawl4ai_import_error():
     """If crawl4ai is not installed, extract_page should return a failed CrawlResult."""
     with patch.dict("sys.modules", {"crawl4ai": None}):
-        with patch("aeo_cli.core.crawler.AsyncWebCrawler", side_effect=ImportError("No module")):
-            pass
-    # Simulate import failure by patching the import inside extract_page
-    with patch("builtins.__import__", side_effect=ImportError("No module named 'crawl4ai'")):
         result = await extract_page("https://example.com")
 
     assert result.success is False
     assert result.error is not None
-    assert "crawl4ai" in result.error.lower() or "module" in result.error.lower()
 
 
 @pytest.mark.asyncio
 async def test_crawler_generic_exception():
     """A generic exception during crawl should be captured, not raised."""
-    with patch("aeo_cli.core.crawler.AsyncWebCrawler") as mock_cls:
+    with patch("crawl4ai.AsyncWebCrawler") as mock_cls:
         mock_crawler = AsyncMock()
         mock_crawler.arun.side_effect = RuntimeError("Browser crashed")
         mock_cls.return_value.__aenter__ = AsyncMock(return_value=mock_crawler)
@@ -48,7 +43,7 @@ async def test_crawler_returns_failed_result():
     mock_result.markdown = ""
     mock_result.links = {}
 
-    with patch("aeo_cli.core.crawler.AsyncWebCrawler") as mock_cls:
+    with patch("crawl4ai.AsyncWebCrawler") as mock_cls:
         mock_crawler = AsyncMock()
         mock_crawler.arun.return_value = mock_result
         mock_cls.return_value.__aenter__ = AsyncMock(return_value=mock_crawler)
@@ -71,7 +66,7 @@ async def test_crawler_successful_extraction():
         "external": [{"href": "https://other.com"}],
     }
 
-    with patch("aeo_cli.core.crawler.AsyncWebCrawler") as mock_cls:
+    with patch("crawl4ai.AsyncWebCrawler") as mock_cls:
         mock_crawler = AsyncMock()
         mock_crawler.arun.return_value = mock_result
         mock_cls.return_value.__aenter__ = AsyncMock(return_value=mock_crawler)

tests/test_mcp_server.py

@@ -9,14 +9,18 @@
 from aeo_cli.core.models import (
     AuditReport,
     ContentReport,
+    DiscoveryResult,
     LlmsTxtReport,
     RobotsReport,
     SchemaReport,
     SiteAuditReport,
-    DiscoveryResult,
 )
 from aeo_cli.server import audit
 
+# FastMCP 2.x wraps @mcp.tool functions in a FunctionTool object.
+# The underlying async function is accessible via .fn
+_audit_fn = audit.fn
+
 
 def _mock_single_report() -> AuditReport:
     return AuditReport(
@@ -49,7 +53,7 @@ async def test_audit_tool_single_page():
     with patch("aeo_cli.server.audit_url", new_callable=AsyncMock) as mock_audit:
         mock_audit.return_value = _mock_single_report()
 
-        result = await audit("https://example.com", single_page=True)
+        result = await _audit_fn("https://example.com", single_page=True)
 
         mock_audit.assert_called_once_with("https://example.com")
         assert result["url"] == "https://example.com"
@@ -64,7 +68,7 @@ async def test_audit_tool_site_audit():
     with patch("aeo_cli.server.audit_site", new_callable=AsyncMock) as mock_audit:
         mock_audit.return_value = _mock_site_report()
 
-        result = await audit("https://example.com")
+        result = await _audit_fn("https://example.com")
 
         mock_audit.assert_called_once_with("https://example.com", max_pages=10)
         assert result["domain"] == "example.com"
@@ -78,7 +82,7 @@ async def test_audit_tool_custom_max_pages():
     with patch("aeo_cli.server.audit_site", new_callable=AsyncMock) as mock_audit:
         mock_audit.return_value = _mock_site_report()
 
-        await audit("https://example.com", max_pages=5)
+        await _audit_fn("https://example.com", max_pages=5)
 
         mock_audit.assert_called_once_with("https://example.com", max_pages=5)
 
@@ -89,6 +93,6 @@ async def test_audit_tool_returns_dict():
     with patch("aeo_cli.server.audit_url", new_callable=AsyncMock) as mock_audit:
         mock_audit.return_value = _mock_single_report()
 
-        result = await audit("https://example.com", single_page=True)
+        result = await _audit_fn("https://example.com", single_page=True)
 
         assert isinstance(result, dict)