Skip to content

fix: Correct remaining wrong command names from Dart/UI confusion #61

Description

@markus-lassfolk

Root Cause (from Reaver analysis)

Three naming layers were confused:

  • MQTT topic (wire format) — correct
  • Dart function name — internal only
  • UI/i18n key — localization string

Corrections Needed

Wrong Name Correct Name Root Cause
read_all_clean_area read_clean_area Pattern guess
readCleanArea read_clean_area camelCase confusion
obstacle_toggle ignore_obstacles Two distinct commands confused
setIgnoreObstacle ignore_obstacles Dart error-log string
shutdownYarbo shutdown Dart method name
restart_yarbo_system restart_container UI localization key

Also

  • ignore_obstacles payload is {"state": int} not {"enabled": bool}
  • set_person_detect requires {"disable": bool, "key": ???} — empty {} returns state:-1

Audit all command strings in codebase against MQTT_PROTOCOL.md.

Metadata

Metadata

Assignees

No one assigned

    Labels

    bugSomething isn't working

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions