Enhance extension handling by supporting both unscoped extension-* and scoped @namespace/extension-* packages. Update documentation and tests to reflect these changes, ensuring clarity in extension management and side-effect imports.

Author: erdalkaraca

packages/app-e2e/README.md

@@ -21,7 +21,7 @@ Minimal Docks shell for Playwright only (not the public demo app in `packages/ap
 Entry is [src/main.ts](./src/main.ts): `registerApp` plus auxiliary/toolbar contributions as specs require.
 
 - **Extensions:** list them in `AppDefinition.extensions` (e.g. `@eclipse-docks/extension-ai-system`, `@eclipse-docks/extension-monaco-editor` for workspace file editing in E2E).
-- **Side-effect imports:** [vite.config.ts](./vite.config.ts) uses `resolveDepVersionsPlugin()` (automatic `@eclipse-docks/extension-*` side-effect imports are on by default). [src/main.ts](./src/main.ts) additionally imports the in-repo [`extension-ai-system/src/ai-system-extension.ts`](../../packages/extension-ai-system/src/ai-system-extension.ts) so that registration runs after the package entry and **before** the rest of `main.ts` adds E2E auxiliary tabs — order **`[aiview, …e2e tabs]`** — avoiding `wa-tab-group` “first tab” fallbacks being mistaken for successful `coupledEditors` coupling.
+- **Side-effect imports:** [vite.config.ts](./vite.config.ts) uses `resolveDepVersionsPlugin()` (automatic `extension-*` / `@scope/extension-*` side-effect imports are on by default). [src/main.ts](./src/main.ts) additionally imports the in-repo [`extension-ai-system/src/ai-system-extension.ts`](../../packages/extension-ai-system/src/ai-system-extension.ts) so that registration runs after the package entry and **before** the rest of `main.ts` adds E2E auxiliary tabs — order **`[aiview, …e2e tabs]`** — avoiding `wa-tab-group` “first tab” fallbacks being mistaken for successful `coupledEditors` coupling.
 
 ### `coupledEditors` example
 

packages/core/src/vite-plugin-resolve-deps.ts

@@ -98,9 +98,10 @@ export const VIRTUAL_EXTENSION_IMPORTS = 'virtual:eclipse-docks-extension-import
 
 const RESOLVED_VIRTUAL_EXTENSION_IMPORTS = `\0${VIRTUAL_EXTENSION_IMPORTS}`;
 
-const DEFAULT_EXTENSION_PATTERN = /^@eclipse-docks\/extension-/;
+/** Unscoped `extension-*` or scoped `@namespace/extension-*` (any npm scope). */
+const DEFAULT_EXTENSION_PATTERN = /^(?:@[^/]+\/)?extension-/;
 
-const DEFAULT_PRIORITY_FIRST = ['@eclipse-docks/extension-pwa'];
+const DEFAULT_PRIORITY_FIRST = ['@eclipse-docks/extension-pwa', 'extension-pwa'];
 
 /** Matches Vite’s default app entry in index.html. */
 const MAIN_TS_SCRIPT_RE =
@@ -119,8 +120,8 @@ export interface ExtensionSideEffectsOptions {
    */
   priorityFirst?: string[];
   /**
-   * Which direct `dependencies` keys qualify as Eclipse Docks extensions.
-   * @default /^@eclipse-docks\/extension-/
+   * Which direct `dependencies` keys qualify as Docks-style extensions (`extension-*`, with an optional npm scope).
+   * @default /^(?:@[^/]+\/)?extension-/
    */
   packageNamePattern?: RegExp;
 }

packages/core/test/units/vite-plugin-resolve-deps.test.ts

@@ -16,8 +16,8 @@ const minimalCtx = {
 
 const defaultSideEffects = {
   exclude: new Set<string>(),
-  priorityFirst: ['@eclipse-docks/extension-pwa'],
-  pattern: /^@eclipse-docks\/extension-/,
+  priorityFirst: ['@eclipse-docks/extension-pwa', 'extension-pwa'],
+  pattern: /^(?:@[^/]+\/)?extension-/,
 };
 
 describe('listExtensionSideEffectPackages', () => {
@@ -59,6 +59,20 @@ describe('listExtensionSideEffectPackages', () => {
       }),
     ).toEqual(['@eclipse-docks/extension-b', '@eclipse-docks/extension-a']);
   });
+
+  it('matches unscoped extension-* and sorts with scoped packages', () => {
+    const deps = {
+      'extension-zebra': '*',
+      '@eclipse-docks/extension-a': '*',
+      'extension-mine': '*',
+      'not-extension': '*',
+    };
+    expect(listExtensionSideEffectPackages(deps, defaultSideEffects)).toEqual([
+      '@eclipse-docks/extension-a',
+      'extension-mine',
+      'extension-zebra',
+    ]);
+  });
 });
 
 function runConfigResolved(plugin: Plugin, root: string) {

packages/create-app/template/README.md

@@ -23,7 +23,7 @@ Then open the URL shown in the terminal (e.g. https://localhost:5173/).
 
 ## Extensions
 
-Add `@eclipse-docks/extension-*` packages to `packages/app/package.json`, then list them in `packages/app/src/main.ts` (`registerApp({ extensions: [...] })`). Vite loads their side-effect entry points automatically via `resolveDepVersionsPlugin()` in `vite.config.ts` (no separate `extensions.ts` file). Pass `extensionSideEffects: false` if you need to disable that.
+Add packages named `extension-*` or `@your-scope/extension-*` to `packages/app/package.json`, then list them in `packages/app/src/main.ts` (`registerApp({ extensions: [...] })`). Vite loads their side-effect entry points automatically via `resolveDepVersionsPlugin()` in `vite.config.ts` (no separate `extensions.ts` file). Pass `extensionSideEffects: false` if you need to disable that.
 
 ## PWA
 

packages/create-app/template/packages/app/src/main.ts

@@ -1,6 +1,6 @@
 import { appLoaderService, contributionRegistry, type HTMLContribution, TOOLBAR_MAIN } from '@eclipse-docks/core';
 
-/** Which extensions the shell offers; keep in sync with `@eclipse-docks/extension-*` entries in package.json (those are auto side-effect-imported in vite.config.ts). */
+/** Which extensions the shell offers; keep in sync with `extension-*` / `@scope/extension-*` entries in package.json (auto side-effect-imported via resolveDepVersionsPlugin). */
 contributionRegistry.registerContribution(TOOLBAR_MAIN, {
   label: 'Brand',
   slot: 'start',

packages/create-app/template/packages/app/vite.config.ts

@@ -22,7 +22,7 @@ export default defineConfig({
   },
   plugins: [
     appSplashPlugin(),
-    /** Side-effect-imports every `@eclipse-docks/extension-*` direct dependency from package.json (virtual module before main). */
+    /** Side-effect-imports every `extension-*` / `@scope/extension-*` direct dependency from package.json (virtual module before main). */
     resolveDepVersionsPlugin(),
     localAliasesPlugin({
       useSrcInDev: true,