Skip to content

♻️ Replace CLI-based app logs with native WebSocket SDK#119

Merged
renardeinside merged 3 commits intomainfrom
improv/use-app-logs
Feb 22, 2026
Merged

♻️ Replace CLI-based app logs with native WebSocket SDK#119
renardeinside merged 3 commits intomainfrom
improv/use-app-logs

Conversation

@renardeinside
Copy link
Copy Markdown
Collaborator

Summary

  • Replace CLI-based databricks apps logs invocation with native WebSocket streaming via databricks_sdk for lower latency and structured output
  • Harden SDK types: ComputeState enum with #[serde(other)] for forward compat, Validation error variant, url::Url-based URL helpers, idle timeout for early stream termination, debug logging for deser errors
  • Improve MCP handler: ResolvedAppName struct (replaces &mut bool), per-profile DatabricksClient cache (RwLock with double-check), u32 tail_lines, restore from_serializable structured MCP content pattern

Test plan

  • 9 new API-level deserialization/validation tests in apx-databricks-sdk (all pass)
  • All 32 SDK tests pass (cargo test -p apx-databricks-sdk)
  • All 30 MCP lib tests pass (cargo test -p apx-mcp --lib)
  • Zero clippy warnings across workspace
  • cargo fmt clean

🤖 Generated with Claude Code

@renardeinside renardeinside changed the title Replace CLI-based app logs with native WebSocket SDK ♻️ Replace CLI-based app logs with native WebSocket SDK Feb 22, 2026
@renardeinside renardeinside merged commit 537d3f4 into main Feb 22, 2026
3 checks passed
@renardeinside renardeinside deleted the improv/use-app-logs branch February 22, 2026 20:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant