hyperlight-dev / hyperlight-dev/hyperlight-js
Unify usage of "host:" prefix for host modules between Rust and NodeJS
Nobody has claimed this yet.
- Dominant language
- Rust
- Stars
- 13
- Forks
- 5
- Avg merge
- 8d 10h
- Merged PRs (30d)
- 13
Description
More bikeshedding: Do we want to add the "host:" prefix? If we do, should we also add it on the rust impl?
Originally posted by @jprendes in https://github.com/hyperlight-dev/hyperlight-js/pull/31#discussion_r2878570790
Currently host modules registered from NodeJS get an automatic "host:" prefix, while modules registered from Rust do not. We should unify the behaviour in one way or the other.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Compare the host-module registration entry points in Rust and NodeJS, focusing on where the automatic "host:" prefix is applied. Resolve the intended consistent behavior and verify that both registration paths use the same prefix policy.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- node.js, rust
- Domain
- backend-api-design
- Issue type
- Refactor
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 30/100