Skip to content

fix(skill.md): change PATCH to PUT for gigs and posts update routes#254

Open
forgou37 wants to merge 1 commit into
profullstack:masterfrom
forgou37:fix/skill-md-patch-to-put
Open

fix(skill.md): change PATCH to PUT for gigs and posts update routes#254
forgou37 wants to merge 1 commit into
profullstack:masterfrom
forgou37:fix/skill-md-patch-to-put

Conversation

@forgou37
Copy link
Copy Markdown

Fixes #253

Problem

public/skill.md documents PATCH /api/gigs/:id and PATCH /api/posts/:id but the server only accepts PUT, returning 405 Method Not Allowed.

The OpenAPI spec at /api/openapi.json already correctly documents both as PUT.

Fix

Updated public/skill.md to match the actual server behavior and the OpenAPI spec:

  • PATCH /api/gigs/:idPUT /api/gigs/:id
  • PATCH /api/posts/:idPUT /api/posts/:id

Verification

# Confirmed PATCH returns 405
curl -X PATCH https://ugig.net/api/posts/<id> -H "Authorization: Bearer $KEY" ...
# → 405

# Confirmed PUT returns 200
curl -X PUT https://ugig.net/api/posts/<id> -H "Authorization: Bearer $KEY" ...
# → 200

Reported via issue #253. Found during accepted bug-hunt gig.

@greptile-apps
Copy link
Copy Markdown

greptile-apps Bot commented May 24, 2026

Greptile Summary

Corrects two HTTP method references in public/skill.md from PATCH to PUT for the gigs and posts update routes, matching the server's actual behavior and the existing OpenAPI spec. No logic or server-side code is changed.

  • PATCH /api/gigs/:idPUT /api/gigs/:id and PATCH /api/posts/:idPUT /api/posts/:id are updated; both are confirmed as PUT in the CLI client (cli/dist/commands/gigs.js line 211, cli/dist/commands/posts.js line 89) and in public/openapi.json.
  • The remaining patch entries in openapi.json are for unrelated endpoints (/api/gigs/{id}/status, video calls, MCP marketplace) and are unaffected.

Confidence Score: 5/5

Documentation-only fix that brings skill.md in sync with the server and OpenAPI spec — safe to merge.

The change is a two-line documentation correction. Both the CLI source and the OpenAPI spec independently confirm that PUT is the correct method for these routes, so the fix is verified.

No files require special attention.

Important Files Changed

Filename Overview
public/skill.md Two HTTP method corrections (PATCH → PUT) for gigs and posts update routes, aligning documentation with the actual server behavior confirmed in the CLI client code.

Sequence Diagram

sequenceDiagram
    participant Client
    participant Server as ugig.net API

    Note over Client,Server: Before fix (skill.md documented PATCH)
    Client->>Server: PATCH /api/gigs/:id
    Server-->>Client: 405 Method Not Allowed
    Client->>Server: PATCH /api/posts/:id
    Server-->>Client: 405 Method Not Allowed

    Note over Client,Server: After fix (skill.md now documents PUT)
    Client->>Server: PUT /api/gigs/:id
    Server-->>Client: 200 OK
    Client->>Server: PUT /api/posts/:id
    Server-->>Client: 200 OK
Loading

Reviews (1): Last reviewed commit: "fix(skill.md): change PATCH to PUT for g..." | 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.

skill.md documents PATCH for gigs and posts but server requires PUT (405)

1 participant