2233admin/opencli-Razormind·#66

feat(channels): migrate doubao/douyin to thick fetch() with bounded retry

Analysed against a440d81d4 files changedby 1012839419a-alt
Open PR on GitHub

Change-risk score

9.1out of 10Elevated
Repository health
8.5to8.4base against this head

This change sits in the top third of this repository's own risk distribution, the band worth reviewing closely. It is riskier than 73% of this repository's own commits.

The largest health movement is in test_douyin_detail_channel.py, down 0.8. Nothing outside this PR depends on a contract it changed.

Files changed
4
in this diff
Contracts changed
0
removed or signature
Outside callers
0
not in this diff
Findings added
7
0 resolved

Blast radius on the repo map

Every file in the repository, grouped into its own directories and sized by lines. The ones this PR changes are lit, and everything that imports them is marked.

backendchromecompatdocsfrontendiiiopenspecscriptstestsapibrowser_act_packsschemasservicesskillsworkflowdify_gra…appcomponentslibchangesintegrationunitecommercelead-generationsocial-listeningintellige…(app)flowrecordsuiunlumen-uiflowworkflowapichannelscontrolpipeline
changed, health fellchanged, health rosechanged, health heldimports a changed fileuntouchedchanged by this PR

Drawing 1,488 of 1,631 files at a readable size, grouped by directory and sized by lines of code. 4 changed here reach 1 more by import. 143 are too small to see at this width, which is what the frame below is for.

Inside backend/channels

api_channe…base.pybrowser_act_channel.pycli_channel.pycrawl4ai_channel.pydoubao_research_channel.…douyin_detail_channel.pyopencli_channel.pyregis…rss_channel.pyskill_channel.pyweb_scrape…

13 files, 2 changed by this PR and 1 that import one. This PR changes 4 files in total, spread across more than one top-level directory. Click any directory above to frame it instead.

Changed

What this PR added, and what it took away

Introduced findings are the ones inside the lines this PR added, not the file's existing ones. The resolved column is the only place in the product that lists them.

Introduced (7)
  • mediumdry violation
    tests/unit/channels/test_douyin_detail_channel.py:99

    32% of file duplicated; worst clone shares 41 lines with tests/unit/channels/test_doubao_research_channel.py

  • mediumduplicated assertion block
    tests/unit/channels/test_douyin_detail_channel.py:128

    assertion block at lines 128-129 is duplicated in tests/unit/channels/test_doubao_research_channel.py

  • mediumduplicated assertion block
    tests/unit/channels/test_douyin_detail_channel.py:110

    assertion block at lines 110-111 is duplicated in tests/unit/channels/test_doubao_research_channel.py

  • mediumduplicated assertion block
    tests/unit/channels/test_doubao_research_channel.py:179

    assertion block at lines 179-180 is duplicated in tests/unit/channels/test_doubao_research_channel.py

  • mediumduplicated assertion block
    tests/unit/channels/test_doubao_research_channel.py:197

    assertion block at lines 197-198 is duplicated in tests/unit/channels/test_doubao_research_channel.py

  • mediumduplicated assertion block
    tests/unit/channels/test_doubao_research_channel.py:215

    assertion block at lines 215-216 is duplicated in tests/unit/channels/test_doubao_research_channel.py

  • lowbumpy road
    backend/channels/douyin_detail_channel.py:147

    collect has 3 nested blocks at the same level (bumpy road)

Resolved (0)
  • Nothing removed.

Who else knows this code

The primary author of each changed file by share of its recent commits, mined from git history. Advisory: it says who has context, not who must review.

  • backend/channels/doubao_research_channel.py2233admin 71% of 2 commits
  • tests/unit/channels/test_doubao_research_channel.py1012839419a-alt 64% of 2 commits
  • backend/channels/douyin_detail_channel.py2233admin 100% of 1 commits

Get this on your own pull requests

This page came from an index of 2233admin/opencli-Razormind. No model calls, no configuration, and it refreshes on every push. Install the bot and every pull request gets one of these.

Install the PR BotIndex a repository
feat(channels): migrate doubao/douyin to thick fetch() with bounded retry · 2233admin/opencli-Razormind #66 · Repowise