chore: fix plugin types

they somehow got broken by updates

Author: wmertens

Diff for: packages/docs/src/routes/api/qwik-optimizer/api.json

@@ -484,7 +484,7 @@
         }
       ],
       "kind": "Interface",
-      "content": "```typescript\nexport interface QwikVitePlugin \n```\n\n\n<table><thead><tr><th>\n\nProperty\n\n\n</th><th>\n\nModifiers\n\n\n</th><th>\n\nType\n\n\n</th><th>\n\nDescription\n\n\n</th></tr></thead>\n<tbody><tr><td>\n\n[api](#)\n\n\n</td><td>\n\n\n</td><td>\n\n[QwikVitePluginApi](#qwikvitepluginapi)\n\n\n</td><td>\n\n\n</td></tr>\n<tr><td>\n\n[name](#)\n\n\n</td><td>\n\n\n</td><td>\n\n'vite-plugin-qwik'\n\n\n</td><td>\n\n\n</td></tr>\n</tbody></table>",
+      "content": "```typescript\nexport interface QwikVitePlugin extends P<QwikVitePluginApi> \n```\n**Extends:** P&lt;[QwikVitePluginApi](#qwikvitepluginapi)<!-- -->&gt;\n\n\n<table><thead><tr><th>\n\nProperty\n\n\n</th><th>\n\nModifiers\n\n\n</th><th>\n\nType\n\n\n</th><th>\n\nDescription\n\n\n</th></tr></thead>\n<tbody><tr><td>\n\n[name](#)\n\n\n</td><td>\n\n\n</td><td>\n\n'vite-plugin-qwik'\n\n\n</td><td>\n\n\n</td></tr>\n</tbody></table>",
       "editUrl": "https://github.com/QwikDev/qwik/tree/main/packages/qwik/src/optimizer/src/plugins/vite.ts",
       "mdFile": "qwik.qwikviteplugin.md"
     },

Diff for: packages/docs/src/routes/api/qwik-optimizer/index.md

@@ -2216,9 +2216,11 @@ _(Optional)_
 ## QwikVitePlugin
 
 ```typescript
-export interface QwikVitePlugin
+export interface QwikVitePlugin extends P<QwikVitePluginApi>
 ```
 
+**Extends:** P&lt;[QwikVitePluginApi](#qwikvitepluginapi)&gt;
+
 <table><thead><tr><th>
 
 Property
@@ -2238,19 +2240,6 @@ Description
 </th></tr></thead>
 <tbody><tr><td>
 
-[api](#)
-
-</td><td>
-
-</td><td>
-
-[QwikVitePluginApi](#qwikvitepluginapi)
-
-</td><td>
-
-</td></tr>
-<tr><td>
-
 [name](#)
 
 </td><td>

Diff for: packages/qwik-city/package.json

@@ -7,11 +7,11 @@
     "@mdx-js/mdx": "^3.0.1",
     "@types/mdx": "^2.0.13",
     "source-map": "^0.7.4",
-    "svgo": "^3.2.0",
+    "svgo": "3.3.2",
     "undici": "*",
     "vfile": "6.0.1",
-    "vite": "^5.2.10",
-    "vite-imagetools": "^6.2.9",
+    "vite": "5.3.4",
+    "vite-imagetools": "7.0.4",
     "zod": "^3.22.4"
   },
   "devDependencies": {

Diff for: packages/qwik-city/src/adapters/shared/api.md

@@ -38,7 +38,7 @@ export const STATIC_PATHS_ID = "@qwik-city-static-paths";
 // Warning: (ae-forgotten-export) The symbol "ViteAdapterPluginOptions" needs to be exported by the entry point index.d.ts
 //
 // @public (undocumented)
-export function viteAdapter(opts: ViteAdapterPluginOptions): Plugin_2<any>;
+export function viteAdapter(opts: ViteAdapterPluginOptions): Plugin_2<never>;
 
 // (No @packageDocumentation comment for this package)
 

Diff for: packages/qwik-city/src/adapters/shared/vite/index.ts

@@ -21,7 +21,7 @@ export function viteAdapter(opts: ViteAdapterPluginOptions) {
   let format = 'esm';
   const outputEntries: string[] = [];
 
-  const plugin: Plugin = {
+  const plugin: Plugin<never> = {
     name: `vite-plugin-qwik-city-${opts.name}`,
     enforce: 'post',
     apply: 'build',

Diff for: packages/qwik-city/src/buildtime/vite/api.md

@@ -8,6 +8,7 @@ import type { BuiltinsWithOptionalParams } from 'svgo/plugins/plugins-types';
 import type { CompileOptions } from '@mdx-js/mdx';
 import type { Config } from 'svgo';
 import { ConfigEnv } from 'vite';
+import type { Plugin as Plugin_2 } from 'vite';
 import type { PluginOption } from 'vite';
 import { UserConfigExport } from 'vite';
 
@@ -20,12 +21,11 @@ export type MdxOptions = CompileOptions;
 // @public (undocumented)
 export function qwikCity(userOpts?: QwikCityVitePluginOptions): PluginOption[];
 
+// Warning: (ae-forgotten-export) The symbol "P" needs to be exported by the entry point index.d.ts
+// Warning: (ae-forgotten-export) The symbol "QwikCityPluginApi" needs to be exported by the entry point index.d.ts
+//
 // @public (undocumented)
-export interface QwikCityPlugin {
-    // Warning: (ae-forgotten-export) The symbol "QwikCityPluginApi" needs to be exported by the entry point index.d.ts
-    //
-    // (undocumented)
-    api: QwikCityPluginApi;
+export interface QwikCityPlugin extends P<QwikCityPluginApi> {
     // (undocumented)
     name: 'vite-plugin-qwik-city';
 }

Diff for: packages/qwik-city/src/buildtime/vite/plugin.ts

@@ -57,7 +57,9 @@ function qwikCityPlugin(userOpts?: QwikCityVitePluginOptions): any {
     },
   };
 
-  const plugin: Plugin = {
+  type P<T> = Plugin<T> & { api: T };
+
+  const plugin: P<QwikCityPluginApi> = {
     name: 'vite-plugin-qwik-city',
     enforce: 'pre',
     api,

Diff for: packages/qwik-city/src/buildtime/vite/types.ts

@@ -3,6 +3,7 @@ import type { MdxTransform } from '../markdown/mdx';
 import type { BuildContext, BuildEntry, BuildRoute, PluginOptions, MdxPlugins } from '../types';
 import type { Config as SVGOConfig } from 'svgo';
 import type { BuiltinsWithOptionalParams as SVGOBuiltinPluginsWithOptionalParams } from 'svgo/plugins/plugins-types';
+import type { Plugin as VitePlugin } from 'vite';
 
 /** @public */
 export interface ImageOptimizationOptions {
@@ -39,10 +40,11 @@ export interface PluginContext {
   mdxTransform: MdxTransform | null;
 }
 
+type P<T> = VitePlugin<T> & { api: T };
+
 /** @public */
-export interface QwikCityPlugin {
+export interface QwikCityPlugin extends P<QwikCityPluginApi> {
   name: 'vite-plugin-qwik-city';
-  api: QwikCityPluginApi;
 }
 
 /** @public */

Diff for: packages/qwik-react/package.json

@@ -35,11 +35,11 @@
   "license": "MIT",
   "main": "./lib/index.qwik.mjs",
   "peerDependencies": {
-    "@builder.io/qwik": "workspace:^",
-    "@types/react": "^18.2.79",
-    "@types/react-dom": "^18.2.25",
-    "react": "^18.2.0",
-    "react-dom": "^18.2.0"
+    "@builder.io/qwik": "1.7.3",
+    "@types/react": "18.3.3",
+    "@types/react-dom": "18.3.0",
+    "react": "18.3.1",
+    "react-dom": "18.3.1"
   },
   "qwik": "./lib/index.qwik.mjs",
   "repository": {

Diff for: packages/qwik-worker/package.json

@@ -24,7 +24,7 @@
   "license": "MIT",
   "main": "./lib/index.qwik.mjs",
   "peerDependencies": {
-    "@builder.io/qwik": "workspace:^"
+    "@builder.io/qwik": "1.7.3"
   },
   "qwik": "./lib/index.qwik.mjs",
   "repository": {

Diff for: packages/qwik/src/optimizer/src/api.md

@@ -4,6 +4,8 @@
 
 ```ts
 
+import type { Plugin as Plugin_2 } from 'vite';
+
 // @public (undocumented)
 export interface ComponentEntryStrategy {
     // (undocumented)
@@ -296,10 +298,10 @@ export interface QwikViteDevResponse {
     _qwikRenderResolve?: () => void;
 }
 
+// Warning: (ae-forgotten-export) The symbol "P" needs to be exported by the entry point index.d.ts
+//
 // @public (undocumented)
-export interface QwikVitePlugin {
-    // (undocumented)
-    api: QwikVitePluginApi;
+export interface QwikVitePlugin extends P<QwikVitePluginApi> {
     // (undocumented)
     name: 'vite-plugin-qwik';
 }

Diff for: packages/qwik/src/optimizer/src/plugins/rollup.ts

@@ -7,6 +7,7 @@ import type {
   QwikManifest,
   TransformModuleInput,
   TransformModule,
+  Optimizer,
 } from '../types';
 import {
   createPlugin,
@@ -18,6 +19,11 @@ import {
 } from './plugin';
 import { versions } from '../versions';
 
+type QwikRollupPluginApi = {
+  getOptimizer: () => Optimizer;
+  getOptions: () => NormalizedQwikPluginOptions;
+};
+
 /** @public */
 export function qwikRollup(qwikRollupOpts: QwikRollupPluginOptions = {}): any {
   const qwikPlugin = createPlugin(qwikRollupOpts.optimizerOptions);
@@ -323,5 +329,5 @@ export interface QwikRollupPluginOptions {
    */
   lint?: boolean;
 }
-
-export interface QwikRollupPlugin extends Rollup.Plugin {}
+type P<T> = Rollup.Plugin<T> & { api: T };
+export interface QwikRollupPlugin extends P<QwikRollupPluginApi> {}

Diff for: packages/qwik/src/optimizer/src/plugins/vite.ts

@@ -40,8 +40,15 @@ const DEDUPE = [QWIK_CORE_ID, QWIK_JSX_RUNTIME_ID, QWIK_JSX_DEV_RUNTIME_ID];
 const STYLING = ['.css', '.scss', '.sass', '.less', '.styl', '.stylus'];
 const FONTS = ['.woff', '.woff2', '.ttf'];
 
+/**
+ * Workaround to make the api be defined in the type.
+ *
+ * @internal
+ */
+type P<T> = VitePlugin<T> & { api: T; config: Extract<VitePlugin<T>['config'], Function> };
+
 /** @public */
-export function qwikVite(qwikViteOpts: QwikVitePluginOptions = {}): any {
+export function qwikVite(qwikViteOpts: QwikVitePluginOptions = {}) {
   let isClientDevOnly = false;
   let clientDevInput: undefined | string = undefined;
   let tmpClientManifestPath: undefined | string = undefined;
@@ -91,7 +98,7 @@ export function qwikVite(qwikViteOpts: QwikVitePluginOptions = {}): any {
   // Vite hooks. The second plugin is a post plugin that is called after the build has finished.
   // The post plugin is used to generate the Qwik manifest file that is used during SSR to
   // generate QRLs for event handlers.
-  const vitePluginPre: VitePlugin = {
+  const vitePluginPre: P<QwikVitePluginApi> = {
     name: 'vite-plugin-qwik',
     enforce: 'pre',
     api,
@@ -478,7 +485,7 @@ export function qwikVite(qwikViteOpts: QwikVitePluginOptions = {}): any {
     },
   };
 
-  const vitePluginPost: VitePlugin = {
+  const vitePluginPost: VitePlugin<never> = {
     name: 'vite-plugin-qwik-post',
     enforce: 'post',
 
@@ -729,7 +736,7 @@ export function qwikVite(qwikViteOpts: QwikVitePluginOptions = {}): any {
     },
   };
 
-  return [vitePluginPre, vitePluginPost];
+  return [vitePluginPre, vitePluginPost] as const;
 }
 
 const ANSI_COLOR = {
@@ -1063,9 +1070,8 @@ export interface QwikVitePluginApi {
 }
 
 /** @public */
-export interface QwikVitePlugin {
+export interface QwikVitePlugin extends P<QwikVitePluginApi> {
   name: 'vite-plugin-qwik';
-  api: QwikVitePluginApi;
 }
 
 /** @public */

Diff for: packages/qwik/src/optimizer/src/plugins/vite.unit.ts

@@ -320,7 +320,7 @@ test('should use the dist/ fallback with client target', async () => {
   const plugin = qwikVite(initOpts)[0];
   const c: any = (await plugin.config(
     { build: { assetsDir: 'my-assets-dir/' } },
-    { command: 'serve' }
+    { command: 'serve', mode: 'development' }
   ))!;
 
   assert.equal(c.build.outDir, normalizePath(resolve(cwd, `dist`)));
@@ -333,7 +333,7 @@ test('should use build.outDir config with client target', async () => {
   const plugin = qwikVite(initOpts)[0];
   const c: any = (await plugin.config(
     { build: { outDir: 'my-dist/', assetsDir: 'my-assets-dir' } },
-    { command: 'serve' }
+    { command: 'serve', mode: 'development' }
   ))!;
 
   assert.equal(c.build.outDir, normalizePath(resolve(cwd, `my-dist`)));
@@ -347,7 +347,10 @@ test('should use build.outDir config when assetsDir is _astro', async () => {
   const plugin = qwikVite(initOpts)[0];
 
   // Astro sets a build.assetsDir of _astro, but we don't want to change that
-  const c: any = (await plugin.config({ build: { assetsDir: '_astro' } }, { command: 'serve' }))!;
+  const c: any = (await plugin.config(
+    { build: { assetsDir: '_astro' } },
+    { command: 'serve', mode: 'development' }
+  ))!;
 
   assert.equal(c.build.outDir, normalizePath(resolve(cwd, `dist/`)));
 });