fix: prevent duplicate empty args in generated completions #440
AmirSa12ci.yml
on: pull_request
Lint and Type Check
21s
Benchmarks
17s
Matrix: Tests
Annotations
6 errors and 6 warnings
|
Tests (macos-latest)
Process completed with exit code 1.
|
|
tests/shell-empty-argv.test.ts > generated shell argv protocol > PowerShell sends exactly one empty arg for root empty completion:
tests/shell-empty-argv.test.ts#L309
AssertionError: promise rejected "Error: ENOENT: no such file or directory,… { …(4) }" instead of resolving
❯ assertPowerShellCase tests/shell-empty-argv.test.ts:309:57
❯ tests/shell-empty-argv.test.ts:372:9
❯ withFixture tests/shell-empty-argv.test.ts:177:5
❯ tests/shell-empty-argv.test.ts:370:5
Caused by: Caused by: Error: ENOENT: no such file or directory, open '/var/folders/bh/cdl3cft92_dbvg852_fp7y7w0000gn/T/tab-shell-empty-argv-YFKlx1/captured-argv.jsonl'
❯ readLastCapturedArgs tests/shell-empty-argv.test.ts:159:19
❯ assertPowerShellCase tests/shell-empty-argv.test.ts:309:3
❯ tests/shell-empty-argv.test.ts:372:9
❯ withFixture tests/shell-empty-argv.test.ts:177:5
❯ tests/shell-empty-argv.test.ts:370:5
⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯
Serialized Error: { errno: -2, code: 'ENOENT', syscall: 'open', path: '/var/folders/bh/cdl3cft92_dbvg852_fp7y7w0000gn/T/tab-shell-empty-argv-YFKlx1/captured-argv.jsonl' }
|
|
Tests (windows-latest)
The strategy configuration was canceled because "test.macos-latest" failed
|
|
Tests (windows-latest)
The operation was canceled.
|
|
Tests (ubuntu-latest)
The strategy configuration was canceled because "test.macos-latest" failed
|
|
Tests (ubuntu-latest)
The operation was canceled.
|
|
Lint and Type Check
Node.js 20 is deprecated. The following actions target Node.js 20 but are being forced to run on Node.js 24: actions/checkout@v4, actions/setup-node@v4, pnpm/action-setup@v4.0.0. For more information see: https://github.blog/changelog/2025-09-19-deprecation-of-node-20-on-github-actions-runners/
|
|
Lint and Type Check:
src/commander.ts#L2
Imports "TabCommand" and "OptionHandler" are only used as type
|
|
Benchmarks
Node.js 20 is deprecated. The following actions target Node.js 20 but are being forced to run on Node.js 24: actions/checkout@v4, actions/setup-node@v4, pnpm/action-setup@v4.0.0. For more information see: https://github.blog/changelog/2025-09-19-deprecation-of-node-20-on-github-actions-runners/
|
|
Tests (macos-latest)
Node.js 20 is deprecated. The following actions target Node.js 20 but are being forced to run on Node.js 24: actions/checkout@v4, actions/setup-node@v4, pnpm/action-setup@v4.0.0. For more information see: https://github.blog/changelog/2025-09-19-deprecation-of-node-20-on-github-actions-runners/
|
|
Tests (windows-latest)
Node.js 20 is deprecated. The following actions target Node.js 20 but are being forced to run on Node.js 24: actions/checkout@v4, actions/setup-node@v4, pnpm/action-setup@v4.0.0. For more information see: https://github.blog/changelog/2025-09-19-deprecation-of-node-20-on-github-actions-runners/
|
|
Tests (ubuntu-latest)
Node.js 20 is deprecated. The following actions target Node.js 20 but are being forced to run on Node.js 24: actions/checkout@v4, actions/setup-node@v4, pnpm/action-setup@v4.0.0. For more information see: https://github.blog/changelog/2025-09-19-deprecation-of-node-20-on-github-actions-runners/
|