Skip to content

Fix stale Incipit completion UI - #20

Merged
readwrightexecute merged 2 commits into
masterfrom
fix/incipit-status-actions
Oct 14, 2026
Merged

readwrightexecute merged 2 commits into
masterfrom
fix/incipit-status-actions

Conversation

@readwrightexecute

@readwrightexecute readwrightexecute commented Oct 14, 2026 •

Copy link
Copy Markdown
Owner

Summary

  • Clear the cold-start status bar from polling fallback responses when clarify questions or section cards are ready
  • Refresh the Step 3 action bar out-of-band from the final section-card update so Party review / Finish appear even if the job_done SSE event is missed
  • Add route-level regression coverage for the missed-SSE fallback behavior

Verification

  • .venv/bin/python -m pytest: 247 passed, 1 xfailed
  • ReadLints reported no diagnostics for the edited files

Note

Low Risk
UI-only HTMX template and render-context changes with regression tests; no auth, data, or API behavior changes.

Overview
Fixes stale Incipit completion UI when the browser misses SSE events by letting HTMX polling fallbacks push targeted out-of-band updates.

On the clarify step, questions_partial now passes status_oob=True, and question_list.html separates status_oob from questions_oob. When questions are ready, polling can clear #status-bar via OOB without marking #questions as OOB—so the primary poll swap still works and users don’t keep seeing the cold-start model message.

On section drafting, section_partial enables oob=True, and when every section is terminal the section-card partial also OOB-clears #status-bar and refreshes #step3-actions (Party review / Finish) if job_done SSE was missed.

Route-level tests in test_routes.py lock in both fallback behaviors.

Reviewed by Cursor Bugbot for commit ee63b4e. Bugbot is set up for automated code reviews on this repo. Configure here.

@cursor cursor Bot 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.

Cursor Bugbot has reviewed your changes using high effort and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 212f640. Configure here.

Comment thread webapp/app/main.py Outdated
@readwrightexecute
readwrightexecute marked this pull request as ready for review October 14, 2026 16:35
@readwrightexecute
readwrightexecute merged commit 38a62a5 into master Oct 14, 2026
5 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.

1 participant