Skip to content

Toad's lock takes formbridge with shows= (formbridge#19) - #1550

Merged
trissim merged 1 commit into
mainfrom
fix/lock-formbridge-with-shows
Nov 6, 2026
Merged

trissim merged 1 commit into
mainfrom
fix/lock-formbridge-with-shows

Conversation

@trissim

@trissim trissim commented Nov 6, 2026

Copy link
Copy Markdown

Hotfix. Toad main can't import against its own lock: #1543 uses formbridge#19's shows= class keyword, but uv.lock pinned formbridge b67c97e from before #19. A test env built by bin/toad-test-env fails collection with DOMNode.__init_subclass__() got an unexpected keyword argument 'shows' (tests/textual/test_application_actions_in_the_running_app.py), and so would CI now that Actions are back on.

One move, 2 lines: formbridge b67c97e → 0547625 (the #19 merge). My miss: I merged formbridge#19 without relocking Toad in the same breath.

🤖 Generated with Claude Code

https://claude.ai/code/session_01XbFMSqJMarXn2ZE8iAL7Sd

#1543 declares FieldView's views with the shows= class keyword that formbridge#19 added, but
uv.lock still pinned formbridge b67c97e from before #19. Any environment built from Toad's own
lock (bin/toad-test-env, CI) raised at import: DOMNode.__init_subclass__() got an unexpected
keyword argument 'shows'. One move: formbridge b67c97e -> 0547625, the #19 merge.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01XbFMSqJMarXn2ZE8iAL7Sd
Copilot AI balanced review requested due to automatic review settings November 6, 2026 04:59

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot wasn't able to review any files in this pull request. Check if the Files changed in this pull request are included in default exclusions.

@trissim
trissim merged commit aecaeff into main Nov 6, 2026
1 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants