@abgov/nx-adsp 13.17.0 → 13.17.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@abgov/nx-adsp",
-  "version": "13.17.0",
+  "version": "13.17.1",
   "license": "Apache-2.0",
   "main": "src/index.js",
   "description": "Government of Alberta - Nx plugin for ADSP apps.",

package/src/generators/react-app/files/AGENTS.md__tmpl__

@@ -64,14 +64,15 @@ from `@abgov/react-components` over those components:
 import { GoabButton, GoabAppHeader, GoabButtonGroup } from '@abgov/react-components';
 ```
 
-**Events**: GoA React components use `onGoab*` handlers (not standard React synthetic
-events); the payload is on `e.detail`:
+**Events**: GoA React components use plain `onClick`/`onChange` handlers, not the
+`onGoab*`/`e.detail` pattern some other GoA framework wrappers use. `onChange` receives its
+detail object directly as the callback's sole argument — never wrapped in a synthetic event:
 
 ```tsx
-<GoabButton onGoabClick={handleSave}>Save</GoabButton>
-<GoabInput onGoabChange={(e) => setValue(e.detail.value)} />
-<GoabDropdown onGoabChange={(e) => setSelected(e.detail.value)} />
-<GoabCheckbox onGoabChange={(e) => setChecked(e.detail.checked)} />
+<GoabButton onClick={handleSave}>Save</GoabButton>
+<GoabInput onChange={(detail) => setValue(detail.value)} />
+<GoabDropdown onChange={(detail) => setSelected(detail.value)} />
+<GoabCheckbox onChange={(detail) => setChecked(detail.checked)} />
 ```
 
 Event `detail` shapes vary by component — check the component's gallery page or the

package/src/generators/react-app/react-app.spec.ts

@@ -99,6 +99,20 @@ describe('React App Generator', () => {
     expect(agents).not.toContain('ui-components.alberta.ca');
   }, 30000);
 
+  it('AGENTS.md documents the real onClick/onChange event pattern, not the never-real onGoab*/e.detail one', async () => {
+    // @abgov/react-components has never exposed onGoab*-prefixed handlers or wrapped a
+    // callback's payload in e.detail -- onChange receives its detail object directly. Regression
+    // guard: this doc text drifted from the real API without the generated app code ever being
+    // affected (app.tsx has always used plain onClick), so nothing else would have caught it.
+    const host = createTreeWithEmptyWorkspace({ layout: 'apps-libs' });
+    await generator(host, options);
+    const agents = host.read('apps/test/AGENTS.md').toString();
+    expect(agents).not.toContain('onGoabClick');
+    expect(agents).not.toContain('onGoabChange');
+    expect(agents).not.toMatch(/e\.detail\.(value|checked)/);
+    expect(agents).toContain('onChange={(detail) => setValue(detail.value)}');
+  }, 30000);
+
   it('can add nginx proxy', async () => {
     const host = createTreeWithEmptyWorkspace({ layout: 'apps-libs' });
     await generator(host, {