@micha.bigler/ui-core-micha 3.13.0 → 3.13.1
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/CHANGELOG.md +5 -0
- package/WORK_ORDERS.md +2 -2
- package/dist/layout/SectionNav.js +3 -3
- package/package.json +1 -1
- package/src/layout/SectionNav.jsx +22 -18
- package/tests/SectionNav.test.jsx +40 -0
- package/work-orders/UCM-SHELL-8.md +113 -0
package/CHANGELOG.md
CHANGED
|
@@ -1,5 +1,10 @@
|
|
|
1
1
|
# Changelog
|
|
2
2
|
|
|
3
|
+
## 3.13.1 - UCM-SHELL-8
|
|
4
|
+
|
|
5
|
+
`SectionNav` now renders grouped and overview navigation buttons inside list items on
|
|
6
|
+
desktop and in the mobile drawer. Selection, callbacks, labels and spacing are preserved.
|
|
7
|
+
|
|
3
8
|
## 3.13.0 - UCM-AUTH-12
|
|
4
9
|
|
|
5
10
|
`AccountPage` accepts `showProfileStatements` (default `true`). Set it to `false` in consumers
|
package/WORK_ORDERS.md
CHANGED
|
@@ -28,8 +28,8 @@ prefixed ID; never reuse a bare flat number across workstreams.
|
|
|
28
28
|
|
|
29
29
|
| ID | Titel | Beschreibung | Datum | Status | Commit(s) | Notiz |
|
|
30
30
|
|---|---|---|---|---|---|---|
|
|
31
|
-
| UCM-AUTH-12 | [Optional account profile statement text](work-orders/UCM-AUTH-12.md) | AccountPage showProfileStatements defaults true; false suppresses statement panels/requests while preserving both consent controls and their payload. Minor release 3.13.0, consumed by innoservice INNO-FIX-13. | 2026-10-13 |
|
|
32
|
-
| UCM-SHELL-8 | SectionNav lists contain direct button children |
|
|
31
|
+
| UCM-AUTH-12 | [Optional account profile statement text](work-orders/UCM-AUTH-12.md) | AccountPage showProfileStatements defaults true; false suppresses statement panels/requests while preserving both consent controls and their payload. Minor release 3.13.0, consumed by innoservice INNO-FIX-13. | 2026-10-13 | done | 97b7d83 | Tier 3 (shared-core account surface). Gate #1 explicitly approved in the innoservice Codex chat on 2026-10-13, including release and app pin bump. Separate configured Luna implementation and independent reviews; existing default and consent behavior are invariants. review: codex/gpt-6-luna · lenses: envelope, regression, duplication, tests, ui, security · 0 raised · 0 accepted · worst accepted: none. Parent scoped tests: AccountPage.profileStatements.test.jsx + AccountPage.test.jsx, 6 passed, correct kit cwd, maxWorkers2. Native author attempt EPERM before collection; parent gate green. Reviewed additive API/default and package release metadata; main push triggers the existing release build/suite/publish. Released 3.13.0: CI38422682968 build, release suite and npm Trusted Publishing all succeeded. App consumption/capture remains tracked under INNO-FIX-13. |
|
|
32
|
+
| UCM-SHELL-8 | SectionNav lists contain direct button children | Grouped and overview entries need genuine list items containing the existing navigation buttons, preserving selection, drawer behavior and presentation. | 2026-10-13 | in-review | `d399f51` | Tier 3. Gate #1 approved by the operator on 2026-10-13: structural correction, tests/review, patch release and innoservice pin bump. Full scope and context: [work-orders/UCM-SHELL-8.md](work-orders/UCM-SHELL-8.md). Original CI38387014900/source9937adb evidence remains in innoservice/work-orders/assets/INNO-FIX-11-axe-nodes.json; CI38425506007/sourceebc35e1 reproduces the three account list violations on 3.13.0, recorded in INNO-FIX-11-candidate-b-axe-nodes.json. Initial native author /root/kit_sectionnav requested codex/gpt-6-luna; effective native metadata unexposed. Source and two focused test cases delivered; author run EPERM before collection, zero tests. Parent tests and independent review pending. review: codex/gpt-6-luna · lenses: envelope, regression, duplication, tests + UI · 1 raised · 0 accepted · worst accepted: none. Disposition in WO Part C: fabricated class-stub identity demand rejected against the real revert failure and preserved existing behavior checks. Parent 21 scoped tests passed (maxWorkers=2), TypeScript build exit 0. Core source accepted; publication, app pin and fresh consumer capture pending. |
|
|
33
33
|
| UCM-THEME-17 | Decide a less compressed baseline type scale | Design round (design-language process): inventory, instrument with 2-3 candidate scales next to today's baseline and MUI on real screens (hram, a survey site, jg-ferien, kerzenziehen), value rounds, frozen sheet. Measured 2026-10-04: page/card/section titles differ by 1-3 px, hierarchy rests on weight alone. Full order [`work-orders/UCM-THEME-17.md`](work-orders/UCM-THEME-17.md) | 2026-10-04 | done | `68dfc6a`, `998e36c`, `cb9dd89`, `c452224` | Tier 2 (document only) · operator decision 2026-10-04: baseline finding, not per-app overrides · implementation order follows the freeze · survey_app SVA-THEME-1 typography revert held until then · **frozen 2026-10-04** after two value rounds: candidate B, mobile h4 23, h1-h3 36/32/28; value source [`docs/TYPE-SCALE.md`](docs/TYPE-SCALE.md) · review: none, document order, the operator freeze is the gate · implementation UCM-THEME-18 |
|
|
34
34
|
| UCM-THEME-18 | Ship the frozen baseline type scale | `tokens.js` typography per `docs/TYPE-SCALE.md`: h1-h3 36/32/28, h4 24 (23 below `sm`), h5 20, subtitle1 13; light fixture typography entries; minor release 3.12.0. Full order [`work-orders/UCM-THEME-18.md`](work-orders/UCM-THEME-18.md) | 2026-10-04 | 5be83d5 | | Tier 3 (shared-core) · reviewer + ui_reviewer · downstream per-app orders (h5 page titles -> h4, per-level weight overrides) written after the release - 2026-10-04: Codex implemented (gpt-6-luna; probe of stale 2026-10-03 record succeeded, available) - orchestrator fixed review findings - review: codex/gpt-6-luna - lenses: envelope,regression,duplication,tests + ui_review - 3 raised - 2 accepted - worst accepted: defect (h4 mobile key now follows an app-overridden sm, test added; completeness test strengthened; rejected: breakpointDownQuery vs theme.breakpoints.down, no theme exists at token time) - light fixture diff limited to typography entries - no rendered side-by-side (no prototype; values asserted from the resolved theme in light and dark) - release 3.12.0 |
|
|
35
35
|
| UCM-I18N-5 | [User-list pagination range text is hard-coded English](work-orders/UCM-I18N-5.md) | `UserListComponent`'s `TablePagination` sets `labelRowsPerPage` but no `labelDisplayedRows`, so German shows "1–4 of 4". | 2026-10-01 | done | 6b9d557 | Found in jg-ferien's rendered check 2026-10-01. Follows `UCM-I18N-4`. Tier 3. Done = released AND jg-ferien pin bumped. Envelope written 2026-10-01; operator: hand to the jg Paket-4 session. Implementation map filled 2026-10-02 (`a732025`): the translation bundle carries FOUR locales (de/fr/en/sw), not three — confirmed by reading `authTranslations.ts` directly; proposed two keys (exact-count + "more than" form) with a best-effort, not native-verified, Swahili wording. Implemented by codex/gpt-5.6-luna exactly per the map: `labelDisplayedRows` resolves `UserList.DISPLAYED_ROWS`/`UserList.DISPLAYED_ROWS_MORE_THAN` through `t()`, both keys added in all four locales. review: codex/gpt-5.6-luna · lenses: envelope,regression,duplication,tests,ui_reviewer (one concurrent batch, diff inline) · 4 raised · 3 accepted (deduplicated) · worst accepted: defect. Accepted: the rendered English test would still pass against a reverted implementation, since the EN wording was deliberately made identical to MUI's own hardcoded default — added a second test swapping in a distinctive marker string for just that key, proving the render genuinely resolves through i18next, not MUI's fallback; `authTranslations.test.js`'s new key-parity check only asserted non-empty strings, not that each locale actually carries the `{{from}}`/`{{to}}`/`{{count}}` placeholders the others do — strengthened with an explicit per-locale placeholder check; self-identified (corroborated by the `tests` lens's own closing remark) that the `count === -1` ("more than") branch can never be reached through a render of `UserListComponent` itself (its own `count` prop is always a real array length) — extracted the inline callback into an exported, directly unit-tested `formatDisplayedRowsLabel(t)` rather than leaving it permanently untested dead code. Rejected: `regression`'s claim that hard-coding `labelDisplayedRows` "overrides any consuming app's MUI theme customisation" — speculative, no actual consumer override exists (same explicit-prop design as `labelRowsPerPage` since `UCM-I18N-4`), and the reviewer's own repo search found zero consumer snapshots/assertions relying on the old English text. `envelope`'s "release + pin bump missing from the diff" was correct but not a code defect — addressed by the remaining steps below, not a fix to the reviewed diff. `duplication`: 0 findings. `ui_reviewer`: 0 blocking findings; confirmed placeholder consistency across all 8 strings; Swahili wording flagged as advisory-only ("structurally plausible to a non-native reviewer, native-speaker verification advisable" — not re-asked of a native speaker here, recorded as open). Tests: 37/37 green (`userScreenTranslations.test.jsx`, `authTranslations.test.js`, `UserListComponent.test.jsx`). Released as `3.11.1` (patch — continues `UCM-I18N-4`'s i18n-gap category, not a new capability area), commit `6b9d557`; npm Trusted Publishing confirmed via the Actions job log (not a registry re-query — provenance publishes can lag), tarball `@micha.bigler/ui-core-micha@3.11.1`. jg-ferien's pin bumped 3.7.0 -> 3.11.1 same day, closed as jg's own `JG-DEPS-3` (`2c1d6e3`) — see that repo's `WORK_ORDERS.md` for the empirical safety checks (status-colour palette diff, `UserListComponent`-unused confirmation, full-suite run) this multi-version jump required. |
|
|
@@ -1,7 +1,7 @@
|
|
|
1
1
|
import { jsx as _jsx, jsxs as _jsxs, Fragment as _Fragment } from "react/jsx-runtime";
|
|
2
2
|
import { useId, useState } from 'react';
|
|
3
3
|
import KeyboardArrowDownIcon from '@mui/icons-material/KeyboardArrowDown';
|
|
4
|
-
import { Box, ButtonBase, Drawer, List, ListItemButton, ListItemText, Paper, Stack, Typography, } from '@mui/material';
|
|
4
|
+
import { Box, ButtonBase, Drawer, List, ListItem, ListItemButton, ListItemText, Paper, Stack, Typography, } from '@mui/material';
|
|
5
5
|
import { useTheme } from '@mui/material/styles';
|
|
6
6
|
import { useTranslation } from 'react-i18next';
|
|
7
7
|
const defaultZIndex = (theme) => theme.zIndex.drawer + 3;
|
|
@@ -82,11 +82,11 @@ const SECTION_NAV_DRAWER_SCROLL_SX = {
|
|
|
82
82
|
};
|
|
83
83
|
function SectionNavList({ groups, activeKey, rememberedKey, overviewItem, onSelect, rememberedLabel, }) {
|
|
84
84
|
const headingId = useId();
|
|
85
|
-
return (_jsxs(Stack, { spacing: 2, children: [overviewItem && (_jsx(Paper, { variant: "outlined", sx: SECTION_NAV_GROUP_PAPER_SX, children: _jsx(List, { disablePadding: true, children: _jsx(ListItemButton, { selected: activeKey == null, "aria-current": activeKey == null ? 'page' : undefined, onClick: () => onSelect === null || onSelect === void 0 ? void 0 : onSelect(null), children: _jsx(ListItemText, { primary: overviewItem.label }) }) }) })), groups.map((group, groupIndex) => {
|
|
85
|
+
return (_jsxs(Stack, { spacing: 2, children: [overviewItem && (_jsx(Paper, { variant: "outlined", sx: SECTION_NAV_GROUP_PAPER_SX, children: _jsx(List, { disablePadding: true, children: _jsx(ListItem, { disablePadding: true, children: _jsx(ListItemButton, { selected: activeKey == null, "aria-current": activeKey == null ? 'page' : undefined, onClick: () => onSelect === null || onSelect === void 0 ? void 0 : onSelect(null), children: _jsx(ListItemText, { primary: overviewItem.label }) }) }) }) })), groups.map((group, groupIndex) => {
|
|
86
86
|
const groupHeadingId = `${headingId}-${groupIndex}`;
|
|
87
87
|
return (_jsxs(Paper, { variant: "outlined", sx: SECTION_NAV_GROUP_PAPER_SX, children: [_jsx(Box, { id: groupHeadingId, sx: SECTION_NAV_GROUP_HEADER_SX, children: _jsx(Typography, { variant: "subtitle2", fontWeight: 700, children: group.label }) }), _jsx(List, { disablePadding: true, "aria-labelledby": groupHeadingId, children: group.items.map((item) => {
|
|
88
88
|
const selected = item.key === activeKey;
|
|
89
|
-
return (_jsx(ListItemButton, { selected: selected, "aria-current": selected ? 'page' : undefined, onClick: () => onSelect === null || onSelect === void 0 ? void 0 : onSelect(item.key), children: _jsx(ListItemText, { primary: item.label, secondary: !selected && item.key === rememberedKey ? rememberedLabel : null }) }, item.key));
|
|
89
|
+
return (_jsx(ListItem, { disablePadding: true, children: _jsx(ListItemButton, { selected: selected, "aria-current": selected ? 'page' : undefined, onClick: () => onSelect === null || onSelect === void 0 ? void 0 : onSelect(item.key), children: _jsx(ListItemText, { primary: item.label, secondary: !selected && item.key === rememberedKey ? rememberedLabel : null }) }) }, item.key));
|
|
90
90
|
}) })] }, group.key));
|
|
91
91
|
})] }));
|
|
92
92
|
}
|
package/package.json
CHANGED
|
@@ -5,6 +5,7 @@ import {
|
|
|
5
5
|
ButtonBase,
|
|
6
6
|
Drawer,
|
|
7
7
|
List,
|
|
8
|
+
ListItem,
|
|
8
9
|
ListItemButton,
|
|
9
10
|
ListItemText,
|
|
10
11
|
Paper,
|
|
@@ -118,13 +119,15 @@ function SectionNavList({
|
|
|
118
119
|
{overviewItem && (
|
|
119
120
|
<Paper variant="outlined" sx={SECTION_NAV_GROUP_PAPER_SX}>
|
|
120
121
|
<List disablePadding>
|
|
121
|
-
<
|
|
122
|
-
|
|
123
|
-
|
|
124
|
-
|
|
125
|
-
|
|
126
|
-
|
|
127
|
-
|
|
122
|
+
<ListItem disablePadding>
|
|
123
|
+
<ListItemButton
|
|
124
|
+
selected={activeKey == null}
|
|
125
|
+
aria-current={activeKey == null ? 'page' : undefined}
|
|
126
|
+
onClick={() => onSelect?.(null)}
|
|
127
|
+
>
|
|
128
|
+
<ListItemText primary={overviewItem.label} />
|
|
129
|
+
</ListItemButton>
|
|
130
|
+
</ListItem>
|
|
128
131
|
</List>
|
|
129
132
|
</Paper>
|
|
130
133
|
)}
|
|
@@ -143,17 +146,18 @@ function SectionNavList({
|
|
|
143
146
|
{group.items.map((item) => {
|
|
144
147
|
const selected = item.key === activeKey;
|
|
145
148
|
return (
|
|
146
|
-
<
|
|
147
|
-
|
|
148
|
-
|
|
149
|
-
|
|
150
|
-
|
|
151
|
-
|
|
152
|
-
|
|
153
|
-
|
|
154
|
-
|
|
155
|
-
|
|
156
|
-
|
|
149
|
+
<ListItem key={item.key} disablePadding>
|
|
150
|
+
<ListItemButton
|
|
151
|
+
selected={selected}
|
|
152
|
+
aria-current={selected ? 'page' : undefined}
|
|
153
|
+
onClick={() => onSelect?.(item.key)}
|
|
154
|
+
>
|
|
155
|
+
<ListItemText
|
|
156
|
+
primary={item.label}
|
|
157
|
+
secondary={!selected && item.key === rememberedKey ? rememberedLabel : null}
|
|
158
|
+
/>
|
|
159
|
+
</ListItemButton>
|
|
160
|
+
</ListItem>
|
|
157
161
|
);
|
|
158
162
|
})}
|
|
159
163
|
</List>
|
|
@@ -68,6 +68,17 @@ function renderNav(props = {}, currentTheme = theme) {
|
|
|
68
68
|
);
|
|
69
69
|
}
|
|
70
70
|
|
|
71
|
+
function expectListsToContainListItemButtons(lists) {
|
|
72
|
+
expect(lists.length).toBeGreaterThan(0);
|
|
73
|
+
lists.forEach((list) => {
|
|
74
|
+
expect(Array.from(list.children).every((child) => child.tagName === 'LI')).toBe(true);
|
|
75
|
+
expect(list.querySelectorAll(':scope > li')).toHaveLength(list.children.length);
|
|
76
|
+
list.querySelectorAll(':scope > li').forEach((item) => {
|
|
77
|
+
expect(item.querySelectorAll(':scope > .MuiListItemButton-root')).toHaveLength(1);
|
|
78
|
+
});
|
|
79
|
+
});
|
|
80
|
+
}
|
|
81
|
+
|
|
71
82
|
describe('SectionNav', () => {
|
|
72
83
|
afterEach(() => {
|
|
73
84
|
cleanup();
|
|
@@ -133,6 +144,35 @@ describe('SectionNav', () => {
|
|
|
133
144
|
expect(container.querySelectorAll('.MuiPaper-root')).toHaveLength(groups.length);
|
|
134
145
|
});
|
|
135
146
|
|
|
147
|
+
it('renders overview and grouped controls inside list items on desktop and mobile', () => {
|
|
148
|
+
const overviewItem = { label: 'Overview' };
|
|
149
|
+
const { container } = renderNav({ overviewItem });
|
|
150
|
+
|
|
151
|
+
expectListsToContainListItemButtons(container.querySelectorAll('nav ul'));
|
|
152
|
+
expect(container.querySelectorAll('nav ul')).toHaveLength(groups.length + 1);
|
|
153
|
+
|
|
154
|
+
cleanup();
|
|
155
|
+
renderNav({ mode: 'mobile', overviewItem, open: true });
|
|
156
|
+
const dialog = screen.getByRole('dialog', { name: 'Switch section' });
|
|
157
|
+
const lists = dialog.querySelectorAll('ul');
|
|
158
|
+
|
|
159
|
+
expectListsToContainListItemButtons(lists);
|
|
160
|
+
expect(lists).toHaveLength(groups.length + 1);
|
|
161
|
+
});
|
|
162
|
+
|
|
163
|
+
it('passes null for overview selection and closes the mobile drawer', async () => {
|
|
164
|
+
const onSelect = vi.fn();
|
|
165
|
+
renderNav({ mode: 'mobile', onSelect, overviewItem: { label: 'Overview' } });
|
|
166
|
+
|
|
167
|
+
fireEvent.click(screen.getByRole('button', { name: /Section Profile/i }));
|
|
168
|
+
expect(screen.getByRole('dialog', { name: 'Switch section' })).toBeTruthy();
|
|
169
|
+
fireEvent.click(screen.getByRole('button', { name: 'Overview' }));
|
|
170
|
+
|
|
171
|
+
expect(onSelect).toHaveBeenCalledOnce();
|
|
172
|
+
expect(onSelect).toHaveBeenCalledWith(null);
|
|
173
|
+
await waitFor(() => expect(screen.queryByRole('dialog', { name: 'Switch section' })).toBeNull());
|
|
174
|
+
});
|
|
175
|
+
|
|
136
176
|
it('shows the remembered secondary line only on its unselected item', () => {
|
|
137
177
|
renderNav({ activeKey: 'profile', rememberedKey: 'security' });
|
|
138
178
|
expect(screen.getByText('Last opened')).toBeTruthy();
|
|
@@ -0,0 +1,113 @@
|
|
|
1
|
+
# UCM-SHELL-8: Valid SectionNav list items
|
|
2
|
+
|
|
3
|
+
# A. Envelope — operator-approved scope
|
|
4
|
+
|
|
5
|
+
## Goal and expected outcome
|
|
6
|
+
|
|
7
|
+
SectionNav navigation entries are genuine list items containing the existing buttons.
|
|
8
|
+
The account capture in innoservice reports three serious axe list violations at 1280 px;
|
|
9
|
+
these remain in ui-core-micha 3.13.0. The correction removes their structural cause.
|
|
10
|
+
|
|
11
|
+
## Scope and non-goals
|
|
12
|
+
|
|
13
|
+
- Correct grouped and optional overview entries in SectionNav's shared list renderer.
|
|
14
|
+
- Keep selection, aria-current, labels, remembered entry, keyboard operation, callbacks,
|
|
15
|
+
controlled/uncontrolled drawer behavior and appearance unchanged.
|
|
16
|
+
- Release the correction as patch 3.13.1 and update innoservice's exact package pin.
|
|
17
|
+
- No navigation redesign, new text, token delta, authentication, permissions, data contract,
|
|
18
|
+
other component, CI configuration or dependency changes beyond that release/pin.
|
|
19
|
+
- No new visual design or prototype: retain the existing presentation.
|
|
20
|
+
|
|
21
|
+
## Tier, risk and required tests to write
|
|
22
|
+
|
|
23
|
+
Tier 3: shared-core source with a rendered DOM correction used by consumers.
|
|
24
|
+
Risk: wrapper defaults could alter spacing or interrupt existing button interaction.
|
|
25
|
+
Write focused assertions that every grouped and overview list has li children containing
|
|
26
|
+
the existing button controls, on desktop and inside the mobile drawer. Preserve existing
|
|
27
|
+
selection and drawer tests; prove overview selection still passes null and closes the drawer.
|
|
28
|
+
|
|
29
|
+
# B. Implementation map — ADDRESSED TO THE IMPLEMENTER
|
|
30
|
+
|
|
31
|
+
## Target and ownership
|
|
32
|
+
|
|
33
|
+
Target: `C:/Users/Micha Bigler/Documents/webapps/ui-core-micha`.
|
|
34
|
+
Native bootstrap retains workspace-root cwd; use absolute paths and explicit workdirs.
|
|
35
|
+
You are not alone in the codebase; preserve other agents' changes. Own only:
|
|
36
|
+
- `src/layout/SectionNav.jsx`
|
|
37
|
+
- `tests/SectionNav.test.jsx`
|
|
38
|
+
|
|
39
|
+
## Context package
|
|
40
|
+
|
|
41
|
+
`src/layout/SectionNav.jsx:120`, SectionNavList, is shared by the desktop nav and mobile
|
|
42
|
+
drawer. The overview List currently directly contains ListItemButton; grouped Lists at
|
|
43
|
+
159 map item keys to direct ListItemButton children. MUI List defaults to ul and the
|
|
44
|
+
button defaults to div. Correct list containment while preserving the interactive button.
|
|
45
|
+
The overview handler passes null; grouped handlers pass item.key. Selected buttons carry
|
|
46
|
+
aria-current=page; remembered secondary text is shown only on its unselected entry.
|
|
47
|
+
Keep exported sx constants, drawer state handlers, prop defaults and group label IDs.
|
|
48
|
+
Use existing MUI primitives; prevent wrapper padding from changing the presentation.
|
|
49
|
+
|
|
50
|
+
`tests/SectionNav.test.jsx` renders the real named SectionNav with ThemeProvider and a
|
|
51
|
+
translation mock. Existing labels: Profile, Security, Users, Invite, Support, Notifications;
|
|
52
|
+
optional Overview; trigger accessible name matches /Section Profile/i; drawer title
|
|
53
|
+
Switch section. Existing tests cover selection, remembered text, controlled/uncontrolled
|
|
54
|
+
drawer, theme tokens and grid. Extend this file without mocking SectionNav or changing
|
|
55
|
+
existing tests to accept weaker behavior. Work from these two files, no broad exploration.
|
|
56
|
+
|
|
57
|
+
## Preamble and progress contract
|
|
58
|
+
|
|
59
|
+
> This committed WO is the COMPLETE spec. You are the initial implementer, not the reviewer.
|
|
60
|
+
> Edit only the owned source/test paths using explicit UTF-8 I/O. Preserve other changes.
|
|
61
|
+
> Do not edit dependencies, lockfiles, package/version/changelog/register/README/governance,
|
|
62
|
+
> settings, environment, other repos or memory. Never read .env or dump environment/settings.
|
|
63
|
+
> Your tools are for editing source/test files and running the tests you wrote, nothing else.
|
|
64
|
+
> No installs, package-manager commands, containers, git operations, self-review or subagents.
|
|
65
|
+
> Run your newly written SectionNav cases ONCE with the existing Node/Vitest runtime,
|
|
66
|
+
> maxWorkers=2, selected by test names; no affected/full suite, build or app startup.
|
|
67
|
+
> If blocked, report it rather than changing repo state. Leave all changes uncommitted.
|
|
68
|
+
> The orchestrator's tests and independent reviews are the acceptance gate, not your run.
|
|
69
|
+
> Start PLAN, emit PROGRESS before relevant actions with no silence over two minutes,
|
|
70
|
+
> then exactly one final RESULT: DONE or RESULT: BLOCKED.
|
|
71
|
+
|
|
72
|
+
# C. Orchestrator only — NOT ADDRESSED TO THE IMPLEMENTER
|
|
73
|
+
|
|
74
|
+
> IMPLEMENTER STOP HERE. You ARE the initial implementation instance. No steps below belong to you.
|
|
75
|
+
|
|
76
|
+
Gate #1: operator explicitly approved this exact structural correction, preserved behavior
|
|
77
|
+
and appearance, tests/review, release and innoservice pin bump on 2026-10-13.
|
|
78
|
+
No source approval was inferred from the earlier registration or clarification question.
|
|
79
|
+
|
|
80
|
+
Dispatch: fresh native plain worker, requested codex/gpt-6-luna, full WO inline; inherited
|
|
81
|
+
cwd is workspace root, target is the absolute core repo above. Record accepted identity
|
|
82
|
+
and only exposed effective metadata. Assigned ownership is not technical containment.
|
|
83
|
+
Parent owns repo state, package version, changelog, README/register and release/pin.
|
|
84
|
+
|
|
85
|
+
Independent envelope/regression/duplication/tests lenses plus UI run concurrently on the
|
|
86
|
+
same frozen diff via fresh read-only codex/gpt-6-luna processes. Security reviewer is not
|
|
87
|
+
required: no auth/security behavior changes. Parent also checks exposure/permission parity.
|
|
88
|
+
Parent runs SectionNav and both existing AccountPage test files, then build. Existing main
|
|
89
|
+
publish workflow supplies the full shared-core regression suite and publishing evidence;
|
|
90
|
+
normal reviewed main push, no manual publish dispatch. Confirm successful publication of
|
|
91
|
+
3.13.1 before innoservice exact pin and lockfile update. Keep all app capture/axe checks.
|
|
92
|
+
Fresh app capture at 375/1280 must verify navigation and absence of the three list violations.
|
|
93
|
+
Finalize register with named reviews, deduplicated dispositions and actual test/release proof.
|
|
94
|
+
|
|
95
|
+
Accepted native implementer: `/root/kit_sectionnav`, explicit requested gpt-6-luna, high
|
|
96
|
+
reasoning, fresh context. Effective native provider/model metadata was not exposed and
|
|
97
|
+
remains unverified. Only the two assigned files changed; no author git/review/repo-state
|
|
98
|
+
operations. Author's single permitted test run failed before collection with temporary-file
|
|
99
|
+
EPERM (zero tests). Parent runs the authoritative gate through the normal runtime path.
|
|
100
|
+
|
|
101
|
+
Core acceptance: five fresh concurrent CLI reviews (envelope, regression, duplication,
|
|
102
|
+
tests, UI), all exit 0, banners confirm codex/gpt-6-luna, provider openai, read-only.
|
|
103
|
+
Deduplicated disposition: 1 raised, 0 accepted, worst accepted none. Tests lens R1 proposed
|
|
104
|
+
proving the MUI implementation identity against a fabricated class-bearing stub. Rejected:
|
|
105
|
+
the structural assertion fails against the actual pre-change direct div children, existing
|
|
106
|
+
selection/theme/drawer tests retain their behavior checks, and the source still directly
|
|
107
|
+
imports and renders the real MUI ListItemButton. No behavior or identity abstraction is added.
|
|
108
|
+
Other lenses report no findings. Parent targeted pass confirms callbacks/permission filtering,
|
|
109
|
+
labels and selected/remembered state unchanged; wrappers use disablePadding.
|
|
110
|
+
Parent gate: 21 tests in SectionNav.test.jsx, AccountPage.test.jsx and
|
|
111
|
+
AccountPage.profileStatements.test.jsx passed with maxWorkers=2; TypeScript build exit 0.
|
|
112
|
+
Publish workflow's full suite is justified by shared-core consumer ripple. App pin and fresh
|
|
113
|
+
consumer capture remain pending after the reviewed core source/release lands.
|