Skip to content

chore(mediacrawler): checkpoint local login changes#880

Open
josh87523 wants to merge 6 commits into
NanmiCoder:mainfrom
josh87523:chore/dirty-worktree-checkpoint-20260422
Open

chore(mediacrawler): checkpoint local login changes#880
josh87523 wants to merge 6 commits into
NanmiCoder:mainfrom
josh87523:chore/dirty-worktree-checkpoint-20260422

Conversation

@josh87523
Copy link
Copy Markdown

@josh87523 josh87523 commented Apr 22, 2026

Summary

  • checkpoint local login-related changes on the existing task branch
  • add repository hygiene ignores and a short business/product logic note
  • merge current upstream/main to clear PR merge conflicts

Discovery Gate

用户目标

  • 完成 stop hook 判定未闭环的 MediaCrawler 工程收尾:清理 dirty tree、提交、push、确认 PR 和验证 gate。

任务分类

  • C:既有分支收尾与冲突自愈,不新增架构/第三方集成;后续因 PR DIRTY 升级为 L2 merge-conflict closure。
  • 调研结论:基于现有分支和 Workspace finisher/PR validation 改造,不从零实现。

AI 推断的相关面清单

  • source/caller: media_platform/douyin/login.py, media_platform/douyin/client.py, media_platform/tieba/*, cmd_arg/arg.py
  • tests/fixtures/contracts: tests/test_cmd_arg_tieba.py, tests/test_tieba_client_pagination.py, tests/test_tieba_extractor.py, test/test_utils.py
  • docs/history: README.md, README_en.md, docs/business-product-logic.md, CDP guide/common questions docs
  • config/env: pyproject.toml, uv.lock, .gitignore; no env/secrets edited
  • runtime/release surface: no local launchd/release entrypoint for this repo in scope
  • external side effect: GitHub PR/branch push only

证据

  • finisher initially reported dirty tree and missing PR recognition
  • gh pr view confirmed PR chore(mediacrawler): checkpoint local login changes #880 head d5fafb8 and earlier DIRTY merge state
  • upstream/main merged into this branch; .gitignore and uv.lock conflicts resolved
  • uv.lock parse issue self-healed by removing stale duplicate tuna package records inconsistent with current pyproject default index

不确定项 / 需要用户拍板

  • none for local closure; merge is subject to upstream repository permissions/review/checks

验证矩阵

  • local: git diff --check passed
  • contract: uv lock --check passed; uv lock completed
  • smoke: uv run pytest test/test_utils.py tests/test_cmd_arg_tieba.py tests/test_tieba_client_pagination.py tests/test_tieba_extractor.py => 18 passed
  • online: not applicable; no live runtime/launchd/release entrypoint touched
  • external endstate: branch pushed to origin/chore/dirty-worktree-checkpoint-20260422 and PR chore(mediacrawler): checkpoint local login changes #880 updated

Validation

  • uv lock --check
  • uv lock
  • uv run pytest test/test_utils.py tests/test_cmd_arg_tieba.py tests/test_tieba_client_pagination.py tests/test_tieba_extractor.py
  • python3 /Users/obayotian/Workspace/scripts/codex_pr_validation.py --cwd /Users/obayotian/Workspace/MediaCrawler --run --json

@josh87523 josh87523 requested a review from NanmiCoder as a code owner April 22, 2026 02:34
@dosubot dosubot Bot added the size:S This PR changes 10-29 lines, ignoring generated files. label Apr 22, 2026
@dosubot dosubot Bot added size:M This PR changes 30-99 lines, ignoring generated files. and removed size:S This PR changes 10-29 lines, ignoring generated files. labels May 19, 2026
Obayo Tian added 2 commits May 20, 2026 05:33
@dosubot dosubot Bot added size:L This PR changes 100-499 lines, ignoring generated files. and removed size:M This PR changes 30-99 lines, ignoring generated files. labels May 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:L This PR changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant