acumatica-cli 0.9.0__tar.gz → 0.9.1__tar.gz

{acumatica_cli-0.9.0 → acumatica_cli-0.9.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.3
 Name: acumatica-cli
-Version: 0.9.0
+Version: 0.9.1
 Summary: Acumatica ERP Config-as-Code: tenant provisioning, baseline config, and reference data
 Author: Konstantin Borovik
 Author-email: Konstantin Borovik <kb@lab5.ca>

{acumatica_cli-0.9.0 → acumatica_cli-0.9.1}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "acumatica-cli"
-version = "0.9.0"
+version = "0.9.1"
 description = "Acumatica ERP Config-as-Code: tenant provisioning, baseline config, and reference data"
 authors = [{ name = "Konstantin Borovik", email = "kb@lab5.ca" }]
 readme = "README.md"

{acumatica_cli-0.9.0 → acumatica_cli-0.9.1}/src/acumatica_cli/client.py

@@ -93,6 +93,9 @@ class AcumaticaClient:
         self._http = httpx.Client(
             base_url=instance.base_url, timeout=timeout, transport=transport
         )
+        # V5/B24: first successful Company PUT this session → one re-login
+        # so later branch selectors resolve; warm re-PUT is re-login-safe
+        self._refreshed_after_company = False
 
     def __enter__(self) -> AcumaticaClient:
         # tenant guard (V5, docs/rest-api.md): an omitted or empty tenant is
@@ -105,27 +108,10 @@ class AcumaticaClient:
                 "an explicit tenant silently lands on the default tenant; "
                 "set ACU_TENANT in .env or pass --tenant"
             )
-        creds: dict[str, str] = {
-            "name": self.instance.user,
-            "password": self.instance.password,
-            "tenant": self.instance.tenant,
-        }
-        self._checked(self._http.post("/entity/auth/login", json=creds))
-        # tenant guard, landed side (V5, B5): login accepting the name proves
-        # nothing - a stale tenant map reroutes named logins to the default
-        # tenant, and a single-tenant instance accepts ANY name (both verified
-        # live, docs/rest-api.md). Verify where the session actually landed
-        # and refuse on mismatch; logout first, since __exit__ never runs
-        # when __enter__ raises (V6 - sessions count against the license).
+        # logout first on failure: __exit__ never runs when __enter__ raises
+        # (V6 - sessions count against the license)
         try:
-            landed = self._landed_tenant()
-            if landed.casefold() != self.instance.tenant.casefold():
-                raise RuntimeError(
-                    f"tenant guard: asked for tenant {self.instance.tenant!r} "
-                    f"but the session landed on {landed!r} - the instance "
-                    "tenant map is stale or the tenant does not exist; check "
-                    "acu tenant list and recycle the app pool"
-                )
+            self._login()
         except BaseException:
             self.__exit__()
             raise
@@ -138,6 +124,50 @@ class AcumaticaClient:
         finally:
             self._http.close()
 
+    def _login(self) -> None:
+        """POST login + refuse on landed-tenant mismatch (V5, B5).
+
+        Login accepting the name proves nothing - a stale tenant map
+        reroutes named logins to the default tenant, and a single-tenant
+        instance accepts ANY name (both verified live, docs/rest-api.md).
+        """
+        creds: dict[str, str] = {
+            "name": self.instance.user,
+            "password": self.instance.password,
+            "tenant": self.instance.tenant,
+        }
+        self._checked(self._http.post("/entity/auth/login", json=creds))
+        landed = self._landed_tenant()
+        if landed.casefold() != self.instance.tenant.casefold():
+            raise RuntimeError(
+                f"tenant guard: asked for tenant {self.instance.tenant!r} "
+                f"but the session landed on {landed!r} - the instance "
+                "tenant map is stale or the tenant does not exist; check "
+                "acu tenant list and recycle the app pool"
+            )
+
+    def relogin(self) -> None:
+        """Drop the cookie session and open a new one without closing transport.
+
+        Mid-session Company create leaves branch selectors empty until a
+        fresh login (V5, B24). Unlike ``__exit__``, the httpx client stays
+        open so the outer context manager still owns the session lifecycle.
+        """
+        # empty body → Content-Length: 0 (V6 / IIS 411)
+        self._http.post("/entity/auth/logout", content=b"")
+        self._login()
+
+    def refresh_after_company(self) -> None:
+        """Re-login once after the first successful Company PUT this session.
+
+        Warm-tenant Company re-PUT is re-login-safe (V5); further Company
+        records in the same run skip the second bounce.
+        """
+        if self._refreshed_after_company:
+            return
+        self.relogin()
+        self._refreshed_after_company = True
+
     def _landed_tenant(self) -> str:
         """The tenant this session actually landed on (login name).
 

{acumatica_cli-0.9.0 → acumatica_cli-0.9.1}/src/acumatica_cli/seed.py

@@ -72,6 +72,10 @@ BOOTSTRAP_ENDPOINT, BOOTSTRAP_ENTITIES = bootstrap.parse_endpoint(
 # the Instance field default rather than hand-synced (V11: one spelling).
 _DEFAULT_ENDPOINT: str = f"Default/{Instance.model_fields['api_version'].default}"
 _SYMBOLIC_BOOTSTRAP = "bootstrap"
+# Mid-session branch-selector failure (V5/B24): virgin-tenant apply opens
+# the REST session pre-Company; INPreferences TransitBranchID (and kin)
+# 500 with this message until a fresh login sees the new branch.
+_BRANCH_EMPTY = "'Branch' cannot be empty"
 
 
 def active_bootstrap(root: Path | None = None) -> tuple[str, frozenset[str]]:
@@ -303,6 +307,10 @@ def apply(
 ) -> int:
     """PUT every record (upsert by key); an action file invokes its action.
 
+    After the first successful Company PUT this session, re-login once so
+    branch selectors resolve for later files (V5/B24). A PUT that still
+    500s with ``'Branch' cannot be empty`` gets one re-login + retry.
+
     Returns the record count.
     """
     if isinstance(baseline, ActionFile):
@@ -315,11 +323,30 @@ def apply(
             body = record
             if any(isinstance(v, list) for v in record.values()):
                 body = _with_detail_ids(client, baseline, record)
-            client.put(baseline.entity, body, endpoint=baseline.endpoint)
+            _put(client, baseline.entity, body, baseline.endpoint)
+            if baseline.entity == "Company":
+                client.refresh_after_company()
             output.data(f"  PUT {baseline.entity} [{label}]")
     return len(baseline.records)
 
 
+def _put(
+    client: AcumaticaClient,
+    entity: str,
+    body: dict[str, Any],
+    endpoint: str | None,
+) -> None:
+    """PUT one record; one re-login + retry on branch-empty 500 (V5/B24)."""
+    try:
+        client.put(entity, body, endpoint=endpoint)
+    except RuntimeError as err:
+        if _BRANCH_EMPTY not in str(err):
+            raise
+        output.info(f"re-login and retry {entity} (branch empty)")
+        client.relogin()
+        client.put(entity, body, endpoint=endpoint)
+
+
 def _with_detail_ids(
     client: AcumaticaClient, baseline: BaselineFile, record: dict[str, Any]
 ) -> dict[str, Any]: