Skip to content

fix: repair OperationsCenter propose heartbeat watcher path - #455

Merged
ProtocolWarden merged 2 commits into
mainfrom
oc-watchdog/20260713-1920-propose-heartbeat
Jul 14, 2026
Merged

fix: repair OperationsCenter propose heartbeat watcher path#455
ProtocolWarden merged 2 commits into
mainfrom
oc-watchdog/20260713-1920-propose-heartbeat

Conversation

@ProtocolWarden

Copy link
Copy Markdown
Owner

No description provided.

@ProtocolWarden

ProtocolWarden commented Jul 13, 2026

Copy link
Copy Markdown
Owner Author

Resolved: superseded by new push — re-review resumed

Self-review concerns — auto-fixing (up to 6 attempts; re-queued if still unresolved):

Failed checks: code_quality

@ProtocolWarden

ProtocolWarden commented Jul 14, 2026

Copy link
Copy Markdown
Owner Author

Resolved: superseded by new push — re-review resumed

Needs human attention (reason=fix_pass_no_progress). Left open — not merged (unresolved) and not closed (work preserved).

The automated fix passes exhausted the Self-Heal Ladder (reached L2/2) without changing the branch; a fresh self-review on the same PR head still finds concerns. Further autonomous retries would repeat without progress.

Latest concerns:

Failed checks: code_quality

…_trigger

ty flagged Path | None passed where touch_liveness requires Path at
main.py:160 — the unconditional call ran before the existing
status_dir is not None guard that gates the heartbeat thread start.
Root cause: OperationsCenter src/operations_center/entrypoints/pipeline_trigger/main.py.
Fixes: Type check (ty) CI gate on PR #455.
@ProtocolWarden
ProtocolWarden merged commit 2b8d408 into main Jul 14, 2026
25 checks passed
@ProtocolWarden
ProtocolWarden deleted the oc-watchdog/20260713-1920-propose-heartbeat branch July 14, 2026 01:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant