@agoric/cosmos 0.35.0-u12.0 → 0.35.0-u14.0

package/x/vstorage/client/cli/query.go

@@ -4,6 +4,7 @@ import (
 	"github.com/Agoric/agoric-sdk/golang/cosmos/x/vstorage/types"
 	"github.com/cosmos/cosmos-sdk/client"
 	"github.com/cosmos/cosmos-sdk/client/flags"
+	"github.com/gogo/protobuf/proto"
 	"github.com/spf13/cobra"
 )
 
@@ -18,6 +19,7 @@ func GetQueryCmd(storeKey string) *cobra.Command {
 	swingsetQueryCmd.AddCommand(
 		GetCmdGetData(storeKey),
 		GetCmdGetChildren(storeKey),
+		GetCmdGetPath(storeKey),
 	)
 
 	return swingsetQueryCmd
@@ -26,8 +28,8 @@ func GetQueryCmd(storeKey string) *cobra.Command {
 // GetCmdGetData queries information about storage
 func GetCmdGetData(queryRoute string) *cobra.Command {
 	cmd := &cobra.Command{
-		Use:   "data [path]",
-		Short: "get vstorage data for path",
+		Use:   "data <path>",
+		Short: "get data for vstorage path",
 		Args:  cobra.ExactArgs(1),
 		RunE: func(cmd *cobra.Command, args []string) error {
 			clientCtx, err := client.GetClientQueryContext(cmd)
@@ -58,8 +60,12 @@ func GetCmdGetChildren(queryRoute string) *cobra.Command {
 	cmd := &cobra.Command{
 		Use:     "children [path]",
 		Aliases: []string{"keys"},
-		Short:   "get vstorage subkey names for path",
-		Args:    cobra.MaximumNArgs(1),
+		Short:   "get child path segments under vstorage path",
+		Long: `get child path segments under vstorage path.
+When absent, path defaults to the empty root path.
+Path segments are dot-separated, so a child "baz" under path "foo.bar" has path
+"foo.bar.baz".`,
+		Args: cobra.MaximumNArgs(1),
 		RunE: func(cmd *cobra.Command, args []string) error {
 			clientCtx, err := client.GetClientQueryContext(cmd)
 			if err != nil {
@@ -86,3 +92,44 @@ func GetCmdGetChildren(queryRoute string) *cobra.Command {
 	flags.AddQueryFlagsToCmd(cmd)
 	return cmd
 }
+
+// GetCmdGetPath queries vstorage data or children, depending on the path
+func GetCmdGetPath(queryRoute string) *cobra.Command {
+	cmd := &cobra.Command{
+		Use:   "path <path>",
+		Short: "get vstorage data, or children if path ends with '.'",
+		Args:  cobra.ExactArgs(1),
+		RunE: func(cmd *cobra.Command, args []string) error {
+			clientCtx, err := client.GetClientQueryContext(cmd)
+			if err != nil {
+				return err
+			}
+			queryClient := types.NewQueryClient(clientCtx)
+
+			path := args[0]
+
+			var res proto.Message
+
+			// if path ends with '.' then remove it and query children
+			if path[len(path)-1] == '.' {
+				path = path[:len(path)-1]
+				res, err = queryClient.Children(cmd.Context(), &types.QueryChildrenRequest{
+					Path: path,
+				})
+			} else {
+				res, err = queryClient.Data(cmd.Context(), &types.QueryDataRequest{
+					Path: path,
+				})
+			}
+
+			if err != nil {
+				return err
+			}
+
+			return clientCtx.PrintProto(res)
+		},
+	}
+
+	flags.AddQueryFlagsToCmd(cmd)
+	return cmd
+}

package/x/vstorage/keeper/grpc_query.go

@@ -2,10 +2,14 @@ package keeper
 
 import (
 	"context"
+	"encoding/json"
+	"fmt"
+	"strings"
 
 	"google.golang.org/grpc/codes"
 	"google.golang.org/grpc/status"
 
+	"github.com/Agoric/agoric-sdk/golang/cosmos/x/vstorage/capdata"
 	"github.com/Agoric/agoric-sdk/golang/cosmos/x/vstorage/types"
 	sdk "github.com/cosmos/cosmos-sdk/types"
 )
@@ -17,6 +21,11 @@ type Querier struct {
 
 var _ types.QueryServer = Querier{}
 
+// ===================================================================
+// /agoric.vstorage.Query/Data
+// ===================================================================
+
+// /agoric.vstorage.Query/Data returns data for a specified path.
 func (k Querier) Data(c context.Context, req *types.QueryDataRequest) (*types.QueryDataResponse, error) {
 	if req == nil {
 		return nil, status.Error(codes.InvalidArgument, "empty request")
@@ -30,6 +39,218 @@ func (k Querier) Data(c context.Context, req *types.QueryDataRequest) (*types.Qu
 	}, nil
 }
 
+// ===================================================================
+// /agoric.vstorage.Query/CapData
+// ===================================================================
+
+const (
+	// Media types.
+	JSONLines = "JSON Lines"
+
+	// CapData transformation formats.
+	FormatCapDataFlat = "flat"
+
+	// CapData remotable value formats.
+	FormatRemotableAsObject = "object"
+	FormatRemotableAsString = "string"
+)
+
+var capDataResponseMediaTypes = map[string]string{
+	JSONLines: JSONLines,
+	// Default to JSON Lines.
+	"": JSONLines,
+}
+var capDataTransformationFormats = map[string]string{
+	FormatCapDataFlat: FormatCapDataFlat,
+	// Default to no transformation.
+	"": "",
+}
+var capDataRemotableValueFormats = map[string]string{
+	FormatRemotableAsObject: FormatRemotableAsObject,
+	FormatRemotableAsString: FormatRemotableAsString,
+	// No default because both formats are lossy.
+}
+
+// flatten converts data into a flat structure in which each deep leaf entry is replaced with
+// a top-level entry having the same value but a key generated by joining the keys on its path
+// with separating dashes.
+// For example,
+// ```
+// { "contacts": [
+//
+//	{ "name": "Alice", "email": "a@example.com" },
+//	{ "name": "Bob", "email": "b@example.com" }
+//
+// ] }
+// ```
+// becomes
+// ```
+//
+//	{
+//	  "contacts-0-name": "Alice",
+//	  "contacts-0-email": "a@example.com",
+//	  "contacts-1-name": "Bob",
+//	  "contacts-1-email": "b@example.com"
+//	}
+//
+// ```
+// cf. https://github.com/Agoric/agoric-sdk/blob/6e5b422b80e47c4dac151404f43faea5ab41e9b0/scripts/get-flattened-publication.sh
+func flatten(input interface{}, output map[string]interface{}, key string, top bool) error {
+	// Act on the raw representation of a Remotable.
+	if remotable, ok := input.(*capdata.CapdataRemotable); ok {
+		var replacement interface{}
+		repr, err := capdata.JsonMarshal(remotable)
+		if err == nil {
+			err = json.Unmarshal(repr, &replacement)
+		}
+		if err != nil {
+			return err
+		}
+		input = replacement
+	}
+
+	childKeyPrefix := key
+	if !top {
+		childKeyPrefix = childKeyPrefix + "-"
+	}
+	if arr, ok := input.([]interface{}); ok {
+		for i, v := range arr {
+			if err := flatten(v, output, childKeyPrefix+fmt.Sprintf("%d", i), false); err != nil {
+				return err
+			}
+		}
+	} else if obj, ok := input.(map[string]interface{}); ok {
+		for k, v := range obj {
+			if err := flatten(v, output, childKeyPrefix+k, false); err != nil {
+				return err
+			}
+		}
+	} else {
+		if _, has := output[key]; has {
+			return fmt.Errorf("key conflict: %q", key)
+		}
+		output[key] = input
+	}
+	return nil
+}
+
+// capdataBigintToDigits represents a bigint as a string consisting of
+// an optional "-" followed by a sequence of digits with no extraneous zeroes
+// (e.g., "0" or "-40").
+func capdataBigintToDigits(bigint *capdata.CapdataBigint) interface{} {
+	return bigint.Normalized
+}
+
+// capdataRemotableToString represents a Remotable as a bracketed string
+// containing its alleged name and id from `slots`
+// (e.g., "[Alleged: IST brand <board007>]").
+func capdataRemotableToString(r *capdata.CapdataRemotable) interface{} {
+	iface := "Remotable"
+	if r.Iface != nil || *r.Iface != "" {
+		iface = *r.Iface
+	}
+	return fmt.Sprintf("[%s <%s>]", iface, r.Id)
+}
+
+// capdataRemotableToObject represents a Remotable as an object containing
+// its id from `slots` and its alleged name minus any "Alleged:" prefix
+// (e.g., `{ "id": "board007", "allegedName": "IST brand" }`).
+func capdataRemotableToObject(r *capdata.CapdataRemotable) interface{} {
+	iface := "Remotable"
+	if r.Iface != nil || *r.Iface != "" {
+		iface = *r.Iface
+		iface, _ = strings.CutPrefix(iface, "Alleged: ")
+	}
+	return map[string]interface{}{"id": r.Id, "allegedName": iface}
+}
+
+// /agoric.vstorage.Query/CapData returns data for a specified path,
+// interpreted as CapData in a StreamCell (auto-promoting isolated CapData
+// into a single-item StreamCell) and transformed as specified.
+func (k Querier) CapData(c context.Context, req *types.QueryCapDataRequest) (*types.QueryCapDataResponse, error) {
+	if req == nil {
+		return nil, status.Error(codes.InvalidArgument, "empty request")
+	}
+	ctx := sdk.UnwrapSDKContext(c)
+
+	valueTransformations := capdata.CapdataValueTransformations{
+		Bigint: capdataBigintToDigits,
+	}
+
+	// A response Value is "<prefix><separator-joined items><suffix>".
+	prefix, separator, suffix := "", "\n", ""
+
+	// Read options.
+	mediaType, ok := capDataResponseMediaTypes[req.MediaType]
+	if !ok {
+		return nil, status.Error(codes.InvalidArgument, "invalid media_type")
+	}
+	transformation, ok := capDataTransformationFormats[req.ItemFormat]
+	if !ok {
+		return nil, status.Error(codes.InvalidArgument, "invalid item_format")
+	}
+	switch remotableFormat, ok := capDataRemotableValueFormats[req.RemotableValueFormat]; {
+	case !ok:
+		return nil, status.Error(codes.InvalidArgument, "invalid remotable_value_format")
+	case remotableFormat == FormatRemotableAsObject:
+		valueTransformations.Remotable = capdataRemotableToObject
+	case remotableFormat == FormatRemotableAsString:
+		valueTransformations.Remotable = capdataRemotableToString
+	}
+
+	// Read data, auto-upgrading a standalone value to a single-value StreamCell.
+	entry := k.GetEntry(ctx, req.Path)
+	if !entry.HasValue() {
+		return nil, status.Error(codes.FailedPrecondition, "no data")
+	}
+	value := entry.StringValue()
+	var cell StreamCell
+	_ = json.Unmarshal([]byte(value), &cell)
+	if cell.BlockHeight == "" {
+		cell = StreamCell{Values: []string{value}}
+	}
+
+	// Format each StreamCell value.
+	responseItems := make([]string, len(cell.Values))
+	for i, capDataJson := range cell.Values {
+		item, err := capdata.DecodeSerializedCapdata(capDataJson, valueTransformations)
+		if err != nil {
+			return nil, status.Error(codes.FailedPrecondition, err.Error())
+		}
+		if transformation == FormatCapDataFlat {
+			flattened := map[string]interface{}{}
+			if err := flatten(item, flattened, "", true); err != nil {
+				return nil, status.Error(codes.Internal, err.Error())
+			}
+			// Replace the item, unless it was a scalar that "flattened" to `{ "": ... }`.
+			if _, singleton := flattened[""]; !singleton {
+				item = flattened
+			}
+		}
+		switch mediaType {
+		case JSONLines:
+			jsonText, err := capdata.JsonMarshal(item)
+			if err != nil {
+				return nil, status.Error(codes.Internal, err.Error())
+			}
+			responseItems[i] = string(jsonText)
+		}
+	}
+
+	return &types.QueryCapDataResponse{
+		BlockHeight: cell.BlockHeight,
+		Value:       prefix + strings.Join(responseItems, separator) + suffix,
+	}, nil
+}
+
+// ===================================================================
+// /agoric.vstorage.Query/Children
+// ===================================================================
+
+// /agoric.vstorage.Query/Children returns the list of path segments
+// that exist immediately underneath a specified path, including
+// those corresponding with "empty non-terminals" having children
+// but no data of their own.
 func (k Querier) Children(c context.Context, req *types.QueryChildrenRequest) (*types.QueryChildrenResponse, error) {
 	if req == nil {
 		return nil, status.Error(codes.InvalidArgument, "empty request")

package/x/vstorage/keeper/keeper.go

@@ -10,6 +10,7 @@ import (
 	"strconv"
 	"strings"
 
+	storetypes "github.com/cosmos/cosmos-sdk/store/types"
 	sdk "github.com/cosmos/cosmos-sdk/types"
 	db "github.com/tendermint/tm-db"
 
@@ -62,7 +63,7 @@ func cutPrefix(s, prefix []byte) (after []byte, found bool) {
 // for the various parts of the state machine
 type Keeper struct {
 	changeManager ChangeManager
-	storeKey      sdk.StoreKey
+	storeKey      storetypes.StoreKey
 }
 
 func (bcm *BatchingChangeManager) Track(ctx sdk.Context, k Keeper, entry agoric.KVEntry, isLegacy bool) {
@@ -116,7 +117,7 @@ func NewBatchingChangeManager() *BatchingChangeManager {
 	return &bcm
 }
 
-func NewKeeper(storeKey sdk.StoreKey) Keeper {
+func NewKeeper(storeKey storetypes.StoreKey) Keeper {
 	return Keeper{
 		storeKey:      storeKey,
 		changeManager: NewBatchingChangeManager(),

package/x/vstorage/keeper/keeper_grpc_test.go

@@ -0,0 +1,300 @@
+package keeper
+
+import (
+	"fmt"
+	"reflect"
+	"strings"
+	"testing"
+
+	grpcCodes "google.golang.org/grpc/codes"
+	grpcStatus "google.golang.org/grpc/status"
+
+	agoric "github.com/Agoric/agoric-sdk/golang/cosmos/types"
+	"github.com/Agoric/agoric-sdk/golang/cosmos/x/vstorage/capdata"
+	"github.com/Agoric/agoric-sdk/golang/cosmos/x/vstorage/types"
+
+	sdk "github.com/cosmos/cosmos-sdk/types"
+)
+
+func ptr[T any](v T) *T {
+	return &v
+}
+
+func mustJsonMarshal(val any) string {
+	jsonText, err := capdata.JsonMarshal(val)
+	if err != nil {
+		panic(err)
+	}
+	return string(jsonText)
+}
+
+func mustMarshalStreamCell(blockHeight string, values []string) string {
+	cell := map[string]any{
+		"blockHeight": blockHeight,
+		"values":      values,
+	}
+	return mustJsonMarshal(cell)
+}
+
+func TestCapData(t *testing.T) {
+	testKit := makeTestKit()
+	ctx, keeper := testKit.ctx, testKit.vstorageKeeper
+	querier := Querier{keeper}
+
+	type testCase struct {
+		label       string
+		data        *string
+		request     types.QueryCapDataRequest
+		expected    types.QueryCapDataResponse
+		errCode     grpcCodes.Code
+		errContains *string
+	}
+	testCases := []testCase{}
+
+	// Test simple cases (various kinds of bad data up to simple flat JSON-compatible CapData).
+	decodableSmallcaps := `{"body":"#true","slots":[]}`
+	decodableLegacy := `{"body":"true","slots":[]}`
+	testCases = append(testCases, []testCase{
+		{label: "no data",
+			data:        nil,
+			request:     types.QueryCapDataRequest{RemotableValueFormat: "string"},
+			errCode:     grpcCodes.FailedPrecondition,
+			errContains: ptr("no data"),
+		},
+		{label: "zero-length data",
+			data:        ptr(""),
+			request:     types.QueryCapDataRequest{RemotableValueFormat: "string"},
+			errCode:     grpcCodes.FailedPrecondition,
+			errContains: ptr("JSON"),
+		},
+		{label: "cell with empty string",
+			data:        ptr(mustMarshalStreamCell("1", []string{decodableSmallcaps, "", decodableLegacy})),
+			request:     types.QueryCapDataRequest{RemotableValueFormat: "string"},
+			errCode:     grpcCodes.FailedPrecondition,
+			errContains: ptr("JSON"),
+		},
+		{label: "non-JSON data",
+			data:        ptr("foo"),
+			request:     types.QueryCapDataRequest{RemotableValueFormat: "string"},
+			errCode:     grpcCodes.FailedPrecondition,
+			errContains: ptr("invalid"),
+		},
+		{label: "cell with non-JSON",
+			data:        ptr(mustMarshalStreamCell("1", []string{decodableSmallcaps, "foo", decodableLegacy})),
+			request:     types.QueryCapDataRequest{RemotableValueFormat: "string"},
+			errCode:     grpcCodes.FailedPrecondition,
+			errContains: ptr("invalid"),
+		},
+		{label: "lone non-CapData",
+			data:        ptr("{}"),
+			request:     types.QueryCapDataRequest{RemotableValueFormat: "string"},
+			errCode:     grpcCodes.FailedPrecondition,
+			errContains: ptr("invalid CapData"),
+		},
+		{label: "cell with non-CapData",
+			data:        ptr(mustMarshalStreamCell("1", []string{decodableSmallcaps, "{}", decodableLegacy})),
+			request:     types.QueryCapDataRequest{RemotableValueFormat: "string"},
+			errCode:     grpcCodes.FailedPrecondition,
+			errContains: ptr("invalid CapData"),
+		},
+		{label: "lone smallcaps CapData",
+			data:     ptr(decodableSmallcaps),
+			request:  types.QueryCapDataRequest{RemotableValueFormat: "string"},
+			expected: types.QueryCapDataResponse{Value: `true`},
+		},
+		{label: "cell with smallcaps CapData",
+			data:     ptr(mustMarshalStreamCell("1", []string{decodableSmallcaps})),
+			request:  types.QueryCapDataRequest{RemotableValueFormat: "string"},
+			expected: types.QueryCapDataResponse{BlockHeight: "1", Value: `true`},
+		},
+		{label: "lone legacy CapData",
+			data:     ptr(decodableLegacy),
+			request:  types.QueryCapDataRequest{RemotableValueFormat: "string"},
+			expected: types.QueryCapDataResponse{Value: `true`},
+		},
+		{label: "cell with legacy CapData",
+			data:     ptr(mustMarshalStreamCell("1", []string{decodableLegacy})),
+			request:  types.QueryCapDataRequest{RemotableValueFormat: "string"},
+			expected: types.QueryCapDataResponse{BlockHeight: "1", Value: `true`},
+		},
+	}...)
+
+	// Test option validation.
+	testCases = append(testCases, []testCase{
+		{label: "explicit JSON Lines",
+			data:     ptr(decodableSmallcaps),
+			request:  types.QueryCapDataRequest{MediaType: "JSON Lines", RemotableValueFormat: "string"},
+			expected: types.QueryCapDataResponse{Value: `true`},
+		},
+		{label: "invalid media type",
+			data:        ptr(decodableSmallcaps),
+			request:     types.QueryCapDataRequest{MediaType: "JSONLines", RemotableValueFormat: "string"},
+			errCode:     grpcCodes.InvalidArgument,
+			errContains: ptr("media_type"),
+		},
+		{label: "invalid item format",
+			data:        ptr(decodableSmallcaps),
+			request:     types.QueryCapDataRequest{ItemFormat: "deep", RemotableValueFormat: "string"},
+			errCode:     grpcCodes.InvalidArgument,
+			errContains: ptr("item_format"),
+		},
+		{label: "missing remotable value format",
+			data:        ptr(decodableSmallcaps),
+			request:     types.QueryCapDataRequest{},
+			errCode:     grpcCodes.InvalidArgument,
+			errContains: ptr("remotable_value_format"),
+		},
+		{label: "invalid remotable value format",
+			data:        ptr(decodableSmallcaps),
+			request:     types.QueryCapDataRequest{RemotableValueFormat: "foo"},
+			errCode:     grpcCodes.InvalidArgument,
+			errContains: ptr("remotable_value_format"),
+		},
+	}...)
+
+	// Test formatting options against sufficiently complex CapData,
+	// deriving legacy encoding from smallcaps encoding to ensure equivalence
+	// and deriving expectations from marshalling to avoid spurious mismatches
+	// from Go's unpredictable field ordering (e.g., `{"a":0,"b":1}` vs. `{"b":1,"a":0}`).
+	slots := []any{"a"}
+	deepSmallcapsBody := `{"arr":[{"bigint":"+42","remotable":"$0.Alleged: Foo brand","ref2":"$0"}]}`
+	deepLegacyBody := deepSmallcapsBody
+	legacyFromSmallcaps := [][2]string{
+		[2]string{`"+42"`, `{"@qclass":"bigint","digits":"42"}`},
+		[2]string{`"$0.Alleged: Foo brand"`, `{"@qclass":"slot","index":0,"iface":"Alleged: Foo brand"}`},
+		[2]string{`"$0"`, `{"@qclass":"slot","index":0}`},
+	}
+	for _, pair := range legacyFromSmallcaps {
+		deepLegacyBody = strings.Replace(deepLegacyBody, pair[0], pair[1], -1)
+	}
+	cell := mustMarshalStreamCell("1", []string{
+		mustJsonMarshal(map[string]any{"body": "#" + deepSmallcapsBody, "slots": slots}),
+		mustJsonMarshal(map[string]any{"body": deepLegacyBody, "slots": slots}),
+	})
+	mustMarshalTwoLines := func(val any) string {
+		line := mustJsonMarshal(val)
+		return fmt.Sprintf("%s\n%s", line, line)
+	}
+	testCases = append(testCases, testCase{label: "remotables as strings",
+		data:    ptr(cell),
+		request: types.QueryCapDataRequest{RemotableValueFormat: "string"},
+		expected: types.QueryCapDataResponse{
+			BlockHeight: "1",
+			Value: mustMarshalTwoLines(map[string]any{
+				"arr": []any{
+					map[string]any{
+						"bigint":    "42",
+						"remotable": "[Alleged: Foo brand <a>]",
+						"ref2":      "[Alleged: Foo brand <a>]",
+					},
+				},
+			}),
+		},
+	})
+	testCases = append(testCases, testCase{label: "remotables as strings, flat",
+		data:    ptr(cell),
+		request: types.QueryCapDataRequest{ItemFormat: "flat", RemotableValueFormat: "string"},
+		expected: types.QueryCapDataResponse{
+			BlockHeight: "1",
+			Value: mustMarshalTwoLines(map[string]any{
+				"arr-0-bigint":    "42",
+				"arr-0-remotable": "[Alleged: Foo brand <a>]",
+				"arr-0-ref2":      "[Alleged: Foo brand <a>]",
+			}),
+		},
+	})
+	testCases = append(testCases, testCase{label: "remotables as objects",
+		data:    ptr(cell),
+		request: types.QueryCapDataRequest{RemotableValueFormat: "object"},
+		expected: types.QueryCapDataResponse{
+			BlockHeight: "1",
+			Value: mustMarshalTwoLines(map[string]any{
+				"arr": []any{
+					map[string]any{
+						"bigint":    "42",
+						"remotable": map[string]any{"id": "a", "allegedName": "Foo brand"},
+						"ref2":      map[string]any{"id": "a", "allegedName": "Foo brand"},
+					},
+				},
+			}),
+		},
+	})
+	testCases = append(testCases, testCase{label: "remotables as objects, flat",
+		data:    ptr(cell),
+		request: types.QueryCapDataRequest{ItemFormat: "flat", RemotableValueFormat: "object"},
+		expected: types.QueryCapDataResponse{
+			BlockHeight: "1",
+			Value: mustMarshalTwoLines(map[string]any{
+				"arr-0-bigint":                "42",
+				"arr-0-remotable-id":          "a",
+				"arr-0-remotable-allegedName": "Foo brand",
+				"arr-0-ref2-id":               "a",
+				"arr-0-ref2-allegedName":      "Foo brand",
+			}),
+		},
+	})
+
+	// Test errors from CapData that includes unsupported values.
+	expectNotImplemented := func(label, capdataBody string, slots []any) testCase {
+		if slots == nil {
+			slots = []any{}
+		}
+		serialized := mustJsonMarshal(map[string]any{
+			"body":  capdataBody,
+			"slots": slots,
+		})
+		return testCase{
+			label:       label,
+			data:        ptr(serialized),
+			request:     types.QueryCapDataRequest{RemotableValueFormat: "string"},
+			errCode:     grpcCodes.FailedPrecondition,
+			errContains: ptr("not implemented"),
+		}
+	}
+	testCases = append(testCases, []testCase{
+		expectNotImplemented("smallcaps undefined", `#"#undefined"`, nil),
+		expectNotImplemented("smallcaps NaN", `#"#NaN"`, nil),
+		expectNotImplemented("smallcaps infinity", `#"#Infinity"`, nil),
+		expectNotImplemented("smallcaps negative infinity", `#"#-Infinity"`, nil),
+		expectNotImplemented("smallcaps symbol", `#"%foo"`, nil),
+		expectNotImplemented("smallcaps promise", `#"&0"`, []any{"a"}),
+		expectNotImplemented("smallcaps tagged", `#{"#tag":"copySet","payload":[]}`, nil),
+		expectNotImplemented("smallcaps error", `#{"#error":"foo","name":"Error"}`, nil),
+		expectNotImplemented("legacy undefined", `{"@qclass":"undefined"}`, nil),
+		expectNotImplemented("legacy NaN", `{"@qclass":"NaN"}`, nil),
+		expectNotImplemented("legacy infinity", `{"@qclass":"Infinity"}`, nil),
+		expectNotImplemented("legacy negative infinity", `{"@qclass":"-Infinity"}`, nil),
+		expectNotImplemented("legacy symbol", `{"@qclass":"symbol","name":"foo"}`, nil),
+		expectNotImplemented("smallcaps tagged", `{"@qclass":"tagged","tag":"copySet","payload":[]}`, nil),
+		expectNotImplemented("smallcaps error", `{"@qclass":"error","message":"foo","name":"Error"}`, nil),
+		expectNotImplemented("smallcaps Hilbert Hotel", `{"@qclass":"hilbert","original":"foo"}`, nil),
+	}...)
+	for _, desc := range testCases {
+		desc.request.Path = "key"
+		if desc.data == nil {
+			keeper.SetStorage(ctx, agoric.NewKVEntryWithNoValue(desc.request.Path))
+		} else {
+			keeper.SetStorage(ctx, agoric.NewKVEntry(desc.request.Path, *desc.data))
+		}
+		resp, err := querier.CapData(sdk.WrapSDKContext(ctx), &desc.request)
+		if desc.errCode == grpcCodes.OK {
+			if err != nil {
+				t.Errorf("%s: got unexpected error %v", desc.label, err)
+			} else if reflect.DeepEqual(resp, &desc.expected) {
+				continue
+			}
+			if resp.Value != desc.expected.Value {
+				lines := strings.Split(resp.Value, "\n")
+				t.Errorf("%s: wrong result value lines: %#q", desc.label, lines)
+			} else {
+				t.Errorf("%s: wrong result: %#v", desc.label, resp)
+			}
+		} else if err == nil {
+			t.Errorf("%s: got no error, want error %q", desc.label, *desc.errContains)
+		} else if code := grpcStatus.Code(err); code != desc.errCode {
+			t.Errorf("%s: got error code %q, want %q", desc.label, code, desc.errCode)
+		} else if desc.errContains != nil && !strings.Contains(err.Error(), *desc.errContains) {
+			t.Errorf("%s: got error %v, want error %q", desc.label, err, *desc.errContains)
+		}
+	}
+}

package/x/vstorage/keeper/keeper_test.go

@@ -31,7 +31,7 @@ func makeTestKit() testKit {
 
 	db := dbm.NewMemDB()
 	ms := store.NewCommitMultiStore(db)
-	ms.MountStoreWithDB(vstorageStoreKey, sdk.StoreTypeIAVL, db)
+	ms.MountStoreWithDB(vstorageStoreKey, storetypes.StoreTypeIAVL, db)
 	err := ms.LoadLatestVersion()
 	if err != nil {
 		panic(err)

package/x/vstorage/keeper/querier.go

@@ -33,7 +33,11 @@ func getVstorageEntryPath(urlPathSegments []string) (string, error) {
 // be used to extend it to a vstorage path such as "foo.bar.baz").
 func NewQuerier(keeper Keeper, legacyQuerierCdc *codec.LegacyAmino) sdk.Querier {
 	return func(ctx sdk.Context, urlPathSegments []string, req abci.RequestQuery) (res []byte, err error) {
-		switch urlPathSegments[0] {
+		var queryType string
+		if len(urlPathSegments) > 0 {
+			queryType = urlPathSegments[0]
+		}
+		switch queryType {
 		case QueryData:
 			entryPath, entryPathErr := getVstorageEntryPath(urlPathSegments[1:])
 			if entryPathErr != nil {
@@ -47,7 +51,7 @@ func NewQuerier(keeper Keeper, legacyQuerierCdc *codec.LegacyAmino) sdk.Querier
 			}
 			return queryChildren(ctx, entryPath, req, keeper, legacyQuerierCdc)
 		default:
-			return nil, sdkerrors.Wrap(sdkerrors.ErrUnknownRequest, "unknown vstorage query endpoint")
+			return nil, sdkerrors.Wrap(sdkerrors.ErrUnknownRequest, "unknown vstorage query path")
 		}
 	}
 }