@geonosis/oxlint-plugin-biological-architecture 2.6.1 → 2.8.0

package/CHANGELOG.md

@@ -1,5 +1,64 @@
 # @geonosis/oxlint-plugin-biological-architecture
 
+## 2.8.0
+
+### Minor Changes
+
+- e880a6c: The tiers have no dead ends a consumer can reach (#1, #2). Three shapes from one migration had no
+  legal expression, because two rules together forbade every place the file could live:
+  
+  - **`tissue-no-data-props`** accepts layout configuration: a `number` or `boolean` prop the tissue
+    gives a literal default (`initialCount = 10`, `wrap = false`). A stateless adapter between two
+    cells is a tissue, and now it can be one. A string, a default that is not a literal of the prop's
+    type, and a primitive with no default are still data.
+  - **`no-hook-in-component-disguise`** accepts the client-boundary beacon: a file whose directive
+    prologue says `'use client'`, whose component calls a hook and returns only `null`. Its parent — a
+    Server Component, or a tissue under `tissue-no-hooks` — cannot call the hook, so there was no
+    real component to move it into. A `'use client'` component that returns its `children`, or calls
+    no hook, is still reported. The no-hook message now tells a layout about metadata
+    (`passthroughLayout`) and tells any other component nothing of the kind.
+  - **`no-invalid-feature-folders`** takes `roots` (default: none). Directly under each declared
+    directory, a folder that is not a tier is a finding per file, so an unmigrated pile
+    (`src/components/organisms/`) is debt the ratchet counts and moving a file into it grows the
+    count, instead of silently clearing every tier rule.
+  
+  `cell-must-not-compose-cell` keeps no exception; its fix now names the stateful host: two cells
+  side by side, coordinating through the store, arranged by a tissue.
+
+## 2.7.0
+
+### Minor Changes
+
+- 08c9653: The kit knows no document kind. `document-sagas-are-generic` and `documents-share-one-table` are
+  gone from the plugin and from the `backend-workflows` preset, with their corpus and fixtures: both
+  took a list of one consumer's document kinds (`perKindPrefixes`, `perKindTables`) and enforced a
+  vocabulary the kit has no business holding — a document entity is whatever extends `Document`, and
+  the registry a consumer builds is the only enumeration there is. Their intent is held by types
+  now: a consumer's per-entity maps are folds over the registry, and a per-entity saga has nowhere to
+  be written once `documentWorkflows` mints them. Their ids stay for one release: oxlint refuses a
+  config that names a rule its plugin does not export, so a config naming either still parses, the
+  id reports nothing and says once that it is retired, and `geonosis doctor` WARNs naming it where
+  it would have failed. Migration: remove the block before the next major, when the ids go; the
+  doctor's floor table no longer lists them.
+
+### Patch Changes
+
+- c8c5d51: `documentWorkflows(entity, { effects, ports })` at `@geonosis/document/workflows` — T3, the org
+  workflows over the engine, one body per Document: `create`, `save`, `setStatus`, `remove`, `send`
+  (durable), `convert`, `duplicate`, `share`, each named for the entity. A host decides
+  `effects.announce(fact)` — the product's events off a `DocumentFact` — and answers three ports,
+  `door`, `link` and `mail` (`Result<void, MailUndelivered>` from `better-result`); `beforeCreate`,
+  `conversion`, the mail words, `counter`, `actor`, `guard`, `now` and `seed` default, and a
+  counter-numbered entity or a template on a host that declares neither is refused by name. The
+  factory makes the two steps that reach the outside world, so the durable-body law holds on every
+  host. A third entry, like `./store`, so the core stays off every floor and browser-safe. The
+  Standard Schema builders the surface wrote for itself move to the core (`objectOf`, `enumOf`,
+  `optional`, `nullable`, `arrayOf`, `merge`, the scalars, `validate`) and `SchemaAdapter` is the
+  adapter's name (`SurfaceSchemaAdapter` stays as an alias). `@geonosis/workflows` and
+  `better-result` join the package's dependencies. The plugin's `document-sagas-are-generic` and
+  `documents-share-one-table` no longer carry `floor: pending`: the store shipped at 0.4 and the
+  workflows ship here. `documentIdentifiers` spells every identifier as a literal type off the name it was given (`CamelOf`, `DocumentIdentifiers`), so a consumer indexes a typed router map by the camel name without a cast. `documentEntities(map).keyed(pick)` answers one value per entity under its name, total by construction.
+
 ## 2.6.1
 
 ## 2.6.0

package/README.md

@@ -136,8 +136,6 @@ months. Carry these across, or turn the rule off:
 | `no-brand-names` | `{ brands, allowedSubstrings }` | refuses the run |
 | `organelle-dependency` | `{ sharedFeatures }` | only `shared` crosses a feature boundary |
 | `cell-no-cross-feature-organelles` | `{ sharedFeatures }` | only `shared` crosses a feature boundary |
-| `documents-share-one-table` | `{ perKindTables }` | refuses the run |
-| `document-sagas-are-generic` | `{ perKindPrefixes }` | refuses the run |
 | `step-opens-its-own-cell` | `{ engine, within, wrapper }` | refuses the run |
 | `no-raw-html-atoms` | the positional element→atom map | refuses the run |
 | `layer-walls` | `{ layers: [{ name, paths, mayImport }], externals? }` | refuses the run |

package/RULES.md

@@ -9,11 +9,14 @@ Read from the two source trees on 2026-08-29:
 One plugin name, two rule sets, no shared code. **33 rules had the same filename in both trees; only
 3 of those 33 were byte-identical.** A fix in one never reached the other.
 
-The union here is **54 rules**, plus twenty-eight the kit wrote for itself — `layer-walls`,
-`emit-declares-attempts`, `cell-no-cross-feature-organelles`, the `backend-medusa` ten,
-`no-unregistered-plugin-dir` + `router-schema-parity`, the `themekit` two,
-`no-mock-db-in-integration`, `todo-names-its-plan`, `boundary-time-is-primitive`,
-`durable-body-reads-through-steps`, `subpath-over-barrel` and the six of the W67 batch — **82**.
+The union here is **52 rules** — 54 when it was cut, less `document-sagas-are-generic` and
+`documents-share-one-table`, retired in 2.7.0 because both enforced one consumer's document kinds —
+plus twenty-eight the kit wrote for itself: `layer-walls`, `emit-declares-attempts`,
+`cell-no-cross-feature-organelles`, the `backend-medusa` ten, `no-unregistered-plugin-dir` +
+`router-schema-parity`, the `themekit` two, `no-mock-db-in-integration`, `todo-names-its-plan`,
+`boundary-time-is-primitive`, `durable-body-reads-through-steps`, `subpath-over-barrel` and the six
+of the W67 batch — **80**. The two retired ids still resolve for one release, as no-ops that say
+they are retired, so a config naming them parses; they go at the next major.
 Every one of the twenty-eight is marked **written here** in the inventory, and the table below is
 held against that column rather than kept by hand.
 
@@ -22,10 +25,10 @@ held against that column rather than kept by hand.
 | in both trees, byte-identical | 3 |
 | in both trees, **diverged** | 30 |
 | dielime only | 3 |
-| during.day only | 18 |
-| **union** | **54** |
+| during.day only | 16 |
+| **union** | **52** |
 | written here | 28 |
-| **total** | **82** |
+| **total** | **80** |
 
 ## How divergence was resolved
 
@@ -513,26 +516,6 @@ Closing it also fixed a hole in the barrel rule's own pattern, which required a
 therefore no-op'd on the cwd-relative `cells/index.ts` oxlint reports for a package whose own root
 holds `cells/`.
 
-### `documents-share-one-table` — matches `pgTable` as well as `sqliteTable`
-
-during.day's fork matched `literalNamedBy(node, 'sqliteTable')` and nothing else, and every tenant
-table on that tree has been `pgTable` since its plan 037. Its own probe: a well-formed per-kind
-table
-
-```ts
-export const invoices = pgTable('invoices', { id: text('id').primaryKey(), orgId: orgIdColumn() })
-```
-
-under `packages/db/src/schema/tenant/` produced **zero** findings from this rule; only the malformed
-variant was caught, by `tenant-tables-carry-org-id`, and by accident. The fork's rule had been dead
-since the tenant plane moved to Postgres.
-
-The kit matching both constructors is therefore a fix, not a widening for its own sake. It stands
-because the rule is about the second storage a base entity retired — which is true whatever the
-dialect — and because `dialect-through-the-seam` already says the dialect is a seam fact rather than
-a thing a document kind gets to have an opinion about. Both branches are exercised through the real
-binary in `src/rules/__fixtures__/packages/db/src/schema/tenant/`, so neither can go quiet again.
-
 ### `no-raw-html-atoms` — the tier it was always documented for (0.3.0)
 
 Its own description said levels 2-6 and it fired at every level, `atoms/` included, where a raw
@@ -567,13 +550,9 @@ rule the option refusal exists to stop, one option along.
 | `organelle-dependency` | `sharedFeatures` | `['shared']` — during.day's `charts`/`document` are config, not law |
 | `mutating-step-declares-retry` | `retryKey` | `maxRetries` — Medusa's key. `@geonosis/workflows` spells the same budget `retry: { limit, backoff }` (`create-step.ts`), and with the key compiled in a correctly budgeted step on that engine was a finding for ever |
 | `mutating-step-declares-retry` | `retryLimitKey` | `limit` — the key that holds the NUMBER when the budget is an object. #191: reading only the bare form let `retry: { limit: 0 }` pass, which is the same silence as `maxRetries: 0` |
-| `documents-share-one-table` | `perKindTables` | **refuses the run** |
 | `worker-handles-are-scoped` | `within` + `wrapper` | **refuses the run** — `withCells` and `/apps/api/src/` were compiled in, so the rule served one repo's layout and one repo's vocabulary and a scope opener called anything else read as unscoped for ever |
 | `subpath-over-barrel` | `packages` (`{ name, subpaths: { <subpath>: string[] } }[]`) | **refuses the run** — which of a repo's own packages expose subpaths, and which names live under each, is a map only that package's owner has |
 | `durable-body-reads-through-steps` | `ports` | **refuses the run** — what a body reaches the outside world through is the repo's vocabulary; `durableKey` / `durableValue` (default `durable: true`) say how a workflow declares itself durable |
-| `document-sagas-are-generic` | `perKindPrefixes` | **refuses the run** |
-| `document-sagas-are-generic` | `within` | **refuses the run** — `/packages/workflows/src/` was compiled in, so the same per-kind saga one directory over was silent and no other layout could enable the rule at all |
-| `document-sagas-are-generic` | `factories` | `['saga', 'createWorkflow']` — the callee was `saga` alone, so a per-entity `createWorkflow('invoice.create', …)` produced no finding (026-W2, measured). A name assembled from the entity is a template literal the rule cannot read: that is the mint the rule sends people to, not a gap in it |
 | `unique-query-step-name` | `queryStepCalls` | **refuses the run** — the two names it shipped with are Medusa framework calls with no counterpart, so on the org engine the rule had zero reach while reading as enforced (026-W2, measured through the binary). There is no second spelling to inherit: `@geonosis/workflows` has no query-step primitive, and a repeated step name there is DEFINED rather than lost — `reserve`, `reserve#2` in call order (`engine.ts:124`) |
 | `no-raw-html-atoms` | (positional) element→atom map | **refuses the run** |
 | `no-raw-html-atoms` | `scope` (second positional, `{ scope: ["cells", …] }`) | every file except the atom tier |
@@ -710,8 +689,6 @@ has, and one a kit that guessed at it would keep stale on somebody else's behalf
 | `compound-no-stores` | frontend | — | `directionRule` | yes | yes | diverged |
 | `constants-in-constants-file` | backend-workflows | — | `paths` | — | yes | during only |
 | `dialect-through-the-seam` | backend-workflows | — | `onImportSource` + `mergeVisitors` + `paths` | — | yes | during only |
-| `document-sagas-are-generic` | backend-workflows | `perKindPrefixes` | `paths` + `options` + `literal-call` | — | yes | during only |
-| `documents-share-one-table` | backend-workflows | `perKindTables` | `paths` + `options` + `literal-call` | — | yes | during only |
 | `durable-body-reads-through-steps` | backend-workflows | `ports` | `nodes` + `objects` + `steps` + `options` | — | — | **written here** (#167, design-026 §8) |
 | `effect-hook-naming` | frontend | — | `paths` | yes | yes | diverged |
 | `emit-declares-attempts` | backend-medusa | — | `options` | — | — | **written here** |
@@ -867,8 +844,6 @@ option not named here is one no rule reads.
 | `compound-no-stores` | — |
 | `constants-in-constants-file` | — |
 | `dialect-through-the-seam` | — |
-| `document-sagas-are-generic` | `factories`, `perKindPrefixes`, `within` |
-| `documents-share-one-table` | `perKindTables` |
 | `durable-body-reads-through-steps` | `durableKey`, `durableValue`, `ports`, `stepFactories`, `stepProxies`, `workflowFactories` |
 | `effect-hook-naming` | — |
 | `emit-declares-attempts` | `budgetHelpers`, `emitters` |
@@ -892,7 +867,7 @@ option not named here is one no rule reads.
 | `no-index-count-as-exact` | `estimateFields`, `labelKey` |
 | `no-inert-hidden-jsx` | — |
 | `no-inline-data-in-jsx` | — |
-| `no-invalid-feature-folders` | — |
+| `no-invalid-feature-folders` | `roots` |
 | `no-loader-side-effect-in-index-file` | `directories`, `resourceFactories` |
 | `no-logic-in-component-files` | — |
 | `no-mock-db-in-integration` | `dbModules`, `integrationDirs`, `mockNames` |
@@ -951,8 +926,6 @@ itself, and the doctor says so on the `exercised` line.
 
 | Rule | Floor | Shipped |
 |---|---|---|
-| `document-sagas-are-generic` | `document` | PENDING |
-| `documents-share-one-table` | `document` | PENDING |
 | `step-opens-its-own-cell` | `workflows` | yes |
 
 <!-- END GENERATED FLOORS -->

package/corpus/.oxlintrc.json

@@ -37,24 +37,6 @@
     "biological-architecture/compound-no-stores": "error",
     "biological-architecture/constants-in-constants-file": "error",
     "biological-architecture/dialect-through-the-seam": "error",
-    "biological-architecture/document-sagas-are-generic": [
-      "error",
-      {
-        "perKindPrefixes": [
-          "invoice"
-        ],
-        "within": "(?:__fixtures__|corpus)/domains/[^/]+/workflows/"
-      }
-    ],
-    "biological-architecture/documents-share-one-table": [
-      "error",
-      {
-        "perKindTables": [
-          "invoices",
-          "quotes"
-        ]
-      }
-    ],
     "biological-architecture/durable-body-reads-through-steps": [
       "error",
       {

package/corpus/manifest.json

@@ -12,8 +12,6 @@
     "biological-architecture/compound-no-stores",
     "biological-architecture/constants-in-constants-file",
     "biological-architecture/dialect-through-the-seam",
-    "biological-architecture/document-sagas-are-generic",
-    "biological-architecture/documents-share-one-table",
     "biological-architecture/durable-body-reads-through-steps",
     "biological-architecture/effect-hook-naming",
     "biological-architecture/emit-declares-attempts",

package/dist/{chunk-KV7JRNAP.js → chunk-24K5232K.js}

@@ -42,8 +42,6 @@ var BACKEND_WORKFLOWS = [
   "boundary-time-is-primitive",
   "constants-in-constants-file",
   "dialect-through-the-seam",
-  "document-sagas-are-generic",
-  "documents-share-one-table",
   "durable-body-reads-through-steps",
   "mutating-step-declares-retry",
   "no-date-string-compare",

package/dist/index.d.ts

@@ -36,6 +36,16 @@ type ProbeTree = {
 };
 type ProbeFn = (options: readonly unknown[]) => Probe | ProbeTree;
 
+type RetiredRule = Rule & {
+    meta: Rule['meta'] & {
+        retired: {
+            reason: string;
+            since: string;
+        };
+    };
+};
+declare const RETIRED_RULES: Record<'document-sagas-are-generic' | 'documents-share-one-table', RetiredRule>;
+
 type Rule = {
     create: (context: never) => Record<string, unknown>;
     fixShape: string;
@@ -123,4 +133,4 @@ declare const plugin: {
     rules: Record<string, Rule>;
 };
 
-export { Preset, type Rule, type RuleName, plugin as default, rules };
+export { Preset, RETIRED_RULES, type Rule, type RuleName, plugin as default, rules };