Skip to content

Add support for fixed pipeline ID via environment variable #5

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

Merged
merged 3 commits into from
Mar 28, 2025

Conversation

devin-ai-integration[bot]
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: [email protected]

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 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.

1 participant