Skip to content

fix: Remove npm latest install, add PR checks, improve error handling and formatting - #26

Merged
glesage merged 3 commits into
mainfrom
fix/remove-npm-latest-custom-install
Jul 10, 2026
Merged

fix: Remove npm latest install, add PR checks, improve error handling and formatting#26
glesage merged 3 commits into
mainfrom
fix/remove-npm-latest-custom-install

Conversation

@glesage

@glesage glesage commented Jul 10, 2026

Copy link
Copy Markdown
Owner

Summary

This PR cleans up the CI pipelines, fixes formatting, improves error handling with proper error chaining, and updates the TypeScript target to ES2022.

Changes

CI & Tooling

  • New PR checks workflow (.github/workflows/pr-checks.yml): Adds automated CI that runs lint, format check, build, example build/test, and unit tests on every PR to catch issues early.
  • Remove npm install -g npm@latest (.github/workflows/publish-npm.yml): Eliminates an unnecessary global npm upgrade step from the publish workflow.
  • Add format:check script (package.json): Adds a bun run format:check script for CI to verify formatting without writing changes.

Error handling

  • Error chaining with .cause across bedrock-client.ts, config.ts, and validators.ts: Wraps caught errors with the original error attached via Error.cause, enabling better debugging by preserving the full error chain.

Bug fixes

  • Fix OpenRouter SDK API (openrouter-client.ts): Wraps chat parameters in a chatRequest object to match the updated SDK interface.

Formatting

  • Fix self-closing textarea formatting (PromptModal.svelte): Closes textarea tags on the same line to satisfy prettier formatting rules.
  • Update TypeScript target (tsconfig.json): Bumps target and lib from ES2020 to ES2022, which is required for native Error.cause support.

@glesage
glesage marked this pull request as ready for review July 10, 2026 01:25
@glesage
glesage merged commit 3997402 into main Jul 10, 2026
1 check passed
@glesage
glesage deleted the fix/remove-npm-latest-custom-install branch July 10, 2026 01:48
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