Merged
Conversation
- Add @biomejs/biome as dev dependency with biome.json config - Add lint, format, and fix npm scripts - Apply Biome formatting across all source and test files - Update AGENTS.md with linting guidelines and commands - Sort devDependencies alphabetically in package.json Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
CI uses bun install --frozen-lockfile which requires an up-to-date bun.lockb. The npm package-lock.json was committed by mistake. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
- Fix fetch mock assignments with `as unknown as typeof fetch` for Bun's preconnect property requirement - Narrow FetchResult discriminated union before accessing .data/.error - Add missing properties to OpenCodeModelEntry type (max_completion_tokens, supported_parameters, is_moderated, default_parameters) - Fix createToastBody to accept input type (variant optional with default) - Import AddressInfo from node:net instead of node:http - Fix non-null assertions for array access and possibly-undefined values - Cast ZodType to any for .shape access in schema tests - Fix plugin test to match actual plugin behavior (no return value) Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
The 'normal' state checked `next !== undefined` before advancing `i`, causing an infinite loop when processing the final character of input (where next is always undefined). Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
- Add GritQL plugin to detect dynamic import() expressions (no lazy imports) - Enable noCommonJs rule at error level to enforce ESM imports over require() These rules ensure consistent use of static ESM imports for better tree-shaking and static analysis.
Tests that write to the real config now reset it to empty models in beforeEach, ensuring updateModels starts from a clean state regardless of what previous tests may have written. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
The project uses bun.lockb (not package-lock.json), so npm ci fails. npm install works without a lockfile. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
npm citonpm installin the publish workflow since the project usesbun.lockb(notpackage-lock.json)Test plan
🤖 Generated with Claude Code