Add a PropertyListRef resolver (MCP get_property_list, auth_token bearer, pagination, cache_valid_until)
Maintainers usually reply within 1 day
Nobody has claimed this yet.
Assessment
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Newbie friendliness
- 48/100
- Issue type
- Feature
- Clarity
- Clearly specified
- Activity status
- Active
- Tech stack
- go, typescript
- Domain
- backend-api-design, networking, security
Research direction
Start with the existing PropertyListRef type and TypeScript resolvePropertyList, then trace adcp/v3/internal/safefetch and the MCP client’s request/authentication patterns. Implement ResolvePropertyList(ctx, ref) around those entry points, covering bearer auth, complete pagination, returned identifiers, cache_valid_until, optional caching, and SSRF protection. Done means Go sellers can resolve every supported property-list reference consistently with the TypeScript SDK, with focused tests for pagination, cache expiry, auth, and unsafe URLs.
Written by the indexing model from the issue text.
Description
Problem
PropertyListRef{AgentURL, ListID, AuthToken} is a type only. Nothing in the SDK resolves a reference, so every Go seller that receives a property list (in targeting_overlay.property_list, property_list_exclude or criteria.offer_filters.property_list) writes its own fetcher. Each one guesses the transport, auth, paging and caching.
Ask
A ResolvePropertyList(ctx, ref) helper that:
- calls the MCP tool
get_property_listatAgentURL, withAuthTokenas the bearer token; - follows
paginationto the end; - returns identifiers plus
cache_valid_until, with an optional cache honouring it; - goes through
adcp/v3/internal/safefetchfor SSRF protection.
This should match the TypeScript SDK's resolvePropertyList, so Go and TypeScript sellers read lists the same way. The spec doesn't yet say how a reference is fetched; adcontextprotocol/adcp#8282 asks it to.
- Dominant language
- Go
- Stars
- 4
- Forks
- 2
- Avg merge
- 17h
- Merged PRs (30d)
- 21
Getting set up
This project ships no dev container, Dockerfile or contributing guide, so setting up is up to you: start from its README, and see our first-contribution guide for the general steps.
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.
More from adcontextprotocol/adcp-go
-
CreativeInput carries no revision_id or localization (follow-up to #545)Possibly taken @sujanchalla0510 claimed this 6 days ago. Open
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
adcontextprotocol/adcp-go#567 ·
Maintainers usually reply within 1 day
-
Difficulty 4/5 3-5 days Newbie friendliness 35/100
adcontextprotocol/adcp-go#590 ·
Maintainers usually reply within 1 day
-
Difficulty 5/5 Over a week Newbie friendliness 30/100
adcontextprotocol/adcp-go#588 · 2 comments ·
Maintainers usually reply within 1 day
-
Replace the hand-written RFC 9421 layer in adcp/v3/signing with httpsign and httpsfvPossibly taken A pull request linked to this issue is open or already merged. Open
Difficulty 5/5 Over a week Newbie friendliness 25/100
adcontextprotocol/adcp-go#579 ·
Maintainers usually reply within 1 day
-
Difficulty 4/5 3-5 days Newbie friendliness 52/100
adcontextprotocol/adcp-go#577 ·
Maintainers usually reply within 1 day
All issues in adcontextprotocol/adcp-go
Similar issues
-
Project submission:Open
Difficulty 1/5 Under an hour Newbie friendliness 72/100
slavakurilyak/awesome-ai-agents#845 ·
Maintainers usually reply within 1 day
-
bug
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
Maintainers usually reply within 3 days
-
Difficulty 2/5 1-3 hours Newbie friendliness 85/100
-
[BUG] deploy.restart_policy without condition drops the restart policyPossibly taken @akshat-lakhera claimed this 1 day ago. Open
Difficulty 2/5 1-3 hours Newbie friendliness 82/100
docker/compose#14383 · 1 comment ·
Maintainers usually reply within 1 day
-
Difficulty 1/5 Under an hour Newbie friendliness 72/100
Azure/azure-storage-fuse#2429 ·
Maintainers usually reply within 1 day