@akanjs/cli 3.0.0-alpha.33 → 3.0.0-alpha.35

package/.build-stamp

@@ -1 +1 @@
-92c020a25f9a2117ffc1aac1e1581b005e93ce7da63656e22aa9756cad18ea57
+19d013737b1748d534b589ea7399041a437db6f36d396f7ce0e2617c8883bf31

package/guidelines/conventions/conventions.instruction.md

@@ -59,10 +59,13 @@ that looks wrong; do not "fix" it back.
 - **Never redeclare a generated CRUD endpoint name** in `*.signal.ts` (`no-redeclare-predefined-endpoint.grit`).
 - **Never type a `*.Util.tsx` / `*.Zone.tsx` prop as a `cnst` model** (`no-model-type-in-util-zone.grit`). Those two
   roles are always client components, so a `cnst.Banner` / `cnst.LightBanner` prop is a class instance the server has
-  to hand across the boundary; take `bannerId: string` and read the model from the store instead. Two shapes are
-  exempt because neither is an instance: `cnst.<Enum>["value"]`, whose indexed access resolves to a string union, and
-  a `ClientInit` / `ClientView` / `ClientEdit` type argument, which the framework maps to `GetStateObject<…>` plain
-  data. Any *other* indexed access is still flagged — `cnst.Banner["image"]` is a `File`.
+  to hand across the boundary; take `bannerId: string` and read the model from the store instead. Three shapes are
+  exempt because none of them is an instance: `cnst.<Enum>["value"]`, whose indexed access resolves to a string
+  union; a `ClientInit` / `ClientView` / `ClientEdit` type argument, which the framework maps to `GetStateObject<…>`
+  plain data; and a `ModelsProps<cnst.Setting>` type argument, whose only use of the model is the `onClickItem`
+  callback, so nothing model-shaped ever lands on a prop. `ModelProps<"setting", cnst.LightSetting>` is *not* exempt
+  — that one spreads the model onto the props themselves, and `Unit` / `View`, which take it, are server components
+  this rule does not cover. Any *other* indexed access is still flagged — `cnst.Banner["image"]` is a `File`.
   **Only prop positions are read** — a `*Props` interface or type alias, and the inline object type on the
   component's own parameter. A `cnst` type that never leaves the file is not a boundary crossing and stays legal: a
   local annotation, a callback parameter the framework itself types with the model
@@ -678,6 +681,10 @@ apps and libs never import it directly (`no-import-external-library`) — everyt
   providers the way middleware is applied: `option.applyAdaptor(LlmAdaptorRole, ClaudeLlm)`, where the
   implementation is an `adapt()` class in a `srvkit/` implementing `LlmAdaptor.chat(request, onDelta?)` — ignore
   `onDelta` and the chat still answers whole.
+- **A dialog's close is the dialog's own dismissal, not a state flip.** `closeDialogIn<Ns>` (and `Dialog.Close`)
+  run through whatever `Dialog.Modal` registered, so the agent takes the exact path the X button takes —
+  `confirmClose` still prompts and `onCancel` still fires. A close that only set `open` to false would skip both,
+  which is a different action wearing the same name.
 - **`<Agent.Zone id="comments">` runs a second agent over one section, in parallel with the root.** Everything
   mounted inside — `st.use` subscriptions, hook tools, Guides — belongs to that zone's own conversation *and*
   stays visible to the root agent: **zones are views, never walls**, so wrapping a section costs the root nothing.
@@ -715,17 +722,36 @@ apps and libs never import it directly (`no-import-external-library`) — everyt
   callable still drives the click a person makes. It does not throw: a tool schema is built during render, and an
   agent-tooling mistake that aborted the render would cost the route its server rendering. `st.useState`'s `set`
   degrades the same way, to read-only.
-- **A component that renders once per row publishes nothing.** A tool registered under one name by fifty rows is
-  forty-nine collisions and one survivor. The container publishes one tool taking the id instead —
-  `removeTask(taskId)`, never fifty `removeTask` — and the agent reads the ids from the `<slice>.items` resource
-  `Load.Units` and `Data.ListContainer` already expose.
+- **A component that renders once per row never closes over its row's id — it takes the id as an argument.** A
+  tool that captured its own row would be fifty registrations of one name, forty-nine of them shadowed, and the
+  survivor would remove whichever row happened to mount last. Take the id instead — `removeTask(taskId)`, never
+  fifty `removeTask` — and every row's registration is then interchangeable, so a row component may publish after
+  all: say so with `shared: true` and the repeats are one declaration rather than a warned-about clash. The ids
+  come from the `<slice>.items` resource `Load.Units` and `Data.ListContainer` already expose. `shared` is a claim
+  about the *tool*, not a way to quiet a console: two rows whose tools would do different things (a different
+  `modal`, a different redirect) are not interchangeable, and that one wants a `namespace` or nothing at all.
 - **`akanjs/ui` publishes its own controls, so an app writes nothing for them.** `Data.ListContainer` (and every
-  `Model.AdminPanel`) publishes its toolbar and its row and modal verbs; `Load.Units`, `Load.Pagination` and
-  `Data.Pagination` publish `setPageOf<Model>`; `Layout.Sider`, `System.SelectLanguage`, `Link.Back` and
-  `System.ThemeToggle` publish the shell. **A component that can render twice on one screen takes a
-  `namespace` prop and publishes nothing without it** — `Tab`, `Dialog`, `ScreenNavigator`. Pass one
-  (`<Tab namespace="detail">`) and the tool becomes `switchTabInDetail`; leave it off and that tab is invisible to
-  the agent, because two tabs answering to `switchTab` would mean the first to mount loses.
+  `Model.AdminPanel`) publishes its toolbar and its row verbs; `Model.NewWrapper` (so `Model.New` too) publishes
+  `new<Model>`; `Load.Units`, `Load.Pagination` and `Data.Pagination` publish `setPageOf<Model>`; `Layout.Sider`,
+  `System.SelectLanguage`, `Link.Back` and `System.ThemeToggle` publish the shell. **A component that can render
+  twice on one screen takes a `namespace` prop and publishes nothing without it** — `Tab`, `Dialog`,
+  `ScreenNavigator`. Pass one (`<Tab namespace="detail">`) and the tool becomes `switchTabInDetail`; leave it off
+  and that tab is invisible to the agent, because two tabs answering to `switchTab` would mean the first to mount
+  loses. `Model.NewWrapper` takes the same prop but publishes without one, because its slice already names it —
+  a second create trigger for the same slice, opening a form seeded differently, is what needs the suffix.
+- **The `Model.*` row wrappers publish their verb, taking the id.** `Model.EditWrapper`, `Model.ViewWrapper`,
+  `Model.RemoveWrapper` and `Model.Remove` publish `edit<Model>` / `view<Model>` / `remove<Model>` with a
+  `modelId` argument, so a list built from `Load.Units` and an app's own `Unit` reaches the same verbs an
+  `AdminPanel` does. `Model.SureToRemove` publishes the same — except under `typeNameToRemove`, where it
+  publishes nothing: that gate makes a person retype the model's name, an approval card is one click, and
+  offering the lever at a friction the screen does not have is not the same control.
+- **A modal publishes its verbs while it is open, and only then.** `Model.EditModal` publishes `submit<Model>`
+  and `cancelEditOf<Model>`, `Model.ViewModal` publishes `closeViewOf<Model>`, and `Model.ViewEditModal`
+  publishes `edit<Model>` / `submit<Model>` / `closeViewOf<Model>` — each from a subtree that mounts with the
+  open modal, never from the component that merely holds it. That is what makes a list legal: `Data.CardList`
+  renders one editor per row, and at most one of them is ever open, so one name is registered rather than fifty.
+  It also means the verb is absent from the catalogue while nothing is open, which is honest — and costs the
+  agent nothing, because the catalogue is re-read on the turn that follows the tool call that opened the form.
 - **A form control publishes its own setter, and reading a form publishes one tool that fills several at once.**
   Both are free: an app writes no `st.tool` for a form. A `Field.*` / `Input.*` / `Select` / `Switch` handed
   `onChange={st.do.setTitleOnTask}` **by reference** publishes `setTitleOnTask` while it is on screen — the same

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@akanjs/cli",
-  "version": "3.0.0-alpha.33",
+  "version": "3.0.0-alpha.35",
   "sourceType": "module",
   "type": "module",
   "publishConfig": {
@@ -34,7 +34,7 @@
     "@langchain/openai": "^1.4.6",
     "@tailwindcss/node": "^4.3.0",
     "@trapezedev/project": "^7.1.4",
-    "akanjs": "3.0.0-alpha.33",
+    "akanjs": "3.0.0-alpha.35",
     "chalk": "^5.6.2",
     "commander": "^14.0.3",
     "dayjs": "^1.11.20",