Bugfix/primitives cleanup

CHANGELOG.md

@@ -6,6 +6,54 @@ This project follows [Semantic Versioning](https://semver.org/).
 
 ---
 
+## [1.4.0] — 2026-04-11
+
+Cleanup release. Removes six public aliases that contradicted the SibuJS philosophy — plain verbs, no framework ceremony, no redundant synonyms for the same primitive. All of the removed APIs were either one-line forwards to an existing primitive or identity wrappers; every existing example can be rewritten by deleting the wrapper and calling the underlying primitive directly.
+
+### Removed
+
+- **`createSignal`** — was `return signal(value)`. Use `signal()` directly.
+- **`createMemo`** — was `return derived(fn)`. Use `derived()` directly.
+- **`createEffect`** — was `return effect(fn)`. Use `effect()` directly.
+- **`memo`** — was `return derived(factory)`. Use `derived()` directly.
+- **`memoFn`** — was `return derived(callback)`. Use `derived()` directly.
+- **`composable`** — was `return setup` (identity function). Plain functions are already composables in SibuJS; just write one and call it.
+
+The three removed files (`src/patterns/primitives.ts`, `src/core/signals/memo.ts`, `src/core/signals/memoFn.ts`) are currently empty stubs exporting nothing — they can be deleted from disk in a follow-up commit without further code changes.
+
+### Migration
+
+```ts
+// before
+import { createSignal, createMemo, createEffect, memo, memoFn, composable } from "sibujs";
+
+const [count, setCount] = createSignal(0);
+const doubled = createMemo(() => count() * 2);
+const sorted = memo(() => items().slice().sort());
+const handler = memoFn(() => (e: Event) => process(e));
+createEffect(() => console.log(count()));
+const useCounter = composable(() => { /* … */ });
+
+// after
+import { signal, derived, effect } from "sibujs";
+
+const [count, setCount] = signal(0);
+const doubled = derived(() => count() * 2);
+const sorted = derived(() => items().slice().sort());
+const handler = derived(() => (e: Event) => process(e));
+effect(() => console.log(count()));
+function useCounter() { /* … */ }
+```
+
+### Also updated
+
+- `generateComponentMetadata`, `generateTypeStubs`, and the Vite/Webpack pure-annotation factory list in `sibujs/build` no longer mention the removed names.
+- Lint rule `no-signals-in-conditionals` no longer checks `memo` / `memoFn` (they don't exist).
+- `SignalNodeSnapshot.kind` comment updated to drop the `"memo"` tag.
+- Test suite: `tests/primitives.test.ts`, `tests/memo.test.ts`, `tests/memoFn.test.ts` reduced to placeholder stubs; `tests/types.test.ts` and `tests/ide.test.ts` updated to assert the aliases are gone. Suite: **2105/2105 passing** (down from 2113 by exactly the 8 deleted alias-specific tests).
+
+---
+
 ## [1.3.0] — 2026-04-11
 
 Large minor release. Adds **27 new reactive/DOM primitives**, a full **SSR + OWASP security hardening pass** (A01, A02, A03, A10 + CWE-1321 prototype pollution), **10 ergonomic features** that stay inside the SibuJS philosophy (No VDOM, No JSX, No compilation, Zero dependencies, fine-grained reactivity), **typed tag factory overloads** for common elements, and a new **`tag(props, children)` positional shorthand** that removes the need for the `nodes:` key at every level of the tree. Test suite grew from **1875 → 2113** passing tests (+238, **0 regressions**).

index.ts

@@ -57,8 +57,6 @@ export * from "./src/core/signals/derived";
 export * from "./src/core/signals/watch";
 export * from "./src/core/signals/store";
 export * from "./src/core/signals/ref";
-export * from "./src/core/signals/memo";
-export * from "./src/core/signals/memoFn";
 export * from "./src/core/signals/array";
 export * from "./src/core/signals/deepSignal";
 export * from "./src/core/signals/writable";

package.json

@@ -1,6 +1,6 @@
 {
   "name": "sibujs",
-  "version": "1.3.0",
+  "version": "1.4.0",
   "description": "A lightweight, function-based frontend framework that combines the best of React, Svelte, and Vue — with zero VDOM and maximum simplicity. Designed for developers who want fine-grained reactivity and full control without compilation or magic.",
   "keywords": [
     "frontend",

patterns.ts

@@ -10,7 +10,6 @@ export * from "./src/patterns/persist";
 export * from "./src/patterns/optimistic";
 export * from "./src/patterns/timeTravel";
 export * from "./src/patterns/globalStore";
-export * from "./src/patterns/primitives";
 
 // Component patterns
 export * from "./src/patterns/hoc";

src/build/analyzer.ts

@@ -46,14 +46,11 @@ export const moduleSizes: Record<string, number> = {
   "core/watch": 300,
   "core/store": 380,
   "core/ref": 150,
-  "core/memo": 180,
-  "core/memoFn": 160,
   "core/array": 420,
   "core/deepSignal": 500,
   "core/lifecycle": 300,
   "core/context": 350,
   "core/persist": 400,
-  "core/primitives": 200,
   "core/hoc": 280,
   "core/transition": 600,
   "core/form": 750,

src/build/ide.ts

@@ -68,31 +68,6 @@ export function getComponentMetadata(): ComponentMeta[] {
         },
       ],
     },
-    {
-      name: "memo",
-      description:
-        "Returns a memoized value that only recomputes when its reactive dependencies change. Alias for derived.",
-      props: [
-        {
-          name: "factory",
-          type: "() => T",
-          required: true,
-          description: "Function that computes the memoized value",
-        },
-      ],
-    },
-    {
-      name: "memoFn",
-      description: "Returns a memoized callback that only updates when its reactive dependencies change.",
-      props: [
-        {
-          name: "callback",
-          type: "() => T",
-          required: true,
-          description: "The callback factory function to memoize",
-        },
-      ],
-    },
     {
       name: "ref",
       description:
@@ -140,44 +115,6 @@ export function getComponentMetadata(): ComponentMeta[] {
       ],
     },
 
-    // ── SolidJS-style Primitives ─────────────────────────────────────
-    {
-      name: "createSignal",
-      description: "Creates a reactive signal. SolidJS-style alias for signal. Returns [getter, setter].",
-      props: [
-        {
-          name: "value",
-          type: "T",
-          required: true,
-          description: "Initial value",
-        },
-      ],
-    },
-    {
-      name: "createMemo",
-      description: "Creates a derived/computed reactive value. SolidJS-style alias for derived.",
-      props: [
-        {
-          name: "fn",
-          type: "() => T",
-          required: true,
-          description: "Computation function that reads other signals",
-        },
-      ],
-    },
-    {
-      name: "createEffect",
-      description: "Creates a reactive side effect. SolidJS-style alias for effect.",
-      props: [
-        {
-          name: "fn",
-          type: "() => void",
-          required: true,
-          description: "Effect function that reads reactive signals",
-        },
-      ],
-    },
-
     // ── Lifecycle ────────────────────────────────────────────────────
     {
       name: "onMount",
@@ -531,11 +468,6 @@ export function generateVSCodeSnippets(): Record<
       ],
       description: "Create a reactive form with validation",
     },
-    "SibuJS createSignal": {
-      prefix: "sibu-signal",
-      body: ["const [${1:value}, ${2:setValue}] = createSignal(${3:initialValue});"],
-      description: "Create a reactive signal (SolidJS-style alias for signal)",
-    },
   };
 }
 
@@ -583,12 +515,6 @@ export function generateTypeStubs(): Record<string, string> {
 
     derived: ["declare function derived<T>(getter: () => T): () => T;"].join("\n"),
 
-    memo: ["declare function memo<T>(factory: () => T): () => T;"].join("\n"),
-
-    memoFn: ["declare function memoFn<T extends (...args: unknown[]) => unknown>(callback: () => T): () => T;"].join(
-      "\n",
-    ),
-
     ref: [
       "interface Ref<T> { current: T; }",
       "declare function ref<T>(initial: T): Ref<T>;",
@@ -610,14 +536,6 @@ export function generateTypeStubs(): Record<string, string> {
       "declare function store<T extends object>(initialState: T): [{ readonly [K in keyof T]: T[K] }, StoreActions<T>];",
     ].join("\n"),
 
-    createSignal: ["declare function createSignal<T>(value: T): [() => T, (next: T | ((prev: T) => T)) => void];"].join(
-      "\n",
-    ),
-
-    createMemo: ["declare function createMemo<T>(fn: () => T): () => T;"].join("\n"),
-
-    createEffect: ["declare function createEffect(fn: () => void): () => void;"].join("\n"),
-
     mount: [
       "declare function mount(component: (() => HTMLElement) | HTMLElement | Node, container: HTMLElement | null): { node: Node; unmount: () => void };",
     ].join("\n"),

src/build/linting.ts

@@ -45,7 +45,7 @@ export const lintRules = {
     description: "Signal functions should not be called inside conditionals, loops, or nested functions",
     check(source: string): LintViolation[] {
       const violations: LintViolation[] = [];
-      const hookNames = ["signal", "effect", "derived", "memo", "memoFn", "ref", "watch", "store"];
+      const hookNames = ["signal", "effect", "derived", "ref", "watch", "store"];
       const hookPattern = new RegExp(`\\b(${hookNames.join("|")})\\s*\\(`, "g");
       const lines = source.split("\n");
 

src/build/vite.ts

@@ -77,7 +77,6 @@ function injectPureAnnotations(code: string): string {
   const sibuFactories = [
     "tagFactory",
     "context",
-    "composable",
     "defineComponent",
     "withProps",
     "withDefaults",

src/build/webpack.ts

@@ -16,7 +16,6 @@ export interface SibuWebpackPluginOptions {
 const PURE_FACTORIES = [
   "tagFactory",
   "context",
-  "composable",
   "defineComponent",
   "withProps",
   "withDefaults",

src/core/signals/memo.ts

@@ -1,16 +1,4 @@
-import { derived } from "./derived";
-import type { Accessor } from "./signal";
-
-/**
- * memo returns a memoized value that only recomputes when its
- * reactive dependencies change. This is semantically identical to
- * derived but named for convenience.
- *
- * Use this to avoid expensive computations on every render cycle.
- *
- * @param factory Function that computes the memoized value
- * @returns Getter function that returns the memoized value
- */
-export function memo<T>(factory: () => T): Accessor<T> {
-  return derived(factory);
-}
+// Removed in 1.4.0: memo was a thin alias for derived. Use derived()
+// directly. This file is kept as an empty stub and can be removed in a
+// follow-up commit.
+export {};

src/core/signals/memoFn.ts

@@ -1,14 +1,4 @@
-import { derived } from "./derived";
-import type { Accessor } from "./signal";
-
-/**
- * memoFn returns a memoized callback function that only updates
- * when its reactive dependencies change. This prevents unnecessary
- * re-creations of callback functions passed to child components.
- *
- * @param callback The callback function to memoize
- * @returns Getter that returns the current memoized callback
- */
-export function memoFn<T extends (...args: any[]) => any>(callback: () => T): Accessor<T> {
-  return derived(callback);
-}
+// Removed in 1.4.0: memoFn was a thin alias for derived. Use derived()
+// directly. This file is kept as an empty stub and can be removed in a
+// follow-up commit.
+export {};

src/core/signals/signal.ts

@@ -7,7 +7,7 @@ import { isDev } from "../dev";
 declare const __accessor: unique symbol;
 
 /**
- * A reactive signal getter returned by signal(), derived(), memo(), and similar primitives.
+ * A reactive signal getter returned by signal(), derived(), and similar primitives.
  *
  * Pass an Accessor directly into reactive prop positions — never call it there:
  * ```ts

src/devtools/signalGraph.ts

@@ -50,7 +50,7 @@ export interface SignalNodeSnapshot {
   id: string;
   /** Debug name, if the caller tagged the signal. */
   name: string | null;
-  /** Runtime type tag: `"signal"`, `"derived"`, `"memo"`, `"effect"`. */
+  /** Runtime type tag: `"signal"`, `"derived"`, `"effect"`. */
   kind: string;
   /** Best-effort preview of the current value. */
   value: string;

src/patterns/composable.ts

@@ -1,14 +1,10 @@
 // ============================================================================
-// COMPOSABLES & COMPONENT PATTERNS
+// COMPONENT PATTERNS
 // ============================================================================
-
-/**
- * composable wraps a setup function to create reusable stateful logic.
- * Similar to Vue 3 composables — encapsulates reactive state and methods.
- */
-export function composable<T>(setup: () => T): () => T {
-  return setup;
-}
+//
+// Note: `composable()` was removed in 1.4.0 — it was an identity wrapper
+// (`return setup`) that added nothing over calling the setup function
+// directly. Plain functions are already composables in SibuJS.
 
 /**
  * RenderProp implements the render-prop pattern.

src/patterns/primitives.ts

@@ -1,60 +1,5 @@
-import { derived } from "../core/signals/derived";
-import { effect } from "../core/signals/effect";
-import { signal } from "../core/signals/signal";
-
-/**
- * SolidJS-style reactive primitives — standalone APIs that don't require
- * being inside a component. These are thin wrappers around the signal system.
- */
-
-/**
- * Creates a reactive signal. Equivalent to signal but with SolidJS naming.
- *
- * @param value Initial value
- * @returns Tuple [getter, setter]
- *
- * @example
- * ```ts
- * const [count, setCount] = createSignal(0);
- * console.log(count()); // 0
- * setCount(5);
- * ```
- */
-export function createSignal<T>(value: T): [() => T, (next: T | ((prev: T) => T)) => void] {
-  return signal(value);
-}
-
-/**
- * Creates a derived/computed reactive value. Equivalent to derived.
- *
- * @param fn Computation function that reads other signals
- * @returns Getter for the computed value
- *
- * @example
- * ```ts
- * const [count] = createSignal(5);
- * const doubled = createMemo(() => count() * 2);
- * console.log(doubled()); // 10
- * ```
- */
-export function createMemo<T>(fn: () => T): () => T {
-  return derived(fn);
-}
-
-/**
- * Creates a reactive side effect. Equivalent to effect.
- *
- * @param fn Effect function that reads reactive signals
- * @returns Cleanup/teardown function
- *
- * @example
- * ```ts
- * const [count] = createSignal(0);
- * const cleanup = createEffect(() => {
- *   console.log("Count is:", count());
- * });
- * ```
- */
-export function createEffect(fn: () => void): () => void {
-  return effect(fn);
-}
+// Removed in 1.4.0: createSignal / createMemo / createEffect were thin
+// aliases for signal / derived / effect. Use the canonical primitives from
+// "sibujs" instead. This file is kept as an empty stub to avoid a delete
+// in this change set; it can be removed in a follow-up commit.
+export {};

src/plugins/ecosystem.ts

@@ -86,7 +86,7 @@ export const bundlerMetadata = {
   sideEffects: false as const,
   modules: {
     core: ["html", "mount", "each", "slots", "fragment", "catch", "portal", "directives"],
-    hooks: ["signal", "effect", "derived", "watch", "store", "ref", "memo", "memoFn", "array", "deepSignal"],
+    hooks: ["signal", "effect", "derived", "watch", "store", "ref", "array", "deepSignal"],
     plugins: ["router", "i18n"],
     components: ["ErrorBoundary", "Loading"],
     ssr: ["ssr"],