@llm4ts/shell 0.17.0 → 0.18.1

package/kits/j2ee-nextjs/fixtures/demo-bank/legacy-j2ee/src/main/webapp/transferStep1.jsp

@@ -0,0 +1,73 @@
+<%@ page contentType="text/html; charset=ISO-8859-1" %>
+<%@ page import="java.util.List" %>
+<%@ page import="com.demobank.legacy.dto.BenfDTO" %>
+<%@ taglib uri="http://java.sun.com/jsp/jstl/core" prefix="c" %>
+<%-- HERO 3, step 1 of 3: wire transfer entry. Posts step=1 to TransferServlet
+     which stores a TransferDraft in HttpSession. Scriptlet + JSTL mix is
+     historical: the beneficiary combo predates the JSTL migration. --%>
+<jsp:include page="header.jsp" />
+
+<h1>Wire Transfer &mdash; Step 1 of 3</h1>
+
+<c:if test="${not empty stepErr}">
+  <p class="msgErr"><c:out value="${stepErr}" /></p>
+</c:if>
+
+<form id="trfForm" class="legacy" method="post" action="transfer">
+  <input type="hidden" name="step" value="1" />
+  <div class="row">
+    <label for="benfId">Beneficiary</label>
+    <select id="benfId" name="benfId">
+      <option value="">-- select --</option>
+<%
+    // Legacy: combo rendered with a scriptlet since 2007 (pre-JSTL page).
+    List benfCombo = (List) request.getAttribute("benfCombo");
+    if (benfCombo != null) {
+        for (int i = 0; i < benfCombo.size(); i++) {
+            BenfDTO b = (BenfDTO) benfCombo.get(i);
+%>
+      <option value="<%= b.getBenfId() %>"><%= b.getBenfNm() %> - <%= b.getBenfIban() %></option>
+<%
+        }
+    }
+%>
+    </select>
+    <a href="beneficiary?action=new">new beneficiary</a>
+  </div>
+  <div class="row">
+    <label for="trfAmt">Amount</label>
+    <input type="text" id="trfAmt" name="trfAmt" size="12" value="<c:out value='${draft.trfAmt}' />" />
+    <select id="trfCcy" name="trfCcy">
+      <option value="EUR" selected="selected">EUR</option>
+    </select>
+  </div>
+  <div class="row">
+    <label for="trfDesc">Description</label>
+    <input type="text" id="trfDesc" name="trfDesc" size="50" maxlength="140"
+           value="<c:out value='${draft.trfDesc}' />" />
+  </div>
+  <div class="row">
+    <label for="valDt">Value date</label>
+    <input type="text" id="valDt" name="valDt" size="10" value="<c:out value='${draft.valDt}' />" />
+    <span class="small">(dd/mm/yyyy, leave blank for first available)</span>
+  </div>
+  <div class="row">
+    <input type="submit" class="btn" value="Continue" />
+  </div>
+</form>
+
+<script type="text/javascript">
+$(document).ready(function() {
+  $("#trfForm").submit(function() {
+    var ok = true;
+    if (dbTrim($("#benfId").val()) == "") { dbMark("#benfId", "Select a beneficiary"); ok = false; }
+    else dbUnmark("#benfId");
+    if (!dbAmount("#trfAmt")) ok = false;
+    // Description optional client-side; host rejects some characters,
+    // server strips them silently (see TransferServlet.sanitizeDesc).
+    return ok;
+  });
+});
+</script>
+
+<jsp:include page="footer.jsp" />

package/kits/j2ee-nextjs/fixtures/demo-bank/legacy-j2ee/src/main/webapp/transferStep2.jsp

@@ -0,0 +1,40 @@
+<%@ page contentType="text/html; charset=ISO-8859-1" %>
+<%@ page import="com.demobank.legacy.web.TransferDraft" %>
+<%-- HERO 3, step 2 of 3: review. All data comes from the TransferDraft in
+     HttpSession (attribute "trfDraft"); nothing is re-posted except the
+     confirmation itself. Direct GET without a draft bounces to step 1. --%>
+<jsp:include page="header.jsp" />
+
+<h1>Wire Transfer &mdash; Step 2 of 3: Review</h1>
+
+<%
+    TransferDraft draft = (TransferDraft) session.getAttribute("trfDraft");
+    if (draft == null) {
+        // Session expired or deep link: back to step 1.
+        response.sendRedirect("transfer");
+        return;
+    }
+%>
+
+<table class="data" style="width: 60%;">
+  <tr><th>Beneficiary</th><td><%= draft.getBenfNm() %></td></tr>
+  <tr><th>IBAN</th><td><%= draft.getBenfIban() %></td></tr>
+  <tr><th>Amount</th><td><%= draft.getTrfCcy() %> <%= draft.getTrfAmt() %></td></tr>
+  <tr><th>Description</th><td><%= draft.getTrfDesc() == null ? "" : draft.getTrfDesc() %></td></tr>
+  <tr><th>Value date</th><td><%= draft.getValDt() %></td></tr>
+  <tr><th>Fee</th><td>EUR <%= draft.getFeeAmt() %> (standard SEPA)</td></tr>
+</table>
+
+<p class="msgErr">
+  Please verify the details above. On confirmation the transfer is sent to
+  the bank host and can no longer be amended online.
+</p>
+
+<form method="post" action="transfer">
+  <input type="hidden" name="step" value="3" />
+  <input type="submit" class="btn" value="Confirm transfer" />
+  <input type="button" class="btn btnGrey" value="Back"
+         onclick="window.location='transfer';" />
+</form>
+
+<jsp:include page="footer.jsp" />

package/kits/j2ee-nextjs/fixtures/demo-bank/nextjs/CONTRIBUTING.md

@@ -0,0 +1,76 @@
+# DemoBank house rules
+
+These rules apply to every page added to this repository — by people or by
+automated converters. The exemplar pages (`src/app/cards/page.tsx`,
+`src/app/profile/page.tsx`) demonstrate all of them; imitate the exemplars
+before inventing anything.
+
+## File layout
+
+```text
+contracts/<domain>.openapi.yaml   wire contract (OpenAPI 3.0)
+src/app/<route>/page.tsx          one page per route, always "use client"
+src/auth/AuthProvider.tsx         SSO stub — consume, never reimplement
+src/components/                   design system (import from '@/components')
+src/services/<domain>/port.ts     typed interface + factory type
+src/services/<domain>/mock.ts     mock adapter with fixture data
+src/services/registry.ts          the only place adapters are wired to ports
+src/theme/                        tokens + component CSS
+tests/<route>.page.test.tsx       one component test file per page
+```
+
+Imports use the `@/` alias (`@/components`, `@/services/registry`); no deep
+relative paths across top-level folders.
+
+## Pages
+
+- Every page starts with `'use client'` and default-exports the page
+  component. No server components fetching data, no API routes.
+- Every page renders exactly one `PageLayout` at its root and puts content
+  in `Card`s.
+- Get the signed-in user from `useAuth()`. Never implement login, tokens, or
+  redirects.
+- Loading states: hold `null` until the port resolves and render
+  `<p className="db-loading">…</p>` meanwhile (see the exemplars).
+
+## Components
+
+- Compose the design system: `Button`, `Field`, `Form`, `DataTable`,
+  `Stepper`, `Card`, `PageLayout`. Do not add external UI kits and do not
+  hand-roll tables, labels, or submit handling that a component covers.
+- Tabular data goes through `DataTable` with declarative columns.
+- Every form control sits inside a `Field` (label + error handled for you)
+  and every form is a `Form` render prop (values, errors, setValue,
+  submitting). Validation is a pure function next to the page returning a
+  `ValidationErrors<T>` map.
+- Styling: use the `db-*` classes from `src/theme/components.css`, which
+  consume only the custom properties in `src/theme/tokens.css`. No inline
+  colors, no new CSS files per page, no CSS-in-JS.
+
+## Services (ports and adapters)
+
+- No `fetch`/network calls in components — ever. Pages call ports; adapters
+  talk to the gateway.
+- Each domain gets `src/services/<domain>/port.ts` (interface + `...Factory`
+  type), `src/services/<domain>/mock.ts` (deterministic fixture data, small
+  artificial delay), and `contracts/<domain>.openapi.yaml` matching the port.
+- Pages obtain ports only from `src/services/registry.ts`. Swapping a mock
+  for a real adapter is a one-line change there; pages never import adapters
+  directly.
+
+## Tests
+
+- One test file per page under `tests/`, named `<route>.page.test.tsx`,
+  written with Vitest + React Testing Library.
+- Style: `vi.mock('@/services/registry', …)` with instrumented port
+  functions, render the page inside `AuthProvider`, then assert (1) fields
+  or rows render, (2) validation fires and blocks the port on bad input,
+  (3) the port is called with the expected payload.
+- Tests are deterministic and network-free.
+
+## TypeScript and lint
+
+- `strict` TypeScript; no `any`, no type assertions (`as`), no `@ts-ignore`.
+- Model domain data with `readonly` interfaces exported from the port.
+- Gates that must stay green: `pnpm typecheck && pnpm lint && pnpm test &&
+  pnpm build`.

package/kits/j2ee-nextjs/fixtures/demo-bank/nextjs/README.md

@@ -0,0 +1,46 @@
+# demo-bank-nextjs
+
+Synthetic, client-only Next.js SPA for a fictional bank ("DemoBank"). This is
+the **destination fixture** for the llm4ts bank-conversion PoC: converted
+legacy pages land here, and the convert flow judges them by how well they
+imitate this repo's house style (see `CONTRIBUTING.md` and the exemplar
+pages `src/app/cards/page.tsx` and `src/app/profile/page.tsx`).
+
+Everything is fake: SSO is a stubbed `AuthProvider`, service adapters are
+in-memory mocks behind typed ports, and the data is deterministic fixture
+data. No real bank, brand, or person is referenced.
+
+## Install and run
+
+This package is standalone — it is **not** part of the llm4ts pnpm
+workspace, so install with the workspace ignored:
+
+```bash
+pnpm install --ignore-workspace
+pnpm dev        # local dev server
+```
+
+Gates (the convert flow's hard gates — all must pass):
+
+```bash
+pnpm typecheck && pnpm lint && pnpm test && pnpm build
+```
+
+`pnpm build` produces a static export in `out/` (`output: 'export'`; there
+is no server runtime).
+
+## Offline notes
+
+- `pnpm-lock.yaml` is committed; installs are deterministic.
+- To run on a machine without network (e.g. workshop stage): warm the pnpm
+  store once while online with `pnpm install --ignore-workspace` here, then
+  in a seeded copy run `pnpm install --ignore-workspace --offline` — pnpm
+  resolves everything from the lockfile and the local store.
+- Tests, typecheck, lint, and build never touch the network.
+
+## Seeding a working copy
+
+The fixture is materialized as a standalone git repository by
+`../seed-nextjs.mjs` (node_modules and build output are excluded; the
+initial commit is deterministic). `../smoke-nextjs.mjs` seeds into a temp
+dir and asserts the expected files exist.

package/kits/j2ee-nextjs/fixtures/demo-bank/nextjs/contracts/cards.openapi.yaml

@@ -0,0 +1,57 @@
+# Wire contract for the cards domain. Mirrors src/services/cards/port.ts —
+# keep the two in sync. Served by the API gateway; the SPA never calls it
+# directly outside the cards adapter.
+openapi: 3.0.3
+info:
+  title: DemoBank Cards API
+  version: 1.0.0
+paths:
+  /customers/{customerId}/cards:
+    get:
+      operationId: listCards
+      summary: List payment cards for a customer
+      parameters:
+        - name: customerId
+          in: path
+          required: true
+          schema:
+            type: string
+      responses:
+        '200':
+          description: The customer's cards
+          content:
+            application/json:
+              schema:
+                type: array
+                items:
+                  $ref: '#/components/schemas/CardSummary'
+components:
+  schemas:
+    Money:
+      type: object
+      required: [amount, currency]
+      properties:
+        amount:
+          type: number
+        currency:
+          type: string
+          description: ISO 4217 code
+    CardSummary:
+      type: object
+      required: [id, label, maskedPan, kind, status, balance]
+      properties:
+        id:
+          type: string
+        label:
+          type: string
+        maskedPan:
+          type: string
+          description: Masked PAN, e.g. "**** **** **** 4242"
+        kind:
+          type: string
+          enum: [debit, credit]
+        status:
+          type: string
+          enum: [active, blocked]
+        balance:
+          $ref: '#/components/schemas/Money'

package/kits/j2ee-nextjs/fixtures/demo-bank/nextjs/contracts/profile.openapi.yaml

@@ -0,0 +1,73 @@
+# Wire contract for the profile domain. Mirrors src/services/profile/port.ts —
+# keep the two in sync. Served by the API gateway; the SPA never calls it
+# directly outside the profile adapter.
+openapi: 3.0.3
+info:
+  title: DemoBank Profile API
+  version: 1.0.0
+paths:
+  /customers/{customerId}/profile:
+    get:
+      operationId: getProfile
+      summary: Fetch a customer's profile
+      parameters:
+        - name: customerId
+          in: path
+          required: true
+          schema:
+            type: string
+      responses:
+        '200':
+          description: The customer's profile
+          content:
+            application/json:
+              schema:
+                $ref: '#/components/schemas/Profile'
+    put:
+      operationId: updateProfile
+      summary: Update a customer's profile
+      parameters:
+        - name: customerId
+          in: path
+          required: true
+          schema:
+            type: string
+      requestBody:
+        required: true
+        content:
+          application/json:
+            schema:
+              $ref: '#/components/schemas/ProfileUpdate'
+      responses:
+        '200':
+          description: The updated profile
+          content:
+            application/json:
+              schema:
+                $ref: '#/components/schemas/Profile'
+components:
+  schemas:
+    Profile:
+      type: object
+      required: [customerId, fullName, email, phone]
+      properties:
+        customerId:
+          type: string
+        fullName:
+          type: string
+        email:
+          type: string
+          format: email
+        phone:
+          type: string
+    ProfileUpdate:
+      type: object
+      required: [fullName, email, phone]
+      properties:
+        fullName:
+          type: string
+        email:
+          type: string
+          format: email
+        phone:
+          type: string

package/kits/j2ee-nextjs/fixtures/demo-bank/nextjs/eslint.config.mjs

@@ -0,0 +1,24 @@
+// Minimal flat config: typescript-eslint recommended plus a few house rules.
+// Deliberately no framework plugin zoo — the hard gates are typecheck, lint,
+// test, build; the house style lives in CONTRIBUTING.md and the exemplars.
+import tseslint from 'typescript-eslint';
+
+export default tseslint.config(
+  {
+    ignores: ['node_modules/', '.next/', 'out/', 'next-env.d.ts'],
+  },
+  ...tseslint.configs.recommended,
+  {
+    rules: {
+      '@typescript-eslint/no-explicit-any': 'error',
+      '@typescript-eslint/consistent-type-assertions': [
+        'error',
+        { assertionStyle: 'never' },
+      ],
+      '@typescript-eslint/consistent-type-imports': 'error',
+      eqeqeq: ['error', 'always'],
+      'no-console': 'error',
+      'prefer-const': 'error',
+    },
+  },
+);

package/kits/j2ee-nextjs/fixtures/demo-bank/nextjs/next-env.d.ts

@@ -0,0 +1,5 @@
+/// <reference types="next" />
+/// <reference types="next/image-types/global" />
+
+// NOTE: This file should not be edited
+// see https://nextjs.org/docs/app/api-reference/config/typescript for more information.

package/kits/j2ee-nextjs/fixtures/demo-bank/nextjs/next.config.mjs

@@ -0,0 +1,10 @@
+/**
+ * Client-only SPA: static export, served from object storage behind a CDN.
+ * Mirrors kits/j2ee-nextjs/scaffolds/nextjs-spa (the bank-provided scaffold).
+ * @type {import('next').NextConfig}
+ */
+const nextConfig = {
+  output: 'export',
+};
+
+export default nextConfig;

package/kits/j2ee-nextjs/fixtures/demo-bank/nextjs/package.json

@@ -0,0 +1,29 @@
+{
+  "name": "demo-bank-nextjs",
+  "version": "0.1.0",
+  "private": true,
+  "scripts": {
+    "dev": "next dev",
+    "build": "next build",
+    "typecheck": "tsc --noEmit",
+    "lint": "eslint .",
+    "test": "vitest run"
+  },
+  "dependencies": {
+    "next": "15.3.4",
+    "react": "19.1.0",
+    "react-dom": "19.1.0"
+  },
+  "devDependencies": {
+    "@testing-library/dom": "^10.4.0",
+    "@testing-library/react": "^16.3.0",
+    "@types/node": "^22.15.0",
+    "@types/react": "^19.1.0",
+    "@types/react-dom": "^19.1.0",
+    "eslint": "^9.29.0",
+    "jsdom": "^26.1.0",
+    "typescript": "5.8.3",
+    "typescript-eslint": "^8.34.0",
+    "vitest": "^3.2.0"
+  }
+}