Skip to content

Add debugger implementor surface hooks#4

Merged
Gabriel-Darbord merged 2 commits into
mainfrom
codex/debugger-compatibility-hooks
Jun 26, 2026
Merged

Add debugger implementor surface hooks#4
Gabriel-Darbord merged 2 commits into
mainfrom
codex/debugger-compatibility-hooks

Conversation

@Gabriel-Darbord

@Gabriel-Darbord Gabriel-Darbord commented Jun 26, 2026

Copy link
Copy Markdown
Member

Summary

  • add debuggerMethodImplementorForSession: to the Pharo13Surface runtime packages
  • keep runtime behavior surface-owned: P13/P14 send directly to StDebuggerMethodImplementor, P12 returns nil for callers that should fall back
  • cover each runtime surface with focused tests

Verification

  • P12: PharoCompatibilityTest + PharoCompatibilityP13SurfacePharo12Test, 7 passed
  • P13: PharoCompatibilityTest + PharoCompatibilityP13SurfacePharo13Test, 4 passed
  • P14: PharoCompatibilityTest + PharoCompatibilityP13SurfacePharo14Test, 10 passed

Co-authored-by: Codex <codex@openai.com>
@Gabriel-Darbord Gabriel-Darbord force-pushed the codex/debugger-compatibility-hooks branch from f475508 to 3466c20 Compare June 26, 2026 12:48
@Gabriel-Darbord Gabriel-Darbord changed the title Add debugger implementor compatibility hook Add debugger implementor surface hooks Jun 26, 2026
@Gabriel-Darbord Gabriel-Darbord marked this pull request as ready for review June 26, 2026 12:49
Co-authored-by: Codex <codex@openai.com>
@Gabriel-Darbord Gabriel-Darbord merged commit 6ed6e78 into main Jun 26, 2026
3 checks passed
@Gabriel-Darbord Gabriel-Darbord deleted the codex/debugger-compatibility-hooks branch June 26, 2026 13:17
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