Skip to content

[app-server] fix account/read response annotation #5642

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 2 commits into from
Oct 24, 2025
Merged

[app-server] fix account/read response annotation #5642

merged 2 commits into from
Oct 24, 2025
+32 −8

Conversation

@owenlin0
Copy link
Contributor

@owenlin0 owenlin0 commented Oct 24, 2025 •
edited
Loading

The API schema export is currently broken:

> cargo run -p codex-app-server-protocol --bin export -- --out DIR
Error: this type cannot be exported

This PR fixes the error message so we get more info:

> cargo run -p codex-app-server-protocol --bin export -- --out DIR
Error: failed to export client responses: dependency core::option::Option<codex_protocol::account::Account> cannot be exported

And fixes the root cause which is the account/read response.
@owenlin0 owenlin0 marked this pull request as ready for review October 24, 2025 17:47
@owenlin0 owenlin0 merged commit 190e7eb into main Oct 24, 2025
34 of 37 checks passed
@owenlin0 owenlin0 deleted the owen/fix_app_server_export branch October 24, 2025 18:17
@github-actions github-actions bot locked and limited conversation to collaborators Oct 24, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

3 participants

@owenlin0 @rasmusrygaard