Merge pull request 'feat(snapshot): refresh API video metadata; keep removed visible' (#9) from auto/refresh-meta-removed-1103 into main

Reviewed-on: #9
This commit was merged in pull request #9.
This commit is contained in:
kairee committed 2026-11-03 23:59:00 +08:00
commit bffd0c0183
13 files changed
+445 -45

No files matched your search

+70
View File
@@ -410,6 +410,75 @@ class Repository:
return [dict(row) for row in rows] return [dict(row) for row in rows]
return self.db.read(run) return self.db.read(run)
def refresh_video_metadata(
self,
*,
aid: int,
bvid: str | None,
title: str,
up_mid: int | None,
up_name: str | None,
pubdate: int,
duration: int | None = None,
pages: int | None = None,
cover: str | None = None,
up_face: str | None = None,
now: int | None = None,
) -> bool:
"""用接口返回字段覆盖更新已有视频元数据;不改 status,空值不冲掉旧值。
入库与快照成功路径共用。视频不存在时返回 False。
"""
now = self._now() if now is None else int(now)
aid = int(aid)
def run(conn: sqlite3.Connection) -> bool:
existing = conn.execute(
"SELECT aid FROM video WHERE aid = ?", (aid,)
).fetchone()
if existing is None:
return False
conn.execute(
"""
UPDATE video SET
bvid = COALESCE(?, bvid),
title = CASE
WHEN ? != '' THEN ?
ELSE title
END,
up_mid = COALESCE(?, up_mid),
up_name = COALESCE(?, up_name),
pubdate = CASE
WHEN ? > 0 THEN ?
ELSE pubdate
END,
duration = COALESCE(?, duration),
pages = COALESCE(?, pages),
cover = COALESCE(?, cover),
up_face = COALESCE(?, up_face),
last_sync = ?
WHERE aid = ?
""",
(
bvid,
title or "",
title or "(无标题)",
up_mid,
up_name,
int(pubdate or 0),
int(pubdate or 0),
None if duration is None else int(duration),
None if pages is None else int(pages),
cover,
up_face,
now,
aid,
),
)
return True
return self.db.write(run)
def ingest_video( def ingest_video(
self, self,
*, *,
@@ -430,6 +499,7 @@ class Repository:
不写 B 站收藏夹、不设置 ``folder_id``;同时删除同 aid 的候选行。 不写 B 站收藏夹、不设置 ``folder_id``;同时删除同 aid 的候选行。
既有的 ``removed`` / ``dropped`` / ``unfaved`` 状态都会被重新激活。 既有的 ``removed`` / ``dropped`` / ``unfaved`` 状态都会被重新激活。
已存在时仍以本次接口字段为准刷新元数据。
""" """
now = self._now() if now is None else int(now) now = self._now() if now is None else int(now)
aid = int(aid) aid = int(aid)
+26
View File
@@ -116,12 +116,38 @@ class LibraryService:
if candidate is None: if candidate is None:
video = self.repo.get_video(aid) video = self.repo.get_video(aid)
if video is not None: if video is not None:
# 已在库:仍以接口为准刷新元数据(不重复建档)
try:
detail = self.read_provider.get().view_detail(aid=aid)
except BilibiliError as exc:
self._log("warning", "ingest.detail_failed", aid=aid, error=str(exc))
return IngestResult( return IngestResult(
ok=True, aid=aid, state="exists", ok=True, aid=aid, state="exists",
title=str(video.get("title") or ""), title=str(video.get("title") or ""),
up_name=str(video.get("up_name") or ""), up_name=str(video.get("up_name") or ""),
message="已在库,无需重复入库", message="已在库,无需重复入库",
) )
self.repo.refresh_video_metadata(
aid=detail.aid,
bvid=detail.bvid,
title=detail.title,
up_mid=detail.up_mid,
up_name=detail.up_name,
pubdate=detail.pubdate,
duration=detail.duration,
pages=detail.pages,
cover=detail.cover,
up_face=detail.up_face,
now=self._now(),
)
self._write_snapshot(detail)
self.repo.bump_cache_version("videos")
return IngestResult(
ok=True, aid=aid, state="exists",
title=detail.title,
up_name=detail.up_name or "",
message="已在库,已刷新元数据",
)
return IngestResult( return IngestResult(
ok=False, aid=aid, state="missing", message="候选不存在或已处理" ok=False, aid=aid, state="missing", message="候选不存在或已处理"
) )
+31 -16
View File
@@ -3,12 +3,13 @@
规则(DEVELOPMENT.md §7.4): 规则(DEVELOPMENT.md §7.4):
* 对 ``active`` / ``unfaved`` 视频逐条请求 ``view``; * 对 ``active`` / ``unfaved`` 视频逐条请求 ``view_detail``(完整元数据 + 七项);
* 成功写入七项指标;``ok=1`` 的同日快照直接跳过(断点续跑;含小时任务 1 小时内刚刷新的); * 成功时以接口字段覆盖更新 ``video`` 元数据(title/duration/pages/cover/UP 等),并写入七项快照;
* ``ok=1`` 的同日快照直接跳过(断点续跑;含小时任务 1 小时内刚刷新的);
* 午夜 ``seed_at`` 用上一完成日 ok=1 行克隆出当日壳(``snapshot:seeded`` / ``fans:seeded``),不推进完成日; * 午夜 ``seed_at`` 用上一完成日 ok=1 行克隆出当日壳(``snapshot:seeded`` / ``fans:seeded``),不推进完成日;
* 晚间 ``snapshot_at`` 全量必须 force 打穿 seed,成功后才写完成日并清除 seeded; * 晚间 ``snapshot_at`` 全量必须 force 打穿 seed,成功后才写完成日并清除 seeded;
* ``run_recent_videos`` 只刷新近 N 天新片的当日行,不改完成日 / 备份; * ``run_recent_videos`` 只刷新近 N 天新片的当日行,不改完成日 / 备份;
* 视频不可访问时标记 ``removed``; * 视频不可访问时标记 ``removed``(保留元数据与历史快照,站内继续展示,后续采集跳过);
* 其他错误最多重试 3 次,仍失败写 ``ok=0`` 占位,不写假零值; * 其他错误最多重试 3 次,仍失败写 ``ok=0`` 占位,不写假零值;
* 对 ``track_fans`` 逐个请求 ``relation/stat``(独立任务,不影响视频完成日); * 对 ``track_fans`` 逐个请求 ``relation/stat``(独立任务,不影响视频完成日);
* 视频采集全部成功后写入当日完成标记并生成日备份。 * 视频采集全部成功后写入当日完成标记并生成日备份。
@@ -224,7 +225,7 @@ class SnapshotService:
force: bool = False, force: bool = False,
keep_scraped: bool = False, keep_scraped: bool = False,
) -> SnapshotSummary: ) -> SnapshotSummary:
"""采集视频七项指标并在全部成功后标记完成日 / 备份。 """采集视频元数据 + 七项指标,并在全部成功后标记完成日 / 备份。
``force``:忽略完成标记,重跑。 ``force``:忽略完成标记,重跑。
``keep_scraped``:force 时仍跳过非 seed 的 ok=1(seal 断点续跑); ``keep_scraped``:force 时仍跳过非 seed 的 ok=1(seal 断点续跑);
@@ -264,7 +265,7 @@ class SnapshotService:
continue continue
# force 且非 keep_scraped:CLI 全量重采,不 skip # force 且非 keep_scraped:CLI 全量重采,不 skip
stat, error, unavailable = self._fetch_view(client, aid) detail, error, unavailable = self._fetch_view_detail(client, aid)
if unavailable: if unavailable:
summary.removed += 1 summary.removed += 1
self.repo.set_video_status(aid, "removed", now=now) self.repo.set_video_status(aid, "removed", now=now)
@@ -272,17 +273,16 @@ class SnapshotService:
"warning", "snapshot.removed", "warning", "snapshot.removed",
stat_date=stat_date, aid=aid, error=error, stat_date=stat_date, aid=aid, error=error,
) )
elif stat is not None: elif detail is not None:
self._apply_detail(detail, now=self._now())
self.repo.upsert_snapshot( self.repo.upsert_snapshot(
stat_date, stat_date,
aid, aid,
metrics=stat.metrics(), metrics=detail.metrics(),
ok=True, ok=True,
err=None, err=None,
captured_at=self._now(), captured_at=self._now(),
) )
if getattr(stat, "up_face", None):
self.repo.apply_video_up_face(aid, stat.up_face)
summary.success += 1 summary.success += 1
else: else:
summary.failed += 1 summary.failed += 1
@@ -369,7 +369,7 @@ class SnapshotService:
videos = self.repo.video_aids_recent_for_snapshot(since_pubdate=since_pubdate) videos = self.repo.video_aids_recent_for_snapshot(since_pubdate=since_pubdate)
summary.total = len(videos) summary.total = len(videos)
for index, aid in enumerate(videos, 1): for index, aid in enumerate(videos, 1):
stat, error, unavailable = self._fetch_view(client, aid) detail, error, unavailable = self._fetch_view_detail(client, aid)
if unavailable: if unavailable:
summary.removed += 1 summary.removed += 1
self.repo.set_video_status(aid, "removed", now=now) self.repo.set_video_status(aid, "removed", now=now)
@@ -377,17 +377,16 @@ class SnapshotService:
"warning", "snapshot.recent_removed", "warning", "snapshot.recent_removed",
stat_date=stat_date, aid=aid, error=error, stat_date=stat_date, aid=aid, error=error,
) )
elif stat is not None: elif detail is not None:
self._apply_detail(detail, now=self._now())
self.repo.upsert_snapshot( self.repo.upsert_snapshot(
stat_date, stat_date,
aid, aid,
metrics=stat.metrics(), metrics=detail.metrics(),
ok=True, ok=True,
err=None, err=None,
captured_at=self._now(), captured_at=self._now(),
) )
if getattr(stat, "up_face", None):
self.repo.apply_video_up_face(aid, stat.up_face)
summary.success += 1 summary.success += 1
else: else:
summary.failed += 1 summary.failed += 1
@@ -693,11 +692,27 @@ class SnapshotService:
) )
return state.is_login, state.message return state.is_login, state.message
def _fetch_view(self, client, aid: int): def _apply_detail(self, detail, *, now: int) -> None:
"""快照成功:以 view_detail 字段覆盖 video 元数据(不改 status)。"""
self.repo.refresh_video_metadata(
aid=detail.aid,
bvid=detail.bvid,
title=detail.title,
up_mid=detail.up_mid,
up_name=detail.up_name,
pubdate=detail.pubdate,
duration=detail.duration,
pages=detail.pages,
cover=detail.cover,
up_face=detail.up_face,
now=now,
)
def _fetch_view_detail(self, client, aid: int):
last_error = "" last_error = ""
for attempt in range(1, self.retries + 1): for attempt in range(1, self.retries + 1):
try: try:
return client.view(aid), None, False return client.view_detail(aid=aid), None, False
except BilibiliAPIError as exc: except BilibiliAPIError as exc:
last_error = str(exc) last_error = str(exc)
if exc.code in VIDEO_UNAVAILABLE_CODES: if exc.code in VIDEO_UNAVAILABLE_CODES:
+11 -7
View File
@@ -44,7 +44,10 @@ from ..timeutil import (
) )
from .cache import DASHBOARD_CACHE, TTL_FANS, TTL_VIDEOS, build_cache_key from .cache import DASHBOARD_CACHE, TTL_FANS, TTL_VIDEOS, build_cache_key
TRACKED_STATUSES = ("active", "unfaved") TRACKED_STATUSES = ("active", "unfaved") # 采集队列口径;站内展示勿用
# 站内展示/统计:含原站失效 removed(继续展示);不含人工 dropped
VISIBLE_STATUSES = ("active", "unfaved", "removed")
_VISIBLE_IN = ",".join(f"'{s}'" for s in VISIBLE_STATUSES)
VIDEO_SORTS = { VIDEO_SORTS = {
"heat", "view", "like", "coin", "favorite", "share", "reply", "danmaku", "heat", "view", "like", "coin", "favorite", "share", "reply", "danmaku",
"interaction", "growth", "growth_rate", "added", "published", "interaction", "growth", "growth_rate", "added", "published",
@@ -340,7 +343,7 @@ class DashboardQueries:
def _videos( def _videos(
conn: sqlite3.Connection, conn: sqlite3.Connection,
*, *,
statuses: Sequence[str] = TRACKED_STATUSES, statuses: Sequence[str] = VISIBLE_STATUSES,
folder_id: int | None = None, folder_id: int | None = None,
q: str | None = None, q: str | None = None,
q_fields: Sequence[str] | None = None, q_fields: Sequence[str] | None = None,
@@ -609,7 +612,7 @@ class DashboardQueries:
if first_seen_ts is not None: if first_seen_ts is not None:
initial_import_count = int(conn.execute( initial_import_count = int(conn.execute(
"SELECT COUNT(*) FROM video WHERE first_seen = ? " "SELECT COUNT(*) FROM video WHERE first_seen = ? "
"AND status IN ('active','unfaved')", f"AND status IN ({_VISIBLE_IN})",
(int(first_seen_ts),), (int(first_seen_ts),),
).fetchone()[0]) ).fetchone()[0])
@@ -680,7 +683,7 @@ class DashboardQueries:
"total": sum(by_status.values()), "total": sum(by_status.values()),
"active_ups": int(conn.execute( "active_ups": int(conn.execute(
"SELECT COUNT(DISTINCT up_mid) FROM video " "SELECT COUNT(DISTINCT up_mid) FROM video "
"WHERE status IN ('active','unfaved') AND up_mid IS NOT NULL" f"WHERE status IN ({_VISIBLE_IN}) AND up_mid IS NOT NULL"
).fetchone()[0]), ).fetchone()[0]),
"candidates_pending": candidate_pending, "candidates_pending": candidate_pending,
"candidates_total": candidate_total, "candidates_total": candidate_total,
@@ -955,6 +958,7 @@ class DashboardQueries:
within_start = _date_minus(latest, int(within_days) - 1) within_start = _date_minus(latest, int(within_days) - 1)
videos = self._videos( videos = self._videos(
conn, conn,
statuses=VISIBLE_STATUSES,
folder_id=folder_id, folder_id=folder_id,
q=q, q=q,
within_field=within_field, within_field=within_field,
@@ -1157,7 +1161,7 @@ class DashboardQueries:
aid = int(aid) aid = int(aid)
def run(conn: sqlite3.Connection) -> dict[str, Any] | None: def run(conn: sqlite3.Connection) -> dict[str, Any] | None:
video_rows = self._videos(conn) video_rows = self._videos(conn, statuses=VISIBLE_STATUSES)
video = next((item for item in video_rows if int(item["aid"]) == aid), None) video = next((item for item in video_rows if int(item["aid"]) == aid), None)
if video is None: if video is None:
row = conn.execute( row = conn.execute(
@@ -1752,7 +1756,7 @@ class DashboardQueries:
SELECT COUNT(*) AS total, SELECT COUNT(*) AS total,
SUM(CASE WHEN up_follower IS NOT NULL THEN 1 ELSE 0 END) AS covered SUM(CASE WHEN up_follower IS NOT NULL THEN 1 ELSE 0 END) AS covered
FROM video FROM video
WHERE status IN ('active', 'unfaved') AND up_mid IS NOT NULL WHERE status IN ('active', 'unfaved', 'removed') AND up_mid IS NOT NULL
""" """
).fetchone() ).fetchone()
follower_total = int(follower_row["total"] or 0) follower_total = int(follower_row["total"] or 0)
@@ -1964,7 +1968,7 @@ class DashboardQueries:
""" """
SELECT b.*, SELECT b.*,
(SELECT COUNT(*) FROM video v WHERE v.up_mid = b.mid (SELECT COUNT(*) FROM video v WHERE v.up_mid = b.mid
AND v.status IN ('active','unfaved')) AS video_count AND v.status IN ('active','unfaved','removed')) AS video_count
FROM up_blacklist b ORDER BY b.created_at, b.mid FROM up_blacklist b ORDER BY b.created_at, b.mid
""" """
).fetchall() ).fetchall()
+9
View File
@@ -134,6 +134,15 @@ function toast(message, kind = "info") {
}, 2600); }, 2600);
} }
document.addEventListener("click", (event) => {
const mark = event.target.closest("[data-removed-tip]");
if (!mark) return;
event.preventDefault();
event.stopPropagation();
toast("该视频已失效");
}, true);
window.addEventListener("error", (event) => { window.addEventListener("error", (event) => {
const target = event.target; const target = event.target;
if (target instanceof HTMLImageElement) { if (target instanceof HTMLImageElement) {
+9
View File
@@ -13,6 +13,13 @@ export function cover(item, className = "thumb") {
loading="lazy" referrerpolicy="no-referrer"></div>`; loading="lazy" referrerpolicy="no-referrer"></div>`;
} }
export function removedCoverMark(item) {
if (item?.status !== "removed") return "";
return `<button type="button" class="cover-removed-mark" data-removed-tip
title="该视频已失效" aria-label="该视频已失效">!</button>`;
}
export function rankBadge(rank, className = "rank-badge") { export function rankBadge(rank, className = "rank-badge") {
if (!rank) return ""; if (!rank) return "";
const top = rank <= 3 ? ` rank-${rank}` : ""; const top = rank <= 3 ? ` rank-${rank}` : "";
@@ -73,6 +80,7 @@ export function videoCard(item, {
<div class="video-cover"> <div class="video-cover">
${cover(item, "video-thumb")} ${cover(item, "video-thumb")}
${durationBadge} ${durationBadge}
${removedCoverMark(item)}
</div> </div>
<div class="video-card-body"> <div class="video-card-body">
<h3 class="video-title">${title}</h3> <h3 class="video-title">${title}</h3>
@@ -150,6 +158,7 @@ export function mediaCard(item, { rank = null, highlight = "" } = {}) {
<div class="media-cover"> <div class="media-cover">
${cover(item, "media-thumb")} ${cover(item, "media-thumb")}
${durationBadge} ${durationBadge}
${removedCoverMark(item)}
</div> </div>
<span class="media-title">${escapeHtml(item.title || "(无标题)")}</span> <span class="media-title">${escapeHtml(item.title || "(无标题)")}</span>
<span class="media-byline"> <span class="media-byline">
+26
View File
@@ -388,6 +388,32 @@ a.live-chip:hover { border-color: color-mix(in srgb, var(--accent) 55%, var(--bo
background: color-mix(in srgb, var(--accent) 70%, transparent); background: color-mix(in srgb, var(--accent) 70%, transparent);
color: #fff; color: #fff;
} }
.detail-cover-wrap { position: relative; }
.cover-removed-mark {
position: absolute;
z-index: 2;
top: 6px;
right: 6px;
display: grid;
place-items: center;
width: 22px;
height: 22px;
padding: 0;
border: 0;
border-radius: 999px;
background: color-mix(in srgb, var(--warning) 82%, #1a1208);
color: #fff8f0;
font-size: 13px;
font-weight: 700;
line-height: 1;
cursor: help;
box-shadow: 0 1px 3px rgba(0, 0, 0, 0.35);
}
.cover-removed-mark:focus-visible {
outline: 2px solid var(--accent);
outline-offset: 2px;
}
.cover-duration { .cover-duration {
position: absolute; position: absolute;
z-index: 1; z-index: 1;
+4
View File
@@ -1,6 +1,7 @@
import { api } from "../api.js"; import { api } from "../api.js";
import { import {
badge, centerActiveChips, cover, emptyState, metricChips, metricGrid, badge, centerActiveChips, cover, emptyState, metricChips, metricGrid,
removedCoverMark,
sectionHeader, videoCard, sectionHeader, videoCard,
} from "../components.js"; } from "../components.js";
import { lineChart, themeColor } from "../charts.js"; import { lineChart, themeColor } from "../charts.js";
@@ -53,7 +54,10 @@ export async function renderVideo(root, { params, query }) {
</header> </header>
<div class="detail-hero"> <div class="detail-hero">
<div class="detail-cover-wrap">
${cover(detail, "hero-cover")} ${cover(detail, "hero-cover")}
${removedCoverMark(detail)}
</div>
<div class="detail-hero-body"> <div class="detail-hero-body">
<h1>${escapeHtml(detail.title)}</h1> <h1>${escapeHtml(detail.title)}</h1>
<div class="detail-meta"> <div class="detail-meta">
+30
View File
@@ -256,11 +256,41 @@ def view_for(
reply: int = 16, reply: int = 16,
danmaku: int = 7, danmaku: int = 7,
bvid: str | None = None, bvid: str | None = None,
title: str | None = None,
duration: int | None = None,
pages: int | None = None,
cover: str | None = None,
up_mid: int | None = None,
up_name: str | None = None,
up_face: str | None = None,
pubdate: int | None = None,
) -> dict[str, Any]: ) -> dict[str, Any]:
item = copy.deepcopy(VIEW_PAYLOAD) item = copy.deepcopy(VIEW_PAYLOAD)
data = item["data"] data = item["data"]
data["aid"] = aid data["aid"] = aid
data["bvid"] = bvid or f"BV{aid}" data["bvid"] = bvid or f"BV{aid}"
if title is not None:
data["title"] = title
if duration is not None:
data["duration"] = duration
if cover is not None:
data["pic"] = cover
if pubdate is not None:
data["pubdate"] = pubdate
data["ctime"] = pubdate
if pages is not None:
data["videos"] = pages
data["pages"] = [
{"cid": 1000 + i, "page": i + 1, "from": "vupload", "part": f"P{i + 1}"}
for i in range(max(1, int(pages)))
]
owner = data.setdefault("owner", {})
if up_mid is not None:
owner["mid"] = up_mid
if up_name is not None:
owner["name"] = up_name
if up_face is not None:
owner["face"] = up_face
data["stat"].update({ data["stat"].update({
"aid": aid, "aid": aid,
"view": view, "view": view,
+34
View File
@@ -0,0 +1,34 @@
# -*- coding: utf-8 -*-
"""失效封面徽标静态落地检查: uv run python tests/test_removed_cover_mark.py"""
from pathlib import Path
ROOT = Path(__file__).resolve().parents[1]
STATIC = ROOT / "reaction_board" / "web" / "static"
def test_removed_cover_mark_markup():
components = (STATIC / "components.js").read_text(encoding="utf-8")
assert "export function removedCoverMark" in components
assert 'data-removed-tip' in components
assert "该视频已失效" in components
assert "${removedCoverMark(item)}" in components
app = (STATIC / "app.js").read_text(encoding="utf-8")
assert 'closest("[data-removed-tip]")' in app
assert 'toast("该视频已失效")' in app
css = (STATIC / "style.css").read_text(encoding="utf-8")
assert ".cover-removed-mark" in css
video = (STATIC / "views" / "video.js").read_text(encoding="utf-8")
assert "removedCoverMark(detail)" in video
def main():
test_removed_cover_mark_markup()
print("OK: 失效封面徽标静态落地检查通过")
if __name__ == "__main__":
main()
+68
View File
@@ -408,6 +408,72 @@ def test_up_follower_snapshot_and_backfill():
db.close() db.close()
def test_refresh_video_metadata_overwrites_api_fields_keeps_nulls():
with tempfile.TemporaryDirectory() as tmp:
db, repo = setup(tmp)
try:
repo.upsert_from_fav(resource(9, pages=1), F1.media_id, was_added=True)
ok = repo.refresh_video_metadata(
aid=9,
bvid="BVnew9",
title="刷新后",
up_mid=200,
up_name="新名",
pubdate=1_710_000_000,
duration=999,
pages=5,
cover="http://example/new.png",
up_face="http://example/face.png",
now=1_800_000_100,
)
assert ok is True
video = repo.get_video(9)
assert video["title"] == "刷新后"
assert video["duration"] == 999
assert video["pages"] == 5
assert video["bvid"] == "BVnew9"
assert video["up_mid"] == 200
assert video["cover"] == "http://example/new.png"
assert video["status"] == "active"
# 空值不冲掉
ok = repo.refresh_video_metadata(
aid=9,
bvid=None,
title="",
up_mid=None,
up_name=None,
pubdate=0,
duration=None,
pages=None,
cover=None,
up_face=None,
now=1_800_000_200,
)
assert ok is True
video = repo.get_video(9)
assert video["title"] == "刷新后"
assert video["duration"] == 999
assert video["pages"] == 5
assert video["bvid"] == "BVnew9"
# 不存在的 aid
assert repo.refresh_video_metadata(
aid=404,
bvid=None, title="x", up_mid=None, up_name=None, pubdate=1,
) is False
# removed 后 refresh 也不改 status
repo.set_video_status(9, "removed")
repo.refresh_video_metadata(
aid=9, bvid="BVkeep", title="仍在", up_mid=200, up_name="新名",
pubdate=1_710_000_000, duration=1000, pages=6,
)
video = repo.get_video(9)
assert video["status"] == "removed"
assert video["title"] == "仍在"
assert video["pages"] == 6
finally:
db.close()
def main(): def main():
test_upsert_and_folder_resource() test_upsert_and_folder_resource()
test_invalid_attr_and_missing() test_invalid_attr_and_missing()
@@ -422,8 +488,10 @@ def main():
test_pending_snapshot_aids() test_pending_snapshot_aids()
test_upsert_snapshot_preserves_view_open() test_upsert_snapshot_preserves_view_open()
test_up_follower_snapshot_and_backfill() test_up_follower_snapshot_and_backfill()
test_refresh_video_metadata_overwrites_api_fields_keeps_nulls()
print("OK: 收藏夹/视频状态机与黑名单测试全部通过") print("OK: 收藏夹/视频状态机与黑名单测试全部通过")
if __name__ == "__main__": if __name__ == "__main__":
main() main()
+81 -6
View File
@@ -216,14 +216,86 @@ def test_unavailable_marks_removed():
try: try:
add_video(repo, 1) add_video(repo, 1)
add_video(repo, 2) add_video(repo, 2)
before = repo.get_video(1)
result = service.run(date=DATE) result = service.run(date=DATE)
assert result.removed == 2 assert result.removed == 2
assert result.failed == 0 assert result.failed == 0
assert result.completed is True assert result.completed is True
assert repo.get_video(1)["status"] == "removed" video1 = repo.get_video(1)
assert repo.get_video(2)["status"] == "removed" video2 = repo.get_video(2)
assert video1["status"] == "removed"
assert video2["status"] == "removed"
# 失效只改 status,不得抹元数据 / 不得写空快照
assert video1["title"] == before["title"]
assert video1["duration"] == before["duration"]
assert video1["bvid"] == before["bvid"]
assert repo.get_snapshot(DATE, 1) is None assert repo.get_snapshot(DATE, 1) is None
assert repo.get_snapshot(DATE, 2) is None assert repo.get_snapshot(DATE, 2) is None
# removed 不再进入后续快照队列
assert 1 not in repo.video_aids_for_snapshot()
assert 2 not in repo.video_aids_for_snapshot()
finally:
db.close()
def test_snapshot_refreshes_video_metadata():
"""快照成功路径以 view_detail 覆盖 pages/duration/title 等元数据。"""
with tempfile.TemporaryDirectory() as tmp:
transport = SnapshotTransport({
1: view_for(
1, view=111, title="新标题P3", duration=3600, pages=3,
cover="http://example/new.jpg", up_name="新UP",
),
})
db, repo, service = make_service(tmp, transport, track_fans=())
try:
add_video(repo, 1)
old = repo.get_video(1)
assert old["title"] == "视频1"
assert old["duration"] == 600
assert old.get("pages") in (None, 0) or old["pages"] != 3
result = service.run_videos(date=DATE)
assert result.success == 1
video = repo.get_video(1)
assert video["title"] == "新标题P3"
assert video["duration"] == 3600
assert video["pages"] == 3
assert video["cover"] == "http://example/new.jpg"
assert video["up_name"] == "新UP"
assert video["status"] == "active"
snap = repo.get_snapshot(DATE, 1)
assert snap["view"] == 111 and snap["ok"] == 1
finally:
db.close()
def test_recent_snapshot_refreshes_metadata():
with tempfile.TemporaryDirectory() as tmp:
now = 1_800_000_000
transport = SnapshotTransport({
7: view_for(7, view=77, title="近新片加P", duration=1200, pages=2,
pubdate=now - 3600),
})
db, repo, service = make_service(tmp, transport, track_fans=())
try:
repo.upsert_from_fav(
FavResource(
aid=7, bvid="BV7", title="旧标题", up_mid=100,
up_name="UP", pubdate=now - 3600, duration=300,
cover=None, attr=0, fav_time=None, pages=1,
),
FOLDER.media_id,
was_added=True,
)
result = service.run_recent_videos(date=DATE, recent_days=2)
assert result.success == 1
video = repo.get_video(7)
assert video["title"] == "近新片加P"
assert video["duration"] == 1200
assert video["pages"] == 2
assert repo.get_snapshot(DATE, 7)["view"] == 77
# recent 不推进完成日
assert repo.kv_get("snapshot:last_completed") in (None, "")
finally: finally:
db.close() db.close()
@@ -440,11 +512,12 @@ def test_run_recent_videos_refreshes_without_completion_markers():
"""小时任务只刷新近窗新片,不写完成日 / 备份。""" """小时任务只刷新近窗新片,不写完成日 / 备份。"""
clock = 1_800_000_000 clock = 1_800_000_000
with tempfile.TemporaryDirectory() as tmp: with tempfile.TemporaryDirectory() as tmp:
# view_for 须带与库内一致的 pubdate,否则快照回写会把近窗片挤出窗口
transport = SnapshotTransport({ transport = SnapshotTransport({
1: view_for(1, view=111), 1: view_for(1, view=111, pubdate=clock - 3600),
2: view_for(2, view=222), 2: view_for(2, view=222, pubdate=clock - 86400),
3: view_for(3, view=333), 3: view_for(3, view=333, pubdate=clock - 10 * 86400),
4: view_for(4, view=444), 4: view_for(4, view=444, pubdate=clock - 10 * 86400),
}) })
db, repo, service = make_service(tmp, transport, track_fans=()) db, repo, service = make_service(tmp, transport, track_fans=())
try: try:
@@ -776,6 +849,8 @@ def main():
test_snapshot_success_resume_and_force() test_snapshot_success_resume_and_force()
test_resume_skips_ok_and_retries_failed() test_resume_skips_ok_and_retries_failed()
test_unavailable_marks_removed() test_unavailable_marks_removed()
test_snapshot_refreshes_video_metadata()
test_recent_snapshot_refreshes_metadata()
test_other_error_writes_failed_placeholder() test_other_error_writes_failed_placeholder()
test_fan_failure_placeholder() test_fan_failure_placeholder()
test_auth_invalid_is_warned_but_snapshot_continues() test_auth_invalid_is_warned_but_snapshot_continues()
+46 -16
View File
@@ -38,11 +38,11 @@ def test_overview():
assert overview["data_date"] == DATES[-1] assert overview["data_date"] == DATES[-1]
assert overview["snapshot_days"] == 3 assert overview["snapshot_days"] == 3
assert overview["is_first_day"] is False assert overview["is_first_day"] is False
assert overview["initial_import_count"] == 2 assert overview["initial_import_count"] == 3 # 含 removed 104
assert overview["counts"]["active"] == 2 assert overview["counts"]["active"] == 2
assert overview["counts"]["unfaved"] == 1 assert overview["counts"]["unfaved"] == 1
assert overview["counts"]["removed"] == 1 assert overview["counts"]["removed"] == 1
assert overview["counts"]["active_ups"] == 2 assert overview["counts"]["active_ups"] == 3 # 含 UP C(仅 removed)
assert overview["counts"]["candidates_pending"] == 1 assert overview["counts"]["candidates_pending"] == 1
assert overview["totals"]["view"] == 1200 + 5200 + 120 assert overview["totals"]["view"] == 1200 + 5200 + 120
assert overview["interactions"] > 0 assert overview["interactions"] > 0
@@ -68,13 +68,16 @@ def test_video_listing_filters_and_sorting():
heat = q.list_videos(sort="heat") heat = q.list_videos(sort="heat")
aids = [item["aid"] for item in heat["items"]] aids = [item["aid"] for item in heat["items"]]
assert 104 not in aids, "removed 视频不应进入榜单" # 热度门槛榜口径含 removed;104 无成功快照仍不进热度
assert 104 not in aids
# 综合热度榜应用最小播放门槛 2000:101(1200)与 103(120)被排除 # 综合热度榜应用最小播放门槛 2000:101(1200)与 103(120)被排除
assert aids == [102], aids assert aids == [102], aids
assert heat["items"][0]["heat_percentile"] == 100.0 assert heat["items"][0]["heat_percentile"] == 100.0
views = q.list_videos(sort="view") views = q.list_videos(sort="view")
assert [item["aid"] for item in views["items"]] == [102, 101, 103] # removed(104) 站内继续展示;无指标排在末尾
assert [item["aid"] for item in views["items"]] == [102, 101, 103, 104]
assert views["items"][-1]["status"] == "removed"
interactions = q.list_videos(sort="interaction") interactions = q.list_videos(sort="interaction")
assert interactions["items"][0]["aid"] == 101 assert interactions["items"][0]["aid"] == 101
@@ -91,14 +94,40 @@ def test_video_listing_filters_and_sorting():
assert [item["aid"] for item in searched["items"]] == [102] assert [item["aid"] for item in searched["items"]] == [102]
folder = q.list_videos(sort="view", folder_id=3700023351) folder = q.list_videos(sort="view", folder_id=3700023351)
assert folder["total"] == 3 assert folder["total"] == 4 # 含 removed 104
folder = q.list_videos(sort="view", folder_id=999) folder = q.list_videos(sort="view", folder_id=999)
assert folder["total"] == 0 assert folder["total"] == 0
page = q.list_videos(sort="view", page=2, page_size=1) page = q.list_videos(sort="view", page=2, page_size=1)
assert page["total"] == 3 assert page["total"] == 4
assert page["pages"] == 3 assert page["pages"] == 4
assert [item["aid"] for item in page["items"]] == [101] assert [item["aid"] for item in page["items"]] == [101]
detail = q.video_detail(104)
assert detail is not None
assert detail["status"] == "removed"
assert detail["aid"] == 104
finally:
db.close()
def test_heat_ranking_includes_removed_with_metrics():
"""removed 进入热度门槛榜口径;有成功快照且过门槛时可见。"""
with tempfile.TemporaryDirectory() as tmp:
db, q = setup(tmp)
try:
db.write(lambda conn: conn.execute(
"UPDATE snapshot SET ok = 1, view = 5000, like = 10, coin = 1, "
"favorite = 1, share = 0, reply = 0, danmaku = 0, err = NULL "
"WHERE aid = 104 AND stat_date = ?",
(DATES[1],),
))
DASHBOARD_CACHE.clear()
heat = q.list_videos(sort="heat")
aids = [item["aid"] for item in heat["items"]]
assert 104 in aids
assert next(item for item in heat["items"] if item["aid"] == 104)["status"] == "removed"
finally: finally:
db.close() db.close()
@@ -183,13 +212,14 @@ def test_up_queries():
try: try:
ups = q.list_ups() ups = q.list_ups()
assert ups["sort"] == "avg_views" assert ups["sort"] == "avg_views"
assert ups["total"] == 2 assert ups["total"] == 3 # 含仅有 removed 视频的 UP C
assert ups["items"][0]["up_mid"] == 1 assert ups["items"][0]["up_mid"] == 1
assert ups["items"][0]["video_count"] == 2 assert ups["items"][0]["video_count"] == 2
assert ups["items"][0]["total_views"] == 6400 assert ups["items"][0]["total_views"] == 6400
assert ups["items"][0]["avg_views"] == 3200.0 assert ups["items"][0]["avg_views"] == 3200.0
assert ups["items"][0]["up_face"] == "https://i0.hdslb.com/bfs/face/a.jpg" assert ups["items"][0]["up_face"] == "https://i0.hdslb.com/bfs/face/a.jpg"
assert ups["items"][0]["follower_latest"] == 12345 assert ups["items"][0]["follower_latest"] == 12345
assert {item["up_mid"] for item in ups["items"]} == {1, 2, 3}
one = q.list_ups(min_videos=2) one = q.list_ups(min_videos=2)
assert one["total"] == 1 assert one["total"] == 1
@@ -220,8 +250,8 @@ def test_up_queries():
# 大小写不敏感 # 大小写不敏感
lower = q.list_ups(q="up b") lower = q.list_ups(q="up b")
assert lower["total"] == 1 and lower["items"][0]["up_mid"] == 2 assert lower["total"] == 1 and lower["items"][0]["up_mid"] == 2
# 命中多个 UP # 命中多个 UP(含 UP C)
assert q.list_ups(q="UP")["total"] == 2 assert q.list_ups(q="UP")["total"] == 3
finally: finally:
db.close() db.close()
@@ -457,7 +487,7 @@ def test_year_filter_and_published_sort():
db, q = setup(tmp) db, q = setup(tmp)
try: try:
by_year = q.list_videos(sort="view", year=2026) by_year = q.list_videos(sort="view", year=2026)
assert [item["aid"] for item in by_year["items"]] == [102, 101, 103] assert [item["aid"] for item in by_year["items"]] == [102, 101, 103, 104]
assert by_year["year"] == 2026 assert by_year["year"] == 2026
none = q.list_videos(sort="view", year=2025) none = q.list_videos(sort="view", year=2025)
@@ -465,13 +495,13 @@ def test_year_filter_and_published_sort():
assert none["total"] == 0 assert none["total"] == 0
published = q.list_videos(sort="published") published = q.list_videos(sort="published")
assert [item["aid"] for item in published["items"]] == [103, 101, 102] assert [item["aid"] for item in published["items"]] == [103, 101, 102, 104]
added = q.list_videos(sort="added") added = q.list_videos(sort="added")
assert added["total"] == 3 assert added["total"] == 4 # 含 removed
ups = q.list_ups(sort="efficiency", year=2026) ups = q.list_ups(sort="efficiency", year=2026)
assert ups["year"] == 2026 and ups["total"] == 2 assert ups["year"] == 2026 and ups["total"] == 3
ups_none = q.list_ups(sort="views", year=2025) ups_none = q.list_ups(sort="views", year=2025)
assert ups_none["total"] == 0 assert ups_none["total"] == 0
finally: finally:
@@ -498,9 +528,9 @@ def test_about_payload():
"growth_rate_prior": 1000, "growth_rate_prior": 1000,
} }
assert about["heat_weights"]["view"] == 0.30 assert about["heat_weights"]["view"] == 0.30
assert about["followers"]["videos_total"] == 3 assert about["followers"]["videos_total"] == 4 # 含 removed
assert about["followers"]["videos_covered"] == 3 assert about["followers"]["videos_covered"] == 3
assert about["followers"]["videos_missing"] == 0 assert about["followers"]["videos_missing"] == 1
assert about["folders"][0]["media_id"] == 3700023351 assert about["folders"][0]["media_id"] == 3700023351
assert about["preview"] is False assert about["preview"] is False
finally: finally: