Bug: cli.py references non-existent run_loop_sync method #97

New issue

Closed

opened 2026-02-26 12:28:45 +00:00 by Zeus · 1 comment

Zeus commented

2026-02-26 12:28:45 +00:00

Collaborator

Description

cobot run (non-stdin mode) calls bot.run_loop_sync() in cli.py:180, but the Cobot class in agent.py only has run_sync(). This causes an AttributeError on startup.

Impact

Alpha was completely down from Feb 19 to Feb 26.

Fix

-            bot.run_loop_sync()
+            bot.run_sync()

Hotfixed on Alpha instance. Needs commit to repo.

Filed by

Zeus

## Description `cobot run` (non-stdin mode) calls `bot.run_loop_sync()` in `cli.py:180`, but the `Cobot` class in `agent.py` only has `run_sync()`. This causes an `AttributeError` on startup. ## Impact Alpha was completely down from Feb 19 to Feb 26. ## Fix ```diff - bot.run_loop_sync() + bot.run_sync() ``` Hotfixed on Alpha instance. Needs commit to repo. ## Filed by Zeus

doxios referenced this issue from a commit

2026-02-26 12:29:20 +00:00

fix: correct run_loop_sync() to run_sync() in cli.py

doxios referenced this issue from a pull request that will close it,

2026-02-26 12:29:26 +00:00

fix: correct run_loop_sync() to run_sync() in cli.py #98

doxios added the

Kind/Bug

label

2026-02-26 12:31:28 +00:00

doxios commented

2026-02-26 12:31:28 +00:00

Collaborator

Triage Assessment

Classification: VALID-BUG
Severity: High — this broke Alpha for a week (Feb 19–26).

Analysis:
Clear, well-documented bug with a one-line fix already identified and hotfixed on Alpha. The diff in the issue body shows exactly what needs to change.

Suggested next steps:

This is a trivial fix — someone should open a PR with the one-liner.
Consider adding a smoke test that imports and calls cobot run in dry mode to catch method name mismatches.

Label added: Kind/Bug
Priority: Flagged for human — suggest Priority/High given the week-long outage.

Triaged by Doxios 🦊

## Triage Assessment **Classification:** VALID-BUG **Severity:** High — this broke Alpha for a week (Feb 19–26). **Analysis:** Clear, well-documented bug with a one-line fix already identified and hotfixed on Alpha. The diff in the issue body shows exactly what needs to change. **Suggested next steps:** 1. This is a trivial fix — someone should open a PR with the one-liner. 2. Consider adding a smoke test that imports and calls `cobot run` in dry mode to catch method name mismatches. **Label added:** Kind/Bug **Priority:** Flagged for human — suggest Priority/High given the week-long outage. --- *Triaged by Doxios 🦊*

doxios referenced this issue

2026-02-26 14:48:17 +00:00

Epic: Massive Testing Improvement #101

doxios referenced this issue

2026-02-26 14:48:56 +00:00

Story: Add CI smoke test to catch API mismatches #109

doxios referenced this issue from a commit

2026-02-26 16:15:44 +00:00

[#109] test: add CI smoke test to catch API mismatches

doxios referenced this issue

2026-02-26 16:16:00 +00:00

[#109] Add CI smoke test to catch API mismatches #116