feat: check if CID exists in local store

[2-towns]

A small method that checks if a CID exists in the local store. Useful for integration with status-go and can be exposed in the Go wrapper.

[emizzle]

Awesome, thank you for doing that Arnaud!

[marcinczenko]

Thanks!

[marcinczenko]

Suggested change

	test "Should returns true when a cid is already in the local store":
	test "Should return true when a cid is already in the local store":

[marcinczenko]

To use this method in my current CodexClient - the hasLocalBlock needs to be exposed via API - Maybe you can already use in the lib, it would be good to have it available via API as well....

I am using this in codex/rest/api.nim:

  router.api(MethodGet, "/api/codex/v1/data/{cid}/has") do(
    cid: Cid, resp: HttpResponseRef
  ) -> RestApiResponse:
    ## Only test if the give CID is available in the local store
    ##
    var headers = buildCorsHeaders("GET", allowedOrigin)

    if cid.isErr:
      return RestApiResponse.error(Http400, $cid.error(), headers = headers)


    let cid = cid.get()
    
    let hasCid = await node.hasLocalBlock(cid)

    let json = %*{$cid: hasCid}

    return RestApiResponse.response($json, contentType = "application/json")

[emizzle]

I'd definitely say let's add this in a commit @marcinczenko. Ok for you @2-towns?

[2-towns]

I'd definitely say let's add this in a commit @marcinczenko. Ok for you @2-towns?

I am already on it

[2-towns]

Okay so sorry I was already working on it before your message @emizzle.

I pushed with the tests and updated the openapi description.

Don't hesitate next time @marcinczenko to commit / push.

[emizzle]

My preference would be to name the endpoint /api/codex/v1/data/{cid}/exists instead of has, but definitely not a blocker for me.

[marcinczenko]

exists sounds indeed more pro, but is harder to type ;)

[marcinczenko]

I have updated CodexClient to use exists API - if everything sounds ok, it should be ok to merge - or maybe we better wait a little bit till I adjust archive downloader?

[marcinczenko]

I think we have all - but as I said, if it is not blocking other things, I would suggest to wait with merging - I am now changing archive downloader to use first only trigger the downloads, then to poll using "exists" API to finally locally fetch.