Merge branch 'main' of ssh://github.com/BuilderIO/qwik into fixed_ser…

…ialize
genki · Sep 16, 2023 · c42dc77 · c42dc77
2 parents 7074c76 + bfe8040
commit c42dc77
Show file tree

Hide file tree

Showing 47 changed files with 5,619 additions and 596 deletions.
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -414,7 +414,7 @@ jobs:
             packages/qwik-labs/package.json
           if-no-files-found: error
 
-    ############ RELEASE ############
+  ############ RELEASE ############
   release:
     name: Release
     runs-on: ubuntu-latest
@@ -680,7 +680,7 @@ jobs:
         if: ${{ always() }}
         run: pnpm run lint.eslint
 
-  ############ TRIGGER QWIKCITY E2E Test ############
+  ############ TRIGGER QWIKCITY E2E TEST ############
   trigger-qwikcity-e2e:
     name: Trigger Qwik City E2E
     runs-on: ubuntu-latest

diff --git a/.github/workflows/pr.yml b/.github/workflows/pr.yml
@@ -0,0 +1,22 @@
+name: Qwik PR Checks
+
+on:
+  pull_request_target:
+    types:
+      - opened
+      - edited
+      - synchronize
+      - ready_for_review
+
+permissions:
+  pull-requests: read
+
+jobs:
+  ############ SEMANTIC PR TITLE VALIDATION ############
+  semantic-pr:
+    name: Validate PR title
+    runs-on: ubuntu-latest
+    steps:
+      - uses: amannn/action-semantic-pull-request@v5
+        env:
+          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
diff --git a/README.md b/README.md
@@ -31,6 +31,8 @@ npm create qwik@latest
 pnpm create qwik@latest
 # or
 yarn create qwik@latest
+# or
+bun create qwik@latest
 ```
 
 - Understand the difference between [resumable and replayable](https://qwik.builder.io/docs/concepts/resumable/) applications.

diff --git a/packages/create-qwik/src/helpers/jokes.json b/packages/create-qwik/src/helpers/jokes.json
@@ -41,7 +41,7 @@
   ["What's red and bad for your teeth?", "A Brick."],
   [
     "What's the difference between a guitar and a fish?",
-    "You can tune a guitar but you can't \"tuna\"fish!"
+    "You can tune a guitar but you can't \"tuna\" fish!"
   ],
   ["Why did the coffee file a police report?", "It got mugged."],
   ["What do you do when you see a space man?", "Park your car, man."],

diff --git a/packages/docs/scripts/pages.json b/packages/docs/scripts/pages.json
@@ -7,6 +7,11 @@
     "href": "https://www.builder.io/",
     "tags": "saas"
   },
+  {
+    "href": "https://www.expobeds.com/",
+    "size": "large",
+    "tags": "conpany"
+  },
   {
     "href": "https://jose-aguilar.vercel.app/",
     "tags": "portfolio"

diff --git a/packages/docs/src/repl/monaco.tsx b/packages/docs/src/repl/monaco.tsx
@@ -333,7 +333,8 @@ const MONACO_VS_URL = getCdnUrl('monaco-editor', MONACO_VERSION, '/min/vs');
 const MONACO_LOADER_URL = `${MONACO_VS_URL}/loader.js`;
 
 const CLIENT_LIB = `
-declare module '*.css' {
+declare module '*.css' {}
+declare module '*.css?inline' {
   const css: string
   export default css
 }

diff --git a/packages/docs/src/repl/worker/app-bundle-client.ts b/packages/docs/src/repl/worker/app-bundle-client.ts
@@ -128,7 +128,7 @@ export const getInputs = (options: ReplInputOptions) => {
   });
 };
 
-const MODULE_EXTS = ['.tsx', '.ts', '.js', '.jsx', '.mjs'];
+const MODULE_EXTS = ['.tsx', '.ts', '.js', '.jsx', '.mjs', '.css'];
 
 export const getOutput = (o: OutputChunk | OutputAsset) => {
   const f: ReplModuleOutput = {

diff --git a/packages/docs/src/repl/worker/repl-plugins.ts b/packages/docs/src/repl/worker/repl-plugins.ts
@@ -115,7 +115,8 @@ const getRuntimeBundle = (runtimeBundle: string) => {
 };
 
 export const replCss = (options: ReplInputOptions): Plugin => {
-  const isStylesheet = (id: string) => ['.css', '.scss', '.sass'].some((ext) => id.endsWith(ext));
+  const isStylesheet = (id: string) =>
+    ['.css', '.scss', '.sass'].some((ext) => id.endsWith(`${ext}?inline`));
 
   return {
     name: 'repl-css',
@@ -129,7 +130,7 @@ export const replCss = (options: ReplInputOptions): Plugin => {
 
     load(id) {
       if (isStylesheet(id)) {
-        const input = options.srcInputs.find((i) => i.path.endsWith(id));
+        const input = options.srcInputs.find((i) => i.path.endsWith(id.replace(/\?inline$/, '')));
         if (input && typeof input.code === 'string') {
           return `const css = ${JSON.stringify(input.code)}; export default css;`;
         }

diff --git a/packages/docs/src/routes/api/qwik-city-vite-netlify-edge/api.json b/packages/docs/src/routes/api/qwik-city-vite-netlify-edge/api.json
@@ -26,7 +26,7 @@
         }
       ],
       "kind": "Interface",
-      "content": "```typescript\nexport interface NetlifyEdgeAdapterOptions extends ServerAdapterOptions \n```\n**Extends:** ServerAdapterOptions\n\n\n|  Property | Modifiers | Type | Description |\n|  --- | --- | --- | --- |\n|  [excludedPath?](#) |  | string | _(Optional)_ Manually add path pattern that should be excluded from the edge function routes that are created by the 'manifest.json' file. |\n|  [functionRoutes?](#) |  | boolean | <p>_(Optional)_ Determines if the build should generate the edge functions declarations <code>manifest.json</code> file.</p><p>https://docs.netlify.com/edge-functions/declarations/</p><p>Defaults to <code>true</code>.</p> |\n|  [staticPaths?](#) |  | string\\[\\] | _(Optional)_ Manually add pathnames that should be treated as static paths and not SSR. For example, when these pathnames are requested, their response should come from a static file, rather than a server-side rendered response. |",
+      "content": "```typescript\nexport interface NetlifyEdgeAdapterOptions extends ServerAdapterOptions \n```\n**Extends:** ServerAdapterOptions\n\n\n|  Property | Modifiers | Type | Description |\n|  --- | --- | --- | --- |\n|  [excludedPath?](#) |  | string \\| string\\[\\] | <p>_(Optional)_ Manually add path pattern that should be excluded from the edge function routes that are created by the 'manifest.json' file.</p><p>If not specified, the following paths are excluded by default: /build/\\* /favicon.ico /robots.txt /mainifest.json /\\~partytown/\\* /service-worker.js /sitemap.xml</p><p>https://docs.netlify.com/edge-functions/declarations/\\#declare-edge-functions-in-netlify-toml</p> |\n|  [functionRoutes?](#) |  | boolean | <p>_(Optional)_ Determines if the build should generate the edge functions declarations <code>manifest.json</code> file.</p><p>https://docs.netlify.com/edge-functions/declarations/</p><p>Defaults to <code>true</code>.</p> |\n|  [staticPaths?](#) |  | string\\[\\] | _(Optional)_ Manually add pathnames that should be treated as static paths and not SSR. For example, when these pathnames are requested, their response should come from a static file, rather than a server-side rendered response. |",
       "editUrl": "https://github.com/BuilderIO/qwik/tree/main/packages/qwik-city/adapters/netlify-edge/vite/index.ts",
       "mdFile": "qwik-city.netlifyedgeadapteroptions.md"
     }

diff --git a/packages/docs/src/routes/api/qwik-city-vite-netlify-edge/index.md b/packages/docs/src/routes/api/qwik-city-vite-netlify-edge/index.md
@@ -30,10 +30,10 @@ export interface NetlifyEdgeAdapterOptions extends ServerAdapterOptions
 
 **Extends:** ServerAdapterOptions
 
-| Property             | Modifiers | Type     | Description                                                                                                                                                                                                                          |
-| -------------------- | --------- | -------- | ------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------ |
-| [excludedPath?](#)   |           | string   | _(Optional)_ Manually add path pattern that should be excluded from the edge function routes that are created by the 'manifest.json' file.                                                                                           |
-| [functionRoutes?](#) |           | boolean  | <p>_(Optional)_ Determines if the build should generate the edge functions declarations <code>manifest.json</code> file.</p><p>https://docs.netlify.com/edge-functions/declarations/</p><p>Defaults to <code>true</code>.</p>        |
-| [staticPaths?](#)    |           | string[] | _(Optional)_ Manually add pathnames that should be treated as static paths and not SSR. For example, when these pathnames are requested, their response should come from a static file, rather than a server-side rendered response. |
+| Property             | Modifiers | Type               | Description                                                                                                                                                                                                                                                                                                                                                                                                                   |
+| -------------------- | --------- | ------------------ | ----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- |
+| [excludedPath?](#)   |           | string \| string[] | <p>_(Optional)_ Manually add path pattern that should be excluded from the edge function routes that are created by the 'manifest.json' file.</p><p>If not specified, the following paths are excluded by default: /build/\* /favicon.ico /robots.txt /mainifest.json /\~partytown/\* /service-worker.js /sitemap.xml</p><p>https://docs.netlify.com/edge-functions/declarations/\#declare-edge-functions-in-netlify-toml</p> |
+| [functionRoutes?](#) |           | boolean            | <p>_(Optional)_ Determines if the build should generate the edge functions declarations <code>manifest.json</code> file.</p><p>https://docs.netlify.com/edge-functions/declarations/</p><p>Defaults to <code>true</code>.</p>                                                                                                                                                                                                 |
+| [staticPaths?](#)    |           | string[]           | _(Optional)_ Manually add pathnames that should be treated as static paths and not SSR. For example, when these pathnames are requested, their response should come from a static file, rather than a server-side rendered response.                                                                                                                                                                                          |
 
 [Edit this section](https://github.com/BuilderIO/qwik/tree/main/packages/qwik-city/adapters/netlify-edge/vite/index.ts)