supermemory
supermemory copied to clipboard
redirect when the extension button is clicked
Extension Manifest Update
- Purpose: Update the Chrome extension manifest to include new functionality.
- Key Changes:
- Added
home.htmlfile to handle extension popup behavior - Expanded
web_accessible_resourcesto includecontent/*files - Added
actionconfiguration to set the default popup for the extension
- Added
- Impact: The extension will now open the SuperMemory.ai home page when the user clicks the extension icon, providing a more seamless user experience.
✨ Generated with love by Kaizen ❤️
Original Description
# Update Chrome Extension Manifest- **Purpose: ** Update the Chrome extension manifest to include a default popup page.
- Key Changes:
- Added a new
home.htmlfile that redirects the user to thehttps://supermemory.ai/homeURL. - Updated the
manifest.jsonfile to include anactionsection with adefault_popupproperty pointing to thehome.htmlfile. - Added the necessary permissions for the extension to access the
webRequest,storage, andcontextMenusAPIs, as well as all URLs.
- Added a new
- **Impact: ** This change will provide users with a default popup page when they click on the extension icon, which will redirect them to the SuperMemory.ai home page. This can improve the user experience and discoverability of the extension's functionality.
✨ Generated with love by Kaizen ❤️
Original Description
A simple redirect when the extension button is clicked to make it easier to access the home page and to ask supermemory
The code for host permissions was the same as before (refer to line 27 of the previous version)
Hi! Thank you so much for this!!!
This works great, but I found an unrelated issue that we can just add to this PR.
Open chrome console on any website ->
#supermemory-extension-host {
position: fixed;
bottom: 0;
right: 0;
z-index: 99999;
}
this resource is blocked because of web_accessible_resource policy
🔍 Code Review Summary
❗ Attention Required: This push has potential issues. 🚨
Overview
- Total Feedbacks: 1 (Critical: 1, Refinements: 0)
- Files Affected: 1
- Code Quality: [█████████████████░░░] 85% (Good)
🚨 Critical Issues
security (1 issues)
1. Excessive permissions in manifest.json
📁 File: apps/extension/manifest.json 🔍 Reasoning: Using '<all_urls>' for 'host_permissions' grants the extension access to all URLs, which can be a security risk if not necessary.
💡 Solution: Restrict 'host_permissions' to only the necessary URLs.
Current Code:
"host_permissions":["<all_urls>"]
Suggested Code:
"host_permissions":["https://specific-domain.com/*"]
✨ Generated with love by Kaizen ❤️
Useful Commands
- Feedback: Reply with
!feedback [your message] - Ask PR: Reply with
!ask-pr [your question] - Review: Reply with
!review - Explain: Reply with
!explain [issue number]for more details on a specific issue - Ignore: Reply with
!ignore [issue number]to mark an issue as false positive - Update Tests: Reply with
!unittestto create a PR with test changes
@Dhravya Are we still going to close this PR?