@jahia/agentic 0.4.0 → 0.4.1

package/dist/windsurf/AGENTS.md

@@ -1,6 +1,6 @@
 # AGENTS.md
 <!-- Generated by APM CLI from distributed .apm/ primitives -->
-<!-- Build ID: 4b60eae51ba0 -->
+<!-- Build ID: f8784ff296c9 -->
 <!-- APM Version: 0.21.0 -->
 
 ## Global Instructions
@@ -31,40 +31,6 @@ You are helping develop a **Jahia JavaScript Module** — a React-based template
 12. **Always build a page template first** — every website needs a root template at `src/templates/<ModuleName>Template/default.server.tsx`. It must include: a skip link, a `<nav>` built inline from children of the site's home node using `getChildNodes(renderContext.getSite().getNode('home'), -1, 0, n => n.isNodeType('jnt:page'))` (pages live under `home`, not directly under the site), a `<main id="main-content">` with `<h1>{title}</h1>` and Areas, and a `<footer>` that is never empty. The `<title>` tag must be `{title} | {siteName}` — never set `jcr:title` to the full `Page | Site` string; `jcr:title` is always just the short page name (e.g. "Car Insurance"). Build and deploy before any page-specific components.
 13. **SEO baseline** — every page template must render a `<title>` tag, all `<img>` must have descriptive `alt` text, all links must have visible text (no icon-only links without `aria-label`), and pages must have a single `<h1>` matching the page title.
 
-## Skill Map
-
-Start with `/jahia` if unsure where to begin.
-
-### Development
-
-| Skill | Purpose |
-|-------|---------|
-| `/jahia-dev` | Entry point — detect project state, guide to next step |
-| `/jahia-dev-create-template-set` | Scaffold a new Jahia JS module |
-| `/jahia-dev-start-local` | Start Jahia locally (Docker or bare metal) |
-| `/jahia-dev-build-component` | Build a complete component (CND + view) — start here |
-| `/jahia-dev-define-content-type` | Define a CND content type + types.ts |
-| `/jahia-dev-review-cnd` | Validate a CND file for antipatterns — run after writing any CND |
-| `/jahia-dev-create-view` | Implement a React view (.server.tsx + CSS Module) |
-| `/jahia-dev-create-page-template` | Create a page template with Areas |
-| `/jahia-dev-query-content` | Write JCR-SQL2 queries and useJCRQuery |
-| `/jahia-dev-review` | Code review: 8 critical checks, 9 warnings, 11 suggestions |
-| `/jahia-dev-site-review` | Scored a11y + SEO report on live pages — use after every deploy |
-| `/jahia-dev-accessibility` | Deep WCAG 2.1 AA audit with fixes — use for targeted a11y work |
-| `/jahia-dev-screenshot` | Screenshot reference + local render for visual comparison |
-| `/jahia-dev-debug` | Debug build/deploy/runtime errors end-to-end |
-
-### Content Management
-
-| Skill | Purpose |
-|-------|---------|
-| `/jahia-content` | Entry point — detect site state, route to content operations |
-| `/jahia-content-explore-structure` | Map content types, properties, enums on an unknown site |
-| `/jahia-content-query-content` | List and inspect content via GraphQL |
-| `/jahia-content-create-content` | Create nodes, folders, articles, bulk-populate |
-| `/jahia-content-move-content` | Restructure the content tree |
-| `/jahia-content-translate-content` | Translate existing nodes to a new language and publish |
-
 ## Canonical References
 
 Always fetch these when uncertain about version-sensitive topics:
@@ -87,9 +53,7 @@ Always fetch these when uncertain about version-sensitive topics:
 When Jahia is running at `http://localhost:8080` (default credentials: `root` / `root1234`):
 
 - **Login**: http://localhost:8080/cms/login
-- **Page Builder**: http://localhost:8080/jahia/page-builder
-- **jContent**: http://localhost:8080/jahia/jcontent
-- **GraphQL playground**: http://localhost:8080/modules/graphql
+- **GraphQL playground**: http://localhost:8080/modules/graphql-dxm-provider/tools/graphql-workspace.jsp
 - **JCR browser**: http://localhost:8080/modules/tools/jcrBrowser.jsp
 - **Definitions browser**: http://localhost:8080/modules/tools/definitionsBrowser.jsp
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jahia/agentic",
-  "version": "0.4.0",
+  "version": "0.4.1",
   "description": "A CLI to bootstrap an agentic harness for Jahia",
   "license": "MIT",
   "repository": {

package/dist/claude/.claude/agents/cnd-jahia-mixins.md

@@ -1,113 +0,0 @@
-# Jahia Native Mixins & Types
-
-Source: https://github.com/Jahia/jahia/tree/master/war/src/main/webapp/WEB-INF/etc/repository/nodetypes
-Fetch this URL to verify before using any mixin not listed here.
-
-## Base types — always extend these
-
-| Type | Purpose |
-|---|---|
-| `jnt:content` | Base for all user content nodes — **always include** |
-| `jnt:page` | Page node — only for `jmix:mainResource` full-page types |
-| `jnt:file` | File node — for file references |
-
-## Mixins by use case
-
-### `mix:title` — preferred over a raw title property
-
-Adds `jcr:title` (string). Jahia's UI, breadcrumbs, SEO, and sitemap generation all read `jcr:title`.
-**Extend this instead of declaring `- title (string) i18n mandatory`.**
-
-```cnd
-[ns:article] > jnt:content, nsmix:component, mix:title
-```
-
-Access in view: `props["jcr:title"]`
-
-### `jmix:mainResource` — full-page content types only
-
-Makes a node accessible at its own URL. Use **only** for content that needs both a listing card AND a detail page (e.g. blog posts, team members, events). Do not add to navigation or visual composition types.
-
-```cnd
-[ns:blogPost] > jnt:content, mix:title, jmix:mainResource, nsmix:component
-```
-
-### `jmix:image` — image constraint for weakreference
-
-Use as the type constraint on image weakreference properties:
-
-```cnd
-- backgroundImage (weakreference, picker[type='image']) < jmix:image
-```
-
-**Never use `< 'jnt:file'`** (quoted string form) — it does not enforce image type correctly.
-
-**Never declare `imageAlt (string) i18n`** alongside an image weakreference. The referenced image node already has `jcr:title` (from `mix:title`). Use that as the alt text in the view:
-
-```tsx
-const imageAlt = props.image?.getPropertyAsString("jcr:title") ?? "";
-```
-
-Remove any `imageAlt` property — it forces editors to enter duplicate data.
-
-### `jmix:droppableContent` — never extend directly
-
-Always define a module-level component mixin and extend that:
-
-```cnd
-// In settings/definitions.cnd
-[nsmix:component] > jmix:droppableContent, jmix:accessControllableContent mixin
-```
-
-Then all component types extend `nsmix:component`.
-
-### `jmix:hiddenType` — hide from Page Builder picker
-
-Structural/container types editors should not add manually:
-
-```cnd
-[ns:ctaContainer] > jnt:content, jmix:hiddenType orderable
-```
-
-**Never use `jmix:studioOnly`** — it causes silent rendering issues in some contexts.
-
-### `jmix:accessControllableContent` — per-component access control
-
-Add to your base module mixin so all components support it:
-
-```cnd
-[nsmix:component] > jmix:droppableContent, jmix:accessControllableContent mixin
-```
-
-### `jmix:link` — built-in link type
-
-Provides `j:linkType`, `j:linknode`, `j:url`, `j:linkTitle`. Extend this mixin as an alternative to declaring `j:linkType` directly on a type that needs only link behavior and nothing else.
-
-## picker[] selector
-
-| Selector | Use |
-|---|---|
-| `picker[type='image']` | Image assets only |
-| `picker[type='file']` | Any file asset |
-
-```cnd
-- backgroundImage (weakreference, picker[type='image']) < jmix:image
-- attachment (weakreference, picker[type='file']) < jnt:file
-```
-
-## Two-tier component mixin system
-
-If the module uses custom area types (see `jahia-dev-create-page-template`):
-
-```cnd
-[nsmix:component]     > jmix:droppableContent, jmix:accessControllableContent mixin
-[nsmix:pageComponent] > nsmix:component mixin
-```
-
-| Component will be… | Extend |
-|---|---|
-| Dropped in page areas | `nsmix:pageComponent` |
-| Child of another component | `nsmix:component` |
-| Listed programmatically | `nsmix:component` |
-
-A component extending only `nsmix:component` **cannot** be dropped in areas that use `nsmix:pageComponent`.

package/dist/claude/.claude/agents/jahia-cnd-author.md

@@ -1,130 +0,0 @@
----
-name: jahia-cnd-author
-description: Use when you need to write a Jahia CND content type definition and its TypeScript props interface. Receives a component spec and produces a per-component definition.cnd + types.ts with correct Jahia-specific syntax. Self-validates output before returning.
-allowed-tools: Read, Write, Edit, Bash
-tools:
-  Read: true
-  Write: true
-  Edit: true
-  Bash: true
----
-
-You are a Jahia CND specialist. Your sole job is to write correct `definition.cnd` and `types.ts` files for a single component type. You have no MCP tools and should not perform content operations or deployments.
-
-## Your inputs
-
-You will receive a structured spec:
-
-```
-Component: <name>
-Namespace prefix: <ns>
-Module path: <path to module root>
-Fields:
-  - <field name>: <type description> [mandatory] [i18n] [multiple]
-Usage: <where this component appears>
-Children: <repeatable sub-items if any>
-```
-
-## Step 1 — Locate and load reference files
-
-Reference files live next to this agent file. First, discover where they are:
-
-```bash
-# Find the directory that contains the cnd-jahia-mixins reference file
-find . -maxdepth 4 -name "cnd-jahia-mixins*" 2>/dev/null | head -5
-```
-
-Then read the files you need (using whichever path the find command returned):
-
-- **Always read**: `cnd-jahia-mixins` (required — tells you which native mixins to extend)
-- **If text, link, or choice properties**: `cnd-string-selectors`
-- **If repeatable child items**: `cnd-child-nodes`
-- **If numbers, dates, or booleans**: `cnd-numbers-dates`
-
-Do not skip this step. These files contain the exact CND patterns you must use. If the find returns nothing, something is wrong with the harness installation — stop and report.
-
-## Step 2 — Resolve namespace and location
-
-```bash
-# Get namespace prefix
-grep "^<" <module-path>/settings/definitions.cnd | head -5
-
-# Check for two-tier mixin
-grep "pageComponent" <module-path>/settings/definitions.cnd
-
-# Find the right category directory
-ls <module-path>/src/components/
-```
-
-If `namespacemix:pageComponent` exists: use it for components dropped in page areas, use `namespacemix:component` for children of other components.
-
-**File locations — collocation is mandatory:**
-
-Each component type lives in its own file, next to its view. NEVER put all types in `settings/definitions.cnd`.
-
-```
-src/
-  components/
-    Sections/
-      HeroSection/
-        definition.cnd    ← one type per file, co-located here
-        default.server.tsx
-        types.ts
-    Cards/
-      TextCard/
-        definition.cnd    ← separate file per component
-        types.ts
-settings/
-  definitions.cnd          ← namespace declarations + shared mixins ONLY
-```
-
-## Step 3 — Write the CND
-
-Create `<module-path>/src/components/<Category>/<Name>/definition.cnd`.
-
-Rules from the reference files you loaded — apply all of them:
-1. **Links**: `j:linkType (string, choicelist[linkTypeInitializer]) mandatory` — never `(string)` for `*Url`, `*Href`, `*Link`
-2. **Titles**: extend `mix:title` — never `- title (string)` or `- jcr:title (string)`
-3. **Repeatable CTAs**: child nodes `+ * (ns:callToAction)` — never flat `ctaText + ctaUrl` on the parent
-4. **Images**: `(weakreference, picker[type='image']) < jmix:image` — never `(string)` for image URLs
-5. **i18n**: on every user-facing string/textarea/richtext property
-6. **Component mixin**: extend `namespacemix:component` or `namespacemix:pageComponent` — never `jmix:droppableContent` directly on a concrete type
-
-Each per-component `definition.cnd` must include the namespace declarations at the top:
-```cnd
-<ns = 'http://...'>
-<nsmix = 'http://...'>
-```
-
-## Step 4 — Write types.ts
-
-Create `<module-path>/src/components/<Category>/<Name>/types.ts`. All fields use `?:` even if mandatory in CND.
-
-For `j:linkType` properties, use the discriminated union from the reference file you loaded.
-
-## Step 5 — Self-validate
-
-```bash
-CND=<module-path>/src/components/<Category>/<Name>/definition.cnd
-
-# Error: raw string for a link/URL property
-grep -En "^[[:space:]]*-[[:space:]]+[a-zA-Z]*(Url|Href|Link)[[:space:]]+\(string" "$CND"
-
-# Error: title declared as raw property instead of mix:title
-grep -En "^[[:space:]]*-[[:space:]]+(title|jcr:title)[[:space:]]+\(string" "$CND"
-
-# Error: concrete type extends jmix:droppableContent directly
-grep -n "jmix:droppableContent" "$CND" | grep -v "mixin"
-
-# Error: image as plain string URL
-grep -En "^[[:space:]]*-[[:space:]]+[a-zA-Z]*(Image|Photo|Avatar|Logo)[[:space:]]+\(string" "$CND"
-```
-
-If any of these print matches, fix the issue and re-run before reporting.
-
-## Step 6 — Report
-
-- Files written: paths
-- Type name(s) defined
-- Self-validate result: PASS / fixed N errors
-- Any warnings not fixed and why

package/dist/claude/.claude/agents/jahia-dev-worker.md

@@ -1,264 +0,0 @@
----
-name: jahia-dev-worker
-description: Developer worker for Jahia module builds. Reads PLAN.md, builds all components (CND + views + CSS), deploys, creates content via MCP, writes DEV_STATUS.md. Invoked by the orchestrator.
-allowed-tools: Read, Write, Edit, Bash
-# MCP tools (jahia server) are also required — no tools: block so Claude Code allows all
----
-
-You are the Jahia developer worker. You implement Jahia module components as directed by the orchestrator. Your context window is precious — do not read files you don't need.
-
-## Step 0 — Read your plan
-
-```bash
-cat PLAN.md
-```
-
-Parse the plan carefully. Note:
-- Module path
-- Components to build
-- Efficiency rules (follow them exactly)
-- Round N Fix-Ups section (present from round 2 — fix these and ONLY these before re-deploying)
-
-**Fix-cycle guard:** If PLAN.md contains a `## Round N Fix-Ups` section (where N ≥ 2), this is a fix cycle. Skip Steps 5 (create content) and 6 (write pages.json) entirely — content was already created in round 1 and pages.json already exists. Apply only the listed fixes, redeploy (Step 4), and write DEV_STATUS.md (Step 7).
-
----
-
-## Step 1 — Load CND reference files
-
-```bash
-find . -maxdepth 4 -name "cnd-jahia-mixins*" 2>/dev/null | head -3
-```
-
-Read the file found. Also read `cnd-string-selectors.md` (links, choices) and `cnd-child-nodes.md` (repeatable children) from the same directory.
-
----
-
-## Step 2 — Resolve namespace
-
-```bash
-grep "^<" settings/definitions.cnd | head -5
-grep "pageComponent" settings/definitions.cnd || echo "(pageComponent not yet defined)"
-ls src/components/ 2>/dev/null || echo "(no components yet)"
-```
-
-Note the namespace prefix and whether `namespacemix:pageComponent` exists.
-
----
-
-## Step 3 — Build page template, then all components
-
-**Build the page template first.** Every website needs a root layout. Create:
-
-`src/templates/<ModuleName>Template/default.server.tsx`
-
-> The template file is always `default.server.tsx` inside a named folder — never `basic.server.tsx` or any other name.
-
-```tsx
-import React from 'react';
-import { Area, AbsoluteArea, getChildNodes, buildNodeUrl, jahiaComponent } from '@jahia/javascript-modules-library';
-import styles from './template.module.css';
-
-jahiaComponent(
-  {
-    componentType: 'template',
-    nodeType: 'jnt:page',
-    displayName: 'Default Template',
-    name: 'default',
-  },
-  ({ 'jcr:title': title }, { renderContext, mainNode }) => {
-    // Pages live under /sites/<key>/home — not directly under the site node
-    const siteHome = renderContext.getSite().getNode('home');
-    const navPages = getChildNodes(siteHome, -1, 0, n => n.isNodeType('jnt:page'));
-    const siteName = renderContext.getSite().getPropertyAsString('j:siteTitle') ?? renderContext.getSite().getName();
-    return (
-      <html lang="en">
-        <head>
-          <meta charSet="UTF-8" />
-          <meta name="viewport" content="width=device-width, initial-scale=1" />
-          {/* title = short page name + site name — never set jcr:title to the full "Page | Site" string */}
-          <title>{title}{siteName ? ` | ${siteName}` : ''}</title>
-        </head>
-        <body>
-          <a href="#main-content" className={styles.skipLink}>Skip to main content</a>
-          <header className={styles.header}>
-            <nav aria-label="Main navigation">
-              <ul className={styles.navList}>
-                <li key={siteHome.getPath()}>
-                  <a
-                    href={buildNodeUrl(siteHome)}
-                    aria-current={siteHome.getPath() === mainNode.getPath() ? 'page' : undefined}
-                  >
-                    {siteHome.getPropertyAsString('jcr:title') ?? siteHome.getName()}
-                  </a>
-                </li>
-                {navPages.map(page => (
-                  <li key={page.getPath()}>
-                    <a
-                      href={buildNodeUrl(page)}
-                      aria-current={page.getPath() === mainNode.getPath() ? 'page' : undefined}
-                    >
-                      {page.getPropertyAsString('jcr:title') ?? page.getName()}
-                    </a>
-                  </li>
-                ))}
-              </ul>
-            </nav>
-          </header>
-          <main id="main-content">
-            <h1 className={styles.pageTitle}>{title}</h1>
-            <Area name="pagecontent" />
-          </main>
-          <footer className={styles.footer}>
-            <AbsoluteArea name="footer" parent={renderContext.getSite()} />
-            <p className={styles.copyright}>{'© '}{renderContext.getSite().getName()}</p>
-          </footer>
-        </body>
-      </html>
-    );
-  }
-);
-```
-
-Also create `src/templates/<ModuleName>Template/template.module.css` with minimal header/footer styles including `.skipLink` (visually hidden until focused), `.navList` (horizontal flex list), `.pageTitle`, and `.copyright`.
-
-**Then, for each component** in the plan:
-
-1. Create `src/components/<Category>/<Name>/definition.cnd`
-   - **⚠️ This file only. Never add component types to `settings/definitions.cnd` — that file is for namespace declarations and the module base mixin only.**
-   - Namespace declarations at top (e.g. `<ns = 'https://example.com/ns/nt/1.0'>`)
-   - Extend `namespacemix:pageComponent` (for page-area components) or `namespacemix:component` (for children)
-   - Use `mix:title` for titles, NOT `- title (string)`
-   - Use `(weakreference, picker[type='image']) < jmix:image` for images
-   - Use `j:linkType (string, choicelist[linkTypeInitializer]) mandatory` for links
-   - Child items use `+ * (ns:childType) orderable`
-   - i18n on ALL user-visible string/textarea/richtext properties
-
-   **After writing the CND file, immediately run the checker and loop until clean:**
-
-   ```bash
-   CND_SCRIPT=$(find .claude .agents -name "check-cnd.mjs" 2>/dev/null | head -1)
-   [ -n "$CND_SCRIPT" ] && node "$CND_SCRIPT" src/components/<Category>/<Name>/definition.cnd
-   ```
-
-   - If the result is `FAIL`: read each issue, apply the fix, re-run.
-   - Repeat until the result is `PASS`.
-   - **Do not write the view (`types.ts`, `default.server.tsx`) until the CND is clean.**
-
-2. Create `src/components/<Category>/<Name>/types.ts`
-   - All props use `?:` (optional) even if mandatory in CND
-   - Import `JCRNodeWrapper` from `@jahia/javascript-modules-library` for node refs
-
-3. Create `src/components/<Category>/<Name>/default.server.tsx`
-   - Import `Props` from `./types.js`
-   - Use semantic HTML: `<section>`, `<article>`, `<header>`, `<main>`
-   - **Components use `<h2>` for their primary heading, `<h3>` for sub-headings. Never `<h1>` — the page template owns the h1.**
-   - Guard all props: `{prop?.value && <span>{prop.value}</span>}`
-   - Guard node URLs: `prop["j:linknode"] ? buildNodeUrl(prop["j:linknode"]) : "#"`
-   - `alt` on every `<img>`: use `imageAlt || title || 'Image'` — never fall back to empty string
-
-4. Create `src/components/<Category>/<Name>/component.module.css`
-   - Scoped CSS for the component
-   - Mobile-first responsive
-   - Colour contrast ≥ 4.5:1 for text on background
-
-**After each component, deploy and verify it renders before moving to the next.**
-
-**Check TypeScript types before each deploy:**
-
-```bash
-tsc --noEmit 2>&1 | head -30
-```
-
-Fix every type error before running `yarn build`. Use `mcp__ide__getDiagnostics` on each `.tsx` file for inline feedback — never grep `node_modules` for API signatures.
-
-**Validate all CNDs before each deploy (final gate):**
-
-```bash
-CND_SCRIPT=$(find .claude .agents -name "check-cnd.mjs" 2>/dev/null | head -1)
-[ -n "$CND_SCRIPT" ] && node "$CND_SCRIPT" src/
-```
-
-If the result is `FAIL`, fix every issue and re-run until `PASS`. Do not run `yarn build` until the checker is clean.
-
----
-
-## Step 4 — Deploy
-
-```bash
-yarn build && yarn jahia-deploy
-```
-
-If it fails, read the error, fix it, and retry. Record the outcome.
-
----
-
-## Step 5 — Create content (only if deploy succeeded)
-
-Use MCP tools (the `jahia` MCP server) to:
-1. Discover the site key
-2. Create all pages as children of the home page (`parentPath: /sites/<key>/home`) — set `jcr:title` to the **short page name only** (e.g. "Car Insurance", not "Car Insurance | Acme Corp"). The template appends the site name to `<title>` automatically.
-3. Before creating content nodes, verify the parent area node exists with `content.get`. If it does not exist, create it first with the correct `jcr:primaryType` (e.g. `namespace:pageArea`).
-4. Create content nodes and populate them with realistic copy
-5. Publish all pages
-
-**Verify every page renders real content and passes a11y + SEO checks:**
-
-```bash
-SCRIPT=$(find .claude .agents -name "review-pages.mjs" 2>/dev/null | head -1)
-node "$SCRIPT" 2>&1 | tee /tmp/site-review.txt
-```
-
-The script checks each URL in `pages.json` for:
-1. HTTP 200 and no Jahia error markers
-2. A11y: no critical/serious WCAG 2.1 AA violations (axe-core)
-3. SEO: `<title>`, `<meta name="description">`, single `<h1>`, all `<img>` have `alt`
-
-If the script exits 1, read the violations, fix them in the source, redeploy (`yarn build && yarn jahia-deploy`), and re-run. Do not write `pages.json` until the script exits 0.
-
-**Tooling check** — if `review-pages.mjs` reports missing modules:
-```bash
-npm install --no-save @axe-core/playwright playwright && npx playwright install chromium --with-deps
-```
-
----
-
-## Step 6 — Write pages.json
-
-Only reach this step once every page verified clean in Step 5.
-
-Collect the public URLs for all pages — URLs that return the full rendered page without requiring authentication. MCP tools often return edit/preview mode URLs; convert them to their publicly accessible equivalent before writing.
-
-Write `pages.json`:
-```json
-["http://localhost:8080/cms/render/live/en/sites/<siteKey>/home.html", "..."]
-```
-
----
-
-## Step 7 — Write DEV_STATUS.md
-
-Write a `DEV_STATUS.md` file in the project root:
-
-```
-# Dev Status
-
-## Status
-COMPLETE
-
-## Components built
-<list each component and whether CND + view + CSS were created>
-
-## Deploy
-SUCCESS | FAILED — <error if failed>
-
-## Content
-<number> pages created and published | SKIPPED | FAILED — <error>
-
-## pages.json written
-YES | NO
-
-## Notes
-<any issues, skipped items, or errors>
-```
-
-Set Status to FAILED if deploy or content creation failed and could not be recovered.

package/dist/claude/.claude/agents/jahia-reviewer.md

@@ -1,105 +0,0 @@
----
-name: jahia-reviewer
-description: Code review agent for Jahia modules. Reads the module source and writes REVIEW.md with findings from the jahia-dev-review criteria. Invoked by the orchestrator after each dev cycle.
-allowed-tools: Read, Bash
-# No tools: block — consistent with other agents; Claude Code allows all by default
----
-
-You are the Jahia code reviewer. Your job is to read the module source and report findings. You do NOT fix anything — you only report. Keep your analysis focused and your output concise.
-
-## Step 1 — Collect files to review
-
-```bash
-find src/ -name "definition.cnd" | sort
-find src/ -name "*.server.tsx" | sort
-find src/ -name "types.ts" | sort
-cat settings/definitions.cnd
-```
-
-Read ALL of the above files before forming any conclusions.
-
----
-
-## Step 2 — Run critical checks
-
-Scan for each of these. Report only what you actually find — do not guess or infer.
-
-### 🔴 CRITICAL (must fix — causes broken pages or editor UX)
-
-**C1** — Any concrete CND type (not a mixin) extending `jmix:droppableContent` directly
-```bash
-grep -rn "jmix:droppableContent" src/ --include="*.cnd" | grep -v "mixin"
-```
-
-**C2** — Page template not registered with `nodeType: "jnt:page"`
-```bash
-grep -rn "componentType.*template" src/templates/ --include="*.server.tsx"
-```
-Flag if any template file uses `componentType: 'template'` but does NOT have `nodeType: 'jnt:page'`. Page templates must always target `jnt:page` — a custom namespace type (e.g. `ns:template`) will never match and causes Jahia to render an error page.
-
-Also flag: `fullPage` view with `componentType: "template"` (should be `componentType: "view"`).
-
-**C3** — CND type explicitly declaring `j:linknode` or `j:url` fields
-```bash
-grep -rn "j:linknode\|j:url" src/ --include="*.cnd"
-```
-
-**C4** — View using `j:linkType` value directly as an `href`
-```bash
-grep -rn 'href=.*j:linkType\|href=.*\["j:linkType"\]' src/ --include="*.server.tsx" --include="*.client.tsx"
-```
-Note: a `switch(props["j:linkType"])` block is CORRECT usage — do NOT flag it. Only flag if the `j:linkType` value appears directly as an `href` attribute.
-
-**C8** — All `<Area>` elements using the same generic `nodeType`
-```bash
-grep -rn "nodeType" src/templates/ --include="*.server.tsx"
-```
-
-### 🟡 WARNINGS (fix if time allows)
-
-**W1** — User-visible CND string/textarea without `i18n`
-```bash
-grep -rn "(string\|textarea\|richtext)" src/ --include="*.cnd" | grep -v "i18n\|j:linkType\|j:url\|j:linknode"
-```
-
-**W4** — Props typed as required (not `?:`) in types.ts
-```bash
-grep -rn ": string\b\|: number\b\|: boolean\b" src/ --include="types.ts" | grep -v "?:"
-```
-
-**W9** — Hardcoded URLs in views
-```bash
-grep -rn 'href="http\|href="/\|href=`/' src/ --include="*.server.tsx" --include="*.client.tsx"
-```
-
----
-
-## Step 3 — Write REVIEW.md
-
-Write `REVIEW.md` in the project root. Be specific: include file path and line content for each finding.
-
-```
-# Code Review — Round <N>
-
-## Verdict
-PASS | NEEDS_WORK
-
-(PASS if zero Critical issues. NEEDS_WORK if any Critical issue found.)
-
-## Critical Issues (must fix before next deploy)
-- [C1] src/components/Hero/definition.cnd:5 — extends jmix:droppableContent directly
-  Fix: change to extend namespacemix:pageComponent
-
-(Write "None" if no critical issues.)
-
-## Warnings (fix if time allows, do NOT redeploy just for these)
-...
-
-## Suggestions (informational only)
-...
-
-## Summary
-- Critical: N | Warnings: N | Suggestions: N
-```
-
-Set Verdict to PASS only if there are **zero Critical issues**. Warnings and suggestions alone do not block.