Skip to content

fix: declare SOAR dotenv dependency#259

Open
Haihan-Jiang wants to merge 1 commit into
google:mainfrom
Haihan-Jiang:add-secops-soar-dotenv-dependency
Open

fix: declare SOAR dotenv dependency#259
Haihan-Jiang wants to merge 1 commit into
google:mainfrom
Haihan-Jiang:add-secops-soar-dotenv-dependency

Conversation

@Haihan-Jiang
Copy link
Copy Markdown

The SecOps SOAR package imports dotenv from secops_soar_mcp.bindings, but python-dotenv is not listed in the package dependencies.

This adds the missing runtime dependency to server/secops-soar/pyproject.toml.

Tests:

  • python3 -m py_compile server/secops-soar/secops_soar_mcp/bindings.py
  • python3 -m pip install --dry-run ./server/secops-soar (metadata prepared; dependency resolution then stopped because the local Python is 3.9 and the existing mcp>=1.4.1 requirement needs Python 3.10+)
  • git diff --check

@Haihan-Jiang Haihan-Jiang requested a review from a team May 26, 2026 08:55
@Haihan-Jiang
Copy link
Copy Markdown
Author

This is ready for review. The SOAR dotenv dependency is now declared, and the visible checks are green.

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