Skip to content

fix(ddg): улучшить fallback для метаданных Alpina - #2

Open
gman-dev-nov wants to merge 1 commit into
mainfrom
fix/ddg-alpina-fallback
Open

fix(ddg): улучшить fallback для метаданных Alpina#2
gman-dev-nov wants to merge 1 commit into
mainfrom
fix/ddg-alpina-fallback

Conversation

@gman-dev-nov

Copy link
Copy Markdown
Owner

Что изменилось

Улучшен fallback-поиск метаданных книг Alpina через DuckDuckGo. Бот точнее обрабатывает результаты выдачи и стабильнее извлекает данные книги в сценариях, где раньше возвращались неполные или неточные метаданные.

Зачем это нужно

Ранее часть книг Alpina плохо находилась через текущий DDG fallback:

  • бот мог не распознать подходящий результат;
  • мог вернуть неполные поля;
  • мог не дойти до корректной карточки, хотя нужная книга была в поисковой выдаче.

Из-за этого добавление книги по ISBN или поисковым подсказкам было менее надежным, чем ожидалось.

Что получает пользователь

  • выше шанс автоматически получить корректные метаданные книги;
  • меньше ручных исправлений после поиска;
  • меньше ложных пустых результатов для книг Alpina.

Технически

  • доработана логика выбора и обработки результатов DDG;
  • обновлены тесты провайдеров на проблемные сценарии;
  • lint-правки, относящиеся к этому же блоку, включены в тот же логический fix.

Проверка

  • venv/bin/python -m pytest tests/test_home_library_providers.py -q

Уточнил обработку и выбор результатов DDG для книг Alpina, чтобы бот стабильнее восстанавливал корректные название, автора и другие метаданные в сценариях, где прежний fallback часто возвращал неполные или неточные данные.

Коммит собирает исходное улучшение и последующий lint-fix в один логический bugfix, чтобы история ветки отражала единственную пользовательскую проблему: слабое извлечение метаданных Alpina через DDG.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 3836cc5d98

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment on lines +108 to +112
for raw_snippet in snippets:
snippet = html.unescape(raw_snippet)
author_match = AUTHOR_LABEL_PATTERN.search(snippet)
if author_match:
return _normalize_author(

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Match snippet author to selected title

The new fallback returns the first Автор ... match from snippets without checking whether that snippet belongs to the title chosen as best, so mixed DuckDuckGo results can produce a valid title paired with the wrong author. This happens whenever multiple snippets contain different authors and no em-dash author is found, because _extract_ddg_book calls this helper globally rather than by result index.

Useful? React with 👍 / 👎.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant