gitpr-cli 0.0.37__tar.gz → 0.0.38__tar.gz

{gitpr_cli-0.0.37 → gitpr_cli-0.0.38}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: gitpr-cli
-Version: 0.0.37
+Version: 0.0.38
 Summary: AI-powered PR automation, commit messages, and code review (Gemini, DeepSeek, and Ollama)
 Author-email: Natan Fiuza <contato@natanfiuza.dev.br>
 Requires-Python: >=3.10

{gitpr_cli-0.0.37 → gitpr_cli-0.0.38}/gitpr_cli.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: gitpr-cli
-Version: 0.0.37
+Version: 0.0.38
 Summary: AI-powered PR automation, commit messages, and code review (Gemini, DeepSeek, and Ollama)
 Author-email: Natan Fiuza <contato@natanfiuza.dev.br>
 Requires-Python: >=3.10

{gitpr_cli-0.0.37 → gitpr_cli-0.0.38}/gitpr_cli.egg-info/SOURCES.txt

@@ -22,6 +22,7 @@ src/linter_wizard.py
 src/main.py
 src/mcp_server.py
 src/metrics.py
+src/net.py
 src/security.py
 src/spinner.py
 src/tui_issue.py
@@ -38,14 +39,18 @@ tests/test_blame_metrics.py
 tests/test_chat_backend.py
 tests/test_core.py
 tests/test_external_linters.py
+tests/test_github_api.py
 tests/test_i18n.py
 tests/test_install_wizard.py
+tests/test_issue_engine.py
 tests/test_linter_metrics.py
 tests/test_mcp_prompts.py
 tests/test_mcp_server.py
 tests/test_mcp_server_e2e.py
 tests/test_metrics.py
+tests/test_net_timeouts.py
 tests/test_plugins.py
+tests/test_pr_publish_app.py
 tests/test_pr_publish_linter_modal.py
 tests/test_pre_save.py
 tests/test_skill_command.py

{gitpr_cli-0.0.37 → gitpr_cli-0.0.38}/src/ai_providers.py

@@ -4,13 +4,36 @@ from datetime import datetime
 import click
 from google import genai
 from openai import OpenAI
-import urllib.request
 from pathlib import Path
 
+from src.net import bounded_resolve, bounded_urlopen
 from src.spinner import Spinner
 from src.i18n import __, CURRENT_LANG
 
 
+def _make_gemini_client(api_key, timeout_s):
+    """Builds a Gemini client with an explicit request timeout.
+
+    google-genai expresses http_options.timeout in MILLISECONDS.
+    """
+    bounded_resolve("generativelanguage.googleapis.com")
+    return genai.Client(
+        api_key=api_key, http_options={"timeout": int(timeout_s * 1000)}
+    )
+
+
+def _make_openai_client(api_key, provider, timeout_s):
+    """Builds an OpenAI-compatible client (DeepSeek/Ollama) with a timeout in seconds."""
+    base_url = (
+        "https://api.deepseek.com"
+        if provider == "deepseek"
+        else "http://localhost:11434/v1"
+    )
+    if provider == "deepseek":
+        bounded_resolve("api.deepseek.com")
+    return OpenAI(api_key=api_key, base_url=base_url, timeout=timeout_s)
+
+
 # Hidden --pre-save debug flag: when enabled, every AI payload is dumped
 # to a JSON file in the current directory before being sent to the model.
 PRE_SAVE_ENABLED = False
@@ -88,8 +111,11 @@ def call_ai_model(
     Unified engine for AI calls.
     Supports 'gemini' and 'deepseek'.
     """
+    from src.config import get_ai_timeout
+
     max_retries = 3
     retry_delay = 2
+    timeout_s = get_ai_timeout()
 
     if PRE_SAVE_ENABLED:
         saved_file = _save_pre_save_payload(
@@ -112,7 +138,7 @@ def call_ai_model(
             try:
                 meta_raw = {}
                 if provider == "gemini":
-                    client = genai.Client(api_key=api_key)
+                    client = _make_gemini_client(api_key, timeout_s)
                     response = client.models.generate_content(
                         model=api_model,
                         contents=prompt,
@@ -141,12 +167,7 @@ def call_ai_model(
 
                 elif provider in ["deepseek", "ollama"]:
                     # DeepSeek and Ollama are 100% compatible with the OpenAI library.
-                    base_url = (
-                        "https://api.deepseek.com"
-                        if provider == "deepseek"
-                        else "http://localhost:11434/v1"
-                    )
-                    client = OpenAI(api_key=api_key, base_url=base_url)
+                    client = _make_openai_client(api_key, provider, timeout_s)
 
                     response = client.chat.completions.create(
                         model=api_model,
@@ -244,17 +265,22 @@ def load_chat_commands():
         except Exception:
             pass
 
-    # If it is not in the cache, download it from the remote repository
+    # If it is not in the cache, download it from the remote repository.
+    # bounded_urlopen also bounds DNS resolution, which urllib's timeout does not.
     try:
-        req = urllib.request.Request(url, headers={"User-Agent": "GitPR-Chat"})
-        with urllib.request.urlopen(req, timeout=5) as response:
-            data = json.loads(response.read().decode())
+        raw = bounded_urlopen(
+            url, timeout=5, headers={"User-Agent": "GitPR-Chat"}
+        )
+        if raw is not None:
+            data = json.loads(raw.decode())
             with open(cache_file, "w", encoding="utf-8") as f:
                 json.dump(data, f, ensure_ascii=False, indent=2)
             return data
     except Exception:
-        # Safety fallback if the user is offline
-        return {
+        pass  # Fall through to the offline defaults below
+
+    # Safety fallback if the user is offline
+    return {
             "/explain": "Explains the diff line by line.",
             "/tests": "Generates unit tests for the changed functions.",
             "/optimize": "Analyzes cyclomatic complexity and performance.",
@@ -295,6 +321,10 @@ def call_ai_chat(
     Dedicated engine for the Interactive Chat.
     Keeps the historical context and returns free Markdown (does not force JSON).
     """
+    from src.config import get_ai_timeout
+
+    timeout_s = get_ai_timeout()
+
     if PRE_SAVE_ENABLED:
         saved_file = _save_pre_save_payload(
             "chat",
@@ -316,7 +346,7 @@ def call_ai_chat(
 
     try:
         if provider == "gemini":
-            client = genai.Client(api_key=api_key)
+            client = _make_gemini_client(api_key, timeout_s)
 
             # Format the history into the Gemini SDK format
             formatted_contents = []
@@ -341,12 +371,7 @@ def call_ai_chat(
             result_text = response.text
 
         elif provider in ["deepseek", "ollama"]:
-            base_url = (
-                "https://api.deepseek.com"
-                if provider == "deepseek"
-                else "http://localhost:11434/v1"
-            )
-            client = OpenAI(api_key=api_key, base_url=base_url)
+            client = _make_openai_client(api_key, provider, timeout_s)
 
             messages = [{"role": "system", "content": system_instruction}]
             for msg in chat_history:

{gitpr_cli-0.0.37 → gitpr_cli-0.0.38}/src/blame_engine.py

@@ -3,7 +3,7 @@ import click
 import re
 import os
 from datetime import datetime
-from src.core import get_current_branch, resolve_output_path
+from src.core import get_current_branch, get_skill_context, resolve_output_path
 from src.config import get_api_key, get_api_model, get_ai_provider, resolve_skill_path
 from src.ai_providers import call_ai_model
 from src.i18n import __
@@ -77,8 +77,13 @@ def get_commit_info(commit_hash):
     return {"author": __("Unknown"), "date": __("Unknown"), "message": __("No message")}
 
 
-def analyze_commit_with_ai(commit_hash, file_path):
-    """Uses AI to read the diff and classify as ORIGIN or REFACTORING."""
+def analyze_commit_with_ai(commit_hash, file_path, sys_inst=None):
+    """Uses AI to read the diff and classify as ORIGIN or REFACTORING.
+
+    sys_inst: skill context pre-loaded once by run_blame_analysis() (avoids
+    repeating the confirmation message for every commit). When None, the
+    skill file is loaded silently (return_data/silent modes).
+    """
     diff = execute_git_show(commit_hash, file_path)
     if not diff:
         return {
@@ -94,11 +99,14 @@ def analyze_commit_with_ai(commit_hash, file_path):
     # Use the 'simple' model (Flash/Lite) to save money in the loop
     api_model = get_api_model(provider, task_complexity="simple")
 
-    skill_path = resolve_skill_path(".gitpr.blame.md")
-    if os.path.exists(skill_path):
-        with open(skill_path, "r", encoding="utf-8") as f:
-            sys_inst = f.read()
-    else:
+    if sys_inst is None:
+        # Silent modes: load the skill file without the confirmation message
+        skill_path = resolve_skill_path(".gitpr.blame.md")
+        if os.path.exists(skill_path):
+            with open(skill_path, "r", encoding="utf-8") as f:
+                sys_inst = f.read()
+
+    if not sys_inst:
         sys_inst = __(
             'You are a Software Architect. Analyze the diff and determine if it is the ORIGIN of the rule (new logic) or REFACTORING. Respond ONLY with JSON: {"status": "ORIGIN", "reason": "Explain what was introduced"} or {"status": "REFACTORING", "reason": "Explain what was changed"}'
         )
@@ -153,6 +161,12 @@ def run_blame_analysis(file_path, start_line, end_line, return_data=False):
             click.secho(__("⚠️ No traceable commits found in these lines."), fg="yellow")
         return [] if return_data else None
 
+    # Load the blame skill ONCE per run — passing it by parameter avoids the
+    # "found and loaded" message repeating for every commit analyzed.
+    # Silent modes (return_data) stay quiet and fall back to the internal
+    # silent load inside analyze_commit_with_ai.
+    skill_context = get_skill_context("blame") if not return_data else None
+
     if not return_data:
         click.secho(
             __(
@@ -177,7 +191,7 @@ def run_blame_analysis(file_path, start_line, end_line, return_data=False):
 
             seen_hashes.add(current_commit)
             info = get_commit_info(current_commit)
-            ai_analysis = analyze_commit_with_ai(current_commit, file_path)
+            ai_analysis = analyze_commit_with_ai(current_commit, file_path, skill_context)
 
             status = str(ai_analysis.get("status", "ORIGIN")).upper()
             reason = str(ai_analysis.get("reason", ""))

{gitpr_cli-0.0.37 → gitpr_cli-0.0.38}/src/config.py

@@ -36,8 +36,14 @@ DEFAULT_CONFIG = {
     "PR_PUBLISH_LOG": "true",
     "GITPR_AUTO_MERGE": "false",
     "OUTPUT_FILE_NAME_LINTER": "{branch}_{datetime}_LINTER.md",
+    "GITPR_AI_TIMEOUT": "180",
+    "GITPR_LINTER_TIMEOUT": "120",
 }
 
+# Fallbacks used when the .env value is missing or not a positive number.
+_DEFAULT_AI_TIMEOUT = 180.0
+_DEFAULT_LINTER_TIMEOUT = 120.0
+
 
 def get_skill_dir():
     """Returns the absolute path to the project's skill folder (.gitpr/skill)."""
@@ -93,6 +99,31 @@ def get_ai_provider():
     return os.getenv("DEFAULT_AI_PROVIDER", "gemini").lower()
 
 
+def _positive_float(raw, fallback):
+    """Parses *raw* as a positive float, falling back on junk or non-positive values."""
+    try:
+        value = float(raw)
+    except (TypeError, ValueError):
+        return fallback
+    return value if value > 0 else fallback
+
+
+def get_ai_timeout():
+    """Returns the AI SDK request timeout in seconds (GITPR_AI_TIMEOUT, default 600).
+
+    Bounds a single model call so a hung provider can never freeze the CLI
+    indefinitely. Invalid or non-positive values fall back to the default.
+    """
+    load_dotenv(ENV_FILE)
+    return _positive_float(os.getenv("GITPR_AI_TIMEOUT"), _DEFAULT_AI_TIMEOUT)
+
+
+def get_linter_timeout():
+    """Returns the external linter subprocess timeout in seconds (default 120)."""
+    load_dotenv(ENV_FILE)
+    return _positive_float(os.getenv("GITPR_LINTER_TIMEOUT"), _DEFAULT_LINTER_TIMEOUT)
+
+
 def coauthor_enabled():
     """Returns True if the Gitpr-cli co-author trailer should be appended (default True).
 

{gitpr_cli-0.0.37 → gitpr_cli-0.0.38}/src/core.py

@@ -617,7 +617,7 @@ def get_repo_name():
 
 
 def get_skill_context(action_type="pr"):
-    """Reads the correct context file based on the action (PR/Commit or Review)."""
+    """Reads the correct context file based on the action (PR/Commit/Review/Issue/Blame)."""
 
     # Define which file to look for
     if action_type == "commit":
@@ -628,6 +628,8 @@ def get_skill_context(action_type="pr"):
         target_file = ".gitpr.filereview.md"
     elif action_type == "issue":
         target_file = ".gitpr.issue.md"
+    elif action_type == "blame":
+        target_file = ".gitpr.blame.md"
     else:  # review or fullreview
         target_file = ".gitpr.review.md"
 
@@ -644,10 +646,10 @@ def get_skill_context(action_type="pr"):
     )
 
     if file_to_load:
+        nome_arquivo = os.path.basename(file_to_load)
         try:
             with open(file_to_load, "r", encoding="utf-8") as f:
                 conteudo = f.read()
-                nome_arquivo = os.path.basename(file_to_load)
                 click.secho(
                     __(
                         "🧠 File {file_name} (Skill) found and loaded!",

{gitpr_cli-0.0.37 → gitpr_cli-0.0.38}/src/i18n.py

@@ -1,10 +1,11 @@
 import os
 import json
 import locale
-import urllib.request
 from pathlib import Path
 from dotenv import load_dotenv, set_key
 
+from src.net import bounded_urlopen
+
 # Global path to the .env file
 env_path = Path.home() / ".gitpr" / ".env"
 load_dotenv(env_path)
@@ -55,18 +56,21 @@ def get_translations(lang_code):
     if not local_file.exists() or needs_update:
         remote_url = f"https://raw.githubusercontent.com/natanfiuza/gitpr/main/langs/{lang_code}.json"
         try:
-            with urllib.request.urlopen(remote_url, timeout=3) as response:
-                content = response.read().decode("utf-8")
-
-            with open(local_file, "w", encoding="utf-8") as f:
-                f.write(content)
-
-            # Update .env with the new version after successful download
-            set_key(env_path, "LANG_VERSION", __lang_version__)
+            # bounded_urlopen also bounds DNS resolution, which urllib's
+            # timeout does not — a stalled resolver must never block startup.
+            raw = bounded_urlopen(remote_url, timeout=3)
+            if raw is not None:
+                with open(local_file, "w", encoding="utf-8") as f:
+                    f.write(raw.decode("utf-8"))
+
+                # Update .env with the new version after successful download
+                set_key(env_path, "LANG_VERSION", __lang_version__)
         except Exception:
-            # In case of failure (e.g.: offline), if the old file exists, use it
-            if not local_file.exists():
-                return {}
+            pass  # Fall through: the stale local file below is the fallback
+
+        # In case of failure (e.g.: offline), if the old file exists, use it
+        if not local_file.exists():
+            return {}
 
     if local_file.exists():
         try:

{gitpr_cli-0.0.37 → gitpr_cli-0.0.38}/src/issue_engine.py

@@ -1,14 +1,19 @@
 import subprocess
 import re
-import os
 import time
 import click
 from src.ai_providers import call_ai_model
 from src.cache import get_cached_response, save_cached_response
-from src.config import get_api_key, get_api_model, get_ai_provider, resolve_skill_path
+from src.config import get_api_key, get_api_model, get_ai_provider
 from src.ai_providers import call_ai_model
 from src.i18n import __
-from src.core import estimate_token_count, split_diff_into_chunks, get_doc_url
+from src.core import (
+    estimate_token_count,
+    split_diff_into_chunks,
+    get_doc_url,
+    get_skill_context,
+    get_changed_docs_list,
+)
 
 
 def get_github_repo_info():
@@ -56,17 +61,44 @@ def generate_issue_content(context_text, context_type="diff"):
     # Use the advanced model to ensure Issue structure quality
     api_model = get_api_model(provider, task_complexity="advanced")
 
-    skill_path = resolve_skill_path(".gitpr.issue.md")
-    sys_inst = ""
-
-    if os.path.exists(skill_path):
-        with open(skill_path, "r", encoding="utf-8") as f:
-            sys_inst = f.read()
-    else:
+    # Load the issue skill with visual feedback (parity with the PR flow);
+    # the default Software Architect persona remains the fallback.
+    sys_inst = get_skill_context("issue")
+    if not sys_inst:
         sys_inst = __(
             "You are a Software Architect. Follow the What / Why / Where / How format to document the Issue."
         )
 
+    # ── Inject changed documentation file list as metadata (no content) ──
+    # Parity with the PR flow: the diff excludes docs via Smart Excludes
+    # pathspec, so the AI must know which docs were touched without their
+    # full prose/markup content (excluded from the diff).
+    if context_type == "diff":
+        try:
+            changed_docs = get_changed_docs_list()
+            if changed_docs:
+                docs_section = __(
+                    "Changed documentation (content excluded from diff):\n"
+                )
+                for doc in changed_docs:
+                    docs_section += f"- {doc}\n"
+                sys_inst = docs_section + "\n" + sys_inst
+                click.secho(
+                    __(
+                        "📄 {count} documentation file(s) excluded from diff (Smart Excludes).",
+                        count=len(changed_docs),
+                    ),
+                    fg="blue",
+                    dim=True,
+                )
+                click.secho(
+                    f"📚 {__('Learn more:')} {get_doc_url('smart-excludes.md')}",
+                    fg="blue",
+                    underline=True,
+                )
+        except Exception:
+            pass  # Non-critical — never block the main flow for this metadata
+
     # Adaptive Brain (Dynamic Prompt)
     if context_type == "blame":
         target_action = __(

{gitpr_cli-0.0.37 → gitpr_cli-0.0.38}/src/linter_engine.py

@@ -1,9 +1,11 @@
 import re
 import fnmatch
+import shlex
+import shutil
 import subprocess
 import os
 import xml.etree.ElementTree as ET
-from src.config import load_linter_rules, load_external_linters
+from src.config import load_linter_rules, load_external_linters, get_linter_timeout
 from src.i18n import __
 from src.metrics import log_local_metric
 
@@ -69,18 +71,44 @@ def _apply_rule(rule, code_line, line_number, current_file, alerts):
         )
 
 
-def _run_external_linter(command, file_path):
-    """Executes an external linter command and returns its stdout (Checkstyle XML)."""
+def _split_command(command):
+    """Splits a configured linter command string into an argv list.
+
+    Uses POSIX quoting rules (so "--report=checkstyle" loses its quotes and
+    quoted paths stay whole) but with backslash escaping DISABLED, otherwise
+    Windows paths like C:\\tools\\lint.exe would be mangled into C:toolslint.exe.
+    """
+    lexer = shlex.shlex(command, posix=True)
+    lexer.whitespace_split = True
+    lexer.escape = ""
+    return list(lexer)
+
+
+def _run_external_linter(command, file_path, timeout=None):
+    """Executes an external linter command and returns its stdout (Checkstyle XML).
+
+    The command runs as an argv list WITHOUT a shell, so metacharacters in the
+    configured command or in the target path are never interpreted (a file named
+    'a; rm -rf ~' is passed as one literal argument).  Because CreateProcess only
+    auto-appends .exe, the executable is resolved through shutil.which() so
+    PATHEXT shims such as npx.cmd keep working on Windows without shell=True.
+    """
     try:
-        # Resolve the command by injecting the target file
-        full_command = f'{command} "{file_path}"'
+        argv = _split_command(command)
+        if not argv:
+            return ""
+
+        resolved = shutil.which(argv[0])
+        if resolved:
+            argv[0] = resolved
+
+        argv.append(file_path)
 
         result = subprocess.run(
-            full_command,
-            shell=True,
+            argv,
             capture_output=True,
             stdin=subprocess.DEVNULL,
-            timeout=120,
+            timeout=timeout if timeout is not None else get_linter_timeout(),
             text=True,
             encoding="utf-8",
             errors="replace",
@@ -92,7 +120,11 @@ def _run_external_linter(command, file_path):
 
 
 def _parse_checkstyle_xml(xml_content):
-    """Extracts errors from Checkstyle XML into a dictionary list."""
+    """Extracts errors from Checkstyle XML into a dictionary list.
+
+    Each entry carries the owning <file name=...> so callers can attribute a
+    violation to its source file instead of matching on line number alone.
+    """
     results = []
     if not xml_content or not xml_content.strip():
         return results
@@ -100,6 +132,7 @@ def _parse_checkstyle_xml(xml_content):
     try:
         root = ET.fromstring(xml_content)
         for file_node in root.findall("file"):
+            source_file = file_node.get("name", "")
             for error_node in file_node.findall("error"):
                 try:
                     line = int(error_node.get("line", 0))
@@ -107,6 +140,7 @@ def _parse_checkstyle_xml(xml_content):
                     continue
                 results.append(
                     {
+                        "file": source_file,
                         "line": line,
                         "severity": error_node.get("severity", "error").lower(),
                         "message": error_node.get("message", ""),
@@ -117,6 +151,61 @@ def _parse_checkstyle_xml(xml_content):
     return results
 
 
+def _checkstyle_file_matches(reported_path, target_path):
+    """True when a Checkstyle <file name=...> refers to *target_path*.
+
+    Checkstyle reports absolute paths while the diff yields repo-relative ones,
+    so the comparison is suffix-based over normalized separators.  A report with
+    no name attribute cannot be discriminated and is accepted, so a linter that
+    omits it never silently loses every violation.
+    """
+    if not reported_path:
+        return True
+
+    reported = os.path.normpath(reported_path).replace("\\", "/").lower()
+    target = os.path.normpath(target_path).replace("\\", "/").lower()
+
+    return (
+        reported == target
+        or reported.endswith("/" + target)
+        or target.endswith("/" + reported)
+    )
+
+
+def _collect_external_alerts(
+    external_linters, file_path, file_extension, alerts, allowed_lines=None
+):
+    """Runs every external linter matching *file_extension* against *file_path*.
+
+    Violations are attributed by file AND line: the file guard drops results a
+    linter reports for OTHER files (project-wide configs, followed imports),
+    which previously leaked in whenever their line number happened to collide
+    with an added line.  When *allowed_lines* is None (full-file mode) every
+    line of the target file counts; otherwise only the diff's added lines do.
+    """
+    for ext_linter in external_linters:
+        if file_extension not in ext_linter.get("extensions", []):
+            continue
+
+        command = ext_linter.get("command")
+        if not command:
+            continue
+
+        xml_output = _run_external_linter(command, file_path)
+
+        for err in _parse_checkstyle_xml(xml_output):
+            if not _checkstyle_file_matches(err["file"], file_path):
+                continue
+            if allowed_lines is not None and err["line"] not in allowed_lines:
+                continue
+
+            msg = f"🚨 [{ext_linter.get('name', 'External linter')}] {err['message']} ({file_path}, Line {err['line']})"
+            if err["severity"] == "warning":
+                alerts["warnings"].append(msg)
+            else:
+                alerts["errors"].append(msg)
+
+
 def parse_diff_and_lint(diff_text, is_full_file=False, file_path=None):
     """
     Analyzes the git diff OR a full file and applies the rules defined in .gitpr.linter.yml.
@@ -153,6 +242,17 @@ def parse_diff_and_lint(diff_text, is_full_file=False, file_path=None):
                     continue
                 _apply_rule(rule, code_line, i, current_file, alerts)
 
+        # External linters audit the whole file here: with no diff to intersect,
+        # every violation the linter reports for this file is in scope.
+        if external_linters:
+            _collect_external_alerts(
+                external_linters,
+                current_file,
+                file_extension,
+                alerts,
+                allowed_lines=None,
+            )
+
         log_local_metric(
             command="linter",
             status="success",
@@ -204,24 +304,13 @@ def parse_diff_and_lint(diff_text, is_full_file=False, file_path=None):
         for f_path, modified_lines in modified_files.items():
             f_ext = f_path.split(".")[-1] if "." in f_path else ""
 
-            for ext_linter in external_linters:
-                if f_ext not in ext_linter.get("extensions", []):
-                    continue
-
-                command = ext_linter.get("command")
-                if not command:
-                    continue
-
-                xml_output = _run_external_linter(command, f_path)
-                ext_errors = _parse_checkstyle_xml(xml_output)
-
-                for err in ext_errors:
-                    if err["line"] in modified_lines:
-                        msg = f"🚨 [{ext_linter.get('name', 'External linter')}] {err['message']} ({f_path}, Line {err['line']})"
-                        if err["severity"] == "warning":
-                            alerts["warnings"].append(msg)
-                        else:
-                            alerts["errors"].append(msg)
+            _collect_external_alerts(
+                external_linters,
+                f_path,
+                f_ext,
+                alerts,
+                allowed_lines=set(modified_lines),
+            )
 
     log_local_metric(
         command="linter",

{gitpr_cli-0.0.37 → gitpr_cli-0.0.38}/src/main.py

@@ -158,7 +158,7 @@ HELP_MAP: dict[str, dict[str, str]] = {
         ),
     },
     "chat": {
-        "url": get_doc_url("chat-interativo.md"),
+        "url": get_doc_url("understanding_chat_functionality.md"),
         "title": __("Interactive Pair Programming Chat (--chat)"),
         "description": __(
             "Opens an interactive terminal (TUI) to chat with the AI about the current uncommitted changes. Features memory, auto-patching (F5), and live diff refresh (F2)."
@@ -172,7 +172,7 @@ HELP_MAP: dict[str, dict[str, str]] = {
         ),
     },
     "metrics": {
-        "url": get_doc_url("metricas_analytics_dashboard.md"),
+        "url": get_doc_url("metricas-telemetria.md"),
         "title": __("Metrics & Analytics (--metrics)"),
         "description": __("Export or purge local telemetry data for team analytics."),
     },