Skip to content

fix(agents): pass --version to downstream binary and suppress banners in non-TTY mode - #521

Merged
mykolanehrych merged 4 commits into
codemie-ai:mainfrom
AlexStasko:non-tty-session
Oct 6, 2026
Merged

mykolanehrych merged 4 commits into
codemie-ai:mainfrom
AlexStasko:non-tty-session

Conversation

@AlexStasko

@AlexStasko AlexStasko commented Aug 31, 2026 •

Copy link
Copy Markdown
Contributor

Summary

In order to use codemie-* agents with wrappers like https://github.com/pingdotgg/t3code

Ticket: EPMCDME-15648

Changes

  • --version option is passed to the downstream binary, so the upstream tool can understand the real version of the harness.
  • suppress banners in non-TTY mode. Mostly for the codemie-claude binary triggered via the Claude SDK

Impact

Checklist

  • Self-reviewed
  • Manual testing performed
  • Documentation updated (if needed)
  • No breaking changes (or clearly documented)

@AlexStasko AlexStasko changed the title Non tty session fix(agents): pass --version to downstream binary and suppress banners in non-TTY mode Aug 31, 2026
@AlexStasko

Copy link
Copy Markdown
Contributor Author

Hi @8nevil8, is there any chance this MR gets reviewed?

@8nevil8

8nevil8 commented Sep 8, 2026

Copy link
Copy Markdown
Collaborator

@AlexStasko, pls confirm that codemie --version still respect this flag

@AlexStasko

Copy link
Copy Markdown
Contributor Author

@8nevil8, yes. Please check the screenshot
image

@8nevil8

8nevil8 commented Sep 10, 2026

Copy link
Copy Markdown
Collaborator

@AlexStasko pls fix tests. and we'll proceed

@8nevil8

8nevil8 commented Sep 16, 2026

Copy link
Copy Markdown
Collaborator

@AlexStasko pls fix CI

@AlexStasko

Copy link
Copy Markdown
Contributor Author

@8nevil8, sorry, I was off for a few weeks. I've pushed a change, hopefully it will fix the tests.

Comment thread src/agents/core/AgentCLI.ts Outdated
@mykolanehrych

Copy link
Copy Markdown
Collaborator

You have a conflict in src/agents/core/AgentCLI.ts, please resolve it

@AlexStasko

Copy link
Copy Markdown
Contributor Author

@mykolanehrych, the branch is rebased, and conflicts are resolved.

@AlexStasko

Copy link
Copy Markdown
Contributor Author

@mykolanehrych, edited description to add ticket number: EPMCDME-15648

@mykolanehrych
mykolanehrych merged commit ccfd027 into codemie-ai:main Oct 6, 2026
10 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.

3 participants