Skip to content

feat(desktop): put the Ask Luke composer at the foot of the History tab - #643

Merged
dastratakos merged 1 commit into
mainfrom
dastratakos/nagoya-v3
Sep 2, 2026
Merged

feat(desktop): put the Ask Luke composer at the foot of the History tab#643
dastratakos merged 1 commit into
mainfrom
dastratakos/nagoya-v3

Conversation

@dastratakos

@dastratakos dastratakos commented Sep 2, 2026

Copy link
Copy Markdown
Contributor

Summary

  • The "Ask Luke" composer now stands at the foot of the History tab as well as under the sessions list. It is the same AskLuke component wired to the same ask handler, so a typed ask from either tab enters the one conversation, and on History the reply lands as a bubble directly above the field that asked.
  • The composer rides inside History's ph-no-capture subtree, so a draft there is as unrecorded as the thread around it. Nothing new leaves the machine.
  • The ask key no longer forces a switch to the Sessions tab: it lands in whichever composer is showing and only gives way to the sessions list from Settings, which has no field.
  • The History thread's own gap already separates the pill from the last bubble, so the composer's list margin is zeroed there.
  • Luke's guide now says the History tab carries the composer too, and the ConversationHistoryPanel test covers the field in the empty and threaded states and its placement after the list.

Evidence

  • Platform-independent checks: ./scripts/check.sh passed (exit 0) on the Linux cloud workspace, including the new conversation-history-panel test.
  • macOS Electron verification (./scripts/verify.sh): not run locally (Linux workspace); CI's macOS job runs it and links the evidence below.

Automated visual evidence

Download the deterministic macOS evidence · workflow run

  • Commit: 0d8e9ef67074d0ed78273ceddd02de7a893b0cd4
  • Scenario: smoke
  • Physical-notch check: not performed by CI

Physical-device evidence

  • Screenshot or screen recording: not attached
  • Physical-notch check: not performed
  • Device/display configuration: not recorded

Notes

  • Blockers or follow-up verification: inspect CI's History-tab evidence for the composer's spacing under the thread.

🤖 Generated with Claude Code


Open workspace in Conductor

Open in Alchemize

The History tab is where a typed ask's reply lands as a bubble, yet the
field that asks lived only under the sessions list, so reading the thread
and continuing it meant a tab change between every exchange. The same
composer now stands at the thread's foot, addressed to the same
conversation, inside the subtree recordings never see. The ask key still
lands in whichever field is showing and gives way to the sessions list
only from Settings, which holds none; Luke's guide learns where the second
field stands.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
@vercel

vercel Bot commented Sep 2, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
luke-web Ready Ready Preview Sep 2, 2026 7:24pm UTC

Request Review

@dastratakos
dastratakos added this pull request to the merge queue Sep 2, 2026
Merged via the queue into main with commit 44c85fe Sep 2, 2026
10 checks passed
@dastratakos
dastratakos deleted the dastratakos/nagoya-v3 branch September 2, 2026 19:26
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