Merge pull request #18 from zinja-coder/fix_them_all

Fixed following issue zinja-coder/jadx-ai-mcp#33 and following issues: - fix: fixed the following error -> "HTTP error 400: {"error":"Missing required parameter 'class'"}." in following mcp tools -> get_fields_of_class, get_methods_of_class - fix: fixed the following error -> Unexpected error: Expecting value: line 1 column 1 (char 0). - fix: added dict as a return type in mcp tools to keep up to date with fastmcp

Author: zinja-coder

jadx_mcp_server.py

@@ -207,7 +207,10 @@ async def get_from_jadx(endpoint: str, params: dict = {}) -> Union[str, dict]:
  resp.raise_for_status()
  response = resp.text
  if isinstance(response, str):
- return json.loads(response)
+ try:
+ return json.loads(response)
+ except Exception as e: # fix the `Unexpected error: Expecting value: line 1 column 1 (char 0).` error
+ response = {"response":resp.text}
  return response
  except httpx.HTTPStatusError as e:
  error_message = f"HTTP error {e.response.status_code}: {e.response.text}"
@@ -320,7 +323,7 @@ async def get_methods_of_class(class_name: str) -> dict:
  A list of all methods in the class.
  """
 
- response = await get_from_jadx("methods-of-class", {"method": class_name})
+ response = await get_from_jadx("methods-of-class", {"class_name": class_name})
  if isinstance(response, str):
  return json.loads(response)
  return response
@@ -336,7 +339,7 @@ async def get_fields_of_class(class_name: str) -> dict:
  A list of all fields in the class.
  """
 
- response = await get_from_jadx("fields-of-class", {"method": class_name})
+ response = await get_from_jadx("fields-of-class", {"class_name": class_name})
  if isinstance(response, str):
  return json.loads(response)
  return response

test.sh

@@ -123,7 +123,7 @@ call_tool "get_methods_of_class" '{"class_name":"com.zin.dvac.AuthActivity"}' 18
 
 # 12) get_fields_of_class
 echo "--- get_fields_of_class ---"
-call_tool "get_fields_of_class" '{"class_name":"com.zin.dvac.AuthActivity"}' 19 | jq -r '.result[]? // .'
+call_tool "get_fields_of_class" '{"class_name":"com.zin.dvac.DatabaseHelper"}' 19 | jq -r '.result[]? // .'
 
 # 13) get_smali_of_class
 echo "--- get_smali_of_class ---"
@@ -152,11 +152,11 @@ echo "--- get_main_application_classes_code (offset=0,count=3) ---"
 call_tool "get_main_application_classes_code" '{"offset":0,"count":3}' 25 | jq -r '.result.items[]?.name, .result.items[]?.content'
 
 # 19) rename operations (use with care; examples commented)
-echo "--- rename_class ---"
-call_tool "rename_class" '{"class_name":"com.zin.dvac.AuthActivity","new_name":"WebViewActivity"}' 26 | jq
-echo "--- rename_method ---"
-call_tool "rename_method" '{"method_name":"com.zin.dvac.AuthActivity.onCreate","new_name":"initializeWebView"}' 27 | jq
-echo "--- rename_field ---"
-call_tool "rename_field" '{"class_name":"com.zin.dvac.LoginActivity","field_name":"editTextLoginPassword","new_name":"passwordInputField"}' 28 | jq
+#echo "--- rename_class ---"
+#call_tool "rename_class" '{"class_name":"com.zin.dvac.AuthActivity","new_name":"WebViewActivity"}' 26 | jq
+#echo "--- rename_method ---"
+#call_tool "rename_method" '{"method_name":"com.zin.dvac.AuthActivity.onCreate","new_name":"initializeWebView"}' 27 | jq
+#echo "--- rename_field ---"
+#call_tool "rename_field" '{"class_name":"com.zin.dvac.LoginActivity","field_name":"editTextLoginPassword","new_name":"passwordInputField"}' 28 | jq
 
 echo "== done =="