mammoth-cli 2.2.52__tar.gz → 2.2.53__tar.gz

{mammoth_cli-2.2.52 → mammoth_cli-2.2.53}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: mammoth-cli
-Version: 2.2.52
+Version: 2.2.53
 Summary: Command-line interface for the Mammoth Analytics platform
 Project-URL: homepage, https://mammoth.io
 Project-URL: repository, https://github.com/EdgeMetric/mammothsdk

{mammoth_cli-2.2.52 → mammoth_cli-2.2.53}/mammoth_cli/__init__.py

@@ -6,7 +6,7 @@ not implement a second HTTP client and does not call private SDK members.
 
 from __future__ import annotations
 
-__version__ = "2.2.52"
+__version__ = "2.2.53"
 
 # The versioned machine output envelope contract.
 SCHEMA_VERSION = 1

{mammoth_cli-2.2.52 → mammoth_cli-2.2.53}/mammoth_cli/bundled_skill/mammoth-cli/references/report-checklist.md

@@ -9,7 +9,7 @@ Every "yes" must point at a command you ran in this task and its response.
 | Read back every value-changing transform | `view data get VIEW_ID DATASET_ID` (full page; `view data query` with `limit`/`condition` for a slice) after the job reached `success`; the sample shows the intended change and nothing else changed |
 | No measure column is all-zero, all-null, or all one value after a transform | inspect the readback sample; an all-`0` / all-`Unknown` column means the condition or column mapping was wrong, not that the data is like that |
 | A clean check is stated as checked | `column_checks` (issue types, `rows_checked`, `found`) on a data read or upload preview, `deliverable_check.checked`, `spelling_variants_checked`, `dataset_health[].health`, `project check` `checked`/`skipped`. `column_checks.error`, `preview_error`, `checked: false` or `skipped` means the check did not run: say so, do not report it as clean |
-| Decide on every blank the deliverable uses | one report line for each `blank_values` entry (`project check` lists them all: from every view and dashboard): the column, the count, and filled, filtered, or kept and why. Kept is a decision only when you write it down |
+| Decide on every blank the deliverable uses | one report line for each `blank_values` entry (`project check` lists them all: from every view and dashboard): the column, the count, and filled, filtered, or kept and why. Kept is a decision only when you write it down. A blank TEXT column carries `fill_template` (not `fix`): offer the fill, ask the user for the value, put it in place of `<value>`, never run the template as is |
 | A join states its match rate and explains unmatched rows | row count before/after, `IS_EMPTY` filter count on a right-side column, and the key columns named exactly as both view schemas show them |
 | Group counts add up | Σ of per-group counts from a pivot/crosstab equals the source row count (or nulls in the group key explain the difference) |
 | Every job you submitted reached a terminal state | `job get JOB_ID` shows `success`; report anything else as failed/unknown, not done |

{mammoth_cli-2.2.52 → mammoth_cli-2.2.53}/mammoth_cli/services/data_quality.py

@@ -378,21 +378,36 @@ def _blank_values_warning(
     total: int,
     checked: int,
     remove_fix: str | None,
+    fill_template: str | None = None,
 ) -> dict[str, Any]:
-    """The ``blank_values`` record; *remove_fix* when a blank row holds no figure at all."""
+    """The ``blank_values`` record.
+
+    *remove_fix* when a blank row holds no figure at all; *fill_template* (never a
+    ``fix``) for blank TEXT cells: it holds ``<value>``, which only the user can give.
+    """
     detail = f"{blanks} of {total} rows are blank. "
     if remove_fix is None:
-        detail += (
-            "Decide before you summarise: fill them (view transform set-values with "
-            "IS_EMPTY, or fill-missing), remove the rows (view transform filter), or "
-            "keep them and say so."
-        )
-        return {
+        if fill_template is not None:
+            detail += (
+                "Offer to fill them first: ask the user which value to write, put it in "
+                "fill_template in place of <value> and run that only once they answer. "
+                "Otherwise remove the rows (view transform filter) or keep them and say so."
+            )
+        else:
+            detail += (
+                "Decide before you summarise: fill them (view transform set-values with "
+                "IS_EMPTY, or fill-missing), remove the rows (view transform filter), or "
+                "keep them and say so."
+            )
+        record: dict[str, Any] = {
             "column": column,
             "issue": "blank_values",
             "detail": detail,
             "rows_checked": checked,
         }
+        if fill_template is not None:
+            record["fill_template"] = fill_template
+        return record
     detail += (
         f"{column} is the only number that varies between rows, so these rows hold no "
         "figure: removing them (fix) is the clean-up. Fill them only with a value the "
@@ -407,6 +422,24 @@ def _blank_values_warning(
     }
 
 
+def _fill_blank_text_template(
+    view_id: int, dataset_id: int, column: str, project_id: int | None
+) -> str:
+    """``set-values`` writing ``<value>`` into the column's blank cells: a template, not a fix."""
+    spec = json.dumps(
+        {
+            "existing_column": column,
+            "condition": {"column": column, "operator": "IS_EMPTY"},
+            "values": [{"value": "<value>"}],
+            "dataset_id": dataset_id,
+        }
+    )
+    return (
+        f"mammoth view transform set-values {view_id} --input {shlex.quote(spec)}"
+        f"{_project_flag(project_id)}"
+    )
+
+
 def _remove_blank_rows_hint(
     view_id: int, dataset_id: int, column: str, project_id: int | None
 ) -> str:
@@ -445,8 +478,9 @@ def column_warnings(
         (``numbers_stored_as_text``, ``dates_stored_as_text``,
         ``variant_spellings``, ``renamed_label``, ``blank_values`` or the table-level
         ``duplicate_rows``), ``detail`` and, where one command fixes it,
-        ``fix``. Counts are over the rows given (``rows_checked`` on each
-        record).
+        ``fix``. A blank TEXT column carries ``fill_template`` instead: a command
+        with a ``<value>`` placeholder the user must supply, never run as is. Counts
+        are over the rows given (``rows_checked`` on each record).
     """
     materialised = [row for row in rows if isinstance(row, Mapping)]
     if not materialised:
@@ -515,5 +549,17 @@ def column_warnings(
             remove_fix = None
             if figures == [column] and view_id is not None and dataset_id is not None:
                 remove_fix = _remove_blank_rows_hint(view_id, dataset_id, column, project_id)
-            warnings.append(_blank_values_warning(column, blanks, len(values), checked, remove_fix))
+            fill_template = None
+            if (
+                remove_fix is None
+                and str(col_type).upper() == "TEXT"
+                and view_id is not None
+                and dataset_id is not None
+            ):
+                fill_template = _fill_blank_text_template(view_id, dataset_id, column, project_id)
+            warnings.append(
+                _blank_values_warning(
+                    column, blanks, len(values), checked, remove_fix, fill_template
+                )
+            )
     return warnings

{mammoth_cli-2.2.52 → mammoth_cli-2.2.53}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "mammoth-cli"
-version = "2.2.52"
+version = "2.2.53"
 description = "Command-line interface for the Mammoth Analytics platform"
 authors = [{name = "Mammoth Analytics", email = "support@mammoth.io"}]
 readme = "README.md"