Skip to content

Add feedback form with star rating and Socket security workflow #32

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
ewalid merged 4 commits into from
Mar 3, 2026
Merged

Add feedback form with star rating and Socket security workflow #32

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
2b583ad
feat(feedback): stars rating, optional improvements, optional email
ewalid Mar 2, 2026
7c197a5
chore: add socket security workflow and refine feedback UI
ewalid Mar 3, 2026
439a60c
Merge branch 'master' into feature/feedback-form-stars-and-email
ewalid Mar 3, 2026
97a7206
chore: remove socket security workflow
ewalid Mar 3, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 4 additions & 3 deletions frontend/src/components/features/feedback/Feedback.css
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -186,9 +186,10 @@
}

.feedback-chip.selected {
background: var(--color-accent-100);
border-color: var(--color-accent-500);
color: var(--color-accent-700);
background: #bbf7d0;
border-color: var(--color-accent-600);
color: var(--color-accent-800);
box-shadow: 0 0 0 1px rgba(16, 185, 129, 0.4);
}

:root.dark .feedback-chip.selected {
Expand Down
31 changes: 18 additions & 13 deletions package.json
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,6 +1,5 @@
{
"name": "@ewalid/rosetta-mcp",
"mcpName": "io.github.ewalid/rosetta-mcp",
"version": "0.1.1",
"description": "AI-powered Excel translation MCP server that preserves formatting, formulas, and data integrity",
"keywords": [
Expand All @@ -13,18 +12,23 @@
"localization",
"i18n"
],
"author": "Walid Elmselmi <ewalid@example.com>",
"license": "MIT",
"repository": {
"type": "git",
"url": "https://github.com/ewalid/rosetta.git"
},
"homepage": "https://github.com/ewalid/rosetta",
"bugs": {
"url": "https://github.com/ewalid/rosetta/issues"
},
"repository": {
"type": "git",
"url": "git+https://github.com/ewalid/rosetta.git"
},
"license": "MIT",
"author": "Walid Elmselmi <ewalid@example.com>",
"type": "commonjs",
"main": "index.js",
Copy link
Copy Markdown

@cursor cursor bot Mar 3, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

main field points to non-existent file

Medium Severity

The newly added "main": "index.js" field references a file that doesn't exist in the repository (no index.js is present at the root). Additionally, index.js is not listed in the files array, so it wouldn't be included in the published npm package even if it were created. Any attempt to programmatically require or import this package will fail at runtime with a "module not found" error.

Fix in Cursor Fix in Web

"bin": {
"rosetta-mcp": "./bin/rosetta-mcp.js"
"rosetta-mcp": "bin/rosetta-mcp.js"
},
"directories": {
"test": "tests"
},
"files": [
"bin/",
Expand All @@ -33,16 +37,17 @@
"README.md",
"LICENSE"
],
"scripts": {
"postinstall": "echo 'Please ensure Python 3.11+ and uv are installed. Run: pip install rosetta-xl'"
},
"devDependencies": {},
"peerDependencies": {},
"engines": {
"node": ">=18",
"python": ">=3.11"
},
"publishConfig": {
"access": "public"
},
"scripts": {
"postinstall": "echo 'Please ensure Python 3.11+ and uv are installed. Run: pip install rosetta-xl'"
},
"peerDependencies": {},
"devDependencies": {}
"mcpName": "io.github.ewalid/rosetta-mcp"
}