pjdev-gitlab 5.1.12__tar.gz → 5.1.13__tar.gz

{pjdev_gitlab-5.1.12 → pjdev_gitlab-5.1.13}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: pjdev-gitlab
-Version: 5.1.12
+Version: 5.1.13
 Project-URL: Documentation, https://gitlab.purplejay.io/keystone/python/-/tree/main/pjdev-gitlab/README.md
 Project-URL: Issues, https://gitlab.purplejay.io/keystone/python/-/issues
 Project-URL: Source, https://gitlab.purplejay.io/keystone/python
@@ -234,6 +234,31 @@ async def main() -> None:
 asyncio.run(main())
 ```
 
+#### Custom statuses
+
+The workflow status an instance defines for itself (`Ready`, `In progress`, ...)
+is neither a label nor the open/closed `state`, and REST does not expose it.
+Reading it is opt-in, because GitLab ships the status widget as an experiment
+(**17.11+**) and selecting it against an older instance fails the whole query:
+
+```python
+# Just the status — one small query. Preferred on a hot path.
+status = await work_items_service.get_work_item_status(
+    42, project_path="my-group/my-project"
+)
+print(status.name if status else "no status set")
+
+# Or folded into a read you were making anyway.
+item = await work_items_service.get_work_item(
+    42, project_path="my-group/my-project", include_status=True
+)
+items = await work_items_service.search_work_items(
+    project_path="my-group/my-project", include_status=True
+)
+```
+
+Status names are configured per namespace — match them case-insensitively.
+
 ## Bundled agent skills
 
 Skill files for AI agents ship under `.agents/skills/` inside the installed package, following the [library-skills.io](https://library-skills.io/create/) convention. Topics: issues, workItems, merge requests, repository files, generic packages.

{pjdev_gitlab-5.1.12 → pjdev_gitlab-5.1.13}/README.md

@@ -206,6 +206,31 @@ async def main() -> None:
 asyncio.run(main())
 ```
 
+#### Custom statuses
+
+The workflow status an instance defines for itself (`Ready`, `In progress`, ...)
+is neither a label nor the open/closed `state`, and REST does not expose it.
+Reading it is opt-in, because GitLab ships the status widget as an experiment
+(**17.11+**) and selecting it against an older instance fails the whole query:
+
+```python
+# Just the status — one small query. Preferred on a hot path.
+status = await work_items_service.get_work_item_status(
+    42, project_path="my-group/my-project"
+)
+print(status.name if status else "no status set")
+
+# Or folded into a read you were making anyway.
+item = await work_items_service.get_work_item(
+    42, project_path="my-group/my-project", include_status=True
+)
+items = await work_items_service.search_work_items(
+    project_path="my-group/my-project", include_status=True
+)
+```
+
+Status names are configured per namespace — match them case-insensitively.
+
 ## Bundled agent skills
 
 Skill files for AI agents ship under `.agents/skills/` inside the installed package, following the [library-skills.io](https://library-skills.io/create/) convention. Topics: issues, workItems, merge requests, repository files, generic packages.

{pjdev_gitlab-5.1.12 → pjdev_gitlab-5.1.13}/src/pjdev_gitlab/.agents/skills/pjdev_gitlab_work_items/SKILL.md

@@ -87,7 +87,7 @@ across_group = await work_items_service.search_work_items(
 )
 ```
 
-## View / update status
+## Open/closed state
 
 ```python
 from pjdev_gitlab.models import WorkItemStateEvent
@@ -100,6 +100,39 @@ await work_items_service.set_work_item_state(
 )
 ```
 
+## Custom status
+
+Separate from open/closed: the workflow status an instance defines for itself
+(`Ready`, `In progress`, `Code Complete`, ...). It is **not** a label, and the
+REST API does not expose it at all — GraphQL's status widget is the only way to
+read it.
+
+Reading it is opt-in, because the widget is a GitLab *experiment* introduced in
+**17.11**: selecting it against an older instance fails the entire query, so it
+is never requested unless asked for.
+
+```python
+# Just the status — one small query, no discussions dragged along.
+# This is the right call on a hot path (e.g. per webhook event).
+status = await work_items_service.get_work_item_status(42, project_path="group/proj")
+print(status.name if status else "no status set")
+
+# Or alongside everything else, when you were fetching the item anyway.
+item = await work_items_service.get_work_item(
+    42, project_path="group/proj", include_status=True
+)
+print(item.status.name if item.status else "no status set")
+
+# Also available on searches.
+items = await work_items_service.search_work_items(
+    project_path="group/proj", include_status=True
+)
+```
+
+`None` means the item has no status assigned. Status names are configured per
+namespace, so **match them case-insensitively** rather than hard-coding an enum —
+`"Code Complete"` on one instance may be `"code complete"` on another.
+
 ## Comments and threaded replies
 
 ```python

{pjdev_gitlab-5.1.12 → pjdev_gitlab-5.1.13}/src/pjdev_gitlab/__about__.py

@@ -1,4 +1,4 @@
 # SPDX-FileCopyrightText: 2026-present Chris O'Neill <chris@purplejay.io>
 #
 # SPDX-License-Identifier: MIT
-__version__ = "5.1.12"
+__version__ = "5.1.13"

{pjdev_gitlab-5.1.12 → pjdev_gitlab-5.1.13}/src/pjdev_gitlab/models.py

@@ -265,6 +265,32 @@ class WorkItemIteration(GraphQLBase):
     web_url: Optional[str] = None
 
 
+class WorkItemStatus(GraphQLBase):
+    """A custom work-item status (``Ready``, ``In progress``, ``Code Complete``, ...).
+
+    Distinct from :class:`WorkItemState`, which is only ``OPEN``/``CLOSED``.
+    Statuses are defined per namespace, so the names are whatever the instance's
+    administrators configured — match them case-insensitively rather than
+    hard-coding an enum.
+
+    GitLab exposes this as an **experiment** (``WorkItemWidgetStatus``,
+    introduced in 17.11), which is why the widget is opt-in throughout
+    :mod:`pjdev_gitlab.work_items_service` — see ``include_status``. ``category``
+    and ``description`` are 18.1+ and are deliberately not selected, so the
+    queries here stay valid against 17.11.
+
+    Every field is optional: an instance may return a status object before it has
+    been fully configured, and a parse failure here would take down the whole
+    work-item read.
+    """
+
+    id: Optional[str] = None
+    name: Optional[str] = None
+    icon_name: Optional[str] = None
+    color: Optional[str] = None
+    position: Optional[int] = None
+
+
 class WorkItemNote(GraphQLBase):
     id: str
     body: str
@@ -316,6 +342,9 @@ class WorkItem(GraphQLBase):
     start_date: Optional[str] = None
     due_date: Optional[str] = None
     discussions: List[WorkItemDiscussion] = []
+    status: Optional[WorkItemStatus] = None
+    """Only populated when the read requested ``include_status=True``; ``None``
+    otherwise, which is indistinguishable from "this item has no status set"."""
 
     widgets: List[Dict[str, Any]] = []
 
@@ -345,6 +374,11 @@ class WorkItem(GraphQLBase):
             elif widget_type == "START_AND_DUE_DATE":
                 flat["start_date"] = widget.get("startDate")
                 flat["due_date"] = widget.get("dueDate")
+            elif widget_type == "STATUS":
+                # Only present when the query opted in via include_status; a work
+                # item with no status assigned still sends the widget, with a
+                # null status.
+                flat["status"] = widget.get("status")
             elif widget_type == "NOTES":
                 nodes = (widget.get("discussions") or {}).get("nodes") or []
                 flat["discussions"] = [

{pjdev_gitlab-5.1.12 → pjdev_gitlab-5.1.13}/src/pjdev_gitlab/work_items_service.py

@@ -31,6 +31,7 @@ from pjdev_gitlab.models import (
     WorkItemNote,
     WorkItemState,
     WorkItemStateEvent,
+    WorkItemStatus,
     is_global_id,
     make_label_gid,
     make_work_item_gid,
@@ -79,7 +80,18 @@ fragment LabelFields on Label {
 }
 """
 
-_WORK_ITEM_FRAGMENT = (
+# The status widget is a GitLab *experiment* (17.11+). Selecting an inline
+# fragment on a type the instance does not know about fails the whole query, so
+# this is spliced in only when a caller asks for it — an older instance keeps
+# working for every other field. `category`/`description` are 18.1+ and are left
+# out for the same reason.
+_STATUS_WIDGET_SELECTION = """
+    ... on WorkItemWidgetStatus {
+      status { id name iconName color position }
+    }
+"""
+
+_WORK_ITEM_FRAGMENT_TEMPLATE = (
     _USER_FRAGMENT
     + _NOTE_FRAGMENT
     + _LABEL_FRAGMENT
@@ -130,12 +142,20 @@ fragment WorkItemFields on WorkItem {
         }
       }
     }
+__STATUS_WIDGET__
   }
 }
 """
 )
 
 
+def _work_item_fragment(include_status: bool = False) -> str:
+    """Build the ``WorkItemFields`` fragment, optionally selecting the status widget."""
+    return _WORK_ITEM_FRAGMENT_TEMPLATE.replace(
+        "__STATUS_WIDGET__", _STATUS_WIDGET_SELECTION if include_status else ""
+    )
+
+
 # ---------------------------------------------------------------------------
 # Helpers
 # ---------------------------------------------------------------------------
@@ -213,6 +233,7 @@ async def get_work_item(
     *,
     project_path: Optional[ProjectId] = None,
     group_path: Optional[GroupId] = None,
+    include_status: bool = False,
     client: Optional[AsyncClient] = None,
 ) -> WorkItem:
     """Fetch a single workItem.
@@ -223,11 +244,17 @@ async def get_work_item(
       - A project-scoped ``iid`` (``42`` or ``"42"``) along with
         ``project_path`` or ``group_path`` — looked up via the namespace's
         ``workItems`` connection.
+
+    Set ``include_status=True`` to populate :attr:`WorkItem.status` with the
+    custom work-item status. It is off by default because the underlying widget
+    is an experiment that only exists on GitLab 17.11+, and asking for it on an
+    older instance fails the whole query. When the status is all you need,
+    :func:`get_work_item_status` is a much smaller round-trip.
     """
     if (project_path or group_path) and not is_global_id(str(work_item)):
         path = project_path if project_path is not None else group_path
         query = (
-            _WORK_ITEM_FRAGMENT
+            _work_item_fragment(include_status)
             + """
             query GetWorkItemByIid($path: ID!, $iid: String!) {
               namespace(fullPath: $path) {
@@ -250,7 +277,7 @@ async def get_work_item(
 
     gid = make_work_item_gid(work_item)
     query = (
-        _WORK_ITEM_FRAGMENT
+        _work_item_fragment(include_status)
         + """
         query GetWorkItem($id: WorkItemID!) {
           workItem(id: $id) { ...WorkItemFields }
@@ -264,6 +291,87 @@ async def get_work_item(
     return WorkItem.from_node(node)
 
 
+def _extract_status(node: Optional[Dict[str, Any]]) -> Optional[WorkItemStatus]:
+    """Pull the status out of a work-item node's ``widgets`` list.
+
+    GitLab returns *every* widget on the item, but only the status widget matches
+    the inline fragment — the rest arrive as empty objects and are skipped here.
+    An item with no status assigned yields ``None``.
+    """
+    for widget in (node or {}).get("widgets") or []:
+        status = widget.get("status")
+        if status:
+            return WorkItemStatus.model_validate(status)
+    return None
+
+
+@async_retry_http(status_codes_to_ignore=_IGNORE_4XX)
+async def get_work_item_status(
+    work_item: WorkItemRef,
+    *,
+    project_path: Optional[ProjectId] = None,
+    group_path: Optional[GroupId] = None,
+    client: Optional[AsyncClient] = None,
+) -> Optional[WorkItemStatus]:
+    """Fetch only the custom status of a workItem.
+
+    Same identifier rules as :func:`get_work_item`. Prefer this over
+    ``get_work_item(..., include_status=True)`` when the status is all you need:
+    it skips the full field set, most notably the 50 discussions the shared
+    fragment pulls in, which matters on a per-event hot path.
+
+    Returns ``None`` when the item exists but has no status assigned. Raises when
+    the item cannot be found, or — because the widget is a 17.11+ experiment —
+    when the instance does not know the ``WorkItemWidgetStatus`` type at all.
+    """
+    if (project_path or group_path) and not is_global_id(str(work_item)):
+        path = project_path if project_path is not None else group_path
+        query = """
+            query GetWorkItemStatusByIid($path: ID!, $iid: String!) {
+              namespace(fullPath: $path) {
+                workItems(iid: $iid, first: 1) {
+                  nodes {
+                    id
+                    widgets {
+                      ... on WorkItemWidgetStatus {
+                        status { id name iconName color position }
+                      }
+                    }
+                  }
+                }
+              }
+            }
+        """
+        data = await graphql(
+            query, {"path": str(path), "iid": str(work_item)}, client=client
+        )
+        nodes = (((data or {}).get("namespace") or {}).get("workItems") or {}).get(
+            "nodes"
+        ) or []
+        if not nodes:
+            raise RuntimeError(f"workItem !{work_item} not found under {path}")
+        return _extract_status(nodes[0])
+
+    gid = make_work_item_gid(work_item)
+    query = """
+        query GetWorkItemStatus($id: WorkItemID!) {
+          workItem(id: $id) {
+            id
+            widgets {
+              ... on WorkItemWidgetStatus {
+                status { id name iconName color position }
+              }
+            }
+          }
+        }
+    """
+    data = await graphql(query, {"id": gid}, client=client)
+    node = (data or {}).get("workItem")
+    if not node:
+        raise RuntimeError(f"workItem {gid} not found")
+    return _extract_status(node)
+
+
 @async_retry_http(status_codes_to_ignore=_IGNORE_4XX)
 async def search_work_items(
     *,
@@ -273,6 +381,7 @@ async def search_work_items(
     labels: Optional[List[str]] = None,
     search: Optional[str] = None,
     work_item_type_ids: Optional[List[str]] = None,
+    include_status: bool = False,
     paginate_results: bool = True,
     page_size: int = 100,
     client: Optional[AsyncClient] = None,
@@ -288,6 +397,9 @@ async def search_work_items(
 
     When ``paginate_results`` is ``True`` (the default) the helper follows
     ``pageInfo.endCursor`` until ``hasNextPage`` is false.
+
+    ``include_status`` populates :attr:`WorkItem.status` on every result; see
+    :func:`get_work_item` for the version caveat.
     """
     if (project_path is None) == (group_path is None):
         raise ValueError("provide exactly one of project_path / group_path")
@@ -296,7 +408,7 @@ async def search_work_items(
     path = project_path if project_path is not None else group_path
 
     query = (
-        _WORK_ITEM_FRAGMENT
+        _work_item_fragment(include_status)
         + f"""
         query SearchWorkItems(
           $path: ID!,
@@ -418,7 +530,7 @@ async def set_work_item_state(
             client=_client,
         )
         query = (
-            _WORK_ITEM_FRAGMENT
+            _work_item_fragment()
             + """
             mutation SetWorkItemState($id: WorkItemID!, $stateEvent: WorkItemStateEvent!) {
               workItemUpdate(input: { id: $id, stateEvent: $stateEvent }) {
@@ -471,7 +583,7 @@ async def set_work_item_labels(
             client=_client,
         )
         query = (
-            _WORK_ITEM_FRAGMENT
+            _work_item_fragment()
             + """
             mutation SetWorkItemLabels($input: WorkItemUpdateInput!) {
               workItemUpdate(input: $input) {

{pjdev_gitlab-5.1.12 → pjdev_gitlab-5.1.13}/test_report_pjdev-gitlab.txt

@@ -3,9 +3,9 @@ platform linux -- Python 3.12.3, pytest-9.1.1, pluggy-1.6.0 -- /venv/bin/python3
 cachedir: .pytest_cache
 rootdir: /builds/keystone/python/pjdev-gitlab
 configfile: pyproject.toml
-plugins: asyncio-1.4.0, respx-0.23.1, anyio-4.14.2
+plugins: asyncio-1.4.0, anyio-4.14.2, respx-0.23.1
 asyncio: mode=Mode.AUTO, debug=False, asyncio_default_fixture_loop_scope=None, asyncio_default_test_loop_scope=function
-collecting ... collected 89 items
+collecting ... collected 96 items
 
 tests/tests_for_api_utilities.py::test_build_auth_defaults_to_private_token PASSED [  1%]
 tests/tests_for_api_utilities.py::test_build_auth_selects_bearer_for_oauth PASSED [  2%]
@@ -15,78 +15,85 @@ tests/tests_for_api_utilities.py::test_paginate_consumes_x_next_page PASSED [  5
 tests/tests_for_api_utilities.py::test_async_retry_http_aborts_on_ignored_status PASSED [  6%]
 tests/tests_for_api_utilities.py::test_async_retry_http_retries_on_5xx PASSED [  7%]
 tests/tests_for_api_utilities.py::test_upload_to_project_returns_markdown PASSED [  8%]
-tests/tests_for_git_sync_service.py::test_build_remote_url[https://gitlab.example.com-g/s/n-https://gitlab.example.com/g/s/n.git] PASSED [ 10%]
-tests/tests_for_git_sync_service.py::test_build_remote_url[https://gitlab.example.com/-/g/n/-https://gitlab.example.com/g/n.git] PASSED [ 11%]
-tests/tests_for_git_sync_service.py::test_build_remote_url[https://gitlab.example.com-g/n.git-https://gitlab.example.com/g/n.git] PASSED [ 12%]
-tests/tests_for_git_sync_service.py::test_build_remote_url_empty_repo_errors PASSED [ 13%]
-tests/tests_for_git_sync_service.py::test_default_target_dir[developers/claude-skills-claude-skills] PASSED [ 14%]
-tests/tests_for_git_sync_service.py::test_default_target_dir[g/s/name.git-name] PASSED [ 15%]
-tests/tests_for_git_sync_service.py::test_default_target_dir[solo-solo] PASSED [ 16%]
-tests/tests_for_git_sync_service.py::test_clone_when_target_missing PASSED [ 17%]
-tests/tests_for_git_sync_service.py::test_clone_passes_token_via_askpass_env_only PASSED [ 19%]
-tests/tests_for_git_sync_service.py::test_clone_with_branch PASSED       [ 20%]
-tests/tests_for_git_sync_service.py::test_update_when_target_is_repo PASSED [ 21%]
-tests/tests_for_git_sync_service.py::test_update_with_existing_branch_checks_out_and_ff PASSED [ 22%]
-tests/tests_for_git_sync_service.py::test_update_with_new_branch_creates_from_fetch_head PASSED [ 23%]
-tests/tests_for_git_sync_service.py::test_update_adds_origin_when_absent PASSED [ 24%]
-tests/tests_for_git_sync_service.py::test_update_detached_head_errors PASSED [ 25%]
-tests/tests_for_git_sync_service.py::test_clone_failure_removes_partial_target PASSED [ 26%]
-tests/tests_for_git_sync_service.py::test_target_exists_but_not_a_repo_errors PASSED [ 28%]
-tests/tests_for_git_sync_service.py::test_missing_token_errors PASSED    [ 29%]
-tests/tests_for_git_sync_service.py::test_ff_only_failure_propagates PASSED [ 30%]
-tests/tests_for_issues_service.py::test_search_issues_paginates PASSED   [ 31%]
-tests/tests_for_issues_service.py::test_create_issue_with_image_uploads_first_then_creates PASSED [ 32%]
-tests/tests_for_issues_service.py::test_set_issue_state PASSED           [ 33%]
-tests/tests_for_issues_service.py::test_set_issue_labels_modes PASSED    [ 34%]
-tests/tests_for_issues_service.py::test_aggregate_issues_by_label PASSED [ 35%]
-tests/tests_for_issues_service.py::test_comment_on_issue PASSED          [ 37%]
-tests/tests_for_merge_requests_service.py::test_get_merge_request PASSED [ 38%]
-tests/tests_for_merge_requests_service.py::test_comment_on_merge_request PASSED [ 39%]
-tests/tests_for_merge_requests_service.py::test_inline_diff_comment_payload_shape PASSED [ 40%]
-tests/tests_for_merge_requests_service.py::test_approve_and_unapprove PASSED [ 41%]
-tests/tests_for_models.py::test_config_api_base_url_strips_trailing_slash PASSED [ 42%]
-tests/tests_for_models.py::test_issue_extra_fields_ignored PASSED        [ 43%]
-tests/tests_for_models.py::test_merge_request_basic PASSED               [ 44%]
-tests/tests_for_models.py::test_config_env_loading PASSED                [ 46%]
-tests/tests_for_oauth_cli.py::test_missing_client_id_errors PASSED       [ 47%]
-tests/tests_for_oauth_cli.py::test_success_prints_only_token PASSED      [ 48%]
-tests/tests_for_oauth_cli.py::test_reads_config_from_env PASSED          [ 49%]
-tests/tests_for_oauth_cli.py::test_bad_redirect_port_errors PASSED       [ 50%]
-tests/tests_for_oauth_cli.py::test_oauth_error_returns_one PASSED        [ 51%]
-tests/tests_for_oauth_service.py::test_make_pkce_challenge_is_s256_of_verifier PASSED [ 52%]
-tests/tests_for_oauth_service.py::test_cache_path_is_per_host_and_private PASSED [ 53%]
-tests/tests_for_oauth_service.py::test_token_if_valid_expiry PASSED      [ 55%]
-tests/tests_for_oauth_service.py::test_store_token_response_roundtrip PASSED [ 56%]
-tests/tests_for_oauth_service.py::test_store_token_response_requires_access_token PASSED [ 57%]
-tests/tests_for_oauth_service.py::test_refresh_access_token_success PASSED [ 58%]
-tests/tests_for_oauth_service.py::test_refresh_access_token_returns_none_on_error PASSED [ 59%]
-tests/tests_for_oauth_service.py::test_get_access_token_reuses_valid_cache PASSED [ 60%]
-tests/tests_for_oauth_service.py::test_get_access_token_refreshes_expired_cache PASSED [ 61%]
-tests/tests_for_oauth_service.py::test_get_access_token_requires_client_id PASSED [ 62%]
-tests/tests_for_packages_service.py::test_upload_generic_package_url_shape PASSED [ 64%]
-tests/tests_for_packages_service.py::test_download_generic_package PASSED [ 65%]
-tests/tests_for_packages_service.py::test_list_packages PASSED           [ 66%]
-tests/tests_for_repo_files_service.py::test_get_file_url_encoded PASSED  [ 67%]
-tests/tests_for_repo_files_service.py::test_download_file_writes_to_dest PASSED [ 68%]
-tests/tests_for_repo_files_service.py::test_download_directory_walks_tree PASSED [ 69%]
-tests/tests_for_sync_cli.py::test_missing_repo_errors PASSED             [ 70%]
-tests/tests_for_sync_cli.py::test_missing_client_id_errors PASSED        [ 71%]
-tests/tests_for_sync_cli.py::test_success_prints_path_and_forwards_args PASSED [ 73%]
-tests/tests_for_sync_cli.py::test_reads_config_from_env PASSED           [ 74%]
-tests/tests_for_sync_cli.py::test_oauth_error_returns_one PASSED         [ 75%]
-tests/tests_for_sync_cli.py::test_sync_error_returns_one PASSED          [ 76%]
-tests/tests_for_sync_cli.py::test_bad_redirect_port_errors PASSED        [ 77%]
-tests/tests_for_work_items_service.py::test_make_work_item_gid_passthrough_and_coerce PASSED [ 78%]
-tests/tests_for_work_items_service.py::test_work_item_from_node_flattens_widgets PASSED [ 79%]
-tests/tests_for_work_items_service.py::test_graphql_raises_on_errors PASSED [ 80%]
-tests/tests_for_work_items_service.py::test_paginate_graphql_follows_end_cursor PASSED [ 82%]
-tests/tests_for_work_items_service.py::test_get_work_item_by_global_id PASSED [ 83%]
-tests/tests_for_work_items_service.py::test_get_work_item_by_project_path_and_iid_uses_namespace_lookup PASSED [ 84%]
-tests/tests_for_work_items_service.py::test_get_work_item_missing_raises PASSED [ 85%]
+tests/tests_for_git_sync_service.py::test_build_remote_url[https://gitlab.example.com-g/s/n-https://gitlab.example.com/g/s/n.git] PASSED [  9%]
+tests/tests_for_git_sync_service.py::test_build_remote_url[https://gitlab.example.com/-/g/n/-https://gitlab.example.com/g/n.git] PASSED [ 10%]
+tests/tests_for_git_sync_service.py::test_build_remote_url[https://gitlab.example.com-g/n.git-https://gitlab.example.com/g/n.git] PASSED [ 11%]
+tests/tests_for_git_sync_service.py::test_build_remote_url_empty_repo_errors PASSED [ 12%]
+tests/tests_for_git_sync_service.py::test_default_target_dir[developers/claude-skills-claude-skills] PASSED [ 13%]
+tests/tests_for_git_sync_service.py::test_default_target_dir[g/s/name.git-name] PASSED [ 14%]
+tests/tests_for_git_sync_service.py::test_default_target_dir[solo-solo] PASSED [ 15%]
+tests/tests_for_git_sync_service.py::test_clone_when_target_missing PASSED [ 16%]
+tests/tests_for_git_sync_service.py::test_clone_passes_token_via_askpass_env_only PASSED [ 17%]
+tests/tests_for_git_sync_service.py::test_clone_with_branch PASSED       [ 18%]
+tests/tests_for_git_sync_service.py::test_update_when_target_is_repo PASSED [ 19%]
+tests/tests_for_git_sync_service.py::test_update_with_existing_branch_checks_out_and_ff PASSED [ 20%]
+tests/tests_for_git_sync_service.py::test_update_with_new_branch_creates_from_fetch_head PASSED [ 21%]
+tests/tests_for_git_sync_service.py::test_update_adds_origin_when_absent PASSED [ 22%]
+tests/tests_for_git_sync_service.py::test_update_detached_head_errors PASSED [ 23%]
+tests/tests_for_git_sync_service.py::test_clone_failure_removes_partial_target PASSED [ 25%]
+tests/tests_for_git_sync_service.py::test_target_exists_but_not_a_repo_errors PASSED [ 26%]
+tests/tests_for_git_sync_service.py::test_missing_token_errors PASSED    [ 27%]
+tests/tests_for_git_sync_service.py::test_ff_only_failure_propagates PASSED [ 28%]
+tests/tests_for_issues_service.py::test_search_issues_paginates PASSED   [ 29%]
+tests/tests_for_issues_service.py::test_create_issue_with_image_uploads_first_then_creates PASSED [ 30%]
+tests/tests_for_issues_service.py::test_set_issue_state PASSED           [ 31%]
+tests/tests_for_issues_service.py::test_set_issue_labels_modes PASSED    [ 32%]
+tests/tests_for_issues_service.py::test_aggregate_issues_by_label PASSED [ 33%]
+tests/tests_for_issues_service.py::test_comment_on_issue PASSED          [ 34%]
+tests/tests_for_merge_requests_service.py::test_get_merge_request PASSED [ 35%]
+tests/tests_for_merge_requests_service.py::test_comment_on_merge_request PASSED [ 36%]
+tests/tests_for_merge_requests_service.py::test_inline_diff_comment_payload_shape PASSED [ 37%]
+tests/tests_for_merge_requests_service.py::test_approve_and_unapprove PASSED [ 38%]
+tests/tests_for_models.py::test_config_api_base_url_strips_trailing_slash PASSED [ 39%]
+tests/tests_for_models.py::test_issue_extra_fields_ignored PASSED        [ 40%]
+tests/tests_for_models.py::test_merge_request_basic PASSED               [ 41%]
+tests/tests_for_models.py::test_config_env_loading PASSED                [ 42%]
+tests/tests_for_oauth_cli.py::test_missing_client_id_errors PASSED       [ 43%]
+tests/tests_for_oauth_cli.py::test_success_prints_only_token PASSED      [ 44%]
+tests/tests_for_oauth_cli.py::test_reads_config_from_env PASSED          [ 45%]
+tests/tests_for_oauth_cli.py::test_bad_redirect_port_errors PASSED       [ 46%]
+tests/tests_for_oauth_cli.py::test_oauth_error_returns_one PASSED        [ 47%]
+tests/tests_for_oauth_service.py::test_make_pkce_challenge_is_s256_of_verifier PASSED [ 48%]
+tests/tests_for_oauth_service.py::test_cache_path_is_per_host_and_private PASSED [ 50%]
+tests/tests_for_oauth_service.py::test_token_if_valid_expiry PASSED      [ 51%]
+tests/tests_for_oauth_service.py::test_store_token_response_roundtrip PASSED [ 52%]
+tests/tests_for_oauth_service.py::test_store_token_response_requires_access_token PASSED [ 53%]
+tests/tests_for_oauth_service.py::test_refresh_access_token_success PASSED [ 54%]
+tests/tests_for_oauth_service.py::test_refresh_access_token_returns_none_on_error PASSED [ 55%]
+tests/tests_for_oauth_service.py::test_get_access_token_reuses_valid_cache PASSED [ 56%]
+tests/tests_for_oauth_service.py::test_get_access_token_refreshes_expired_cache PASSED [ 57%]
+tests/tests_for_oauth_service.py::test_get_access_token_requires_client_id PASSED [ 58%]
+tests/tests_for_packages_service.py::test_upload_generic_package_url_shape PASSED [ 59%]
+tests/tests_for_packages_service.py::test_download_generic_package PASSED [ 60%]
+tests/tests_for_packages_service.py::test_list_packages PASSED           [ 61%]
+tests/tests_for_repo_files_service.py::test_get_file_url_encoded PASSED  [ 62%]
+tests/tests_for_repo_files_service.py::test_download_file_writes_to_dest PASSED [ 63%]
+tests/tests_for_repo_files_service.py::test_download_directory_walks_tree PASSED [ 64%]
+tests/tests_for_sync_cli.py::test_missing_repo_errors PASSED             [ 65%]
+tests/tests_for_sync_cli.py::test_missing_client_id_errors PASSED        [ 66%]
+tests/tests_for_sync_cli.py::test_success_prints_path_and_forwards_args PASSED [ 67%]
+tests/tests_for_sync_cli.py::test_reads_config_from_env PASSED           [ 68%]
+tests/tests_for_sync_cli.py::test_oauth_error_returns_one PASSED         [ 69%]
+tests/tests_for_sync_cli.py::test_sync_error_returns_one PASSED          [ 70%]
+tests/tests_for_sync_cli.py::test_bad_redirect_port_errors PASSED        [ 71%]
+tests/tests_for_work_items_service.py::test_make_work_item_gid_passthrough_and_coerce PASSED [ 72%]
+tests/tests_for_work_items_service.py::test_work_item_from_node_flattens_widgets PASSED [ 73%]
+tests/tests_for_work_items_service.py::test_graphql_raises_on_errors PASSED [ 75%]
+tests/tests_for_work_items_service.py::test_paginate_graphql_follows_end_cursor PASSED [ 76%]
+tests/tests_for_work_items_service.py::test_get_work_item_by_global_id PASSED [ 77%]
+tests/tests_for_work_items_service.py::test_get_work_item_by_project_path_and_iid_uses_namespace_lookup PASSED [ 78%]
+tests/tests_for_work_items_service.py::test_get_work_item_omits_the_status_widget_by_default PASSED [ 79%]
+tests/tests_for_work_items_service.py::test_get_work_item_include_status_selects_the_widget_and_lifts_it PASSED [ 80%]
+tests/tests_for_work_items_service.py::test_get_work_item_missing_raises PASSED [ 81%]
+tests/tests_for_work_items_service.py::test_get_work_item_status_by_project_path_and_iid PASSED [ 82%]
+tests/tests_for_work_items_service.py::test_get_work_item_status_by_global_id_skips_the_iid_lookup PASSED [ 83%]
+tests/tests_for_work_items_service.py::test_get_work_item_status_returns_none_when_no_status_is_assigned PASSED [ 84%]
+tests/tests_for_work_items_service.py::test_get_work_item_status_missing_item_raises PASSED [ 85%]
 tests/tests_for_work_items_service.py::test_search_work_items_paginates_and_translates_state PASSED [ 86%]
 tests/tests_for_work_items_service.py::test_search_work_items_group_scope_uses_group_field PASSED [ 87%]
-tests/tests_for_work_items_service.py::test_search_work_items_requires_one_scope PASSED [ 88%]
-tests/tests_for_work_items_service.py::test_set_work_item_state_closes_via_workItemUpdate PASSED [ 89%]
+tests/tests_for_work_items_service.py::test_search_work_items_include_status_selects_the_widget PASSED [ 88%]
+tests/tests_for_work_items_service.py::test_search_work_items_requires_one_scope PASSED [ 89%]
+tests/tests_for_work_items_service.py::test_set_work_item_state_closes_via_workItemUpdate PASSED [ 90%]
 tests/tests_for_work_items_service.py::test_set_work_item_state_mutation_errors_raise PASSED [ 91%]
 tests/tests_for_work_items_service.py::test_list_work_item_notes_filters_system_notes PASSED [ 92%]
 tests/tests_for_work_items_service.py::test_comment_on_work_item_creates_note PASSED [ 93%]
@@ -97,4 +104,4 @@ tests/tests_for_work_items_service.py::test_list_labels_for_project_includes_anc
 tests/tests_for_work_items_service.py::test_create_label_passes_project_path_color_and_description PASSED [ 98%]
 tests/tests_for_work_items_service.py::test_create_label_group_scope_passes_group_path PASSED [100%]
 
-============================== 89 passed in 1.80s ==============================
+============================== 96 passed in 1.91s ==============================

{pjdev_gitlab-5.1.12 → pjdev_gitlab-5.1.13}/tests/tests_for_work_items_service.py

@@ -33,6 +33,7 @@ def _wi_node(
     labels: Optional[List[Dict[str, Any]]] = None,
     assignees: Optional[List[Dict[str, Any]]] = None,
     discussions: Optional[List[Dict[str, Any]]] = None,
+    status: Optional[Dict[str, Any]] = None,
     numeric_id: int = 12345,
 ) -> Dict[str, Any]:
     """Build a fake GraphQL workItem node — only the bits the service uses."""
@@ -41,6 +42,8 @@ def _wi_node(
         {"type": "LABELS", "labels": {"nodes": labels or []}},
         {"type": "ASSIGNEES", "assignees": {"nodes": assignees or []}},
     ]
+    if status is not None:
+        widgets.append({"type": "STATUS", "status": status})
     if discussions is not None:
         widgets.append(
             {
@@ -70,6 +73,16 @@ def _wi_node(
     }
 
 
+def _status_node(*, name: str = "Ready", id_: int = 3) -> Dict[str, Any]:
+    return {
+        "id": f"gid://gitlab/WorkItems::Statuses::Custom::Status/{id_}",
+        "name": name,
+        "iconName": "status-running",
+        "color": "#1f75cb",
+        "position": 10,
+    }
+
+
 def _label_node(*, id_: int = 99, title: str = "bug", color: str = "#FF0000") -> Dict[str, Any]:
     return {
         "id": f"gid://gitlab/ProjectLabel/{id_}",
@@ -256,6 +269,45 @@ async def test_get_work_item_by_project_path_and_iid_uses_namespace_lookup():
     assert "namespace(fullPath: $path)" in captured["query"]
 
 
+@respx.mock
+async def test_get_work_item_omits_the_status_widget_by_default():
+    """The widget is a 17.11+ experiment, so an unasked-for query must not select it."""
+    captured: Dict[str, Any] = {}
+
+    def handler(request: httpx.Request) -> httpx.Response:
+        captured.update(_decode_request(request))
+        return httpx.Response(200, json={"data": {"workItem": _wi_node()}})
+
+    respx.post(GRAPHQL_URL).mock(side_effect=handler)
+
+    item = await work_items_service.get_work_item("gid://gitlab/WorkItem/12345")
+    assert "WorkItemWidgetStatus" not in captured["query"]
+    assert "__STATUS_WIDGET__" not in captured["query"]
+    assert item.status is None
+
+
+@respx.mock
+async def test_get_work_item_include_status_selects_the_widget_and_lifts_it():
+    captured: Dict[str, Any] = {}
+
+    def handler(request: httpx.Request) -> httpx.Response:
+        captured.update(_decode_request(request))
+        return httpx.Response(
+            200,
+            json={"data": {"workItem": _wi_node(status=_status_node(name="Ready"))}},
+        )
+
+    respx.post(GRAPHQL_URL).mock(side_effect=handler)
+
+    item = await work_items_service.get_work_item(
+        "gid://gitlab/WorkItem/12345", include_status=True
+    )
+    assert "... on WorkItemWidgetStatus" in captured["query"]
+    assert item.status is not None
+    assert item.status.name == "Ready"
+    assert item.status.icon_name == "status-running"
+
+
 @respx.mock
 async def test_get_work_item_missing_raises():
     respx.post(GRAPHQL_URL).mock(
@@ -267,6 +319,113 @@ async def test_get_work_item_missing_raises():
         await work_items_service.get_work_item(99, project_path="group/proj")
 
 
+# ---------------------------------------------------------------------------
+# get_work_item_status
+# ---------------------------------------------------------------------------
+
+
+@respx.mock
+async def test_get_work_item_status_by_project_path_and_iid():
+    captured: Dict[str, Any] = {}
+
+    def handler(request: httpx.Request) -> httpx.Response:
+        captured.update(_decode_request(request))
+        return httpx.Response(
+            200,
+            json={
+                "data": {
+                    "namespace": {
+                        "workItems": {
+                            "nodes": [
+                                {
+                                    "id": "gid://gitlab/WorkItem/12345",
+                                    # Shaped like a real response: GitLab returns
+                                    # every widget, so the non-status ones come
+                                    # back as empty objects.
+                                    "widgets": [{}, {}, {"status": _status_node()}],
+                                }
+                            ]
+                        }
+                    }
+                }
+            },
+        )
+
+    respx.post(GRAPHQL_URL).mock(side_effect=handler)
+
+    status = await work_items_service.get_work_item_status(425, project_path="group/proj")
+    assert status is not None
+    assert status.name == "Ready"
+    assert captured["variables"] == {"path": "group/proj", "iid": "425"}
+    # The point of this helper is that it does NOT drag the full fragment along.
+    assert "...WorkItemFields" not in captured["query"]
+    assert "discussions" not in captured["query"]
+
+
+@respx.mock
+async def test_get_work_item_status_by_global_id_skips_the_iid_lookup():
+    captured: Dict[str, Any] = {}
+
+    def handler(request: httpx.Request) -> httpx.Response:
+        captured.update(_decode_request(request))
+        return httpx.Response(
+            200,
+            json={
+                "data": {
+                    "workItem": {
+                        "id": "gid://gitlab/WorkItem/12345",
+                        "widgets": [{"status": _status_node(name="In progress")}],
+                    }
+                }
+            },
+        )
+
+    respx.post(GRAPHQL_URL).mock(side_effect=handler)
+
+    status = await work_items_service.get_work_item_status("gid://gitlab/WorkItem/12345")
+    assert status is not None and status.name == "In progress"
+    assert captured["variables"] == {"id": "gid://gitlab/WorkItem/12345"}
+
+
+@respx.mock
+async def test_get_work_item_status_returns_none_when_no_status_is_assigned():
+    respx.post(GRAPHQL_URL).mock(
+        return_value=httpx.Response(
+            200,
+            json={
+                "data": {
+                    "namespace": {
+                        "workItems": {
+                            "nodes": [
+                                {
+                                    "id": "gid://gitlab/WorkItem/12345",
+                                    "widgets": [{}, {"status": None}],
+                                }
+                            ]
+                        }
+                    }
+                }
+            },
+        )
+    )
+
+    assert (
+        await work_items_service.get_work_item_status(425, project_path="group/proj")
+        is None
+    )
+
+
+@respx.mock
+async def test_get_work_item_status_missing_item_raises():
+    respx.post(GRAPHQL_URL).mock(
+        return_value=httpx.Response(
+            200, json={"data": {"namespace": {"workItems": {"nodes": []}}}}
+        )
+    )
+    with pytest.raises(ExceptionGroup):
+        await work_items_service.get_work_item_status(99, project_path="group/proj")
+
+
 # ---------------------------------------------------------------------------
 # search_work_items
 # ---------------------------------------------------------------------------
@@ -358,6 +517,35 @@ async def test_search_work_items_group_scope_uses_group_field():
     assert "group(fullPath: $path)" in captured["query"]
 
 
+@respx.mock
+async def test_search_work_items_include_status_selects_the_widget():
+    captured: Dict[str, Any] = {}
+
+    def handler(request: httpx.Request) -> httpx.Response:
+        captured.update(_decode_request(request))
+        return httpx.Response(
+            200,
+            json={
+                "data": {
+                    "project": {
+                        "workItems": {
+                            "nodes": [_wi_node(status=_status_node(name="PJ Test"))],
+                            "pageInfo": {"endCursor": None, "hasNextPage": False},
+                        }
+                    }
+                }
+            },
+        )
+
+    respx.post(GRAPHQL_URL).mock(side_effect=handler)
+
+    items = await work_items_service.search_work_items(
+        project_path="group/proj", include_status=True
+    )
+    assert "... on WorkItemWidgetStatus" in captured["query"]
+    assert items[0].status is not None and items[0].status.name == "PJ Test"
+
+
 async def test_search_work_items_requires_one_scope():
     with pytest.raises(ExceptionGroup):
         await work_items_service.search_work_items()