-
Notifications
You must be signed in to change notification settings - Fork 21
fix: add Jest 30 support and fix time limit in loop-runner #1318
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Open
mohammedahmed18
wants to merge
43
commits into
main
Choose a base branch
from
fix/js-jest30-loop-runner
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
43 commits
Select commit
Hold shift + click to select a range
04a87cf
fix: add Jest 30 support, fix time limit, and fix async function looping
mohammedahmed18 4c61d08
Merge branch 'main' into fix/js-jest30-loop-runner
mohammedahmed18 a3764f1
Merge branch 'main' of github.com:codeflash-ai/codeflash into fix/js-…
mohammedahmed18 4157534
fix: use getter functions for env var constants in capture.js
mohammedahmed18 017bde1
refactor: improve code quality and documentation in loop-runner and c…
mohammedahmed18 71b38d5
fix: Parse timing markers from console output for JavaScript benchmar…
mohammedahmed18 7273f27
chore: trigger CI workflows
mohammedahmed18 b83e516
fix: use lazy % formatting for logger.debug to pass ruff G004
mohammedahmed18 0592d92
Merge branch 'main' into fix/js-jest30-loop-runner
mohammedahmed18 b4d0b0f
fix: support monorepo hoisted dependencies in JS requirements check
mohammedahmed18 3b56d24
debug: add extensive Jest execution logging for troubleshooting
mohammedahmed18 9cd5d5a
fix: calculate correct import paths for JavaScript tests in temp dire…
mohammedahmed18 0a8d120
fix: preserve ./ prefix in JS import paths and fix TestType enum
mohammedahmed18 6febd69
debug: add extensive performance test debugging
mohammedahmed18 6c74adc
fix: disable custom loop-runner to enable basic performance testing
mohammedahmed18 202bdc4
Merge branch 'main' into fix/js-jest30-loop-runner
mohammedahmed18 bab3bd4
style: auto-fix linting issues
github-actions[bot] 535c640
fix: resolve all linting issues from ruff and mypy
mohammedahmed18 d0b859a
Optimize PrComment.to_json
codeflash-ai[bot] c151b6c
Merge pull request #1383 from codeflash-ai/codeflash/optimize-pr1318-…
claude[bot] ae31ca7
Fix JavaScript test generation and benchmarking
mohammedahmed18 9bb05f6
style: auto-fix linting issues
github-actions[bot] 8fcb8cc
cleanup
mohammedahmed18 67ea0c9
Merge branch 'fix/js-jest30-loop-runner' of github.com:codeflash-ai/c…
mohammedahmed18 a6b9364
fix: include same-class helper methods inside class wrapper for TypeS…
mohammedahmed18 f800ae3
Merge branch 'main' into fix/js-jest30-loop-runner
mohammedahmed18 b65711d
fix: resolve merge conflict in function_optimizer.py
github-actions[bot] 4545b8c
fix: add export keywords to remaining JavaScript/TypeScript tests
mohammedahmed18 183d800
fix: detect CommonJS exports (module.exports) for function discovery
mohammedahmed18 6c23255
version upgrade for cf package
mohammedahmed18 ce13a6d
Merge branch 'main' into fix/js-jest30-loop-runner
Saga4 599a0e3
fix: resolve merge conflicts in verifier.py
github-actions[bot] dcd9e2a
some fixes for test runner and instrumentation
mohammedahmed18 1b8f701
Merge branch 'fix/js-jest30-loop-runner' of github.com:codeflash-ai/c…
mohammedahmed18 175226b
fix: correct loop index calculation in JS performance benchmarking
mohammedahmed18 536c1d0
remove debug statements
mohammedahmed18 e9b7154
Merge branch 'main' of github.com:codeflash-ai/codeflash into fix/js-…
mohammedahmed18 e07fd1d
fix tests
mohammedahmed18 4c9f4ef
Optimize StandaloneCallTransformer._parse_bracket_standalone_call
codeflash-ai[bot] 6b77be5
ignore calls inside string litrals for instrumentation and fix e2e test
mohammedahmed18 f5dd109
Merge pull request #1465 from codeflash-ai/codeflash/optimize-pr1318-…
mohammedahmed18 9937fe0
fixes for unit tests
mohammedahmed18 4dedb9f
Merge branch 'fix/js-jest30-loop-runner' of github.com:codeflash-ai/c…
mohammedahmed18 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Oops, something went wrong.
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
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
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
Oops, something went wrong.
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Potential breaking API change: throughput fields changed from
strtointPreviously these were serialized as
str(self.original_async_throughput)andstr(self.best_async_throughput). Now they're passed as rawintvalues. This JSON is sent to the CodeFlash API server (viacfapi.pywherepr_comment.to_json()is called). If the server expects string values for these fields, this will cause a type mismatch or API error.Verify the server-side API accepts
intfor these fields before merging.