lhumina_code/hero_lib
22
0
Fork 1
Code Issues 17 Pull requests 4 Projects Releases Packages 1 Wiki Activity Actions

fix: correct ClientError re-export path in OSIS SDK generator #100

Closed
mik-tf wants to merge 1 commit from development_offline_import into development
pull from: development_offline_import
merge into: lhumina_code:development
Conversation 1 Commits 1 Files changed 1 +1 -1
mik-tf commented 2026-02-14 18:36:40 +00:00
Owner
Copy link

Summary

  • Fix the SDK code generator to emit herolib_osis::offline::ClientError instead of herolib_osis::offline::transport::ClientError
  • The offline module re-exports ClientError directly; the transport sub-path does not exist in the public API
  • This is a 1-line fix in packages/osis/src/generators/rust/rust_client.rs

Closes #101
Relates to: lhumina_code/hero_osis#6

## Summary - Fix the SDK code generator to emit `herolib_osis::offline::ClientError` instead of `herolib_osis::offline::transport::ClientError` - The `offline` module re-exports `ClientError` directly; the `transport` sub-path does not exist in the public API - This is a 1-line fix in `packages/osis/src/generators/rust/rust_client.rs` Closes #101 Relates to: https://forge.ourworld.tf/lhumina_code/hero_osis/issues/6
fix: use correct re-export path for ClientError in SDK generator
Some checks failed
Lint / lint-linux (pull_request) Failing after 23s
Details
Tests with Services / test-linux (pull_request) Has been cancelled
Details
Lint / lint-macos (pull_request) Has been cancelled
Details
Tests with Services / test-macos (pull_request) Has been cancelled
Details
d9b253bf47 
The offline module re-exports ClientError at herolib_osis::offline::ClientError,
not herolib_osis::offline::transport::ClientError. This caused all generated
SDK clients to fail to compile.

Closes lhumina_code/hero_osis#6

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
mik-tf commented 2026-02-19 02:59:32 +00:00
Author
Owner
Copy link

Superseded by #110 — consolidated to development_mik branch with latest development merged in.

Superseded by #110 — consolidated to `development_mik` branch with latest `development` merged in.
mik-tf closed this pull request 2026-02-19 02:59:37 +00:00
Some checks failed
Lint / lint-linux (pull_request) Failing after 23s
Details
Tests with Services / test-linux (pull_request) Has been cancelled
Details
Lint / lint-macos (pull_request) Has been cancelled
Details
Tests with Services / test-macos (pull_request) Has been cancelled
Details

Pull request closed

This pull request cannot be reopened because the branch was deleted.
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
lhumina_code/hero_lib!100
No description provided.