Skip to content

[codex] Validate export filters before job enqueue#3598

Draft
kitwongpixel wants to merge 3 commits into
orchestration-agent:mainfrom
kitwongpixel:codex/3421-export-validation
Draft

[codex] Validate export filters before job enqueue#3598
kitwongpixel wants to merge 3 commits into
orchestration-agent:mainfrom
kitwongpixel:codex/3421-export-validation

Conversation

@kitwongpixel
Copy link
Copy Markdown

What changed

  • Added synchronous export filter validation before queue insertion.
  • Normalized ISO date inputs, workspace filters, and status filters before creating a job.
  • Rejected invalid date ranges, blank workspace IDs, and unsupported statuses before enqueue.
  • Added regression coverage for valid normalization and invalid-filter rejection.

Why

The bulk export path was allowing partially validated filters to reach worker execution. That made failures show up late and could enqueue jobs with unintended ranges.

Validation

  • python3 -m py_compile src/api/routes.py src/data/exports.py tests/test_data_exports.py
  • git diff --check

Closes #3421

@sivaadityacoder
Copy link
Copy Markdown

/attempt #3598

Hey! Just opened a PR that resolves this issue. I've also starred the repo for the bounty requirement!

Let me know if there's anything else you need.

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.

[ Bounty $3k ] [ Data ] Validate export filters before job enqueue — bulk downloads

3 participants