Skip to content

Avoid job-backed method rewrite application#56

Merged
Gabriel-Darbord merged 1 commit into
mainfrom
codex/avoid-rewrite-job-wrapper
Jun 15, 2026
Merged

Avoid job-backed method rewrite application#56
Gabriel-Darbord merged 1 commit into
mainfrom
codex/avoid-rewrite-job-wrapper

Conversation

@Gabriel-Darbord

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

Copy link
Copy Markdown
Member

Summary

  • Apply method rewrite changes directly through the refactoring change manager instead of the job-backed NewTools applier.
  • Use PharoCompatibility refactoringChangeManagerClass for the Pharo 13/14 change-manager rename instead of probing globals in MCP.

Dependency

Verification

  • P13: MCPToolRewriteMethodsTest
  • P14 scratch: PharoCompatibility refactoringChangeManagerClass resolves to ReChangeManager
  • P14 scratch: MCPToolRewriteMethodsTest
  • Changed Tonel file parsed TonelParser before the hook refactor
  • git diff --check

@Gabriel-Darbord Gabriel-Darbord force-pushed the codex/avoid-rewrite-job-wrapper branch 2 times, most recently from caefc20 to 1c138a0 Compare June 15, 2026 14:59
Co-authored-by: Codex <codex@openai.com>
@Gabriel-Darbord Gabriel-Darbord force-pushed the codex/avoid-rewrite-job-wrapper branch from 1c138a0 to 22751f2 Compare June 15, 2026 15:21
@Gabriel-Darbord Gabriel-Darbord merged commit cc532f7 into main Jun 15, 2026
3 of 6 checks passed
@Gabriel-Darbord Gabriel-Darbord deleted the codex/avoid-rewrite-job-wrapper branch June 15, 2026 15:30
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