Skip to content

fix: escape sandbox permission helper reference#620

Merged
chubes4 merged 1 commit into
mainfrom
fix/escaped-permission-helper
Jun 4, 2026
Merged

fix: escape sandbox permission helper reference#620
chubes4 merged 1 commit into
mainfrom
fix/escaped-permission-helper

Conversation

@chubes4
Copy link
Copy Markdown
Collaborator

@chubes4 chubes4 commented Jun 4, 2026

Summary

  • Escape the generated PHP namespace separator for the Data Machine PermissionHelper class reference.
  • Add smoke coverage so the sandbox generated code keeps the namespaced class reference and does not collapse it to DataMachineAbilitiesPermissionHelper.

Verification

  • npm run agent-sandbox-code-smoke
  • npm run build
  • git diff --check

Evidence

AI assistance

  • AI assistance: Yes
  • Tool(s): OpenCode (GPT-5.5)
  • Used for: Diagnosed the proof-run artifact failure, drafted the minimal escaping fix, added smoke assertions, and ran local verification. Chris remains responsible for review and merge.

@chubes4 chubes4 merged commit 1a2069c into main Jun 4, 2026
@chubes4 chubes4 deleted the fix/escaped-permission-helper 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