Skip to content

Remove VS Code base64 transport and use stable study IDs - #1319

Merged
c-bata merged 3 commits into
optuna:mainfrom
c-bata:vscode-worker
Aug 12, 2026
Merged

Remove VS Code base64 transport and use stable study IDs#1319
c-bata merged 3 commits into
optuna:mainfrom
c-bata:vscode-worker

Conversation

@c-bata

@c-bata c-bata commented Aug 12, 2026

Copy link
Copy Markdown
Member

Summary

  • Send storage files from the VS Code extension host to the Webview as Uint8Array instead of base64-encoded strings.
  • Remove the corresponding atob and binary-string conversion logic.
  • Clean up the Webview message listener when the component unmounts.
  • Change OptunaStorage.getStudy to use a study ID instead of a list index.
  • Update SQLite and Journal lookups, routes, and Study links to use stable study IDs.
  • Add a regression test covering Journal study IDs with deleted-study gaps.

The study detail route is now /study/:studyId.

@c-bata
c-bata merged commit 0a04f93 into optuna:main Aug 12, 2026
10 checks passed
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