Skip to content

Conversation

@bowlofarugula
Copy link
Contributor

No description provided.

@bowlofarugula
Copy link
Contributor Author

bowlofarugula commented Nov 20, 2025

Could we use this image or are movie scenes off limits lol

image

image

@bowlofarugula bowlofarugula force-pushed the wasmcp branch 8 times, most recently from fc98c33 to 96059c0 Compare November 20, 2025 12:55
@radu-matei
Copy link
Member

This is an awesome article, @bowlofarugula! Thank you so much for taking the time to write it.

My only suggestion — would it make sense to add one short paragraph at the beginning of the article with some context on how this links to Spin/Wasm?

Thank you!

@radu-matei
Copy link
Member

(as for the image, I'm all for good puns, but will defer to @michelleN on this one 😂)

@bowlofarugula
Copy link
Contributor Author

@radu-matei Thank you for the review! Good call on adding some clarity up front. I've pushed an update with that change as well as a section explicitly showing how MCP components can be distributed on OCI registries this way.

Also, I looked into the failing build and noticed that a job is tripping broken over links to OpenAI, which are apparently being blocked by them. I have confirmed these links are accessible in the browser.

@bowlofarugula bowlofarugula force-pushed the wasmcp branch 5 times, most recently from bfdc537 to d9180af Compare November 20, 2025 20:30
Copy link
Contributor

@kate-goldenring kate-goldenring left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for all of your work on this! I left a few preliminary comments. I think this is ready to mark as "Ready for Review".

@bowlofarugula bowlofarugula marked this pull request as ready for review November 21, 2025 07:36
@bowlofarugula bowlofarugula force-pushed the wasmcp branch 8 times, most recently from bc87ae2 to e134dde Compare November 21, 2025 11:14
@bowlofarugula
Copy link
Contributor Author

bowlofarugula commented Nov 21, 2025

That failing build is due to OpenAI blocking some of the requests that job is making to validate links to their docs.

Is there some way to override that check for specific links?

@bowlofarugula bowlofarugula force-pushed the wasmcp branch 4 times, most recently from 270c783 to a10d3f4 Compare November 24, 2025 08:43
Signed-off-by: bowlofarugula <[email protected]>
Copy link
Contributor

@kate-goldenring kate-goldenring left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Just some optional nits.

Comment on lines 63 to 65
The [Fundamental Theorem of Software Engineering](https://en.wikipedia.org/wiki/Fundamental_theorem_of_software_engineering) states:

> We can solve any problem by introducing an extra level of [indirection](https://en.wikipedia.org/wiki/Indirection).
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nit: In markdown, this seems to hide this quote rather than highlight it. I'd just do a classic quote.

Suggested change
The [Fundamental Theorem of Software Engineering](https://en.wikipedia.org/wiki/Fundamental_theorem_of_software_engineering) states:
> We can solve any problem by introducing an extra level of [indirection](https://en.wikipedia.org/wiki/Indirection).
The [Fundamental Theorem of Software Engineering](https://en.wikipedia.org/wiki/Fundamental_theorem_of_software_engineering) states: "We can solve any problem by introducing an extra level of [indirection](https://en.wikipedia.org/wiki/Indirection)."

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That seems like a bug in our stylesheets (or something)? > should render as blockquote. Or am I misunderstanding your comment Kate?

@bowlofarugula bowlofarugula force-pushed the wasmcp branch 2 times, most recently from 8a38471 to a984b27 Compare November 25, 2025 01:14
@bowlofarugula
Copy link
Contributor Author

The list of broken links are

     1	├─BROKEN─ https://github.com/spinframework/spin/tree/main/wit/deps/http (HTTP_404)
     2	├─BROKEN─ https://github.com/spinframework/spin/blob/main/wit/deps/http/handler.wit (HTTP_404)

Oh no I think these are from the other blog post?

@itowlson
Copy link
Contributor

Oh bother, that's because a Spin PR renamed those WIT directories. Definitely not your problem @bowlofarugula. I will force merge if that's okay?

@itowlson itowlson merged commit b05e3cd into spinframework:main Nov 25, 2025
2 of 3 checks passed
@bowlofarugula
Copy link
Contributor Author

Woohoo thanks everyone!!

@itowlson
Copy link
Contributor

Thank you for all the hard yards on the writing!

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.

4 participants