commons-metrics 0.0.22__py3-none-any.whl → 0.0.24__py3-none-any.whl

commons_metrics/__init__.py

@@ -12,4 +12,4 @@ from .text_simplifier import TextSimplifier
 from .variable_finder import VariableFinder
 
 __all__ = ['Util', 'DatabaseConnection', 'ComponentRepository', 'UpdateDesignSystemComponents', 'GitHubAPIClient', 'AzureDevOpsClient', 'S3FileManager', 'CacheManager', 'CommonsReposClient', 'DateUtils', 'TextSimplifier', 'VariableFinder']
-__version__ = '0.0.22'
+__version__ = '0.0.24'

commons_metrics/azure_devops_client.py

@@ -5,7 +5,7 @@ import re
 from typing import List, Dict, Optional
 from requests.exceptions import HTTPError, Timeout, ConnectionError
 import urllib3
-from .commons_repos_client import extract_package_version_from_pubspec, extract_package_version_from_package_json
+from .commons_repos_client import CommonsReposClient
 
 # Suprimir warnings de SSL para Azure DevOps con certificados corporativos
 urllib3.disable_warnings(urllib3.exceptions.InsecureRequestWarning)
@@ -383,11 +383,11 @@ class AzureDevOpsClient:
         if platform == 'mobile':
             config_file = 'pubspec.yaml'
             default_package_name = 'bds_mobile'
-            extract_version_method = extract_package_version_from_pubspec
+            extract_version_method = CommonsReposClient.extract_package_version_from_pubspec
         else:  # web
             config_file = 'package.json'
             default_package_name = '@bancolombia/design-system-web'
-            extract_version_method = extract_package_version_from_package_json
+            extract_version_method = CommonsReposClient.extract_package_version_from_package_json
         
         # Usar nombre de paquete por defecto si no se proporciona
         package_name = design_system_name or default_package_name

commons_metrics/github_api_client.py

@@ -1,15 +1,46 @@
 import requests
 import base64
+import time
+from threading import Lock
 from typing import List, Dict, Optional
 import os
-from .commons_repos_client import extract_package_version_from_pubspec, extract_package_version_from_package_json
+from .commons_repos_client import CommonsReposClient
+
+
+class RateLimiter:
+    """
+    Controla el rate limiting para respetar los límites de API de GitHub.
+    GitHub permite 5000 requests/hora autenticados = ~1.4 TPS.
+    Usamos 720ms entre requests (1.39 TPS) para estar seguros.
+    """
+    def __init__(self, delay: float = 0.72):
+        self.delay = delay
+        self.last_request_time = 0
+        self.lock = Lock()
+    
+    def wait(self):
+        """Espera el tiempo necesario antes de hacer el siguiente request"""
+        with self.lock:
+            current_time = time.time()
+            time_since_last_request = current_time - self.last_request_time
+            
+            if time_since_last_request < self.delay:
+                sleep_time = self.delay - time_since_last_request
+                time.sleep(sleep_time)
+            
+            self.last_request_time = time.time()
+
 
 class GitHubAPIClient:
     """
-    Cliente para interactuar con la API de GitHub
+    Cliente para interactuar con la API de GitHub con rate limiting integrado.
+    Controla automáticamente el TPS para no exceder los límites de GitHub (5000 req/hora).
     """
     
-    def __init__(self, token: str, owner: str = None, repo: str = None):
+    # Rate limiter compartido entre todas las instancias
+    _rate_limiter = RateLimiter(delay=0.72)  # 720ms entre requests = 1.39 TPS
+    
+    def __init__(self, token: str, owner: str = None, repo: str = None, enable_rate_limit: bool = True):
         """
         Inicializa el cliente de GitHub API
         
@@ -17,10 +48,12 @@ class GitHubAPIClient:
             token: Personal Access Token de GitHub
             owner: Dueño del repositorio (ej: 'grupobancolombia-innersource') - Opcional
             repo: Nombre del repositorio (ej: 'NU0066001_BDS_MOBILE_Lib') - Opcional
+            enable_rate_limit: Si True, aplica rate limiting automático (por defecto True)
         """
         self.token = token
         self.owner = owner
         self.repo = repo
+        self.enable_rate_limit = enable_rate_limit
         self.base_url = f"https://api.github.com/repos/{owner}/{repo}" if owner and repo else "https://api.github.com"
         self.headers = {
             "Authorization": f"token {token}",
@@ -37,6 +70,9 @@ class GitHubAPIClient:
         Returns:
             Lista de diccionarios con información de archivos/carpetas
         """
+        if self.enable_rate_limit:
+            self._rate_limiter.wait()
+        
         url = f"{self.base_url}/contents/{path}"
         response = requests.get(url, headers=self.headers)
         
@@ -57,6 +93,9 @@ class GitHubAPIClient:
         Returns:
             Contenido del archivo como string, o None si no existe
         """
+        if self.enable_rate_limit:
+            self._rate_limiter.wait()
+        
         url = f"{self.base_url}/contents/{path}"
         response = requests.get(url, headers=self.headers)
         
@@ -150,6 +189,9 @@ class GitHubAPIClient:
         page = 1
         
         while True:
+            if self.enable_rate_limit:
+                self._rate_limiter.wait()
+            
             url = "https://api.github.com/search/code"
             params = {
                 'q': query,
@@ -207,11 +249,11 @@ class GitHubAPIClient:
         if platform == 'mobile':
             config_file = 'pubspec.yaml'
             default_package_name = 'bds_mobile'
-            extract_version_method = extract_package_version_from_pubspec
+            extract_version_method = CommonsReposClient.extract_package_version_from_pubspec
         else:  # web
             config_file = 'package.json'
             default_package_name = '@bancolombia/design-system-web'
-            extract_version_method = extract_package_version_from_package_json
+            extract_version_method = CommonsReposClient.extract_package_version_from_package_json
         
         # Usar nombre de paquete por defecto si no se proporciona
         package_name = design_system_name or default_package_name
@@ -255,6 +297,9 @@ class GitHubAPIClient:
         search_url = "https://api.github.com/search/repositories"
         
         while True:
+            if self.enable_rate_limit:
+                self._rate_limiter.wait()
+            
             params = {
                 'q': query,
                 'per_page': per_page,
@@ -294,6 +339,9 @@ class GitHubAPIClient:
         Returns:
             Contenido del archivo como string, o None si no existe
         """
+        if self.enable_rate_limit:
+            self._rate_limiter.wait()
+        
         response = requests.get(api_url, headers=self.headers)
         
         if response.status_code == 200:

{commons_metrics-0.0.22.dist-info → commons_metrics-0.0.24.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: commons_metrics
-Version: 0.0.22
+Version: 0.0.24
 Summary: A simple library for basic statistical calculations
 Author: Bancolombia
 Author-email: omar.david.pino@email.com

{commons_metrics-0.0.22.dist-info → commons_metrics-0.0.24.dist-info}/RECORD

@@ -1,18 +1,18 @@
-commons_metrics/__init__.py,sha256=OApw2zEoANZrVtA4Ro-h9mOaeRFW26Tazz5d2rRThsY,796
-commons_metrics/azure_devops_client.py,sha256=sD130ggzZWUsNqkBVNrLH80_AN00CxQ4preP9sBdzHM,16778
+commons_metrics/__init__.py,sha256=xt4Tm9YIRbfAwr9oKzjoH6lNHX4LuDZJXJE7UIgeUqc,796
+commons_metrics/azure_devops_client.py,sha256=XrGqhr4voFUtGkdB27tnzsMSiNdpfY4stQUU51e1a0U,16755
 commons_metrics/cache_manager.py,sha256=HOeup9twUizjJAbh1MNXdPT8BMVeLFoolOWlAzMTXkE,2651
 commons_metrics/commons_repos_client.py,sha256=PiAMLWuDnI8AlZzE3sfQ3s2P23UrYbbqaq63AFRroHc,4695
 commons_metrics/database.py,sha256=wzcrK8q09J28NMLExkhWy8w7N2fQPkckrGf62C9rdQ0,1989
 commons_metrics/date_utils.py,sha256=8465712QJDGcshqry97Gi90lbMEbvbX3uiuHRVwGHbE,2654
-commons_metrics/github_api_client.py,sha256=yPLsqn_KvQDIXcNp7Ma8qQwWy8svpXNLpDX69vwL9BU,11465
+commons_metrics/github_api_client.py,sha256=2gynYtEzPxDETX-KRshCHLvP4AsRceSrRd-7VWSfUNc,13200
 commons_metrics/repositories.py,sha256=47JK9rhcpx_X6RWRkM3768qTwk39ODm8LLvi6QzZDdQ,9480
 commons_metrics/s3_file_manager.py,sha256=Mm7vlPJeXB46LnCXFs9oz7PQezcIcgYLlSJqMzVf72g,3384
 commons_metrics/text_simplifier.py,sha256=jRYckQ5APR0A3wY4hg70kOHxHUFuGzd6D-CJC3__j78,1518
 commons_metrics/update_design_components.py,sha256=QpY0GCCCMjdYOZ7b8oNigU9iTpiGx91CYsyWwN8WVDA,7660
 commons_metrics/util.py,sha256=98zuynalXumQRh-BB0Bcjyoh6vS2BTOUM8tVgr7iS9Q,1225
 commons_metrics/variable_finder.py,sha256=pxI_XSd-lq_AiUjDbcUC4knIZhWZwt7HQrQOa-F0ud4,8061
-commons_metrics-0.0.22.dist-info/licenses/LICENSE,sha256=jsHZ2Sh1wCL74HC25pDDGXCyQ0xgsTAy62FvEnehKIg,1067
-commons_metrics-0.0.22.dist-info/METADATA,sha256=ayPIDGEVQgYsQySmNneULcEyQ42-bejwGFV6HGJ_q0I,402
-commons_metrics-0.0.22.dist-info/WHEEL,sha256=_zCd3N1l69ArxyTb8rzEoP9TpbYXkqRFSNOD5OuxnTs,91
-commons_metrics-0.0.22.dist-info/top_level.txt,sha256=lheUN-3OKdU3A8Tg8Y-1IEB_9i_vVRA0g_FOiUsTQz8,16
-commons_metrics-0.0.22.dist-info/RECORD,,
+commons_metrics-0.0.24.dist-info/licenses/LICENSE,sha256=jsHZ2Sh1wCL74HC25pDDGXCyQ0xgsTAy62FvEnehKIg,1067
+commons_metrics-0.0.24.dist-info/METADATA,sha256=urWGuJg84vEuiiFTjw7ArflhFL4SC8hDLsiutYUIJqc,402
+commons_metrics-0.0.24.dist-info/WHEEL,sha256=_zCd3N1l69ArxyTb8rzEoP9TpbYXkqRFSNOD5OuxnTs,91
+commons_metrics-0.0.24.dist-info/top_level.txt,sha256=lheUN-3OKdU3A8Tg8Y-1IEB_9i_vVRA0g_FOiUsTQz8,16
+commons_metrics-0.0.24.dist-info/RECORD,,