nocfo-cli 1.5.2__tar.gz → 1.5.3__tar.gz

{nocfo_cli-1.5.2 → nocfo_cli-1.5.3}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: nocfo-cli
-Version: 1.5.2
+Version: 1.5.3
 Summary: NoCFO CLI, MCP server, and Cursor skill toolkit.
 License: MIT
 License-File: LICENSE

{nocfo_cli-1.5.2 → nocfo_cli-1.5.3}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "poetry.core.masonry.api"
 
 [tool.poetry]
 name = "nocfo-cli"
-version = "1.5.2"
+version = "1.5.3"
 description = "NoCFO CLI, MCP server, and Cursor skill toolkit."
 authors = ["NoCFO"]
 readme = "README.md"

{nocfo_cli-1.5.2 → nocfo_cli-1.5.3}/src/nocfo_toolkit/mcp/curated/bookkeeping/relation.py

@@ -8,12 +8,10 @@ from fastmcp.tools import tool
 from nocfo_toolkit.mcp.curated.confirmation import confirm_mutation
 from nocfo_toolkit.mcp.curated.runtime import business_slug, get_client
 from nocfo_toolkit.mcp.curated.bookkeeping.document import document_by_number
-from nocfo_toolkit.mcp.curated.errors import raise_tool_error
 from nocfo_toolkit.mcp.curated.schemas import (
     DeletedResponse,
     DocumentRelationCreateInput,
     DocumentRelationIdInput,
-    DocumentRelationUpdateInput,
     EntryListInput,
     ListEnvelope,
     RelationSummary,
@@ -97,36 +95,6 @@ async def bookkeeping_document_relation_create(
     return dump_model_from_backend(RelationSummary, result)
 
 
-@tool(
-    name="bookkeeping_document_relation_update",
-    description="Update a relation listed for the same document context.",
-)
-async def bookkeeping_document_relation_update(
-    params: DocumentRelationUpdateInput,
-) -> dict[str, Any]:
-    args = params
-    slug = await business_slug(args.business)
-    document = await document_by_number(slug, args.document_number)
-    payload = await _resolve_relation_update_payload(slug, args.payload.model_dump())
-    path = f"/v1/business/{slug}/document/{document['id']}/relation/{args.relation_id}/"
-    await confirm_mutation(
-        business=slug,
-        tool_name="bookkeeping_document_relation_update",
-        target_resource={
-            "type": "document_relation",
-            "id": args.relation_id,
-        },
-        parameters=payload,
-    )
-    result = await get_client().request(
-        "PATCH",
-        path,
-        json_body=payload,
-        business_slug=slug,
-    )
-    return dump_model_from_backend(RelationSummary, result)
-
-
 @tool(
     name="bookkeeping_document_relation_delete",
     description="Delete a relation listed for the same document context.",
@@ -152,21 +120,3 @@ async def bookkeeping_document_relation_delete(
         business_slug=slug,
     )
     return dump_model(DeletedResponse(relation_id=args.relation_id))
-
-
-async def _resolve_relation_update_payload(
-    slug: str, raw_payload: dict[str, Any]
-) -> dict[str, Any]:
-    payload = {key: value for key, value in raw_payload.items() if value is not None}
-    related_document_number = payload.pop("related_document_number", None)
-    if related_document_number is not None:
-        related = await document_by_number(slug, str(related_document_number))
-        payload["related_document"] = related["id"]
-    if not payload:
-        raise_tool_error(
-            "invalid_request",
-            "At least one mutable relation field is required.",
-            "Provide one or more of related_document_number, role, or type.",
-            status_code=400,
-        )
-    return payload

{nocfo_cli-1.5.2 → nocfo_cli-1.5.3}/src/nocfo_toolkit/mcp/curated/common.py

@@ -5,14 +5,12 @@ from __future__ import annotations
 from typing import Any
 
 from fastmcp.tools import tool
-from nocfo_toolkit.mcp.curated.confirmation import confirm_mutation
 from nocfo_toolkit.mcp.curated.runtime import get_client
 from nocfo_toolkit.mcp.curated.schemas import (
     BusinessContextInput,
     BusinessSummary,
     ListEnvelope,
     PaginationInput,
-    PayloadOnlyInput,
     ResolvedBusiness,
     UserSummary,
     dump_model,
@@ -56,20 +54,3 @@ async def common_accessible_businesses_list(
 async def common_user_retrieve() -> dict[str, Any]:
     payload = await get_client().request("GET", "/v1/user/")
     return dump_model_from_backend(UserSummary, payload)
-
-
-@tool(
-    name="common_user_update",
-    description="Update basic authenticated user profile fields.",
-)
-async def common_user_update(params: PayloadOnlyInput) -> dict[str, Any]:
-    args = params
-    path = "/v1/user/"
-    await confirm_mutation(
-        business="current",
-        tool_name="common_user_update",
-        target_resource={"type": "user", "id": "current"},
-        parameters=args.payload,
-    )
-    result = await get_client().request("PATCH", path, json_body=args.payload)
-    return dump_model_from_backend(UserSummary, result)

{nocfo_cli-1.5.2 → nocfo_cli-1.5.3}/src/nocfo_toolkit/mcp/curated/schema/bookkeeping/document.py

@@ -13,7 +13,6 @@ from nocfo_toolkit.mcp.curated.schema.common import (
     DocumentAction,
     DocumentSide,
     RelationRole,
-    StrictModel,
     RelationType,
     enum_or_str,
     tool_handle,
@@ -197,28 +196,7 @@ class DocumentRelationCreateInput(DocumentNumberInput):
 
 class DocumentRelationIdInput(DocumentNumberInput):
     relation_id: int = Field(
-        description="Relation ID from bookkeeping_document_relations_list. Use bookkeeping_document_relation_update or bookkeeping_document_relation_delete for this ID."
-    )
-
-
-class DocumentRelationUpdatePayload(StrictModel):
-    related_document_number: str | None = Field(
-        default=None,
-        description="Document number of the other bookkeeping document in the relation.",
-    )
-    role: enum_or_str(RelationRole) | None = Field(
-        default=None,
-        description="Whether the current document is the accrual or settlement side of the relation.",
-    )
-    type: enum_or_str(RelationType) | None = Field(
-        default=None,
-        description="Type/category shown for this record.",
-    )
-
-
-class DocumentRelationUpdateInput(DocumentRelationIdInput):
-    payload: DocumentRelationUpdatePayload = Field(
-        description="Relation update payload. Supported fields: related_document_number, role, and type."
+        description="Relation ID from bookkeeping_document_relations_list. Use bookkeeping_document_relation_delete for this ID."
     )
 
 
@@ -434,12 +412,12 @@ class EntrySummary(AgentModel):
 class RelationSummary(AgentModel):
     tool_handle: str | None = Field(
         default=None,
-        description="Copy this value from bookkeeping_document_relations_list.items[].tool_handle and pass it unchanged to bookkeeping_document_relation_update or bookkeeping_document_relation_delete.",
+        description="Copy this value from bookkeeping_document_relations_list.items[].tool_handle and pass it unchanged to bookkeeping_document_relation_delete.",
     )
     relation_id: int | None = Field(
         default=None,
         validation_alias=AliasChoices("relation_id", "id"),
-        description="Relation ID. Use bookkeeping_document_relation_update or bookkeeping_document_relation_delete for this ID.",
+        description="Relation ID. Use bookkeeping_document_relation_delete for this ID.",
     )
     related_document_handle: str | None = Field(
         default=None,

{nocfo_cli-1.5.2 → nocfo_cli-1.5.3}/src/nocfo_toolkit/mcp/curated/schema/common.py

@@ -385,12 +385,6 @@ class ExactResourceInput(BusinessContextInput):
         return self
 
 
-class PayloadOnlyInput(StrictModel):
-    payload: dict[str, Any] = Field(
-        description="Fields to update on the current user profile."
-    )
-
-
 class IdentifierInput(BusinessContextInput):
     identifier: str = Field(
         description="Identifier for this resource. Prefer the exact value named in the tool description."

{nocfo_cli-1.5.2 → nocfo_cli-1.5.3}/src/nocfo_toolkit/mcp/curated/schema/invoicing/sales_invoice.py

@@ -19,15 +19,23 @@ from nocfo_toolkit.mcp.curated.schema.common import (
 
 
 def _derive_invoice_next_action(
-    *, status_raw: str | None, is_sendable: bool | None, last_delivery_at: str | None
+    *,
+    status_raw: str | None,
+    is_sendable: bool | None,
+    last_delivery_at: str | None,
+    invoice_number: int | str | None = None,
 ) -> tuple[str | None, str | None]:
     status = str(status_raw).strip().upper() if status_raw is not None else ""
     can_send = bool(is_sendable)
 
     if status == "DRAFT":
+        if invoice_number is None:
+            identifier_hint = "Use `tool_handle` from this response because `invoice_number` is assigned only after acceptance."
+        else:
+            identifier_hint = "You can use either `invoice_number` or `tool_handle`."
         return (
             "accept",
-            "Invoice is in DRAFT status. Next step: call invoicing_sales_invoice_action with action=accept.",
+            f"Invoice is in DRAFT status. Next step: call invoicing_sales_invoice_action with action=accept. {identifier_hint}",
         )
 
     if can_send and not last_delivery_at:
@@ -286,6 +294,7 @@ class SalesInvoiceSummary(AgentModel):
                 status_raw=self.status,
                 is_sendable=self.is_sendable,
                 last_delivery_at=self.last_delivery_at,
+                invoice_number=self.invoice_number,
             )
             if self.next_action is None:
                 self.next_action = next_action
@@ -356,6 +365,7 @@ class SalesInvoiceListItem(AgentModel):
             status_raw=data.get("status"),
             is_sendable=data.get("is_sendable"),
             last_delivery_at=data.get("last_delivery_at"),
+            invoice_number=data.get("invoice_number"),
         )
         if "next_action" not in data:
             data["next_action"] = next_action