Highlight draft project in sidebar

[marmeladema]

Summary by CodeRabbit

• Bug Fixes
  • Improved sidebar highlighting for active projects and threads.
  • Draft conversations now correctly highlight their associated project.
  • Sidebar selection remains accurate after navigation and re-rendering.
  • Added clearer spacing and visual emphasis for selected projects.

[coderabbitai]

📝 Walkthrough

Walkthrough

Draft project rows now share active sidebar styling and accessibility state with thread rows. Highlight synchronization tracks both persisted thread selection and the project owning the current draft, with an E2E test covering the transition from draft to active thread.

Changes

Draft sidebar highlighting

Layer / File(s)	Summary
Recompute sidebar selection state crates/giskard-server/static/app.js	Sidebar selection now uses a shared helper and recomputes active and aria-current state for both thread rows and draft project rows.
Style and validate draft project highlighting crates/giskard-server/static/app.css, tests/e2e/tests/draft-composer.spec.ts	Project rows receive active and hover styling with updated padding, while the E2E test verifies draft highlighting and the subsequent active thread state.

Estimated code review effort: 3 (Moderate) | ~20 minutes

Sequence Diagram(s)

sequenceDiagram
  participant DraftComposer
  participant syncActiveThreadHighlight
  participant Sidebar
  DraftComposer->>syncActiveThreadHighlight: draft project id and thread state
  syncActiveThreadHighlight->>Sidebar: set project active and aria-current
  DraftComposer->>Sidebar: send message
  Sidebar->>syncActiveThreadHighlight: active thread state
  syncActiveThreadHighlight->>Sidebar: replace project highlight with thread highlight

Loading

Possibly related PRs

• marmeladema/Giskard#123: Overlaps with sidebar active-selection logic and styling.
• marmeladema/Giskard#124: Modifies related sidebar active and hover CSS.
• marmeladema/Giskard#141: Affects draft and thread selection state coupled to highlighting.

Suggested reviewers: claude

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately summarizes the main change: highlighting the draft project in the sidebar.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch draft-project-sidebar-selection

---

_{Comment @coderabbitai help to get the list of available commands.}

[coderabbitai]

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@crates/giskard-server/static/app.css`:
- Line 99: Regenerate the mobile screenshot at docs/screenshots/ide-mobile.png
after the sidebar styling change in the app.css rule, ensuring it reflects the
updated styling consistently with the desktop screenshot.

🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

• Push a commit to this branch (recommended)
• Create a new PR with the fixes

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: 6d0f96bc-5377-44e2-a7a4-4d173d26fb07

📥 Commits

Reviewing files that changed from the base of the PR and between 775b7cd and 3b77472.

⛔ Files ignored due to path filters (1)

• docs/screenshots/ide-desktop.png is excluded by !**/*.png, !docs/screenshots/**

📒 Files selected for processing (3)

• crates/giskard-server/static/app.css
• crates/giskard-server/static/app.js
• tests/e2e/tests/draft-composer.spec.ts