Skip to content

Test session table keyboard activation#31

Draft
cursor[bot] wants to merge 2 commits into
mainfrom
cursor/missing-test-coverage-fb72
Draft

Test session table keyboard activation#31
cursor[bot] wants to merge 2 commits into
mainfrom
cursor/missing-test-coverage-fb72

Conversation

@cursor

@cursor cursor Bot commented Jul 17, 2026

Copy link
Copy Markdown

Risky behavior now covered

  • Session rows remain keyboard-focusable.
  • Mouse clicks and Enter/Space activate the correct session.
  • Space activation prevents page scrolling.
  • Non-activation keys neither select a session nor cancel normal keyboard behavior.

Test files added/updated

  • Added web/components/session-table.test.tsx with five deterministic component tests.
  • Added web/vitest.config.ts and web test dependencies.
  • Updated the root npm test command to run both core and web suites.

Regression-risk reduction

PR #25 introduced a keyboard-only interaction path with no prior component coverage. These tests lock down the complete activation contract and session identity without network calls or timers. A mutation removing Space handling caused the targeted test to fail, confirming regression sensitivity.

Validation

  • npm test — 40 tests passed (35 core, 5 web)
  • New test/config ESLint checks passed
  • Web TypeScript check passed
Open in Web View Automation 

cursoragent and others added 2 commits July 17, 2026 10:05
Co-authored-by: Parvez Kose <koseparvez@gmail.com>
Co-authored-by: Parvez Kose <koseparvez@gmail.com>
@vercel

vercel Bot commented Jul 17, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
polyagent Ready Ready Preview, Comment Jul 17, 2026 10:11am

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