Skip to content

Conversation

@lwsinclair
Copy link

Hi there,

This pull request shares a security update on basic-memory.

We also have an entry for basic-memory in our directory, MseeP.ai, where we provide regular security and trust updates on your app.

We invite you to add our badge for your MCP server to your README to help your users learn from a third party that provides ongoing validation of basic-memory.

You can easily take control over your listing for free: visit it at https://mseep.ai/app/basicmachines-co-basic-memory.

Yours Sincerely,

Lawrence W. Sinclair
CEO/SkyDeck AI
Founder of MseeP.ai
MCP servers you can trust


MseeP.ai Security Assessment Badge

Here are our latest evaluation results of basic-memory

Security Scan Results

Security Score: 100/100

Risk Level: low

Scan Date: 2025-05-08

Score starts at 100, deducts points for security issues, and adds points for security best practices

Security Findings

Medium Severity Issues

  • semgrep: Use of subprocess with shell=True detected. This can be dangerous if used with untrusted input.
    • Location: installer/installer.py
    • Line: 18

Low Severity Issues

  • semgrep: Use of base64 decoding detected. This might indicate obfuscated code.
  • ... and 2 more low severity issues

This security assessment was conducted by MseeP.ai, an independent security validation service for MCP servers. Visit our website to learn more about our security reviews.

@CLAassistant
Copy link

CLAassistant commented May 14, 2025

CLA assistant check
All committers have signed the CLA.

@phernandez
Copy link
Member

@lwsinclair Hi, I've removed the code referenced in the scan. How do you make this run again?

@lwsinclair
Copy link
Author

Our scans should automatically get run again regularly. If you haven't seen an update after a day or two, please ask again on https://mseep.ai/contact as something has gone wrong.

@phernandez phernandez closed this Jun 18, 2025
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.

3 participants