Skip to content

Conversation

@Chan9390
Copy link
Contributor

Context

  • Lighthouse AI previously used hardcoded tools implemented directly in the NextJS application
  • Prowler MCP server was released containing most of these tools
  • This PR refactors Lighthouse to consume tools from the MCP server instead of maintaining hardcoded implementations

Description

Tool Integration:

  • Removed all hardcoded tool implementations from Lighthouse
  • Removed getLighthouse-prefixed functions in the actions directory that were created specifically for tool calling
  • Created a singleton MCP client to connect to Prowler MCP server

Dependency Upgrades:

  • Added langgraph/mcp-adapters to v1.0 to support passing custom headers (authentication) to MCP client
  • Updated related dependencies: langchain-core, openai, and AWS SDK libraries to v1

Architecture Simplification:

  • Replaced langgraph-supervisor with a simple LangChain agent
  • Implemented two meta tools for MCP interaction:
    • describe_tool: Retrieves tool schema from MCP server
    • execute_tool: Executes tools via MCP server
  • Simplified overall architecture by removing complexity and better separating responsibilities

Note: MCP Server is now a critical dependency for Lighthouse. If the MCP server is unavailable or Lighthouse cannot connect to it, Lighthouse will lose the ability to retrieve tenant data and execute core functionality.

Steps to review

TODO

Checklist

UI

  • All issue/task requirements work as expected on the UI
  • Screenshots/Video of the functionality flow (if applicable) - Mobile (X < 640px)
  • Screenshots/Video of the functionality flow (if applicable) - Table (640px > X < 1024px)
  • Screenshots/Video of the functionality flow (if applicable) - Desktop (X > 1024px)
  • Ensure new entries are added to CHANGELOG.md, if applicable.

API

  • Verify if API specs need to be regenerated.
  • Check if version updates are required (e.g., specs, Poetry, etc.).
  • Ensure new entries are added to CHANGELOG.md, if applicable.

License

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@Chan9390 Chan9390 requested review from a team as code owners November 18, 2025 09:04
@github-actions
Copy link
Contributor

github-actions bot commented Nov 18, 2025

Conflict Markers Resolved

All conflict markers have been successfully resolved in this pull request.

@github-actions
Copy link
Contributor

github-actions bot commented Nov 18, 2025

✅ All necessary CHANGELOG.md files have been updated.

@github-actions
Copy link
Contributor

github-actions bot commented Nov 18, 2025

🔒 Container Security Scan

Image: prowler-ui:a870e86
Last scan: 2025-12-16 18:42:18 UTC

✅ No Vulnerabilities Detected

The container image passed all security checks. No known CVEs were found.

📋 Resources:

@github-actions github-actions bot removed documentation provider/aws Issues/PRs related with the AWS provider component/api provider/iac Issues/PRs related with the IaC provider metadata-review labels Dec 15, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Dec 15, 2025

🔒 Container Security Scan

Image: prowler:a870e86
Last scan: 2025-12-16 18:42:06 UTC

📊 Vulnerability Summary

Severity Count
🔴 Critical 3
Total 3

3 package(s) affected

⚠️ Action Required

Critical severity vulnerabilities detected. These should be addressed before merging:

  • Review the detailed scan results
  • Update affected packages to patched versions
  • Consider using a different base image if updates are unavailable

📋 Resources:

alejandrobailo
alejandrobailo previously approved these changes Dec 15, 2025
alejandrobailo
alejandrobailo previously approved these changes Dec 15, 2025
@alejandrobailo alejandrobailo self-assigned this Dec 16, 2025
@alejandrobailo alejandrobailo self-requested a review December 16, 2025 12:56
@alejandrobailo alejandrobailo merged commit b9bfdc1 into master Dec 17, 2025
35 checks passed
@alejandrobailo alejandrobailo deleted the PRWLR-8351-lighthouse-prowler-mcp branch December 17, 2025 09:10
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.

5 participants