perf(refactor): consolidate getState calls in resolveWebviewView

[0xMink]

Summary

• Consolidate three separate this.getState().then(...) calls into a single await this.getState() with destructuring in resolveWebviewView()
• Eliminates two redundant executions of getState() which includes CloudService calls, ContextProxy reads, and other work
• No behavioral change — same defaults, same setter calls, same values

Test plan

• Existing ClineProvider.spec.ts tests pass (90/90, 6 skipped unchanged)
• Verify terminal settings (shell integration timeout, command delay, zsh options) are applied correctly on webview load
• Verify TTS enabled/speed settings are applied correctly on webview load

---

Important

Refactor resolveWebviewView in ClineProvider.ts to consolidate multiple getState calls into a single call, improving performance without changing behavior.

• Refactor:
  • Consolidate three this.getState().then(...) calls into a single await this.getState() in resolveWebviewView() in ClineProvider.ts.
  • Use destructuring to extract state values, reducing redundant getState() executions.
• Behavior:
  • No change in behavior; same defaults, setter calls, and values are maintained.
• Testing:
  • Existing tests in ClineProvider.spec.ts pass (90/90, 6 skipped unchanged).

^{This description was created by for e69da98. You can customize this summary. It will automatically update as commits are pushed.}

[roomote]

Rooviewer   See task

Reviewed 5519dcd against the previous review at e69da98. The consolidation of three getState() calls into a single await this.getState() is correct -- all destructured defaults, setter calls, and values match the original behavior. No new issues found.

Previous reviews

• e69da98: Review #1
• e69da98: Review #2

_{Mention @roomote in a comment to request specific changes to this pull request or fix all unresolved issues.}

[roomote]

Rooviewer   See task

No issues found. The consolidation of three getState() calls into a single await this.getState() is correct -- all destructured defaults, setter calls, and values match the original behavior. All 88 existing tests pass.

_{Mention @roomote in a comment to request specific changes to this pull request or fix all unresolved issues.}