gemcode 0.3.97__tar.gz → 0.3.99__tar.gz

{gemcode-0.3.97/src/gemcode.egg-info → gemcode-0.3.99}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: gemcode
-Version: 0.3.97
+Version: 0.3.99
 Summary: Local-first coding agent on Google Gemini + ADK
 Author: GemCode Contributors
 License:                                  Apache License

{gemcode-0.3.97 → gemcode-0.3.99}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "gemcode"
-version = "0.3.97"
+version = "0.3.99"
 description = "Local-first coding agent on Google Gemini + ADK"
 readme = "README.md"
 requires-python = ">=3.11"

{gemcode-0.3.97 → gemcode-0.3.99}/src/gemcode/invoke.py

@@ -184,15 +184,33 @@ async def run_turn(
         # user-granted "Files and Folders" permissions on first access.
         # If approved once, we don't re-prompt for the rest of this session.
         attach_allow = True
-        if cfg is not None:
-          attach_allow = bool(getattr(cfg, "interactive_permission_ask", False))
-          attach_allow = attach_allow and hasattr(sys.stdin, "isatty") and sys.stdin.isatty()
-          if attach_allow and not bool(getattr(cfg, "_attachments_allowed", False)):
-            attach_allow = _prompt_yes_no(
-              "Allow GemCode to read and upload the attached file(s) from disk? (y/n) "
-            )
-            if attach_allow:
+        if hasattr(sys.stdin, "isatty") and sys.stdin.isatty():
+          # Default-on: attachments can read any local file path (not workspace-scoped),
+          # but we ask once per session so the user is in control and macOS can trigger
+          # its permission prompt at the moment we attempt the read.
+          attach_allow = os.environ.get("GEMCODE_ATTACHMENTS_ASK", "1").lower() not in (
+            "0",
+            "false",
+            "no",
+            "off",
+          )
+          if cfg is not None:
+            # If user already approved earlier in this session, don't prompt again.
+            if bool(getattr(cfg, "_attachments_allowed", False)):
+              attach_allow = True
+            # If yes-to-all is enabled, auto-allow attachments.
+            elif bool(getattr(cfg, "yes_to_all", False)):
+              attach_allow = True
               object.__setattr__(cfg, "_attachments_allowed", True)
+            elif attach_allow:
+              attach_allow = _prompt_yes_no(
+                "Allow GemCode to read and upload the attached file(s) from disk? (y/n) "
+              )
+              if attach_allow:
+                object.__setattr__(cfg, "_attachments_allowed", True)
+        else:
+          # Non-interactive sessions can't prompt; default to allow.
+          attach_allow = True
         effective_attachments = attachment_paths if attach_allow else None
 
         root = cfg.project_root if cfg is not None else Path.cwd()

{gemcode-0.3.97 → gemcode-0.3.99}/src/gemcode/multimodal_input.py

@@ -97,6 +97,18 @@ def _infer_mime(path: Path, data: bytes) -> tuple[str, list[str]]:
   return "application/octet-stream", warnings
 
 
+def _is_supported_mime(m: str) -> bool:
+  mm = (m or "").strip().lower()
+  if not mm:
+    return False
+  if mm.startswith(("image/", "audio/", "video/", "text/")):
+    return True
+  if mm == "application/pdf":
+    return True
+  # Most other application/* types are rejected by Gemini file parts.
+  return False
+
+
 def build_user_content(
   prompt: str,
   attachment_paths: Sequence[Path | str] | None,
@@ -135,6 +147,12 @@ def build_user_content(
         continue
       mime, mw = _infer_mime(p, data)
       warnings.extend(mw)
+      if not _is_supported_mime(mime) or mime == "application/octet-stream":
+        warnings.append(
+          f"unsupported attachment type for {p} (mime={mime}); "
+          "skipping (export to PDF/image/text if needed)"
+        )
+        continue
       parts.append(types.Part(inline_data=types.Blob(data=data, mime_type=mime)))
 
   text = (prompt or "").strip() or (

{gemcode-0.3.97 → gemcode-0.3.99}/src/gemcode/session_summariser.py

@@ -100,16 +100,51 @@ def _build_prompt(transcript_lines: list[str], *, focus: str = "") -> str:
   )
 
 
-def _call_summary_model(*, model: str, prompt: str) -> dict[str, Any]:
+def _extract_json_object(text: str) -> dict[str, Any] | None:
+  """
+  Best-effort parse:
+  - strict JSON object
+  - JSON object embedded in extra text (common model behavior)
+  """
+  raw = (text or "").strip()
+  if not raw:
+    return None
+  try:
+    obj = json.loads(raw)
+    return obj if isinstance(obj, dict) else None
+  except Exception:
+    pass
+
+  # Try to recover embedded JSON: find the largest {...} span.
+  start = raw.find("{")
+  end = raw.rfind("}")
+  if start == -1 or end == -1 or end <= start:
+    return None
+  candidate = raw[start : end + 1].strip()
+  try:
+    obj = json.loads(candidate)
+    return obj if isinstance(obj, dict) else None
+  except Exception:
+    return None
+
+
+def _call_summary_model(*, model: str, prompt: str) -> tuple[dict[str, Any] | None, str]:
   api_key = os.environ.get("GOOGLE_API_KEY")
   if not api_key:
     raise RuntimeError("GOOGLE_API_KEY not set")
 
   client = Client(api_key=api_key)
+  cfg = types.GenerateContentConfig(temperature=0.2)
+  # Prefer structured JSON output when supported by the SDK/model.
+  try:
+    setattr(cfg, "response_mime_type", "application/json")
+  except Exception:
+    pass
+
   resp = client.models.generate_content(
-    model=model,
-    contents=[types.Content(role="user", parts=[types.Part(text=prompt)])],
-    config=types.GenerateContentConfig(temperature=0.2),
+      model=model,
+      contents=[types.Content(role="user", parts=[types.Part(text=prompt)])],
+      config=cfg,
   )
 
   out_parts: list[str] = []
@@ -127,14 +162,7 @@ def _call_summary_model(*, model: str, prompt: str) -> dict[str, Any]:
   text = "".join(out_parts).strip()
   if not text:
     raise RuntimeError("session summariser returned empty text")
-
-  try:
-    data = json.loads(text)
-  except Exception as e:
-    raise RuntimeError(f"session summariser returned invalid JSON: {e}") from e
-  if not isinstance(data, dict):
-    raise RuntimeError("session summariser returned non-object JSON")
-  return data
+  return _extract_json_object(text), text
 
 
 def summarise_session(
@@ -149,7 +177,18 @@ def summarise_session(
     return {"error": "session transcript is empty", "session_id": session_id}
 
   prompt = _build_prompt(transcript_lines, focus=focus)
-  data = _call_summary_model(model=model, prompt=prompt)
+  data, raw_text = _call_summary_model(model=model, prompt=prompt)
+  if data is None:
+    # Graceful fallback: save raw text as summary_markdown so /summarise never hard-fails
+    # due to JSON formatting drift.
+    data = {
+      "title": f"Session {session_id[:8]}",
+      "summary_markdown": raw_text[:20_000],
+      "memory_facts": [],
+      "user_facts": [],
+      "notes_markdown": "",
+      "open_items": [],
+    }
 
   title = str(data.get("title") or f"Session {session_id[:8]}").strip()[:120]
   summary_markdown = str(data.get("summary_markdown") or "").strip()
@@ -219,6 +258,7 @@ def summarise_session(
     "summary_path": str(out_path),
     "title": title,
     "summary_markdown": summary_markdown,
+    "used_json": bool(data is not None and _extract_json_object(raw_text) is not None),
     "memory_facts_saved": saved_memory,
     "user_facts_saved": saved_user,
     "notes_status": note_status,

{gemcode-0.3.97 → gemcode-0.3.99/src/gemcode.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: gemcode
-Version: 0.3.97
+Version: 0.3.99
 Summary: Local-first coding agent on Google Gemini + ADK
 Author: GemCode Contributors
 License:                                  Apache License