commons-metrics 0.0.22__tar.gz → 0.0.24__tar.gz

{commons_metrics-0.0.22 → commons_metrics-0.0.24}/PKG-INFO

@@ -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 → commons_metrics-0.0.24}/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-0.0.22 → commons_metrics-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-0.0.22 → commons_metrics-0.0.24}/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 → commons_metrics-0.0.24}/commons_metrics.egg-info/PKG-INFO

@@ -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 → commons_metrics-0.0.24}/setup.py

@@ -2,7 +2,7 @@ from setuptools import setup, find_packages
 
 setup(
     name='commons_metrics',
-    version='0.0.22',
+    version='0.0.24',
     description='A simple library for basic statistical calculations',
     #long_description=open('USAGE.md').read(),
     #long_description_content_type='text/markdown',