Commit dcc1a72
committed
fix: clear ruff F811/F841 findings across frontends and helpers
Resolve all F811 (redefined-while-unused) and F841 (unused-local) issues
reported by ruff so the lint baseline is cleaner:
- frontends/tui_v3.py: drop a duplicate `from dataclasses import
dataclass` line that was shadowed by the very next import; remove the
dead `from prompt_toolkit.keys import Keys` block in
_ptk_keypress_to_bytes — the symbol was never referenced in the
function body, only `getattr(kp, 'key', None)` is used.
- frontends/fsapp.py: handle_message unpacks data.event but never reads
the event object — rename to `_event` to flag intent.
- frontends/genericagent_acp_bridge.py: write_message computed `method`
but only logged the payload — embed method in the ACP-BRIDGE log line
so the local is used and debugging is easier.
- frontends/qtapp.py: drop unused `as e` from an exception handler that
already prints via traceback.print_exc().
- frontends/tuiapp.py: drop the unused `args` assignment in main();
parse_args still validates argv and surfaces errors.
- memory/autonomous_operation_sop/helper.py: remove an unused `errors`
list in complete_task.
Verification:
- `python3 -m py_compile` on all six files passes.
- `ruff check <files> --select F811,F841` reports `All checks passed!`.
- No behavioural change; the only logic tweak is the ACP-BRIDGE log
gaining a `[method]` prefix.1 parent c85b59e commit dcc1a72
6 files changed
Lines changed: 4 additions & 12 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
809 | 809 | | |
810 | 810 | | |
811 | 811 | | |
812 | | - | |
| 812 | + | |
813 | 813 | | |
814 | 814 | | |
815 | 815 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
147 | 147 | | |
148 | 148 | | |
149 | 149 | | |
150 | | - | |
| 150 | + | |
151 | 151 | | |
152 | 152 | | |
153 | 153 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
833 | 833 | | |
834 | 834 | | |
835 | 835 | | |
836 | | - | |
| 836 | + | |
837 | 837 | | |
838 | 838 | | |
839 | 839 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
22 | 22 | | |
23 | 23 | | |
24 | 24 | | |
25 | | - | |
26 | 25 | | |
27 | 26 | | |
28 | 27 | | |
| |||
821 | 820 | | |
822 | 821 | | |
823 | 822 | | |
824 | | - | |
825 | | - | |
826 | | - | |
827 | | - | |
828 | | - | |
829 | 823 | | |
830 | 824 | | |
831 | 825 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
721 | 721 | | |
722 | 722 | | |
723 | 723 | | |
724 | | - | |
| 724 | + | |
725 | 725 | | |
726 | 726 | | |
727 | 727 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
70 | 70 | | |
71 | 71 | | |
72 | 72 | | |
73 | | - | |
74 | | - | |
75 | 73 | | |
76 | 74 | | |
77 | 75 | | |
| |||
0 commit comments