Update app.py

app.py

@@ -54,24 +54,31 @@ def create_labeled_zip(image, detections):
     zip_path = os.path.join(tempfile.gettempdir(), "rosetta_glyphs.zip")
     
     try:
+        # Load a font (try-except block for system compatibility)
         try:
+            # Try loading a standard font, fallback to default if fails
             font = ImageFont.truetype("arial.ttf", 16)
         except IOError:
             font = ImageFont.load_default()
 
         for i, d in enumerate(detections):
+            # Crop
             box = d['box']
             crop = image.crop((box[0], box[1], box[2], box[3]))
             
+            # Prepare Label
             label_text = f"{d['code']} {int(d['confidence']*100)}%"
             draw = ImageDraw.Draw(crop)
             
+            # Calculate text size using textbbox (newer PIL versions)
             left, top, right, bottom = draw.textbbox((0, 0), label_text, font=font)
             text_w = right - left
             text_h = bottom - top
             
             img_w, img_h = crop.size
             
+            # Draw background rectangle (bottom right)
+            # Check if image is too small for label, if so, skip drawing to avoid crash
             if img_w > text_w and img_h > text_h:
                 rect_x0 = img_w - text_w - 4
                 rect_y0 = img_h - text_h - 4
@@ -81,9 +88,11 @@ def create_labeled_zip(image, detections):
                 draw.rectangle([rect_x0, rect_y0, rect_x1, rect_y1], fill="black")
                 draw.text((rect_x0 + 2, rect_y0), label_text, fill="white", font=font)
             
+            # Save crop
             filename = f"{d['code']}_{i}.png"
             crop.save(os.path.join(zip_dir, filename))
 
+        # Create Zip
         shutil.make_archive(zip_path.replace('.zip', ''), 'zip', zip_dir)
         return zip_path
         
@@ -113,6 +122,7 @@ def core_detect(image, conf_threshold):
                 conf = float(box.conf[0])
                 xyxy = box.xyxy[0].tolist()
                 
+                # Create detection object
                 detection = {
                     "code": code,
                     "description": gardiner_map.get(code, "Unknown"),
@@ -120,6 +130,8 @@ def core_detect(image, conf_threshold):
                     "box": xyxy
                 }
                 detections.append(detection)
+                
+                # Create crop (Clean crop for Gallery display)
                 crop_img = image.crop((xyxy[0], xyxy[1], xyxy[2], xyxy[3]))
                 crops.append((crop_img, f"{code}\n({int(conf*100)}%)"))
                 
@@ -227,9 +239,11 @@ def process_pipeline(image, conf_threshold):
         annotated_img, detections, crops = core_detect(image, conf_threshold)
         
         # 2. Prepare Downloads
+        # A. Annotated Image
         ann_path = os.path.join(tempfile.gettempdir(), "annotated_hieroglyphs.jpg")
         annotated_img.save(ann_path)
         
+        # B. Zip File with Labels
         zip_path = create_labeled_zip(image, detections)
         
         # 3. Extract Keywords & Translate
@@ -251,18 +265,16 @@ def process_pipeline(image, conf_threshold):
         print(f"Pipeline Error: {e}")
         return None, None, None, f"System Failure: {str(e)}", "", None, str(e), None, []
 
-# --- 6. MCP API FUNCTIONS (RENAMED FOR CLAUDE) ---
+# --- 6. MCP API FUNCTIONS ---
 
-def detect_glyphs(image: Image.Image, conf: float = 0.25) -> Tuple[str, Dict[str, Any]]:
-    """Detects hieroglyphs in an image. Returns path to annotated image and JSON data."""
+def detect_hieroglyphs_api(image: Image.Image, conf: float = 0.25) -> Tuple[str, Dict[str, Any]]:
     ann_img, dets, _ = core_detect(image, conf)
     with tempfile.NamedTemporaryFile(dir="/tmp/gradio_results", suffix=".jpg", delete=False) as t:
         ann_img.save(t.name)
         path = t.name
     return path, {"count": len(dets), "detections": dets}
 
-def translate_story(keywords_text: str) -> Tuple[str, str]:
-    """Translates a list of codes or keywords into a mystical story and academic text."""
+def translate_codes_api(keywords_text: str) -> Tuple[str, str]:
     if isinstance(keywords_text, str):
         keywords = [k.strip() for k in keywords_text.split(',')]
     else:
@@ -271,8 +283,7 @@ def translate_story(keywords_text: str) -> Tuple[str, str]:
     clean_mystical = re.sub('<[^<]+?>', '', mystical) 
     return clean_mystical, academic
 
-def get_stats(json_data: Dict[str, Any]) -> str:
-    """Generates analytics charts from detection JSON. Returns path to chart image."""
+def get_analytics_chart_api(json_data: Dict[str, Any]) -> str:
     dets = json_data.get("detections", [])
     fig = core_analytics(dets, 640, 640) 
     if fig is None: return "No data."
@@ -282,8 +293,7 @@ def get_stats(json_data: Dict[str, Any]) -> str:
     plt.close(fig)
     return path
 
-def list_codes() -> Dict[str, Any]:
-    """Returns the full dictionary of supported Gardiner codes and descriptions."""
+def list_all_codes_api() -> Dict[str, Any]:
     return gardiner_data
 
 # --- 7. HTML GENERATORS ---
@@ -392,18 +402,18 @@ guide_html = """
     <div class="card-title" style="color: #d4af37;">🤖 CLAUDE DESKTOP SETUP GUIDE</div>
     <div class="guide-step"><span class="step-title">STEP 0: PREREQUISITE</span><p>Ensure you have <b>Node.js</b> installed.</p></div>
     <div class="guide-step"><span class="step-title">STEP 1: PREPARE WORKSPACE</span>1. Create: <span class="path-highlight">C:\\Claude_Work</span><br>2. Move images inside.</div>
-    <div class="guide-step"><span class="step-title">STEP 2: CONFIGURE CLAUDE</span>1. Edit: <code>%APPDATA%\\Claude\\claude_desktop_config.json</code><br>2. Paste the JSON below.<br>3. Restart Claude (make sure to completely close from task manager or system tray)</div>
+    <div class="guide-step"><span class="step-title">STEP 2: CONFIGURE CLAUDE</span>1. Edit: <code>%APPDATA%\\Claude\\claude_desktop_config.json</code><br>2. Paste the JSON below.<br>3. Restart Claude.</div>
 </div>
 """
 
-# CHANGED: "gradio" key changed to "Rosetta Decoder"
+# URL UPDATED TO: youkii-xr/hieroglyph-mcp-server
 claude_json_content = """{
   "mcpServers": {
-    "Rosetta Decoder": {
+    "gradio": {
       "command": "npx",
       "args": [
         "mcp-remote",
-        "https://mcp-1st-birthday-rosetta-decoder.hf.space/gradio_api/mcp/",
+        "https://youkii-xr-hieroglyph-mcp-server.hf.space/gradio_api/mcp/",
         "--transport",
         "streamable-http"
       ]
@@ -414,7 +424,7 @@ claude_json_content = """{
         "-m",
         "gradio",
         "upload-mcp",
-        "https://mcp-1st-birthday-rosetta-decoder.hf.space/",
+        "https://youkii-xr-hieroglyph-mcp-server.hf.space/",
         "C:\\\\Claude_Work"
       ]
     }
@@ -430,19 +440,18 @@ with gr.Blocks(title="Rosetta Decoder Ultimate") as demo:
     gr.HTML(trail_script)
     
     # --- MCP TOOL REGISTRATION LAYER (Hidden) ---
-    # CHANGED: Renamed API names to be short verbs (detect, translate, etc.)
     with gr.Row(visible=False):
         btn_detect = gr.Button("Detect")
-        btn_detect.click(fn=detect_glyphs, inputs=[gr.Image(label="img"), gr.Number(label="conf")], outputs=[gr.Textbox(label="path"), gr.JSON(label="json")], api_name="detect")
+        btn_detect.click(fn=detect_hieroglyphs_api, inputs=[gr.Image(label="img"), gr.Number(label="conf")], outputs=[gr.Textbox(label="path"), gr.JSON(label="json")], api_name="detect")
         
         btn_trans = gr.Button("Translate")
-        btn_trans.click(fn=translate_story, inputs=[gr.Textbox(label="text")], outputs=[gr.Textbox(label="mystic"), gr.Textbox(label="academic")], api_name="translate")
+        btn_trans.click(fn=translate_codes_api, inputs=[gr.Textbox(label="text")], outputs=[gr.Textbox(label="mystic"), gr.Textbox(label="academic")], api_name="translate")
         
         btn_anal = gr.Button("Analytics")
-        btn_anal.click(fn=get_stats, inputs=[gr.JSON(label="data")], outputs=[gr.Textbox(label="chart_path")], api_name="analyze")
+        btn_anal.click(fn=get_analytics_chart_api, inputs=[gr.JSON(label="data")], outputs=[gr.Textbox(label="chart_path")], api_name="analytics")
         
         btn_list = gr.Button("List")
-        btn_list.click(fn=list_codes, inputs=[], outputs=[gr.JSON(label="data")], api_name="list_codes")
+        btn_list.click(fn=list_all_codes_api, inputs=[], outputs=[gr.JSON(label="data")], api_name="list_codes")
 
     # --- Visible UI ---
     with gr.Row(elem_classes="header-row"):