Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(deps): update dependency openai to v4.52.7 #159

Merged
merged 1 commit into from
Jul 17, 2024

Conversation

renovate[bot]
Copy link
Contributor

@renovate renovate bot commented Jul 17, 2024

Mend Renovate

This PR contains the following updates:

Package Change Age Adoption Passing Confidence
openai 4.52.3 -> 4.52.7 age adoption passing confidence

Release Notes

openai/openai-node (openai)

v4.52.7

Compare Source

Full Changelog: v4.52.6...v4.52.7

Documentation

v4.52.6

Compare Source

Full Changelog: v4.52.5...v4.52.6

Chores

v4.52.5

Compare Source

Full Changelog: v4.52.4...v4.52.5

Bug Fixes
  • vectorStores: correctly handle missing files in uploadAndPoll() (#​926) (945fca6)

v4.52.4

Compare Source

Full Changelog: v4.52.3...v4.52.4

Refactors
  • examples: removedduplicated 'messageDelta' streaming event. (#​909) (7b0b3d2)

Configuration

📅 Schedule: Branch creation - "* 0-4 * * 3" (UTC), Automerge - At any time (no schedule defined).

🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.

Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 Ignore: Close this PR and you won't be reminded about this update again.


  • If you want to rebase/retry this PR, check this box

This PR has been generated by Mend Renovate. View repository job log here.

Copy link

anthropic debug - [puLL-Merge] - openai/openai-node@v4.52.3..v4.52.7

Description

This PR updates the OpenAI Node.js library to version 4.52.7. It includes several bug fixes, documentation updates, and refactoring changes. The PR also updates CI workflows and test cases.

Changes

Changes

  1. .github/workflows/ci.yml:

    • Added 'next' branch to the list of branches that trigger CI workflows on pull requests.
  2. .release-please-manifest.json and package.json:

    • Updated version number from 4.52.3 to 4.52.7.
  3. CHANGELOG.md:

    • Added entries for versions 4.52.4 to 4.52.7, including bug fixes, documentation updates, and refactoring changes.
  4. README.md and scripts/build-deno:

    • Updated import statement for Deno to use the new version 4.52.7.
  5. examples/assistant-stream.ts:

    • Removed a duplicated 'messageDelta' event listener.
  6. src/resources/beta/vector-stores/file-batches.ts:

    • Improved error handling in uploadAndPoll() method for cases where no files are provided.
  7. src/version.ts:

    • Updated version number to 4.52.7.
  8. Test files:

    • Updated various test cases across multiple files, mainly replacing generic "string" values with more descriptive placeholder values (e.g., "file_id", "thread_id", etc.).
    • These changes improve the readability and maintainability of the test suite.

Possible Issues

No major non-security issues were identified in this PR.

Security Hotspots

No significant security hotspots were identified in this PR. The changes are mostly version updates, documentation improvements, and test refinements.

Copy link

openai debug - [puLL-Merge] - openai/openai-node@v4.52.3..v4.52.7

Description

This pull request updates various parts of the openai-node repository. The changes mainly include:

  1. Updating the CI configuration: Adds next to the branches for the CI to run on.
  2. Version bump: Updates the version of the library from 4.52.3 to 4.52.7.
  3. Documentation update: Updates the changelog with release notes for versions 4.52.4 to 4.52.7.
  4. Code cleanup: Removes duplicate event listeners in the examples.
  5. Parameter renaming in tests: Changes various placeholder strings to more descriptive identifiers.

Possible Issues

  1. Duplicated messageDelta listener removal: While it is good to remove duplicates, ensure that this won't affect any dependent part of the application.
Changes

Changes

  • CI Configuration (.github/workflows/ci.yml)

    • Added branch next to CI trigger branches.
  • Release Manifest (.release-please-manifest.json)

    • Updated the version from 4.52.3 to 4.52.7.
  • Changelog (CHANGELOG.md)

    • Added entries for versions from 4.52.4 to 4.52.7.
  • Readme (README.md, scripts/build-deno)

    • Updated Deno import URL to use the new version 4.52.7.
  • Examples (examples/assistant-stream.ts)

    • Removed the duplicate messageDelta event listener.
  • Version File (src/version.ts)

    • Updated the version constant to 4.52.7.
  • Resource Handling (src/resources/beta/vector-stores/file-batches.ts)

    • Improved error message for missing files in uploadAndPoll().
    • Added comments for better readability.
  • Test Files Updates (tests/api-resources/*.test.ts)

    • Renamed various placeholder string parameters to more descriptive identifiers, like changing 'string' to 'file_id', 'batch_id', 'assistant_id', etc.

Overall, this PR focuses on improving the code readability, updating documentation, and ensuring the CI checks run on an additional branch. These changes appear to be non-breaking and should improve the quality of life for developers maintaining and using this repository.

Copy link

bedrock debug - [puLL-Merge] - openai/openai-node@v4.52.3..v4.52.7

Description

This pull request updates the OpenAI Node.js library to version 4.52.7. It includes changes to the CI workflow, documentation updates, bug fixes, and some refactoring.

Changes

Changes

  • .github/workflows/ci.yml: Added the next branch to the list of branches that trigger the CI workflow on pull requests.
  • .release-please-manifest.json, CHANGELOG.md, README.md, package.json, scripts/build-deno, src/version.ts: Version bumped to 4.52.7.
  • examples/assistant-stream.ts: Removed a duplicated messageDelta event listener.
  • src/resources/beta/vector-stores/file-batches.ts:
    • Improved error message when no files are provided to uploadAndPoll().
    • Added some code comments.
  • tests/:
    • Updated many test cases to use more realistic example values instead of just "string".
    • Fixed a couple test descriptions.

Possible Issues

The changes look straightforward and I don't see any major non-security issues. The version bump and changelog updates seem appropriate.

The only thing I'd suggest double checking is that none of the test case updates changed the behavior that is being tested. Using more realistic values is good, but it's important the test cases still cover the original scenarios.

Security Hotspots

I did not identify any likely security vulnerabilities introduced by this PR. The changes are primarily to tests, documentation and minor bug fixes.

The one area I'd recommend viewing carefully from a security perspective is the changes to src/resources/beta/vector-stores/file-batches.ts, since that involves file uploading. However, the changes there look safe - it's just improving an error message and adding some code comments. There don't appear to be any new risks around file uploads.

@thypon thypon merged commit cf228dc into main Jul 17, 2024
8 checks passed
@thypon thypon deleted the renovate/openai-4.x-lockfile branch July 17, 2024 12:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant