3

我有一个监听所有代码审查的 webhook,然后我获取此 PR 审查的评论,以便获得评论在差异中的位置。我正在使用 GitHub REST API,但我遇到的问题与 GraphQL API 相同。

所以工作流程是:

  1. 从 webhook 中获取评论 ID
  2. 获取该评论的评论列表
  3. 对于每条评论,获取 diff hunk 和找到编辑行的位置

所有这些在 99% 的时间里都能正常工作。有时我null在这个位置上,我忽略了这些评论。

但这一次,我遇到了另一个奇怪的问题。通常,位置是指 diff 中行的索引。

例如,在:

@@ -1 +1,3 @@
-# sedy-test
\\ No newline at end of file
+# sedy-test
+
+This repository is used to test [sedy](https://github.com/marmelab/sedy).

如果位置为 3,则编辑的行为+# sedy-test

问题是对于某些评论,我得到的位置不能在差异内。例如,请参阅此 PR

当我尝试通过以下请求获取评论的评论位置时:

{
  repository(owner: "Kmaschta", name: "comfygure") {
    pullRequest(number: 1) {
      reviews(last: 1) {
        edges {
          node {
            state
            comments(first: 1) {
              edges {
                node {
                  bodyText
                  authorAssociation
                  position
                  originalPosition
                  diffHunk
                }
              }
            }
          }
        }
      }
    }
  }
}

响应如下:

{
  "data": {
    "repository": {
      "pullRequest": {
        "reviews": {
          "edges": [
            {
              "node": {
                "state": "COMMENTED",
                "comments": {
                  "edges": [
                    {
                      "node": {
                        "bodyText": "s/fot/for/",
                        "authorAssociation": "OWNER",
                        "position": 71,
                        "originalPosition": 71,
                        "diffHunk": "@@ -24,31 +34,39 @@ const ls = (ui, modules) => function* () {\n };\n \n const add = (ui, modules, options) => function* () {\n-    const { red, bold } = ui.colors;\n+    const { red, bold, green } = ui.colors;\n \n     if (!options.length) {\n         ui.error(`${red('No environment specified.')}`);\n-        help(ui, 1);\n     }\n \n     if (options.length > 1) {\n         ui.error(`${red('Invalid environment format. The environment name should be one word.')}`);\n-        help(ui, 1);\n+    }\n+\n+    if (options.length !== 1) {\n+        ui.print(`${bold('SYNOPSIS')}\n+        ${bold('comfy')} env add <environment>\n+\n+Type ${green('comfy env --help')} for details`);\n+\n+        return ui.exit(0);\n     }\n \n     const project = yield modules.project.retrieveFromConfig();\n     const environment = yield modules.environment.add(project, options[0]);\n-    const addCommand = `comfy add ${environment.name}`;\n+    const addCommand = `comfy setall ${environment.name}`;\n \n-    ui.print(`${bold('Cool!')} Your new environment \"${bold(environment.name)}\" was successfully saved.`);\n-    ui.print(`You can now add a configuration, try ${bold(addCommand)}`);\n+    ui.print(`${bold(green('Environment successfully created'))}`);\n+    ui.print(`You can now set a configuration fot this environment using ${bold(addCommand)}`);"
                      }
                    }
                  ]
                }
              }
            }
          ]
        }
      }
    }
  }
}

位置是 71,但差异不包含超过 40 行。

如果是 GitHub API,或者我不理解 position 字段的意义,那么这是一个错误吗?

注意:我在 GitHub 论坛上发布了同样的问题

4

2 回答 2

3

来自Github API 评论文档

位置值等于要添加注释的文件中从第一个“@@”大块标题向下的行数。"@@" 行正下方的行是位置 1,下一行是位置 2,依此类推。diff 中的位置通过空白行和其他大块继续增加,直到新文件的开头。

这里diffHunk给你当前的差异大块,这不是文件中的第一个

如果你得到完整的差异文件,那就更清楚了:

curl "https://api.github.com/repos/Kmaschta/comfygure/pulls/1" \
     -H "Accept: application/vnd.github.v3.diff"

注释env.js的第一个块从第 77 行开始,您的注释在第 148 行,而diffHunk您的请求从第 114 行开始

我认为目前无法使用 GraphQL 请求完整的 PR 差异,但您可以像上面一样使用 Rest v3

于 2018-10-21T01:10:37.817 回答
1

我有同样的问题。我终于找到了如何确定位置。

让我们看看你的公关。

https://github.com/Kmaschta/comfygure/pull/1/files?utf8=%E2%9C%93&diff=unified#diff-10b371776dce3b12ed817f3fb8704a7d

该文件有 2 个差异块。

位置从第一个大块开始。

第一个大块下方的行是位置 1。

https://github.com/Kmaschta/comfygure/pull/1/files?utf8=%E2%9C%93&diff=unified#diff-10b371776dce3b12ed817f3fb8704a7dL1

第一个大块的结尾是位置 36。

https://github.com/Kmaschta/comfygure/pull/1/files?utf8=%E2%9C%93&diff=unified#diff-10b371776dce3b12ed817f3fb8704a7dL18

并且不知何故github在第二个大块开始之前添加了第一个大块结束的+1。(36 + 1)

所以,第二个大块的起始线是 38。

在第二个大块中,您的评论上方有 34 行。

这就是为什么你的评论是 71。

https://github.com/Kmaschta/comfygure/pull/1/files?utf8=%E2%9C%93&diff=unified#diff-10b371776dce3b12ed817f3fb8704a7dR61

Github的计算方法是这样的。

我认为这个计算有问题。但是如果你想计算,你可以通过这种方法来做。

于 2018-11-06T14:46:52.063 回答