diffron 0.1.10__tar.gz → 0.1.11__tar.gz

{diffron-0.1.10/diffron.egg-info → diffron-0.1.11}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: diffron
-Version: 0.1.10
+Version: 0.1.11
 Summary: Git commit message and PR description generator using Lemonade
 Home-page: https://github.com/diffron/diffron
 Author: Diffron Contributors

{diffron-0.1.10 → diffron-0.1.11}/diffron/__init__.py

@@ -24,7 +24,7 @@ from .models import (
     get_model_config,
 )
 
-__version__ = "0.1.10"
+__version__ = "0.1.11"
 __author__ = "Diffron Contributors"
 __all__ = [
     "DiffronClient",

{diffron-0.1.10 → diffron-0.1.11}/diffron/commit_gen.py

@@ -28,8 +28,8 @@ COMMIT_TYPES = [
 ]
 
 DEFAULT_MAX_CHARS = 4000
-DEFAULT_MAX_TOKENS = 100
-DEFAULT_TEMPERATURE = 0.2
+DEFAULT_MAX_TOKENS = 500
+DEFAULT_TEMPERATURE = 0.4
 
 
 def generate_commit_message(
@@ -45,8 +45,8 @@ def generate_commit_message(
     Args:
         diff: Git diff string. If None, gets staged diff automatically.
         max_chars: Maximum characters of diff to send. Defaults to 4000.
-        max_tokens: Maximum tokens to generate. Defaults to 100.
-        temperature: Sampling temperature. Defaults to 0.2.
+        max_tokens: Maximum tokens to generate. Defaults to 500.
+        temperature: Sampling temperature. Defaults to 0.4.
         client: LemonadeClient instance. Creates new one if not provided.
 
     Returns:
@@ -74,10 +74,38 @@ def generate_commit_message(
     # Build prompt
     commit_types = ", ".join(COMMIT_TYPES)
     prompt = (
-        f"Write a concise Git commit message in Conventional Commits format. "
-        f"Use one of these types: {commit_types}. "
-        f"Format: 'type: description' (e.g., 'feat: add user authentication'). "
-        f"Output ONLY the commit message, nothing else. No quotes, no explanations.\n\n"
+        f"Write a Git commit message following the Conventional Commits specification.\n\n"
+        f"Format:\n"
+        f"  type(scope): short summary\n"
+        f"\n"
+        f"  Optional body explaining what changed and why.\n"
+        f"  Use bullet points for multiple changes.\n"
+        f"  Wrap lines at 72 characters.\n\n"
+        f"Rules:\n"
+        f"- Use one of: {commit_types}\n"
+        f"- Scope is optional — use it when the change affects a specific module/component\n"
+        f"- The summary line should be imperative mood (\"add\" not \"added\"), max 72 chars\n"
+        f"- The body lists WHAT changed per file/area and WHY, not a diff walkthrough\n"
+        f"- Separate title from body with a blank line\n"
+        f"- If the change is trivial (single file, <10 lines), a title-only message is fine\n\n"
+        f"Examples:\n"
+        f"  feat(auth): add JWT token refresh logic\n"
+        f"\n"
+        f"  Implement automatic token refresh when the access token expires.\n"
+        f"  Uses a background thread to refresh 30 seconds before expiry,\n"
+        f"  preventing interrupted user sessions.\n"
+        f"\n"
+        f"  refactor(api): extract route handlers into separate modules\n"
+        f"\n"
+        f"  router.py:\n"
+        f"  - Move auth routes to auth_router.py\n"
+        f"  - Move user routes to user_router.py\n"
+        f"\n"
+        f"  Reduces router.py from 800 to 120 lines for easier maintenance.\n"
+        f"\n"
+        f"  fix(parser): handle empty input gracefully\n"
+        f"\n"
+        f"Output ONLY the commit message, nothing else.\n\n"
         f"Diff:\n{diff}"
     )
 
@@ -97,19 +125,24 @@ def generate_commit_message(
     commit_message = re.sub(r"<think>.*?</think>", "", commit_message, flags=re.DOTALL)
     commit_message = commit_message.replace("<think>", "").replace("</think>", "").strip()
 
-    # Remove any surrounding quotes
-    if commit_message.startswith('"') and commit_message.endswith('"'):
-        commit_message = commit_message[1:-1]
-    if commit_message.startswith("'") and commit_message.endswith("'"):
-        commit_message = commit_message[1:-1]
-
     # Remove markdown code blocks if present
     if commit_message.startswith("```"):
         lines = commit_message.split("\n")
-        commit_message = "\n".join(
-            line for line in lines
-            if not line.startswith("```")
-        ).strip()
+        # Find opening and closing ``` and strip them
+        start = 0
+        end = len(lines)
+        for i, line in enumerate(lines):
+            if line.strip().startswith("```") and i == 0:
+                start = 1
+            elif line.strip() == "```" and i > 0:
+                end = i
+                break
+        commit_message = "\n".join(lines[start:end]).strip()
+
+    # Remove surrounding quotes (only if entire message is wrapped)
+    if (commit_message.startswith('"') and commit_message.endswith('"')) or \
+       (commit_message.startswith("'") and commit_message.endswith("'")):
+        commit_message = commit_message[1:-1].strip()
 
     return commit_message
 

{diffron-0.1.10 → diffron-0.1.11/diffron.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: diffron
-Version: 0.1.10
+Version: 0.1.11
 Summary: Git commit message and PR description generator using Lemonade
 Home-page: https://github.com/diffron/diffron
 Author: Diffron Contributors

{diffron-0.1.10 → diffron-0.1.11}/diffron.egg-info/SOURCES.txt

@@ -38,9 +38,6 @@ diffron/hooks/prepare-commit-msg
 diffron/hooks/prepare-commit-msg-global.py
 diffron/hooks/prepare-commit-msg.py
 docs/HOOKS.md
-docs/LEMONADE_PORT_VERIFICATION_FIX.md
-docs/LEMONADE_UNRESPONSIVE_FIX.md
-docs/MODEL_NAME_CASE_FIX.md
 docs/SETUP.md
 docs/USAGE.md
 hooks/aipr.py

{diffron-0.1.10 → diffron-0.1.11}/diffron.egg-info/top_level.txt

@@ -1,4 +1,5 @@
 build
+dev-docs
 diffron
 dist
 docs

{diffron-0.1.10 → diffron-0.1.11}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "diffron"
-version = "0.1.10"
+version = "0.1.11"
 description = "Git commit message and PR description generator using Lemonade"
 readme = "README.md"
 license = {text = "MIT"}

diffron-0.1.10/docs/LEMONADE_PORT_VERIFICATION_FIX.md

@@ -1,105 +0,0 @@
-# Lemonade Port Verification Fix
-
-## Problem
-
-Diffron incorrectly connected to **port 8000** (running Axo Biomarker Analyzer) instead of **port 8020** (running Lemonade), causing:
-
-```
-Error code: 404 - {'detail': 'Not Found'}
-```
-
-### Root Cause
-
-The `detect_lemonade_port()` function in `diffron/lemonade.py` only checked if a **port was open** (`is_port_open`), not whether it was actually a **Lemonade server**.
-
-When port 8000 responded (Axo Biomarker web app), Diffron mistakenly identified it as Lemonade and sent API requests to the wrong service.
-
-| Port | Service | Response |
-|------|---------|----------|
-| 8000 | Axo Biomarker Analyzer | HTML web page |
-| 8020 | **Lemonade** (correct) | `{"data": [...]}` API |
-
-## Solution
-
-### Use lemonade-python-sdk's `verify_lemonade_server`
-
-The **lemonade-python-sdk** provides `verify_lemonade_server(port, host)` which:
-
-1. Queries the native Lemonade API endpoint `/api/v1/models`
-2. Validates the response format (`{"data": [...]}` or list)
-3. Distinguishes Lemonade from other services (Ollama, web apps, etc.)
-
-### Changes
-
-**1. Added `lemonade-sdk` dependency**
-
-Updated `requirements.txt`:
-```txt
-lemonade-sdk>=0.1.0
-```
-
-**2. Updated `detect_lemonade_port()`**
-
-`diffron/lemonade.py` now:
-
-- Imports `verify_lemonade_server` and `find_available_lemonade_port` from `lemonade_sdk.port_scanner`
-- **Verifies** each open port is actually Lemonade before returning it
-- Falls back to legacy `is_port_open` behavior if SDK is not installed
-
-```python
-# Import from lemonade-python-sdk for proper Lemonade verification
-try:
-    from lemonade_sdk.port_scanner import verify_lemonade_server, find_available_lemonade_port
-    HAS_LEMONADE_SDK = True
-except ImportError:
-    HAS_LEMONADE_SDK = False
-
-def detect_lemonade_port(...) -> Optional[int]:
-    # ... existing env var checks ...
-
-    # Try SDK's find_available_lemonade_port first
-    if HAS_LEMONADE_SDK:
-        found = find_available_lemonade_port(hostname, scan_ports)
-        if found:
-            return found
-
-    # Fallback: manual scan with SDK verification
-    for port in ports_to_scan:
-        if is_port_open(hostname, port):
-            if HAS_LEMONADE_SDK:
-                if verify_lemonade_server(port, hostname):  # ← Key fix!
-                    return port
-            else:
-                return port  # Legacy: no verification
-```
-
-**3. Added graceful fallback**
-
-If `lemonade-sdk` is not installed, Diffron falls back to the original `is_port_open` behavior (first open port wins), maintaining backward compatibility.
-
-## Testing
-
-```bash
-# Before fix: returned 8000 (Axo Biomarker)
-# After fix: returns 8020 (Lemonade)
-python -c "from diffron.lemonade import detect_lemonade_port; print(detect_lemonade_port())"
-# Output: 8020
-```
-
-## Impact
-
-- **Fixes 404 errors** when other services run on ports 8000/8080
-- **No breaking changes** - existing setups continue to work
-- **Better diagnostics** - clear separation between Lemonade and non-Lemonade services
-
-## Files Changed
-
-| File | Change |
-|------|--------|
-| `diffron/lemonade.py` | Use `verify_lemonade_server` for port detection |
-| `requirements.txt` | Add `lemonade-sdk>=0.1.0` dependency |
-
-## Version
-
-- **Fixed in**: 0.1.3
-- **Date**: 2026-04-13

diffron-0.1.10/docs/LEMONADE_UNRESPONSIVE_FIX.md

@@ -1,97 +0,0 @@
-# Lemonade Model Unresponsive Fix
-
-## Problem
-
-Lemonade-Modell wurde manchmal unresponsive, was zu Fehlern in Diffron führte:
-
-```
-Diffron error: 'NoneType' object is not subscriptable
-```
-
-oder
-
-```
-Lemonade model failed after 3 attempts: Lemonade model returned no response
-```
-
-## Lösung
-
-In [`diffron/lemonade.py`](diffron/lemonade.py) wurde die Retry-Logik mit automatischem Entladen und Neuladen des Modells implementiert.
-
-### Änderungen in `LemonadeClient.chat_completion()`
-
-```python
-max_retries = 3
-for attempt in range(1, max_retries + 1):
-    try:
-        response = self.client.chat.completions.create(...)
-        # Handle None response (model unresponsive)
-        if response is None or not response.choices:
-            if attempt < max_retries:
-                self._reload_model()
-                continue
-            raise RuntimeError("Lemonade model returned no response")
-        return response.choices[0].message.content
-    except Exception as e:
-        if attempt < max_retries:
-            self._reload_model()
-            continue
-        raise RuntimeError(
-            f"Lemonade model failed after {max_retries} attempts: {e}"
-        )
-```
-
-### Neue Methode `_reload_model()`
-
-Entlädt und lädt das Lemonade-Modell neu, wenn es unresponsive wird. Verwendet die `unload_model()` und `load_model()` Methoden des lemonade-python-sdk, um einen echten Server-seitigen Neustart zu erzwingen.
-
-```python
-def _reload_model(self) -> None:
-    """Unload and reload Lemonade model when unresponsive.
-    
-    Uses lemonade-python-sdk's unload_model() and load_model() methods
-    to properly force a server-side model restart.
-    """
-    try:
-        # Use SDK's LemonadeClient for server operations
-        sdk_client = _LemonadeSDKClient(
-            base_url=f"http://{self.host}:{self._port}"
-        )
-
-        # Unload current model from server
-        sdk_client.unload_model()
-
-        # Small delay to allow server to complete unload
-        import time
-        time.sleep(0.5)
-
-        # Load the model back on server
-        sdk_client.load_model(model_name=self.model)
-
-        # Reinitialize our client (same port, model should be fresh)
-        self._init_client()
-
-    except Exception:
-        pass  # Continue even if reload fails
-```
-
-### Wichtige Änderungen
-
-1. **Import-Alias**: Der SDK-Import wurde zu `_LemonadeSDKClient` umbenanmt, um Namenskonflikte mit der eigenen `LemonadeClient` Klasse zu vermeiden.
-
-2. **Echter Server-Neustart**: Statt nur den Port zu wechseln (was nicht funktioniert hat), wird jetzt das Modell über die SDK-API entladen und neu geladen:
-   - `sdk_client.unload_model()` - Entlädt das aktuelle Modell vom Server
-   - `time.sleep(0.5)` - Kurze Wartezeit für sauberen unload
-   - `sdk_client.load_model(model_name=self.model)` - Lädt das Modell neu
-
-3. **Retry-Logik**: Bei 3 Versuchen wird jeweils versucht, das Modell neu zu laden, bevor ein Fehler geworfen wird.
-
-## Technische Details
-
-- **Maximale Wiederholungen**: 3
-- **Wartezeit zwischen Reloads**: 0.5 Sekunden
-- **Fehlerbehandlung**: Bei jedem Reload-Versuch werden alle Exceptions abgefangen, damit der Prozess fortgesetzt werden kann
-
-## Version
-
-Diese Änderung wurde in Version 0.1.5 veröffentlicht.

diffron-0.1.10/docs/MODEL_NAME_CASE_FIX.md

@@ -1,88 +0,0 @@
-# Model Name Case Sensitivity Fix
-
-## Problem
-
-Diffron returned a **404 Not Found** error from the Lemonade server when using the `Qwen3.5-0.8B-GGUF` model.
-
-### Root Cause
-
-The Lemonade server is **case-sensitive** when matching model names. Diffron's curated model configuration used the wrong casing:
-
-| Location | Model Name (Before) | Model Name (Expected) |
-|----------|---------------------|------------------------|
-| `diffron/models.py` | `qwen3.5-0.8b-gguf` | `Qwen3.5-0.8B-GGUF` |
-
-This caused the OpenAI SDK to send a request with the wrong model identifier, resulting in:
-
-```
-Error code: 404 - {'detail': 'Not Found'}
-```
-
-## Solution
-
-### 1. Corrected Model Name
-
-Updated the canonical model name in `diffron/models.py`:
-
-```python
-# Before
-ModelConfig(name="qwen3.5-0.8b-gguf", ...)
-
-# After
-ModelConfig(name="Qwen3.5-0.8B-GGUF", ...)
-```
-
-### 2. Case-Insensitive Model Lookup
-
-Enhanced `get_model_config()` to accept case-insensitive input while always returning the **canonical name**:
-
-```python
-def get_model_config(model_name: str) -> Optional[ModelConfig]:
-    # Exact match first
-    for model in AVAILABLE_MODELS:
-        if model.name == model_name:
-            return model
-    # Case-insensitive fallback
-    for model in AVAILABLE_MODEOLS:
-        if model.name.lower() == model_name.lower():
-            return model
-    return None
-```
-
-This allows users to type `qwen3.5-0.8b-gguf`, `QWEN3.5-0.8B-GGUF`, etc., but Diffron always sends the correct name to the server.
-
-### 3. CLI Canonical Name Enforcement
-
-The `setup_model_cli()` now stores the **canonical name** from the config, not the user's raw input:
-
-```python
-# Always uses config.name (canonical) instead of args.model (user input)
-os.environ["DIFFRON_MODEL"] = config.name
-subprocess.run(["setx", "DIFFRON_MODEL", config.name], check=True)
-```
-
-## Testing
-
-```bash
-# All of these now work correctly:
-diffron-setup-model --model Qwen3.5-0.8B-GGUF   # exact
-diffron-setup-model --model qwen3.5-0.8b-gguf   # lowercase (fallback)
-diffron-setup-model --model QWEN3.5-0.8B-GGUF   # uppercase (fallback)
-```
-
-## Impact
-
-- **No breaking changes** - existing `DIFFRON_MODEL` values with wrong casing are now resolved correctly
-- **Improved UX** - users don't need to memorize exact casing
-- **Server compatibility** - correct model name is always sent to Lemonade
-
-## Files Changed
-
-| File | Change |
-|------|--------|
-| `diffron/models.py` | Fixed `Qwen3.5-0.8B-GGUF` name, added case-insensitive lookup |
-
-## Version
-
-- **Fixed in**: 0.1.3
-- **Date**: 2026-04-13