scalar / openapi-parser

Modern OpenAPI parser written in TypeScript
MIT License
26 stars 7 forks source link

fix: node prefix breaks old webpack setup #124

Closed hanspagel closed 1 month ago

hanspagel commented 1 month ago

WIP: I’m not if it’s just the prefix that breaks Webpack yet. Need to test it.

I received this in a Docusaurus Webpack setup:

Module build failed: UnhandledSchemeError: Reading from "node:fs" is not handled by plugins (Unhandled scheme). Webpack supports "data:" and "file:" URIs by default. You may need an additional plugin to handle "node:" URIs.

changeset-bot[bot] commented 1 month ago

πŸ¦‹ Changeset detected

Latest commit: 52b617e405b4fbbc70fc7d0db2738fffafdca2c4

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package | Name | Type | | ---------------------- | ----- | | @scalar/openapi-parser | Patch |

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

relativeci[bot] commented 1 month ago

#85 Bundle Size β€” 114.7KiB (~-0.01%).

52b617e(current) vs 72703c8 main#83(baseline)

Bundle metrics  Change 2 changes Improvement 1 improvement
                 Current
#85
     Baseline
#83
Improvement  Initial JS 114.7KiB(~-0.01%) 114.71KiB
No change  Initial CSS 0B 0B
Change  Cache Invalidation 1.04% 4.51%
No change  Chunks 45 45
No change  Assets 45 45
No change  Modules 45 45
No change  Duplicate Modules 0 0
No change  Duplicate Code 0% 0%
No change  Packages 0 0
No change  Duplicate Packages 0 0

Bundle size by type  Change 1 change Improvement 1 improvement
|            |       Current
[#85](https://app.relative-ci.com/projects/hXQ54UQKluDUZAKvpt9b/jobs/85-3Gz0o8PcO9q4aqOxUD2q?utm_source=github&utm_medium=pr-report "View bundle analysis report") |      Baseline
[#83](https://app.relative-ci.com/projects/hXQ54UQKluDUZAKvpt9b/jobs/83-emhfC7lQeFeVjZbyXkxO?utm_source=github&utm_medium=pr-report "View baseline bundle analysis report") | |:--|--:|--:| | Improvement  [JS](https://app.relative-ci.com/projects/hXQ54UQKluDUZAKvpt9b/jobs/85-3Gz0o8PcO9q4aqOxUD2q/assets?ba=%7B%22filters%22%3A%22ft.CSS-0_ft.JS-1_ft.IMG-0_ft.MEDIA-0_ft.FONT-0_ft.HTML-0_ft.OTHER-0%22%7D "View JS assets") | `114.7KiB` (`~-0.01%`) | `114.71KiB` |

Bundle analysis report Branch fix/node-prefix-breaks-old-webpa... Project dashboard

cloudflare-pages[bot] commented 1 month ago

Deploying openapi-parser with  Cloudflare Pages  Cloudflare Pages

Latest commit: 52b617e
Status: βœ…  Deploy successful!
Preview URL: https://68ca2fb4.openapi-parser.pages.dev
Branch Preview URL: https://fix-node-prefix-breaks-old-w.openapi-parser.pages.dev

View logs

hanspagel commented 1 month ago

Doesn’t help. :) Got another solution, though.