Skip to content

Fix maxSteps in Markdown agent definitions#535

Merged
ericdallo merged 4 commits into
editor-code-assistant:masterfrom
itkonen:fix/markdown-agent-max-steps
Jul 20, 2026
Merged

Fix maxSteps in Markdown agent definitions#535
ericdallo merged 4 commits into
editor-code-assistant:masterfrom
itkonen:fix/markdown-agent-max-steps

Conversation

@itkonen

@itkonen itkonen commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

Summary

  • recognize the documented maxSteps field in Markdown agent frontmatter
  • preserve the existing steps field as a backward-compatible alias
  • cover the documented spelling in agent conversion tests

Test

  • clojure -M:test --focus eca.features.agents-test

  • I added a entry in changelog under unreleased section.

  • This is not an AI slop.

🤖 Generated with ECA (openai/gpt-5.6-sol - high)

itkonen and others added 2 commits July 15, 2026 21:16
Markdown agents document maxSteps, but conversion only recognized the legacy steps key. Accept the documented field while preserving compatibility.

🤖 Generated with [ECA](https://eca.dev) (openai/gpt-5.6-sol - high)

Co-Authored-By: eca-agent <git@eca.dev>

@ericdallo ericdallo left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thank you!

@ericdallo

Copy link
Copy Markdown
Member

@itkonen can you solve the conflicts please?

itkonen and others added 2 commits July 16, 2026 19:32
Preserve maxSteps precedence while retaining newer Markdown agent fields from master.

🤖 Generated with [ECA](https://eca.dev)

Co-Authored-By: eca-agent <git@eca.dev>
@ericdallo
ericdallo merged commit 92bf915 into editor-code-assistant:master Jul 20, 2026
7 checks passed
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.

2 participants