@pikku/core 0.12.0 → 0.12.2

package/src/services/user-session-service.test.ts

@@ -0,0 +1,113 @@
+import { describe, test } from 'node:test'
+import assert from 'node:assert'
+import {
+  PikkuSessionService,
+  createMiddlewareSessionWireProps,
+  createFunctionSessionWireProps,
+} from './user-session-service.js'
+
+describe('PikkuSessionService', () => {
+  test('should set and get session', () => {
+    const service = new PikkuSessionService()
+    service.setInitial({ userId: 'user-1' })
+    const session = service.get()
+    assert.deepStrictEqual(session, { userId: 'user-1' })
+  })
+
+  test('should return undefined when no session set', () => {
+    const service = new PikkuSessionService()
+    assert.strictEqual(service.get(), undefined)
+  })
+
+  test('should mark sessionChanged when set is called', () => {
+    const service = new PikkuSessionService()
+    assert.strictEqual(service.sessionChanged, false)
+    service.set({ userId: 'user-1' })
+    assert.strictEqual(service.sessionChanged, true)
+  })
+
+  test('should clear session', () => {
+    const service = new PikkuSessionService()
+    service.setInitial({ userId: 'user-1' })
+    service.clear()
+    assert.strictEqual(service.get(), undefined)
+    assert.strictEqual(service.sessionChanged, true)
+  })
+
+  test('should freeze initial session', () => {
+    const service = new PikkuSessionService()
+    service.setInitial({ userId: 'user-1' })
+    const frozen = service.freezeInitial()
+    assert.deepStrictEqual(frozen, { userId: 'user-1' })
+  })
+
+  test('should only freeze initial once', () => {
+    const service = new PikkuSessionService()
+    service.setInitial({ userId: 'user-1' })
+    service.freezeInitial()
+    service.set({ userId: 'user-2' })
+    const frozen2 = service.freezeInitial()
+    assert.deepStrictEqual(frozen2, { userId: 'user-1' })
+  })
+
+  test('should throw when channelStore provided without channelId', () => {
+    assert.throws(() => new PikkuSessionService({} as any), {
+      message: 'Channel ID is required when using channel store',
+    })
+  })
+
+  test('should use channelStore when provided', async () => {
+    let storedSession: any
+    const store = {
+      setUserSession: (id: string, session: any) => {
+        storedSession = session
+      },
+      getChannelAndSession: (id: string) => ({ session: { userId: 'stored' } }),
+    }
+    const service = new PikkuSessionService(store as any, 'ch-1')
+    service.set({ userId: 'new' })
+    assert.deepStrictEqual(storedSession, { userId: 'new' })
+
+    const session = service.get()
+    assert.deepStrictEqual(session, { userId: 'stored' })
+  })
+})
+
+describe('createMiddlewareSessionWireProps', () => {
+  test('should return session wire props', () => {
+    const service = new PikkuSessionService()
+    service.setInitial({ userId: 'user-1' })
+    const props = createMiddlewareSessionWireProps(service)
+    assert.ok('session' in props)
+    assert.ok('setSession' in props)
+    assert.ok('getSession' in props)
+    assert.ok('hasSessionChanged' in props)
+  })
+
+  test('setSession should call setInitial', () => {
+    const service = new PikkuSessionService()
+    const props = createMiddlewareSessionWireProps(service)
+    props.setSession({ userId: 'user-2' })
+    assert.deepStrictEqual(service.get(), { userId: 'user-2' })
+  })
+})
+
+describe('createFunctionSessionWireProps', () => {
+  test('should return function session wire props', () => {
+    const service = new PikkuSessionService()
+    service.setInitial({ userId: 'user-1' })
+    const props = createFunctionSessionWireProps(service)
+    assert.ok('session' in props)
+    assert.ok('setSession' in props)
+    assert.ok('clearSession' in props)
+    assert.ok('getSession' in props)
+    assert.ok('hasSessionChanged' in props)
+  })
+
+  test('session should be frozen initial', () => {
+    const service = new PikkuSessionService()
+    service.setInitial({ userId: 'user-1' })
+    const props = createFunctionSessionWireProps(service)
+    assert.deepStrictEqual(props.session, { userId: 'user-1' })
+  })
+})

package/src/services/user-session-service.ts

@@ -1,5 +1,5 @@
-import { ChannelStore } from '../wirings/channel/channel-store.js'
-import { CoreUserSession } from '../types/core.types.js'
+import type { ChannelStore } from '../wirings/channel/channel-store.js'
+import type { CoreUserSession } from '../types/core.types.js'
 
 export interface SessionService<UserSession extends CoreUserSession> {
   sessionChanged: boolean
@@ -66,7 +66,7 @@ export function createMiddlewareSessionWireProps<
   UserSession extends CoreUserSession,
 >(session: SessionService<UserSession>) {
   return {
-    session: session.get(),
+    session: session.get() as UserSession | undefined,
     setSession: (s: UserSession) => session.setInitial(s),
     getSession: () => session.get(),
     hasSessionChanged: () => session.sessionChanged,

package/src/services/workflow-service.ts

@@ -1,10 +1,5 @@
-import {
-  SerializedError,
-  CoreSingletonServices,
-  CreateWireServices,
-  CoreConfig,
-} from '../types/core.types.js'
-import {
+import type { SerializedError } from '../types/core.types.js'
+import type {
   WorkflowRun,
   WorkflowRunWire,
   StepState,
@@ -37,11 +32,6 @@ export interface WorkflowService {
 
   // Orchestration operations
   resumeWorkflow(runId: string): Promise<void>
-  setServices(
-    singletonServices: CoreSingletonServices,
-    createWireServices: CreateWireServices | undefined,
-    config: CoreConfig
-  ): void
   startWorkflow<I>(
     name: string,
     input: I,

package/src/time-utils.test.ts

@@ -1,10 +1,10 @@
 import { describe, test } from 'node:test'
 import assert from 'node:assert/strict'
 
+import type { RelativeTimeInput } from './time-utils'
 import {
   getRelativeTimeOffset,
   getRelativeTimeOffsetFromNow,
-  RelativeTimeInput,
 } from './time-utils'
 
 const approxEqual = (a: number, b: number, delta: number = 10): boolean =>

package/src/types/core.types.ts

@@ -1,33 +1,35 @@
 import type { Logger, LogLevel } from '../services/logger.js'
-import { VariablesService } from '../services/variables-service.js'
-import { SecretService } from '../services/secret-service.js'
-import { SchemaService } from '../services/schema-service.js'
-import { JWTService } from '../services/jwt-service.js'
-import { PikkuHTTP } from '../wirings/http/http.types.js'
-import {
+import type { VariablesService } from '../services/variables-service.js'
+import type { SecretService } from '../services/secret-service.js'
+import type { SchemaService } from '../services/schema-service.js'
+import type { JWTService } from '../services/jwt-service.js'
+import type { PikkuHTTP } from '../wirings/http/http.types.js'
+import type {
   PikkuChannel,
   CorePikkuChannelMiddleware,
   CorePikkuChannelMiddlewareFactory,
 } from '../wirings/channel/channel.types.js'
-import { PikkuRPC } from '../wirings/rpc/rpc-types.js'
-import { PikkuMCP } from '../wirings/mcp/mcp.types.js'
-import { PikkuScheduledTask } from '../wirings/scheduler/scheduler.types.js'
-import { PikkuQueue, QueueService } from '../wirings/queue/queue.types.js'
-import { PikkuCLI } from '../wirings/cli/cli.types.js'
-import {
+import type { PikkuRPC } from '../wirings/rpc/rpc-types.js'
+import type { PikkuMCP } from '../wirings/mcp/mcp.types.js'
+import type { PikkuScheduledTask } from '../wirings/scheduler/scheduler.types.js'
+import type { PikkuQueue, QueueService } from '../wirings/queue/queue.types.js'
+import type { PikkuCLI } from '../wirings/cli/cli.types.js'
+import type {
   PikkuWorkflowWire,
   WorkflowService,
   WorkflowServiceConfig,
   WorkflowStepWire,
 } from '../wirings/workflow/workflow.types.js'
 import type { PikkuGraphWire } from '../wirings/workflow/graph/workflow-graph.types.js'
-import { PikkuTrigger } from '../wirings/trigger/trigger.types.js'
-import { SchedulerService } from '../services/scheduler-service.js'
-import { DeploymentService } from '../services/deployment-service.js'
-import { AIStorageService } from '../services/ai-storage-service.js'
-
-import { AIAgentRunnerService } from '../services/ai-agent-runner-service.js'
-import { AIRunStateService } from '../services/ai-run-state-service.js'
+import type { PikkuTrigger } from '../wirings/trigger/trigger.types.js'
+import type { PikkuGateway } from '../wirings/gateway/gateway.types.js'
+import type { SchedulerService } from '../services/scheduler-service.js'
+import type { DeploymentService } from '../services/deployment-service.js'
+import type { AIStorageService } from '../services/ai-storage-service.js'
+
+import type { ContentService } from '../services/content-service.js'
+import type { AIAgentRunnerService } from '../services/ai-agent-runner-service.js'
+import type { AIRunStateService } from '../services/ai-run-state-service.js'
 import type { PikkuAIMiddlewareHooks } from '../wirings/ai-agent/ai-agent.types.js'
 
 export type PikkuWiringTypes =
@@ -41,6 +43,7 @@ export type PikkuWiringTypes =
   | 'cli'
   | 'workflow'
   | 'agent'
+  | 'gateway'
 
 export interface FunctionServicesMeta {
   optimized: boolean
@@ -101,10 +104,13 @@ export type FunctionRuntimeMeta = {
   expose?: boolean
   remote?: boolean
   mcp?: boolean
+  readonly?: boolean
   sessionless?: boolean
   version?: number
   requiresApproval?: boolean
   contractHash?: string
+  inputHash?: string
+  outputHash?: string
 }
 
 export type FunctionMeta = FunctionRuntimeMeta &
@@ -208,6 +214,8 @@ export interface CoreSingletonServices<Config extends CoreConfig = CoreConfig> {
   /** AI agent storage service (threads, messages, working memory) */
   aiStorage?: AIStorageService
 
+  /** Content service for file storage and retrieval */
+  content?: ContentService
   /** AI agent runner service (model calls + tool loop) */
   aiAgentRunner?: AIAgentRunnerService
   /** AI run state service (run lifecycle + approval persistence) */
@@ -243,6 +251,7 @@ export type PikkuWire<
   workflowStep: WorkflowStepWire
   graph: PikkuGraphWire
   trigger: PikkuTrigger<TriggerOutput>
+  gateway: PikkuGateway
   session: HasInitialSession extends true
     ? UserSession
     : UserSession | undefined

package/src/types/state.types.ts

@@ -1,44 +1,54 @@
-import { PikkuError, ErrorDetails } from '../errors/error-handler.js'
-import {
+import type { PikkuError, ErrorDetails } from '../errors/error-handler.js'
+import type {
   CorePikkuFunctionConfig,
   CorePermissionGroup,
   CorePikkuPermission,
 } from '../function/functions.types.js'
-import { CorePikkuTriggerFunctionConfig } from '../wirings/trigger/trigger.types.js'
-import { CoreChannel, ChannelsMeta } from '../wirings/channel/channel.types.js'
-import { CLIMeta, CLIProgramState } from '../wirings/cli/cli.types.js'
-import {
+import type { CorePikkuTriggerFunctionConfig } from '../wirings/trigger/trigger.types.js'
+import type {
+  CoreChannel,
+  ChannelsMeta,
+} from '../wirings/channel/channel.types.js'
+import type { CLIMeta, CLIProgramState } from '../wirings/cli/cli.types.js'
+import type {
   HTTPMethod,
   CoreHTTPFunctionWiring,
   HTTPWiringsMeta,
 } from '../wirings/http/http.types.js'
-import {
+import type {
   CoreMCPResource,
   MCPResourceMeta,
   MCPToolMeta,
   CoreMCPPrompt,
   MCPPromptMeta,
 } from '../wirings/mcp/mcp.types.js'
-import { CoreAIAgent, AIAgentMeta } from '../wirings/ai-agent/ai-agent.types.js'
-import {
+import type {
+  CoreAIAgent,
+  AIAgentMeta,
+} from '../wirings/ai-agent/ai-agent.types.js'
+import type {
+  CoreGateway,
+  GatewaysMeta,
+} from '../wirings/gateway/gateway.types.js'
+import type {
   CoreQueueWorker,
   QueueWorkersMeta,
 } from '../wirings/queue/queue.types.js'
-import {
+import type {
   CoreScheduledTask,
   ScheduledTasksMeta,
 } from '../wirings/scheduler/scheduler.types.js'
-import {
+import type {
   CoreWorkflow,
   WorkflowsRuntimeMeta,
 } from '../wirings/workflow/workflow.types.js'
-import {
+import type {
   CoreTrigger,
   CoreTriggerSource,
   TriggerMeta,
   TriggerSourceMeta,
 } from '../wirings/trigger/trigger.types.js'
-import {
+import type {
   FunctionsMeta,
   CorePikkuMiddleware,
   CorePikkuMiddlewareGroup,
@@ -70,7 +80,10 @@ export interface PikkuPackageState {
       }
     >
     /** Maps namespace aliases to package config (e.g., 'ext' -> { package: '@pikku/...', rpcEndpoint: '...' }) */
-    externalPackages: Map<string, { package: string; rpcEndpoint?: string }>
+    addons: Map<
+      string,
+      { package: string; rpcEndpoint?: string; auth?: boolean; tags?: string[] }
+    >
   }
   http: {
     middleware: Map<string, CorePikkuMiddleware<any, any>[]>
@@ -112,6 +125,10 @@ export interface PikkuPackageState {
     agents: Map<string, CoreAIAgent>
     agentsMeta: AIAgentMeta
   }
+  gateway: {
+    gateways: Map<string, CoreGateway>
+    meta: GatewaysMeta
+  }
   cli: {
     meta: CLIMeta | Record<string, any> // Backward compatible with old published CLI format
     programs: Record<string, CLIProgramState>
@@ -148,7 +165,7 @@ export interface PikkuPackageState {
     } | null
   }
   package: {
-    /** Service factory functions for external packages */
+    /** Service factory functions for addon packages */
     factories: {
       createConfig?: CreateConfig<CoreConfig>
       createSingletonServices?: CreateSingletonServices<