okstra 0.188.0 → 0.188.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "okstra",
-  "version": "0.188.0",
+  "version": "0.188.1",
   "description": "Host-aware multi-provider cross-verification orchestrator runtime and agent skills.",
   "license": "MIT",
   "author": "devonshin",

package/runtime/BUILD.json

@@ -1,5 +1,5 @@
 {
-  "package": "0.188.0",
-  "builtAt": "2026-09-03T19:30:32.235Z",
+  "package": "0.188.1",
+  "builtAt": "2026-09-03T21:11:05.123Z",
   "repoRoot": "/home/runner/work/okstra/okstra"
 }

package/runtime/python/okstra_ctl/report_finalize.py

@@ -36,6 +36,7 @@ from __future__ import annotations
 
 import argparse
 import json
+from datetime import datetime, timezone
 import subprocess
 import sys
 from dataclasses import dataclass
@@ -45,7 +46,11 @@ from typing import Any, Callable, Mapping, Sequence
 from .agent.activity import ActivityProjectionError, project_agent_activity
 from .report_contract import apply_execution_roles
 from .report_assembly import ReportAssemblyError, assemble_report
-from .dispatch_state import DispatchError, link_agent_dispatch_result
+from .dispatch_state import (
+    DispatchError,
+    link_agent_dispatch_result,
+    mutate_team_state,
+)
 from .final_report_paths import (
     final_report_data_path,
     final_report_markdown_path,
@@ -495,6 +500,7 @@ def run_finalize(
         commands = [(name, cmd) for name, cmd in commands if name in selected]
 
     first_failure = ""
+    validated = False
     for name, command in commands:
         # 실패한 시퀀스가 worktree 를 거두면 재작업 대상이 사라진다.
         if name == STEP_TEARDOWN_STAGES and first_failure:
@@ -505,6 +511,10 @@ def run_finalize(
         steps.append(step_payload(name, command, result))
         if result.returncode != 0 and not first_failure:
             first_failure = f"{name} failed with exit code {result.returncode}"
+        if name == STEP_VALIDATE_RUN and result.returncode == 0:
+            validated = True
+    if validated:
+        _record_run_end(ctx.team_state_path)
     pointer, pointer_error = _recorded_next_phase(ctx)
     payload: dict[str, Any] = {
         "ok": not first_failure,
@@ -519,6 +529,35 @@ def run_finalize(
     return payload
 
 
+def _record_run_end(team_state_path: Path) -> None:
+    """검증이 처음 통과한 시각을 `team-state.runEndedAt` 에 적는다. 이미 있으면 그대로.
+
+    토큰 수집기(`okstra_token_usage.collect.resolve_run_window`)는 이 값을 창의
+    끝으로 읽는데 어떤 코드도 적지 않았다. 그래서 완료된 run 을 다시 finalize
+    하면 창의 끝이 지금이 되어, 같은 세션이 그 뒤에 돌린 run 이 이 run 의 리드
+    사용량에 들어왔다(관측 2026-09-03, dev-10626 error-analysis r04: 하루 뒤
+    재수집에 오늘 run 이 포함, 리드 3h → 11h). run 이 끝났다고 판정하는 자리는
+    검증 통과이므로 여기서 한 번만 적는다. 실패해도 finalize 는 이미 끝난
+    시퀀스라 결과를 바꾸지 않고 경고만 낸다.
+    """
+    ended = datetime.now(timezone.utc).strftime("%Y-%m-%dT%H:%M:%SZ")
+
+    def stamp(state: dict[str, Any]) -> bool:
+        if state.get("runEndedAt"):
+            return False
+        state["runEndedAt"] = ended
+        return True
+
+    try:
+        mutate_team_state(team_state_path, stamp)
+    except (DispatchError, OSError) as exc:
+        print(
+            f"report-finalize: could not record runEndedAt on {team_state_path} "
+            f"({exc}); the next usage collection will end at the current time",
+            file=sys.stderr,
+        )
+
+
 def _recorded_next_phase(ctx: FinalizeContext) -> tuple[dict[str, str], str]:
     """`validate-run` 이 방금 태스크 매니페스트에 쓴 다음 phase 포인터.
 

package/runtime/python/okstra_ctl/report_projections.py

@@ -191,7 +191,9 @@ def project_token_usage(team_state: Mapping[str, Any]) -> dict[str, Any]:
         "grand": _usage_row(
             summary,
             "grand",
-            None if lead_cost is None or worker_cost is None else lead_cost + worker_cost,
+            # 두 소수의 합은 부동소수점 잔재(85.05619999999999)를 남긴다 — 정본
+            # JSON 에는 수집기와 같은 소수 넷째 자리로 적는다.
+            None if lead_cost is None or worker_cost is None else round(lead_cost + worker_cost, 4),
         ),
         "workerDetails": _worker_details(team_state),
         "cli": {"costUsd": costs.get("cliWorkers")},

package/runtime/python/okstra_token_usage/collect.py

@@ -230,6 +230,32 @@ def run_window_relaxation_floor(team_state_path: Path) -> str | None:
     return relaxation_floor(run_dir, suffix, manifest)
 
 
+def _recorded_run_end(state: dict, manifest: dict | None) -> str | None:
+    """완료된 run 이 지난 수집에서 기록한 창의 끝 — 리드·워커 usage 블록의 endedAt 최댓값.
+
+    run 의 끝을 적는 코드가 없다(`runEndedAt` 은 읽기만, status 파일은 안 쓰인다).
+    그래서 완료된 run 을 다시 수집하면 창의 끝이 지금이 되어, 같은 세션이 그
+    뒤에 돌린 다른 run 까지 이 run 의 리드 창에 들어온다(관측 2026-09-03:
+    dev-10626 error-analysis r04 를 하루 뒤 재finalize 하면 오늘 run 이 들어옴).
+    매니페스트가 `completed` 인 run 은 처음 완료됐을 때 수집한 창이 곧 run 의
+    끝이므로 그 값으로 고정한다. 아직 완료 전이면 None — 리드가 서술문을 고쳐
+    Phase 7 을 다시 도는 동안은 창이 지금까지 늘어나는 것이 맞다.
+    """
+    if not isinstance(manifest, dict) or manifest.get("status") != "completed":
+        return None
+    ends: list[str] = []
+    blocks = [state.get("leadUsage")] + [
+        worker.get("usage") for worker in (state.get("workers") or []) if isinstance(worker, dict)
+    ]
+    for block in blocks:
+        if not isinstance(block, dict) or block.get("source") == "unavailable":
+            continue
+        ended = block.get("endedAt")
+        if isinstance(ended, str) and ended:
+            ends.append(ended)
+    return max(ends) if ends else None
+
+
 def _run_end_estimate(run_dir: Path, suffix: str) -> str | None:
     """run 종료 근사 — 같은 run 의 status 산출물 mtime(reconcile 후 고정, Phase 7
     재렌더로도 바뀌지 않음). 완료 전(status 부재)이면 None."""
@@ -353,7 +379,8 @@ def resolve_run_window(
     섞여 폭증한다(관측: requirements-discovery 한 run 에 lead 1.7억 토큰 /
     $416 / 3h). 토큰 집계를 이 윈도우로 스코핑해 그 run 분만 센다. 시작 =
     이 run 의 run-manifest createdAt, 종료 = team-state.runEndedAt → 이 run 의
-    status mtime → 현재 시각(아직 진행 중) 순으로 해소한다. 접미사를 못 뽑으면
+    status mtime → 완료된 run 이 지난 수집에서 기록한 창의 끝(`_recorded_run_end`)
+    → 현재 시각(아직 진행 중) 순으로 해소한다. 접미사를 못 뽑으면
     (None, None) — 윈도우 없이 전체를 세는 기존 동작으로 안전 폴백.
 
     시작 완화(`relax_start`, 기본 True — 토큰 수집용): run-manifest createdAt 이
@@ -386,7 +413,12 @@ def resolve_run_window(
         if not window_start_is_pinned(since, floor):
             earliest = _earliest_lead_session_ts(state, team_state_path)
             since = relax_window_start(since, earliest, floor)
-    until = state.get("runEndedAt") or _run_end_estimate(run_dir, suffix) or utc_now()
+    until = (
+        state.get("runEndedAt")
+        or _run_end_estimate(run_dir, suffix)
+        or _recorded_run_end(state, manifest)
+        or utc_now()
+    )
     return since, until
 
 

package/runtime/python/okstra_token_usage/report.py

@@ -238,7 +238,7 @@ def populate_data_token_cells(data_path: Path, team_state: dict) -> int:
         "billableTokens": summary.get("grandBillableEquivalentTokens"),
         # CLI tracked on its own row per the template — grand here means
         # lead + claudeWorkers, not lead + claudeWorkers + cli.
-        "costUsd": lead_cost + worker_cost,
+        "costUsd": round(lead_cost + worker_cost, 4),
     })
     token_usage.setdefault("cli", {})["costUsd"] = cli_cost
     token_usage["workerDetails"] = _worker_detail_rows(team_state)