okstra 0.49.0 → 0.51.0

package/runtime/python/okstra_ctl/report_views.py

@@ -3,9 +3,9 @@
 Single product, single source of truth:
 
 * ``render_html(src_md, *, run_meta)`` — deterministic self-contained
-  HTML renderer for human readers. Sections §5/§6/§7 user-actionable
-  rows (those reachable from §5 ``C-*`` IDs) get embedded ``<form>``
-  controls. §4.6 / §4.7 / §4.8 deliverable sub-sections are explicitly
+  HTML renderer for human readers. Sections §1/§3/§4 user-actionable
+  rows (those reachable from §1 ``C-*`` IDs) get embedded ``<form>``
+  controls. §5.6 / §5.7 / §5.8 deliverable sub-sections are explicitly
   excluded from form attachment — they are read-only deliverables.
 
 User responses are NEVER merged back into the original report. The HTML
@@ -57,7 +57,7 @@ def _strip_leading_frontmatter(text: str) -> str:
 
 from .clarification_items import (
     _is_separator_row,
-    _section_5_slice,
+    _section_1_slice,
     _split_pipe_row,
     parse_clarification_items,
 )
@@ -79,7 +79,7 @@ _LINK_PATTERN = re.compile(r"\[([^\]]+)\]\(([^)]+)\)")
 
 # Sections whose Response-ID-bearing rows must NOT get form attachment
 # (read-only deliverables — see plan §1.4).
-_NO_FORM_SECTION_PREFIXES = ("## 4.6", "### 4.6", "## 4.7", "### 4.7", "## 4.8", "### 4.8")
+_NO_FORM_SECTION_PREFIXES = ("## 5.6", "### 5.6", "## 5.7", "### 5.7", "## 5.8", "### 5.8")
 
 
 @dataclass(frozen=True)
@@ -161,7 +161,7 @@ def _markdown_to_html(
     headings: list[tuple[int, str, str]] = []
     i = 0
     n = len(lines)
-    current_section_path: list[str] = []  # ['## 5. ...', '### 5.1 ...'] etc.
+    current_section_path: list[str] = []  # ['## 1. ...', '### 1.1 ...'] etc.
 
     while i < n:
         line = lines[i]
@@ -394,7 +394,7 @@ class _GroupedSpec:
     value`` metadata cell led by ``headline_col``; the long columns
     (``wide_cols``) each keep their own min-width column.
 
-    ``kind == "clarification"`` additionally re-attaches the §5 form
+    ``kind == "clarification"`` additionally re-attaches the §1 form
     widget to the ``user_input_col`` cell and the ``data-*`` row attrs."""
     headline_col: int
     group_cols: tuple[int, ...]
@@ -410,7 +410,7 @@ class _GroupedSpec:
 def _grouped_table_spec(
     header_cells: list[str], section_path: list[str]
 ) -> Optional[_GroupedSpec]:
-    """Only §5 Clarification Items is grouped in the HTML view (it keeps the
+    """Only §1 Clarification Items is grouped in the HTML view (it keeps the
     interactive form and stays flat in the .md). All other narrative tables are
     already rendered compactly by the template, so no grouping is applied here."""
     norm = [h.strip() for h in header_cells]
@@ -420,7 +420,7 @@ def _grouped_table_spec(
         group = tuple(c for c in range(len(norm)) if c != headline and c not in wide_set)
         return _GroupedSpec(headline_col=headline, group_cols=group, wide_cols=wide, **kw)
 
-    # §5 Clarification Items — keep the interactive form, and widen the three
+    # §1 Clarification Items — keep the interactive form, and widen the three
     # long-prose columns (Expected form is prose too, not a code column).
     if (
         any("Clarification Items" in h for h in section_path)
@@ -474,7 +474,7 @@ def _grouped_meta_cell(
 def _grouped_clarification_row(
     row: list[str], spec: _GroupedSpec
 ) -> tuple[str, str]:
-    """Return ``(tr_attrs, wide_cells_html)`` for one §5 row, re-attaching
+    """Return ``(tr_attrs, wide_cells_html)`` for one §1 row, re-attaching
     the form widget + ``data-*`` attrs to ``C-\\d+`` rows exactly as the
     non-grouped path does."""
     rid = row[spec.id_col] if 0 <= spec.id_col < len(row) else ""
@@ -822,7 +822,7 @@ def serialize_user_response(
 # --------------------------------------------------------------------------- #
 
 def report_has_clarification_items(src_md: str) -> bool:
-    """True when the final-report MD has at least one §5 ``C-*``
+    """True when the final-report MD has at least one §1``C-*``
     clarification row. This is the single predicate that gates HTML-view
     generation: the self-contained html's only value over the markdown is
     the embedded ``<form>`` widgets for those rows, so a clarification-free
@@ -844,7 +844,7 @@ def render_html_view(
 ) -> Path | None:
     """Write ``<stem>.html`` next to ``src_md_path`` and return its path,
     or return ``None`` when generation is skipped because the report has
-    no §5 clarification rows (see ``report_has_clarification_items``).
+    no §1 clarification rows (see ``report_has_clarification_items``).
     Idempotent — overwrites an existing html sibling, and removes a stale
     one when a previously-clarification-bearing report no longer has rows."""
     src_text = src_md_path.read_text(encoding="utf-8")