lsm / neokai / 26674465293 / 10
82%
dev: 82%

DEFAULT BRANCH: dev
Ran
Files 334
Run time 10s
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
coverage: 18.908%. Remained the same
26674465293.10

push

github

web-flow 
fix(credentials): fall back to DatabaseCredentialStore when macOS daemon has no TTY (#2047)

* fix(credentials): fall back to DatabaseCredentialStore when macOS daemon has no TTY

KeychainCredentialStore.set spawns `security add-generic-password -U -w`
and writes the password via stdin. When no existing Keychain entry exists,
`security` opens an interactive retype prompt on /dev/tty, bypassing stdin.
This blocks the daemon during startup because a server process cannot
respond to terminal prompts.

Fix: in createCredentialStore, when on darwin but process.stdout.isTTY is
false, return DatabaseCredentialStore instead of KeychainCredentialStore.
This keeps Keychain for interactive shells and uses the encrypted SQLite
fallback for daemon/non-TTY mode.

Tests added for all platform/TTY combinations.

* fix(credentials): tighten TTY guard and correct comment

- Change guard from `!== false` to `=== true` so undefined is treated
  as non-TTY (falls back to DatabaseCredentialStore).
- Correct comment: `security` reads retype confirmation from /dev/tty,
  not stdin.
- Update test: undefined isTTY now expects DatabaseCredentialStore.

* fix(credentials): use -p flag in KeychainCredentialStore.set to avoid /dev/tty prompt

Replaces the -w stdin approach with -p DATA flag. The -w flag caused
`security` to open an interactive retype prompt on /dev/tty when no
existing Keychain entry existed, blocking daemon startup.

- Revert TTY guard in createCredentialStore (back to platform-only check)
- KeychainCredentialStore.set now passes secret via -p flag
- Comment updated to explain the /dev/tty issue
- Tests simplified: remove isTTY tests, keep platform-based tests

17862 of 94466 relevant lines covered (18.91%)

9.81 hits per line

Source Files on job daemon-online-mcp - 26674465293.10