@smartsoft001-mobilems/claude-plugins 2.101.0 → 2.103.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@smartsoft001-mobilems/claude-plugins",
-  "version": "2.101.0",
+  "version": "2.103.0",
   "publishConfig": {
     "access": "public"
   },

package/plugins/mobilems/.claude-plugin/plugin.json

@@ -1,5 +1,5 @@
 {
   "name": "mobilems",
   "description": "MobilEMS-specific Claude Code skills (MuzaCMS / MobileMS indexer API)",
-  "version": "2.101.0"
+  "version": "2.103.0"
 }

package/plugins/mobilems/skills/mobilems-project-init/references/app-shell.md

@@ -4,41 +4,79 @@ Build the root `App` component, the static pages, and the app routing by copying
 `apps/web/src/app` and threading `<PREFIX>`. The foundation owns the shell, the static pages, and the
 routing table's static routes; each feature module skill adds its own routes.
 
-## App root — `app.ts` / `app.html` / `app.scss`
+## App root — `app.ts` / `app.html`
 
-Copy verbatim and thread `<PREFIX>`. The root component imports the `COMPONENTS` array +
-`SkipLinksComponent` from `@<PREFIX>/angular`, injects `ConfigsFacade` + `SeoService` (from
-`@smartsoft001-mobilems/angular`), `LocalStorageService` (shared lib) and `Router`; it inits the
-language, loads the global `Config` in `ngOnInit`, and focuses `#skip-links` on each `NavigationEnd`.
+The shell lives in the framework (`AppComponent`, selector `smart-mobilems-app`). The root
+component is a declaration only — thread `<PREFIX>`:
 
-> **QA gotcha (skip links):** the copied skip-link labels/targets describe the _reference_ museum's
-> sections — a project shipped skip links that still said "Światowid". Rewrite them for this app's
-> actual sections and localize them (PL/EN keys). See `references/qa-hardening.md` §1.
-> `app.html`:
+```ts
+import { ChangeDetectionStrategy, Component } from '@angular/core';
+import { AppComponent } from '@smartsoft001-mobilems/angular';
+
+import {
+  ConfirmationModalComponent,
+  FooterComponent,
+  HeaderComponent,
+} from '@<PREFIX>/angular';
+
+@Component({
+  imports: [
+    AppComponent,
+    HeaderComponent,
+    FooterComponent,
+    ConfirmationModalComponent,
+  ],
+  selector: 'app-root',
+  templateUrl: './app.html',
+  changeDetection: ChangeDetectionStrategy.Eager,
+})
+export class App {}
+```
+
+`app.html`:
 
 ```html
-<app-skip-links></app-skip-links>
-<app-app [options]="options">
-  <header tabindex="-1"><app-header></app-header></header>
-  <footer><app-footer></app-footer></footer>
-</app-app>
+<smart-mobilems-app>
+  <header tabindex="-1"><app-header /></header>
+  <footer><app-footer /></footer>
+  <app-confirmation-modal />
+</smart-mobilems-app>
 ```
 
+Language init, the global `Config` load, the `app-version` meta tag, skip links and
+focus-after-navigation all come from `provideMobilemsApp()` in `app.config.ts`
+(`references/config-providers.md`). Do **not** copy `skip-links`, `local-storage`,
+`configs-cache`, `components/app` or `route-focus` from the reference app — they are framework
+code now.
+
+> **QA gotcha (skip links):** the skip-link targets are the `skipLinks.home` / `skipLinks.subpage`
+> entries of `provideMobilemsApp()`. They must describe _this_ museum's sections and be localized
+> (PL/EN keys) — a project shipped skip links that still said "Światowid". The framework adds the
+> nav and footer links itself, so the header's main navigation MUST carry `id="main-nav" tabindex="-1"`
+> and the footer `id="main-footer" tabindex="-1"`; every other `targetId` must exist on the page it is
+> listed for — a missing id is a silent no-op. See `references/qa-hardening.md` §1.
+
 ## Static pages
 
-Copy `pages/{home,page,contact,search,not-found}` + the `home` and `page` resolvers, and thread
-`<PREFIX>`. These use only foundation shared components:
-
-- `home.component` uses the foundation carousels (`NewObjectsCarouselComponent`,
-  `NewsCarouselComponent`, `PartnersCarouselComponent`) and the `IObject`/`IArticle`/`IPartner` types
-  from the shared `models/` dir, plus the `DataService` home loaders — all foundation code (see
-  `references/shared-lib.md`). The home page renders fully; its carousels populate from the API at
-  runtime. **QA gotchas:** home sections must reuse the SAME shared card components as the module
-  list pages (an ad-hoc home card was a High bug); Swiper `loop` only when there are enough slides
-  for `slidesPerView` (else a console warning on every load); don't fire the same home API call
-  multiple times on load (`object-of-the-week` went out 3× — share/replay it). See
-  `references/qa-hardening.md` §6–7.
-- `search.component` renders results through a `<router-outlet>`; it has no feature import.
+The framework ships the bases; the app keeps the templates (design) and the data:
+
+- `home.component` extends `HomeBaseComponent`: `page` (CMS slug from `provideMobilemsApp({ homePage })`)
+  and `applySeo()` come from the base; the template is `<h1 class="smart-sr-only">` +
+  `<smart-mobilems-page-sections [page]="page" (pageLoaded)="applySeo($event)" />` and the component
+  provides `{ provide: PAGE_SECTION_COMPONENTS_TOKEN, useValue: PAGE_SECTION_COMPONENTS }` (the museum's
+  section variants, `references/shared-lib.md`). No hard-coded home layout.
+- `page.component` extends `PageStaticComponent`: provides `PAGE_SECTION_COMPONENTS_TOKEN` and
+  `PageHeadingOwner`; the template renders `loading()`, `loadFailed()` (→ `app-error-retry (retry)="retry()"`),
+  the breadcrumb declaration (`pageRoute ?? '/strona/' + id + '/' + (title | smartSlug)`), a hidden `h1`
+  unless `sectionsHaveH1()`, and `<smart-mobilems-page-sections [sections]="pageData.sections ?? null">`
+  with `[innerHTML]="contentHtml()"` in the slot. Routes use `pageResolver` (`strona/:id`) and
+  `createSlugPageResolver('kontakt', { emptyTitleKey: 'CONTACT.pageTitle' })` with
+  `runGuardsAndResolvers: 'always'` and `data: { pageRoute: '/kontakt' }` for `/kontakt`.
+- `search.component` extends `SearchBaseComponent<'obiekty' | …>` and declares only `tabDefinitions`
+  (`{ id, labelKey: 'SEARCH.tabs.*', count: (d) => d.objects }` per module the app ships); the template
+  renders the heading, `app-tabs` with `[(activeTabId)]="activeTab"` and a `<router-outlet>`.
+- `not-found.component` extends `NotFoundComponent` and renders `<smart-mobilems-not-found-content
+  [fullHeight]="true" />` (plus the breadcrumb declaration with `url`).
 
 Run a feature-package grep over the pages you copy as a guard. **Do not copy `article-preview`** — the
 reference's `article-preview` page imports `@smartsoft001-mobilems/articles-domain` (it previews an

package/plugins/mobilems/skills/mobilems-project-init/references/config-providers.md

@@ -5,135 +5,72 @@ version). Thread `<PREFIX>`. `app.config.server.ts` and
 `app.routes.server.ts` stay as the base emitted them; `main.ts`, `main.server.ts` and `server.ts`
 are overwritten in step 5 (`references/server-env.md` §3).
 
-The imports from `@<PREFIX>/angular` (`LanguageInterceptor`, `PreviewAuthInterceptor`,
-`SsrTimeoutInterceptor`, `LocalStorageService`, `EnvironmentService`) come from the shared lib
-populated in step 7 — so in **foundation-only** mode, omit the three interceptor providers and the
-`@<PREFIX>/angular` interceptor imports (they don't exist yet).
+The framework provides the whole provider set (`provideMobilemsCore()`: hydration, zoneless, router,
+HttpClient + the MobilEMS interceptors, NgRx, Masonry, ngx-translate, `SharedModule.forRoot`) and the app
+plumbing (`provideMobilemsApp()`: language, translations fetch, config load, SEO constants, skip links,
+focus). Only `environment` / `EnvironmentService` come from the shared lib (step 7). Museum-specific
+interceptors (e.g. a JWT one) go through `provideMobilemsCore({ interceptors: [...] })`.
 
 ```ts
-import {
-  HTTP_INTERCEPTORS,
-  HttpClient,
-  provideHttpClient,
-  withFetch,
-  withInterceptorsFromDi,
-} from '@angular/common/http';
 import {
   ApplicationConfig,
-  Injectable,
-  provideBrowserGlobalErrorListeners,
-  provideZonelessChangeDetection,
-  importProvidersFrom,
   inject,
-  provideEnvironmentInitializer,
   isDevMode,
-  provideAppInitializer,
+  provideEnvironmentInitializer,
 } from '@angular/core';
-import {
-  provideClientHydration,
-  withEventReplay,
-  withI18nSupport,
-} from '@angular/platform-browser';
-import { provideRouter, withInMemoryScrolling } from '@angular/router';
-import { EffectsModule, provideEffects } from '@ngrx/effects';
-import { provideStore, StoreModule } from '@ngrx/store';
 import { provideStoreDevtools } from '@ngrx/store-devtools';
-import {
-  InterpolatableTranslationObject,
-  TranslateModule,
-  TranslateService,
-} from '@ngx-translate/core';
-import {
-  AppService,
-  ErrorService,
-  NgrxSharedModule,
-  StyleService,
-} from '@smartsoft001/angular';
 import {
   ISearchData,
+  provideMobilemsApp,
+  provideMobilemsCore,
   SEARCH_CONFIG_TOKEN,
-  SharedModule,
 } from '@smartsoft001-mobilems/angular';
-import { MasonryModule } from '@thisissoon/angular-masonry';
-import { catchError, firstValueFrom, map, of, timeout } from 'rxjs';
 
-import {
-  environment,
-  EnvironmentService,
-  LanguageInterceptor,
-  LocalStorageService,
-  PreviewAuthInterceptor,
-  SsrTimeoutInterceptor,
-} from '@<PREFIX>/angular';
+import { environment, EnvironmentService } from '@<PREFIX>/angular';
 
 import { appRoutes } from './app.routes';
 
-@Injectable()
-class NoOpAppService extends AppService {
-  override initTitle(): void {
-    // No-op: disable AppService.initTitle() which overrides
-    // page titles set by components via SeoService.updateTitle()
-  }
-}
-
 export const appConfig: ApplicationConfig = {
   providers: [
-    provideClientHydration(withEventReplay(), withI18nSupport()),
-    provideBrowserGlobalErrorListeners(),
-    provideZonelessChangeDetection(),
-    provideRouter(
-      appRoutes,
-      withInMemoryScrolling({ scrollPositionRestoration: 'top' }),
-    ),
-    provideHttpClient(withFetch(), withInterceptorsFromDi()),
-    { provide: HTTP_INTERCEPTORS, useClass: LanguageInterceptor, multi: true },
-    {
-      provide: HTTP_INTERCEPTORS,
-      useClass: PreviewAuthInterceptor,
-      multi: true,
-    },
-    {
-      provide: HTTP_INTERCEPTORS,
-      useClass: SsrTimeoutInterceptor,
-      multi: true,
-    },
-    importProvidersFrom(
-      StoreModule.forRoot(
-        {},
-        {
-          runtimeChecks: {
-            strictStateImmutability: false,
-            strictActionImmutability: false,
-          },
-        },
-      ),
-      MasonryModule.forRoot(),
-      TranslateModule.forRoot(),
-      EffectsModule.forRoot([]),
-      NgrxSharedModule,
-      SharedModule.forRoot({
+    provideMobilemsCore({
+      routes: appRoutes,
+      shared: {
         apiUrl: environment.apiUrl,
         multimediaUrl: environment.multimediaUrl,
         mainLogoPath: 'logo.png',
         styleVariables: {},
-      }),
-    ),
-    { provide: StyleService, useClass: StyleService },
-    { provide: AppService, useClass: NoOpAppService },
-    {
-      provide: ErrorService,
-      useValue: {
-        log: (err: any) => {
-          console.error('Global error handler:', err);
-        },
       },
-    },
+    }),
+    provideMobilemsApp({
+      version: environment.version,
+      languages: { available: ['pl', 'en'], default: 'pl' },
+      skipLinks: {
+        home: [
+          { targetId: 'main-new-objects', translationKey: 'HOME.accessibility.skipToNewObjects' },
+        ],
+        subpage: [
+          { targetId: 'main-content', translationKey: 'HOME.accessibility.skipToContent' },
+        ],
+      },
+      // Produkcyjnie translacje są własnością CMS-a klienta (setting/theme),
+      // a lokalny plik służy developmentowi. USE_LOCAL_TRANSLATIONS wymusza
+      // plik także w buildzie produkcyjnym (zrzuty benchmarku, review makiety,
+      // demo bez backendu).
+      translations: {
+        url:
+          environment.production && !environment.useLocalTranslations
+            ? environment.apiUrl + 'setting/theme'
+            : '/api/translations',
+      },
+      seo: {
+        siteNameFallback: { pl: '<NAZWA MUZEUM>', en: '<MUSEUM NAME>' },
+        defaultOgImage: { url: '/gfx/og-default.jpg', width: 1200, height: 630 },
+        siteUrl: () => environment.siteUrl,
+      },
+    }),
     provideEnvironmentInitializer(() => {
-      const envService = inject(EnvironmentService);
-      envService.init();
+      inject(EnvironmentService).init();
     }),
-    provideStore(),
-    provideEffects(),
     provideStoreDevtools({
       maxAge: 25,
       logOnly: !isDevMode(),
@@ -154,44 +91,6 @@ export const appConfig: ApplicationConfig = {
         },
       },
     },
-    provideAppInitializer(async () => {
-      const httpClient = inject(HttpClient);
-      const translateService = inject(TranslateService);
-      const localStorageService = inject(LocalStorageService);
-
-      // Initialize language before fetching translations so the
-      // language interceptor sends the correct Accept-Language header.
-      localStorageService.initLang();
-
-      async function fetchTranslations() {
-        const url = environment.production
-          ? environment.apiUrl + 'setting/theme'
-          : '/api/translations';
-
-        const translations = (await firstValueFrom(
-          httpClient.get<{ data: any }>(url).pipe(
-            timeout(5000),
-            map((response) => {
-              return response?.data?.translations ?? { en: {}, pl: {} };
-            }),
-            catchError(() => of(null)),
-          ),
-        )) as {
-          en: InterpolatableTranslationObject;
-          pl: InterpolatableTranslationObject;
-        } | null;
-
-        if (!translations || !translations.en || !translations.pl) {
-          console.warn('Translations are missing or malformed:', translations);
-          return;
-        }
-
-        translateService.setTranslation('en', translations.en);
-        translateService.setTranslation('pl', translations.pl);
-      }
-
-      await fetchTranslations();
-    }),
   ],
 };
 ```

package/plugins/mobilems/skills/mobilems-project-init/references/shared-lib.md

@@ -48,11 +48,12 @@ FEATURE-OWNED. A typical FEATURE-OWNED set (confirm with the classifier on your
 | `news-grid-item`, `news-list-item`         | `mobilems-articles` (news)     |
 | `widget/gallery-article`                   | `mobilems-articles`            |
 
-Build everything else — `app`, `header`, `footer`, `page`, the `filter*` family, `breadcrumbs`,
-`pagination`, `searchbar`, `loader`, `error-retry`, `scroll-top`, `skip-links`, `wcag`, `gallery`,
+Build everything else — `header`, `footer`, `page`, the `filter*` family, `breadcrumbs`,
+`pagination`, `searchbar`, `loader`, `error-retry`, `scroll-top`, `wcag`, `gallery`,
 `map`, `slider`, `carousel`, the three carousels above, `category-dropdown`, `dropdown`, `menu-item`,
 `next-page`, `calendar`, `confirmation-modal`, `newsletter-*`, `image-info-panel`, `icons`, `tabs`,
-and the `widget` container with its foundation widgets.
+and the `widget` container with its foundation widgets. The `app` shell and `skip-links` are
+framework components (`smart-mobilems-app`, `smart-mobilems-skip-links`) — do not copy them.
 
 > Trust the script over this table for the **leaf** components (the grid/list cards and the
 > `widget/gallery-*` sub-widgets) — if the reference diverges, the script is authoritative there.
@@ -102,13 +103,17 @@ components duplicate exports that now exist; copying those is the single most co
 step. The app-level set the framework does **not** provide, and that the module skills expect to find
 here:
 
-| Kind         | Expected members                                                                                                                   |
-| ------------ | ---------------------------------------------------------------------------------------------------------------------------------- |
-| pipes        | `altText`, `capitalize`, `fileUrl`, `truncateHtml`                                                                                 |
-| tools        | `isInternalLink`, `normalizeUrl`, `stripHtml` (+ the shared `tools.ts` helpers)                                                    |
-| directives   | `externalLink`                                                                                                                     |
-| interceptors | `language`, `preview-auth`, `ssr-timeout`                                                                                          |
-| services     | `museum-seo`, `filter-persistence`, `local-storage`, `orientation`, `confirmation-modal`, `contact`, `data`, `environment`, `wcag` |
+| Kind         | Expected members                                                                                                  |
+| ------------ | ----------------------------------------------------------------------------------------------------------------- |
+| pipes        | `altText`, `fileUrl`, `truncateHtml` (`capitalize` is a framework pipe)                                           |
+| tools        | `isInternalLink`, `normalizeUrl`, `stripHtml` (+ the shared `tools.ts` helpers)                                   |
+| directives   | `externalLink`                                                                                                    |
+| interceptors | none — `LanguageInterceptor`, `PreviewAuthInterceptor`, `SsrTimeoutInterceptor`, `DedupeRequestInterceptor` are framework classes |
+| services     | `filter-persistence`, `orientation`, `confirmation-modal`, `contact`, `data`, `environment`, `wcag` (`museum-seo` is the framework `MuseumSeoService`) |
+
+`local-storage`, `configs-cache`, `museum-seo`, `not-found-content`, the interceptors and `decode-html-entities` are gone: language lives in the framework `TranslationService`
+(`setLanguage()` for the header switcher, `currentLanguage` for `museum-seo`), the config cache in
+`ConfigsFacade.init()`.
 
 Adjust to what the reference actually ships, but keep the rule: one implementation, in this lib,
 exported from the barrel, documented in the app's own README so the module skills can find it.