Repository navigation
chore: Update TypeScript/ESLint configs, Check Circular Dependencies, Sort imports, Bump Node Engine, Test Sharding - #226
Merged
Conversation
…circular dependency
…cripts/**/*.ts - Introduced a new script to sort imports in TypeScript files according to project conventions. - Updated ESLint configuration to include TypeScript files in the scripts directory for linting. - Added new commands in package.json for sorting imports and checking import order.
- Added a new tsconfig.json for the scripts directory, extending the main tsconfig.json. - Updated ESLint configuration to include the new scripts tsconfig for linting. - Modified main tsconfig.json to specify types for node and jest, and adjusted included files.
- Added .nvmrc file to specify Node.js version 24. - Updated ESLint configuration to disable specific rules for test files. - Modified package.json to require Node.js version 24 and updated @types/node dependency. - Adjusted CI and publish workflows to use Node.js version 24. - Enhanced tsconfig-paths-bootstrap.mjs for better baseUrl handling. - Refactored logging.ts to improve type safety in write methods.
- Deleted the SUPPORTED_ATTACHMENT_MIMES constant, which included image and PDF mime types. - This change simplifies the attachment handling by relying solely on SUPPORTED_IMAGE_MIMES.
…as error - Changed the severity of the "no-nested-ternary" rule from warning to error in the ESLint configuration, promoting stricter code quality standards.
…ion engine - Updated the `createWorkspacePolicyHook` to introduce a `resolvePolicy` function for clearer policy resolution based on tool names. - Modified the `CloudflareSandboxExecutionEngine` to use a function for checking if the state is closed, preventing potential issues with concurrent mutations. - Simplified promise handling in `LocalCodingTools` by removing unnecessary checks and added a new utility function for formatting fallback grep display.
danny-avila
changed the title
danny-avila
changed the title
- Changed the shebang in `sort-imports.ts` from Bun to Node for better compatibility. - Updated `package.json` scripts to reflect the change in runtime for sorting imports. - Removed Bun types from `tsconfig.json` to streamline type definitions. - Adjusted the import sorting logic to improve handling of file paths and sorting criteria.
- Renamed the existing test job to unit-tests and added a matrix strategy for sharding. - Created a new integration-tests job for change-gated integration testing. - Updated test execution commands to accommodate sharding in unit tests.
danny-avila
changed the title
…ables - Added a new job for summarization tests to the CI workflow. - Updated the unit test command to exclude an additional test file. - Introduced environment variables for various API keys to support the new summarization tests.
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.
No description provided.