fix(daemon): use real terminal size for PTY instead of hardcoded 24x80

[Wirasm]

Summary

• Daemon PTYs were created with hardcoded 24x80 dimensions, causing agents to render incorrectly until the Ghostty attach window connected and sent a resize
• Add a 4-tier resolution chain for initial PTY dimensions: --rows/--cols CLI flags > [daemon] default_rows/default_cols config > ioctl(TIOCGWINSZ) on stdout > 80×24 fallback
• Fixes the real-terminal case (ioctl detects actual size) and enables non-TTY callers like Claude Code to specify dimensions via flags or config

Test plan

• kild create test --daemon --rows 50 --cols 200 → daemon log shows rows:50, cols:200
• Config [daemon] default_rows = 50 / default_cols = 200 → used when no CLI flags and no TTY
• From real Ghostty terminal: kild create test --daemon → uses actual terminal dimensions
• cargo test --all passes
• cargo clippy --all -- -D warnings clean

[Wirasm]

PR Review Summary

Critical Issues (1 found)

Agent	Issue	Location
code-reviewer, silent-failure-hunter, code-simplifier	Partial CLI/config override silently discarded — resolve_pty_size() requires both --rows AND --cols to be set for either to take effect. kild create foo --cols 220 silently ignores the flag and falls through to ioctl/80×24. Same for config: setting only default_cols has no effect.	daemon_spawn.rs:270-282

Fix: Resolve each dimension independently:

fn resolve_pty_size(params: &AgentSpawnParams<'_>) -> (u16, u16) {
    let cfg = &params.kild_config.daemon;
    let (terminal_cols, terminal_rows) = query_terminal_size();
    let cols = params.cols.or(cfg.default_cols).unwrap_or(terminal_cols);
    let rows = params.rows.or(cfg.default_rows).unwrap_or(terminal_rows);
    (cols, rows)
}

Important Issues (1 found)

Agent	Issue	Location
code-reviewer, code-simplifier	#[allow(clippy::too_many_arguments)] — open_session now has 9 positional args. Project enforces -D warnings. An OpenSessionRequest struct (matching existing CreateSessionRequest pattern) would fix this properly.	open.rs:48

Suggestions (3 found)

Agent	Suggestion	Location
silent-failure-hunter	Add debug! log in query_terminal_size() fallback path and in resolve_pty_size() showing which source won	daemon_spawn.rs:288-302
code-simplifier	Merge with_rows()/with_cols() into with_pty_size(rows, cols)	request.rs:118-126
docs-impact-agent	Missing --rows/--cols in CLAUDE.md CLI quick-reference and README config block	CLAUDE.md, README.md

Strengths

• Clean 4-tier resolution chain design
• Correct and well-documented unsafe ioctl block
• Proper config merge for new fields
• nix::libc is already a transitive dep — no new crate added

Verdict: NEEDS FIXES

1. Fix resolve_pty_size to resolve each dimension independently (critical — user-visible bug)
2. Consider OpenSessionRequest struct to eliminate #[allow(clippy::too_many_arguments)]