 snowy-yang
|
65adfc9089
|
refactor: 删除 aiohttp_compat 垫片，music_api 改走原生 core.http_client
core/aiohttp_compat 是「同步转异步」迁移期的 requests 同形异步垫片：它存在的
唯一理由是让既有 `requests.post/get + raise_for_status()` 风格的代码以最小
改动迁到异步。其两个调用方（music_api/api.py、music_api/client.py）**本就已是
async def**——垫片只剩「保留 requests 语义」这一层包装，属于迁移残留。
改动：
- api.py：23 处 requests.post/get → await core.http_client.request()。
该文件每处都在请求后立刻 raise_for_status() 才取 .json()，从不把 4xx 当数据读，
故「_once 对非 2xx 直接抛 UpstreamHTTPError」与旧语义等价（原 raise_for_status
抛的 HTTPError 也一律落入 RequestException 分支）；
except requests.RequestException → except UpstreamError。
- client.py：两个自管重试的方法改用新公开的 core.http_client.request_once()
（单次、不重试），保留原有「5xx 退避重试一次 / 4xx 立即失败」的分流——
把 status_code>=500 的判断换成 UpstreamHTTPError.status，行为逐条对齐
（4xx=风控不重试，网络/5xx 重试一次）。响应体上限交由客户端 max_bytes 统一把关。
post_request_full 返回类型改为 HTTPResult（qr.py 用到的 .text/.cookies/.headers
三者在 HTTPResult 上齐备）。
- core/http_client：_once 更名并公开为 request_once（自管重试的调用方需要它）。
注：这是为「删垫片」而补的唯一一个小口子，不是新增抽象。
注意一处**有意的行为变化**：api.py 原经垫片请求零重试，现在 request() 自带
「网络异常/5xx 退避重试一次、4xx 不重试」（core.http_client 既定口径）。
429/460 属 4xx 不重试，不加剧风控。
- 删除 core/aiohttp_compat.py；web/app.py 去掉其 close()；pyproject 去掉其
per-file-ignore；core/http_errors.py 注释更正为「仅供 core.sync_http」。
- 测试：test_aiohttp_compat.py 拆解为 test_async_bridge.py（只留桥的两条契约，
垫片自身的 4 条随垫片消亡）；test_search_music / test_playlist_detail 的替身
边界从 api.requests.post 换成 api._http.request，异常类型换成 UpstreamNetworkError。
门禁：ruff 全过 / pyright 0-0 / pytest 383 项全绿（0 失败；383=387-6+2，账面吻合）。
|
2026-09-28 19:17:03 +08:00 |
|
|
snowy-yang
|
d2e6856899
|
refactor: get_playlist_detail 批次拉取拆解（94 行 / 圈 24 / 嵌套 3 → 60 行 / 圈 9 / 嵌套 1）
为什么：歌单解析的唯一上游通道，94 行单函数把「歌单头解析 / trackIds
分批拉明细 / 失败批次记账 / incomplete 标记 / 错误映射」五件事揉在
一个 try 里；曲目字段映射与搜索歌曲路逐字段重复。
拆出（模块级，与 search_music 拆解同风格）：
- _song_base_fields：歌曲条目公共字段（搜索/歌单共用同一容错口径），
_map_song_item 改为其加 type 键的包装
- _playlist_track_ids：trackIds → 字符串 ID 列表
- _collect_playlist_tracks：批次循环整体（失败批次计数、脏 ID 跳批、
明细缺 id 跳过、incomplete 判定输入），嵌套 3 是批次语义固有，
集中在单一职责函数内
保留语义：脏 ID（'①'.isdigit() 为真但 int() 抛 ValueError）跳过整批
且不计 failed_batches；批次阶段网络错误仍映射「获取歌单详情请求失败」。
先写测试后动代码：新增 tests/test_playlist_detail.py 11 例，在未改动
实现上跑绿（修正 1 例测试自身的替身消费顺序），覆盖此前零测试路径——
playlist:null 防枚举、业务 code 错误、非 dict 响应、批次失败记账、
脏 ID 跳批、100/批分批、错误映射。
门禁：ruff 全过 · pyright 0/0 · pytest 359 passed（348→359）。
|
2026-09-27 10:53:48 +08:00 |
|
|
snowy-yang
|
e00ac14460
|
refactor: search_music 四路映射拆解（101 行 / 圈 32 / 嵌套 6 → 42 行 / 圈 7 / 嵌套 1）
为什么：站内搜索的唯一上游通道，101 行里 if/elif 四路条目映射占了
2/3，错误映射与字段容错口径（歌曲 artists 空缺兜底未知歌手、专辑
不兜底、alias 缺省 '' 等）全埋在分支里，改一路要通读全函数。
拆出（模块级，与既有 _ensure_dict 同风格）：
- _map_song_item / _map_album_item / _map_artist_item / _map_playlist_item
——四路口径原样，docstring 标注各路容错差异
- _SEARCH_RESULT_KEYS + _SEARCH_ITEM_MAPPERS 派发表（未知类型不在表内
→ 空结果，原 if/elif 无 else 行为一致）
- _collect_search_items 统一「取键 → 无 id 跳过 → 映射」
先写测试后动代码：新增 tests/test_search_music.py 12 例，在未改动
实现上跑绿（修正 1 例替身形状：网络异常必须抛垫片自己的
RequestException，aiohttp.ClientError 由 aiohttp_compat 内部翻译，
直接抛底层是不存在的形状）。契约测试不为违反 offset: int 签名的
None 入参背书（防御性 int(offset or 0) 保留，不测）。
门禁：ruff 全过 · pyright 0/0 · pytest 330 passed（319→330）。
|
2026-09-27 09:42:07 +08:00 |
|
|
snowy-yang
|
6288a98ecc
|
refactor: 拆分 4 个巨石模块为包，清理过时注释并删除作废文档
为什么：单文件 1100~1900 行的巨石模块让「改一处要通读全文」；同时注释里残留
大量已删台账的编号，以及「Flask/FastAPI 双栈迁移期」这类已失实的架构描述，
会误导后来者按不复存在的架构去改代码。
拆分（对外导入路径不变，行为零变更）：
- music_downloader.py -> music_downloader/（models/downloader/cover/_http/_log/_concurrency）
- auth_system.py -> auth_system/（db/security/users/credits/ledger/mail/cards/backup...）
- music_api.py -> music_api/（api/client/crypto/qr/cookies/compat/constants/errors）
- services/runtime.py -> services/（api_service/_shared/_taskcore/downloads/batch/podcast/
streaming/tasks）；services/runtime.py 保留为兼容门面
兼容性做法：各包用 __init__.py 以 __all__ 原样再导出全部符号，既有调用点
（auth_system.xxx / services.xxx / from music_api import Y）零改动；路径常量改为经包属性
动态读取（auth_system.DB_PATH），否则测试的 monkeypatch 会静默失效并写到生产库。
修复 9 处代码损伤（上一轮注释清理把 `()` 当「空括号残留」删掉所致；用「与拆分前逐字
等价」校验逐一定位，静态检查只会报出其中 4 处）：
- core/config.py ).strip() or '*'
- core/state.py threading.Lock() x2
- qr_login.py datetime.now() / .strip()
- music_downloader/downloader.py .lower()
- services/podcast.py set() / time.time() x2
清理与删除：
- 删除 部署文档.md / 功能介绍.md / README_私有包警告.txt
- 清理指向已删台账与文档的过时编号注释；把「迁移期 / 两栈 / 双框架 / 已迁移」等
失实描述改写为现状描述（保留 `<-> Flask /path` 映射表等仍有价值的契约来源说明）
- _mkpkg.py：公开包剔除清单去掉已不存在的文档名；私有包警告改为自包含
- pyproject.toml：补 pythonpath = ["."]，使裸跑 pytest 不再 import core 失败
验证：ruff 全通过；pyright 0 errors / 0 warnings（102 文件）；pytest 144 passed；
真实 uvicorn 冒烟 6 条路由符合预期且生产 data/ 指纹未变；忽略注释的逐字等价性校验通过。
|
2026-09-26 12:37:48 +08:00 |
|