Skip to content

Conversation

@Khurdhula-Harshavardhan
Copy link
Collaborator

This pull request introduces several improvements and updates across the codebase, focusing on enhancing the CI workflow, expanding API parameter support, and improving test organization. The most notable changes include a refactor of the GitHub Actions workflow for SDK tests, new and updated interface parameters for API flexibility, and improved test script structure.

CI/CD Workflow Improvements:

  • Refactored .github/workflows/sdk-tests.yml to split the workflow into separate build and test jobs. The build job now caches dependencies and build artifacts, and the test job runs in parallel for different test types (audio, classification, validate, vision, web), restoring the build cache for efficiency. [1] [2]

API Parameter Enhancements:

  • Added new optional parameters to several interfaces for greater flexibility:
    • file_store_key in TranslateImageParams (src/general/interfaces.ts)
    • Optional steps in PredictionParams (src/general/interfaces.ts)
    • dimensions, instruction, and query in EmbeddingV2Params (src/general/interfaces.ts)
    • Optional name in PromptCreateParams (src/prompt_engine/interfaces.ts)
    • fine_grained in VOCRParams for high fidelity bounding boxes (src/vision/interfaces.ts)
    • Expanded element_prompts to accept a dictionary for element extraction in AIScrapeParams (src/web/interfaces/scrape.ts)

Test and Script Organization:

  • Updated package.json test scripts to use a more modular structure, introducing a test:run script and adding a test:classification target. The main test script now delegates to test:all for easier maintenance and extensibility.
  • Commented out two cookie-related tests in tests/web.test.ts to temporarily disable them, likely due to instability or pending fixes. [1] [2]

Version Updates:

  • Bumped package version from 0.3.8 to 0.4.0 in both package.json and jsr.json to reflect new features and breaking changes. [1] [2]

These changes collectively improve CI efficiency, API usability, and maintainability of test scripts, while preparing the package for a new minor release.

Copy link
Collaborator

@VineeTagarwaL-code VineeTagarwaL-code left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to go !

@Khurdhula-Harshavardhan Khurdhula-Harshavardhan merged commit ddfc0ee into main Dec 1, 2025
6 of 7 checks passed
@Khurdhula-Harshavardhan Khurdhula-Harshavardhan deleted the fix/types branch December 1, 2025 18:46
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.

3 participants