Undo service init can fail from lack of await

[thedustinmiller]

Hello,

Currently the undo service doesn't await ui.warning calls, which causes the validation to fail and the application to exit without explanation. This makes silly issues, like git exiting with 128 due to not setting email and name, more difficult to debug.

Very cool work, thanks!

sidekick-cli/src/sidekick/services/undo_service.py

Lines 152 to 161 in 2176d40

	except subprocess.TimeoutExpired as e:
	error = GitOperationError(
	operation="commit", message="Git commit timed out", original_error=e
	)
	ui.warning(str(error))
	return False
	except Exception as e:
	error = GitOperationError(operation="commit", message=str(e), original_error=e)
	ui.warning(f"Error creating undo commit: {e}")
	return False

[geekforbrains]

Thanks for reporting this. Will investigate and patch on next release 😅

[tsbertalan]

This is failing initial setup for me on both Python 3.10 and 3.11.

Output includes:

/sidekick/services/undo_service.py:62: RuntimeWarning: coroutine 'warning' was never awaited
  ui.warning(UNDO_DISABLED_HOME)
RuntimeWarning: Enable tracemalloc to get the object allocation Setup validation failed: Undo System
Setup failed at step 'Undo System': Setup step 'Undo System'    Setup step 'Undo System' failed validation

I get this after entering my OpenAIAPI key and choosing a default model, and then on every subsequent invocation of sidekick.

[tsbertalan]

This is on pip v 0.5.1. 0.4.1 gives a different error, but if I go back to 0.3.4, it starts.