fix checkpoint store lint allowlist

2299741· pfleidi·3w ago·2 files·+2 added/-1 removed

Allow checkpoint.TemporaryStore as a capability interface for ireturn.

This matches the existing committed-store exception and keeps CI's non-fixing golangci-lint run aligned with the local fixer.

Sessions

b8a4d6cdf265View transcript

Changes

2

• M.golangci.yaml+1

• cmd/entire/cli/strategy

• Mmanual_commit.go+1/-1

121 unmodified lines

122
123
124
125
126
127
128

121 unmodified lines

- github.com/entireio/cli/cmd/entire/cli/review/types.AgentReviewer
        - github.com/entireio/cli/cmd/entire/cli/review.SynthesisProvider
        - github.com/entireio/cli/cmd/entire/cli/checkpoint.CommittedReader
        - github.com/entireio/cli/cmd/entire/cli/checkpoint.TemporaryStore
        - github.com/entireio/cli/cmd/entire/cli/strategy.Strategy
        - github.com/entireio/cli/internal/entireclient/tokenstore.store
        - github.com/go-git/go-git/v6/x/plugin.Signer

M.golangci.yaml+1

61 unmodified lines

62
63
64
65
65
66
67
68

61 unmodified lines

// getTemporaryStore returns the git-backed shadow-branch store with the
// strategy's blob fetcher wired in.
func (s *ManualCommitStrategy) getTemporaryStore(ctx context.Context, repo *git.Repository) (checkpoint.TemporaryStore, error) { //nolint:ireturn // temporary store capability is the abstraction boundary
func (s *ManualCommitStrategy) getTemporaryStore(ctx context.Context, repo *git.Repository) (checkpoint.TemporaryStore, error) {
    stores, err := s.getCheckpointStores(ctx, repo)
    if err != nil {
        return nil, err
    }

Mcmd/entire/cli/strategy/manual_commit.go+1/-1