Add debug logging to entry points when collector is initialized or skipp

[ProtocolWarden]

Auto-generated by Operations Center execution.

Goal

Add debug logging to entry points when collector is initialized or skipped

Definition of done (complete ALL before finishing)

1. Complete the task in its ENTIRETY — every acceptance criterion and every
   file the task implies (implementation, tests, and docs as applicable). Do
   not leave TODOs, stubs, or 'follow-up' gaps; a partial change is rejected
   in review.
2. Add or update tests/checks that prove the work is correct.
3. Run the repository's test suite and linters/formatters and make them
   pass locally. If anything fails, fix it before finishing — do not hand
   off a red build.
4. Only consider the task done when the full change is in place AND verified
   green. The PR you open should be mergeable as-is.

[ProtocolWarden]

Resolved: new push — automated review resumed

Needs human attention (reason=ci_persistently_red). Left open — not merged (unresolved) and not closed (work preserved).

CI has not gone green after 20 checks (1 failing: audit: failure). Not merged (red CI) and not closed (work preserved) — needs a human to fix CI.

[ProtocolWarden]

Needs human attention (reason=ci_never_settled). Left open — not merged (unresolved) and not closed (work preserved).

CI has not settled after 21 checks (2 still running: Test (pytest), audit). Not merged (CI incomplete) and not closed (work preserved) — needs a human to investigate stuck CI.

[ProtocolWarden]

Needs human attention (reason=ci_never_settled). Left open — not merged (unresolved) and not closed (work preserved).

CI has not settled-green on the current head after 21 checks (2 still running: Test (pytest), audit). Not merged (CI incomplete) and not closed (work preserved) — needs a human to investigate stuck CI.

[ProtocolWarden]

Resolved: CI green on unchanged head — test suite validates implementation; automated review resumed

Resolved: CI green on unchanged head — test suite validates implementation; automated review resumed

CI has not settled-green on the current head after 21 checks (2 still running: Test (pytest), audit). Not merged (CI incomplete) and not closed (work preserved) — needs a human to investigate stuck CI.

[ProtocolWarden]

Self-review concerns — auto-fixing (up to 6 attempts; re-queued if still unresolved):

['Code style: observe() method uses tuple unpacking for logging side effects (not idiomatic). Pattern (logger.debug(...), value)[1] appears multiple times in service.py and reduces readability. Should refactor to separate statements or if/else blocks.', 'The logging in observe() method for skipped collectors relies on the tuple trick, which is difficult to read and understand. This impacts maintainability of the logging feature itself.', 'Minor: Unrelated formatting change in pr_review_watcher/main.py (spacing before colon) mixed with logging changes.']

[ProtocolWarden]

Self-review concerns — auto-fixing (up to 6 attempts; re-queued if still unresolved):

["CRITICAL: Missing 'import logging' in src/operations_center/entrypoints/autonomy_cycle/main.py — logger is used but import not shown in diff, will cause NameError at runtime", "CODE QUALITY: Non-idiomatic logging pattern in service.py observe() method (~11 instances) — uses tuple unpacking trick '(logger.debug(...), value)[1]' instead of proper if/else blocks, reduces readability and maintainability", "INCIDENTAL: Unrelated formatting change in pr_review_watcher/main.py (spacing in 'len(prefix) :') appears to be from formatter, not related to PR objective", 'SCOPE: PR includes extensive .console/ documentation files with internal stage tracking — appropriate for tracking but significantly expands PR scope for what should be a logging-only feature']