Skip to content

wip: Add new unified mcp implementation #1492

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
wants to merge 2 commits into
base: dev
Choose a base branch
from
Draft

Conversation

creatorrr
Copy link
Contributor

  • feat(sdks): Add docs-mcp package
  • wip: Add new unified mcp implementation

Signed-off-by: Diwank Singh Tomer <[email protected]>
Copy link
Contributor

claude bot commented Jun 19, 2025

Claude encountered an error —— View job


I'll analyze this and get back to you.

Copy link
Contributor

CI Feedback 🧐

A test triggered by this PR failed. Here is an AI-generated analysis of the failure:

Action: claude-review

Failed stage: Run Claude Code Review [❌]

Failure summary:

The action failed due to two main issues:
• Git hash computation failed for sdks/node-sdk and
sdks/python-sdk directories (lines 210-236), with git returning "fatal: Unable to hash" errors and
exit status 128
• API validation error occurred where a tool name didn't match the required pattern
^[a-zA-Z0-9_-]{1,64}$ (line 806), specifically for tools.20.custom.name
• Multiple MCP servers
failed to connect (fetch, gitingest-mcp, search, firecrawl, typespec-mcp)
• The process completed
with exit code 1 (line 833)

Relevant error logs:
1:  ##[group]Operating System
2:  Ubuntu
...

196:  Attempt 1 of 3...
197:  OIDC token successfully obtained
198:  Exchanging OIDC token for app token...
199:  Attempt 1 of 3...
200:  App token successfully obtained
201:  Using GITHUB_TOKEN from OIDC
202:  Checking permissions for actor: creatorrr
203:  Permission level retrieved: admin
204:  Actor has write access: admin
205:  Direct prompt provided, triggering action
206:  Actor type: User
207:  Verified human actor: creatorrr
208:  ✅ Created initial comment with ID: 2988942222
209:  Successfully fetched PR #1492 data
210:  fatal: Unable to hash sdks/node-sdk
211:  Failed to compute SHA for sdks/node-sdk: warn: Command failed: git hash-object "sdks/node-sdk"
212:  fatal: Unable to hash sdks/node-sdk
213:  signal: null,
214:  status: 128,
215:  output: [ null, "", "fatal: Unable to hash sdks/node-sdk\n" ],
216:  pid: 2146,
217:  stdout: "",
218:  stderr: "fatal: Unable to hash sdks/node-sdk\n",
219:  at genericNodeError (node:child_process:941:13)
220:  at checkExecSyncError (node:child_process:449:27)
221:  at execSync (node:child_process:277:31)
222:  at <anonymous> (/home/runner/work/_actions/anthropics/claude-code-action/beta/src/github/data/fetcher.ts:117:21)
223:  at map (1:11)
224:  at fetchGitHubData (/home/runner/work/_actions/anthropics/claude-code-action/beta/src/github/data/fetcher.ts:106:40)
225:  fatal: Unable to hash sdks/python-sdk
226:  Failed to compute SHA for sdks/python-sdk: warn: Command failed: git hash-object "sdks/python-sdk"
227:  fatal: Unable to hash sdks/python-sdk
228:  signal: null,
229:  status: 128,
230:  output: [ null, "", "fatal: Unable to hash sdks/python-sdk\n" ],
231:  pid: 2148,
232:  stdout: "",
233:  stderr: "fatal: Unable to hash sdks/python-sdk\n",
234:  at genericNodeError (node:child_process:941:13)
235:  at checkExecSyncError (node:child_process:449:27)
236:  at execSync (node:child_process:277:31)
...

727:  "TodoWrite",
728:  "mcp__github_file_ops__commit_files",
729:  "mcp__github_file_ops__delete_files",
730:  "mcp__github_file_ops__update_claude_comment",
731:  "mcp__deepwiki__read_wiki_structure",
732:  "mcp__deepwiki__read_wiki_contents",
733:  "mcp__deepwiki__ask_question",
734:  "mcp__llmtxt_dev__llms.txt-file",
735:  "mcp__llmtxt_dev__git",
736:  "mcp__context7__resolve-library-id",
737:  "mcp__context7__get-library-docs"
738:  ],
739:  "mcp_servers": [
740:  {
741:  "name": "fetch",
742:  "status": "failed"
743:  },
744:  {
745:  "name": "gitingest-mcp",
746:  "status": "failed"
747:  },
...

751:  },
752:  {
753:  "name": "linear",
754:  "status": "needs-auth"
755:  },
756:  {
757:  "name": "deepwiki",
758:  "status": "connected"
759:  },
760:  {
761:  "name": "llmtxt.dev",
762:  "status": "connected"
763:  },
764:  {
765:  "name": "search",
766:  "status": "failed"
767:  },
768:  {
769:  "name": "context7",
770:  "status": "connected"
771:  },
772:  {
773:  "name": "firecrawl",
774:  "status": "failed"
775:  },
776:  {
777:  "name": "typespec-mcp",
778:  "status": "failed"
779:  }
...

791:  "stop_reason": "stop_sequence",
792:  "stop_sequence": "",
793:  "type": "message",
794:  "usage": {
795:  "input_tokens": 0,
796:  "output_tokens": 0,
797:  "cache_creation_input_tokens": 0,
798:  "cache_read_input_tokens": 0,
799:  "server_tool_use": {
800:  "web_search_requests": 0
801:  }
802:  },
803:  "content": [
804:  {
805:  "type": "text",
806:  "text": "API Error: 400 {\"type\":\"error\",\"error\":{\"type\":\"invalid_request_error\",\"message\":\"tools.20.custom.name: String should match pattern '^[a-zA-Z0-9_-]{1,64}$'\"}}"
807:  }
808:  ]
809:  },
810:  "parent_tool_use_id": null,
811:  "session_id": "4ecba8a3-d372-460e-83b7-eaddd7b8e818"
812:  }
813:  {
814:  "type": "result",
815:  "subtype": "success",
816:  "is_error": true,
817:  "duration_ms": 768,
818:  "duration_api_ms": 1516,
819:  "num_turns": 1,
820:  "result": "API Error: 400 {\"type\":\"error\",\"error\":{\"type\":\"invalid_request_error\",\"message\":\"tools.20.custom.name: String should match pattern '^[a-zA-Z0-9_-]{1,64}$'\"}}",
821:  "session_id": "4ecba8a3-d372-460e-83b7-eaddd7b8e818",
822:  "total_cost_usd": 0.0005407999999999999,
823:  "usage": {
824:  "input_tokens": 0,
825:  "cache_creation_input_tokens": 0,
826:  "cache_read_input_tokens": 0,
827:  "output_tokens": 0,
828:  "server_tool_use": {
829:  "web_search_requests": 0
830:  }
831:  }
832:  }
833:  ##[error]Process completed with exit code 1.
834:  ##[group]Run bun run ${GITHUB_ACTION_PATH}/src/entrypoints/update-comment-link.ts
...

839:  DISALLOWED_TOOLS: WebSearch,WebFetch
840:  REPOSITORY: julep-ai/julep
841:  PR_NUMBER: 1492
842:  CLAUDE_COMMENT_ID: 2988942222
843:  GITHUB_RUN_ID: 15764757623
844:  GITHUB_TOKEN: ***
845:  GITHUB_EVENT_NAME: pull_request
846:  TRIGGER_COMMENT_ID: 
847:  CLAUDE_BRANCH: 
848:  IS_PR: false
849:  BASE_BRANCH: 
850:  CLAUDE_SUCCESS: false
851:  OUTPUT_FILE: /home/runner/work/_temp/claude-execution-output.json
852:  TRIGGER_USERNAME: creatorrr
853:  PREPARE_SUCCESS: true
854:  PREPARE_ERROR: 
855:  ##[endgroup]

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