Skip to content

refactor: separate Spotlight into standalone extension#33

Open
nisargkolhe wants to merge 1 commit intomainfrom
claude/arcify-spotlight-refactor-uoINs
Open

refactor: separate Spotlight into standalone extension#33
nisargkolhe wants to merge 1 commit intomainfrom
claude/arcify-spotlight-refactor-uoINs

Conversation

@nisargkolhe
Copy link
Owner

@nisargkolhe nisargkolhe commented Jan 13, 2026

  • Create new Arcify Spotlight extension with independent codebase
  • Remove all Spotlight code from main Arcify extension
  • Arcify Spotlight has its own settings (accent color, debug logging)
  • Arcify new tab button now opens chrome://newtab/ directly
  • Remove spotlight-related permissions from Arcify manifest
  • Update build configuration to remove spotlight build steps

The new Arcify Spotlight extension:

  • Located at ../arcify-spotlight/
  • Has its own manifest, background script, options page
  • Independent keyboard shortcuts (Alt+L, Alt+T)
  • No shared code with Arcify
  • Removed PINNED_TAB feature (Arcify-specific)

Note

Separates Spotlight from the main extension by fully removing in-repo Spotlight functionality and wiring.

  • Deletes Spotlight content scripts/pages (spotlight/*), new tab override, and related message handling/shortcuts from background.js
  • Manifest cleanup: drops search, topSites, history permissions; removes chrome_url_overrides.newtab, Spotlight commands, and content_scripts; trims web accessible resources
  • UI/options: newTabBtn now opens a blank tab; removes enableSpotlight option and its persistence
  • Keeps auto-archive, tab activity tracking, and URL copy; prunes unused Spotlight logic and messages
  • Build: removes Spotlight overlay/newtab build steps and copies; simplifies asset copying in Vite plugin

Written by Cursor Bugbot for commit bfa93af. This will update automatically on new commits. Configure here.

- Create new Arcify Spotlight extension with independent codebase
- Remove all Spotlight code from main Arcify extension
- Arcify Spotlight has its own settings (accent color, debug logging)
- Arcify new tab button now opens chrome://newtab/ directly
- Remove spotlight-related permissions from Arcify manifest
- Update build configuration to remove spotlight build steps

The new Arcify Spotlight extension:
- Located at ../arcify-spotlight/
- Has its own manifest, background script, options page
- Independent keyboard shortcuts (Alt+L, Alt+T)
- No shared code with Arcify
- Removed PINNED_TAB feature (Arcify-specific)
Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

This PR is being reviewed by Cursor Bugbot

Details

You are on the Bugbot Free tier. On this plan, Bugbot will review limited PRs each billing cycle.

To receive Bugbot reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.

"installation-onboarding.js",
"assets/*",
"_favicon/*",
"spotlight/shared/*"
Copy link

Choose a reason for hiding this comment

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

Onboarding references removed Spotlight commands and settings

High Severity

The toggleSpotlight and toggleSpotlightNewTab commands were removed from the manifest, and the enableSpotlight setting was removed from options, but the installation-onboarding.js and installation-onboarding.html still reference these. The onboarding flow has an entire step dedicated to Spotlight with a toggle button (spotlightToggle), displays keyboard shortcuts for non-existent commands, and saves/loads the now-defunct enableSpotlight setting. Users will see a Spotlight configuration step for a feature that no longer exists, and the displayed shortcuts (Alt+L, Alt+T) won't work.

Additional Locations (1)

Fix in Cursor Fix in Web

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

Comments