fix: use Query to properly handle --pingpong flag #65

pi-agent · 2026-04-02T20:34:36+08:00

pi-agent commented

2026-04-02 20:34:36 +08:00

Summary

Use nonebot_plugin_alconna Query to correctly handle the --pingpong boolean flag. Previously the flag wasn't working because the parameter wasn't being properly injected.

Changes

Import Query from nonebot_plugin_alconna
Change Option to not have args (just Option('--pingpong'))
Use Query[bool] type annotation with default Query('pingpong')
Check pingpong.available to determine if flag was set

Root Cause

The previous implementation used Option('--pingpong', default=False) which doesn't work as expected. The parameter wasn't being injected into the handler function.

Solution

Use the Query pattern from nonebot_plugin_alconna which properly handles boolean flags:

Option('--pingpong') without args
pingpong: Query[bool] = Query("pingpong") in handler
Check pingpong.available to determine if flag was set

Testing

All existing tests pass (50 passed)
Plugin loads successfully

## Summary Use nonebot_plugin_alconna Query to correctly handle the --pingpong boolean flag. Previously the flag wasn't working because the parameter wasn't being properly injected. ## Changes - Import Query from nonebot_plugin_alconna - Change Option to not have args (just Option('--pingpong')) - Use Query[bool] type annotation with default Query('pingpong') - Check pingpong.available to determine if flag was set ## Root Cause The previous implementation used `Option('--pingpong', default=False)` which doesn't work as expected. The parameter wasn't being injected into the handler function. ## Solution Use the Query pattern from nonebot_plugin_alconna which properly handles boolean flags: - `Option('--pingpong')` without args - `pingpong: Query[bool] = Query("pingpong")` in handler - Check `pingpong.available` to determine if flag was set ## Testing - All existing tests pass (50 passed) - Plugin loads successfully

pi-agent added 1 commit 2026-04-02 20:34:38 +08:00

fix: use Query to properly handle --pingpong flag f21b7067df

Use nonebot_plugin_alconna Query to correctly handle the --pingpong
boolean flag. Previously the flag wasn't working because the
parameter wasn't being properly injected.

Changes:
- Import Query from nonebot_plugin_alconna
- Change Option to not have args (just Option('--pingpong'))
- Use Query[bool] type annotation with default Query('pingpong')
- Check pingpong.available to determine if flag was set

kagami-review commented

2026-04-02 20:36:07 +08:00

Failed to generate code suggestions for PR