Skip to content

Clean up Epicea change history metadata#45

Merged
Gabriel-Darbord merged 1 commit into
mainfrom
codex/p13-epicea-extensions
Jun 3, 2026
Merged

Clean up Epicea change history metadata#45
Gabriel-Darbord merged 1 commit into
mainfrom
codex/p13-epicea-extensions

Conversation

@Gabriel-Darbord
Copy link
Copy Markdown
Member

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

Summary

  • remove MCP extensions for legacy Monticello Epicea save/load events
  • replace change-history selector probing with Epicea event double dispatch
  • remove the Object/OmReference fallback extensions and obsolete MCPChangeHistoryValueExtractor
  • tighten change-history entry initialization contracts and remove redundant trailing receiver returns
  • document that methods without explicit returns answer self
  • keep change-history listing, filtering, apply, and revert behavior unchanged

Epicea code-change events now answer a narrow MCP hook that fills MCPChangeHistoryEntryInfo. The entry DTO owns the output fields, while event subclasses choose the appropriate metadata path by dispatch.

Verification

  • P13 direct MCP: MCPToolChangeHistoryTest and MCPToolChangeHistoryToolsTest passed (22 tests)
  • P13 direct MCP: MCPToolContractsTest passed (101 tests)
  • P13 direct MCP: MCPToolStructuredOutputTest passed (29 tests)
  • P13 direct MCP: no remaining mcpChangeHistoryValueForSelector:, mcpChangeHistoryReferenceString, or MCPChangeHistoryValueExtractor references in MCP packages
  • P13 disk scan: no ifNil: [ ^ self ] remains in change-history classes
  • P14 direct MCP: baseline MCP loaded from the local Tonel checkout without warnings
  • P14 direct MCP: MCPToolChangeHistoryTest and MCPToolChangeHistoryToolsTest passed (22 tests)
  • P13 direct MCP: repository is clean after adopting HEAD
  • git diff --check passed

@Gabriel-Darbord Gabriel-Darbord force-pushed the codex/p13-epicea-extensions branch from 272484e to 79e2e6a Compare June 3, 2026 08:41
@Gabriel-Darbord Gabriel-Darbord changed the title Load Monticello Epicea extensions only where supported Drop legacy Monticello Epicea history metadata Jun 3, 2026
@Gabriel-Darbord Gabriel-Darbord force-pushed the codex/p13-epicea-extensions branch from 79e2e6a to 9ed15a6 Compare June 3, 2026 09:23
@Gabriel-Darbord Gabriel-Darbord changed the title Drop legacy Monticello Epicea history metadata Clean up Epicea change history metadata Jun 3, 2026
@Gabriel-Darbord Gabriel-Darbord force-pushed the codex/p13-epicea-extensions branch from 9ed15a6 to 2ed3e13 Compare June 3, 2026 09:24
Drop legacy Monticello change-history metadata, replace selector-probing extraction with Epicea event double dispatch, and document implicit self returns for initializers and mutators.

Co-authored-by: Codex <codex@openai.com>
@Gabriel-Darbord Gabriel-Darbord force-pushed the codex/p13-epicea-extensions branch from 2ed3e13 to 0f4384b Compare June 3, 2026 09:39
@Gabriel-Darbord Gabriel-Darbord merged commit 017a377 into main Jun 3, 2026
3 checks passed
@Gabriel-Darbord Gabriel-Darbord deleted the codex/p13-epicea-extensions branch June 3, 2026 09:43
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