Skip to content

Add data connectors API, scope filter, and segment duration params#94

Merged
kdr merged 1 commit into
mainfrom
kdr-updates0303
Mar 4, 2026
Merged

Add data connectors API, scope filter, and segment duration params#94
kdr merged 1 commit into
mainfrom
kdr-updates0303

Conversation

@kdr

@kdr kdr commented Mar 3, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Summary

  • Add new Data Connectors API wrapper (client.dataConnectors.list()) for listing configured data connectors
  • Add optional scope field ('file' | 'segment') to video_info filters used in Chat and Search
  • Add minDuration/maxDuration query params to listFileSegments
  • Fix camelCase→snake_case mapping for startTimeAfter/endTimeBefore in listFileSegments
  • Regenerate clients from updated OpenAPI spec (cloudglue-api-spec PR build(deps): bump axios from 1.12.1 to 1.13.5 #86)

Test plan

  • npm run build compiles without errors
  • Verify client.dataConnectors.list() returns configured connectors
  • Verify chat/search filters accept the scope field
  • Verify listFileSegments accepts minDuration/maxDuration params

🤖 Generated with Claude Code

Summary by CodeRabbit

  • New Features

    • Added Data Connectors API support with a new public interface for managing data connectors.
    • Enhanced file segment queries with duration-based filtering capabilities (minimum and maximum duration parameters).

  • Chores

    • Bumped version to 0.7.1.

@kdr @claude
Add data connectors API, scope filter, and segment duration params
566d10a 
- Add new Data Connectors API wrapper exposing list() for configured connectors
- Add scope field ('file' | 'segment') to video_info filter for chat/search
- Add minDuration/maxDuration params to listFileSegments
- Fix camelCase to snake_case mapping for startTimeAfter/endTimeBefore in listFileSegments
- Regenerate clients from updated OpenAPI spec

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Mar 3, 2026 •
edited
Loading

Copy link
Copy Markdown
📝 Walkthrough

Walkthrough

This pull request integrates a new Data Connectors API into the Cloudglue client. It adds a wrapper class for the data connectors API, exposes new types for data connector schemas, enhances the files API with duration filter parameters, and wires the data connectors API into the main client instance.

Changes

Cohort / File(s) Summary
Version & Metadata
package.json, spec		 Version bumped from 0.7.0 to 0.7.1; submodule commit reference updated.
Data Connectors API Integration
src/api/data-connectors.api.ts, src/client.ts		 New EnhancedDataConnectorsApi wrapper class created; integrated into Cloudglue client as public dataConnectors property with initialization and wiring to shared Axios config.
Type Definitions
src/types.ts		 Added DataConnector and DataConnectorList exported types from generated schemas; extended Filter.video_info with optional scope field ('file' | 'segment').
Files API Enhancement
src/api/files.api.ts		 Added minDuration and maxDuration optional parameters to listFileSegments method; parameters destructured and mapped to min_duration and max_duration query keys.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~22 minutes

Poem

🐰 A new API hops in, with grace so fine,
Data connectors and durations align,
Types and clients in harmony dance,
Files flow smoother with filtered expanse! ✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately summarizes the three main changes: adding the data connectors API, scope filter, and segment duration parameters.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch kdr-updates0303

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@kdr kdr requested a review from amyxst March 4, 2026 00:45
@kdr kdr merged commit 8369fbc into main Mar 4, 2026
1 check passed
@kdr kdr deleted the kdr-updates0303 branch March 4, 2026 01:26
@coderabbitai coderabbitai Bot mentioned this pull request Mar 16, 2026
Merged
@coderabbitai coderabbitai Bot mentioned this pull request Mar 30, 2026
Merged
3 tasks
@coderabbitai coderabbitai Bot mentioned this pull request Apr 21, 2026
Merged
@coderabbitai coderabbitai Bot mentioned this pull request Jun 10, 2026
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@amyxst amyxst amyxst approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@kdr @amyxst