feat: 增加 BOSS 猎头岗位过滤 - #30
Conversation
|
@codex reveiw |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 6b79049360
ℹ️ 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".
| if boss_filter_config.exclude_headhunter_jobs && is_headhunter_job_card(&job_card_ele)? { | ||
| return Ok(CardOutcome::Skipped(SkipReason::Headhunter)); | ||
| } |
There was a problem hiding this comment.
Deduplicate headhunter skips across rescans
When loading another batch, the outer loop rescans every card still on the page, but this return neither marks the card as seen nor records its job ID in a task-local filtered set. Because the filter runs before the click that would add is-seen, each headhunter card is counted and logged again after every scroll, so skipped_headhunter can greatly overstate the number of jobs and produce repeated per-card log messages. Track excluded job IDs (or otherwise mark the card as handled) before returning this outcome.
Useful? React with 👍 / 👎.
修改内容
BOSS 猎头岗位过滤开关,位置在配置中心 -> 求职方案 -> 岗位筛选img.job-tag-icon[alt=猎头]检测到猎头岗位,已按配置跳过猎头岗位跳过 N 条统计技术说明
.job-tag-icon的alt属性alt=猎头,避免误判其他岗位标签验证
cargo test --no-default-features boss::handler::position_say_hello --lib通过,40 个测试全过./node_modules/.bin/tsc --noEmit通过