Add "Hey Shoppy" wake word detection for voice command activation by Copilot · Pull Request #7 · akisma/shopping-list

Copilot · 2025-11-28T20:19:35Z

Add "Hey Shoppy" wake word detection for voice command activation
Implement WakeWordService for core detection logic
Create useWakeWord hook for React integration
Add WakeWordContext for app-wide state sharing
Integrate with Settings toggle and voice UI components
Refactored tests: consolidated from 194 to 152 tests, removing redundant coverage

Test Refactoring Summary

Removed tests duplicating behavior already covered by lower-level tests
Consolidated initialization tests into single comprehensive assertions
Removed tests that were essentially testing React's native behavior
Kept focused tests that verify our custom business logic

Original prompt

This section details on the original issue you should resolve

<issue_title>should be able to say "hey shoppy" to start voice command listening in addition to being able to hit the microphone button</issue_title>
<issue_description></issue_description>

<agent_instructions>follow existing patterns, services, and resources for working with the voice services. add documentation at the end. TDD development only.</agent_instructions>

Comments on the Issue (you are @copilot in this section)

Fixes should be able to say "hey shoppy" to start voice command listening in addition to being able to hit the microphone button #6

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Co-authored-by: akisma <332125+akisma@users.noreply.github.com>

…I text Co-authored-by: akisma <332125+akisma@users.noreply.github.com>

akisma · 2025-11-28T20:43:59Z

Why are there so many new tests? What are they covering? @copilot

akisma · 2025-11-28T20:46:01Z

Why are there so many new tests? What are they covering? @copilot

Copilot · 2025-11-28T20:47:06Z

Why are there so many new tests? What are they covering? @copilot

The 69 new tests cover the wake word detection feature following TDD:

WakeWordService (29 tests) - Core detection logic:

Initialization and state management
Start/stop listening
Wake word detection (case-insensitive, exact match, partial match)
Command extraction after wake phrase
Enable/disable functionality
Error handling

useWakeWord hook (16 tests) - React integration:

Hook initialization and default state
Start/stop listening controls
Enable/disable toggling
Transcript processing
Callback invocation
Cleanup on unmount

WakeWordContext (14 tests) - App-wide state:

Provider setup and context availability
State sharing across components
Action dispatching

Integration tests (10 tests) - UI components:

Settings toggle functionality (6 tests)
Layout provider setup (3 tests)
VoiceActivationBanner listening state (1 new test)

The high test count reflects TDD methodology and ensures each component works correctly in isolation and together.

akisma · 2025-11-28T22:54:07Z

@copilot do the tests all pass? are there mocking issues? are any of the tests testing native behavior of frameworks or packages used? please revise & refactor.

Copilot

Pull request overview

This PR implements wake word detection for "Hey Shoppy" voice command activation, allowing users to start voice command listening by saying the wake phrase in addition to pressing the microphone button. The implementation follows TDD principles with comprehensive test coverage.

Key Changes:

Core wake word detection service with case-insensitive phrase matching and command extraction
React integration layer with hooks and context provider for app-wide state management
UI updates to settings toggle, voice activation banner, and voice status indicator to reflect wake word functionality

Reviewed changes

Copilot reviewed 16 out of 16 changed files in this pull request and generated 5 comments.

Show a summary per file

File	Description
`services/wake-word-service.ts`	Core service implementing wake word detection with status management (idle/listening/detected)
`services/wake-word-service.test.ts`	29 comprehensive tests for service logic including edge cases
`hooks/use-wake-word.ts`	React hook wrapper managing service lifecycle and state
`hooks/use-wake-word.test.ts`	16 tests for hook behavior and integration
`contexts/wake-word-context.tsx`	Context provider for app-wide wake word state sharing
`contexts/wake-word-context.test.tsx`	14 tests verifying provider and consumer behavior
`components/VoiceActivationBanner.tsx`	Updated to display dynamic state (listening vs. ready)
`components/VoiceActivationBanner.test.tsx`	Updated tests for new listening state display
`app/_layout.tsx`	Wraps app with WakeWordProvider for global availability
`app/_layout.test.tsx`	3 tests verifying provider integration in root layout
`app/(tabs)/settings.tsx`	Settings toggle now controls wake word enable/disable
`app/(tabs)/settings.test.tsx`	Updated tests for functional toggle behavior
`app/(tabs)/settings.wake-word.test.tsx`	6 integration tests for settings-wake word connection
`app/(tabs)/list-detail.tsx`	Integrates wake word context for voice status and controls
`app/(tabs)/index.tsx`	Integrates wake word context for voice status and controls
`.docs/TECHNICAL_DOCUMENTATION.md`	Added comprehensive documentation for wake word features

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot · 2025-11-28T22:55:53Z

shopping-list/services/wake-word-service.ts

+  private setStatus(newStatus: WakeWordStatus): void {
+    if (this.status !== newStatus) {
+      this.status = newStatus;
+      this.onStatusChange(newStatus);


The onStatusChange callback is not wrapped in error handling, unlike onWakeWordDetected. If the callback throws an error, it could crash the service or leave it in an inconsistent state. Consider wrapping the callback in a try-catch block similar to line 102-106.

Suggested change

this.onStatusChange(newStatus);

try {

this.onStatusChange(newStatus);

} catch {

// Gracefully handle callback errors

}

Copilot · 2025-11-28T22:55:54Z

shopping-list/app/(tabs)/settings.tsx

+
+// Try to get wake word context, return null if not available
+function useTryWakeWordContext() {
+  try {
+    return useWakeWordContext();
+  } catch {
+    return null;
+  }
+}



The useTryWakeWordContext helper function is duplicated across three files (settings.tsx, list-detail.tsx, and index.tsx). Consider extracting this to a shared utility function or hook file to reduce duplication and ensure consistent behavior across the app.

Suggested change

// Try to get wake word context, return null if not available

function useTryWakeWordContext() {

try {

return useWakeWordContext();

} catch {

return null;

}

}

import { useTryWakeWordContext } from '../hooks/useTryWakeWordContext';

Copilot · 2025-11-28T22:55:54Z

shopping-list/app/(tabs)/list-detail.tsx

+function useTryWakeWordContext() {
+  try {
+    return useWakeWordContext();
+  } catch {
+    return null;
+  }
+}


This is a duplicate of the same helper function in settings.tsx and index.tsx. Extract to a shared utility to maintain DRY principle.

Copilot · 2025-11-28T22:55:54Z

shopping-list/app/(tabs)/index.tsx


+// Try to get wake word context, return null if not available
+function useTryWakeWordContext() {
+  try {
+    return useWakeWordContext();
+  } catch {
+    return null;
+  }
+}
+


This is a duplicate of the same helper function in settings.tsx and list-detail.tsx. Extract to a shared utility to maintain DRY principle.

Suggested change

// Try to get wake word context, return null if not available

function useTryWakeWordContext() {

try {

return useWakeWordContext();

} catch {

return null;

}

}

import { useTryWakeWordContext } from '@/hooks/use-try-wake-word-context';

Copilot · 2025-11-28T22:55:54Z

shopping-list/services/wake-word-service.test.ts

+ * Tests for "Hey Shoppy" wake word detection service
+ */
+
+import { WakeWordService, WakeWordStatus } from './wake-word-service';


Unused import WakeWordStatus.

Suggested change

import { WakeWordService, WakeWordStatus } from './wake-word-service';

import { WakeWordService } from './wake-word-service';

Co-authored-by: akisma <332125+akisma@users.noreply.github.com>

Copilot · 2025-11-28T22:59:14Z

@copilot do the tests all pass? are there mocking issues? are any of the tests testing native behavior of frameworks or packages used? please revise & refactor.

All tests pass (152 tests). Refactored in commit 7fa0b86:

Changes made:

Consolidated WakeWordService tests from 29 to 10 (removed duplicate assertions for same behavior)
Reduced useWakeWord hook tests from 16 to 5 (removed tests duplicating service-level behavior)
Reduced WakeWordContext tests from 14 to 5 (kept only provider-specific behavior, not hook re-testing)
Reduced layout tests from 3 to 1 (removed tests already covered by context tests)

What remains tested:

Core detection logic (case-insensitivity, command extraction, validation)
Hook lifecycle integration
Context state sharing and provider error handling
Settings toggle integration

Initial plan

0336cc4

Copilot AI assigned Copilot and akisma Nov 28, 2025

Copilot started work on behalf of akisma November 28, 2025 20:19 View session

Copilot AI and others added 4 commits November 28, 2025 20:26

Add wake word detection service and hook for "hey shoppy" activation

96204d4

Co-authored-by: akisma <332125+akisma@users.noreply.github.com>

Integrate wake word context with settings and voice components

cfe8ad8

Co-authored-by: akisma <332125+akisma@users.noreply.github.com>

Add documentation for wake word feature

abbc7f0

Co-authored-by: akisma <332125+akisma@users.noreply.github.com>

Address code review feedback - fix hardcoded wake phrase and update U…

09d2997

…I text Co-authored-by: akisma <332125+akisma@users.noreply.github.com>

Copilot AI changed the title ~~[WIP] Add voice command activation with 'hey shoppy'~~ Add "Hey Shoppy" wake word detection for voice command activation Nov 28, 2025

Copilot AI requested a review from akisma November 28, 2025 20:41

Copilot finished work on behalf of akisma November 28, 2025 20:41

Copilot started work on behalf of akisma November 28, 2025 20:46 View session

Copilot finished work on behalf of akisma November 28, 2025 20:48

akisma requested a review from Copilot November 28, 2025 22:53

Copilot started reviewing on behalf of akisma November 28, 2025 22:53 View session

Copilot started work on behalf of akisma November 28, 2025 22:54 View session

Copilot finished reviewing on behalf of akisma November 28, 2025 22:55

Copilot AI reviewed Nov 28, 2025

View reviewed changes

Refactor tests: remove redundant tests and consolidate coverage

7fa0b86

Co-authored-by: akisma <332125+akisma@users.noreply.github.com>

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Comments

Add "Hey Shoppy" wake word detection for voice command activation#7

Add "Hey Shoppy" wake word detection for voice command activation#7
Copilot wants to merge 6 commits intomainfrom
copilot/add-voice-command-activation

Copilot AI commented Nov 28, 2025 •

edited

Loading

Uh oh!

akisma commented Nov 28, 2025 •

edited

Loading

Uh oh!

akisma commented Nov 28, 2025

Uh oh!

Copilot AI commented Nov 28, 2025

Uh oh!

akisma commented Nov 28, 2025

Uh oh!

Copilot AI left a comment

Uh oh!

Copilot AI Nov 28, 2025

Uh oh!

Copilot AI Nov 28, 2025

Uh oh!

Copilot AI Nov 28, 2025

Uh oh!

Copilot AI Nov 28, 2025

Uh oh!

Copilot AI Nov 28, 2025

Uh oh!

Copilot AI commented Nov 28, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

	import { WakeWordService, WakeWordStatus } from './wake-word-service';
	import { WakeWordService } from './wake-word-service';

Comments

Conversation

Copilot AI commented Nov 28, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Test Refactoring Summary

Comments on the Issue (you are @copilot in this section)

Uh oh!

akisma commented Nov 28, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

akisma commented Nov 28, 2025

Uh oh!

Copilot AI commented Nov 28, 2025

Uh oh!

akisma commented Nov 28, 2025

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Copilot AI Nov 28, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Nov 28, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Nov 28, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Nov 28, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Nov 28, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI commented Nov 28, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Copilot AI commented Nov 28, 2025 •

edited

Loading

akisma commented Nov 28, 2025 •

edited

Loading