Skip to content

Conversation

@alexandear
Copy link
Contributor

@alexandear alexandear commented Jan 12, 2026

Summary

Remove committed artifacts.

Why

These artifacts mistakenly committed to the repo in #1164 and #1589.

$ file e2e.test
e2e.test: ELF 64-bit LSB executable, x86-64, version 1 (SYSV), dynamically linked, interpreter /lib64/ld-linux-x86-64.so.2, BuildID[sha1]=eba9512f03831a33590136029a1260666a1a3423, with debug_info, not stripped
$ file cmd/mcpcurl/mcpcurl
cmd/mcpcurl/mcpcurl: ELF 64-bit LSB executable, x86-64, version 1 (SYSV), dynamically linked, interpreter /lib64/ld-linux-x86-64.so.2, BuildID[sha1]=7b05c50515653ffa92038aa6db4c399d1c27b5e3, with debug_info, not stripped

What changed

  • Remove e2e.test
  • Remove cmd/mcpcurl/mcpcurl
  • Add e2e.test and mcpcurl to .gitignore

MCP impact

  • No tool or API changes
  • Tool schema or behavior changed
  • New tool added

Prompts tested (tool changes only)

Security / limits

  • No security or limits impact
  • Auth / permissions considered
  • Data exposure, filtering, or token/size limits considered

Tool renaming

  • I am renaming tools as part of this PR (e.g. a part of a consolidation effort)
    • I have added the new tool aliases in deprecated_tool_aliases.go
  • I am not renaming tools as part of this PR

Note: if you're renaming tools, you must add the tool aliases. For more information on how to do so, please refer to the official docs.

Lint & tests

  • Linted locally with ./script/lint
  • Tested locally with ./script/test

Docs

  • Not needed
  • Updated (README / docs / examples)

Copilot AI review requested due to automatic review settings January 12, 2026 18:22
@alexandear alexandear requested a review from a team as a code owner January 12, 2026 18:22
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot wasn't able to review any files in this pull request.

Copy link
Collaborator

@SamMorrowDrums SamMorrowDrums left a comment

Choose a reason for hiding this comment

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

Thank you, very much agreed we should do this.

@SamMorrowDrums SamMorrowDrums merged commit 31b541e into github:main Jan 12, 2026
10 checks passed
@alexandear alexandear deleted the chore/remove-binary branch January 12, 2026 19:05
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