okstra 0.130.3 → 0.130.4

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "okstra",
-  "version": "0.130.3",
+  "version": "0.130.4",
   "description": "Multi-agent cross-verification orchestrator runtime + Claude Code skills.",
   "license": "MIT",
   "author": "devonshin",

package/runtime/BUILD.json

@@ -1,5 +1,5 @@
 {
-  "package": "0.130.3",
-  "builtAt": "2026-07-21T14:59:13.137Z",
+  "package": "0.130.4",
+  "builtAt": "2026-07-21T15:17:32.560Z",
   "repoRoot": "/home/runner/work/okstra/okstra"
 }

package/runtime/prompts/profiles/implementation-planning.md

@@ -78,6 +78,11 @@
 - Required deliverable shape (final report, in addition to the standard sections):
   - at least two implementation options. **Each option must include**:
     - **File Structure**: an explicit list of files to create / modify / delete with each file's responsibility (one-line each). Use the form `Create: path — responsibility` / `Modify: path:line-range — change summary` / `Delete: path — reason`. Write every `path` in full and `<PROJECT_ROOT>`-relative — never ellipsis-abbreviated (`…` / `...` / a trailing `/…`); an abbreviated path does not resolve and is rejected by plan-body verification as a kind-b path mismatch.
+      - **Two-tier change description.** Each `fileStructure` row carries `summary` **and** optional `details`, and they are not interchangeable:
+        - `summary` — one plain-language sentence a reviewer who has never opened this file can follow: what changes and why it is needed. Name behaviour and domain nouns, not identifiers. No function/type/variable names, no call chains, no line numbers, no conditional logic spelled out. **Enforced:** `schemas/final-report-v1.0.schema.json` `$defs.OptionCandidate.fileStructure.items.summary` caps it at 120 characters, so a dense identifier dump fails schema validation.
+        - `details` — the technical specifics that used to be crammed into `summary`: exact symbol names, signatures, comparison semantics, line ranges, enum members. Omit the field when there is nothing beyond the summary.
+        - Bad `summary` (identifier dump, no readable claim): `Widen the return of retrieveFontFamilyGroupWithFontVersionId to also yield the matched FontFamily (own key and enabled flag) and the matched Font.status.`
+        - Good — `summary`: `Carry enough information out of the font lookup to tell whether the matched family is switched off.` / `details`: `Add the matched FontFamily (own key + enabled) and Font.status to the return of retrieveFontFamilyGroupWithFontVersionId. Both are already in scope in the :102-104 loop and dropped at return.`
     - affected interfaces / public contracts and downstream consumers
     - estimated blast radius (units, configs, deployment manifests, data migrations)
   - trade-off matrix across options (rows = options, columns at minimum: complexity, risk, reversibility, test coverage cost, rollout cost)

package/runtime/python/okstra_ctl/report_views.py

@@ -513,6 +513,7 @@ def _emit_table(lines: list[str], start: int, section_path: list[str]) -> tuple[
     )
 
     narrow_cols = _narrow_columns(header_cells, rows)
+    path_cols = _path_columns(header_cells)
     # `User input` carries the embedded form widget (textarea / select /
     # input) which needs all the horizontal space it can get; its
     # markdown plain text is empty or a short placeholder so the auto
@@ -523,7 +524,7 @@ def _emit_table(lines: list[str], start: int, section_path: list[str]) -> tuple[
     head = (
         "<thead><tr>"
         + "".join(
-            f"<th{_col_class(idx, narrow_cols)}>{_inline(c)}</th>"
+            f"<th{_col_class(idx, narrow_cols, path_cols)}>{_inline(c)}</th>"
             for idx, c in enumerate(header_cells)
         )
         + "</tr></thead>"
@@ -562,7 +563,7 @@ def _emit_table(lines: list[str], start: int, section_path: list[str]) -> tuple[
                     )
                 else:
                     cells_html.append(
-                        f"<td{_col_class(idx, narrow_cols)}>{_inline(cell)}</td>"
+                        f"<td{_col_class(idx, narrow_cols, path_cols)}>{_inline(cell)}</td>"
                     )
             body_rows.append(
                 f'<tr id="{html.escape(meta.row_id.lower())}" '
@@ -579,7 +580,7 @@ def _emit_table(lines: list[str], start: int, section_path: list[str]) -> tuple[
             body_rows.append(
                 tr_open
                 + "".join(
-                    f"<td{_col_class(idx, narrow_cols)}>{_inline(c)}</td>"
+                    f"<td{_col_class(idx, narrow_cols, path_cols)}>{_inline(c)}</td>"
                     for idx, c in enumerate(row)
                 )
                 + "</tr>"
@@ -921,8 +922,30 @@ def _narrow_columns(
     return narrow
 
 
-def _col_class(col_idx: int, narrow_cols: set[int]) -> str:
-    return ' class="td-narrow"' if col_idx in narrow_cols else ""
+def _col_class(col_idx: int, narrow_cols: set[int], path_cols: set[int]) -> str:
+    if col_idx in narrow_cols:
+        return ' class="td-narrow"'
+    if col_idx in path_cols:
+        return ' class="td-path"'
+    return ""
+
+
+# A repo-relative path has no spaces, so `overflow-wrap: anywhere` puts its
+# min-content width at one character and the auto table layout hands the
+# column almost nothing while a prose neighbour takes the rest — the path
+# then renders one or two characters per line. `td-path` floors the width.
+_PATH_HEADER_PREFIXES: tuple[str, ...] = ("path", "파일 경로", "경로")
+
+
+def _path_columns(header_cells: list[str]) -> set[int]:
+    return {
+        col
+        for col, header in enumerate(header_cells)
+        if _INLINE_MD_STRIP_RE.sub("", header or "")
+        .strip()
+        .lower()
+        .startswith(_PATH_HEADER_PREFIXES)
+    }
 
 
 _DANGEROUS_URL_SCHEME_RE = re.compile(
@@ -960,6 +983,10 @@ def _inline(text: str) -> str:
     # markdown source intentionally stacks short fields with <br>). html.escape
     # above turned them into &lt;br&gt;; restore the tag.
     out = out.replace("&lt;br&gt;", "<br>").replace("&lt;br/&gt;", "<br>").replace("&lt;br /&gt;", "<br>")
+    # `<small>` demotes a cell's technical detail line below its plain-language
+    # first line (File Structure `details`). Allowlisted alongside <br>; every
+    # other tag stays escaped.
+    out = out.replace("&lt;small&gt;", "<small>").replace("&lt;/small&gt;", "</small>")
     return out
 
 

package/runtime/schemas/final-report-v1.0.schema.json

@@ -1256,7 +1256,8 @@
               "ticketId": { "$ref": "#/$defs/TicketId" },
               "action": { "enum": ["Create", "Modify", "Delete"] },
               "path": { "type": "string", "minLength": 1 },
-              "summary": { "type": "string", "minLength": 1 }
+              "summary": { "type": "string", "minLength": 1, "maxLength": 120 },
+              "details": { "type": "string", "minLength": 1 }
             }
           }
         },

package/runtime/templates/reports/final-report.template.md

@@ -204,10 +204,10 @@ Carried-forward plan items retain their prior verdicts verbatim; each such item
 
 - File Structure:
 
-  | ID | Ticket ID | Action | Path (and line-range) | Change summary |
-  |----|-----------|--------|------------------------|----------------|
+  | {{ t("columns.recordMeta") }} | Path (and line-range) | Change summary |
+  |------|------------------------|----------------|
 {% for fs in opt.fileStructure -%}
-  | {{ fs.id | mdcell }} | `{{ fs.ticketId | mdcell }}` | {{ fs.action | mdcell }} | `{{ fs.path | mdcell }}` | {{ fs.summary | mdcell }} |
+  | **{{ fs.id | mdcell }}**<br>Ticket: `{{ fs.ticketId | mdcell }}`<br>Action: {{ fs.action | mdcell }} | `{{ fs.path | mdcell }}` | {{ fs.summary | mdcell }}{% if fs.details %}<br><small>{{ fs.details | mdcell }}</small>{% endif %} |
 {% endfor %}
 
 - {{ t("implementationPlanning.optionInterfacesLabel") }}: {{ opt.interfaces }}

package/runtime/templates/reports/report.css

@@ -188,6 +188,24 @@ td.td-narrow, th.td-narrow {
   width: 5%;
   white-space: nowrap;
 }
+/* Path columns — assigned by `_path_columns()` in report_views.py. A
+ * repo-relative path holds no spaces, so the `overflow-wrap: anywhere`
+ * above lets the auto table layout starve the column down to a couple of
+ * characters per line. Floor the width and let it wrap at separators. */
+td.td-path, th.td-path {
+  min-width: 26ch;
+  word-break: break-word;
+}
+
+/* Second line of a two-tier cell: the plain-language summary leads, the
+ * technical detail follows demoted. */
+td small {
+  display: inline-block;
+  margin-top: 0.35em;
+  font-size: 0.88em;
+  color: GrayText;
+}
+
 thead th {
   position: sticky;
   top: 3rem;