Skip to content

Use raw body for managed comment comparison#830

Merged
kitsuyui merged 1 commit into
mainfrom
use-raw-managed-comment-body
May 21, 2026
Merged

Use raw body for managed comment comparison#830
kitsuyui merged 1 commit into
mainfrom
use-raw-managed-comment-body

Conversation

@kitsuyui
Copy link
Copy Markdown
Owner

Summary

  • compare existing managed comments with the raw markdown body when available
  • keep body_text as a fallback for comment payloads without body
  • cover the GitHub API shape where body_text omits the HTML comment marker

Validation

  • bun install --frozen-lockfile
  • bun run build
  • bun run lint
  • bun run test
  • git diff --check

Compare existing managed comments with the raw markdown comment body when the API returns it. This keeps the hidden marker available for equality checks while retaining body_text as a fallback.
@github-actions
Copy link
Copy Markdown

🎉 Happy commit!

@github-actions
Copy link
Copy Markdown

gh-counter

PR gate

Removed Added +/-
TODO/FIXME 0 0 0

Repo dashboard

main (048b9dd) #830 (d4bf64f) +/-
TODO/FIXME 0 0 0

Reported by gh-counter

@github-actions
Copy link
Copy Markdown

Code Metrics Report

main (048b9dd) #830 (d4bf64f) +/-
Coverage 99.0% 99.0% 0.0%
Code to Test Ratio 1:0.5 1:0.5 0.0
Test Execution Time 1s 1s 0s
Details
  |                     | main (048b9dd) | #830 (d4bf64f) | +/-  |
  |---------------------|----------------|----------------|------|
  | Coverage            |          99.0% |          99.0% | 0.0% |
  |   Files             |              5 |              5 |    0 |
  |   Lines             |            107 |            107 |    0 |
  |   Covered           |            106 |            106 |    0 |
  | Code to Test Ratio  |          1:0.5 |          1:0.5 |  0.0 |
  |   Code              |           1408 |           1408 |    0 |
  |   Test              |            777 |            777 |    0 |
  | Test Execution Time |             1s |             1s |   0s |

Code coverage of files in pull request scope (100.0% → 100.0%)

Files Coverage +/- Status
src/github.ts 100.0% 0.0% modified

Reported by octocov

@github-actions
Copy link
Copy Markdown

gh-build-size

Target main #830 +/-
Total action artifacts 23,290,052 B 23,290,132 B +80 B
Runtime bundle 1,151,843 B 1,151,854 B +11 B
Source maps 17,122,599 B 17,122,668 B +69 B

Reported by gh-build-size

@kitsuyui kitsuyui merged commit 9cb3a79 into main May 21, 2026
5 checks passed
@kitsuyui kitsuyui deleted the use-raw-managed-comment-body branch May 21, 2026 13:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant