[PR #123] [MERGED] fix: add file_path parameter to save_deliverable for large reports #129

New issue

Closed

opened 2026-02-27 08:09:23 +03:00 by kerem · 0 comments

kerem commented

2026-02-27 08:09:23 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/KeygraphHQ/shannon/pull/123
Author: @ajmallesh
Created: 2/11/2026
Status: ✅ Merged
Merged: 2/11/2026
Merged by: @ajmallesh

Base: main ← Head: fix/large-deliverable-handling

📝 Commits (5)

f7eb3a5 fix: add file_path parameter to save_deliverable for large reports
4ea255b docs: simplify and condense CLAUDE.md
efc492a feat: add issue number detection to pr command
57d82b4 chore: remove CLAUDE_CODE_MAX_OUTPUT_TOKENS env var handling
a3ba152 fix: restore max_output_tokens error handling

📊 Changes

17 files changed (+298 additions, -333 deletions)

View changed files

📝 .claude/commands/pr.md (+18 -0)
📝 CLAUDE.md (+92 -280)
📝 mcp-server/src/tools/save-deliverable.ts (+73 -19)
📝 prompts/exploit-auth.txt (+6 -2)
📝 prompts/exploit-authz.txt (+6 -2)
📝 prompts/exploit-injection.txt (+6 -2)
📝 prompts/exploit-ssrf.txt (+6 -2)
📝 prompts/exploit-xss.txt (+7 -4)
📝 prompts/pre-recon-code.txt (+8 -4)
📝 prompts/recon.txt (+12 -2)
📝 prompts/vuln-auth.txt (+7 -3)
📝 prompts/vuln-authz.txt (+7 -3)
📝 prompts/vuln-injection.txt (+7 -3)
📝 prompts/vuln-ssrf.txt (+7 -3)
📝 prompts/vuln-xss.txt (+7 -3)
📝 src/ai/claude-executor.ts (+11 -0)
📝 src/ai/message-handlers.ts (+18 -1)

📄 Description

Summary

Add file_path parameter to save_deliverable MCP tool, allowing agents to reference files instead of passing large content inline (avoids output token limits)
Simplify and condense CLAUDE.md documentation
Add issue number detection to pr command for automatic linking

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/KeygraphHQ/shannon/pull/123 **Author:** [@ajmallesh](https://github.com/ajmallesh) **Created:** 2/11/2026 **Status:** ✅ Merged **Merged:** 2/11/2026 **Merged by:** [@ajmallesh](https://github.com/ajmallesh) **Base:** `main` ← **Head:** `fix/large-deliverable-handling` --- ### 📝 Commits (5) - [`f7eb3a5`](https://github.com/KeygraphHQ/shannon/commit/f7eb3a57cec88f0c96a7499dced29ad6c0ce7823) fix: add file_path parameter to save_deliverable for large reports - [`4ea255b`](https://github.com/KeygraphHQ/shannon/commit/4ea255b8c542482647c7978ac4e52f228cda290f) docs: simplify and condense CLAUDE.md - [`efc492a`](https://github.com/KeygraphHQ/shannon/commit/efc492a3f9795ca2fb9a13390fff605b0d0f4129) feat: add issue number detection to pr command - [`57d82b4`](https://github.com/KeygraphHQ/shannon/commit/57d82b41aebbc0ef9d2fe5a6d999ab2d7102bb08) chore: remove CLAUDE_CODE_MAX_OUTPUT_TOKENS env var handling - [`a3ba152`](https://github.com/KeygraphHQ/shannon/commit/a3ba1529e0989e15215f30ce8625eb1d9b1471fa) fix: restore max_output_tokens error handling ### 📊 Changes **17 files changed** (+298 additions, -333 deletions) <details> <summary>View changed files</summary> 📝 `.claude/commands/pr.md` (+18 -0) 📝 `CLAUDE.md` (+92 -280) 📝 `mcp-server/src/tools/save-deliverable.ts` (+73 -19) 📝 `prompts/exploit-auth.txt` (+6 -2) 📝 `prompts/exploit-authz.txt` (+6 -2) 📝 `prompts/exploit-injection.txt` (+6 -2) 📝 `prompts/exploit-ssrf.txt` (+6 -2) 📝 `prompts/exploit-xss.txt` (+7 -4) 📝 `prompts/pre-recon-code.txt` (+8 -4) 📝 `prompts/recon.txt` (+12 -2) 📝 `prompts/vuln-auth.txt` (+7 -3) 📝 `prompts/vuln-authz.txt` (+7 -3) 📝 `prompts/vuln-injection.txt` (+7 -3) 📝 `prompts/vuln-ssrf.txt` (+7 -3) 📝 `prompts/vuln-xss.txt` (+7 -3) 📝 `src/ai/claude-executor.ts` (+11 -0) 📝 `src/ai/message-handlers.ts` (+18 -1) </details> ### 📄 Description ## Summary - Add `file_path` parameter to `save_deliverable` MCP tool, allowing agents to reference files instead of passing large content inline (avoids output token limits) - Simplify and condense CLAUDE.md documentation - Add issue number detection to pr command for automatic linking --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>