[CL-507] fix(radar): isolate close attribution by owner #513

Merged

andrei merged 1 commit from fix/radar-attribution-tenant-isolation into master

2026-07-21 21:49:41 +00:00

andrei commented

2026-07-21 21:03:51 +00:00

Owner

Что сделано

Доставлен reviewed patch из /tmp/arnold-cl507-radar-isolation.bundle через cherry-pick commit 11d9101f на актуальный master 5a4ce017.
Новый PR head: c684fe3f6b.
Radar close attribution теперь изолирует owner/tenant user_id и legacy rows относит только к canonical Radar owner.
При rebase на PR #515 conflict был только в backend/tests/test_radar_live.py; сохранены оба набора тестов: #515 insufficient-balance skip и #507 tenant attribution.

Зачем

Закрывает смешивание Radar close attribution между владельцем env-backed Radar account и другими tenant/user contexts.

План тестирования

Patch equivalence по production files: git diff --exit-code 11d9101f523b5963dbf2c155959ce7814ef16de3..HEAD -- backend/bot/radar_live.py backend/bot/runner/futures_position_sync_daemon.py backend/tests/test_futures_position_sync_daemon.py
Focused: cd backend && /srv/apps/arnold-trader/venv/bin/python -m pytest tests/test_radar_live.py tests/test_futures_position_sync_daemon.py -q -> 65 passed
Full backend clean env: cd backend && env -u PM2_HOME -u pm_id -u ARNOLD_APP_ROOT -u ARNOLD_REPO_DIR -u ARNOLD_ENVIRONMENT -u DEPLOY_ENVIRONMENT -u ENVIRONMENT -u SENTRY_ENV -u PORT -u ENCRYPTION_MASTER_KEY -u ENCRYPTION_KEY /srv/apps/arnold-trader/venv/bin/python -m pytest -q -> 7386 passed, 7 skipped
git diff --check origin/master..HEAD -> clean

Fixes #507

## Что сделано - Доставлен reviewed patch из /tmp/arnold-cl507-radar-isolation.bundle через cherry-pick commit 11d9101f на актуальный master 5a4ce017. - Новый PR head: c684fe3f6bf6a3d2bead91b5e6301515baa4847f. - Radar close attribution теперь изолирует owner/tenant user_id и legacy rows относит только к canonical Radar owner. - При rebase на PR #515 conflict был только в backend/tests/test_radar_live.py; сохранены оба набора тестов: #515 insufficient-balance skip и #507 tenant attribution. ## Зачем - Закрывает смешивание Radar close attribution между владельцем env-backed Radar account и другими tenant/user contexts. ## План тестирования - Patch equivalence по production files: git diff --exit-code 11d9101f523b5963dbf2c155959ce7814ef16de3..HEAD -- backend/bot/radar_live.py backend/bot/runner/futures_position_sync_daemon.py backend/tests/test_futures_position_sync_daemon.py - Focused: cd backend && /srv/apps/arnold-trader/venv/bin/python -m pytest tests/test_radar_live.py tests/test_futures_position_sync_daemon.py -q -> 65 passed - Full backend clean env: cd backend && env -u PM2_HOME -u pm_id -u ARNOLD_APP_ROOT -u ARNOLD_REPO_DIR -u ARNOLD_ENVIRONMENT -u DEPLOY_ENVIRONMENT -u ENVIRONMENT -u SENTRY_ENV -u PORT -u ENCRYPTION_MASTER_KEY -u ENCRYPTION_KEY /srv/apps/arnold-trader/venv/bin/python -m pytest -q -> 7386 passed, 7 skipped - git diff --check origin/master..HEAD -> clean Fixes #507

andrei added 1 commit

2026-07-21 21:03:51 +00:00