@rcrsr/rill 0.15.0 → 0.17.0

package/dist/ext/fetch/index.js

@@ -6,7 +6,7 @@
  * All URLs are constructed from config - scripts cannot specify arbitrary URLs.
  */
 import { RuntimeError } from '../../error-classes.js';
-import { isDict } from '../../runtime/core/callable.js';
+import { toCallable } from '../../runtime/core/callable.js';
 import { rillTypeToTypeValue, } from '../../runtime/core/values.js';
 import { buildRequest, executeRequest, createSemaphore, } from './request.js';
 // ============================================================
@@ -52,12 +52,11 @@ function mapEndpointConfig(config) {
 // ARGUMENT PROCESSING
 // ============================================================
 /**
- * Process arguments from positional or dict form.
- * Supports two calling conventions:
- * 1. Positional: endpoint(arg1, arg2, arg3)
- * 2. Dict: endpoint({name: value, ...})
+ * Process named arguments dict, applying defaults and required checks.
+ * Called from endpointFn after runtime has already marshaled positional args
+ * to a named Record<string, RillValue> map.
  *
- * @param args - Raw arguments from call site
+ * @param args - Named argument map from runtime
  * @param params - Parameter definitions
  * @param functionName - Function name for error messages
  * @returns Dict of argument name to value
@@ -65,37 +64,8 @@ function mapEndpointConfig(config) {
  */
 function processArguments(args, params, functionName) {
     const result = {};
-    // Case 1: Single dict argument - extract named parameters
-    const firstArg = args[0];
-    if (args.length === 1 && firstArg !== undefined && isDict(firstArg)) {
-        const argsDict = firstArg;
-        for (const param of params) {
-            const value = argsDict[param.name];
-            if (value === undefined) {
-                // Check if parameter has default value
-                if (param.defaultValue !== undefined) {
-                    result[param.name] = param.defaultValue;
-                }
-                else if (param.required !== false) {
-                    // EC-8: Missing required parameter
-                    throw new RuntimeError('RILL-R001', `parameter "${param.name}" is required`, undefined, {
-                        functionName,
-                        paramName: param.name,
-                    });
-                }
-            }
-            else {
-                result[param.name] = value;
-            }
-        }
-        return result;
-    }
-    // Case 2: Positional arguments - map by position
-    for (let i = 0; i < params.length; i++) {
-        const param = params[i];
-        if (param === undefined)
-            continue;
-        const value = args[i];
+    for (const param of params) {
+        const value = args[param.name];
         if (value === undefined) {
             // Check if parameter has default value
             if (param.defaultValue !== undefined) {
@@ -126,7 +96,7 @@ function processArguments(args, params, functionName) {
  * All URLs constructed from config - scripts cannot create arbitrary URLs.
  *
  * @param config - Fetch configuration with endpoints
- * @returns ExtensionResult with endpoint functions and introspection
+ * @returns ExtensionFactoryResult with endpoint functions and introspection
  * @throws Error on invalid configuration
  *
  * @example
@@ -176,7 +146,7 @@ export function createFetchExtension(config) {
         const params = endpointConfig.params ?? [];
         // Generate function for this endpoint
         const endpointFn = async (args) => {
-            // Process arguments (positional or dict)
+            // Process named arguments (apply defaults, check required)
             const processedArgs = processArguments(args, params, endpointName);
             // Build request
             const { url, options, responseShape } = buildRequest(requestConfig, endpointName, processedArgs);
@@ -194,10 +164,10 @@ export function createFetchExtension(config) {
         };
         // Build parameter definitions for RillFunction
         const rillParams = params.map((param) => {
-            // Map EndpointParam type string to RillType object
+            // Map EndpointParam type string to TypeStructure object
             const rillType = param.type !== 'dict'
-                ? { type: param.type }
-                : { type: 'dict' };
+                ? { kind: param.type }
+                : { kind: 'dict' };
             return {
                 name: param.name,
                 type: rillType,
@@ -206,8 +176,8 @@ export function createFetchExtension(config) {
             };
         });
         const returnTypeValue = (endpointConfig.responseShape ?? defaultResponseShape) === 'full'
-            ? rillTypeToTypeValue({ type: 'dict' })
-            : rillTypeToTypeValue({ type: 'any' });
+            ? rillTypeToTypeValue({ kind: 'dict' })
+            : rillTypeToTypeValue({ kind: 'any' });
         const hostFunctionDef = {
             params: rillParams,
             fn: endpointFn,
@@ -241,7 +211,7 @@ export function createFetchExtension(config) {
         params: [],
         fn: endpoints,
         annotations: { description: 'List configured endpoints' },
-        returnType: rillTypeToTypeValue({ type: 'list' }),
+        returnType: rillTypeToTypeValue({ kind: 'list' }),
     };
     // ============================================================
     // DISPOSAL
@@ -259,9 +229,14 @@ export function createFetchExtension(config) {
     // ============================================================
     // EXTENSION RESULT
     // ============================================================
-    const result = functions;
-    result.dispose = dispose;
-    return result;
+    const callableDict = {};
+    for (const [name, def] of Object.entries(functions)) {
+        callableDict[name] = toCallable(def);
+    }
+    return {
+        value: callableDict,
+        dispose,
+    };
 }
 // ============================================================
 // MANIFEST
@@ -270,4 +245,3 @@ export const extensionManifest = {
     factory: createFetchExtension,
     configSchema,
 };
-//# sourceMappingURL=index.js.map

package/dist/ext/fetch/request.d.ts

@@ -87,4 +87,3 @@ export declare function buildRequest(config: FetchExtensionConfig, endpointName:
     responseShape: ResponseShape;
 };
 export {};
-//# sourceMappingURL=request.d.ts.map

package/dist/ext/fetch/request.js

@@ -412,4 +412,3 @@ export function buildRequest(config, endpointName, args) {
         responseShape: endpoint.responseShape ?? 'body',
     };
 }
-//# sourceMappingURL=request.js.map

package/dist/ext/fs/index.d.ts

@@ -4,7 +4,7 @@
  * Provides sandboxed filesystem operations via mount-based access control.
  * All 12 functions implement path validation, permission checks, and glob filtering.
  */
-import type { ExtensionResult, ExtensionConfigSchema, ExtensionManifest } from '../../runtime/ext/extensions.js';
+import type { ExtensionFactoryResult, ExtensionConfigSchema, ExtensionManifest } from '../../runtime/ext/extensions.js';
 import { type MountConfig } from './sandbox.js';
 /** Filesystem extension configuration */
 export interface FsConfig {
@@ -24,7 +24,7 @@ export declare const configSchema: ExtensionConfigSchema;
  * Returns 12 functions: read, write, append, list, find, exists, remove, stat, mkdir, copy, move, mounts.
  *
  * @param config - Mount configuration and defaults
- * @returns ExtensionResult with 12 filesystem functions
+ * @returns ExtensionFactoryResult with 12 filesystem functions
  * @throws RuntimeError if mount initialization fails
  *
  * @example
@@ -36,6 +36,5 @@ export declare const configSchema: ExtensionConfigSchema;
  * });
  * ```
  */
-export declare function createFsExtension(config: FsConfig): ExtensionResult;
+export declare function createFsExtension(config: FsConfig): ExtensionFactoryResult;
 export declare const extensionManifest: ExtensionManifest;
-//# sourceMappingURL=index.d.ts.map