fix: issue #11947 automated patch

.bot/coordination/acceptance_criteria.md

@@ -0,0 +1,37 @@
+# Acceptance Criteria — Retry for issue #11947
+
+## Source of truth for this retry
+- [ ] Retry scope is derived from `.bot/validation_failures.md`
+- [ ] Only required fixes tied to the recorded failure are included
+
+## Files in scope
+- [ ] `python-package/xgboost/__init__.py`
+- [ ] `tests/python/test_shap.py`
+
+## `python-package/xgboost/__init__.py`
+- [ ] `interpret` is imported on its own line:
+  - `from . import interpret`
+- [ ] `collective` remains on its own line:
+  - `from . import collective`
+- [ ] `"interpret"` remains present in `__all__`
+- [ ] Import layout is stable under ruff/pre-commit (no auto-rewrite on rerun)
+
+## `tests/python/test_shap.py`
+- [ ] No extra blank line remains between `import scipy.special` and `import xgboost as xgb`
+- [ ] The interpret wrapper tests remain present in this file
+- [ ] File formatting is stable under ruff/pre-commit (no auto-rewrite on rerun)
+
+## Validation
+### Required
+- [ ] Exact failed command now passes:
+  - `pre-commit run --files python-package/xgboost/__init__.py tests/python/test_shap.py --show-diff-on-failure`
+
+### Optional if environment supports it
+- [ ] Narrow targeted test run for interpret wrapper coverage passes:
+  - `python -m pytest tests/python/test_shap.py -k "TestInterpret" -q`
+
+## Scope discipline
+- [ ] No unrelated files were edited for this retry
+- [ ] `python-package/xgboost/interpret.py` was left unchanged unless a concrete follow-up error required touching it
+- [ ] No docs, refactors, or broader API work were added in this retry
+- [ ] No PRs were opened and no dangerous commands were run

.bot/coordination/coordinator_plan.md

@@ -0,0 +1,86 @@
+# Coordinator Retry Plan — dmlc/xgboost issue #11947
+
+## Goal of this retry
+This retry is **not** a fresh implementation plan for the whole RFC. It is a **targeted retry** based strictly on the recorded validation failure in `.bot/validation_failures.md`.
+
+## Failure summary
+Only one required check failed:
+- suite: `lint`
+- command:
+  - `pre-commit run --files python-package/xgboost/__init__.py tests/python/test_shap.py --show-diff-on-failure`
+
+The failure log shows that **ruff auto-fixed formatting/import layout** in exactly two files:
+1. `python-package/xgboost/__init__.py`
+2. `tests/python/test_shap.py`
+
+No other required validation failures are recorded.
+
+## What the failure means
+The previous attempt already had the intended retry scope loaded into the linted files, but the patch did not land in a pre-commit-clean form.
+
+### Exact hook-driven fixes from the log
+#### `python-package/xgboost/__init__.py`
+Ruff changed:
+- from:
+  - `from . import collective`
+  - plus implicit/combined import arrangement
+- to separate import lines:
+  - `from . import collective`
+  - `from . import interpret`
+
+This means the retry must preserve **separate first-party import lines**, not collapse them.
+
+#### `tests/python/test_shap.py`
+Ruff changed:
+- removed the extra blank line between:
+  - `import scipy.special`
+  - `import xgboost as xgb`
+
+The file must keep normal import grouping with no stray blank line there.
+
+## Retry scope
+### Required edits only
+Limit this retry to:
+- `python-package/xgboost/__init__.py`
+- `tests/python/test_shap.py`
+
+### Explicitly avoid in this retry
+Do **not** expand scope unless the exact failing command forces it.
+In particular, do not spend this retry on:
+- new docs
+- new result classes
+- new public methods on booster/sklearn models
+- unrelated refactors
+- new test files unless absolutely necessary
+
+## Important repo state note
+There is already a pending implementation file in the worktree:
+- `python-package/xgboost/interpret.py`
+
+That file is **not mentioned in the recorded failure**. The retry worker should treat it as already part of the patch and **avoid churning it** unless a follow-up validation step shows a concrete issue.
+
+## Required retry actions
+1. Ensure `python-package/xgboost/__init__.py` uses the ruff-compatible separate import line for `interpret`.
+2. Ensure `tests/python/test_shap.py` matches the ruff-compatible import spacing.
+3. Keep the new interpret wrapper tests in `tests/python/test_shap.py`; do not move them elsewhere during this retry unless unavoidable.
+4. Re-run the exact failing command:
+   - `pre-commit run --files python-package/xgboost/__init__.py tests/python/test_shap.py --show-diff-on-failure`
+
+## Optional follow-up validation (only if inexpensive and available)
+If the environment supports Python/native test execution, run a narrow targeted test for the newly added interpret wrapper coverage inside `tests/python/test_shap.py`.
+Suggested example:
+- `python -m pytest tests/python/test_shap.py -k "TestInterpret" -q`
+
+If that is not feasible, the exact pre-commit pass is sufficient for this retry plan.
+
+## Success definition for this retry
+This retry succeeds when:
+- the exact failing pre-commit command passes cleanly
+- no additional unrelated edits are introduced
+- the worker leaves the patch focused on the already-implemented interpretability slice
+
+## Main risks to avoid
+- Re-expanding scope beyond the logged failure
+- Moving tests into a different file during the retry
+- Collapsing imports in `__init__.py` back into a form ruff rewrites
+- Touching `interpret.py` without a concrete failure signal

.bot/coordination/worker_handoff.md

@@ -0,0 +1,6 @@
+# Worker Handoff
+
+Issue #11947 (initial)
+- Apply minimal scoped patch only.
+- Update tests relevant to changed behavior.
+- Write .bot/pr_body.md and .bot/coordination/worker_report.md.

.bot/coordination/worker_handoff_retry.md

@@ -0,0 +1,108 @@
+# Worker Handoff Retry — GPT-5.3-Codex
+
+## Context
+This is a **retry handoff**, not a greenfield implementation brief.
+You must scope your work to the recorded validation failure in:
+- `.bot/validation_failures.md`
+
+Read that file first if you need to confirm the failure context.
+
+## Failure to fix
+Only one required check failed:
+
+```bash
+pre-commit run --files python-package/xgboost/__init__.py tests/python/test_shap.py --show-diff-on-failure
+```
+
+The failure log shows `ruff check` auto-modified these two files.
+
+## Exact required changes from the failure log
+### 1) `python-package/xgboost/__init__.py`
+Ruff wanted this import layout:
+
+```python
+from . import tracker  # noqa
+from . import collective
+from . import interpret
+```
+
+Do **not** collapse `collective` and `interpret` into one import line.
+Keep them as separate `from . import ...` statements.
+
+Also keep `"interpret"` in `__all__`.
+
+### 2) `tests/python/test_shap.py`
+Ruff removed one extra blank line in the import section.
+The top of the file should look like:
+
+```python
+import itertools
+import re
+
+import numpy as np
+import scipy.special
+import xgboost as xgb
+from xgboost import testing as tm
+```
+
+Do not leave a stray blank line between `scipy.special` and `import xgboost as xgb`.
+
+## Scope constraints
+### Only touch what is required by this retry
+Focus on:
+- `python-package/xgboost/__init__.py`
+- `tests/python/test_shap.py`
+
+### Do not expand scope
+Do **not** use this retry to:
+- redesign the API
+- move tests into a new file unless absolutely forced
+- add docs
+- add new methods to booster/sklearn wrappers
+- refactor unrelated code
+- touch unrelated files because they “look nicer”
+
+## Important current state
+There is already a pending implementation file in the worktree:
+- `python-package/xgboost/interpret.py`
+
+That file was **not** part of the failing lint command. Assume it is already part of the intended patch and **leave it alone unless a concrete error requires a change**.
+
+## What to preserve
+The retry should preserve the already-added interpret wrapper coverage in `tests/python/test_shap.py`.
+Do not remove those tests. The retry is about making the existing patch pass the exact failed validation.
+
+## Recommended step-by-step
+1. Inspect current diffs/status.
+2. Ensure `python-package/xgboost/__init__.py` matches the ruff-preferred import layout.
+3. Ensure `tests/python/test_shap.py` import spacing matches the ruff-preferred layout.
+4. Do not make unrelated edits.
+5. Run the exact failed validation command:
+   ```bash
+   pre-commit run --files python-package/xgboost/__init__.py tests/python/test_shap.py --show-diff-on-failure
+   ```
+6. If that passes and the environment is healthy enough, optionally run a narrow targeted test:
+   ```bash
+   python -m pytest tests/python/test_shap.py -k "TestInterpret" -q
+   ```
+
+## Expected end state
+- `python-package/xgboost/__init__.py`:
+  - separate import line for `interpret`
+  - `"interpret"` remains in `__all__`
+- `tests/python/test_shap.py`:
+  - clean import spacing
+  - interpret wrapper tests still present
+- exact pre-commit command passes without rewriting files
+
+## Non-goals
+- No PR creation
+- No pushes
+- No dangerous commands
+- No broad cleanup pass
+
+## Deliverable quality bar
+This retry is successful only if it is:
+- minimal
+- directly tied to the logged failure
+- pre-commit clean for the exact failing command

.bot/coordination/worker_report.md

@@ -0,0 +1,44 @@
+# Worker Report: issue #11947 (`repair-2`)
+
+## Status
+- Completed
+
+## Scope executed
+- Read required context files:
+  - `AGENTS.md`
+  - `.bot/retrieved_memory.md`
+  - `.bot/memory/worker_retrieved_memory.md`
+  - `.bot/coordination/worker_handoff_retry.md`
+  - `.bot/validation_failures.md`
+- Kept retry scope tied to the single recorded failure:
+  - `pre-commit run --files python-package/xgboost/__init__.py tests/python/test_shap.py --show-diff-on-failure`
+
+## Concrete remediation notes
+1. `python-package/xgboost/__init__.py`
+- Confirmed/import layout aligned with ruff-stable form:
+  - `from . import collective`
+  - `from . import interpret`
+- Confirmed `"interpret"` remains in `__all__`.
+
+2. `tests/python/test_shap.py`
+- Confirmed no extra blank line remains between:
+  - `import scipy.special`
+  - `import xgboost as xgb`
+- Preserved the previously added `TestInterpret` coverage (no removal or relocation).
+
+## Validation run for retry
+- Required command:
+  - `pre-commit run --files python-package/xgboost/__init__.py tests/python/test_shap.py --show-diff-on-failure`
+  - Result: Pass
+- Additional check:
+  - `python -m py_compile python-package/xgboost/interpret.py tests/python/test_shap.py python-package/xgboost/__init__.py`
+  - Result: Pass
+- Optional targeted test attempt:
+  - `PYTHONPATH=python-package python -m pytest tests/python/test_shap.py -k "TestInterpret" -q`
+  - Result: Blocked (`ModuleNotFoundError: No module named 'numpy'`)
+
+## Scope discipline confirmation
+- No retry-scope expansion beyond the recorded lint failure.
+- `python-package/xgboost/interpret.py` left unchanged in this pass.
+- No docs/refactors/extra API work added during retry.
+- No PRs were opened.