Skip to content

Fix limit parameter to respect config.jobCount for TheirStack fetcher#6

Merged
Mayank-glitch-cpu merged 2 commits intocodex/integrate-ashby-job-ingestion-strategyfrom
copilot/sub-pr-2-yet-again
Feb 11, 2026
Merged

Fix limit parameter to respect config.jobCount for TheirStack fetcher#6
Mayank-glitch-cpu merged 2 commits intocodex/integrate-ashby-job-ingestion-strategyfrom
copilot/sub-pr-2-yet-again

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Feb 11, 2026

The limit parameter was always defaulted to 10 even when not provided as a query parameter, preventing fetchTheirStack from falling back to config.jobCount/JOB_COUNT.

Changes

  • Changed limit default from 10 to undefined in /api/pipeline/step1 endpoint (line 92 of backend/src/routes/pipeline.ts)

This allows fetchTheirStack to use its existing fallback logic:

const to = options.range
  ? options.range.to
  : Math.min(deduped.length, options.limit ?? config.jobCount);

When no limit query param is provided, the function now correctly uses the configured JOB_COUNT environment variable instead of always using 10.


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Co-authored-by: Mayank-glitch-cpu <85422185+Mayank-glitch-cpu@users.noreply.github.com>
Copilot AI changed the title [WIP] Update Ashby slug pool expansion and API updates Fix limit parameter to respect config.jobCount for TheirStack fetcher Feb 11, 2026
@Mayank-glitch-cpu Mayank-glitch-cpu marked this pull request as ready for review February 11, 2026 18:55
@Mayank-glitch-cpu Mayank-glitch-cpu merged commit d985b8e into codex/integrate-ashby-job-ingestion-strategy Feb 11, 2026
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.

2 participants