Skip to content

Add typed modeling connection errors - #1340

Open
pierremtb wants to merge 2 commits into
mainfrom
pierremtb/modeling-terminal-connection-errors
Open

Add typed modeling connection errors#1340
pierremtb wants to merge 2 commits into
mainfrom
pierremtb/modeling-terminal-connection-errors

Conversation

@pierremtb

@pierremtb pierremtb commented Aug 25, 2026

Copy link
Copy Markdown
Contributor

Summary

  • add stable modeling connection error codes for authentication, account access, concurrency limits, and backend disconnects
  • add an explicit retryable flag and human-readable detail
  • expose a backward-compatible ModelingWebSocketResponse envelope while leaving the legacy WebSocketResponse enum unchanged for downstream Rust and generated-SDK consumers
  • rebase onto the automatic OpenAPI generation workflow on current main

Tests

  • cargo test -p kittycad-modeling-cmds --features websocket serialize_websocket_connection_error
  • CI Rust tests, lints, formatting, Wasm check, OpenAPI regeneration, and release check pass

@codecov

codecov Bot commented Aug 25, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 63.07692% with 24 lines in your changes missing coverage. Please review.
⚠️ Please upload report for BASE (main@bef38e1). Learn more about missing BASE report.

Files with missing lines Patch % Lines
modeling-cmds/src/websocket.rs 63.07% 24 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1340   +/-   ##
=======================================
  Coverage        ?   34.65%           
=======================================
  Files           ?       36           
  Lines           ?     1870           
  Branches        ?        0           
=======================================
  Hits            ?      648           
  Misses          ?     1222           
  Partials        ?        0           
Flag Coverage Δ
unittests 34.65% <63.07%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@pierremtb
pierremtb force-pushed the pierremtb/modeling-terminal-connection-errors branch from 2d5a2b2 to a20b74d Compare August 26, 2026 10:05
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