mkdocs2confluence 0.7.25__tar.gz → 0.7.27__tar.gz

{mkdocs2confluence-0.7.25 → mkdocs2confluence-0.7.27}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: mkdocs2confluence
-Version: 0.7.25
+Version: 0.7.27
 Summary: Publish MkDocs Material pages to Confluence Cloud — admonitions, Mermaid diagrams, tabs, page properties and more
 Author: Anders Hybertz
 License: GPL-3.0-or-later

{mkdocs2confluence-0.7.25 → mkdocs2confluence-0.7.27}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "mkdocs2confluence"
-version = "0.7.25"
+version = "0.7.27"
 description = "Publish MkDocs Material pages to Confluence Cloud — admonitions, Mermaid diagrams, tabs, page properties and more"
 readme = "README.md"
 license = { text = "GPL-3.0-or-later" }

{mkdocs2confluence-0.7.25 → mkdocs2confluence-0.7.27}/src/mkdocs2confluence.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: mkdocs2confluence
-Version: 0.7.25
+Version: 0.7.27
 Summary: Publish MkDocs Material pages to Confluence Cloud — admonitions, Mermaid diagrams, tabs, page properties and more
 Author: Anders Hybertz
 License: GPL-3.0-or-later

{mkdocs2confluence-0.7.25 → mkdocs2confluence-0.7.27}/src/mkdocs_to_confluence/publisher/client.py

@@ -380,6 +380,9 @@ class ConfluenceClient:
         # Fetch space states once per run (keyed by space so one run = one API call)
         if cache_key not in self._space_states:
             self._space_states[cache_key] = self._fetch_available_states(page_id)
+            print(
+                f"  [status] space states fetched: {[s.get('name') for s in self._space_states[cache_key]]}",
+            )
 
         matched: dict[str, Any] | None = None
         for state in self._space_states.get(cache_key, []):
@@ -393,6 +396,7 @@ class ConfluenceClient:
                 "name": matched["name"],
                 "color": matched["color"],
             }
+            print(f"  [status] matched space state: {body}")
         else:
             # Fall back: name + color (required together when no id)
             _default_colors = {
@@ -404,9 +408,13 @@ class ConfluenceClient:
                 "outdated": "#ff7452",
             }
             body = {"name": name, "color": _default_colors.get(_normalize(name), "#2684ff")}
+            print(f"  [status] no space state match — using fallback: {body}")
 
         url = self._v1(f"/content/{page_id}/state")
         resp = self._http.put(url, json=body)
+        print(f"  [status] PUT {url} → HTTP {resp.status_code}")
+        if not resp.is_success:
+            print(f"  [status] response body: {resp.text[:300]}")
         self._raise_for_status(resp, f"set_page_status({page_id!r}, {status_key!r})")
 
     def _fetch_available_states(self, page_id: str) -> list[dict[str, Any]]:
@@ -419,7 +427,8 @@ class ConfluenceClient:
         resp = self._http.get(self._v1(f"/content/{page_id}/state/available"))
         if resp.is_success:
             data: dict[str, Any] = resp.json()
-            return data.get("spaceContentStates") or []
+            return list(data.get("spaceContentStates") or [])
+        print(f"  [warn] could not fetch available states (HTTP {resp.status_code}) — using fallback")
         return []
 
     def list_attachments(self, page_id: str) -> dict[str, dict[str, Any]]:

{mkdocs2confluence-0.7.25 → mkdocs2confluence-0.7.27}/src/mkdocs_to_confluence/publisher/pipeline.py

@@ -664,10 +664,12 @@ def _post_process_action(
     # Set Confluence page status (rough-draft / in-progress / etc.) — non-fatal.
     if action.page_id and action.confluence_status and not action.is_folder:
         try:
+            print(f"  [status] setting '{action.confluence_status}' on page {action.page_id!r}...")
             client.set_page_status(action.page_id, action.confluence_status, space_key=space_key)
+            print("  [status] ok")
         except Exception as exc:
             # Always print status errors — user configured status explicitly
-            print(f"  [warn] could not set page status '{action.confluence_status}': {exc}", file=sys.stderr)
+            print(f"  [warn] could not set page status '{action.confluence_status}': {exc}")
 
     # Upload assets — skip files whose mtime is not newer than Confluence.
     if action.page_id and action.attachments:
@@ -730,13 +732,15 @@ def execute_publish(
             # Still apply status even for unchanged pages.
             if action.page_id and action.confluence_status and not action.is_folder:
                 try:
+                    print(
+                        f"  [status] setting '{action.confluence_status}' on page {action.page_id!r}...",
+                    )
                     client.set_page_status(action.page_id, action.confluence_status, space_key=space_key)
+                    print("  [status] ok")
                 except Exception as exc:
-                    if not quiet:
-                        print(
-                            f"  [warn] could not set page status '{action.confluence_status}': {exc}",
-                            file=sys.stderr,
-                        )
+                    print(
+                        f"  [warn] could not set page status '{action.confluence_status}': {exc}",
+                    )
             continue
 
         counter += 1

{mkdocs2confluence-0.7.25 → mkdocs2confluence-0.7.27}/tests/test_publish_pipeline.py

@@ -1945,3 +1945,100 @@ def test_execute_publish_quiet_suppresses_stdout(tmp_path: Path, capsys: pytest.
 
     out, _ = capsys.readouterr()
     assert out == ""
+
+
+# ── Status: end-to-end flow ───────────────────────────────────────────────────
+
+
+def test_compile_page_returns_confluence_status(tmp_path: Path) -> None:
+    """compile_page must return the status: value from front matter."""
+    docs = tmp_path / "docs"
+    docs.mkdir()
+    md = docs / "page.md"
+    md.write_text("---\nstatus: in-progress\n---\n\n# My Page\n\nContent.\n", encoding="utf-8")
+
+    node = _page_node("My Page", md)
+    config = _make_config(docs)
+    _, _, _, confluence_status = compile_page(node, config)
+
+    assert confluence_status == "in-progress"
+
+
+def test_plan_publish_sets_confluence_status_on_create(tmp_path: Path) -> None:
+    """plan_publish must carry confluence_status into a create PageAction."""
+    docs = tmp_path / "docs"
+    docs.mkdir()
+    md = docs / "page.md"
+    md.write_text("---\nstatus: in-progress\n---\n\n# My Page\n\nContent.\n", encoding="utf-8")
+
+    node = _page_node("My Page", md)
+    config = _make_config(docs)
+    conf_config = _make_conf_config()
+
+    client = MagicMock()
+    client.find_page.return_value = None
+
+    plan = plan_publish([node], client, config, conf_config, space_id="42")
+
+    assert plan[0].action == "create"
+    assert plan[0].confluence_status == "in-progress"
+
+
+def test_plan_publish_sets_confluence_status_on_skip(tmp_path: Path) -> None:
+    """plan_publish must carry confluence_status into a skip (unchanged) PageAction."""
+    from mkdocs_to_confluence.publisher.pipeline import _xhtml_hash
+
+    docs = tmp_path / "docs"
+    docs.mkdir()
+    md = docs / "page.md"
+    md.write_text("---\nstatus: in-progress\n---\n\n# My Page\n\nContent.\n", encoding="utf-8")
+
+    node = _page_node("My Page", md)
+    config = _make_config(docs)
+    conf_config = _make_conf_config()
+
+    xhtml, _, _, _ = compile_page(node, config)
+    stored_hash = _xhtml_hash(xhtml)
+
+    existing_page = {"id": "77", "version": {"number": 2}}
+    client = MagicMock()
+    client.find_page.return_value = existing_page
+    client.get_content_hash.return_value = stored_hash
+
+    plan = plan_publish([node], client, config, conf_config, space_id="42")
+
+    assert plan[0].action == "skip"
+    assert plan[0].confluence_status == "in-progress"
+
+
+def test_execute_publish_calls_set_page_status_on_create(tmp_path: Path) -> None:
+    """execute_publish must call client.set_page_status for a created page with confluence_status."""
+    from mkdocs_to_confluence.publisher.pipeline import execute_publish
+
+    node = _make_section_node("P", [])
+    action = PageAction(
+        node=node, title="P", action="create",
+        parent_id=None, xhtml="<p/>", confluence_status="in-progress",
+    )
+    client = MagicMock()
+    client.create_page.return_value = {"id": "99"}
+
+    execute_publish([action], client, space_id="42", docs_dir=tmp_path)
+
+    client.set_page_status.assert_called_once_with("99", "in-progress", space_key=None)
+
+
+def test_execute_publish_calls_set_page_status_on_skip(tmp_path: Path) -> None:
+    """execute_publish must call client.set_page_status even for skipped (unchanged) pages."""
+    from mkdocs_to_confluence.publisher.pipeline import execute_publish
+
+    node = _make_section_node("P", [])
+    action = PageAction(
+        node=node, title="P", action="skip",
+        parent_id=None, page_id="88", confluence_status="in-progress",
+    )
+    client = MagicMock()
+
+    execute_publish([action], client, space_id="42", docs_dir=tmp_path)
+
+    client.set_page_status.assert_called_once_with("88", "in-progress", space_key=None)