Uh oh!
There was an error while loading. Please reload this page.
fix: AttributeError in EnrichPotentialLevers error handler (lever.id → lever.lever_id) - #130
Merged
neoneye merged 1 commit intoMar 4, 2026
Conversation
…→ lever.lever_id) The error logging on line 155 referenced 'lever.id' but InputLever defines the field as 'lever_id'. This caused an AttributeError that crashed the error handler itself, masking the actual LLM failure and preventing proper fallback to the next model in the LLMExecutor chain. Found during local model testing with Qwen 3.5-35B + GLM 4.6v Flash on Mac Mini (M4 Pro, 64GB). The bug exists in both VoynichLabs/PlanExe2026 and upstream PlanExeOrg/PlanExe.
Uh oh!
There was an error while loading. Please reload this page.
neoneye
commented
Mar 4, 2026
Member
Excellent PR. Thank you. |
82deutschmark added a commit
to VoynichLabs/PlanExe2026
that referenced
this pull request
Mar 6, 2026
- CreateScheduleTask: handle both 'server_iso_utc' (web) and 'timestamp' (CLI) keys in start_time.json with clear error if neither present - EnrichLeversTask: fix lever.id -> lever.lever_id in error handler (same class of bug as PR PlanExeOrg#130) - Add CLI pipeline guide (docs/cli_pipeline_guide.md) - Add Gemini 3.1 Flash Lite Preview config for OpenRouter
82deutschmark added a commit
to VoynichLabs/PlanExe2026
that referenced
this pull request
Mar 6, 2026
- CreateScheduleTask: handle both 'server_iso_utc' (web) and 'timestamp' (CLI) keys in start_time.json with clear error if neither present - EnrichLeversTask: fix lever.id -> lever.lever_id in error handler (same class of bug as PR PlanExeOrg#130) - Add CLI pipeline guide (docs/cli_pipeline_guide.md) - Add Gemini 3.1 Flash Lite Preview config for OpenRouter
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Bug
enrich_potential_levers.pyline 155 referenceslever.idin the error handler, but theInputLeverPydantic model defines the field aslever_id(line 34). This causes anAttributeErrorthat crashes the error handler itself, which:LLMExecutorchainFix
One-line change:
lever.id→lever.lever_idon line 155.How Found
Discovered during local model testing (Qwen 3.5-35B + GLM 4.6v Flash on Mac Mini M4 Pro, 64GB). When Qwen failed to produce valid JSON for EnrichLeversTask, the error handler crashed with
AttributeError: 'InputLever' object has no attribute 'id'before the fallback model (GLM) could be properly evaluated.Testing