refactor: remove unused logger and spinner methods and quote utilities

Remove deprecated/unused functionality:
- logger.reason() method and LOG_SYMBOLS.reason
- spinner.reason() and spinner.reasonAndStop() methods
- posixQuote() and win32Quote() functions from argv/quote
- Remove argv/quote subpath export from package.json

Also removed associated tests and updated documentation.

All tests passing after removal.

Author: jdalton

package.json

@@ -111,10 +111,6 @@
       "types": "./dist/argv/parse.d.ts",
       "default": "./dist/argv/parse.js"
     },
-    "./argv/quote": {
-      "types": "./dist/argv/quote.d.ts",
-      "default": "./dist/argv/quote.js"
-    },
     "./arrays": {
       "types": "./dist/arrays.d.ts",
       "default": "./dist/arrays.js"

src/argv/quote.ts

@@ -26,7 +26,6 @@ import { THEMES } from './themes/themes'
  * console.log(`${LOG_SYMBOLS.info} Information message`)
  * console.log(`${LOG_SYMBOLS.step} Processing step`)
  * console.log(`${LOG_SYMBOLS.progress} Working on task`)
- * console.log(`${LOG_SYMBOLS.reason} Working through logic`)
  * ```
  */
 type LogSymbols = {
@@ -36,8 +35,6 @@ type LogSymbols = {
   info: string
   /** Cyan colored progress indicator symbol (∴ or :. in ASCII) */
   progress: string
-  /** Dimmed yellow reasoning/working symbol (∴ or :. in ASCII) */
-  reason: string
   /** Cyan colored skip symbol (↻ or @ in ASCII) */
   skip: string
   /** Cyan colored step symbol (→ or > in ASCII) */
@@ -164,7 +161,6 @@ function applyColor(
  * console.log(`${LOG_SYMBOLS.fail} Build failed`)          // Theme error color ✖
  * console.log(`${LOG_SYMBOLS.info} Starting process`)      // Theme info color ℹ
  * console.log(`${LOG_SYMBOLS.progress} Working on task`)   // Theme step color ∴
- * console.log(`${LOG_SYMBOLS.reason} Analyzing dependencies`) // Dimmed yellow ∴
  * console.log(`${LOG_SYMBOLS.step} Processing files`)      // Theme step color →
  * console.log(`${LOG_SYMBOLS.success} Build completed`)    // Theme success color ✔
  * console.log(`${LOG_SYMBOLS.warn} Deprecated API used`)   // Theme warning color ⚠
@@ -602,9 +598,6 @@ export class Logger {
       fail: applyColor(supported ? '✖' : '×', theme.colors.error, colors),
       info: applyColor(supported ? 'ℹ' : 'i', theme.colors.info, colors),
       progress: applyColor(supported ? '∴' : ':.', theme.colors.step, colors),
-      reason: colors.dim(
-        applyColor(supported ? '∴' : ':.', theme.colors.warning, colors),
-      ),
       skip: applyColor(supported ? '↻' : '@', theme.colors.step, colors),
       step: applyColor(supported ? '→' : '>', theme.colors.step, colors),
       success: applyColor(supported ? '✔' : '√', theme.colors.success, colors),
@@ -1396,30 +1389,6 @@ export class Logger {
     return this
   }
 
-  /**
-   * Logs a reasoning/working message with a dimmed yellow therefore symbol.
-   *
-   * Automatically prefixes the message with `LOG_SYMBOLS.reason` (dimmed yellow ∴).
-   * Useful for showing intermediate reasoning, logic steps, or "working" output
-   * that leads to a conclusion. Always outputs to stderr. If the message starts
-   * with an existing symbol, it will be stripped and replaced.
-   *
-   * @param args - Message and additional arguments to log
-   * @returns The logger instance for chaining
-   *
-   * @example
-   * ```typescript
-   * logger.step('Analyzing package security')
-   * logger.reason('Found 3 direct dependencies')
-   * logger.reason('Checking 47 transitive dependencies')
-   * logger.reason('Risk score: 8.5/10')
-   * logger.fail('Package blocked due to high risk')
-   * ```
-   */
-  reason(...args: unknown[]): this {
-    return this.#symbolApply('reason', args)
-  }
-
   /**
    * Resets all indentation to zero.
    *

src/logger.ts

@@ -32,15 +32,9 @@ import { resolveColor } from './themes/utils'
 
 /**
  * Symbol types for status messages.
- * Maps to log symbols: fail (✗), info (ℹ), reason (∴), skip (↻), success (✓), warn (⚠).
+ * Maps to log symbols: fail (✗), info (ℹ), skip (↻), success (✓), warn (⚠).
  */
-export type SymbolType =
-  | 'fail'
-  | 'info'
-  | 'reason'
-  | 'skip'
-  | 'success'
-  | 'warn'
+export type SymbolType = 'fail' | 'info' | 'skip' | 'success' | 'warn'
 
 /**
  * Progress tracking information for display in spinner.
@@ -150,11 +144,6 @@ export type Spinner = {
   /** Increment progress by specified amount (default: 1) */
   progressStep(amount?: number): Spinner
 
-  /** Show reasoning (∴) message without stopping the spinner */
-  reason(text?: string | undefined, ...extras: unknown[]): Spinner
-  /** Show reasoning (∴) message and stop the spinner, auto-clearing the line */
-  reasonAndStop(text?: string | undefined, ...extras: unknown[]): Spinner
-
   /** Set complete shimmer configuration */
   setShimmer(config: ShimmerConfig): Spinner
 
@@ -996,45 +985,6 @@ export function Spinner(options?: SpinnerOptions | undefined): Spinner {
         return this
       }
 
-      /**
-       * Show a reasoning/working message (∴) without stopping the spinner.
-       * Outputs to stderr and continues spinning.
-       *
-       * @param text - Reasoning message to display
-       * @param extras - Additional values to log
-       * @returns This spinner for chaining
-       */
-      reason(text?: string | undefined, ...extras: unknown[]) {
-        return this.#showStatusAndKeepSpinning('reason', [text, ...extras])
-      }
-
-      /**
-       * Show a reasoning/working message (∴) and stop the spinner.
-       * Auto-clears the spinner line before displaying the message.
-       *
-       * Implementation note: Unlike other *AndStop methods (successAndStop, failAndStop, etc.),
-       * this method cannot use #apply() with a 'reason' method name because yocto-spinner
-       * doesn't have a built-in 'reason' method. Instead, we manually stop the spinner then
-       * log the message with the reason symbol. This matches the pattern used by methods
-       * like debugAndStop() and maintains consistency with normalizeText() usage and empty
-       * string handling (see #apply's stop method handling for the pattern).
-       *
-       * @param text - Reasoning message to display
-       * @param extras - Additional values to log
-       * @returns This spinner for chaining
-       */
-      reasonAndStop(text?: string | undefined, ...extras: unknown[]) {
-        // Stop spinner first (can't use #apply('reason') since yocto-spinner has no 'reason' method)
-        this.#apply('stop', [])
-        // Normalize text (trim leading whitespace) like other methods
-        const normalized = normalizeText(text)
-        // Only log if we have actual content (consistent with #apply's stop method handling)
-        if (normalized) {
-          logger.error(`${LOG_SYMBOLS.reason} ${normalized}`, ...extras)
-        }
-        return this
-      }
-
       /**
        * Show a skip message (↻) without stopping the spinner.
        * Outputs to stderr and continues spinning.
@@ -1051,9 +1001,10 @@ export function Spinner(options?: SpinnerOptions | undefined): Spinner {
        * Show a skip message (↻) and stop the spinner.
        * Auto-clears the spinner line before displaying the message.
        *
-       * Implementation note: Similar to reasonAndStop(), this method cannot use #apply()
-       * with a 'skip' method name because yocto-spinner doesn't have a built-in 'skip'
-       * method. Instead, we manually stop the spinner then log the message with the skip symbol.
+       * Implementation note: Unlike other *AndStop methods (successAndStop, failAndStop, etc.),
+       * this method cannot use #apply() with a 'skip' method name because yocto-spinner doesn't
+       * have a built-in 'skip' method. Instead, we manually stop the spinner then log the message
+       * with the skip symbol.
        *
        * @param text - Skip message to display
        * @param extras - Additional values to log

src/spinner.ts

@@ -1079,36 +1079,6 @@ describe('Logger', () => {
     })
   })
 
-  describe('reason() method', () => {
-    it('should log reason message with symbol', () => {
-      testLogger.reason('processing dependencies')
-      const output = stderrChunks.join('')
-      expect(output).toContain('processing dependencies')
-    })
-
-    it('should support multiple arguments', () => {
-      testLogger.reason('Found', 3, 'issues')
-      const output = stderrChunks.join('')
-      expect(output).toContain('Found')
-    })
-
-    it('should return logger instance for chaining', () => {
-      const result = testLogger.reason('analyzing...')
-      expect(result).toBe(testLogger)
-    })
-
-    it('should handle empty reason', () => {
-      testLogger.reason()
-      expect(stderrChunks.length).toBeGreaterThan(0)
-    })
-
-    it('should strip existing symbols', () => {
-      testLogger.reason('∴ already has symbol')
-      const output = stderrChunks.join('')
-      expect(output).toContain('already has symbol')
-    })
-  })
-
   describe('skip() method', () => {
     it('should log skip message with symbol', () => {
       testLogger.skip('Test skipped')