[Bug] useLabelKeyboard wired but labels array is empty — hotkeys do not trigger labels #2

New issue

Closed

opened 2026-03-04 09:29:56 -08:00 by pyr0ball · 0 comments

pyr0ball commented

2026-03-04 09:29:56 -08:00

Owner

Problem

useLabelKeyboard is called in LabelView.vue but receives an empty labels: [] array:

useLabelKeyboard({
  labels: [],   // ← never populated
  onLabel: ...
})

The composable maps number keys 1-9 to label names by index. With an empty array, pressing 1-9 does nothing.

Fix

Pass labels.value (the fetched labels list) instead of a hardcoded empty array:

useLabelKeyboard({
  labels: labels.value,
  onLabel: (name) => handleLabel(name),
  onSkip: () => handleSkip(),
  onDiscard: () => handleDiscard(),
  onUndo: () => handleUndo(),
  onHelp: () => { /* TODO */ },
})

Note: labels is a ref<string[]> fetched from GET /api/config/labels on mount. The composable call needs to either watch the ref or be called after fetch completes.

Acceptance Criteria

Pressing 1-9 maps to the corresponding label in the fetched list
h skips, S → skip, D → discard, U → undo all work as documented

## Problem `useLabelKeyboard` is called in `LabelView.vue` but receives an empty `labels: []` array: ```ts useLabelKeyboard({ labels: [], // ← never populated onLabel: ... }) ``` The composable maps number keys 1-9 to label names by index. With an empty array, pressing 1-9 does nothing. ## Fix Pass `labels.value` (the fetched labels list) instead of a hardcoded empty array: ```ts useLabelKeyboard({ labels: labels.value, onLabel: (name) => handleLabel(name), onSkip: () => handleSkip(), onDiscard: () => handleDiscard(), onUndo: () => handleUndo(), onHelp: () => { /* TODO */ }, }) ``` Note: `labels` is a `ref<string[]>` fetched from `GET /api/config/labels` on mount. The composable call needs to either watch the ref or be called after fetch completes. ## Acceptance Criteria - Pressing `1`-`9` maps to the corresponding label in the fetched list - `h` skips, `S` → skip, `D` → discard, `U` → undo all work as documented