fix: collections API should parse skills_json into structured response

Open Beginner friendly

#18 opened on Apr 28, 2026 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty: 2/5
Estimated time: 1-3 hours
Newbie friendliness: 76/100

Issue type: Bug
Clarity: Clearly specified
Activity status: Quiet
Tech stack: go
Domain: api, backend

Research direction

Start in internal/store/store.go and internal/handler/collections.go, tracing how skills_json is loaded and serialized for GET /api/v1/collections/{name}. The work is done when the endpoint returns a skills array of structured objects rather than an escaped JSON string, matching the response example.

Written by the indexing model from the issue text.

Description

From code review of PR #15.

GET /api/v1/collections/{name} returns skills_json as a JSON-encoded string inside JSON, which is awkward for API consumers:

{"skills_json": "[{\"name\":\"s1\",\"image\":\"quay.io/org/s1:1.0.0\"}]"}

Should return parsed structure:

{"skills": [{"name": "s1", "image": "quay.io/org/s1:1.0.0"}]}

Options:

Add a Skills []SkillRef field to Collection with custom MarshalJSON
Or use a response DTO that parses skills_json before serialization

Files: internal/store/store.go, internal/handler/collections.go

Dominant language: HTML
Stars: 9
Forks: 3
Avg merge: 18m
Merged PRs (30d): 1

Contributor guide

No contributing guide indexed for this repository

First steps

Read the whole issue, then the project's contributing guide.
Comment on the issue to say you are picking it up — it saves two people doing the same work.
Fork the repository and make your change on a branch.
Open a pull request that references the issue number.

Similar issues

[Bug]: Matrix progress drafts fail with "Matrix runtime not initialized" during tool activity Open

bug clawsweeper:linked-pr-open clawsweeper:needs-live-repro clawsweeper:no-new-fix-pr impact:message-loss issue-rating: 🐚 platinum hermit P2 regression

Difficulty 2/5 1-3 hours Newbie friendliness 78/100

openclaw/openclaw#155298 · 2 comments · 1 reaction · Opened Sep 22, 2026
[Bug] NULL pointer dereference in llm_running_callback (NGZA-AX-LLM #3) Open

Difficulty 1/5 Under an hour Newbie friendliness 90/100

AXERA-TECH/ax-llm#77 · Opened Sep 22, 2026
resume() sends a second HTTP response (400) after a successful resume Open

Difficulty 1/5 Under an hour Newbie friendliness 90/100

games-on-whales/wolf#509 · Opened Sep 21, 2026
MCP client has no httpx timeout, so the event stream fails after 5 seconds Open

Difficulty 2/5 1-3 hours Newbie friendliness 78/100

Azure/azure-functions-agents-runtime#228 · Opened Sep 21, 2026
feat(mcp): friendlier error when `run_cloud_sync` is called on a disabled connection Open

enhancement

Difficulty 2/5 1-3 hours Newbie friendliness 72/100

airbytehq/PyAirbyte#1205 · Opened Sep 21, 2026

More Backend & API Design issues