The adt-server createAdtServerMcpOptions correctly rejects an incomplete safe_execute configuration when the deployment does not supply consumeExecutionAuthorization and reportExecutionOutcome hooks.
However, the shipped packages/adt-server/src/bin/adt-server.ts startup only passes executeWithDeadline and leaves both hooks undefined. Enabling ADT_SERVER_MCP_SAFE_EXECUTE_ENABLED=true in the binary therefore always aborts startup.
We need a supported way for deployments to register these hooks, or sensible default implementations wired from environment variables (e.g. an authorization consumer URL and an outcome reporter URL) so the binary can run with safe_execute enabled.
Related to PR #143 (review thread PRRT_kwDOM80YYc6TfskC).
The
adt-servercreateAdtServerMcpOptionscorrectly rejects an incomplete safe_execute configuration when the deployment does not supplyconsumeExecutionAuthorizationandreportExecutionOutcomehooks.However, the shipped
packages/adt-server/src/bin/adt-server.tsstartup only passesexecuteWithDeadlineand leaves both hooks undefined. EnablingADT_SERVER_MCP_SAFE_EXECUTE_ENABLED=truein the binary therefore always aborts startup.We need a supported way for deployments to register these hooks, or sensible default implementations wired from environment variables (e.g. an authorization consumer URL and an outcome reporter URL) so the binary can run with safe_execute enabled.
Related to PR #143 (review thread PRRT_kwDOM80YYc6TfskC).