Skip to content

fix: remove stale Ax optimizer import#82

Merged
haasonsaas merged 1 commit intomainfrom
codex/kestrel-ax-import-feedback-20260430
Apr 30, 2026
Merged

fix: remove stale Ax optimizer import#82
haasonsaas merged 1 commit intomainfrom
codex/kestrel-ax-import-feedback-20260430

Conversation

@haasonsaas
Copy link
Copy Markdown
Collaborator

Summary

  • remove the unused AxMiPRO named import from evals/optimize-prompt.mjs
  • keep the optimizer script compatible with @ax-llm/ax v20 module exports

Review feedback addressed

Verification

  • git diff --check
  • symlinked the existing Kestrel node_modules into the fresh worktree, then ran node evals/optimize-prompt.mjs and confirmed it reaches the expected OPENROUTER_KEY guard instead of failing ESM import resolution

@cursor
Copy link
Copy Markdown

cursor Bot commented Apr 30, 2026

PR Summary

Low Risk
Low risk: removes an unused/stale named import to match @ax-llm/ax v20 exports, avoiding ESM import resolution failures without changing runtime behavior.

Overview
Updates the prompt optimizer script (evals/optimize-prompt.mjs) to stop importing AxMiPRO, leaving only AxAI so the file remains compatible with @ax-llm/ax v20 module exports.

Reviewed by Cursor Bugbot for commit 074761a. Bugbot is set up for automated code reviews on this repo. Configure here.

@haasonsaas haasonsaas merged commit bdd66e8 into main Apr 30, 2026
5 checks passed
@haasonsaas haasonsaas deleted the codex/kestrel-ax-import-feedback-20260430 branch April 30, 2026 19:06
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