Skip to content

Update to fastly:compute/service@0.1.0.#33

Merged
dgohman-fastly merged 2 commits into
mainfrom
dgohman-fastly/update-wit
Jan 29, 2026
Merged

Update to fastly:compute/service@0.1.0.#33
dgohman-fastly merged 2 commits into
mainfrom
dgohman-fastly/update-wit

Conversation

@dgohman-fastly
Copy link
Copy Markdown
Contributor

No description provided.

@posborne
Copy link
Copy Markdown
Member

posborne commented Jan 29, 2026

Long term we'll need to figure out how we want to pull in WIT but copying for now is fine. Other options might be:

  • Copy but use the single-file output from wasm-tools wit; we transform to this in build.rs anyhow.
  • Git submodule; probably the best option if we end up making https://github.com/fastly/fst-compute-wit public (under some name). We could move to that now but it might just make more work in making this public in the near term.
  • Viceroy git submodule? Don't love this but it would be another option.
  • Pull in data as rust crate dep to get the data.

Gitsm is probably where we'll end up I'm guessing. The final option could make sense at some point as an alternate distribution option but is probably unnecessary unless we end up having a similar build-tool-written-in-rust pattern pop up elsewhere (which, it definitely could in the future).

EDIT: But even with that tooling, we currently call out to componentize-py to generate stubs anyhow so it wouldn't even be a complete solution (without expanding our tooling).

@dgohman-fastly
Copy link
Copy Markdown
Contributor Author

I'm also expecting we'll eventually have a dedicated public git repo we can pull in as a submodule for the WIT interfaces. Until then, the interfaces is now stable so hopefully we won't be doing this copying very often.

@dgohman-fastly dgohman-fastly merged commit c9996ba into main Jan 29, 2026
4 checks passed
@dgohman-fastly dgohman-fastly deleted the dgohman-fastly/update-wit branch January 29, 2026 19:23
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.

2 participants