fix(byte): drop trailing && from authExport (kills byte when token set)

[dcetlin]

Problem

startByte builds authExport as an element of the inner array that is later .join(' && ')-ed:

const inner = [
  `cd ${shq(byteCwd)}`,
  ...
  authExport || null,          // = `export CLAUDE_CODE_OAUTH_TOKEN="$(cat ...)" &&`
  `caffeinate -i claude ...`,
].filter(Boolean).join(' && ')

Both authExport assignments appended their own trailing &&, so the joined command emitted:

... && export CLAUDE_CODE_OAUTH_TOKEN="$(cat ...)" && && caffeinate ...

The && && is a shell syntax error — the byte tmux session dies almost immediately.

It only fires when CLAUDE_CODE_OAUTH_TOKEN is set (or the slack angellist-token path), which is why token-less deployments never hit it. Surfaced during an external-machine bring-up.

Fix

Remove the trailing && from both assignments; the array join(' && ') already supplies the separator.

Verification

• bun build cli/hydra.ts compiles.
• Generated inner command with a token set contains no && &&.
• Default (token-less) path is byte-for-byte unchanged.

🤖 Generated with Claude Code

[sf8193]

Reviewed with dual-agent process (sp-reviewer + typescript-reviewer).

No blockers or should-fixes. The fix is correct — the old trailing && produced a double connector (&& &&) when interpolated into the inner array that's already joined with .join(' && ') on line 75. Removing it makes the shell command well-formed.

Verified: empty authExport path ('' || null → filtered by .filter(Boolean)) is unaffected. No regressions.

[sf8193]

Nit (flagged by typescript-reviewer): The two authExport branches (lines 59 and 63) are identical except for the file path. Could extract a helper like const makeAuthExport = (p: string) => \export CLAUDE_CODE_OAUTH_TOKEN="$(cat ${shq(p)})"`` to reduce duplication. Purely stylistic — not blocking.