recce-nightly 1.25.0.20251118__tar.gz → 1.28.0.20251208__tar.gz

{recce_nightly-1.25.0.20251118 → recce_nightly-1.28.0.20251208}/.gitignore

@@ -1,4 +1,5 @@
 venv
+.venv
 .idea/
 *.egg-info/
 __pycache__/
@@ -16,6 +17,3 @@ build
 .DS_Store
 
 recce.yml
-
-# Generated VERSION file for recce-cloud package
-recce_cloud/VERSION

{recce_nightly-1.25.0.20251118 → recce_nightly-1.28.0.20251208}/PKG-INFO

@@ -1,74 +1,63 @@
 Metadata-Version: 2.4
 Name: recce-nightly
-Version: 1.25.0.20251118
+Version: 1.28.0.20251208
 Summary: Environment diff tool for dbt
-Home-page: https://github.com/InfuseAI/recce
-Author: InfuseAI Dev Team
-Author-email: dev@infuseai.io
 Project-URL: Bug Tracker, https://github.com/InfuseAI/recce/issues
+Project-URL: Homepage, https://github.com/InfuseAI/recce
+Author-email: InfuseAI Dev Team <dev@infuseai.io>
+License: Apache-2.0
+License-File: LICENSE
+Classifier: Development Status :: 4 - Beta
+Classifier: License :: OSI Approved :: Apache Software License
+Classifier: Operating System :: OS Independent
 Classifier: Programming Language :: Python :: 3.9
 Classifier: Programming Language :: Python :: 3.10
 Classifier: Programming Language :: Python :: 3.11
 Classifier: Programming Language :: Python :: 3.12
 Classifier: Programming Language :: Python :: 3.13
-Classifier: License :: OSI Approved :: Apache Software License
-Classifier: Operating System :: OS Independent
-Classifier: Development Status :: 4 - Beta
 Requires-Python: >=3.9
-Description-Content-Type: text/markdown
-License-File: LICENSE
 Requires-Dist: boto3
-Requires-Dist: requests>=2.28.1
-Requires-Dist: ruamel.yaml>=0.18.6
 Requires-Dist: click>=7.1
 Requires-Dist: deepdiff<9.0,>=7.0
-Requires-Dist: portalocker
 Requires-Dist: fastapi
+Requires-Dist: gitpython
 Requires-Dist: itsdangerous
-Requires-Dist: uvicorn
-Requires-Dist: pydantic
 Requires-Dist: jinja2
+Requires-Dist: packaging
+Requires-Dist: portalocker
+Requires-Dist: py-markdown-table
+Requires-Dist: pydantic
+Requires-Dist: pygithub
+Requires-Dist: python-dateutil
+Requires-Dist: python-multipart
+Requires-Dist: pytz
 Requires-Dist: requests>=2.28.1
 Requires-Dist: rich>=12.0.0
+Requires-Dist: ruamel-yaml>=0.18.6
 Requires-Dist: sentry-sdk
+Requires-Dist: sqlglot<28.0,>=27.29.0
+Requires-Dist: uvicorn
 Requires-Dist: watchdog
 Requires-Dist: websockets
-Requires-Dist: py-markdown-table
-Requires-Dist: python-dateutil
-Requires-Dist: python-multipart
-Requires-Dist: GitPython
-Requires-Dist: PyGithub
-Requires-Dist: sqlglot<29
-Requires-Dist: pytz
-Requires-Dist: packaging
-Provides-Extra: mcp
-Requires-Dist: mcp>=1.0.0; extra == "mcp"
 Provides-Extra: dev
-Requires-Dist: pytest>=4.6; extra == "dev"
-Requires-Dist: pytest-asyncio; extra == "dev"
-Requires-Dist: pytest-flake8; extra == "dev"
-Requires-Dist: black>=25.1.0; extra == "dev"
-Requires-Dist: isort>=6.0.1; extra == "dev"
-Requires-Dist: flake8>=7.2.0; extra == "dev"
-Requires-Dist: pre-commit>=4.2.0; extra == "dev"
-Requires-Dist: pytest-mypy; extra == "dev"
-Requires-Dist: pytest-cov; extra == "dev"
-Requires-Dist: twine; extra == "dev"
-Requires-Dist: tox; extra == "dev"
-Requires-Dist: pandas; extra == "dev"
-Requires-Dist: httpx; extra == "dev"
-Dynamic: author
-Dynamic: author-email
-Dynamic: classifier
-Dynamic: description
-Dynamic: description-content-type
-Dynamic: home-page
-Dynamic: license-file
-Dynamic: project-url
-Dynamic: provides-extra
-Dynamic: requires-dist
-Dynamic: requires-python
-Dynamic: summary
+Requires-Dist: black>=25.1.0; extra == 'dev'
+Requires-Dist: dbt-duckdb; extra == 'dev'
+Requires-Dist: flake8>=7.2.0; extra == 'dev'
+Requires-Dist: httpx; extra == 'dev'
+Requires-Dist: isort>=6.0.1; extra == 'dev'
+Requires-Dist: pandas; extra == 'dev'
+Requires-Dist: pre-commit>=4.2.0; extra == 'dev'
+Requires-Dist: pytest-asyncio<1.0,>=0.23; extra == 'dev'
+Requires-Dist: pytest-cov; extra == 'dev'
+Requires-Dist: pytest-flake8; extra == 'dev'
+Requires-Dist: pytest-mypy; extra == 'dev'
+Requires-Dist: pytest>=4.6; extra == 'dev'
+Requires-Dist: tox; extra == 'dev'
+Requires-Dist: tox-uv; extra == 'dev'
+Requires-Dist: twine; extra == 'dev'
+Provides-Extra: mcp
+Requires-Dist: mcp>=1.0.0; (python_version >= '3.10') and extra == 'mcp'
+Description-Content-Type: text/markdown
 
 <p align="center">
     <a href="https://reccehq.com">

recce_nightly-1.28.0.20251208/pyproject.toml

@@ -0,0 +1,103 @@
+[project]
+name = "recce-nightly"
+dynamic = ["version"]
+description = "Environment diff tool for dbt"
+readme = "README.md"
+authors = [{name = "InfuseAI Dev Team", email = "dev@infuseai.io"}]
+requires-python = ">=3.9"
+license = {text = "Apache-2.0"}
+
+dependencies = [
+    "boto3",
+    "requests>=2.28.1",
+    "ruamel.yaml>=0.18.6",
+    "click>=7.1",
+    "deepdiff>=7.0,<9.0",
+    "portalocker",
+    "fastapi",
+    "itsdangerous",
+    "uvicorn",
+    "pydantic",
+    "jinja2",
+    "rich>=12.0.0",
+    "sentry-sdk",
+    "watchdog",
+    "websockets",
+    "py-markdown-table",
+    "python-dateutil",
+    "python-multipart",
+    "GitPython",
+    "PyGithub",
+    "sqlglot>=27.29.0,<28.0",
+    "pytz",
+    "packaging",
+]
+
+classifiers = [
+    "Programming Language :: Python :: 3.9",
+    "Programming Language :: Python :: 3.10",
+    "Programming Language :: Python :: 3.11",
+    "Programming Language :: Python :: 3.12",
+    "Programming Language :: Python :: 3.13",
+    "License :: OSI Approved :: Apache Software License",
+    "Operating System :: OS Independent",
+    "Development Status :: 4 - Beta",
+]
+
+[project.optional-dependencies]
+mcp = ["mcp>=1.0.0; python_version >= '3.10'"]
+dev = [
+    "pytest>=4.6",
+    "pytest-asyncio>=0.23,<1.0",
+    "pytest-flake8",
+    "black>=25.1.0",
+    "isort>=6.0.1",
+    "flake8>=7.2.0",
+    "pre-commit>=4.2.0",
+    "pytest-mypy",
+    "pytest-cov",
+    "twine",
+    "tox",
+    "tox-uv",
+    "pandas",
+    "httpx",
+    "dbt-duckdb",
+]
+
+[project.scripts]
+recce = "recce.cli:cli"
+
+[project.urls]
+"Bug Tracker" = "https://github.com/InfuseAI/recce/issues"
+"Homepage" = "https://github.com/InfuseAI/recce"
+
+[build-system]
+requires = ["hatchling"]
+build-backend = "hatchling.build"
+
+[tool.hatch.version]
+path = "recce/VERSION"
+pattern = "(?P<version>.+)"
+
+[tool.hatch.build.targets.sdist]
+include = [
+    "/recce",
+]
+
+[tool.hatch.build.targets.wheel]
+packages = ["recce"]
+
+[tool.uv.workspace]
+members = ["recce_cloud"]
+
+[tool.black]
+line-length = 120
+
+[tool.isort]
+profile = "black"
+
+[tool.pytest.ini_options]
+markers = [
+    "asyncio: mark test as an async test (used by pytest-asyncio)",
+]
+asyncio_default_fixture_loop_scope = "function"

recce_nightly-1.28.0.20251208/recce/VERSION

@@ -0,0 +1 @@
+1.28.0.20251208

{recce_nightly-1.25.0.20251118 → recce_nightly-1.28.0.20251208}/recce/apis/check_api.py

@@ -1,4 +1,3 @@
-from datetime import datetime, timezone
 from typing import Optional
 from uuid import UUID
 
@@ -153,22 +152,10 @@ class PatchCheckIn(BaseModel):
 
 @check_router.patch("/checks/{check_id}", status_code=200, response_model=CheckOut, response_model_exclude_none=True)
 async def update_check_handler(check_id: UUID, patch: PatchCheckIn, background_tasks: BackgroundTasks):
-    check = CheckDAO().find_check_by_id(check_id)
+    check = CheckDAO().update_check_by_id(check_id, patch)
     if check is None:
         raise HTTPException(status_code=404, detail="Not Found")
 
-    if patch.name is not None:
-        check.name = patch.name
-    if patch.description is not None:
-        check.description = patch.description
-    if patch.params is not None:
-        check.params = patch.params
-    if patch.view_options is not None:
-        check.view_options = patch.view_options
-    if patch.is_checked is not None:
-        check.is_checked = patch.is_checked
-    check.updated_at = datetime.now(timezone.utc).replace(microsecond=0)
-
     background_tasks.add_task(export_persistent_state)
     return CheckOut.from_check(check)
 
@@ -195,3 +182,22 @@ async def reorder_handler(order: ReorderChecksIn):
         CheckDAO().reorder(order.source, order.destination)
     except RecceException as e:
         raise HTTPException(status_code=400, detail=e.message)
+
+
+@check_router.post("/checks/{check_id}/mark-as-preset", status_code=204)
+async def mark_as_preset_check_handler(check_id: UUID, background_tasks: BackgroundTasks):
+    """
+    Mark an existing check as a preset check (cloud users only).
+
+    This creates a preset check from the specified check.
+    Only available for users with cloud mode enabled.
+
+    Returns:
+        204 No Content: Successfully marked check as preset
+        400 Bad Request: Error with detail message (e.g., not in cloud mode, check not found)
+    """
+    try:
+        CheckDAO().mark_as_preset_check(check_id)
+        background_tasks.add_task(export_persistent_state)
+    except RecceException as e:
+        raise HTTPException(status_code=400, detail=e.message)

recce_nightly-1.28.0.20251208/recce/apis/check_events_api.py

@@ -0,0 +1,353 @@
+"""
+Check Events API endpoints.
+
+This module provides REST endpoints for check events (timeline/conversation),
+proxying requests to Recce Cloud. This feature is only available for cloud users.
+"""
+
+import logging
+from typing import List, Optional
+from uuid import UUID
+
+from fastapi import APIRouter, HTTPException
+from pydantic import BaseModel
+
+from recce.core import default_context
+from recce.event import get_recce_api_token
+from recce.exceptions import RecceException
+from recce.util.cloud.check_events import CheckEventsCloud
+from recce.util.recce_cloud import RecceCloud, RecceCloudException
+
+logger = logging.getLogger("uvicorn")
+
+check_events_router = APIRouter(tags=["check_events"])
+
+
+# ============================================================================
+# Helper Functions
+# ============================================================================
+
+
+def _is_cloud_user() -> bool:
+    """Check if the current user is connected to Recce Cloud."""
+    ctx = default_context()
+    if ctx is None or ctx.state_loader is None:
+        return False
+    return hasattr(ctx.state_loader, "session_id") and ctx.state_loader.session_id is not None
+
+
+def _get_session_info() -> tuple:
+    """
+    Get organization ID, project ID, and session ID from state loader.
+
+    Returns:
+        tuple: (org_id, project_id, session_id)
+
+    Raises:
+        HTTPException: If not in cloud mode or session info unavailable
+    """
+    if not _is_cloud_user():
+        raise HTTPException(
+            status_code=400,
+            detail="Check events are only available when connected to Recce Cloud.",
+        )
+
+    ctx = default_context()
+    state_loader = ctx.state_loader
+
+    session_id = state_loader.session_id
+
+    # Check if org_id and project_id are cached
+    if hasattr(state_loader, "org_id") and hasattr(state_loader, "project_id"):
+        return state_loader.org_id, state_loader.project_id, session_id
+
+    # Fetch from cloud API
+    api_token = get_recce_api_token() or state_loader.token
+    if not api_token:
+        raise HTTPException(
+            status_code=401,
+            detail="Cannot access Recce Cloud: no API token available.",
+        )
+
+    try:
+        recce_cloud = RecceCloud(api_token)
+        session = recce_cloud.get_session(session_id)
+
+        org_id = session.get("org_id")
+        project_id = session.get("project_id")
+
+        if not org_id or not project_id:
+            raise HTTPException(
+                status_code=400,
+                detail=f"Session {session_id} does not belong to a valid organization or project.",
+            )
+
+        # Cache for future use
+        state_loader.org_id = org_id
+        state_loader.project_id = project_id
+
+        return org_id, project_id, session_id
+
+    except RecceCloudException as e:
+        logger.error(f"Failed to get session info: {e}")
+        raise HTTPException(status_code=e.status_code, detail=str(e.reason))
+
+
+def _get_events_client() -> CheckEventsCloud:
+    """
+    Get the CheckEventsCloud client.
+
+    Returns:
+        CheckEventsCloud: Cloud client for event operations
+
+    Raises:
+        HTTPException: If client cannot be initialized
+    """
+    ctx = default_context()
+    api_token = get_recce_api_token() or ctx.state_loader.token
+
+    if not api_token:
+        raise HTTPException(
+            status_code=401,
+            detail="Cannot access Recce Cloud: no API token available.",
+        )
+
+    return CheckEventsCloud(api_token)
+
+
+# ============================================================================
+# Pydantic Models
+# ============================================================================
+
+
+class CheckEventActorOut(BaseModel):
+    """Actor who performed the event."""
+
+    type: str  # "user", "recce_ai", "preset_system"
+    user_id: Optional[int] = None
+    login: Optional[str] = None
+    fullname: Optional[str] = None
+
+
+class CheckEventOut(BaseModel):
+    """Check event response model."""
+
+    id: str
+    check_id: str
+    event_type: str
+    actor: CheckEventActorOut
+    content: Optional[str] = None
+    old_value: Optional[str] = None
+    new_value: Optional[str] = None
+    is_edited: bool = False
+    is_deleted: bool = False
+    created_at: str
+    updated_at: str
+
+
+class CreateCommentIn(BaseModel):
+    """Request body for creating a comment."""
+
+    content: str
+
+
+class UpdateCommentIn(BaseModel):
+    """Request body for updating a comment."""
+
+    content: str
+
+
+# ============================================================================
+# API Endpoints
+# ============================================================================
+
+
+@check_events_router.get(
+    "/checks/{check_id}/events",
+    status_code=200,
+    response_model=List[CheckEventOut],
+)
+async def list_check_events(check_id: UUID):
+    """
+    List all events for a check in chronological order.
+
+    This endpoint returns all events (comments, state changes, etc.) for the
+    specified check. Events are returned in chronological order (oldest first).
+
+    Args:
+        check_id: The check ID
+
+    Returns:
+        List of CheckEventOut objects
+
+    Raises:
+        400: Not connected to Recce Cloud
+        401: No API token available
+        404: Check not found
+    """
+    try:
+        org_id, project_id, session_id = _get_session_info()
+        client = _get_events_client()
+
+        events = client.list_events(org_id, project_id, session_id, str(check_id))
+        return events
+
+    except RecceCloudException as e:
+        logger.error(f"Failed to list check events: {e}")
+        raise HTTPException(status_code=e.status_code, detail=str(e.reason))
+    except RecceException as e:
+        logger.error(f"Failed to list check events: {e}")
+        raise HTTPException(status_code=400, detail=str(e))
+
+
+@check_events_router.get(
+    "/checks/{check_id}/events/{event_id}",
+    status_code=200,
+    response_model=CheckEventOut,
+)
+async def get_check_event(check_id: UUID, event_id: UUID):
+    """
+    Get a specific event by ID.
+
+    Args:
+        check_id: The check ID
+        event_id: The event ID
+
+    Returns:
+        CheckEventOut object
+
+    Raises:
+        400: Not connected to Recce Cloud
+        401: No API token available
+        404: Event not found
+    """
+    try:
+        org_id, project_id, session_id = _get_session_info()
+        client = _get_events_client()
+
+        event = client.get_event(org_id, project_id, session_id, str(check_id), str(event_id))
+        return event
+
+    except RecceCloudException as e:
+        logger.error(f"Failed to get check event: {e}")
+        raise HTTPException(status_code=e.status_code, detail=str(e.reason))
+    except RecceException as e:
+        logger.error(f"Failed to get check event: {e}")
+        raise HTTPException(status_code=400, detail=str(e))
+
+
+@check_events_router.post(
+    "/checks/{check_id}/events",
+    status_code=201,
+    response_model=CheckEventOut,
+)
+async def create_comment(check_id: UUID, body: CreateCommentIn):
+    """
+    Create a new comment on a check.
+
+    Args:
+        check_id: The check ID
+        body: Request body containing comment content
+
+    Returns:
+        Created CheckEventOut object
+
+    Raises:
+        400: Not connected to Recce Cloud or invalid content
+        401: No API token available
+        404: Check not found
+    """
+    if not body.content or not body.content.strip():
+        raise HTTPException(status_code=400, detail="Comment content cannot be empty.")
+
+    try:
+        org_id, project_id, session_id = _get_session_info()
+        client = _get_events_client()
+
+        event = client.create_comment(org_id, project_id, session_id, str(check_id), body.content)
+        return event
+
+    except RecceCloudException as e:
+        logger.error(f"Failed to create comment: {e}")
+        raise HTTPException(status_code=e.status_code, detail=str(e.reason))
+    except RecceException as e:
+        logger.error(f"Failed to create comment: {e}")
+        raise HTTPException(status_code=400, detail=str(e))
+
+
+@check_events_router.patch(
+    "/checks/{check_id}/events/{event_id}",
+    status_code=200,
+    response_model=CheckEventOut,
+)
+async def update_comment(check_id: UUID, event_id: UUID, body: UpdateCommentIn):
+    """
+    Update an existing comment.
+
+    Only the author or an admin can update a comment.
+
+    Args:
+        check_id: The check ID
+        event_id: The event ID of the comment to update
+        body: Request body containing new comment content
+
+    Returns:
+        Updated CheckEventOut object
+
+    Raises:
+        400: Not connected to Recce Cloud or invalid content
+        401: No API token available
+        403: Not authorized to update this comment
+        404: Comment not found
+    """
+    if not body.content or not body.content.strip():
+        raise HTTPException(status_code=400, detail="Comment content cannot be empty.")
+
+    try:
+        org_id, project_id, session_id = _get_session_info()
+        client = _get_events_client()
+
+        event = client.update_comment(org_id, project_id, session_id, str(check_id), str(event_id), body.content)
+        return event
+
+    except RecceCloudException as e:
+        logger.error(f"Failed to update comment: {e}")
+        raise HTTPException(status_code=e.status_code, detail=str(e.reason))
+    except RecceException as e:
+        logger.error(f"Failed to update comment: {e}")
+        raise HTTPException(status_code=400, detail=str(e))
+
+
+@check_events_router.delete(
+    "/checks/{check_id}/events/{event_id}",
+    status_code=204,
+)
+async def delete_comment(check_id: UUID, event_id: UUID):
+    """
+    Delete a comment (soft delete).
+
+    Only the author or an admin can delete a comment. The comment will be
+    marked as deleted but remain in the timeline with a "Comment deleted" indicator.
+
+    Args:
+        check_id: The check ID
+        event_id: The event ID of the comment to delete
+
+    Raises:
+        400: Not connected to Recce Cloud
+        401: No API token available
+        403: Not authorized to delete this comment
+        404: Comment not found
+    """
+    try:
+        org_id, project_id, session_id = _get_session_info()
+        client = _get_events_client()
+
+        client.delete_comment(org_id, project_id, session_id, str(check_id), str(event_id))
+
+    except RecceCloudException as e:
+        logger.error(f"Failed to delete comment: {e}")
+        raise HTTPException(status_code=e.status_code, detail=str(e.reason))
+    except RecceException as e:
+        logger.error(f"Failed to delete comment: {e}")
+        raise HTTPException(status_code=400, detail=str(e))

{recce_nightly-1.25.0.20251118 → recce_nightly-1.28.0.20251208}/recce/apis/check_func.py

@@ -86,10 +86,10 @@ def create_check_from_run(
         is_preset=is_preset,
         is_checked=is_checked,
     )
-    CheckDAO().create(check)
-    run.check_id = check.check_id
+    new_check = CheckDAO().create(check)
+    run.check_id = new_check.check_id
 
-    return check
+    return new_check
 
 
 def create_check_without_run(
@@ -105,8 +105,8 @@ def create_check_without_run(
         is_preset=is_preset,
         is_checked=is_checked,
     )
-    CheckDAO().create(check)
-    return check
+    new_check = CheckDAO().create(check)
+    return new_check
 
 
 def purge_preset_checks():