cicaddy-github 0.2.0__tar.gz → 0.3.0__tar.gz

{cicaddy_github-0.2.0 → cicaddy_github-0.3.0}/Dockerfile

@@ -1,4 +1,4 @@
-FROM python:3.12-slim
+FROM python:3.14-slim
 
 WORKDIR /app
 

{cicaddy_github-0.2.0 → cicaddy_github-0.3.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: cicaddy-github
-Version: 0.2.0
+Version: 0.3.0
 Summary: GitHub Actions plugin for cicaddy AI agent framework
 Project-URL: Homepage, https://github.com/redhat-community-ai-tools/cicaddy-action
 Project-URL: Repository, https://github.com/redhat-community-ai-tools/cicaddy-action.git

{cicaddy_github-0.2.0 → cicaddy_github-0.3.0}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "cicaddy-github"
-version = "0.2.0"
+version = "0.3.0"
 description = "GitHub Actions plugin for cicaddy AI agent framework"
 readme = "README.md"
 requires-python = ">=3.11"

{cicaddy_github-0.2.0 → cicaddy_github-0.3.0}/src/cicaddy_github/github_integration/agents.py

@@ -43,6 +43,26 @@ def dedent_code_blocks(text: str) -> str:
     return _FENCED_CODE_BLOCK.sub(_dedent, text)
 
 
+# Matches output wrapped in a single ```markdown ... ``` fence.
+_MARKDOWN_WRAPPER = re.compile(
+    r"^\s*```(?:markdown|md)\s*\n(.*?)\n\s*```\s*$",
+    re.DOTALL | re.IGNORECASE,
+)
+
+
+def strip_markdown_wrapper(text: str) -> str:
+    """Strip a wrapping ```markdown fence from the entire AI output.
+
+    Some models interpret ``output_format: markdown`` as "wrap the response
+    in a markdown code fence", which causes GitHub to render the comment as
+    a literal code block instead of formatted markdown.
+    """
+    m = _MARKDOWN_WRAPPER.match(text.strip())
+    if m:
+        return m.group(1)
+    return text
+
+
 class GitHubTaskAgent(BaseAIAgent):
     """AI Agent for scheduled tasks and changelog generation."""
 
@@ -280,7 +300,8 @@ Please provide your comprehensive analysis in markdown format.
         comment = f"{BOT_COMMENT_MARKER_PR_REVIEW}\n"
 
         if "ai_analysis" in analysis_result:
-            comment += dedent_code_blocks(analysis_result["ai_analysis"]) + "\n"
+            cleaned = strip_markdown_wrapper(analysis_result["ai_analysis"])
+            comment += dedent_code_blocks(cleaned) + "\n"
 
         comment += (
             "\n<!-- cicaddy-footer -->\n---\n"

{cicaddy_github-0.2.0 → cicaddy_github-0.3.0}/tests/unit/test_agents.py

@@ -1,6 +1,6 @@
-"""Tests for dedent_code_blocks in agents module."""
+"""Tests for dedent_code_blocks and strip_markdown_wrapper in agents module."""
 
-from cicaddy_github.github_integration.agents import dedent_code_blocks
+from cicaddy_github.github_integration.agents import dedent_code_blocks, strip_markdown_wrapper
 
 
 class TestDedentCodeBlocks:
@@ -72,3 +72,42 @@ class TestDedentCodeBlocks:
         assert result.startswith("* Item one\n")
         assert result.endswith("* Item two")
         assert "+added line" in result
+
+
+class TestStripMarkdownWrapper:
+    """Test stripping wrapping ```markdown fences from AI output."""
+
+    def test_strips_markdown_wrapper(self):
+        """Output wrapped in ```markdown is unwrapped."""
+        text = "```markdown\n### Summary\nSome analysis.\n```"
+        result = strip_markdown_wrapper(text)
+        assert result == "### Summary\nSome analysis."
+
+    def test_strips_md_wrapper(self):
+        """Output wrapped in ```md is also unwrapped."""
+        text = "```md\n## Title\nContent.\n```"
+        result = strip_markdown_wrapper(text)
+        assert result == "## Title\nContent."
+
+    def test_strips_case_insensitive(self):
+        """Output wrapped in ```Markdown or ```MD is also unwrapped."""
+        for tag in ("Markdown", "MARKDOWN", "MD", "Md"):
+            text = f"```{tag}\nContent here.\n```"
+            result = strip_markdown_wrapper(text)
+            assert result == "Content here.", f"Failed for tag: {tag}"
+
+    def test_no_change_without_wrapper(self):
+        """Plain markdown without wrapper is unchanged."""
+        text = "### Summary\nSome analysis."
+        assert strip_markdown_wrapper(text) == text
+
+    def test_preserves_internal_code_blocks(self):
+        """Code blocks inside the markdown wrapper are preserved."""
+        text = "```markdown\n### Example\n```python\nprint('hi')\n```\n```"
+        result = strip_markdown_wrapper(text)
+        assert "```python\nprint('hi')\n```" in result
+
+    def test_no_change_for_non_markdown_fence(self):
+        """A ```python wrapper is NOT stripped."""
+        text = "```python\nprint('hi')\n```"
+        assert strip_markdown_wrapper(text) == text