fix(steam-sniper): URL slug authoritative resolver (Tiger Tooth → Scorched fix)#52
Merged
fix(steam-sniper): URL slug authoritative resolver (Tiger Tooth → Scorched fix)#52
Conversation
…Tooth → Scorched' skin mix-up
Лёхин feedback по extension v1.4: при добавлении ножей через расширение
вместо Tiger Tooth попадал Scorched (та же модель, другой скин). Stволы
аналогично. Из всех ножей только 1-2 добавились правильно.
Корень: extension шлёт item_name парсенный из DOM h1, но lis-skins-страницы
содержат карусели похожих скинов — первый h1 с wear-маркером может быть
related-item, а не главный. Сервер доверял DOM-имени, exact-match'ил его
в каталоге и писал левый скин. URL же всегда правильный.
Фикс — приоритезировать URL slug над DOM-именем в /api/lists:
- _slugify_name(): генерирует канонический slug из имени каталога
(drops ★/™, lowercase, non-alphanumeric → dash)
- _slug_from_source_url(): извлекает последний path-сегмент URL
- _rebuild_slug_index(): индексирует 3 формы slug на каждый item:
1. generated (ASCII-only fallback)
2. lis-skins-provided URL slug (percent-encoded — authoritative)
3. URL-decoded variant (если location.href декодирован браузером)
- _resolve_from_source_url(): O(1) lookup, возвращает canonical name
- /api/lists добавлен Шаг 0 — URL-резолвер выше DOM-резолва
- WARNING лог при DOM/URL mismatch — диагностика когда DOM врёт
Дополнительно:
- extension/background.js: 8s timeout через AbortController + явное
сообщение "сервер не ответил, проверь дашборд" вместо silent void
- extension/content.js: toast показывает canonical name из ответа сервера
("Добавлено → ★ Karambit | Tiger Tooth (Field-Tested)") — юзер видит
что улетело до закрытия страницы; PATCH таргета идёт по canonical name
- extension manifest: bump 1.4 → 1.5
- static/js/lists.js: in-flight lock в toggleListItem — палиатив против
дубль-кликов на cases-карточках с двумя toggle-кнопками
- _resolve_item_name: WARNING лог на каждый Steam-API fallback (по фидбэку
Codex) — observability для решения нужно ли refuse'ать вместо silent
first-match. Время + chosen hash_name + resolved name.
- /api/lists endpoint: structured лог received → resolve path → resolved
Тесты: tests/test_source_url_resolution.py — 17 кейсов, главный регресс
test_tiger_tooth_url_resolves_to_tiger_tooth (URL Tiger Tooth + DOM Scorched
→ Tiger Tooth). Encoded/decoded slug variants. StatTrak namespace isolation.
133/133 passed.
Smoke на проде:
POST /api/lists item_name="★ Karambit | Scorched" + URL=tiger-tooth/
→ "★ Karambit | Tiger Tooth (Factory New)" via source_url ✓
Не сделано (осознанно):
- Steam-API fallback оставлен по поведению (только лог) — после URL-резолвера
он хитится только для items не в каталоге; решение по refuse — после
недели наблюдения логов
- Cases policy "в один список" — ждём serverных логов на повтор жалобы
чтобы понять кто шлёт левый list_type (UI или event-double-fire)
- Security findings (CORS, /api без auth, HTTP) остаются deferred —
юзер использует только с Лёхой
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
/api/lists. Решает Лёхин баг "добавляю Tiger Tooth — попадает Scorched" (карусели похожих скинов в DOM путалиgetItemName()).toggleListItem(палиатив, ждём логи на повтор жалобы).Что изменено
tools/steam-sniper/server.py_slug_to_nameиндекс (3 формы slug на item: generated + URL-encoded + URL-decoded),_resolve_from_source_url, Шаг 0 в/api/lists, WARNING-логи на mismatch + Steam-API fallbacktools/steam-sniper/extension/{background,content,manifest}.jstools/steam-sniper/static/js/lists.js_inFlightlock наtoggleListItemtools/steam-sniper/tests/test_source_url_resolution.pytest_tiger_tooth_url_resolves_to_tiger_toothSmoke на production (уже задеплоено)
Что НЕ сделано (по решению пользователя)
log-only. Решение по refuse'у — после недели наблюдения логов.list_type(UI или event-double-fire).Test plan
uv run pytest tests/— 133/133 passed (16 новых тестов на slug resolver, 1 на decoded variant)resolved_via: "source_url"🤖 Generated with Claude Code