@compean/core 0.1.0 → 0.1.1

package/RULES.md

@@ -83,7 +83,7 @@ packages/client/   features/<feature>/: controllers, client services, shared vie
 ## Controllers
 
 - A controller extends `Controller<TState>`. It exposes `state`, `initialize()` and its actions; the base class provides `start()`, `subscribe`, `dispose`, `notify()`, `track(cleanup)`, `load(task)` and `run(task)`.
-- `state` holds display-ready values only. Building them is private mapping inside the controller. Screens and components import only view-model types, never the functions that build them. Decisions call domain rules, not view-model fields. [§7.4, §9.1]
+- `state` holds display-ready values only. Building them is done by the controller's private methods, including helpers that don't use `this` (such as formatting a date), so all of a screen's mapping is found in one place. Screens and components import only view-model types, never the functions that build them. Decisions call domain rules, not view-model fields. [§7.4, §9.1]
 - Loading the screen's data goes through `load()`: before the screen is first ready, the status settles when every load started so far has finished ('ready', or 'error' with a retry); once ready it's a refresh whose failure shows as a toast. A screen that loads nothing never calls it and starts ready. Every action goes through `run()`, which never changes the status and shows failures as toasts. Both show a `UserError`'s message, show a generic message for anything else, and report bugs. [§10.7, §18]
 - An action must add something: `run()`, a state change, or combining services. One that only forwards a call is a pass-through. [§7.1]
 - Every decision that reads app state is made in the controller, in a method called directly where the data arrives (for example at the end of `load()`). Subscribe to a service only when the change comes from outside. [§18.2]
@@ -117,7 +117,7 @@ packages/client/   features/<feature>/: controllers, client services, shared vie
 - A service extends `Observable` only when screens must react to its changes live; otherwise it's a plain class. [§19.1]
 - Its interface speaks the domain: one named snapshot for displayed data, intent methods instead of setters, question methods only for domain rules several callers share. A getter per field is a red flag. [§19.6, §5.1]
 - A library (Zustand, query-core, Better Auth) may live inside a service, never in screens or controllers. [§5.1]
-- Dependencies come through the constructor, every argument named, no `undefined` placeholders. A default must be what production wants, so only `{ clock, random }` default (to the real ones) and a dependency never defaults. Product constants live in the module that owns the rule. [§8.2, §10.9]
+- Dependencies come through the constructor, every argument named, no `undefined` placeholders. Declare each as a field and assign it in the constructor; apps never use parameter properties (`constructor(private readonly x: X)`), because Node runs the API by stripping types only. `erasableSyntaxOnly` enforces it. A default must be what production wants, so only `{ clock, random }` default (to the real ones) and a dependency never defaults. Product constants live in the module that owns the rule. [§8.2, §10.9]
 
 ### Server data on the client
 

package/dist/eslint/index.js

@@ -2,6 +2,8 @@
 const PLATFORM_IMPORT = /^(react|react-dom|react-native|react-router|expo)(\/|$)|^react-native-|^react-router-|^@tanstack\/react-|^expo-|^@expo\/|^@react-navigation\//;
 /** What packages/domain may import: its own files and zod. */
 const DOMAIN_ALLOWED = /^(\.{1,2}\/|zod(\/|$))/;
+/** Tests may import their runner and test libraries, which the platform and purity rules would flag. */
+const TEST_FILES = ['**/*.test.{ts,tsx}'];
 /** TanStack Router calls whose options object defines a route. */
 const ROUTE_FACTORIES = new Set([
     'createFileRoute',
@@ -14,8 +16,8 @@ const ROUTE_FACTORIES = new Set([
 const LIBRARY_OWNERS = [
     {
         library: /^@tanstack\/(query-core|react-query)(\/|$)/,
-        owner: /\.service\.tsx?$/,
-        ownerName: 'a client service (*.service.ts)',
+        owner: /(\.service|(^|[\\/])boot)\.tsx?$/,
+        ownerName: 'a client service (*.service.ts) or boot.ts',
     },
     { library: /^better-auth(\/|$)/, owner: /(^|[\\/])auth\.service\.tsx?$/, ownerName: 'the auth service (auth.service.ts)' },
 ];
@@ -135,10 +137,12 @@ export function compeanRules() {
         { plugins: { compean: plugin } },
         {
             files: ['packages/client/**/*.{ts,tsx}', '**/*.controller.{ts,tsx}'],
+            ignores: TEST_FILES,
             rules: { 'compean/platform-free': 'error' },
         },
         {
             files: ['packages/domain/**/*.{ts,tsx}'],
+            ignores: TEST_FILES,
             rules: { 'compean/domain-pure': 'error' },
         },
         {
@@ -147,6 +151,7 @@ export function compeanRules() {
         },
         {
             files: ['packages/client/**/*.{ts,tsx}', 'apps/web/**/*.{ts,tsx}', 'apps/mobile/**/*.{ts,tsx}'],
+            ignores: TEST_FILES,
             rules: { 'compean/library-owners': 'error' },
         },
     ];

package/dist/registry.d.ts

@@ -1,6 +1,13 @@
 type Token<T> = abstract new (...args: never[]) => T;
-/** Makes `instance` the one `$S(token)` returns. Called from boot.ts and from tests. */
-export declare function register<T>(token: Token<T>, instance: T): void;
+/** What callers of `$S(token)` can use: the class's public members, so a test fake needs only those. */
+type PublicShape<T> = {
+    [K in keyof T]: T[K];
+};
+/**
+ * Makes `instance` the one `$S(token)` returns. Called from boot.ts and from tests. The type comes
+ * from the class alone; `instance` must offer its public members, so a fake with extra fields fits.
+ */
+export declare function register<T>(token: Token<T>, instance: NoInfer<PublicShape<T>>): void;
 /**
  * The service registered for this class, or a framework default (ToastService, ErrorReporter).
  *

package/dist/registry.js

@@ -9,7 +9,10 @@ const defaults = new Map([
     [ToastService, () => new ToastService()],
     [ErrorReporter, () => new ConsoleReporter()],
 ]);
-/** Makes `instance` the one `$S(token)` returns. Called from boot.ts and from tests. */
+/**
+ * Makes `instance` the one `$S(token)` returns. Called from boot.ts and from tests. The type comes
+ * from the class alone; `instance` must offer its public members, so a fake with extra fields fits.
+ */
 export function register(token, instance) {
     services.set(token, instance);
 }

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@compean/core",
-    "version": "0.1.0",
+    "version": "0.1.1",
     "description": "Shared core for Compean apps: controllers, services, errors, forms and the framework rules.",
     "license": "MIT",
     "type": "module",