Skip to content

Migrate event bridge to tsgo and strict Oxlint - #4

Merged
jackblanc merged 2 commits into
masterfrom
agent/code-369-tsgo-oxlint
Aug 9, 2026
Merged

Migrate event bridge to tsgo and strict Oxlint#4
jackblanc merged 2 commits into
masterfrom
agent/code-369-tsgo-oxlint

Conversation

@jackblanc

Copy link
Copy Markdown
Owner

Summary

  • replace the legacy Effect TypeScript language-service setup with the Effect tsgo-patched Oxlint path
  • enable the Effect recommended/style presets, type-aware linting, type checking, warning denial, and strict TypeScript/import/Unicorn/Vitest rules without suppressions
  • refactor source and tests to satisfy the strict rules, including Effect clocks/configuration, typed error channels, service contexts, and it.effect
  • update compatible tooling and runtime packages and document the local setup

Why

CODE-369 requires a single Effect diagnostic path through Oxlint instead of duplicating diagnostics between TypeScript and the old Effect plugin. This also aligns local hooks and CI linting with the same strict configuration.

Impact

Developers now receive type-aware TypeScript and Effect diagnostics from pnpm run lint. Warnings fail the lint gate, and the effective config contains no disabled rules or suppression overrides.

Validation

  • pnpm run check
  • pre-commit: typecheck, lint, format, Knip, and unit tests
  • 9 unit test files / 46 tests passed

The deploy-backed e2e test was not run locally.

…-oxlint

# Conflicts:
#	package.json
#	pnpm-lock.yaml
#	pnpm-workspace.yaml
@jackblanc
jackblanc marked this pull request as ready for review August 9, 2026 21:23
@jackblanc
jackblanc merged commit ce59203 into master Aug 9, 2026
7 checks passed
@jackblanc
jackblanc deleted the agent/code-369-tsgo-oxlint branch August 9, 2026 21:23
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.

1 participant