@involvex/youtube-music-cli 0.0.27 → 0.0.29

package/CHANGELOG.md

@@ -1,3 +1,15 @@
+## [0.0.29](https://github.com/involvex/youtube-music-cli/compare/v0.0.28...v0.0.29) (2026-02-22)
+
+### Bug Fixes
+
+- prevent import navigation when in settings view ([d805b5a](https://github.com/involvex/youtube-music-cli/commit/d805b5a182a2241ab5494565dcf6645da82da954))
+
+## [0.0.28](https://github.com/involvex/youtube-music-cli/compare/v0.0.27...v0.0.28) (2026-02-22)
+
+### Bug Fixes
+
+- standardize quote style in snyk_rules.instructions.md ([5906fed](https://github.com/involvex/youtube-music-cli/commit/5906fed587be5f9cd6629281428ef304d8b32749))
+
 ## [0.0.27](https://github.com/involvex/youtube-music-cli/compare/v0.0.26...v0.0.27) (2026-02-20)
 
 ## [0.0.26](https://github.com/involvex/youtube-music-cli/compare/v0.0.25...v0.0.26) (2026-02-20)

package/dist/source/components/layouts/MainLayout.js

@@ -80,7 +80,9 @@ function MainLayout() {
     }, [dispatch, navState.currentView]);
     const goToImport = useCallback(() => {
         // Don't navigate to import if we're in plugins view (i key is used for plugin install there)
-        if (navState.currentView !== VIEW.PLUGINS) {
+        // Don't navigate to import if we're in settings view (user navigates settings items with Enter)
+        if (navState.currentView !== VIEW.PLUGINS &&
+            navState.currentView !== VIEW.SETTINGS) {
             dispatch({ category: 'NAVIGATE', view: VIEW.IMPORT });
         }
     }, [dispatch, navState.currentView]);

package/dist/source/components/layouts/SearchLayout.js

@@ -1,4 +1,4 @@
-import { jsxs as _jsxs, jsx as _jsx } from "react/jsx-runtime";
+import { jsx as _jsx, jsxs as _jsxs } from "react/jsx-runtime";
 // Search view layout
 import { useNavigation } from "../../hooks/useNavigation.js";
 import { useYouTubeMusic } from "../../hooks/useYouTubeMusic.js";
@@ -10,9 +10,11 @@ import SearchBar from "../search/SearchBar.js";
 import { useKeyBinding } from "../../hooks/useKeyboard.js";
 import { KEYBINDINGS, VIEW } from "../../utils/constants.js";
 import { Box, Text } from 'ink';
+import { usePlayer } from "../../hooks/usePlayer.js";
 function SearchLayout() {
     const { theme } = useTheme();
     const { state: navState, dispatch } = useNavigation();
+    const { state: playerState } = usePlayer();
     const { isLoading, error, search } = useYouTubeMusic();
     const [results, setResults] = useState([]);
     const [isTyping, setIsTyping] = useState(true);
@@ -116,7 +118,8 @@ function SearchLayout() {
             lastAutoSearchedQueryRef.current = null;
         };
     }, [dispatch]);
-    return (_jsxs(Box, { flexDirection: "column", children: [_jsxs(Text, { color: theme.colors.dim, children: ["Limit: ", navState.searchLimit, " (Use [ or ] to adjust)"] }), _jsx(SearchBar, { isActive: isTyping && !isSearching, onInput: input => {
+    return (_jsxs(Box, { flexDirection: "column", children: [playerState.currentTrack && (_jsxs(Box, { children: [_jsx(Text, { color: theme.colors.dim, children: playerState.isPlaying ? '▶ ' : '⏸ ' }), _jsx(Text, { color: theme.colors.primary, bold: true, children: playerState.currentTrack.title }), playerState.currentTrack.artists &&
+                        playerState.currentTrack.artists.length > 0 && (_jsxs(Text, { color: theme.colors.secondary, children: [' • ', playerState.currentTrack.artists.map(a => a.name).join(', ')] }))] })), _jsxs(Text, { color: theme.colors.dim, children: ["Limit: ", navState.searchLimit, " (Use [ or ] to adjust)"] }), _jsx(SearchBar, { isActive: isTyping && !isSearching, onInput: input => {
                     void performSearch(input);
                 } }), (isLoading || isSearching) && (_jsx(Text, { color: theme.colors.accent, children: "Searching..." })), error && _jsx(Text, { color: theme.colors.error, children: error }), !isLoading && navState.hasSearched && (_jsx(SearchResults, { results: results, selectedIndex: navState.selectedResult, isActive: !isTyping, onMixCreated: handleMixCreated, onDownloadStatus: handleDownloadStatus })), !isLoading && navState.hasSearched && results.length === 0 && !error && (_jsx(Text, { color: theme.colors.dim, children: "No results found" })), actionMessage && (_jsx(Text, { color: theme.colors.accent, children: actionMessage })), _jsx(Text, { color: theme.colors.dim, children: isTyping
                     ? 'Type to search, Enter to start, Esc to clear'

package/dist/source/hooks/useKeyboard.d.ts

@@ -1,6 +1,7 @@
 /**
  * Hook to bind keyboard shortcuts.
  * This uses a centralized manager to avoid multiple useInput calls and memory leaks.
+ * Uses a ref-based approach to always call the latest handler without stale closures.
  */
 export declare function useKeyBinding(keys: readonly string[], handler: () => void): void;
 /**

package/dist/source/hooks/useKeyboard.js

@@ -1,5 +1,5 @@
 // Keyboard input handling hook
-import { useCallback, useEffect } from 'react';
+import { useEffect, useRef } from 'react';
 import { useInput } from 'ink';
 import { logger } from "../services/logger/logger.service.js";
 import { useKeyboardBlockContext } from "./useKeyboardBlocker.js";
@@ -8,16 +8,18 @@ const registry = new Set();
 /**
  * Hook to bind keyboard shortcuts.
  * This uses a centralized manager to avoid multiple useInput calls and memory leaks.
+ * Uses a ref-based approach to always call the latest handler without stale closures.
  */
 export function useKeyBinding(keys, handler) {
-    const memoizedHandler = useCallback(handler, [handler]);
+    const handlerRef = useRef(handler);
+    handlerRef.current = handler;
     useEffect(() => {
-        const entry = { keys, handler: memoizedHandler };
+        const entry = { keys, handler: () => handlerRef.current() };
         registry.add(entry);
         return () => {
             registry.delete(entry);
         };
-    }, [keys, memoizedHandler]);
+    }, [keys]); // keys is the only dep; handlerRef is a stable ref
 }
 /**
  * Global Keyboard Manager Component
@@ -84,7 +86,15 @@ export function KeyboardManager() {
                             return false;
                         if (hasShift && !key.shift && !uppercaseShiftInput)
                             return false;
-                        if (!hasShift && key.shift)
+                        // Block lowercase-only bindings when shift is active or the input is
+                        // an uppercase letter (which implies Shift was held).
+                        // Example: the 'p' (Plugins) binding must not fire when the user
+                        // presses Shift+P, which should only trigger 'shift+p' (Playlists).
+                        // Note: `input !== input.toLowerCase()` is true only for uppercase
+                        // alphabetical characters, avoiding false positives on symbols/digits.
+                        if (!hasShift &&
+                            (key.shift ||
+                                (input.length === 1 && input !== input.toLowerCase())))
                             return false;
                         // Check the actual key
                         if (mainKey === 'up' && key.upArrow)

package/dist/source/hooks/usePlayer.js

@@ -5,22 +5,30 @@ import { getConfigService } from "../services/config/config.service.js";
 export function usePlayer() {
     const { state, dispatch, ...playerStore } = usePlayerStore();
     const play = useCallback((track, options) => {
-        // Clear queue if requested (e.g., playing from search results)
         if (options?.clearQueue) {
+            // When clearing the queue, always dispatch fresh play commands rather
+            // than relying on the stale queue state captured in this closure.
+            // This fixes a bug where a track already in the queue wouldn't replay
+            // after clearQueue because SET_QUEUE_POSITION would be dispatched
+            // against the (now-empty) queue.
             dispatch({ category: 'CLEAR_QUEUE' });
-        }
-        // Add to queue if not already there
-        const isInQueue = state.queue.some(t => t.videoId === track.videoId);
-        if (!isInQueue) {
             dispatch({ category: 'ADD_TO_QUEUE', track });
-        }
-        // Find position and play
-        const position = state.queue.findIndex(t => t.videoId === track.videoId);
-        if (position >= 0) {
-            dispatch({ category: 'SET_QUEUE_POSITION', position });
+            dispatch({ category: 'PLAY', track });
         }
         else {
-            dispatch({ category: 'PLAY', track });
+            // Add to queue if not already there
+            const isInQueue = state.queue.some(t => t.videoId === track.videoId);
+            if (!isInQueue) {
+                dispatch({ category: 'ADD_TO_QUEUE', track });
+            }
+            // Find position and play
+            const position = state.queue.findIndex(t => t.videoId === track.videoId);
+            if (position >= 0) {
+                dispatch({ category: 'SET_QUEUE_POSITION', position });
+            }
+            else {
+                dispatch({ category: 'PLAY', track });
+            }
         }
         // Add to history
         const config = getConfigService();

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@involvex/youtube-music-cli",
-	"version": "0.0.27",
+	"version": "0.0.29",
 	"description": "- A Commandline music player for youtube-music",
 	"repository": {
 		"type": "git",
@@ -65,11 +65,11 @@
 	"dependencies": {
 		"@distube/ytdl-core": "^4.16.12",
 		"ansi-escapes": "^7.3.0",
-		"ink": "^6.7.0",
+		"ink": "^6.8.0",
 		"ink-table": "^3.1.0",
 		"ink-text-input": "^6.0.0",
 		"jiti": "^2.6.1",
-		"meow": "^14.0.0",
+		"meow": "^14.1.0",
 		"node-notifier": "^10.0.1",
 		"discord-rpc": "^4.0.1",
 		"node-youtube-music": "^0.10.3",
@@ -77,20 +77,20 @@
 		"react": "^19.2.4",
 		"youtube-ext": "^1.1.25",
 		"youtubei.js": "^16.0.1",
-		"ws": "^8.18.0"
+		"ws": "^8.19.0"
 	},
 	"devDependencies": {
 		"@eslint/js": "^10.0.1",
 		"@sindresorhus/tsconfig": "^8.1.0",
-		"@types/node": "^25.2.3",
+		"@types/node": "^25.3.0",
 		"@types/node-notifier": "^8.0.5",
 		"@types/react": "^19.2.14",
-		"@types/ws": "^8.5.13",
+		"@types/ws": "^8.18.1",
 		"@vdemedes/prettier-config": "^2.0.1",
 		"ava": "^6.4.1",
 		"chalk": "^5.6.2",
 		"conventional-changelog-cli": "^5.0.0",
-		"eslint": "^10.0.0",
+		"eslint": "^10.0.1",
 		"eslint-plugin-react": "^7.37.5",
 		"eslint-plugin-react-hooks": "^7.0.1",
 		"globals": "^17.3.0",