Use `__name__` for loggers across `.ib` sub-mods

Change all `.ib` sub-modules to use `get_logger(name=__name__)`
for per-module logger instances vs shared `._util.log`.

Deats,
- change `._util` to use `__name__` vs literal string.
- change `.broker`, `.feed`, `.ledger`, `.symbols` to import
  `get_logger()` from top-level `.log` and call with `__name__`.
- drop `log` imports from `._util` in all affected mods.

Also,
- drop trailing comma in `.cli.services()` conditional for `loglevel`
  passthrough -> fixes an actual kwargs bug!!

(this commit msg was generated in some part by [`claude-code`][claude-code-gh])
[claude-code-gh]: https://github.com/anthropics/claude-code

piker/brokers/ib/_util.py

@@ -40,7 +40,7 @@ if TYPE_CHECKING:
     from .api import Client
     import i3ipc
 
-log = get_logger('piker.brokers.ib')
+log = get_logger(name=__name__)
 
 _reset_tech: Literal[
     'vnc',

piker/brokers/ib/broker.py

@@ -50,6 +50,7 @@ from ib_insync.objects import (
 )
 
 from piker import config
+from piker.log import get_logger
 from piker.types import Struct
 from piker.accounting import (
     Position,
@@ -77,7 +78,6 @@ from piker.clearing._messages import (
     BrokerdFill,
     BrokerdError,
 )
-from ._util import log
 from .api import (
     _accounts2clients,
     get_config,
@@ -95,6 +95,8 @@ from .ledger import (
     update_ledger_from_api_trades,
 )
 
+log = get_logger(name=__name__)
+
 
 def pack_position(
     pos: IbPosition,

piker/brokers/ib/feed.py

@@ -56,11 +56,11 @@ from piker.brokers._util import (
     NoData,
     DataUnavailable,
 )
+from piker.log import get_logger
 from .api import (
     # _adhoc_futes_set,
     Client,
     con2fqme,
-    log,
     load_aio_clients,
     MethodProxy,
     open_client_proxies,
@@ -78,6 +78,9 @@ from .symbols import get_mkt_info
 if TYPE_CHECKING:
     from trio._core._run import Task
 
+log = get_logger(
+    name=__name__,
+)
 
 # XXX NOTE: See available types table docs:
 # https://interactivebrokers.github.io/tws-api/tick_types.html

piker/brokers/ib/ledger.py

@@ -44,6 +44,7 @@ from ib_insync import (
     CommissionReport,
 )
 
+from piker.log import get_logger
 from piker.types import Struct
 from piker.data import (
     SymbologyCache,
@@ -57,7 +58,6 @@ from piker.accounting import (
     iter_by_dt,
 )
 from ._flex_reports import parse_flex_dt
-from ._util import log
 
 if TYPE_CHECKING:
     from .api import (
@@ -65,6 +65,9 @@ if TYPE_CHECKING:
         MethodProxy,
     )
 
+log = get_logger(
+    name=__name__,
+)
 
 tx_sort: Callable = partial(
     iter_by_dt,

piker/brokers/ib/symbols.py

@@ -42,10 +42,7 @@ from piker.accounting import (
 from piker._cacheables import (
     async_lifo_cache,
 )
-
+from piker.log import get_logger
-from ._util import (
-    log,
-)
 
 if TYPE_CHECKING:
     from .api import (
@@ -53,6 +50,10 @@ if TYPE_CHECKING:
         Client,
     )
 
+log = get_logger(
+    name=__name__,
+)
+
 _futes_venues = (
     'GLOBEX',
     'NYMEX',

piker/cli/__init__.py

@@ -326,7 +326,7 @@ def services(
                 loglevel=(
                     config['loglevel']
                     if tl
-                    else None,
+                    else None
                 ),
             ),
             tractor.get_registry(