fix(graphql): fix graphql.operation.name field

[dchambers]

Which problem is this PR solving?

• Graphql should add operation name and hash in span #705
• @opentelemetry/instrumentation-graphql sets graphql.operation.name span attribute to operation type #847

Short description of the changes

• Change graphql.operation.name to send the GraphQL operation-name (a critical field for GraphQL o11y) instead of the GraphQL operation-type.
• Continue to send the GraphQL operation-type as graphql.operation.type, since this is still useful for segmenting read queries from write mutations, etc.

Checklist

• Ran npm run test-all-versions for the edited package(s) on the latest commit if applicable.

[linux-foundation-easycla]

The committers are authorized under a signed CLA.

• ✅ Dominic Chambers (83f7bdf)

[dchambers]

@dyladan, might you be able to help move this PR out of its current state?

[dyladan]

@dyladan, might you be able to help move this PR out of its current state?

Just need to wait for some reviews. It's only been 24 hours since it was opened and the contributors/reviewers are in many timezones so it can take a few days sometimes.

[codecov]

Codecov Report

Merging #903 (aea5851) into main (bf39b90) will decrease coverage by 0.67%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #903      +/-   ##
==========================================
- Coverage   95.91%   95.23%   -0.68%     
==========================================
  Files          13       20       +7     
  Lines         856     1301     +445     
  Branches      178      265      +87     
==========================================
+ Hits          821     1239     +418     
- Misses         35       62      +27     

Impacted Files	Coverage Δ
...nstrumentation-graphql/src/enums/AttributeNames.ts	100.00% <100.00%> (ø)
...try-instrumentation-graphql/src/instrumentation.ts	91.66% <100.00%> (ø)
...opentelemetry-instrumentation-graphql/src/utils.ts	93.40% <0.00%> (ø)
...opentelemetry-instrumentation-graphql/src/types.ts	100.00% <0.00%> (ø)
...tapackages/auto-instrumentations-node/src/utils.ts	97.87% <0.00%> (ø)
...entelemetry-instrumentation-graphql/src/symbols.ts	100.00% <0.00%> (ø)
.../opentelemetry-instrumentation-graphql/src/enum.ts	100.00% <0.00%> (ø)

[dchambers]

Hi @obecny 👋. You seem to be the last reviewer on the roster for this PR. Any chance you'd be able to take a look please?

[dchambers]

Hi @obecny, it's been another week. Any chance you'd be able to look at this PR please?

[dyladan]

@obecny is no longer a maintainer on the project. sorry for the confusion

[Flarna]

he is still part of the approver group and the owner of the graphql instrumentation.

[dchambers]

@obecny is no longer a maintainer on the project. sorry for the confusion

I see, thank you @dyladan. Do you know who would now be the right person to ping regarding getting this PR merged in that case?

[dchambers]

he is still part of the approver group and the owner of the graphql instrumentation.

This makes sense too @Flarna 👍 , because he's showing as the last person I need an approval from before I can merge.

[dyladan]

I am the right person :)

[dchambers]

Thank you guys! 🙏