celerp 2.2.0__tar.gz → 2.2.1__tar.gz

{celerp-2.2.0 → celerp-2.2.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: celerp
-Version: 2.2.0
+Version: 2.2.1
 Summary: Free, self-hosted ERP for small businesses
 License: Business Source License 1.1
         

{celerp-2.2.0 → celerp-2.2.1}/celerp/output/pdf.py

@@ -349,7 +349,7 @@ def generate_document_pdf(doc: dict[str, Any], company: dict[str, Any] | None =
         if li_taxes:
             tax_str = " + ".join(
                 f"{t.get('label') or t.get('code') or 'Tax'} {float(t.get('rate', 0)):.1f}%"
-                for item in li_taxes
+                for t in li_taxes
             )
         else:
             tax_str = f"{float(li.get('tax_rate') or 0):.1f}%"

{celerp-2.2.0 → celerp-2.2.1}/celerp.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: celerp
-Version: 2.2.0
+Version: 2.2.1
 Summary: Free, self-hosted ERP for small businesses
 License: Business Source License 1.1
         

{celerp-2.2.0 → celerp-2.2.1}/celerp.egg-info/SOURCES.txt

@@ -7,6 +7,7 @@ NOTICE
 README.md
 alembic.ini
 conftest.py
+conftest_support.py
 pyproject.toml
 test_helpers.py
 celerp/__init__.py

{celerp-2.2.0 → celerp-2.2.1}/celerp.egg-info/scm_file_list.json

@@ -214,6 +214,7 @@
     "celerp/session_gate.py",
     "celerp/tax_regimes.py",
     "conftest.py",
+    "conftest_support.py",
     "core/.github/workflows/ci.yml",
     "default_modules/__init__.py",
     "default_modules/celerp-accounting/LICENSE",
@@ -709,6 +710,7 @@
     "pyproject.toml",
     "scripts/apply_i18n.py",
     "scripts/bump-version.sh",
+    "scripts/check_undefined_names.py",
     "scripts/download-python.sh",
     "scripts/extract_i18n.py",
     "scripts/gen_first_party_lock.py",
@@ -790,6 +792,7 @@
     "tests/test_browser/test_production_order.py",
     "tests/test_browser/test_reorder_digest_upsell.py",
     "tests/test_browser/test_reprice_preserves_lines.py",
+    "tests/test_browser/test_reprice_wrong_lot.py",
     "tests/test_browser/test_reserved_conflict_modal.py",
     "tests/test_browser/test_settings_tabs.py",
     "tests/test_browser/test_split_ghost_cols.py",
@@ -807,6 +810,7 @@
     "tests/test_browser/test_worksheet_workflow.py",
     "tests/test_bulk_upload_limits.py",
     "tests/test_cell_placeholder.py",
+    "tests/test_check_undefined_names.py",
     "tests/test_clear_fields.py",
     "tests/test_cli.py",
     "tests/test_cloud_disconnect_persist.py",
@@ -907,6 +911,7 @@
     "tests/test_modules/test_registry.py",
     "tests/test_modules/test_slots.py",
     "tests/test_modules/test_slots_lifecycle.py",
+    "tests/test_no_undefined_names.py",
     "tests/test_notifications/__init__.py",
     "tests/test_notifications/test_router.py",
     "tests/test_notifications/test_service.py",
@@ -929,6 +934,7 @@
     "tests/test_projections/test_status_doc_backfill.py",
     "tests/test_purchasing_fields.py",
     "tests/test_python310_floor.py",
+    "tests/test_reconciliation_forms.py",
     "tests/test_regression.py",
     "tests/test_relay_account_proxy.py",
     "tests/test_reorder_v2.py",
@@ -1026,6 +1032,7 @@
     "tests/test_verticals_integrity.py",
     "tests/test_visual.py",
     "tests/test_weight_unit_behavior.py",
+    "tests/test_worker_config_isolation.py",
     "tests/test_xss_hygiene.py",
     "ui/__init__.py",
     "ui/api_client.py",

celerp-2.2.1/celerp.egg-info/scm_version.json

@@ -0,0 +1,8 @@
+{
+  "tag": "2.2.1",
+  "distance": 0,
+  "node": "g2590c97b63d729423ef35d9ba2abdda76459c902",
+  "dirty": false,
+  "branch": "HEAD",
+  "node_date": "2026-08-18"
+}

{celerp-2.2.0 → celerp-2.2.1}/conftest.py

@@ -5,29 +5,26 @@ from __future__ import annotations
 
 import os
 
+from conftest_support import is_own_test_config, resolve_worker_config
+
 # Must be set before celerp.config is imported (JWT guard fires at module load).
 os.environ.setdefault("ALLOW_INSECURE_JWT", "true")
 
-# Point config.toml at a per-worker temp file. Otherwise every xdist worker
-# shares ~/.config/celerp/config.toml, which ensure_instance_id() reads+writes
-# on the cloud-relay endpoints — concurrent access across workers (and leakage
-# of one test's instance_id/token into the next) made those tests flaky.
-import tempfile as _tempfile
+# Point config.toml at a per-worker temp file. Otherwise every xdist worker shares
+# ~/.config/celerp/config.toml, which ensure_instance_id() reads+writes on the
+# cloud-relay endpoints: concurrent access across workers corrupts the file
+# mid-write (a torn read raises tomllib.TOMLDecodeError) and leaks one test's
+# instance_id/token into the next. resolve_worker_config is the shared rule (see
+# its docstring for why a plain setdefault does not achieve this under xdist).
 _worker = os.environ.get("PYTEST_XDIST_WORKER", "main")
-os.environ.setdefault(
-    "CELERP_CONFIG",
-    os.path.join(_tempfile.gettempdir(), f"celerp-test-config-{_worker}.toml"),
-)
+os.environ["CELERP_CONFIG"] = resolve_worker_config(os.environ.get("CELERP_CONFIG"), _worker)
+
 # Start from a clean config: a temp file left by a previous local run (e.g. a
 # persisted [cloud] disconnected = true from the disconnect endpoint) must not
 # leak into this session. Scoped to our own temp path so a real CELERP_CONFIG
 # passed in by CI is never touched.
 _cfg_start = os.environ["CELERP_CONFIG"]
-if (
-    os.path.dirname(_cfg_start) == _tempfile.gettempdir()
-    and os.path.basename(_cfg_start).startswith("celerp-test-config-")
-    and os.path.exists(_cfg_start)
-):
+if is_own_test_config(_cfg_start) and os.path.exists(_cfg_start):
     os.remove(_cfg_start)
 
 # ── Postgres for the whole test suite ──────────────────────────────────────────

celerp-2.2.1/conftest_support.py

@@ -0,0 +1,44 @@
+# Copyright (c) 2026 Noah Severs
+# SPDX-License-Identifier: LicenseRef-Proprietary
+"""Side-effect-free helpers for the root conftest.
+
+Kept in a separate module (never collected as a test, never provisions a
+database on import) so the per-worker config-isolation rule can be unit tested
+without importing the root conftest, which spins up Postgres at import time.
+"""
+from __future__ import annotations
+
+import os
+import tempfile
+
+_CONFIG_PREFIX = "celerp-test-config-"
+
+
+def is_own_test_config(path: str | None, tmpdir: str | None = None) -> bool:
+    """True for a config path this harness itself created (our per-worker temp).
+
+    Used to tell an inherited value we set from a genuine external CELERP_CONFIG:
+    only the former may be recomputed per worker, the latter is always honored.
+    """
+    if path is None:
+        return False
+    tmpdir = tmpdir or tempfile.gettempdir()
+    return os.path.dirname(path) == tmpdir and os.path.basename(path).startswith(_CONFIG_PREFIX)
+
+
+def resolve_worker_config(existing: str | None, worker: str, tmpdir: str | None = None) -> str:
+    """Return the config.toml path this worker must use.
+
+    An xdist worker inherits the controller process's environment, where the root
+    conftest already ran and set CELERP_CONFIG to the controller's ("main") path.
+    A plain setdefault would then leave every worker pointing at that one shared
+    file, so concurrent read/write across workers corrupts it mid-write (a torn
+    file fails tomllib.load and 500s unrelated requests). So whenever the inherited
+    value is one of our own temp paths (or unset), recompute it for THIS worker; a
+    genuine external CELERP_CONFIG is always left untouched.
+    """
+    tmpdir = tmpdir or tempfile.gettempdir()
+    own = os.path.join(tmpdir, f"{_CONFIG_PREFIX}{worker}.toml")
+    if existing is None or is_own_test_config(existing, tmpdir):
+        return own
+    return existing

{celerp-2.2.0 → celerp-2.2.1}/default_modules/celerp-inventory/celerp_inventory/routes.py

@@ -1888,7 +1888,16 @@ async def split_item(entity_id: str, payload: SplitBody, company_id=Depends(get_
     parent_attrs_wo_pieces = {k: v for k, v in parent_attrs.items() if k != "pieces"}
 
     parent_weight: float | None = _read_float(parent.state, "weight")
-    parent_weight_unit = parent.state.get("weight_unit") or "gram"
+    parent_is_weight_unit = is_weight_unit(parent_sell_by, unit_map)
+    # No weight was ever recorded (e.g. a CSV import that never synced it from
+    # quantity): fall back to qty, same as split_preview, so the mother-weight
+    # column the preview showed doesn't demand edit_inventory_amounts to confirm
+    # the exact value it already displayed. A genuinely stored weight (which may
+    # legitimately differ from qty) is never overridden.
+    parent_weight_is_derived = parent_weight is None and parent_is_weight_unit
+    if parent_weight_is_derived:
+        parent_weight = parent_qty
+    parent_weight_unit = parent.state.get("weight_unit") or (parent_sell_by if parent_is_weight_unit else "gram")
     weight_unit_cfg = unit_map.get(parent_weight_unit) or {}
     weight_decimals = weight_unit_cfg.get("decimals", 2)
 
@@ -1952,6 +1961,11 @@ async def split_item(entity_id: str, payload: SplitBody, company_id=Depends(get_
     else:
         _child_cost_totals = [None] * len(children)
 
+    def _child_weight(c) -> float | None:
+        if c.weight is not None:
+            return c.weight
+        return c.quantity if parent_is_weight_unit else None
+
     # Per-child history descriptors with sequential mother deltas (one row per child).
     children_detail: list[dict] = []
     running_qty = parent_qty
@@ -2000,7 +2014,7 @@ async def split_item(entity_id: str, payload: SplitBody, company_id=Depends(get_
 
         # Origin marker on the child: "Split from <mother>" — the child's first history entry.
         ch_pieces = _to_int_pieces(child.attributes.get("pieces", 0)) if parent_pieces is not None else None
-        ch_weight = child.weight if (parent_weight is not None and child.weight is not None) else None
+        ch_weight = _child_weight(child) if parent_weight is not None else None
         origin = await emit_event(
             session,
             company_id=company_id,
@@ -2075,8 +2089,7 @@ async def split_item(entity_id: str, payload: SplitBody, company_id=Depends(get_
                 metadata_={"reason": "from_split"},
             )
 
-    # Reduce parent quantity — use explicit mother_qty override when provided (user re-weighed mother).
-    total_child_weight = sum(c.weight for c in payload.children if c.weight is not None)
+    total_child_weight = sum(w for c in payload.children if (w := _child_weight(c)) is not None)
     derived_parent_qty = round(parent_qty - total_child_qty, 10)
     derived_mother_weight = (
         round(parent_weight - total_child_weight, weight_decimals) if parent_weight is not None else None

{celerp-2.2.0 → celerp-2.2.1}/default_modules/first_party.lock.json

@@ -7,7 +7,7 @@
   "celerp-contacts": "899cf9febb388356290f6eb8c8ab6d67f1f92a9c9fa8377ebb6d46bf04cedbc3",
   "celerp-dashboard": "00d2e8d6ef46a3afd5d6caf8434217095aabd40c963bfac19d0e988c714443ce",
   "celerp-docs": "8b69661d972454099f94fd0f22d0d3ce614bbc6a257effb776fdb12462a56b7b",
-  "celerp-inventory": "3beccf81df7dff80513d8de0b70db17924d2f5bcd8a111068fddcd76475096c4",
+  "celerp-inventory": "0d3b85ddb73ce02a2a97a5e00827435e145bddaae79491a7ad6b7ade88f38c06",
   "celerp-labels": "1175c234602fbc80e20a33a4d03bf7abcc9a7a78d6e83729c73106cb0af53832",
   "celerp-manufacturing": "5cc089a345eeff66be25db51432481487e699f28c0a545731650843fce81dc01",
   "celerp-reports": "f100f83b3fc3e4565685d3a0c8e62dcc724bb35d32238f051b05c5f97ae10d13",

{celerp-2.2.0 → celerp-2.2.1}/ui/routes/documents.py

@@ -2540,12 +2540,16 @@ celerpUpdateBulkAlloc();
                         repriced = 0
                         for line in lines:
                             sku = (line.get("sku") or "").strip()
-                            if sku:
+                            eid = (line.get("item_id") or line.get("entity_id") or "").strip()
+                            if sku or eid:
                                 try:
-                                    resp = await api.list_items(token, {"sku": sku, "limit": 1})
-                                    items = resp.get("items", []) if isinstance(resp, dict) else resp
-                                    if items:
-                                        new_price = resolve_price(items[0], new_pl)
+                                    item = await api.get_item(token, eid) if eid else None
+                                    if item is None and sku:
+                                        resp = await api.list_items(token, {"sku": sku, "limit": 1})
+                                        items = resp.get("items", []) if isinstance(resp, dict) else resp
+                                        item = items[0] if items else None
+                                    if item is not None:
+                                        new_price = resolve_price(item, new_pl)
                                         line = {**line, "unit_price": new_price, "price_list": new_pl}
                                         repriced += 1
                                 except Exception:
@@ -2900,13 +2904,16 @@ celerpUpdateBulkAlloc();
         updated_lines = []
         for line in existing_lines:
             sku = (line.get("sku") or "").strip()
-            if sku:
-                # Look up current item price via catalog endpoint (SKU lookup)
+            eid = (line.get("item_id") or line.get("entity_id") or "").strip()
+            if sku or eid:
                 try:
-                    resp = await api.list_items(token, {"sku": sku, "limit": 1})
-                    items = resp.get("items", []) if isinstance(resp, dict) else resp
-                    if items:
-                        new_price = resolve_price(items[0], price_list)
+                    item = await api.get_item(token, eid) if eid else None
+                    if item is None and sku:
+                        resp = await api.list_items(token, {"sku": sku, "limit": 1})
+                        items = resp.get("items", []) if isinstance(resp, dict) else resp
+                        item = items[0] if items else None
+                    if item is not None:
+                        new_price = resolve_price(item, price_list)
                         line = {**line, "unit_price": new_price, "price_list": price_list}
                         repriced += 1
                 except APIError:

{celerp-2.2.0 → celerp-2.2.1}/ui/routes/inventory.py

@@ -1855,6 +1855,9 @@ def setup_routes(app):
             for f in schema
         ]
         schema = _apply_amount_edit_permission(schema, _get_role(request), company.get("settings") or {})
+        if (str(item.get("status") or "").lower() == "draft"
+                and role_has_permission(company.get("settings") or {}, _get_role(request), "edit_inventory")):
+            schema = [{**f, "editable": True} if f.get("key") in AMOUNT_EDIT_GATED_KEYS else f for f in schema]
         # Merge category-specific fields for this item's category
         item_cat = item.get("category", "")
         if item_cat and item_cat in cat_schemas:
@@ -5438,7 +5441,8 @@ def _inventory_cell_renderers(schema: list[dict], unit_names: list[str] | None =
                     data_col="pieces",
                     data_decimals=str(decimals),
                 )
-            return display_cell(entity_id=entity_id, field="pieces", value=row.get("pieces", ""), cell_type="number", editable=_ed)
+            _rek = set(row.get("_row_editable_keys") or ())
+            return display_cell(entity_id=entity_id, field="pieces", value=row.get("pieces", ""), cell_type="number", editable=_ed or "pieces" in _rek)
         renderers["pieces"] = _pieces_renderer
 
     # Status renderer: a doc-driven status (sold, memo_out, consigned-in stock) carries

{celerp-2.2.0 → celerp-2.2.1}/ui/routes/reconciliation.py

@@ -204,7 +204,7 @@ def _create_form(session_id: str, line_id: str, line: dict, chart: list[dict], c
             Div(
                 Label(t("acct.je_line_party"), cls="form-label"),
                 searchable_select("contact", contact_opts, placeholder=t("acct.je_line_party"),
-                                  cls_extra="form-input", search_url=_PARTY_SEARCH_URL),
+                                  cls_extra="form-input", search_url=PARTY_SEARCH_URL),
                 cls="form-field",
             ),
             Div(
@@ -260,7 +260,7 @@ def _split_form(session_id: str, line_id: str, line: dict, chart: list[dict], cu
                               step="0.01", cls="form-input max-w-sm"),
                         searchable_select("contact_0", contact_opts,
                                           placeholder=t("acct.je_line_party"),
-                                          cls_extra="form-input", search_url=_PARTY_SEARCH_URL),
+                                          cls_extra="form-input", search_url=PARTY_SEARCH_URL),
                         Input(type="text", name="memo_0", placeholder="Memo", cls="form-input"),
                         cls="split-entry-row flex-row gap-sm mb-sm",
                     )

{celerp-2.2.0 → celerp-2.2.1}/ui/routes/setup.py

@@ -204,12 +204,12 @@ def setup_routes(app):
             # Store the vertical name in company settings so the dashboard can use it
             try:
                 async with api._client(token) as c:
-                    raw = _raise(await c.get("/companies/me")).json()
+                    raw = api._raise(await c.get("/companies/me")).json()
                     settings = dict(raw.get("settings") or {})
                     settings["vertical"] = vertical
-                    await c.patch("/companies/me", json={"name": raw.get("name", ""), "settings": settings})
-            except Exception:
-                pass
+                    api._raise(await c.patch("/companies/me", json={"name": raw.get("name", ""), "settings": settings}))
+            except Exception as exc:
+                logger.warning("storing vertical failed during setup wizard: %s", exc)
             # Re-seed demo items with vertical-aware examples now that vertical is set
             try:
                 async with api._client(token) as c:

celerp-2.2.0/celerp.egg-info/scm_version.json

@@ -1,8 +0,0 @@
-{
-  "tag": "2.2.0",
-  "distance": 0,
-  "node": "g7294eca402bb2083fa7d15e2b75ef273f19238f4",
-  "dirty": false,
-  "branch": "HEAD",
-  "node_date": "2026-08-16"
-}