[PR #303] [MERGED] Add resource validation to workflow.validate() #306

Closed
opened 2026-02-16 02:17:11 -05:00 by yindo · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/run-llama/workflows-py/pull/303
Author: @adrianlyjak
Created: 1/23/2026
Status: Merged
Merged: 1/23/2026
Merged by: @adrianlyjak

Base: mainHead: claude/add-resource-validation-jSZFS


📝 Commits (3)

  • 3ca53e5 feat: Add resource and resource config validation to workflow validate()
  • 14b91c1 Create smooth-donkeys-run.md
  • 98af9e2 Update test_resources.py

📊 Changes

3 files changed (+381 additions, -6 deletions)

View changed files

.changeset/smooth-donkeys-run.md (+5 -0)
📝 packages/llama-index-workflows/src/workflows/workflow.py (+104 -6)
📝 packages/llama-index-workflows/tests/test_resources.py (+272 -0)

📄 Description

Workflow validate can pass on something that will then crash later on from failing to load e.g. Resources/ResourceConfigs. By default, validation now loads the configs and validates them as well, and you can opt in to fully resolving Resources (e.g. clients). That one is disabled by default, as environment variables may not be set up.


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/run-llama/workflows-py/pull/303 **Author:** [@adrianlyjak](https://github.com/adrianlyjak) **Created:** 1/23/2026 **Status:** ✅ Merged **Merged:** 1/23/2026 **Merged by:** [@adrianlyjak](https://github.com/adrianlyjak) **Base:** `main` ← **Head:** `claude/add-resource-validation-jSZFS` --- ### 📝 Commits (3) - [`3ca53e5`](https://github.com/run-llama/workflows-py/commit/3ca53e5f527b82e224a61d74b5c6de6360464290) feat: Add resource and resource config validation to workflow validate() - [`14b91c1`](https://github.com/run-llama/workflows-py/commit/14b91c1ac41a539bdbe6e012eaaeeba76613adc9) Create smooth-donkeys-run.md - [`98af9e2`](https://github.com/run-llama/workflows-py/commit/98af9e2951d5b114c779f3dcb4e3637bc5dcedb6) Update test_resources.py ### 📊 Changes **3 files changed** (+381 additions, -6 deletions) <details> <summary>View changed files</summary> ➕ `.changeset/smooth-donkeys-run.md` (+5 -0) 📝 `packages/llama-index-workflows/src/workflows/workflow.py` (+104 -6) 📝 `packages/llama-index-workflows/tests/test_resources.py` (+272 -0) </details> ### 📄 Description Workflow validate can pass on something that will then crash later on from failing to load e.g. Resources/ResourceConfigs. By default, validation now loads the configs and validates them as well, and you can opt in to fully resolving Resources (e.g. clients). That one is disabled by default, as environment variables may not be set up. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
yindo added the pull-request label 2026-02-16 02:17:11 -05:00
yindo closed this issue 2026-02-16 02:17:11 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: run-llama/workflows-py#306