lambda-erp 0.8.15__tar.gz → 0.8.17__tar.gz

{lambda_erp-0.8.15 → lambda_erp-0.8.17}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: lambda-erp
-Version: 0.8.15
+Version: 0.8.17
 Summary: Core ERP logic - accounting, sales, purchasing, inventory
 Author: TORUS INVESTMENTS AG
 License-Expression: Apache-2.0
@@ -462,7 +462,6 @@ This is early. The project needs:
 - Country compliance packs (tax rules, invoice formats, mandatory fields)
 - Industry templates (services, retail, light manufacturing, SaaS)
 - More preset reports
-- A Postgres storage adapter (the current SQLite layer is fine for local evaluation but will need to be swapped for real multi-user write loads)
 - Better observability around token spend per turn
 - Native messenger integration, for WhatsApp, Telegram, etc.
 
@@ -482,13 +481,6 @@ Release notes live in [CHANGELOG.md](./CHANGELOG.md). Releases are tagged
 `vX.Y.Z` and published in lockstep to PyPI (`lambda-erp`) and npm
 (`@lambda-development/erp-core`).
 
----
-
-## Status
-
-Version 0, and it already implements the vision: invoicing, inventory, accounting, and reporting — all driven by chat. Good for demos, internal tools, and real day-to-day work. It's under active development and improving fast.
-
-If you try it, we'd love to know what broke.
 
 ---
 

{lambda_erp-0.8.15 → lambda_erp-0.8.17}/README.md

@@ -434,7 +434,6 @@ This is early. The project needs:
 - Country compliance packs (tax rules, invoice formats, mandatory fields)
 - Industry templates (services, retail, light manufacturing, SaaS)
 - More preset reports
-- A Postgres storage adapter (the current SQLite layer is fine for local evaluation but will need to be swapped for real multi-user write loads)
 - Better observability around token spend per turn
 - Native messenger integration, for WhatsApp, Telegram, etc.
 
@@ -454,13 +453,6 @@ Release notes live in [CHANGELOG.md](./CHANGELOG.md). Releases are tagged
 `vX.Y.Z` and published in lockstep to PyPI (`lambda-erp`) and npm
 (`@lambda-development/erp-core`).
 
----
-
-## Status
-
-Version 0, and it already implements the vision: invoicing, inventory, accounting, and reporting — all driven by chat. Good for demos, internal tools, and real day-to-day work. It's under active development and improving fast.
-
-If you try it, we'd love to know what broke.
 
 ---
 

{lambda_erp-0.8.15 → lambda_erp-0.8.17}/api/routers/masters.py

@@ -10,7 +10,8 @@ from lambda_erp.utils import _dict, now
 # them; imported here (not moved) so existing `from api.routers.masters import
 # MASTER_IDENTITY_ALIAS`-style imports keep working.
 from api.services import (
-    MASTER_TABLES, MASTER_NAME_PREFIXES, MASTER_RANDOM_NAME_TYPES, MASTER_IDENTITY_ALIAS,
+    MASTER_TABLES, MASTER_NAME_PREFIXES, MASTER_NAME_DIGITS, MASTER_RANDOM_NAME_TYPES,
+    MASTER_IDENTITY_ALIAS,
     MASTER_REFERENCE_CHECKS,
     _search_clause, _where_from_filters, master_search_columns, count_query_cached,
 )
@@ -161,7 +162,7 @@ def _find_reference(master_type: str, name: str) -> str | None:
     return None
 
 
-def _generate_master_name(db, doctype: str, prefix: str) -> str:
+def _generate_master_name(db, doctype: str, prefix: str, digits: int = 3) -> str:
     # Take the highest numeric suffix among strict "PREFIX-<digits>" names.
     # Names with a non-numeric tail (e.g. a custom "ITEM-COST-TEST") are ignored
     # rather than parsed — string-sorting them to the top used to reset the
@@ -179,9 +180,9 @@ def _generate_master_name(db, doctype: str, prefix: str) -> str:
 
     # Skip any number already taken by a non-standard name so we never collide.
     number = max_number + 1
-    while db.exists(doctype, f"{prefix}-{number:03d}"):
+    while db.exists(doctype, f"{prefix}-{number:0{digits}d}"):
         number += 1
-    return f"{prefix}-{number:03d}"
+    return f"{prefix}-{number:0{digits}d}"
 
 
 def _generate_random_master_name(db, doctype: str, prefix: str) -> str:
@@ -230,7 +231,9 @@ def create_master_record(master_type: str, data: dict) -> dict:
             if master_type in MASTER_RANDOM_NAME_TYPES:
                 doc["name"] = _generate_random_master_name(db, doctype, prefix)
             else:
-                doc["name"] = _generate_master_name(db, doctype, prefix)
+                doc["name"] = _generate_master_name(
+                    db, doctype, prefix, MASTER_NAME_DIGITS.get(master_type, 3)
+                )
         elif master_type == "company" and doc.get("company_name"):
             # A company's id is conventionally its name (mirrors /setup/company).
             doc["name"] = doc["company_name"]
@@ -300,9 +303,73 @@ def update_master_record(master_type: str, name: str, data: dict) -> dict:
 # ad-hoc equality field filter (validated against the master's real columns).
 _MASTER_LIST_RESERVED = {
     "limit", "offset", "include_disabled", "search", "search_fields", "fields",
+    "order_by", "order",
 }
 
 
+def _master_list_where(db, doctype: str, master_type: str, request: Request,
+                       include_disabled: bool, search: str | None,
+                       search_fields: str | None) -> tuple[list, list, set]:
+    """Build the validated WHERE clause shared by master list and adjacent.
+
+    Keeping this in one place is what makes detail prev/next follow the exact
+    search and field filters of the list the user opened the record from.
+    """
+    columns = db._get_table_columns(doctype)
+    where_parts: list = []
+    params: list = []
+
+    active = None if include_disabled else _with_active_filter(db, doctype)
+    if active:
+        wp, ps = _where_from_filters(active)
+        where_parts += wp
+        params += ps
+
+    for key, value in request.query_params.items():
+        if key in _MASTER_LIST_RESERVED:
+            continue
+        if key not in columns:
+            raise HTTPException(status_code=400, detail=f"Unknown filter field: {key}")
+        where_parts.append(f'"{key}" = ?')
+        params.append(value)
+
+    if search:
+        requested = [f.strip() for f in (search_fields or "").split(",") if f.strip()]
+        unknown = [f for f in requested if f not in columns]
+        if unknown:
+            raise HTTPException(status_code=400, detail=f"Unknown search field(s): {', '.join(unknown)}")
+        sf = requested or master_search_columns(db, doctype)
+        clause, sp = _search_clause(db, doctype, master_type, search, sf)
+        if clause:
+            where_parts.append(clause)
+            params += sp
+
+    return where_parts, params, columns
+
+
+def _master_order(columns: set, order_by: str | None, order: str) -> tuple[str, str]:
+    if order_by is not None and order_by not in columns:
+        raise HTTPException(status_code=400, detail=f"Unknown order_by field: {order_by}")
+    direction = order.lower()
+    if direction not in ("asc", "desc"):
+        raise HTTPException(status_code=400, detail="order must be 'asc' or 'desc'")
+    # Reference masters retain their historical default: stable ID order.
+    return order_by or "name", direction if order_by else "asc"
+
+
+def _master_order_sql(column: str, direction: str, *, reverse: bool = False) -> str:
+    """Deterministic master ordering with NULL values consistently at the end."""
+    effective = direction
+    nulls = "LAST"
+    if reverse:
+        effective = "desc" if direction == "asc" else "asc"
+        nulls = "FIRST"
+    sql = f'"{column}" {effective.upper()}'
+    if column != "name":
+        sql += f" NULLS {nulls}, name {effective.upper()}"
+    return sql
+
+
 @router.get("/{master_type}/filter-values")
 def master_filter_values(
     master_type: str,
@@ -335,52 +402,22 @@ def list_masters(
     search: str | None = None,
     search_fields: str | None = None,
     fields: str | None = None,
+    order_by: str | None = None,
+    order: str = "asc",
     _user: dict = _viewer,
 ):
     doctype, _ = _get_table(master_type)
     if not doctype:
         return {"detail": f"Unknown master type: {master_type}"}
     db = get_db()
-    columns = db._get_table_columns(doctype)
-
-    where_parts: list = []
-    params: list = []
-
-    # Active (non-disabled) filter, unless the caller opts in to disabled rows.
-    active = None if include_disabled else _with_active_filter(db, doctype)
-    if active:
-        wp, ps = _where_from_filters(active)
-        where_parts += wp
-        params += ps
-
-    # Ad-hoc equality field filters: any remaining query param naming a real
-    # column (e.g. /masters/customer?customer_group=Commercial). Unknown field
-    # -> 400 (a filter on a missing column is a client bug, never interpolated).
-    for key, value in request.query_params.items():
-        if key in _MASTER_LIST_RESERVED:
-            continue
-        if key not in columns:
-            raise HTTPException(status_code=400, detail=f"Unknown filter field: {key}")
-        where_parts.append(f'"{key}" = ?')
-        params.append(value)
-
-    # Free-text search — defaults to the master's text columns when the caller
-    # doesn't name search_fields (so a plain ?search=... just works).
-    if search:
-        sf = [f.strip() for f in (search_fields or "").split(",")
-              if f.strip() and f.strip() in columns]
-        if not sf:
-            sf = master_search_columns(db, doctype)
-        clause, sp = _search_clause(db, doctype, master_type, search, sf)
-        if clause:
-            where_parts.append(clause)
-            params += sp
+    where_parts, params, columns = _master_list_where(
+        db, doctype, master_type, request, include_disabled, search, search_fields,
+    )
+    sort_column, sort_direction = _master_order(columns, order_by, order)
 
     where = (" WHERE " + " AND ".join(where_parts)) if where_parts else ""
     total = count_query_cached(f'SELECT COUNT(*) as c FROM "{doctype}"{where}', params)
 
-    # Deterministic order by the `name` PK (already indexed) — required for
-    # OFFSET pagination and the /adjacent prev/next contract.
     requested_fields = [f.strip() for f in (fields or "").split(",") if f.strip()]
     if requested_fields:
         unknown = [f for f in requested_fields if f not in columns]
@@ -391,7 +428,10 @@ def list_masters(
         projection = ", ".join(f'"{f}"' for f in requested_fields)
     else:
         projection = "*"
-    query = f'SELECT {projection} FROM "{doctype}"{where} ORDER BY name LIMIT {int(limit)}'
+    query = (
+        f'SELECT {projection} FROM "{doctype}"{where} '
+        f'ORDER BY {_master_order_sql(sort_column, sort_direction)} LIMIT {int(limit)}'
+    )
     if offset:
         query += f" OFFSET {int(offset)}"
     rows = db.sql(query, params)
@@ -421,26 +461,79 @@ def search_masters(master_type: str, q: str = "", _user: dict = _viewer):
 
 
 @router.get("/{master_type}/{name}/adjacent")
-def adjacent_master(master_type: str, name: str, include_disabled: bool = True,
-                    _user: dict = _viewer):
-    """The records immediately before/after `name` in the list's order (name
-    ASC) — {"prev": name|None, "next": name|None}. Keyset queries on the
-    primary key. include_disabled defaults True to step 1:1 with the master
-    list, which shows disabled records too."""
+def adjacent_master(
+    master_type: str,
+    name: str,
+    request: Request,
+    include_disabled: bool = True,
+    search: str | None = None,
+    search_fields: str | None = None,
+    order_by: str | None = None,
+    order: str = "asc",
+    _user: dict = _viewer,
+):
+    """Records immediately before/after ``name`` in the current master list.
+
+    Search, equality filters, disabled visibility, and ordering are shared with
+    ``list_masters``. Keyset predicates keep navigation proportional to one
+    index lookup rather than numbering the entire result set.
+    """
     doctype, _ = _get_table(master_type)
     if not doctype:
         raise HTTPException(status_code=404, detail=f"Unknown master type: {master_type}")
     db = get_db()
-    active = ("" if include_disabled or "disabled" not in db._get_table_columns(doctype)
-              else "disabled = 0 AND ")
+    where_parts, params, columns = _master_list_where(
+        db, doctype, master_type, request, include_disabled, search, search_fields,
+    )
+    sort_column, sort_direction = _master_order(columns, order_by, order)
+    base_where = (" AND ".join(where_parts)) if where_parts else "1 = 1"
+    current_rows = db.sql(
+        f'SELECT "{sort_column}" AS sort_value FROM "{doctype}" '
+        f"WHERE {base_where} AND name = ? LIMIT 1",
+        [*params, name],
+    )
+    if not current_rows:
+        return {"prev": None, "next": None}
+    current_value = current_rows[0]["sort_value"]
+
+    def neighbor(toward: str):
+        after = toward == "next"
+        same_cmp = ">" if sort_direction == "asc" else "<"
+        primary_cmp = same_cmp
+        if not after:
+            same_cmp = "<" if same_cmp == ">" else ">"
+            primary_cmp = "<" if primary_cmp == ">" else ">"
+
+        if sort_column == "name":
+            keyset = f'name {same_cmp} ?'
+            key_params = [name]
+        elif current_value is None:
+            if after:
+                keyset = f'"{sort_column}" IS NULL AND name {same_cmp} ?'
+            else:
+                keyset = (
+                    f'("{sort_column}" IS NOT NULL OR '
+                    f'("{sort_column}" IS NULL AND name {same_cmp} ?))'
+                )
+            key_params = [name]
+        else:
+            keyset = (
+                f'("{sort_column}" {primary_cmp} ? OR '
+                f'("{sort_column}" = ? AND name {same_cmp} ?)'
+            )
+            key_params = [current_value, current_value, name]
+            if after:
+                keyset += f' OR "{sort_column}" IS NULL'
+            keyset += ")"
 
-    def neighbor(cmp: str, direction: str):
         rows = db.sql(
-            f'SELECT name FROM "{doctype}" WHERE {active}name {cmp} ? '
-            f"ORDER BY name {direction} LIMIT 1", [name])
+            f'SELECT name FROM "{doctype}" WHERE {base_where} AND {keyset} '
+            f'ORDER BY {_master_order_sql(sort_column, sort_direction, reverse=not after)} LIMIT 1',
+            [*params, *key_params],
+        )
         return rows[0]["name"] if rows else None
 
-    return {"prev": neighbor("<", "DESC"), "next": neighbor(">", "ASC")}
+    return {"prev": neighbor("prev"), "next": neighbor("next")}
 
 
 @router.get("/{master_type}/{name}")

{lambda_erp-0.8.15 → lambda_erp-0.8.17}/api/services.py

@@ -90,6 +90,10 @@ MASTER_NAME_PREFIXES = {
     "warehouse": "WH",
 }
 
+# Minimum numeric suffix width for sequential plugin master ids. Core masters
+# retain the historical three-digit form; plugins can opt into e.g. FIDX-0001.
+MASTER_NAME_DIGITS: dict[str, int] = {}
+
 # High-volume plugin masters can opt out of sequential ids. Sequential naming
 # must inspect existing suffixes after a process restart; opaque random ids keep
 # inserts constant-time for index-scale reference tables.
@@ -218,6 +222,7 @@ def document_columns(doctype_slug: str) -> set:
 
 def register_master(slug: str, table: str, name_field: str, *,
                     name_prefix: str | None = None,
+                    name_digits: int = 3,
                     random_name: bool = False,
                     identity_alias: str | None = None,
                     description: str | None = None,
@@ -236,12 +241,16 @@ def register_master(slug: str, table: str, name_field: str, *,
 
     `name_field` is the human display column (e.g. "company_name").
     `name_prefix` enables auto-generated ids (prefix "LEAD" -> LEAD-001) when a
-    record is created without an explicit `name`. `identity_alias` lets callers
-    address the `name` PK under a friendlier key, like item's `item_code`.
+    record is created without an explicit `name`. `name_digits` controls the
+    minimum sequential suffix width. `identity_alias` lets callers address the
+    `name` PK under a friendlier key, like item's `item_code`.
     """
     MASTER_TABLES[slug] = (table, name_field)
     if name_prefix:
+        if name_digits < 1:
+            raise ValueError("name_digits must be at least 1")
         MASTER_NAME_PREFIXES[slug] = name_prefix
+        MASTER_NAME_DIGITS[slug] = name_digits
     if random_name:
         if not name_prefix:
             raise ValueError("random_name requires name_prefix")

{lambda_erp-0.8.15 → lambda_erp-0.8.17}/frontend/src/api/client.ts

@@ -180,10 +180,10 @@ export const api = {
       `/documents/${doctype}/${encodeURIComponent(name)}/adjacent${qs(params)}`,
     ),
 
-  // Prev/next master record around `name` in the master list's order (name ASC).
-  adjacentMaster: (masterType: string, name: string) =>
+  // Prev/next master record around `name` in the same order+filters as its list.
+  adjacentMaster: (masterType: string, name: string, params?: Record<string, string | number | undefined>) =>
     request<{ prev: string | null; next: string | null }>(
-      `/masters/${masterType}/${encodeURIComponent(name)}/adjacent`,
+      `/masters/${masterType}/${encodeURIComponent(name)}/adjacent${qs(params)}`,
     ),
 
   createDocument: (doctype: string, data: any) =>

{lambda_erp-0.8.15 → lambda_erp-0.8.17}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "lambda-erp"
-version = "0.8.15"
+version = "0.8.17"
 description = "Core ERP logic - accounting, sales, purchasing, inventory"
 readme = "README.md"
 license = "Apache-2.0"