Skip to content

Harden run read envelopes#367

Merged
chubes4 merged 1 commit into
mainfrom
cook/run-envelope-secret-boundary-20260624
Jun 24, 2026
Merged

Harden run read envelopes#367
chubes4 merged 1 commit into
mainfrom
cook/run-envelope-secret-boundary-20260624

Conversation

@chubes4

@chubes4 chubes4 commented Jun 24, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Add observer-safe redaction for task, chat, and runtime-package run read/event envelopes without changing stored operator state.
  • Tighten runtime-package run read defaults to operators while retaining filter-based migration paths for hosts.
  • Redact sensitive-looking parameter_defaults and runtime metadata keys in model/request-facing tool declarations.
  • Expand product-boundary smoke coverage for downstream vocabulary and document the redaction/read policy.

Tests

  • composer test
  • composer phpstan
  • Targeted smokes during development: php tests/task-execution-smoke.php, php tests/chat-run-control-smoke.php, php tests/runtime-package-run-contract-smoke.php, php tests/tool-runtime-smoke.php, php tests/no-product-imports-smoke.php

Risks

  • Sensitive parameter_defaults now normalize to [redacted]; hosts that used declaration defaults as credential storage need to move those secrets into executor-owned resolution or explicit non-model-facing bindings.
  • Runtime-package run status reads now default to manage_options; hosts needing non-admin observers can opt in via agents_runtime_package_run_read_permission, and full envelopes via agents_runtime_package_run_unredacted_read_permission.

AI assistance

  • AI assistance: Yes
  • Tool(s): openai/gpt-5.5/OpenCode
  • Used for: Drafting and implementing the safety hardening, smoke coverage, docs, and PR preparation.

@chubes4 chubes4 merged commit 1e838a4 into main Jun 24, 2026
2 checks passed
@chubes4 chubes4 deleted the cook/run-envelope-secret-boundary-20260624 branch June 24, 2026 03:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant