Skip to content

Orphaned options/ folder — manifest has no options page, AccountsSection is dead code #59

@Loule95450

Description

@Loule95450

Context

The directory extension/src/options/ looks like an options-page implementation (it has components/, an api.ts, a styles.css). It is not. The components are all re-imported by SettingsScreen.tsx (which renders the in-popup settings flow), and there is no entrypoints/options/ and no options_ui/options_page in wxt.config.ts. The folder naming misleads readers and contributors.

Problem / Observation

Proposed approach

Either:

  1. Promote it: add extension/src/entrypoints/options/ with an index.html + main.tsx, declare options_ui in wxt.config.ts, and route from the popup's chrome.runtime.openOptionsPage() when desktop screen real estate is available. Bitwarden-style.
  2. Demote it: rename src/options/src/settings/, delete api.ts and styles.css (the popup already owns these), and delete AccountsSection.tsx.

Option 2 is the smaller surgical change. Pick (1) only if a real options UI is on the roadmap.

Acceptance criteria

  • Either an options entrypoint exists and is reachable, OR the folder is renamed to reflect its actual role.
  • AccountsSection.tsx is deleted unless wired up.
  • extension/src/options/api.ts is removed or its duplicate is justified in a comment.
  • Imports across SettingsScreen.tsx etc. are updated.
  • npm run typecheck and npm run lint are green.

Metadata

Metadata

Assignees

No one assigned

    Labels

    P2Medium priorityrefactorCode restructuring without behavior changetech-debtAccumulated debt to repay

    Type

    No type
    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions