Skip to content

Removed unused LangChain OpenAI dependency - #115

Merged
mounacheikho-cmd merged 2 commits into
climateandtech:maintenance/third-round-security-updatesfrom
mounacheikho-cmd:issue-103-remove-unused-langchain-openai
Aug 25, 2026
Merged

Removed unused LangChain OpenAI dependency#115
mounacheikho-cmd merged 2 commits into
climateandtech:maintenance/third-round-security-updatesfrom
mounacheikho-cmd:issue-103-remove-unused-langchain-openai

Conversation

@mounacheikho-cmd

Copy link
Copy Markdown
Collaborator

Part of #103.

Removes unused langchain-openai==0.3.8, addressing Dependabot alert #128.

The application uses LlamaIndex’s OpenAI provider and does not import langchain_openai. Three analyzer tests were still mocking langchain_openai.ChatOpenAI, even though the analyzer calls report_analyst.core.analyzer.get_llm. Those mocks now target the provider factory actually used by the application.

Validation

  • Confirmed langchain-openai is not installed
  • pip check passed
  • Application import passed
  • Analyzer tests: 17 passed, 1 skipped
  • Full suite: 369 passed, 10 skipped
  • Ruff, Black, and isort checks passed

Full-suite validation used PR #104’s Chroma versions because the current Chroma pin does not build in the validation environment.

@mounacheikho-cmd
mounacheikho-cmd merged commit 3ad48fb into climateandtech:maintenance/third-round-security-updates Aug 25, 2026
7 of 8 checks passed
@mounacheikho-cmd
mounacheikho-cmd deleted the issue-103-remove-unused-langchain-openai branch August 25, 2026 16:32
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