refactor: schema-driven prompting via Pydantic JSON Schema

Refactors prompting to be JSON-Schema-first. prompt_for_schema() now uses Pydantic model_json_schema() and tool schemas and Pydantic models go through the same prompting path. Removes the custom x-* hint fields and replaces them with schema-based prompting heuristics (including array-of-strings CSV input and optional object include gating). Fixes default rendering to distinguish “default key present” vs absent and renders defaults via json.dumps.

Adds validate_schema() for early JSON Schema validation, updates tools execute to fail fast on empty IDs and to accept string-encoded schemas when they decode to an object, and restores recursive dict key-type validation for nested Pydantic models. Updates tests accordingly and adjusts interrogate exclusions for generated cforge/_version.py. Cleans up .gitignore duplicates.

Author: MatthewGrigsby

.gitignore

@@ -18,7 +18,6 @@ CLAUDE.local.md
 .scannerwork
 llms-full.txt
 aider*
-.aider*
 todo/
 *.sarif
 devskim-results.sarif
@@ -35,7 +34,6 @@ token.txt
 mcpgateway.sbom.xml
 gateway_service_leader.lock
 docs/docs/test/
-tmp
 *.tgz
 *.gz
 *.bz
@@ -74,9 +72,7 @@ dictionary.dic
 pdm.lock
 .pdm-python
 temp/
-public/
 *history.md
-htmlcov
 test_commands.md
 cover.md
 build/
@@ -99,28 +95,22 @@ scribeflow.log
 coverage_re
 bin/flagged
 flagged/
-certs/
 # VENV
 .python37/
 .python39/
 
 # Byte-compiled / optimized / DLL files
 __pycache__/
-**/__pycache__/
 *.py[cod]
 *$py.class
 mcpgateway-wrapper/src/mcp_gateway_wrapper/__pycache__/
 
-# Bak
-*.bak
-
 # C extensions
 *.so
 
 # Distribution / packaging
 .wily/
 .Python
-build/
 develop-eggs/
 dist/
 downloads/
@@ -165,7 +155,6 @@ coverage.xml
 *.pot
 
 # Django stuff:
-*.log
 local_settings.py
 db.sqlite3
 
@@ -199,8 +188,6 @@ celerybeat-schedule
 *.sage.py
 
 # Environments
-.env
-.venv
 env/
 venv/
 ENV/
@@ -231,9 +218,6 @@ dmypy.json
 
 .idea/
 
-# Sonar
-.scannerwork
-
 # vim
 *.swp
 *,cover
@@ -244,9 +228,6 @@ logging/
 
 .ai*
 
-# downloads
-downloads/
-
 # db_path
 db_path/
 

cforge/commands/resources/plugins.py

@@ -13,10 +13,13 @@
     to support them.
 """
 
+# Standard
 from typing import Any, Dict, Optional
 
+# Third-Party
 import typer
 
+# First-Party
 from cforge.common.console import get_console
 from cforge.common.errors import AuthenticationError, CaseInsensitiveEnum, CLIError, handle_exception
 from cforge.common.http import make_authenticated_request

cforge/commands/resources/tools.py

@@ -186,8 +186,10 @@ def tools_execute(
     console = get_console()
 
     try:
+        if not tool_id.strip():
+            raise CLIError("Tool ID must be a non-empty string")
+
         prefilled_data: Optional[Dict[str, Any]] = None
-        prompt_optional = True
         if data_file:
             if not data_file.exists():
                 console.print(f"[red]File not found: {data_file}[/red]")
@@ -196,7 +198,7 @@ def tools_execute(
             if not isinstance(file_data, dict):
                 raise CLIError("Data file must contain a JSON object")
             prefilled_data = file_data
-            prompt_optional = False
+        prompt_optional = prefilled_data is None
 
         tool_result = make_authenticated_request("GET", f"/tools/{tool_id}")
         assert isinstance(tool_result, dict)
@@ -212,6 +214,14 @@ def tools_execute(
             input_schema = {"type": "object", "properties": {}}
         elif isinstance(raw_schema, dict):
             input_schema = raw_schema
+        elif isinstance(raw_schema, str):
+            try:
+                parsed_schema = json.loads(raw_schema)
+            except json.JSONDecodeError as exc:
+                raise CLIError("Tool input schema must be a JSON object") from exc
+            if not isinstance(parsed_schema, dict):
+                raise CLIError("Tool input schema must be a JSON object")
+            input_schema = parsed_schema
         else:
             raise CLIError("Tool input schema must be a JSON object")