Skip to content

ci: lint, type-check and test the worker and shared packages #86

Closed

Description

@ajithakdev

Where

.github/workflows/ci.yml:38-42 type-checks only server and extension. eslint.config.js has no block for packages/worker or packages/shared (ESLint reports the worker file as ignored). The worker has no tests.

What happens

Contract drift between the worker and Express (headers, /health, sizes) reaches the default production backend without any check failing.

Fix

Add cd packages/worker && npx tsc --noEmit to CI, extend the TS ESLint block to packages/{worker,shared}/src/**/*.ts, and add a small vitest file that calls the worker's fetch(new Request(...)) directly to check sizes, CORS and Timing-Allow-Origin headers, and 404/415 responses.

Activity

  1. ajithakdev commented on Oct 26, 2026

    @ajithakdev
    OwnerAuthor

    Status on 2026-10-27: #100 adds ESLint and tsc --noEmit for services/worker, packages/shared and packages/reports to CI; worker tests already run. Closes when #100 merges.

  2. added this to the 2027.1.0 milestone on Oct 26, 2026
  3. ajithakdev commented on Oct 30, 2026

    @ajithakdev
    OwnerAuthor

    Fixed in v2027.0.2 (#100): CI now lints and type-checks the worker, @meterx/shared and @meterx/reports.

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

Metadata

Metadata

Assignees

No one assigned

    Labels

    github_actionsPull requests that update GitHub Actions code

    Projects

    No projects

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions