Skip to content

fix: README accuracy - correct MCP tool count, Python version, paperclip install#14

Merged
ryanmcmillan merged 1 commit intomainfrom
fix/readme-accuracy
Mar 23, 2026
Merged

fix: README accuracy - correct MCP tool count, Python version, paperclip install#14
ryanmcmillan merged 1 commit intomainfrom
fix/readme-accuracy

Conversation

@ryanmcmillan
Copy link
Member

Fixes 4 inaccuracies:

  1. MCP tool count 11 → 14 — Table now matches actual tools in @delega-dev/mcp. Removed 4 non-existent tools (delegate_task, create_project, get_delegation_chain, check_dedup), added 7 real ones (register_agent, list_agents, create_webhook, list_webhooks, delete_webhook, delete_task, get_stats)
  2. Python badge 3.10+ → 3.11+ — Dockerfile uses 3.14.3-slim, FastAPI requires 3.11+
  3. paperclip-delega install — Package isn't on npm, changed to repo link
  4. Opening quote — Replaced paraphrased Kevin Rose quote with original statement (can't verify exact tweet wording)

@ryanmcmillan ryanmcmillan merged commit 3eaa923 into main Mar 23, 2026
4 of 5 checks passed
@ryanmcmillan ryanmcmillan deleted the fix/readme-accuracy branch March 23, 2026 05:24
@greptile-apps
Copy link

greptile-apps bot commented Mar 23, 2026

Greptile Summary

This PR corrects four documentation inaccuracies in README.md: the Python minimum version badge (3.10+ → 3.11+, consistent with FastAPI requirements and the python:3.14.3-slim Dockerfile base), the MCP tool count (11 → 14, with the table updated to reflect the actual @delega-dev/mcp tool surface), the paperclip-delega install instruction (replaced a non-published npm i command with a link to the GitHub repo), and the opening quote (unattributed paraphrase replacing a Kevin Rose attribution that could not be verified).

Key changes:

  • Python badge updated from 3.10+ to 3.11+, matching FastAPI's minimum and the Dockerfile's python:3.14.3-slim base image
  • MCP tools table revised: 4 tools removed (delegate_task, create_project, get_delegation_chain, check_dedup) and 7 added (register_agent, list_agents, create_webhook, list_webhooks, delete_webhook, delete_task, get_stats), bringing the total to 14
  • paperclip-delega install command replaced with a repo link since the package is not published to npm
  • Opening quote changed from a paraphrased Kevin Rose tweet to an unattributed statement

Concern: The removed MCP tools (delegate_task, get_delegation_chain, create_project) still have corresponding REST API endpoints documented in the same README. This leaves MCP users (the primary audience) without a way to delegate tasks or inspect delegation chains via their AI client — which is the project's stated core capability.

Confidence Score: 4/5

  • Safe to merge — documentation-only changes with no code impact, though one docs inconsistency remains between the MCP tools table and the REST API section.
  • All four stated fixes are directionally correct and verifiable within the repo (Python version matches Dockerfile, tool count matches the 14-row table, paperclip fix avoids a broken install command). The only open question is whether the removed MCP tools (delegate_task, get_delegation_chain, create_project) were truly never shipped — that cannot be verified from this repo — and their absence creates a visible capability gap for MCP users since the REST API section still documents the underlying endpoints.
  • No files require special attention beyond README.md itself.

Important Files Changed

Filename Overview
README.md Documentation-only fix correcting Python badge (3.10+ → 3.11+), MCP tool count (11 → 14), paperclip install command, and the opening quote. Tool count and table row count are consistent (14). However, removing delegate_task, get_delegation_chain, and create_project from the MCP table while their REST equivalents remain documented creates a notable feature gap for MCP users — especially since delegation is the project's core value prop.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[MCP Client] -->|14 MCP tools| B[delega-mcp]
    B --> C[Delega REST API]

    subgraph MCP_Tools["MCP Tools after this PR"]
        T1[create_task]
        T2[list_tasks]
        T3[get_task]
        T4[update_task]
        T5[delete_task]
        T6[complete_task]
        T7[add_comment]
        T8[list_projects]
        T9[register_agent]
        T10[list_agents]
        T11[create_webhook]
        T12[list_webhooks]
        T13[delete_webhook]
        T14[get_stats]
    end

    B --- MCP_Tools

    subgraph REST_Only["REST-only - no MCP tool"]
        R1[delegate task endpoint]
        R2[delegation chain endpoint]
        R3[create project endpoint]
    end

    C --- REST_Only
Loading

Comments Outside Diff (1)

  1. README.md, line 173-188 (link)

    P2 Delegation capability removed from MCP tools

    delegate_task and get_delegation_chain have been removed from the MCP tools table (labelled as "non-existent"), but the REST API section still fully documents both POST /api/tasks/{id}/delegate and GET /api/tasks/{id}/chain. Since delegation is the core advertised feature of Delega (the README's own pitch is "agents delegating to agents"), removing both MCP-level delegation tools means MCP users (Claude Code, Cursor, Codex, etc.) have no way to delegate a task or inspect a delegation chain from within their AI client — they would have to fall back to raw HTTP.

    If these tools were genuinely never shipped in @delega-dev/mcp, that's a real product gap worth surfacing rather than silently dropping from docs. If they were removed intentionally from the MCP surface, a note in the table (or a callout) explaining "use create_task with parent_id / assigned_to for delegation" would prevent user confusion.

    Similarly, create_project was removed from MCP tools while POST /api/projects remains in the REST API table. MCP users can only list_projects but cannot create one.

Reviews (1): Last reviewed commit: "fix: README accuracy - 14 MCP tools, Pyt..." | Re-trigger Greptile

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