Skip to content

fix(tb-prod): holidays schema — replace date filter with after/before#14

Merged
FilipBuhinicek merged 1 commit intomainfrom
fix/filter-serialization-and-holiday-schema
Apr 7, 2026
Merged

fix(tb-prod): holidays schema — replace date filter with after/before#14
FilipBuhinicek merged 1 commit intomainfrom
fix/filter-serialization-and-holiday-schema

Conversation

@FilipBuhinicek
Copy link
Copy Markdown
Contributor

Summary

  • Removed unsupported filter: "date" from holidays schema — the Productive API rejects filter[date] on this endpoint
  • Added after and before filter fields (same pattern as allocations/activities) which the API actually supports
  • This enables tb-prod query holidays --filter '{"holiday_calendar_id": "112", "after": "2026-04-07", "before": "2026-04-21"}'

Context

Discovered while using the release-tracker skill — holiday queries were failing silently and falling back to pagination + client-side filtering.

Direct curl confirmed: filter[date] → 400 error, filter[after]/filter[before] → works.

Test plan

  • tb-prod describe holidays --include schema shows after and before filters instead of date
  • tb-prod query holidays --filter '{"holiday_calendar_id": "112", "after": "2026-04-07", "before": "2026-04-21"}' returns correct results
  • cargo test -p tb-prod --lib filter passes

🤖 Generated with Claude Code

…lidays

The Productive API does not support `filter[date]` on the holidays
endpoint despite the schema listing it. The actual API uses custom
`after` and `before` filters (same pattern as allocations/activities).

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@FilipBuhinicek FilipBuhinicek marked this pull request as ready for review April 7, 2026 13:21
@FilipBuhinicek FilipBuhinicek requested a review from trogulja April 7, 2026 13:21
@FilipBuhinicek FilipBuhinicek merged commit 9b2e070 into main Apr 7, 2026
1 check passed
@FilipBuhinicek FilipBuhinicek deleted the fix/filter-serialization-and-holiday-schema branch April 7, 2026 13:32
trogulja added a commit that referenced this pull request Apr 7, 2026
Regenerated schema.json from ai-agent's generate-schema script.

New resources: approval_policy_assignments, calendar_events, email_threads,
expense_line_items, integrations, resource_requests, scenario_bookings, slack_users

New enums: approval_policy_target_type, booking_approval_status,
integration_type, resource_request_status

Also includes: param field additions across most resources, improved
descriptions, and the holidays after/before filter fix from #14.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
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