@interfaceObject is not supported.

[wangmir]

What happened?

interfaceObject is apollo federation directive. But, when I try to use it, it falsly generate directive handler at generated file.

it generates InterfaceObject at DirectiveRoot on generated exec.go

type DirectiveRoot struct {
	ComposeDirective func(ctx context.Context, obj interface{}, next graphql.Resolver, name string) (res interface{}, err error)
	InterfaceObject  func(ctx context.Context, obj interface{}, next graphql.Resolver) (res interface{}, err error)
}

What did you expect?

Seamlessly handling without adding new directive handler.

I think it's because interfaceObject is missing from

gqlgen/plugin/federation/federation.go

Lines 80 to 88 in 919aded

	if f.Version == 2 {
	cfg.Directives["shareable"] = config.DirectiveConfig{SkipRuntime: true}
	cfg.Directives["link"] = config.DirectiveConfig{SkipRuntime: true}
	cfg.Directives["tag"] = config.DirectiveConfig{SkipRuntime: true}
	cfg.Directives["override"] = config.DirectiveConfig{SkipRuntime: true}
	cfg.Directives["inaccessible"] = config.DirectiveConfig{SkipRuntime: true}
	cfg.Directives["authenticated"] = config.DirectiveConfig{SkipRuntime: true}
	cfg.Directives["requiresScopes"] = config.DirectiveConfig{SkipRuntime: true}
	}

Minimal graphql.schema and models to reproduce

just adding interfaceObject on existing test data of federation2.

extend schema
  @link(url: "https://specs.apollo.dev/federation/v2.3",
        import: ["@key", "@shareable", "@provides", "@external", "@tag", "@extends", "@override", "@inaccessible", "@interfaceObject"])

schema {
    query: CustomQuery
}

type Hello @key(fields:"name", resolvable: false) @interfaceObject {
  name: String!
}

type World @key(fields: "foo bar", resolvable: false) {
    foo: String!
    bar: Int!
}

extend type ExternalExtension @key(fields: "  upc    ") {
    upc: String!
    reviews: [Hello]
}

type CustomQuery {
  hello: Hello!
}

versions

• go run github.com/99designs/gqlgen version?
  0.17.38
• go version?
  1.20.4

[wangmir]

It seems this PR (#2699) is the fix, but it's not getting in. @StevenACoffman can we add this?

[StevenACoffman]

The original author hasn't gotten it to pass both tests and linting and resolved the merge conflicts. Would you make a new PR that does?

[wangmir]

@StevenACoffman Ok, I'll try then.

[StevenACoffman]

Fixed in #2821