Skip to content

feat: Turbowarp cloud debug message compliance - #613

Draft
Boss-1s wants to merge 46 commits into
TimMcCool:mainfrom
Boss-1s:tw-debug-compliance
Draft

feat: Turbowarp cloud debug message compliance#613
Boss-1s wants to merge 46 commits into
TimMcCool:mainfrom
Boss-1s:tw-debug-compliance

Conversation

@Boss-1s

@Boss-1s Boss-1s commented May 11, 2026

Copy link
Copy Markdown
Contributor

Note

this is a recreated pr of #604 due to a branch name change

Important

I will be coming back to this after kms-smever1.3.1 releases.

Solves issue #537

Changes

  • So far added try-except blocks for: loading json on connect; reading json data
  • Utilizing Rich to make messages visually better
  • more natural language
  • Might need more, that's why I'm making this a draft

Tests

Tested in VS Code with simple scratchattach websocket script using CPython 3.12.1 env

Tested:

  • JSON Loading
  • Loading Method
  • Checking Project (set)
  • Loading Cloud Var Value (set)
  • Setting Cloud Var (set)
  • Checking User (handshake)
  • Checking Project (handshake)

Work to-do

  • JSON Loading
  • Loading Method
  • Checking Project (set)
  • Loading Cloud Var Value (set)
  • Setting Cloud Var (set)
  • Checking User (handshake)
  • Checking Project (handshake)

Boss-1s added 5 commits May 7, 2026 17:23
…ng data

TimMcCool#537

Signed-off-by: Boss_1s <95505913+Boss-1s@users.noreply.github.com>
Signed-off-by: Boss_1s <95505913+Boss-1s@users.noreply.github.com>
Signed-off-by: Boss_1s <95505913+Boss-1s@users.noreply.github.com>
Signed-off-by: Boss_1s <95505913+Boss-1s@users.noreply.github.com>
Signed-off-by: Boss_1s <95505913+Boss-1s@users.noreply.github.com>
Signed-off-by: GitHub <noreply@github.com>
Signed-off-by: GitHub <noreply@github.com>
@Boss-1s

Boss-1s commented Aug 25, 2026

Copy link
Copy Markdown
Contributor Author

They're all done; just need to test the last two with the cloud var set and load

…semver3

Signed-off-by: GitHub <noreply@github.com>
(cherry picked from commit 8d30697)
Signed-off-by: GitHub <noreply@github.com>
### sa.eventhandlers._base

- changed list comp to generator comp in set_project_vars()  and set_var()
- reimplemented attribute type hints into BaseCloudServer. since type hints are inhierted, there is no need to restate them in child classes of BaseCloudServer.

### sa.eventhandlers.cloud_server

- added type hitns to __init__ of BaseCloudServer child classes
- added type hints to init_cloud_server and init_ssl_cloud_server
- revert changing warnings.warn to print in 9863d50

Signed-off-by: GitHub <noreply@github.com>
Signed-off-by: GitHub <noreply@github.com>
Signed-off-by: GitHub <noreply@github.com>
Signed-off-by: GitHub <noreply@github.com>
@Boss-1s

Boss-1s commented Sep 2, 2026

Copy link
Copy Markdown
Contributor Author

what is BaseCloudServer.set_var's skip_forward argument type supposed to be

Signed-off-by: GitHub <noreply@github.com>
Boss-1s and others added 12 commits September 2, 2026 22:30
Signed-off-by: GitHub <noreply@github.com>
Signed-off-by: GitHub <noreply@github.com>
Signed-off-by: GitHub <noreply@github.com>
Co-authored-by: TheCommCraft <79996518+TheCommCraft@users.noreply.github.com>
Signed-off-by: Boss_1s <95505913+Boss-1s@users.noreply.github.com>
Signed-off-by: Boss_1s <95505913+Boss-1s@users.noreply.github.com>
Signed-off-by: Boss_1s <95505913+Boss-1s@users.noreply.github.com>
.
Signed-off-by: GitHub <noreply@github.com>
Signed-off-by: GitHub <noreply@github.com>
Signed-off-by: GitHub <noreply@github.com>
Signed-off-by: Boss_1s <95505913+Boss-1s@users.noreply.github.com>
Signed-off-by: GitHub <noreply@github.com>
@TheCommCraft

Copy link
Copy Markdown
Collaborator

what is BaseCloudServer.set_var's skip_forward argument type supposed to be

I am not sure

@TheCommCraft

Copy link
Copy Markdown
Collaborator

It's WebSocket

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.

2 participants