lhumina_code/hero_osis
21
0
Fork 0
Code Issues 2 Pull requests 2 Projects Releases Packages Wiki Activity Actions

refactor: regenerate SDK with herolib_osis_client transport #9

Open
mik-tf wants to merge 2 commits from development_mik into development
pull from: development_mik
merge into: lhumina_code:development
Conversation 0 Commits 2 Files changed 20 +73 -266
mik-tf commented 2026-02-19 02:59:46 +00:00
Owner
Copy link

Summary

  • Regenerate all 17 SDK domain clients using updated hero_lib generator
  • Replace removed herolib_osis::offline::CachingTransport with herolib_osis_client::OsisClient
  • SDK Cargo.toml now depends on herolib_osis_client instead of herolib_osis (removes sled, dirs, WASM boilerplate deps)
  • Enable local hero_lib patch for development builds

Context

hero_lib development branch (commits 369eb82..25519fd) removed the offline module and introduced herolib_osis_client as a shared cross-platform RPC client. The SDK generated code still referenced the old CachingTransport, breaking all downstream consumers. This PR regenerates the SDK to match.

Supersedes PR #7 (which only fixed the import path but didn't address the full offline module removal).

Test plan

  • SDK compiles standalone (cargo check in sdk/rust/)
  • hero_os compiles with code island using regenerated SDK
  • Full WASM build succeeds (dx build --platform web)
  • hero_os deployed and responding on port 8880

🤖 Generated with Claude Code

Consolidated from #8 into development_mik branch (includes latest development merged in).

## Summary - Regenerate all 17 SDK domain clients using updated hero_lib generator - Replace removed `herolib_osis::offline::CachingTransport` with `herolib_osis_client::OsisClient` - SDK Cargo.toml now depends on `herolib_osis_client` instead of `herolib_osis` (removes `sled`, `dirs`, WASM boilerplate deps) - Enable local hero_lib patch for development builds ## Context hero_lib development branch (commits 369eb82..25519fd) removed the `offline` module and introduced `herolib_osis_client` as a shared cross-platform RPC client. The SDK generated code still referenced the old `CachingTransport`, breaking all downstream consumers. This PR regenerates the SDK to match. **Supersedes PR #7** (which only fixed the import path but didn't address the full offline module removal). ## Test plan - [x] SDK compiles standalone (`cargo check` in `sdk/rust/`) - [x] hero_os compiles with code island using regenerated SDK - [x] Full WASM build succeeds (`dx build --platform web`) - [x] hero_os deployed and responding on port 8880 🤖 Generated with [Claude Code](https://claude.com/claude-code) --- Consolidated from #8 into `development_mik` branch (includes latest `development` merged in).
refactor: regenerate SDK with herolib_osis_client (remove offline dep)
Some checks failed
Build and Test / build (pull_request) Failing after 3s
Details
0f23019e08 
Rebuild SDK using updated hero_lib generator that replaces the removed
herolib_osis::offline::CachingTransport with herolib_osis_client::OsisClient.
All 17 domain clients now use thin wrappers around the shared cross-platform
RPC client. SDK Cargo.toml updated to depend on herolib_osis_client instead
of herolib_osis, removing sled/dirs/WASM boilerplate dependencies.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
merge: resolve conflicts keeping herolib_osis_client transport from PR
Some checks failed
Build and Test / build (pull_request) Failing after 7s
Details
bda4198120
Some checks failed
Build and Test / build (pull_request) Failing after 7s
Details
This pull request can be merged automatically.
This branch is out-of-date with the base branch
You are not authorized to merge this pull request.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin development_mik:development_mik
git switch development_mik

Merge

Merge the changes and update on Forgejo.

Warning: The "Autodetect manual merge" setting is not enabled for this repository, you will have to mark this pull request as manually merged afterwards.

git switch development
git merge --no-ff development_mik
git switch development_mik
git rebase development
git switch development
git merge --ff-only development_mik
git switch development_mik
git rebase development
git switch development
git merge --no-ff development_mik
git switch development
git merge --squash development_mik
git switch development
git merge --ff-only development_mik
git switch development
git merge development_mik
git push origin development
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_osis!9
No description provided.