Skip to content

fix: clarify error message for JavaScript evaluation failure#88

Closed
Sana9058 wants to merge 2 commits into
accordproject:mainfrom
Sana9058:fix/improve-evaluate-error-message
Closed

fix: clarify error message for JavaScript evaluation failure#88
Sana9058 wants to merge 2 commits into
accordproject:mainfrom
Sana9058:fix/improve-evaluate-error-message

Conversation

@Sana9058
Copy link
Copy Markdown
Contributor

Summary

Improves the error message thrown in evaluateJavaScript when required
inputs are missing.

What changed

  • Replaced a generic error message that printed raw fn and data
  • Added a clearer, user-friendly message explaining that template data
    or function source is missing

Motivation

The previous error message could expose large objects in logs and was
hard to understand during debugging. This change makes the failure
reason explicit without changing any behavior.

Notes

  • No functional or logic changes
  • Error condition remains the same

apple added 2 commits February 7, 2026 10:46
Signed-off-by: apple <apple@apples-MacBook-Pro.local>
Signed-off-by: apple <apple@apples-MacBook-Pro.local>
@Sana9058
Copy link
Copy Markdown
Contributor Author

Thanks for taking a look!

This change only improves the clarity of the error message when JavaScript
evaluation is attempted without required inputs. No behavior or logic was
changed.

Happy to adjust or split commits if you’d prefer a different structure.

@mttrbrts
Copy link
Copy Markdown
Member

Unfortunately we don't accept documentation changes from first contributors. Thanks for your contribution

@mttrbrts mttrbrts closed this Feb 13, 2026
@Sana9058 Sana9058 deleted the fix/improve-evaluate-error-message branch February 15, 2026 05:10
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.

2 participants