Skip to content

Conversation

@devin-ai-integration
Copy link
Contributor

This PR adds support for setting a fixed pipeline ID via the VECTORIZE_PIPELINE_ID environment variable.

This change allows users to set the pipeline ID once in their environment instead of having to provide it with each request, while maintaining backward compatibility.

Link to Devin run: https://app.devin.ai/sessions/14a808ba3cd146b9a843963caf800778

Requested by: nicolo.boschi@vectorize.io

devin-ai-integration bot and others added 2 commits March 28, 2025 09:41
Co-Authored-By: nicolo.boschi@vectorize.io <nicolo.boschi@vectorize.io>
Co-Authored-By: nicolo.boschi@vectorize.io <nicolo.boschi@vectorize.io>
@devin-ai-integration
Copy link
Contributor Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add "(aside)" to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

@nicoloboschi nicoloboschi requested a review from Copilot March 28, 2025 12:46
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds support for a fixed pipeline ID by allowing users to configure the environment variable VECTORIZE_PIPELINE_ID, enabling the server to use a pre-set pipeline ID rather than requiring it with each request.

  • Updates the required parameters for tools in the server based on the presence of VECTORIZE_PIPELINE_ID.
  • Logs a configuration message when a fixed pipeline ID is used.
  • Updates documentation in README.md to reflect the new environment variable setup.

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
src/index.ts Conditional adjustments in required fields and logging of PIPELINE_ID, affecting how pipeline IDs are passed to tool functions.
README.md Updated documentation with instructions on setting the VECTORIZE_PIPELINE_ID environment variable.

Copy link
Collaborator

@nicoloboschi nicoloboschi left a comment

Choose a reason for hiding this comment

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

+1

@nicoloboschi nicoloboschi merged commit 8b6062d into main Mar 28, 2025
1 check passed
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