Node errors reach the editor as flattened strings instead of structured NodeError

Open
#80 0 comments 0 reactions 1 assignee View on GitHub

Maintainers usually reply within 1 day

@shhlife is already working on this.

Since Oct 7, 2026.

Assessment

This issue has not been assessed yet.

Description

Problem

This library's nodes fail with plain exceptions, so their errors reach the editor as one flattened string. Messages often repeat the node's name (f"{self.name}: …") or print a provider response into the text, and the editor has to parse them back apart.

griptape-ai/griptape-nodes-engine#5738 added NodeError (griptape_nodes.exe_types.core_types) and a structured error field on NodeErrorEvent. Nodes that raise it can send their own message plus labelled fields, the provider's response and links, which the editor shows directly.

What's needed

  • Raise NodeError where a node fails on a provider or user-fixable problem, or a clear plain exception otherwise.
  • Drop the "{node_name}: " prefix from error messages; the engine already sends the node name.
  • Move provider responses and IDs (generation ID, request ID) out of the message text into response / fields.
  • Raise the library's minimum engine version once a stable engine release includes NodeError (v0.103.2, the latest stable, does not).

Origin

Reported by Jason Schleifer, as part of moving every Griptape node library to NodeError.

Dominant language
Python
Stars
1
Forks
0
Avg merge
2d 17h
Merged PRs (30d)
24

Getting set up

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from griptape-ai/griptape-nodes-library-diffusers

All issues in griptape-ai/griptape-nodes-library-diffusers

Similar issues

More Python issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.