feat: QWIKOSS-2 - [✨] When closing over values in server$ the values …

…should be marked as readonly

Co-authored-by: Hameed Abdulrahaman <hameedabdulrahamann@gmail.com>
Co-authored-by: wajihaNiazi <waijehaniazi204@gmail.com>
Co-authored-by: Phu Nguyen <51897872+phunguyenmurcul@users.noreply.github.com>

.all-contributorsrc

@@ -3,9 +3,7 @@
   "projectOwner": "BuilderIO",
   "repoType": "github",
   "repoHost": "https://github.com",
-  "files": [
-    "README.md"
-  ],
+  "files": ["README.md"],
   "imageSize": 100,
   "commit": true,
   "commitConvention": "angular",

.devcontainer/Dockerfile

@@ -1,12 +1,15 @@
-FROM cimg/rust:1.65.0-node
+FROM cimg/rust:1.72.1-node
 
-RUN rustup toolchain install nightly-2022-09-23; \
-    rustup default nightly-2022-09-23; \
-    rustup --version; \
+RUN rustup --version; \
     cargo --version; \
-    rustc --version; \
-    rustup update; \
+    rustc --version; 
+
+RUN rustup update; \
     rustup target add wasm32-unknown-unknown; \
-    cargo install cargo-insta wasm-pack; \
+    cargo install cargo-insta; \
+    cargo install wasm-pack; \
     rustup component add clippy; \
     corepack enable --install-directory ~/bin
+
+RUN mkdir /home/circleci/store; \
+    pnpm config set store-dir /home/circleci/store

.gitignore

@@ -13,7 +13,10 @@ temp
 
 # Application
 qwik-app/
-**/server/
+/server/
+/starters/**/server/
+/packages/*/server/
+/packages/*/src/styled-system/
 todo-express/
 target
 !/packages/docs/src/routes/demo/events/target
@@ -31,9 +34,9 @@ lib
 tsdoc-metadata.json
 
 # IDE and local environment
-.vscode/settings.json
 .idea
 .eslintcache
+test-results
 
 # Package Managers
 .yarn/*

.prettierrc.json

@@ -1,4 +1,7 @@
 {
+  "plugins": ["./node_modules/prettier-plugin-jsdoc/dist/index.js"],
+  "jsdocPreferCodeFences": true,
+  "tsdoc": true,
   "trailingComma": "es5",
   "tabWidth": 2,
   "semi": true,

.vscode/extensions.json

@@ -8,7 +8,8 @@
     "ms-azuretools.vscode-docker",
     "manucorporat.vermoji",
     "vadimcn.vscode-lldb",
-    "streetsidesoftware.code-spell-checker"
+    "streetsidesoftware.code-spell-checker",
+    "ZixuanChen.vitest-explorer"
   ],
   "unwantedRecommendations": []
 }

.vscode/launch.json

@@ -22,21 +22,6 @@
       "program": "${workspaceFolder}/node_modules/jest/bin/jest.js",
       "cwd": "${workspaceFolder}",
       "args": ["--runInBand", "--watchAll=false"]
-    },
-    {
-      "name": "uvu Current File",
-      "type": "node",
-      "request": "launch",
-      "skipFiles": ["<node_internals>/**"],
-      "program": "${workspaceFolder}/node_modules/tsm/bin.js",
-      "args": [
-        "${workspaceFolder}/node_modules/uvu/bin.js",
-        "${fileDirname}",
-        "${fileBasename}",
-        "--tsmconfig",
-        "${workspaceFolder}/tsm.cjs"
-      ],
-      "console": "integratedTerminal"
     }
   ]
 }

.vscode/settings.json

@@ -0,0 +1,10 @@
+{
+  "vitest.include": ["**/*.{test,spec,unit}.{js,mjs,cjs,ts,mts,cts,jsx,tsx}"],
+  "vitest.exclude": [
+    "**/node_modules/**",
+    "**/dist/**",
+    "**/dist-dev/**",
+    "**/cypress/**",
+    "**/.{idea,git,cache,output,temp}/**"
+  ]
+}

CONTRIBUTING.md

@@ -72,7 +72,7 @@ You need to have these tools up and running in your local machine:
 
 If you would like to make use of the devlopment container solution, but don't use VSCode or Dev Containers, you still can do so, by following steps:
 
-- Build development container locally: `cd .devcontainers; docker build -t qwik-container .`
+- Build development container locally: `cd .devcontainer; docker build -t qwik-container .`
 - Run development container from Qwik project root, binding the directory to container: `cd ..; docker run --rm -d --name qwik-container -p 3300:3300 -p 9229:9299 -v $PWD:/home/circleci/project -t qwik-container`
 
 Docker command does:
@@ -210,28 +210,17 @@ To use your build in your project, follow these steps:
     yarn link @builder.io/qwik @builder.io/qwik-city
    ```
 
-If you can't use package linking (npm link) just copy the contents of `package/qwik/dist` into your projects' `node_modules/@builder.io/qwik` folder.
+If you can't use package linking (npm link) just copy the contents of `packages/qwik/dist` into your projects' `node_modules/@builder.io/qwik` folder, and/or the contents of `packages/qwik-city/lib` into your projects' `node_modules/@builder.io/qwik-city` folder.
 
 ### Test against the docs site:
 
-1. Go to `packages/docs/package.json` and update:
-
-   ```diff
-
-   -- "@builder.io/qwik": "0.17.4",
-   -- "@builder.io/qwik-city": "0.1.0-beta13",
-
-   ++ "@builder.io/qwik": "workspace:*",
-   ++ "@builder.io/qwik-city": "workspace:*",
-   ```
-
-2. At the root of the Qwik repo folder run:
+1. At the root of the Qwik repo folder run:
 
 ```shell
 pnpm install
 ```
 
-3. Run the docs site:
+2. Run the docs site:
 
 ```shell
 cd packages/docs && pnpm start
@@ -245,20 +234,12 @@ pnpm serve
 
 ### Unit Tests Only
 
-Unit tests use [uvu](https://github.com/lukeed/uvu)
+Unit tests use [vitest](https://vitest.dev)
 
 ```shell
 pnpm test.unit
 ```
 
-To keep _uvu_ open with the watch mode, run:
-
-```shell
-pnpm test.watch
-```
-
-> Note that the `test.watch` command isn't necessary if you're running the `pnpm start` command, since `start` will also concurrently run the _uvu_ watch process.
-
 ### E2E Tests Only
 
 E2E tests use [Playwright](https://playwright.dev/).

package.json

@@ -1,6 +1,6 @@
 {
   "name": "qwik-monorepo",
-  "version": "1.2.12",
+  "version": "1.2.13",
   "config": {
     "commitizen": {
       "path": "./node_modules/cz-conventional-changelog"
@@ -27,73 +27,73 @@
     "esbuild-plugin-raw": "^0.1.7"
   },
   "devDependencies": {
-    "@builder.io/partytown": "^0.8.0",
+    "@builder.io/partytown": "^0.8.1",
     "@clack/prompts": "^0.7.0",
-    "@microsoft/api-documenter": "^7.22.30",
-    "@microsoft/api-extractor": "7.36.3",
-    "@napi-rs/cli": "2.12.1",
+    "@microsoft/api-documenter": "^7.23.9",
+    "@microsoft/api-extractor": "^7.38.0",
+    "@napi-rs/cli": "^2.16.3",
     "@napi-rs/triples": "1.1.0",
     "@node-rs/helper": "1.3.3",
     "@octokit/action": "3.18.1",
-    "@playwright/test": "1.36.2",
-    "@types/brotli": "1.3.1",
-    "@types/cross-spawn": "6.0.2",
-    "@types/eslint": "8.44.1",
-    "@types/express": "4.17.17",
-    "@types/mri": "1.1.1",
-    "@types/node": "^20.4.5",
+    "@playwright/test": "^1.38.1",
+    "@types/brotli": "^1.3.2",
+    "@types/cross-spawn": "^6.0.3",
+    "@types/eslint": "^8.44.4",
+    "@types/express": "^4.17.19",
+    "@types/mri": "^1.1.2",
+    "@types/node": "^20.8.4",
     "@types/path-browserify": "1.0.0",
-    "@types/prettier": "2.7.3",
-    "@types/prompts": "2.4.4",
-    "@types/semver": "7.5.0",
+    "@types/prompts": "^2.4.6",
+    "@types/semver": "^7.5.3",
     "@types/which-pm-runs": "1.0.0",
-    "@typescript-eslint/eslint-plugin": "6.2.0",
-    "@typescript-eslint/parser": "6.2.0",
-    "@typescript-eslint/rule-tester": "^6.2.0",
-    "@typescript-eslint/utils": "6.2.0",
+    "@typescript-eslint/eslint-plugin": "^6.7.5",
+    "@typescript-eslint/parser": "^6.7.5",
+    "@typescript-eslint/rule-tester": "^6.7.5",
+    "@typescript-eslint/utils": "^6.7.5",
     "all-contributors-cli": "6.26.1",
     "brotli": "1.3.3",
     "commitizen": "4.3.0",
-    "concurrently": "8.2.0",
+    "concurrently": "^8.2.1",
     "create-qwik": "workspace:*",
     "cross-spawn": "7.0.3",
     "cz-conventional-changelog": "3.3.0",
-    "esbuild": "0.18.17",
-    "eslint": "8.45.0",
+    "esbuild": "^0.19.4",
+    "eslint": "^8.51.0",
     "eslint-plugin-no-only-tests": "3.1.0",
-    "eslint-plugin-qwik": "1.2.6",
+    "eslint-plugin-qwik": "^1.2.13",
     "execa": "7.2.0",
     "express": "4.18.2",
     "install": "^0.13.0",
-    "monaco-editor": "^0.39.0",
+    "monaco-editor": "^0.44.0",
     "mri": "1.2.0",
     "ora": "6.3.1",
     "path-browserify": "1.0.1",
-    "prettier": "3.0.0",
+    "prettier": "^3.0.3",
+    "prettier-plugin-jsdoc": "^1.1.1",
     "pretty-quick": "^3.1.3",
     "prompts": "2.4.2",
     "rollup": "3.26.3",
     "semver": "7.5.4",
     "snoop": "^1.0.4",
     "syncpack": "^10.7.3",
-    "terser": "5.19.2",
-    "tsm": "2.2.2",
-    "typescript": "5.1.6",
-    "undici": "5.22.1",
-    "uvu": "0.5.6",
-    "vite": "4.4.7",
-    "vite-tsconfig-paths": "4.2.0",
+    "terser": "^5.21.0",
+    "tsm": "^2.3.0",
+    "typescript": "^5.2.2",
+    "undici": "^5.26.0",
+    "vite": "^4.4.11",
+    "vite-tsconfig-paths": "^4.2.1",
+    "vitest": "0.34.6",
     "watchlist": "0.3.1",
     "which-pm-runs": "1.1.0",
-    "zod": "^3.21.4"
+    "zod": "^3.22.4"
   },
   "engines": {
     "node": ">=16.8.0 <18.0.0 || >=18.11",
     "npm": "please-use-pnpm",
     "yarn": "please-use-pnpm",
     "pnpm": ">=8.6.12"
   },
-  "packageManager": "pnpm@8.6.12",
+  "packageManager": "pnpm@8.9.0",
   "pnpm": {
     "overrides": {
       "vfile": "6.0.1"
@@ -120,7 +120,7 @@
     "cli.validate": "tsm scripts/validate-cli.ts",
     "commit": "git-cz",
     "deps": "pnpm upgrade -i -r --latest",
-    "docs.sync": "tsm scripts/docs_sync/index.ts",
+    "docs.sync": "tsm scripts/docs_sync/index.ts && pnpm fmt",
     "eslint.update": "tsm scripts/eslint-docs.ts",
     "fmt": "pnpm prettier.fix && pnpm syncpack format",
     "fmt.staged": "pretty-quick --staged",
@@ -147,13 +147,14 @@
     "test.e2e.firefox": "playwright test starters --browser=firefox --config starters/playwright.config.ts",
     "test.e2e.webkit": "playwright test starters --browser=webkit --config starters/playwright.config.ts",
     "test.rust": "make test",
-    "test.unit": "tsm node_modules/uvu/bin.js packages unit.ts --tsmconfig tsm.cjs",
-    "test.unit.debug": "tsm --inspect-brk node_modules/uvu/bin.js packages unit.ts --tsmconfig tsm.cjs",
+    "test.unit": "vitest packages",
+    "test.unit.debug": "vitest --inspect-brk packages",
     "test.vite": "playwright test starters/e2e/qwikcity --browser=chromium --config starters/playwright.config.ts",
     "test.watch": "watchlist packages unit.ts -- pnpm test.unit",
     "tsc.check": "tsc --noEmit",
     "tsc.trace": "tsc -p tsconfig.json --traceResolution > tsc.log",
-    "tsc.watch": "tsc --noEmit --watch --preserveWatchOutput"
+    "tsc.watch": "tsc --noEmit --watch --preserveWatchOutput",
+    "update.qwik.builds": "tsm scripts/update-qwik-builds.ts packages/docs && tsm scripts/update-qwik-builds.ts packages/insights; pnpm install"
   },
   "type": "module"
 }

packages/create-qwik/package.json

@@ -1,13 +1,13 @@
 {
   "name": "create-qwik",
   "description": "Interactive CLI for create Qwik projects and adding features.",
-  "version": "1.2.12",
+  "version": "1.2.13",
   "author": "Builder.io Team",
   "bin": "./create-qwik.cjs",
   "bugs": "https://github.com/BuilderIO/qwik/issues",
   "devDependencies": {
     "@clack/prompts": "^0.7.0",
-    "@types/yargs": "17.0.24",
+    "@types/yargs": "^17.0.28",
     "kleur": "4.1.5",
     "yargs": "17.7.2"
   },

packages/create-qwik/src/run-create-cli.ts

@@ -52,9 +52,7 @@ function parseArgs(args: string[], templates: string[]) {
   return parsedArgs;
 }
 
-/**
- * @param args pass here process.argv.slice(2)
- */
+/** @param args Pass here process.argv.slice(2) */
 export async function runCreateCli(...args: string[]): Promise<CreateAppResult> {
   const pkgManager = getPackageManager();
   const templateManager = await makeTemplateManager('app');
@@ -113,8 +111,8 @@ function isStackBlitz() {
   try {
     // /home/projects/abc123
     return process.cwd().startsWith('/home/projects/');
-  } catch (e) {
-    /**/
+  } catch {
+    // ignore
   }
   return false;
 }