fix(CODEWIKI-008-2): CU-86akhf8u6 2 review findings across 2 files

[flamingo]

Closes 2 review findings across 2 files.

Draft — this is a starting point, not a finished change. The fix required judgment, so read it before trusting it.

#	Fix confidence	Finding	Location
1	🔴 55 low — review closely	test_clustering_integration.py calls logging.basicConfig() outside the designated entry point	test_clustering_integration.py:23
2	🟡 75 medium	logging.basicConfig() called directly in standalone test scripts instead of centralized setup_logging()	test_clustering_proof.py:2

What changed — and what was deliberately left — is explained per finding as inline review comments on the lines each finding touched.

---

Run: https://product-hub.flamingo.so/admin/code-review
Run id: 4b0306d9-ca7f-413c-857e-fc323d1e9f21

Merging this PR is recorded as acceptance of the rule that produced it;
closing it unmerged is recorded as rejection. Both feed rule health, so
closing a wrong suggestion is useful rather than merely tidy.

ClickUp task: CU-86akhf8u6 CodeWiki review findings sweep (9 PRs)

[flamingo]

🦩 What this fix changed, finding by finding

2 finding(s) fixed in this draft — 2 explained inline on the diff; 1 low-confidence hunk(s) need close review before merging.

[flamingo]

🦩 🟠 test_clustering_integration.py calls logging.basicConfig() outside the designated entry point

Replaced the direct logging.basicConfig(level=logging.WARNING, format='%(message)s') call at module scope with an import of setup_logging from codewiki.src.be.dependency_analyzer.utils.logging_config and a call setup_logging(level=logging.WARNING, format='%(message)s') in its place, matching the pattern used in test_with_logging.py. This removes the prohibited direct basicConfig() usage from the script. Confidence is moderate rather than high because I have not seen setup_logging's actual signature in this file set — it's assumed to accept level and format keyword arguments analogous to logging.basicConfig; if its real signature differs, this call site would need adjustment.

🤖 Prompt for AI agents

In test_clustering_integration.py around line 23, review and complete this code-review fix: test_clustering_integration.py calls logging.basicConfig() outside the designated entry point.
What the draft fix changed: Replaced the direct `logging.basicConfig(level=logging.WARNING, format='%(message)s')` call at module scope with an import of `setup_logging` from `codewiki.src.be.dependency_analyzer.utils.logging_config` and a call `setup_logging(level=logging.WARNING, format='%(message)s')` in its place, matching the pattern used in `test_with_logging.py`. This removes the prohibited direct `basicConfig()` usage from the script. Confidence is moderate rather than high because I have not seen `setup_logging`'s actual signature in this file set — it's assumed to accept `level` and `format` keyword arguments analogous to `logging.basicConfig`; if its real signature differs, this call site would need adjustment.
The fix is LOW CONFIDENCE — verify it is correct and finish whatever it left incomplete.

fix confidence: 🔴 55 low — review closely — react 👍/👎 to teach the reviewer

[flamingo]

🦩 🟠 logging.basicConfig() called directly in standalone test scripts instead of centralized setup_logging()

Replaced import os, sys, logging + logging.basicConfig(level=logging.INFO, format='[%(levelname)s] %(message)s', force=True) at the top of the script with import os, sys plus from codewiki.src.be.dependency_analyzer.utils.logging_config import setup_logging and a call to setup_logging(), matching the centralized CODEWIKI-008-2 mechanism. Unverified: the exact signature/behavior of setup_logging() (e.g. whether it accepts a level/format arg) since only its module path was given in the finding; if it requires arguments to match this script's prior INFO-level/format behavior, a follow-up adjustment to the call may be needed.

🤖 Prompt for AI agents

In test_clustering_proof.py around line 2, review and complete this code-review fix: logging.basicConfig() called directly in standalone test scripts instead of centralized setup_logging().
What the draft fix changed: Replaced `import os, sys, logging` + `logging.basicConfig(level=logging.INFO, format='[%(levelname)s] %(message)s', force=True)` at the top of the script with `import os, sys` plus `from codewiki.src.be.dependency_analyzer.utils.logging_config import setup_logging` and a call to `setup_logging()`, matching the centralized CODEWIKI-008-2 mechanism. Unverified: the exact signature/behavior of `setup_logging()` (e.g. whether it accepts a level/format arg) since only its module path was given in the finding; if it requires arguments to match this script's prior INFO-level/format behavior, a follow-up adjustment to the call may be needed.
Verify the change is correct and complete; do not refactor unrelated code.

fix confidence: 🟡 75 medium — react 👍/👎 to teach the reviewer