Skip to content

Conversation

@3rd-Son
Copy link
Contributor

@3rd-Son 3rd-Son commented Dec 2, 2025

πŸ”— Linked Issue

Closes #

βœ… Type of Change

  • ✨ New Project/Feature
  • 🐞 Bug Fix
  • πŸ“š Documentation Update
  • πŸ”¨ Refactor or Other

πŸ“ Summary

πŸ“– README Checklist

  • I have created a README.md file for my project.
  • My README.md follows the official .github/README_TEMPLATE.md.
  • I have included clear installation and usage instructions in my README.md.
  • I have added a GIF or screenshot to the assets folder and included it in my README.md.

βœ”οΈ Contributor Checklist

  • I have read the CONTRIBUTING.md document.
  • My code follows the project's coding standards.
  • I have placed my project in the correct directory (e.g., advance_ai_agents, rag_apps).
  • I have included a requirements.txt or pyproject.toml for dependencies.
  • I have added a .env.example file if environment variables are needed and ensured no secrets are committed.
  • My pull request is focused on a single project or change.

πŸ’¬ Additional Comments


EntelligenceAI PR Summary

This PR adds Memori API key configuration support to the AI Consultant Agent Streamlit UI.

  • Added password-protected text input field for Memori API key between OpenAI and ExaAI key inputs
  • Configured default value from environment variables
  • Updated 'Save API Keys' button logic to persist Memori API key to session environment
  • Included Memori API key in validation check for successful key storage
  • Enables access to Memori Advanced Augmentation features and higher API quotas

@entelligence-ai-pr-reviews
Copy link

Entelligence AI Vulnerability Scanner

Status: No security vulnerabilities found

Your code passed our comprehensive security analysis.

Analyzed 1 files in total

@entelligence-ai-pr-reviews
Copy link

Review Summary

🏷️ Draft Comments (1)

Skipped posting 1 draft comments that were valid but scored below your review threshold (>=13/15). Feel free to update them here.

memory_agents/ai_consultant_agent/app.py (1)

138-164: MEMORI_API_KEY is collected and saved to os.environ, but is never actually used to initialize the Memori client, so providing it has no effect and advanced Memori features will not work.

πŸ“Š Impact Scores:

  • Production Impact: 3/5
  • Fix Specificity: 5/5
  • Urgency Impact: 3/5
  • Total Score: 11/15

πŸ€– AI Agent Prompt (Copy & Paste Ready):

In memory_agents/ai_consultant_agent/app.py, lines 138-164: The code collects and saves a MEMORI_API_KEY from the user, but never uses it to initialize the Memori client. Update the Memori initialization so that if MEMORI_API_KEY is set, it is passed to the Memori constructor (e.g., Memori(conn=SessionLocal, api_key=memori_key)). Otherwise, fall back to the current behavior. Ensure the fix preserves all existing logic and indentation.

@entelligence-ai-pr-reviews
Copy link

Walkthrough

This PR adds support for Memori API key configuration in the AI Consultant Agent's Streamlit interface. The update introduces a new password-protected input field that allows users to enter and save their Memori API key, which is positioned between the existing OpenAI and ExaAI key input fields. The key is loaded from environment variables as a default value and persisted to the session environment when saved. The validation logic for the 'Save API Keys' button has been updated to include the Memori API key in the success check. This enhancement enables users to access Memori Advanced Augmentation features and benefit from higher API quotas.

Changes

File(s) Summary
memory_agents/ai_consultant_agent/app.py Added optional Memori API key configuration with password-protected text input field, default value from environment variables, session persistence logic, and updated validation check in the 'Save API Keys' button handler.

Sequence Diagram

This diagram shows the interactions between components:

sequenceDiagram
    actor User
    participant UI as Streamlit UI
    participant Env as Environment Variables

    User->>UI: Enter OpenAI API Key
    Note over UI: Password field with help text
    
    User->>UI: Enter Memori API Key (optional)
    Note over UI: NEW: Optional field for<br/>Advanced Augmentation
    
    User->>UI: Enter ExaAI API Key
    
    User->>UI: Click "Save API Keys"
    
    alt OpenAI key provided
        UI->>Env: Set OPENAI_API_KEY
    end
    
    alt Memori key provided
        UI->>Env: Set MEMORI_API_KEY
        Note over Env: NEW: Optional Memori key storage
    end
    
    alt Exa key provided
        UI->>Env: Set EXA_API_KEY
    end
    
    alt At least one key provided
        UI-->>User: Display success message
        Note over User,UI: "βœ… API keys saved for this session"
    else No keys provided
        UI-->>User: Display warning
        Note over User,UI: "Please enter at least one API key"
    end
Loading

πŸ”— Cross-Repository Impact Analysis

Enable automatic detection of breaking changes across your dependent repositories. β†’ Set up now

Learn more about Cross-Repository Analysis

What It Does

  • Automatically identifies repositories that depend on this code
  • Analyzes potential breaking changes across your entire codebase
  • Provides risk assessment before merging to prevent cross-repo issues

How to Enable

  1. Visit Settings β†’ Code Management
  2. Configure repository dependencies
  3. Future PRs will automatically include cross-repo impact analysis!

Benefits

  • πŸ›‘οΈ Prevent breaking changes across repositories
  • πŸ” Catch integration issues before they reach production
  • πŸ“Š Better visibility into your multi-repo architecture

▢️ ⚑ AI Code Reviews for VS Code, Cursor, Windsurf
Install the extension

Note for Windsurf Please change the default marketplace provider to the following in the windsurf settings:

Marketplace Extension Gallery Service URL: https://marketplace.visualstudio.com/_apis/public/gallery

Marketplace Gallery Item URL: https://marketplace.visualstudio.com/items

Entelligence.ai can learn from your feedback. Simply add πŸ‘ / πŸ‘Ž emojis to teach it your preferences. More shortcuts below

Emoji Descriptions:

  • ⚠️ Potential Issue - May require further investigation.
  • πŸ”’ Security Vulnerability - Fix to ensure system safety.
  • πŸ’» Code Improvement - Suggestions to enhance code quality.
  • πŸ”¨ Refactor Suggestion - Recommendations for restructuring code.
  • ℹ️ Others - General comments and information.

Interact with the Bot:

  • Send a message or request using the format:
    @entelligenceai + *your message*
Example: @entelligenceai Can you suggest improvements for this code?
  • Help the Bot learn by providing feedback on its responses.
    @entelligenceai + *feedback*
Example: @entelligenceai Do not comment on `save_auth` function !

Also you can trigger various commands with the bot by doing
@entelligenceai command

The current supported commands are

  1. config - shows the current config
  2. retrigger_review - retriggers the review

More commands to be added soon.

@Arindam200 Arindam200 merged commit 50b9a46 into Arindam200:main Dec 3, 2025
1 of 4 checks 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