From b25cb3d10e9bc0117e906ec76dbb728b7fadf015 Mon Sep 17 00:00:00 2001 From: Ned Twigg Date: Thu, 18 Sep 2025 13:10:20 -0700 Subject: [PATCH 1/4] strip down the `githubEditComment` to the basics --- .../enhancers/github/githubEditComment.tsx | 44 +++++-------------- 1 file changed, 10 insertions(+), 34 deletions(-) diff --git a/src/lib/enhancers/github/githubEditComment.tsx b/src/lib/enhancers/github/githubEditComment.tsx index 4f6ea46..1f52bcf 100644 --- a/src/lib/enhancers/github/githubEditComment.tsx +++ b/src/lib/enhancers/github/githubEditComment.tsx @@ -1,6 +1,6 @@ import OverType, { type OverTypeInstance } from 'overtype' import type React from 'react' -import type { CommentEnhancer, CommentSpot } from '@/lib/enhancer' +import type { CommentEnhancer, CommentSpot, StrippedLocation } from '@/lib/enhancer' import { logger } from '@/lib/logger' import { modifyDOM } from '../modifyDOM' import { commonGithubOptions } from './ghOptions' @@ -8,10 +8,6 @@ import { prepareGitHubHighlighter } from './githubHighlighter' export interface GitHubEditCommentSpot extends CommentSpot { type: 'GH_EDIT_COMMENT' - title: string - domain: string - slug: string - number: number } export class GitHubEditCommentEnhancer implements CommentEnhancer { @@ -19,32 +15,18 @@ export class GitHubEditCommentEnhancer implements CommentEnhancer', } } @@ -59,17 +41,11 @@ export class GitHubEditCommentEnhancer implements CommentEnhancer - {slug} - PR #{number} - - ) + tableUpperDecoration(_spot: GitHubEditCommentSpot): React.ReactNode { + return N/A } tableTitle(_spot: GitHubEditCommentSpot): string { - return 'TITLE_TODO' + return 'N/A' } } From bce5f389a593412b09c30575b9eb7aacb8d597c1 Mon Sep 17 00:00:00 2001 From: Ned Twigg Date: Thu, 18 Sep 2025 13:22:18 -0700 Subject: [PATCH 2/4] fix conflict between issueAdd and Edit --- .../enhancers/github/githubEditComment.tsx | 22 +++++++- .../github/githubIssueAddComment.tsx | 6 ++ tests/lib/enhancers/github.test.ts | 55 +++---------------- 3 files changed, 33 insertions(+), 50 deletions(-) diff --git a/src/lib/enhancers/github/githubEditComment.tsx b/src/lib/enhancers/github/githubEditComment.tsx index 1f52bcf..455e80a 100644 --- a/src/lib/enhancers/github/githubEditComment.tsx +++ b/src/lib/enhancers/github/githubEditComment.tsx @@ -16,17 +16,33 @@ export class GitHubEditCommentEnhancer implements CommentEnhancer', + unique_key, } } diff --git a/src/lib/enhancers/github/githubIssueAddComment.tsx b/src/lib/enhancers/github/githubIssueAddComment.tsx index 70f7fed..2b1e1bb 100644 --- a/src/lib/enhancers/github/githubIssueAddComment.tsx +++ b/src/lib/enhancers/github/githubIssueAddComment.tsx @@ -31,6 +31,12 @@ export class GitHubIssueAddCommentEnhancer implements CommentEnhancer { { "for": "id=:rc3: name=null className=prc-Textarea-TextArea-13q4j focus-visible overtype-input", "spot": { - "domain": "github.com", - "number": 56, - "slug": "diffplug/gitcasso", - "title": "what about the draft?", - "type": "GH_ISSUE_ADD_COMMENT", - "unique_key": "github.com:diffplug/gitcasso:56", + "type": "GH_EDIT_COMMENT", + "unique_key": "github.com:diffplug/gitcasso:56:edit-body", }, - "title": "what about the draft?", - "upperDecoration": - - - - # - 56 - - diffplug/gitcasso - - , + "title": "N/A", + "upperDecoration": + N/A + , }, { "for": "id=:ra7: name=null className=prc-Textarea-TextArea-13q4j overtype-input", @@ -259,29 +241,8 @@ describe('github', () => { expect(enhancements(document, window)).toMatchInlineSnapshot(` [ { - "for": "id=issue-3429313834-body name=pull_request[body] className=js-comment-field js-paste-markdown js-task-list-field js-quick-submit js-size-to-fit size-to-fit js-session-resumable CommentBox-input FormControl-textarea js-saved-reply-shortcut-comment-field focus-visible overtype-input", - "spot": { - "domain": "github.com", - "number": NaN, - "slug": "diffplug/gitcasso", - "title": "TODO_TITLE", - "type": "GH_EDIT_COMMENT", - "unique_key": "github.com:diffplug/gitcasso:NaN", - }, - "title": "TITLE_TODO", - "upperDecoration": - - diffplug/gitcasso - - - PR # - NaN - - , + "for": "id=issue-3429313834-body name=pull_request[body] className=js-comment-field js-paste-markdown js-task-list-field js-quick-submit js-size-to-fit size-to-fit js-session-resumable CommentBox-input FormControl-textarea js-saved-reply-shortcut-comment-field focus-visible", + "spot": "NO_SPOT", }, { "for": "id=new_comment_field name=comment[body] className=js-comment-field js-paste-markdown js-task-list-field js-quick-submit FormControl-textarea CommentBox-input js-size-to-fit size-to-fit js-session-resumable js-saved-reply-shortcut-comment-field overtype-input", From 89763c11ba666b305888e617e04b69d427e25f60 Mon Sep 17 00:00:00 2001 From: Ned Twigg Date: Thu, 18 Sep 2025 13:25:51 -0700 Subject: [PATCH 3/4] fixup edit so that it works for issues and for PRs. --- src/lib/enhancers/github/githubEditComment.tsx | 8 +++++--- tests/lib/enhancers/github.test.ts | 11 +++++++++-- 2 files changed, 14 insertions(+), 5 deletions(-) diff --git a/src/lib/enhancers/github/githubEditComment.tsx b/src/lib/enhancers/github/githubEditComment.tsx index 455e80a..477d412 100644 --- a/src/lib/enhancers/github/githubEditComment.tsx +++ b/src/lib/enhancers/github/githubEditComment.tsx @@ -23,9 +23,11 @@ export class GitHubEditCommentEnhancer implements CommentEnhancer { expect(enhancements(document, window)).toMatchInlineSnapshot(` [ { - "for": "id=issue-3429313834-body name=pull_request[body] className=js-comment-field js-paste-markdown js-task-list-field js-quick-submit js-size-to-fit size-to-fit js-session-resumable CommentBox-input FormControl-textarea js-saved-reply-shortcut-comment-field focus-visible", - "spot": "NO_SPOT", + "for": "id=issue-3429313834-body name=pull_request[body] className=js-comment-field js-paste-markdown js-task-list-field js-quick-submit js-size-to-fit size-to-fit js-session-resumable CommentBox-input FormControl-textarea js-saved-reply-shortcut-comment-field focus-visible overtype-input", + "spot": { + "type": "GH_EDIT_COMMENT", + "unique_key": "github.com:diffplug/gitcasso:58:edit-body", + }, + "title": "N/A", + "upperDecoration": + N/A + , }, { "for": "id=new_comment_field name=comment[body] className=js-comment-field js-paste-markdown js-task-list-field js-quick-submit FormControl-textarea CommentBox-input js-size-to-fit size-to-fit js-session-resumable js-saved-reply-shortcut-comment-field overtype-input", From 512d53d17693ef199223b7811a32a80647a5a69c Mon Sep 17 00:00:00 2001 From: Ned Twigg Date: Thu, 18 Sep 2025 13:26:20 -0700 Subject: [PATCH 4/4] biome --- src/lib/enhancers/github/githubEditComment.tsx | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/lib/enhancers/github/githubEditComment.tsx b/src/lib/enhancers/github/githubEditComment.tsx index 477d412..6a78d37 100644 --- a/src/lib/enhancers/github/githubEditComment.tsx +++ b/src/lib/enhancers/github/githubEditComment.tsx @@ -25,7 +25,8 @@ export class GitHubEditCommentEnhancer implements CommentEnhancer