Use try/catch to wrap invoking of action in actionFromPath #51
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.
Full context:
@convex-dev/workflowto not patch non-deterministic globals so I could use workflows withconvex-teststep.runActionand other workflow methods worked fine if they did not throwError when running scheduled function complete:complete Error: Could not find module for: "complete"that made me believe the test instance was not correctly catching the error and so the execution runtime was getting messed upI don't think this is ideal to merge as is but wanted to raise the bug and potential fix path. Ideally we'd apply a similar try/catch to
mutationFromPathas well. What is the general policy for logging in this situation since this creates quite cluttered logs with retries?By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.