Tuple unpacking mismatch in app.py line 119 #2

New Issue

llabeyrie · 2026-03-19T17:29:52Z

llabeyrie commented

2026-03-19 17:29:52 +00:00

Description

run_prompt_pipeline() returns a 3-tuple (image_path, full_output, cmd) (line 56), but the call site on line 119 only unpacks 2 values:

# Line 56 — returns 3 values:
return image_path, full_output.strip(), cmd

# Line 119 — unpacks only 2:
image, logs = run_prompt_pipeline(raw_text)

This will raise a ValueError: too many values to unpack at runtime and crash the app every time the generate button is clicked.

Fix

image, logs, _cmd = run_prompt_pipeline(raw_text)

Or change the function to return only 2 values if cmd is not needed externally.

## Description `run_prompt_pipeline()` returns a 3-tuple `(image_path, full_output, cmd)` (line 56), but the call site on line 119 only unpacks 2 values: ```python # Line 56 — returns 3 values: return image_path, full_output.strip(), cmd # Line 119 — unpacks only 2: image, logs = run_prompt_pipeline(raw_text) ``` This will raise a `ValueError: too many values to unpack` at runtime and **crash the app** every time the generate button is clicked. ### Fix ```python image, logs, _cmd = run_prompt_pipeline(raw_text) ``` Or change the function to return only 2 values if `cmd` is not needed externally.

llabeyrie added the bug priority: critical labels 2026-03-19 17:31:38 +00:00

llabeyrie referenced this issue from a commit

2026-03-19 17:42:37 +00:00

fix: broken subprocess command and tuple unpacking in app.py

llabeyrie referenced a pull request that will close this issue

2026-03-19 17:42:49 +00:00

fix: broken subprocess command and tuple unpacking in app.py #19

llabeyrie closed this issue

2026-03-19 18:16:16 +00:00

Sign in to join this conversation.