CBG-5290: Pass context through all methods

[bbrks]

CBG-5290

Pass context through all sg-bucket API methods

[Copilot]

Pull request overview

This PR updates the sgbucket public interfaces to thread context.Context through additional API entrypoints, aligning bucket/query/view/feed operations with context-aware cancellation/logging patterns.

Changes:

• Added ctx context.Context parameters to multiple core interfaces (BucketStore, KVStore, ViewStore, QueryableStore, MutationFeed, and related callback types).
• Updated ViewResult’s iterator methods to match the updated QueryResultIterator interface (NextBytes(ctx), Close(ctx)), and propagated ctx into view doc-loading (ds.Get(ctx, ...)).
• Updated callback types used by DCP feed handling to include ctx.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 5 comments.

File	Description
bucket.go	Adds context.Context to many bucket/datastore/KV interfaces and callback types.
queries.go	Makes query capability + iterators context-aware (incl. NextBytes(ctx) / Close(ctx)).
views.go	Threads ctx through view processing and iterator API; uses ctx for doc fetch via DataStore.
tap.go	Makes mutation feed interface + event callback context-aware.

Comments suppressed due to low confidence (1)

queries.go:71

• The doc comment says QueryResultIterator is implemented by gocb.ViewResults and gocb.QueryResults, but adding ctx parameters to NextBytes/Close means the upstream gocb types will no longer satisfy this interface. Update the comment (and/or introduce adapter wrappers) so consumers aren’t misled about compatibility.

// Common query iterator interface,
// implemented by sgbucket.ViewResult, gocb.ViewResults, and gocb.QueryResults
type QueryResultIterator interface {

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.