Skip to content

fix(workflows): preserve tool canonical modes in API edits - #7772

Closed
j15z wants to merge 1 commit into
codex/fix-atomic-tool-reorderingfrom
codex/fix-api-tool-canonical-remapping
Closed

fix(workflows): preserve tool canonical modes in API edits#7772
j15z wants to merge 1 commit into
codex/fix-atomic-tool-reorderingfrom
codex/fix-api-tool-canonical-remapping

Conversation

@j15z

@j15z j15z commented Sep 11, 2026

Copy link
Copy Markdown
Collaborator

Summary

Match retained tools across API arrays and use the shared positional remapper for every indexed canonical setting. Reserve unchanged configurations before matching edited tools, drop removed indexes, and reject ambiguous duplicates with conflicting saved modes. Explicit selector or variable inputs choose that field’s mode without masking conflicts in other fields.

Type of Change

  • Bug fix
  • New feature
  • Breaking change
  • Documentation
  • Other: ___________

Testing

82 editing regression tests pass on this branch. Coverage includes both edit paths, existing Jira selector modes, explicit choices, duplicate ambiguity, removed/stale indexes, custom/MCP/workflow targets, and 100-tool reversal. The selector reorder regression failed before the fix. App type check passes.

Checklist

  • Code follows project style guidelines
  • Self-reviewed my changes
  • Tests added/updated and passing
  • No new warnings introduced
  • I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

Screenshots/Videos

Not applicable.

@vercel

vercel Bot commented Sep 11, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
docs Ready Ready Preview Sep 12, 2026 11:31pm UTC

Request Review

@j15z
j15z force-pushed the codex/fix-api-tool-canonical-remapping branch from b4f42b3 to 3c840ed Compare September 12, 2026 20:13
@j15z
j15z force-pushed the codex/fix-atomic-tool-reordering branch from 4039a47 to 34420e9 Compare September 12, 2026 20:13
@j15z
j15z force-pushed the codex/fix-atomic-tool-reordering branch from 34420e9 to cb0caed Compare September 12, 2026 20:27
@j15z
j15z force-pushed the codex/fix-api-tool-canonical-remapping branch from 3c840ed to ea8a65c Compare September 12, 2026 20:27
@j15z
j15z force-pushed the codex/fix-atomic-tool-reordering branch from cb0caed to c4006c8 Compare September 12, 2026 23:04
@j15z
j15z force-pushed the codex/fix-api-tool-canonical-remapping branch from ea8a65c to 30eea7f Compare September 12, 2026 23:04
@j15z
j15z force-pushed the codex/fix-api-tool-canonical-remapping branch from 30eea7f to b2a8afd Compare September 12, 2026 23:29
@j15z
j15z force-pushed the codex/fix-atomic-tool-reordering branch from c4006c8 to 09bf6fe Compare September 12, 2026 23:29
@j15z j15z closed this Sep 13, 2026
@waleedlatif1
waleedlatif1 deleted the codex/fix-api-tool-canonical-remapping branch September 13, 2026 03:08
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