Skip to content

Import browser agent bundles with scoped principal#524

Merged
chubes4 merged 1 commit into
mainfrom
fix/browser-agent-bundle-import-principal
Jun 3, 2026
Merged

Import browser agent bundles with scoped principal#524
chubes4 merged 1 commit into
mainfrom
fix/browser-agent-bundle-import-principal

Conversation

@chubes4
Copy link
Copy Markdown
Collaborator

@chubes4 chubes4 commented Jun 3, 2026

Summary

  • Adds a non-secret import_principal contract for browser Playground agent bundles.
  • Uses that principal to set Data Machine agent execution context only around datamachine/import-agent calls in the browser runner.
  • Preserves the principal through browser session/task payload normalization and updates smoke coverage for the generated runner.

Testing

  • php -l packages/wordpress-plugin/src/trait-wp-codebox-abilities-browser-runner.php && php -l packages/wordpress-plugin/src/trait-wp-codebox-abilities-inheritance.php && php -l packages/wordpress-plugin/src/class-wp-codebox-task-input-contract.php && php -l packages/wordpress-plugin/src/class-wp-codebox-agent-sandbox-runner.php && php -l tests/smoke-wordpress-plugin.php
  • npm run wordpress-plugin-smoke
  • npm run build
  • git diff --check

Notes

  • homeboy lint --path "/Users/chubes/Developer/wp-codebox@fix-browser-agent-bundle-import-principal" --extension nodejs failed in lab setup before linting the change because the offloaded workspace could not resolve node_modules/typescript/bin/tsc. The same npm run build command passed locally in this worktree.

AI assistance

  • AI assistance: Yes
  • Tool(s): OpenCode (GPT-5.5)
  • Used for: Drafting the scoped import-principal implementation, updating smoke coverage, running verification, and preparing the PR description for Chris to review.

@chubes4 chubes4 merged commit 5ab16e2 into main Jun 3, 2026
@chubes4 chubes4 deleted the fix/browser-agent-bundle-import-principal branch June 4, 2026 20:59
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