GraphQL-Java 中无效错误位置的处理优化

在 GraphQL 规范中，错误位置（error location）是一个非常重要的概念。它用于指示 GraphQL 文档中出现问题的具体位置，帮助开发者快速定位和修复问题。根据 GraphQL 规范，错误位置应该是一个有效的定位点，即行号和列号都应该是正整数。

最近在 graphql-java 项目中发现了一个问题：当解析器无法确定具体错误位置时，会返回一个包含负值的 SourceLocation 对象（line: -1, column: -1）。这种行为虽然技术上可行，但并不完全符合 GraphQL 规范的最佳实践。

问题背景

在 graphql-java 的实现中，当解析器遇到错误但无法确定具体位置时（例如到达输入流的末尾），会使用一个特殊的 EMPTY 对象来表示错误位置。这个 EMPTY 对象内部使用 -1 作为行号和列号的默认值。

GraphQL 规范明确指出："如果一个错误可以与请求的 GraphQL 文档中的特定点相关联，它应该包含一个带有位置列表的 locations 条目"。这里的"应该"（should）意味着这是一种推荐做法，而不是强制要求。

技术影响

虽然返回负值的位置信息不会导致功能性问题，但可能会带来以下影响：

1. 客户端工具可能无法正确处理这种特殊的位置值
2. 违反了 GraphQL 规范的推荐做法
3. 可能造成开发者困惑，因为负值的位置没有实际意义

解决方案

graphql-java 团队决定在错误位置无法确定的情况下，不在响应中包含 locations 字段，而不是返回无效的位置值。这种处理方式更符合规范精神，因为规范明确指出位置信息应该在能够关联到具体语法元素时才提供。

这种改进将在以下方面带来好处：

1. 更清晰的错误响应结构
2. 更好的规范合规性
3. 更一致的客户端处理体验

结论

graphql-java 团队对错误位置处理的优化展示了他们对规范细节的关注和对开发者体验的重视。这种改进虽然看似微小，但对于构建健壮的 GraphQL 生态系统非常重要。开发者现在可以期待更规范的错误响应，特别是在处理复杂查询或边缘情况时。

这个变更将在 graphql-java 的下一个版本中发布，建议使用者关注更新日志以获取更多细节。