Skip to content

fix(crawler): publication forbidden-name false positive - #2554

Merged
twoimo merged 1 commit into
datafrom
develop
Aug 16, 2026
Merged

twoimo merged 1 commit into
datafrom
develop

Conversation

@twoimo

@twoimo twoimo commented Aug 16, 2026

Copy link
Copy Markdown
Owner

Fix publication bundle PATH error on video IDs containing 'log'. Adds separator-boundary regex and pre-filter in inventory scan.

…filter

The publication bundle validator rejected data files whose YouTube video
ID happened to contain substrings like 'log' (e.g. kvXlRgISLog.jsonl).
Switch from substring matching to separator-boundary regex so embedded
occurrences in Base64-like IDs no longer trigger false positives.

Also pre-filter forbidden-name files in _publication_inventory (skip
instead of crash) and prune forbidden-component directories from the
os.walk traversal.
@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@twoimo
twoimo merged commit 7eddf00 into data Aug 16, 2026
12 of 13 checks passed
@vercel

vercel Bot commented Aug 16, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
tzudong Building Building Preview Aug 16, 2026 1:49pm

This branch was successfully deployed

1 active deployment
Preview — 473a7d53 Deployed Aug 16, 2026 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant