extract call-time capability binding helpers

Author: mgomes

docs/architecture.md

@@ -23,6 +23,7 @@ Key files:
 - `vibes/execution_assign.go` (assignment targets and member assignment flow)
 - `vibes/execution_script.go` (script call surface and call-time orchestration)
 - `vibes/execution_script_helpers.go` (compiled script lookup/order/ownership and call-time cloning helpers)
+- `vibes/execution_call_capabilities.go` (call-time capability binding and contract registration)
 - `vibes/execution_function_args.go` (function argument/default/type/ivar binding helpers)
 - `vibes/execution_calls.go` (callable dispatch + function invocation)
 - `vibes/execution_call_expr.go` (call expression target/args/kwargs/block evaluation)

vibes/execution_call_capabilities.go

@@ -0,0 +1,49 @@
+package vibes
+
+import (
+	"fmt"
+	"strings"
+)
+
+func bindCapabilitiesForCall(exec *Execution, root *Env, rebinder *callFunctionRebinder, capabilities []CapabilityAdapter) error {
+	if len(capabilities) == 0 {
+		return nil
+	}
+
+	binding := CapabilityBinding{Context: exec.ctx, Engine: exec.engine}
+	for _, adapter := range capabilities {
+		if adapter == nil {
+			continue
+		}
+		scope := &capabilityContractScope{
+			contracts: map[string]CapabilityMethodContract{},
+		}
+		if provider, ok := adapter.(CapabilityContractProvider); ok {
+			for methodName, contract := range provider.CapabilityContracts() {
+				name := strings.TrimSpace(methodName)
+				if name == "" {
+					return fmt.Errorf("capability contract method name must be non-empty")
+				}
+				if _, exists := exec.capabilityContractsByName[name]; exists {
+					return fmt.Errorf("duplicate capability contract for %s", name)
+				}
+				exec.capabilityContractsByName[name] = contract
+				scope.contracts[name] = contract
+			}
+		}
+		globals, err := adapter.Bind(binding)
+		if err != nil {
+			return err
+		}
+		for name, val := range globals {
+			rebound := rebinder.rebindValue(val)
+			root.Define(name, rebound)
+			if len(scope.contracts) > 0 {
+				scope.roots = append(scope.roots, rebound)
+			}
+			bindCapabilityContracts(rebound, scope, exec.capabilityContracts, exec.capabilityContractScopes)
+		}
+	}
+
+	return nil
+}

vibes/execution_script.go

@@ -3,7 +3,6 @@ package vibes
 import (
 	"context"
 	"fmt"
-	"strings"
 )
 
 func (s *Script) Call(ctx context.Context, name string, args []Value, opts CallOptions) (Value, error) {
@@ -58,41 +57,8 @@ func (s *Script) Call(ctx context.Context, name string, args []Value, opts CallO
 		allowRequire:              opts.AllowRequire,
 	}
 
-	if len(opts.Capabilities) > 0 {
-		binding := CapabilityBinding{Context: exec.ctx, Engine: s.engine}
-		for _, adapter := range opts.Capabilities {
-			if adapter == nil {
-				continue
-			}
-			scope := &capabilityContractScope{
-				contracts: map[string]CapabilityMethodContract{},
-			}
-			if provider, ok := adapter.(CapabilityContractProvider); ok {
-				for methodName, contract := range provider.CapabilityContracts() {
-					name := strings.TrimSpace(methodName)
-					if name == "" {
-						return NewNil(), fmt.Errorf("capability contract method name must be non-empty")
-					}
-					if _, exists := exec.capabilityContractsByName[name]; exists {
-						return NewNil(), fmt.Errorf("duplicate capability contract for %s", name)
-					}
-					exec.capabilityContractsByName[name] = contract
-					scope.contracts[name] = contract
-				}
-			}
-			globals, err := adapter.Bind(binding)
-			if err != nil {
-				return NewNil(), err
-			}
-			for name, val := range globals {
-				rebound := rebinder.rebindValue(val)
-				root.Define(name, rebound)
-				if len(scope.contracts) > 0 {
-					scope.roots = append(scope.roots, rebound)
-				}
-				bindCapabilityContracts(rebound, scope, exec.capabilityContracts, exec.capabilityContractScopes)
-			}
-		}
+	if err := bindCapabilitiesForCall(exec, root, rebinder, opts.Capabilities); err != nil {
+		return NewNil(), err
 	}
 
 	if exec.strictEffects {