Skip to content

Kristentr pct#2154

Closed
kristentr wants to merge 2 commits intogithub:mainfrom
kristentr:kristentr-pct
Closed

Kristentr pct#2154
kristentr wants to merge 2 commits intogithub:mainfrom
kristentr:kristentr-pct

Conversation

@kristentr
Copy link

Summary

Why

Fixes #

What changed

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)

kristentr and others added 2 commits March 6, 2026 04:12
* Stricter matching for github.com and ghe.com URLs

* Gracefully handle numeric parameters passed as strings (github#2130)

* Gracefully handle numeric parameters passed as strings

* Fix SHA validation in create_or_update_file (github#2134)

* Fix SHA validation in create_or_update_file

* Doc update

* Handle non-404 errors

* Handle directory paths

* Update instructions

* fix: handle empty files in get_file_contents (github#2042)

1. Empty (0-byte) files caused an unhandled error because the GitHub API
   returns null content with base64 encoding for them; GetContent() fails
   with "malformed response: base64 encoding of null content". Return
   empty text/plain content directly, bypassing decoding entirely.

Co-authored-by: Ksenia Bobrova <almaleksia@github.com>

* Cline & Roo code installation guides (github#2146)

* Cline & Roo code installation guides

* Update docs/installation-guides/install-cline.md

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* Apply suggestion from @Copilot

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

---------

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* Correctly wrap GraphQl error (github#2149)

---------

Co-authored-by: Ksenia Bobrova <almaleksia@github.com>
Co-authored-by: Jakub Janusz <32165716+kubajanusz@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@dsyme dsyme closed this Mar 6, 2026
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