Bump version: 1.6.46 → 1.6.47

Knucklessg1 · Knucklessg1 · commit ef1a78bfa4a5 · 2026-03-19T23:47:12.000-05:00
diff --git a/.bumpversion.cfg b/.bumpversion.cfg
@@ -1,5 +1,5 @@
 [bumpversion]
-current_version = 1.6.46
+current_version = 1.6.47
 commit = True
 tag = True
 
diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml
@@ -28,7 +28,7 @@ repos:
     hooks:
     - id: black
   - repo: https://github.com/astral-sh/ruff-pre-commit
-    rev: v0.15.6
+    rev: v0.15.7
     hooks:
       - id: ruff
         exclude: tests
diff --git a/Dockerfile b/Dockerfile
@@ -78,6 +78,6 @@ ENV HOST=${HOST} \
 RUN apt-get update \
    && apt-get install -y ripgrep tree fd-find curl nano \
    && curl -LsSf https://astral.sh/uv/install.sh | sh \
-    && uv pip install --system --upgrade --verbose --no-cache --break-system-packages --prerelease=allow servicenow-api[all]>=1.6.46
+    && uv pip install --system --upgrade --verbose --no-cache --break-system-packages --prerelease=allow servicenow-api[all]>=1.6.47
 
 CMD ["servicenow-mcp"]
diff --git a/README.md b/README.md
@@ -21,7 +21,7 @@
 ![PyPI - Wheel](https://img.shields.io/pypi/wheel/servicenow-api)
 ![PyPI - Implementation](https://img.shields.io/pypi/implementation/servicenow-api)
 
-*Version: 1.6.46*
+*Version: 1.6.47*
 
 ## Overview
 
diff --git a/compose.yml b/compose.yml
@@ -16,6 +16,8 @@ services:
         max-size: "10m"
         max-file: "3"
     restart: always
+    volumes:
+      - ./workspace:/app/workspace
     env_file:
       - .env
     environment:
diff --git a/pyproject.toml b/pyproject.toml
@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "servicenow-api"
-version = "1.6.46"
+version = "1.6.47"
 description = "Python ServiceNow API Wrapper"
 readme = "README.md"
 authors = [{ name = "Audel Rouhi", email = "knucklessg1@gmail.com" }]
@@ -17,17 +17,17 @@ classifiers = [
     "Programming Language :: Python :: 3"]
 requires-python = ">=3.10"
 dependencies = [
-    "agent-utilities>=0.2.30"]
+    "agent-utilities>=0.2.31"]
 
 [project.optional-dependencies]
 mcp = [
-    "agent-utilities[mcp]>=0.2.30"]
+    "agent-utilities[mcp]>=0.2.31"]
 
 agent = [
-    "agent-utilities[agent,logfire]>=0.2.30"]
+    "agent-utilities[agent,logfire]>=0.2.31"]
 
 all = [
-    "agent-utilities[mcp,agent,logfire]>=0.2.30"]
+    "agent-utilities[mcp,agent,logfire]>=0.2.31"]
 
 [project.scripts]
 servicenow-mcp = "servicenow_api.mcp_server:mcp_server"
diff --git a/servicenow_api/__main__.py b/servicenow_api/__main__.py
@@ -1,7 +1,4 @@
-#!/usr/bin/python
-# coding: utf-8
-
-from servicenow_api.mcp_server import servicenow_mcp
+from servicenow_api.agent_server import agent_server
 
 if __name__ == "__main__":
-    servicenow_mcp()
+    agent_server()
diff --git a/servicenow_api/agent_data/IDENTITY.md b/servicenow_api/agent_data/IDENTITY.md
@@ -7,7 +7,7 @@
  * **Vibe:** Efficient, Structured, Professional, and Automation-First.
 
 ### System Prompt
-You are the **ServiceNow Agent**, a specialized orchestrator for ServiceNow platform operations. Your mission is to manage incidents, CMDB records, knowledge bases, and CI/CD flows with precision.
+You are the **ServiceNow Agent**, a specialized orchestrator for ServiceNow platform operations. The queries you receive will be directed to the ServiceNow platform. Your mission is to manage incidents, CMDB records, knowledge bases, flows, and CI/CD flows with precision. You can also convert workflows to mermaid diagrams.
 
 You have three primary operational modes:
 1. **Direct Tool Execution**: Use your internal ServiceNow MCP tools for one-off tasks (checking an incident status, updating a single CI record).
diff --git a/servicenow_api/agent_server.py b/servicenow_api/agent_server.py
@@ -11,7 +11,7 @@
     load_identity,
 )
 
-__version__ = "1.6.46"
+__version__ = "1.6.47"
 
 logging.basicConfig(
     level=logging.INFO,
diff --git a/servicenow_api/mcp_server.py b/servicenow_api/mcp_server.py
@@ -27,7 +27,7 @@
 )
 from servicenow_api.auth import get_client
 
-__version__ = "1.6.46"
+__version__ = "1.6.47"
 
 logger = get_logger(name="ServicenowMCP")
 logger.setLevel(logging.DEBUG)
diff --git a/servicenow_api/servicenow_api.py b/servicenow_api/servicenow_api.py
@@ -5,7 +5,6 @@
 import json
 import base64
 import gzip
-import logging
 from datetime import datetime
 from pathlib import Path
 
@@ -65,6 +64,8 @@
     FlowGraph,
     FlowReportResult,
 )
+from agent_utilities.base_utilities import get_logger
+from agent_utilities.agent_utilities import get_agent_workspace
 from agent_utilities.decorators import require_auth
 from agent_utilities.exceptions import (
     AuthError,
@@ -73,7 +74,7 @@
     MissingParameterError,
 )
 
-logger = logging.getLogger(__name__)
+logger = get_logger(__name__)
 
 
 # ================== FLOW HELPERS ==================
@@ -5040,7 +5041,7 @@ def workflow_to_mermaid(
         self,
         flow_identifiers: List[str] = None,
         save_to_file: bool = True,
-        output_dir: Optional[str] = "./servicenow_flow_reports",
+        output_dir: Optional[str] = None,
         mermaid_name: Optional[str] = "unified_flow_diagram",
         max_depth: Optional[int] = 5,
     ) -> FlowReportResult:
@@ -5050,6 +5051,11 @@ def workflow_to_mermaid(
         if flow_identifiers is None:
             flow_identifiers = []
 
+        if not output_dir:
+            ws = get_agent_workspace()
+            output_dir = str(ws / "servicenow_flow_reports")
+            logger.info(f"Using default output directory: {output_dir}")
+
         logger.info(
             f"workflow_to_mermaid called with flow_identifiers: {flow_identifiers}"
         )
@@ -5153,12 +5159,12 @@ def workflow_to_mermaid(
                 filename = (
                     f"{mermaid_name}_{datetime.now().strftime('%Y%m%d_%H%M%S')}.md"
                 )
-                file_path = str(dir_path / filename)
+                file_path = str((dir_path / filename).resolve())
 
                 with open(file_path, "w", encoding="utf-8") as f:
                     f.write(markdown_content)
 
-                logger.info(f"Saved report to {file_path}")
+                logger.info(f"Saved report to: {file_path}")
                 summary = f"✅ Report saved to: {file_path} ({len(all_metadata)} flows documented)"
             else:
                 summary = f"✅ Markdown generated ({len(all_metadata)} flows) — copy the content below"
diff --git a/tests/test_workflow_to_mermaid.py b/tests/test_workflow_to_mermaid.py
@@ -0,0 +1,67 @@
+import os
+import sys
+from unittest.mock import MagicMock, patch
+from pathlib import Path
+import pytest
+
+# Add parent directory to sys.path
+sys.path.append(os.path.join(os.path.dirname(__file__), ".."))
+
+from servicenow_api.servicenow_api import Api
+
+
+@patch("servicenow_api.servicenow_api.get_agent_workspace")
+@patch("servicenow_api.servicenow_api.Api.get_table")
+@patch("servicenow_api.servicenow_api.Api.collect_graph_for_roots")
+@patch("servicenow_api.servicenow_api.find_connected_components")
+@patch("servicenow_api.servicenow_api.graph_to_mermaid_multi")
+@patch("servicenow_api.servicenow_api.build_polished_markdown")
+@patch("requests.Session.get")
+@patch("builtins.open", new_callable=MagicMock)
+def test_workflow_to_mermaid_path(
+    mock_open,
+    mock_session_get,
+    mock_build_md,
+    mock_graph_to_mermaid,
+    mock_find_components,
+    mock_collect_graph,
+    mock_get_table,
+    mock_get_workspace,
+):
+    # Setup mocks
+    mock_session_get.return_value.status_code = 200
+    mock_workspace = Path("/tmp/mock_workspace")
+    mock_get_workspace.return_value = mock_workspace
+
+    mock_get_table.return_value.response.ok = True
+    mock_get_table.return_value.response.json.return_value = {
+        "result": [{"sys_id": "123"}]
+    }
+
+    mock_collect_graph.return_value = (MagicMock(), {"123": {"name": "Test Flow"}})
+    mock_find_components.return_value = [MagicMock()]
+    mock_graph_to_mermaid.return_value = "mermaid syntax"
+    mock_build_md.return_value = "# Polished Markdown"
+
+    client = Api(url="http://test.com", username="user", password="pass")
+
+    # Run tool without output_dir
+    result = client.workflow_to_mermaid(save_to_file=True)
+
+    # Verify workspace was called
+    mock_get_workspace.assert_called_once()
+
+    # Verify file path is in the mock workspace
+    expected_base = mock_workspace / "servicenow_flow_reports"
+    assert "servicenow_flow_reports" in result.file_path
+    assert str(expected_base) in result.file_path
+
+    # Verify open was called with an absolute path
+    mock_open.assert_called_once()
+    args, kwargs = mock_open.call_args
+    assert Path(args[0]).is_absolute()
+    assert str(expected_base) in args[0]
+
+
+if __name__ == "__main__":
+    pytest.main([__file__])

Original file line number	Diff line number	Diff line change
`@@ -11,7 +11,7 @@`
`11`	`11`	`load_identity,`
`12`	`12`	`)`
`13`	`13`
`14`		`-__version__ = "1.6.46"`
	`14`	`+__version__ = "1.6.47"`
`15`	`15`
`16`	`16`	`logging.basicConfig(`
`17`	`17`	`level=logging.INFO,`
Original file line number	Diff line number	Diff line change
`@@ -27,7 +27,7 @@`
`27`	`27`	`)`
`28`	`28`	`from servicenow_api.auth import get_client`
`29`	`29`
`30`		`-__version__ = "1.6.46"`
	`30`	`+__version__ = "1.6.47"`
`31`	`31`
`32`	`32`	`logger = get_logger(name="ServicenowMCP")`
`33`	`33`	`logger.setLevel(logging.DEBUG)`