Skip to content

Remove duplicated code across the app, libraries and adapters #528

Closed

Description

@vishnuv688

Part of #494.

Problem

A duplicate-block scan over every package's source (8 consecutive normalized lines repeated across or within files) found 33 duplicated blocks. Duplicates have to change together, and CLAUDE.md asks for one source of truth per concept, with logic shared by adapters in core.

Scope

Each finding is either consolidated in the package the import rules allow, or kept with the reason stated in its PR:

  • App styling and tooling: a second full Tailwind build in every component's styles, a Tailwind v3 config v4 never reads, a duplicate --spacing, autoprefixer (which added only -o-object-fit), and repeated component CSS and handlers.
  • Library packages: page-script naming steps, the UiAutomator driver check, the Nightwatch config file names, and repeated conformance allowlist rules.
  • Adapters: screencast lifecycle logging and BiDi sink wiring repeated across adapters (moved to core), plus repeats inside single adapters.

Kept on purpose: the light and dark theme values in colors.css, each component's own Lit context declarations, @wdio/elements' explicit export list (the gate on that published API), and type re-exports under stable local names.

Activity

  1. self-assigned this
    on Nov 2, 2026
  2. vishnuv688 commented on Nov 2, 2026

    @vishnuv688
    MemberAuthor

    Implemented in #520 (app styling), #521 (library packages) and #522 (adapters), stacked in that order on #519. #522 closes this issue when it reaches main.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

enhancementNew feature or request

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions