ida-tui.git/idatui/remote_ops.py, branch maintui for headless idahttps://git.sl0p.foo/ida-tui.git/atom/idatui/remote_ops.py?h=main2026-08-21T10:15:15Zreformat: ruff format + import sort, mechanically (see ruff.toml)2026-08-21T10:15:15Zblastypeter@haxx.in2026-08-21T10:14:46Zurn:sha1:02d02417800184fb76cd0245cdaa94c437aa4081
No behavior. Listed in .git-blame-ignore-revs (next commit).
adopt ruff: pinned formatter + import sorting, opt-in pre-commit hook2026-08-21T10:14:27Zblastypeter@haxx.in2026-08-21T10:14:27Zurn:sha1:93ca90558aa6016e77625c627a70640e693e3303
PR #1 arrived black-formatted, which forced the question. ruff's formatter
is black's output, it is one static binary rather than a dependency chain,
and it is fast enough that a hook is free. ruff.toml pins the version
(a formatter whose output moves between releases turns 'formatted' into
'formatted by whoever committed last') and selects only import sorting on
the lint side -- a hook that refused a commit over an unused variable would
be a different feature.
.githooks/pre-commit (install: git config core.hooksPath .githooks) formats
exactly what is staged, refuses partly-staged files rather than quietly
widening a commit built with git add -p, and prefers the pinned .venv ruff.
Also drops the blanket '# ruff: noqa' PR #1 left in remote_ops.py -- it
would have exempted that file from the only lint rule we enforce.
tests: repair the two seams the ida-nexus port left behind2026-08-21T10:12:53Zblastypeter@haxx.in2026-08-21T10:12:53Zurn:sha1:ea22067c52ec335ea4d3a616cb2107d5fdbdf7b6
Two scenario monkeypatch sites still hooked client.invoke -- the rename to
call() updated the call sites but not the patches, so reprime_is_free and
split_view crashed instead of counting. call() takes the remote_ops
declaration itself now, so both count by its __name__.
remote_ops imported RemoteModule inside _bindings(), which made
test_nexus_client (NEEDS_IDA = False) unrunnable under a stdlib-only
python3 -- the house rule tests/run.py --fast depends on. The import moves
to the same eagerly-if-present, bound-to-None-so-patchable contract
nexus_client uses, and the test injects FakeRemoteModule/FakeRemoteError
exactly like its other fakes: real names when the library is installed,
strict fakes when it is not.
Switch to ida-nexus2026-08-20T21:55:52ZDuncan Ogilviemr.exodia.tpodt@gmail.com2026-08-20T21:55:52Zurn:sha1:f4c1d9b5497fd0d38137b6b345e8171b307c1117Adopt idb_events and remote module features from ida-codemode2026-08-20T21:42:42ZDuncan Ogilviemr.exodia.tpodt@gmail.com2026-08-20T21:42:42Zurn:sha1:f3715d8de0d255c8b14710acfa120ccb9ea953fd