run-llama/workflows-py
1
0
Fork 0
mirror of https://github.com/run-llama/workflows-py.git synced 2026-08-24 20:01:34 -04:00
Code Issues 18 Packages Projects Releases 10 Wiki Activity

[PR #247] [CLOSED] Handler id quote issue #255

New Issue
Closed
opened 2026-02-16 02:17:03 -05:00 by yindo · 0 comments
yindo commented 2026-02-16 02:17:03 -05:00
Owner
Copy Link
Copy Source

📋 Pull Request Information

Original PR: https://github.com/run-llama/workflows-py/pull/247
Author: @adrianlyjak
Created: 12/16/2025
Status: ❌ Closed

Base: main ← Head: adrian/handler-id-quote-issue-53e3

📝 Commits (1)

  • ec09465 Test: Ensure handler_id is alphanumeric and not quoted

📊 Changes

1 file changed (+61 additions, -2 deletions)

View changed files

📝 packages/llama-index-workflows/tests/server/test_server_endpoints.py (+61 -2)

📄 Description

Add regression tests to ensure handler_id is correctly serialized without trailing quotes.

This addresses a reported issue where handler_id was observed to have a trailing double quote, leading to URL-encoding problems when used in subsequent requests. The tests verify that handler_id is alphanumeric, has the correct length, and is correctly formatted in the raw JSON response.

Slack Thread

Open in Cursor Open in Web

🔄 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/247 **Author:** [@adrianlyjak](https://github.com/adrianlyjak) **Created:** 12/16/2025 **Status:** ❌ Closed **Base:** `main` ← **Head:** `adrian/handler-id-quote-issue-53e3` --- ### 📝 Commits (1) - [`ec09465`](https://github.com/run-llama/workflows-py/commit/ec09465ad98296090277a32ad46d36b61f180976) Test: Ensure handler_id is alphanumeric and not quoted ### 📊 Changes **1 file changed** (+61 additions, -2 deletions) <details> <summary>View changed files</summary> 📝 `packages/llama-index-workflows/tests/server/test_server_endpoints.py` (+61 -2) </details> ### 📄 Description Add regression tests to ensure `handler_id` is correctly serialized without trailing quotes. This addresses a reported issue where `handler_id` was observed to have a trailing double quote, leading to URL-encoding problems when used in subsequent requests. The tests verify that `handler_id` is alphanumeric, has the correct length, and is correctly formatted in the raw JSON response. --- [Slack Thread](https://llama-index.slack.com/archives/C078MCV1VF1/p1765893154470029?thread_ts=1765893154.470029&cid=C078MCV1VF1) <a href="https://cursor.com/background-agent?bcId=bc-398e3373-19fb-4824-8036-d798cee508c4"><picture><source media="(prefers-color-scheme: dark)" srcset="https://cursor.com/open-in-cursor-dark.svg"><source media="(prefers-color-scheme: light)" srcset="https://cursor.com/open-in-cursor-light.svg"><img alt="Open in Cursor" src="https://cursor.com/open-in-cursor.svg"></picture></a>&nbsp;<a href="https://cursor.com/agents?id=bc-398e3373-19fb-4824-8036-d798cee508c4"><picture><source media="(prefers-color-scheme: dark)" srcset="https://cursor.com/open-in-web-dark.svg"><source media="(prefers-color-scheme: light)" srcset="https://cursor.com/open-in-web-light.svg"><img alt="Open in Web" src="https://cursor.com/open-in-web.svg"></picture></a> --- <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:03 -05:00
yindo closed this issue 2026-02-16 02:17:03 -05:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
bug
enhancement
enhancement
good first issue
pull-request
Mirrored from GitHub Pull Request
 question
No Label pull-request
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
yindo
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: run-llama/workflows-py#255
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?