Skip to content

feat(web): make pending questions easy to find without moving threads - #9810

Open
maria-rcks wants to merge 5 commits into
pingdotgg:mainfrom
maria-rcks:t3code/pulse-pending-question-threads
Open

feat(web): make pending questions easy to find without moving threads#9810
maria-rcks wants to merge 5 commits into
pingdotgg:mainfrom
maria-rcks:t3code/pulse-pending-question-threads

Conversation

@maria-rcks

@maria-rcks maria-rcks commented Sep 4, 2026

Copy link
Copy Markdown
Collaborator

Pending questions now have a fixed Needs input count that opens each waiting thread in turn, while thread order stays unchanged and a static border marks pending rows. Submitted answers appear beside the activity label, expand into the original question details, and composer questions render Markdown links.

Verified next-question navigation in the real app with 80 demo threads and 8 pending questions, no border animation, web typecheck, targeted lint, and the existing 318-test pass from the initial implementation.

Original sidebar (sampled capture):

Original pending question sidebar

Previous animated proposal:

Before: directional arrow in sidebar

Final static navigation, same demo threads:

After: fixed Needs input control and static border

Implemented with gpt-5.6-sol in the Codex harness.


Note

Low Risk
Mostly sidebar and timeline presentation; work-log derivation changes how resolved user-input rows read in the activity feed but does not touch auth or persistence.

Overview
Adds a fixed Needs input · N control at the top of the sidebar (via SidebarQuestionIndicators) that cycles through non-archived threads with hasPendingUserInput, calling navigateToThread without reordering the list. Pending rows are highlighted with data-pending-question, the sidebar-question-pending indigo border overlay, and the same styling in both Sidebar and LegacySidebar.

Resolved user-input activities in deriveWorkLogEntries now populate optional userInputSummary and richer detail (questions, options, answers); the message timeline shows the summary next to the work-log label and includes it in accessible text. Pending composer questions render through ChatMarkdown instead of plain text.

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

Note

Add pending-question sidebar indicators and user-input summaries to timeline

  • Adds SidebarQuestionIndicators component that lists non-archived threads with pending user input, shows a count, and cycles to the next pending thread when activated
  • SidebarContent renders the indicator above its scroll area when a navigation callback is supplied; both Sidebar and LegacySidebar wire up the callback
  • Sidebar thread rows in both sidebars receive a pending-question class and data attribute; a CSS overlay adds an indigo border to marked rows
  • ComposerPendingUserInputCard now renders the active question as ChatMarkdown instead of plain text
  • deriveWorkLogEntries in session-logic.ts parses submitted answers for resolved user-input activities and attaches a compact userInputSummary plus structured detail to the work-log entry; PlainWorkEntryRow displays the summary as truncated secondary text
  • Risk: WorkLogEntry.userInputSummary is a new optional field — existing consumers of WorkLogEntry that do not handle the field will simply ignore it, but timeline rendering changes for entries that now carry summaries

Macroscope summarized cbbcb3e.

@github-actions github-actions Bot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:L 100-499 changed lines (additions + deletions). labels Sep 4, 2026
Comment thread apps/web/src/components/sidebar/SidebarQuestionIndicators.tsx Outdated
@macroscopeapp

macroscopeapp Bot commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

Approvability

Verdict: Not approved

Macroscope's review found this PR not approvable — This PR introduces a new pending-question navigation workflow across shared sidebar and chat components. The navigation currently ignores active project/search filters, and the collapsed sidebar has a reported overflow issue, so the cross-cutting behavior and unresolved runtime concerns need human review.

Not approved because:

  • 1 blocking correctness issue found at or above your repo's Minimum Blocking Severity

Adjust the Minimum Blocking Severity for this repo — including turning it Off — in Settings. You can add or adjust custom eligibility rules. Learn more.

@maria-rcks maria-rcks changed the title feat(web): highlight pending questions and show submitted answers feat(web): make pending questions easy to find without moving threads Sep 4, 2026
Comment thread apps/web/src/components/ui/sidebar.tsx Outdated
Comment thread apps/web/src/index.css
Comment thread apps/web/src/components/sidebar/SidebarQuestionIndicators.tsx
Comment thread apps/web/src/components/Sidebar.tsx Outdated

@cursor cursor Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Stale Bugbot comment from a previous run.

Comment thread apps/web/src/components/sidebar/SidebarQuestionIndicators.tsx Outdated
Comment thread apps/web/src/components/ui/sidebar.tsx

@cursor cursor Bot left a comment

Copy link
Copy Markdown
Contributor

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 default 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 cbbcb3e. Configure here.

Comment thread apps/web/src/components/sidebar/SidebarQuestionIndicators.tsx
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:L 100-499 changed lines (additions + deletions). vouch:trusted PR author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant