fix(manifest): remove unused scripting permission [v1.0.2]#63
Conversation
CWS v1.0.1 rejected for declaring scripting permission without using it (Violation reference: Purple Potassium). WXT injects inject.ts via web_accessible_resources, not chrome.scripting, so the permission was dead weight in the manifest. Audited the remaining five permissions (storage, tabs, alarms, unlimitedStorage, sidePanel); all have runtime references in background.ts, useDashboardData.ts, conversation-store.ts, or sidepanel. Version bumped 1.0.1 -> 1.0.2 for CWS resubmission.
|
@DevanshuNEU is attempting to deploy a commit to the Dev's projects Team on Vercel. A member of the Team first needs to authorize it. |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (2)
📝 WalkthroughWalkthroughThis PR updates extension metadata and manifest configuration: the version is incremented to 1.0.2 in ChangesRelease Configuration Updates
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Warning There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure. 🔧 ESLint
ESLint skipped: no ESLint configuration detected in root package.json. To enable, add Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Why
CWS rejected the v1.0.1 submission (2026-05-11) with violation code Purple Potassium: the manifest declared the
scriptingpermission without any runtime use. WXT injectsinject.tsviaweb_accessible_resources, notchrome.scripting, so the permission was dead weight.What ships
scriptingfrom the permissions array inwxt.config.tspackage.jsonversion 1.0.1 -> 1.0.2Two lines across two files. No behavior change.
Permission audit (remaining 5)
Verified each has at least one runtime reference:
storagebackground.ts,useDashboardData.ts,conversation-store.ts, sidepaneltabsbackground.ts(tab activation + URL inspection for claude.ai routing)alarmsbackground.ts(periodic conversation cleanup)unlimitedStorageconversation-store.ts(multi-account history retention)sidePanelbackground.ts(chrome.sidePanel.open), sidepanel entrypointTest plan
bun run zipand resubmit to CWS Developer DashboardRelated
Summary by CodeRabbit