一些检查失败了
automerge / automerge (push) Has been cancelled
react-native-android-build-apk / AssembleRelease (push) Has been cancelled
Build macOS M1 / Main (push) Has been cancelled
Joplin Continuous Integration / Main (macos-15-intel) (push) Has been cancelled
Joplin Continuous Integration / Main (ubuntu-22.04) (push) Has been cancelled
Joplin Continuous Integration / Main (ubuntu-22.04-arm) (push) Has been cancelled
Joplin Continuous Integration / Main (windows-11-vs2026-arm) (push) Has been cancelled
Joplin Continuous Integration / Main (windows-2025) (push) Has been cancelled
Joplin Continuous Integration / ServerDockerImage (ubuntu-22.04) (push) Has been cancelled
Joplin Continuous Integration / ServerDockerImage (ubuntu-22.04-arm) (push) Has been cancelled
Joplin UI tests / Main (ubuntu-22.04) (push) Has been cancelled
Joplin UI tests / Main (windows-2025) (push) Has been cancelled
Joplin UI tests / Mobile (push) Has been cancelled
Close stale issues / ProcessStaleIssues (push) Has been cancelled
661 行
56 KiB
JSON
661 行
56 KiB
JSON
{
|
||
"issue": {
|
||
"url": "https://api.github.com/repos/laurent22/joplin/issues/14651",
|
||
"repository_url": "https://api.github.com/repos/laurent22/joplin",
|
||
"labels_url": "https://api.github.com/repos/laurent22/joplin/issues/14651/labels{/name}",
|
||
"comments_url": "https://api.github.com/repos/laurent22/joplin/issues/14651/comments",
|
||
"events_url": "https://api.github.com/repos/laurent22/joplin/issues/14651/events",
|
||
"html_url": "https://github.com/laurent22/joplin/pull/14651",
|
||
"id": 4042718997,
|
||
"node_id": "PR_kwDOBLftOs7I7Sv0",
|
||
"number": 14651,
|
||
"title": "Desktop: Resolves #14621: Application crashes when deleting a notebook ",
|
||
"user": {
|
||
"login": "Pixels57",
|
||
"id": 111073325,
|
||
"node_id": "U_kgDOBp7YLQ",
|
||
"avatar_url": "https://avatars.githubusercontent.com/u/111073325?v=4",
|
||
"gravatar_id": "",
|
||
"url": "https://api.github.com/users/Pixels57",
|
||
"html_url": "https://github.com/Pixels57",
|
||
"followers_url": "https://api.github.com/users/Pixels57/followers",
|
||
"following_url": "https://api.github.com/users/Pixels57/following{/other_user}",
|
||
"gists_url": "https://api.github.com/users/Pixels57/gists{/gist_id}",
|
||
"starred_url": "https://api.github.com/users/Pixels57/starred{/owner}{/repo}",
|
||
"subscriptions_url": "https://api.github.com/users/Pixels57/subscriptions",
|
||
"organizations_url": "https://api.github.com/users/Pixels57/orgs",
|
||
"repos_url": "https://api.github.com/users/Pixels57/repos",
|
||
"events_url": "https://api.github.com/users/Pixels57/events{/privacy}",
|
||
"received_events_url": "https://api.github.com/users/Pixels57/received_events",
|
||
"type": "User",
|
||
"user_view_type": "public",
|
||
"site_admin": false
|
||
},
|
||
"labels": [
|
||
{
|
||
"id": 519700741,
|
||
"node_id": "MDU6TGFiZWw1MTk3MDA3NDE=",
|
||
"url": "https://api.github.com/repos/laurent22/joplin/labels/bug",
|
||
"name": "bug",
|
||
"color": "cf353b",
|
||
"default": true,
|
||
"description": "It's a bug"
|
||
},
|
||
{
|
||
"id": 1080341216,
|
||
"node_id": "MDU6TGFiZWwxMDgwMzQxMjE2",
|
||
"url": "https://api.github.com/repos/laurent22/joplin/labels/desktop",
|
||
"name": "desktop",
|
||
"color": "ffc274",
|
||
"default": false,
|
||
"description": "All desktop platforms"
|
||
}
|
||
],
|
||
"state": "closed",
|
||
"locked": false,
|
||
"assignees": [],
|
||
"milestone": null,
|
||
"comments": 9,
|
||
"created_at": "2026-03-09T00:59:04Z",
|
||
"updated_at": "2026-03-16T21:56:45Z",
|
||
"closed_at": "2026-03-16T20:49:41Z",
|
||
"assignee": null,
|
||
"author_association": "CONTRIBUTOR",
|
||
"active_lock_reason": null,
|
||
"draft": false,
|
||
"pull_request": {
|
||
"url": "https://api.github.com/repos/laurent22/joplin/pulls/14651",
|
||
"html_url": "https://github.com/laurent22/joplin/pull/14651",
|
||
"diff_url": "https://github.com/laurent22/joplin/pull/14651.diff",
|
||
"patch_url": "https://github.com/laurent22/joplin/pull/14651.patch",
|
||
"merged_at": "2026-03-16T20:49:41Z"
|
||
},
|
||
"body": "<!--\r\n\r\nBefore contributing, please read the contribution guidelines: https://github.com/laurent22/joplin/blob/dev/readme/dev/index.md\r\n\r\nIf this is a Google Summer of Code pull request, please read the [GSoC pull request guidelines](https://github.com/joplin/gsoc/blob/master/pull_request_guidelines.md).\r\n\r\n---\r\n\r\n**Pull request title**: Please prefix the title with the platform you are targetting.\r\n\r\nHere are some examples of good titles:\r\n\r\n- Desktop: Resolves #123: Added new setting to change font\r\n- Mobile, Desktop: Fixes #456: Fixed config screen error\r\n- All: Resolves #777: Made synchronisation faster\r\n\r\nAnd here's an explanation of the title format:\r\n\r\n- \"Desktop\" for the Windows/macOS/Linux app (Electron app)\r\n- \"Mobile\" for the mobile app (or \"Android\" / \"iOS\" if the pull request only applies to one of the mobile platforms)\r\n- \"CLI\" for the CLI app\r\n\r\nIf it's two platforms, separate them with commas - \"Desktop, Mobile\" or if it's for all platforms, prefix with \"All\".\r\n\r\nIf it's not related to any platform (such as a translation, change to the documentation, etc.), simply don't add a platform.\r\n\r\nThen please append the issue that you've addressed or fixed. Use \"Resolves #123\" for new features or improvements and \"Fixes #123\" for bug fixes.\r\n\r\n-->\r\n\r\n## Problem\r\nWhen deleting a notebook that contains a large number of notes (Approx 600+), the app crashes with **\"Maximum update depth exceeded\"**. The error occurs due to bulk removal of note list items.\r\n## Solution\r\nThe crash was caused by the note list item hook storing the created DOM element in React state and updating it inside `useEffect`. When many items were removed in one go, each item’s effect ran in the same commit and called `setState`, which causeddmore updates and led to nested updates beyond React’s limit.\r\n\r\n**The fix** is to stop storing the item element in state. The hook now keeps the created DOM node in a **ref** instead. Updating a ref does not cause a re-render, so when many items run their effect during a bulk delete, there are no nested state updates and the crash is avoided.\r\n\r\nChanges:\r\n- useItemElement: Use a ref instead of state for the item DOM element so the effect doesn’t trigger re-renders, and return the ref.\r\n- useItemEventHandlers: Accept an element or a ref; in the effect, read the element from the ref when needed and attach/remove listeners as before.\r\n\r\n## Test Plan\r\n- **Unit tests:** Added tests in `useItemElement.test.ts` and `useItemEventHandlers.test.ts` (6 tests) that cover: hook returns a ref; element is assigned to ref and cleaned up; null rootElement; ref accepted by event handlers; null ref handling; listener cleanup.\r\n- **Run unit tests:** From repo root, `cd packages/app-desktop` then run:\r\n - `yarn test --testPathPattern=\"NoteListItem/utils/useItemElement|NoteListItem/utils/useItemEventHandlers\"` \r\n or, if the script is present, `yarn test:note-list-item`.\r\n- **Manual check:** Delete a notebook that contains many notes (e.g. 100+); the app should not crash and the list and selection should update correctly.\r\n\r\n## AI Disclosure\r\n- Used AI to help me navigate through codebase\r\n- Used AI to refactor the code and optimize\r\n\r\n\r\nFixes #14621",
|
||
"closed_by": {
|
||
"login": "laurent22",
|
||
"id": 1285584,
|
||
"node_id": "MDQ6VXNlcjEyODU1ODQ=",
|
||
"avatar_url": "https://avatars.githubusercontent.com/u/1285584?v=4",
|
||
"gravatar_id": "",
|
||
"url": "https://api.github.com/users/laurent22",
|
||
"html_url": "https://github.com/laurent22",
|
||
"followers_url": "https://api.github.com/users/laurent22/followers",
|
||
"following_url": "https://api.github.com/users/laurent22/following{/other_user}",
|
||
"gists_url": "https://api.github.com/users/laurent22/gists{/gist_id}",
|
||
"starred_url": "https://api.github.com/users/laurent22/starred{/owner}{/repo}",
|
||
"subscriptions_url": "https://api.github.com/users/laurent22/subscriptions",
|
||
"organizations_url": "https://api.github.com/users/laurent22/orgs",
|
||
"repos_url": "https://api.github.com/users/laurent22/repos",
|
||
"events_url": "https://api.github.com/users/laurent22/events{/privacy}",
|
||
"received_events_url": "https://api.github.com/users/laurent22/received_events",
|
||
"type": "User",
|
||
"user_view_type": "public",
|
||
"site_admin": false
|
||
},
|
||
"reactions": {
|
||
"url": "https://api.github.com/repos/laurent22/joplin/issues/14651/reactions",
|
||
"total_count": 0,
|
||
"+1": 0,
|
||
"-1": 0,
|
||
"laugh": 0,
|
||
"hooray": 0,
|
||
"confused": 0,
|
||
"heart": 0,
|
||
"rocket": 0,
|
||
"eyes": 0
|
||
},
|
||
"timeline_url": "https://api.github.com/repos/laurent22/joplin/issues/14651/timeline",
|
||
"performed_via_github_app": null,
|
||
"state_reason": null
|
||
},
|
||
"comments": [
|
||
{
|
||
"url": "https://api.github.com/repos/laurent22/joplin/issues/comments/4020461516",
|
||
"html_url": "https://github.com/laurent22/joplin/pull/14651#issuecomment-4020461516",
|
||
"issue_url": "https://api.github.com/repos/laurent22/joplin/issues/14651",
|
||
"id": 4020461516,
|
||
"node_id": "IC_kwDOBLftOs7vo1_M",
|
||
"user": {
|
||
"login": "github-actions[bot]",
|
||
"id": 41898282,
|
||
"node_id": "MDM6Qm90NDE4OTgyODI=",
|
||
"avatar_url": "https://avatars.githubusercontent.com/in/15368?v=4",
|
||
"gravatar_id": "",
|
||
"url": "https://api.github.com/users/github-actions%5Bbot%5D",
|
||
"html_url": "https://github.com/apps/github-actions",
|
||
"followers_url": "https://api.github.com/users/github-actions%5Bbot%5D/followers",
|
||
"following_url": "https://api.github.com/users/github-actions%5Bbot%5D/following{/other_user}",
|
||
"gists_url": "https://api.github.com/users/github-actions%5Bbot%5D/gists{/gist_id}",
|
||
"starred_url": "https://api.github.com/users/github-actions%5Bbot%5D/starred{/owner}{/repo}",
|
||
"subscriptions_url": "https://api.github.com/users/github-actions%5Bbot%5D/subscriptions",
|
||
"organizations_url": "https://api.github.com/users/github-actions%5Bbot%5D/orgs",
|
||
"repos_url": "https://api.github.com/users/github-actions%5Bbot%5D/repos",
|
||
"events_url": "https://api.github.com/users/github-actions%5Bbot%5D/events{/privacy}",
|
||
"received_events_url": "https://api.github.com/users/github-actions%5Bbot%5D/received_events",
|
||
"type": "Bot",
|
||
"user_view_type": "public",
|
||
"site_admin": false
|
||
},
|
||
"created_at": "2026-03-09T00:59:15Z",
|
||
"updated_at": "2026-03-16T21:56:46Z",
|
||
"body": "****CLA Assistant Lite bot**** All contributors have signed the CLA ✍️ ✅",
|
||
"author_association": "CONTRIBUTOR",
|
||
"reactions": {
|
||
"url": "https://api.github.com/repos/laurent22/joplin/issues/comments/4020461516/reactions",
|
||
"total_count": 0,
|
||
"+1": 0,
|
||
"-1": 0,
|
||
"laugh": 0,
|
||
"hooray": 0,
|
||
"confused": 0,
|
||
"heart": 0,
|
||
"rocket": 0,
|
||
"eyes": 0
|
||
},
|
||
"performed_via_github_app": {
|
||
"id": 15368,
|
||
"client_id": "Iv1.05c79e9ad1f6bdfa",
|
||
"slug": "github-actions",
|
||
"node_id": "MDM6QXBwMTUzNjg=",
|
||
"owner": {
|
||
"login": "github",
|
||
"id": 9919,
|
||
"node_id": "MDEyOk9yZ2FuaXphdGlvbjk5MTk=",
|
||
"avatar_url": "https://avatars.githubusercontent.com/u/9919?v=4",
|
||
"gravatar_id": "",
|
||
"url": "https://api.github.com/users/github",
|
||
"html_url": "https://github.com/github",
|
||
"followers_url": "https://api.github.com/users/github/followers",
|
||
"following_url": "https://api.github.com/users/github/following{/other_user}",
|
||
"gists_url": "https://api.github.com/users/github/gists{/gist_id}",
|
||
"starred_url": "https://api.github.com/users/github/starred{/owner}{/repo}",
|
||
"subscriptions_url": "https://api.github.com/users/github/subscriptions",
|
||
"organizations_url": "https://api.github.com/users/github/orgs",
|
||
"repos_url": "https://api.github.com/users/github/repos",
|
||
"events_url": "https://api.github.com/users/github/events{/privacy}",
|
||
"received_events_url": "https://api.github.com/users/github/received_events",
|
||
"type": "Organization",
|
||
"user_view_type": "public",
|
||
"site_admin": false
|
||
},
|
||
"name": "GitHub Actions",
|
||
"description": "Automate your workflow from idea to production",
|
||
"external_url": "https://help.github.com/en/actions",
|
||
"html_url": "https://github.com/apps/github-actions",
|
||
"created_at": "2018-07-30T09:30:17Z",
|
||
"updated_at": "2026-05-05T14:51:38Z",
|
||
"permissions": {
|
||
"actions": "write",
|
||
"administration": "read",
|
||
"artifact_metadata": "write",
|
||
"attestations": "write",
|
||
"checks": "write",
|
||
"code_quality": "write",
|
||
"contents": "write",
|
||
"copilot_requests": "write",
|
||
"deployments": "write",
|
||
"discussions": "write",
|
||
"issues": "write",
|
||
"merge_queues": "write",
|
||
"metadata": "read",
|
||
"models": "read",
|
||
"packages": "write",
|
||
"pages": "write",
|
||
"pull_requests": "write",
|
||
"repository_hooks": "write",
|
||
"repository_projects": "write",
|
||
"security_events": "write",
|
||
"statuses": "write",
|
||
"vulnerability_alerts": "read"
|
||
},
|
||
"events": [
|
||
"branch_protection_rule",
|
||
"check_run",
|
||
"check_suite",
|
||
"create",
|
||
"delete",
|
||
"deployment",
|
||
"deployment_status",
|
||
"discussion",
|
||
"discussion_comment",
|
||
"fork",
|
||
"gollum",
|
||
"issues",
|
||
"issue_comment",
|
||
"label",
|
||
"merge_group",
|
||
"milestone",
|
||
"page_build",
|
||
"public",
|
||
"pull_request",
|
||
"pull_request_review",
|
||
"pull_request_review_comment",
|
||
"push",
|
||
"registry_package",
|
||
"release",
|
||
"repository",
|
||
"repository_dispatch",
|
||
"status",
|
||
"watch",
|
||
"workflow_dispatch",
|
||
"workflow_run"
|
||
]
|
||
}
|
||
},
|
||
{
|
||
"url": "https://api.github.com/repos/laurent22/joplin/issues/comments/4020462232",
|
||
"html_url": "https://github.com/laurent22/joplin/pull/14651#issuecomment-4020462232",
|
||
"issue_url": "https://api.github.com/repos/laurent22/joplin/issues/14651",
|
||
"id": 4020462232,
|
||
"node_id": "IC_kwDOBLftOs7vo2KY",
|
||
"user": {
|
||
"login": "coderabbitai[bot]",
|
||
"id": 136622811,
|
||
"node_id": "BOT_kgDOCCSy2w",
|
||
"avatar_url": "https://avatars.githubusercontent.com/in/347564?v=4",
|
||
"gravatar_id": "",
|
||
"url": "https://api.github.com/users/coderabbitai%5Bbot%5D",
|
||
"html_url": "https://github.com/apps/coderabbitai",
|
||
"followers_url": "https://api.github.com/users/coderabbitai%5Bbot%5D/followers",
|
||
"following_url": "https://api.github.com/users/coderabbitai%5Bbot%5D/following{/other_user}",
|
||
"gists_url": "https://api.github.com/users/coderabbitai%5Bbot%5D/gists{/gist_id}",
|
||
"starred_url": "https://api.github.com/users/coderabbitai%5Bbot%5D/starred{/owner}{/repo}",
|
||
"subscriptions_url": "https://api.github.com/users/coderabbitai%5Bbot%5D/subscriptions",
|
||
"organizations_url": "https://api.github.com/users/coderabbitai%5Bbot%5D/orgs",
|
||
"repos_url": "https://api.github.com/users/coderabbitai%5Bbot%5D/repos",
|
||
"events_url": "https://api.github.com/users/coderabbitai%5Bbot%5D/events{/privacy}",
|
||
"received_events_url": "https://api.github.com/users/coderabbitai%5Bbot%5D/received_events",
|
||
"type": "Bot",
|
||
"user_view_type": "public",
|
||
"site_admin": false
|
||
},
|
||
"created_at": "2026-03-09T00:59:38Z",
|
||
"updated_at": "2026-03-16T00:17:21Z",
|
||
"body": "<!-- This is an auto-generated comment: summarize by coderabbit.ai -->\n<!-- This is an auto-generated comment: review paused by coderabbit.ai -->\n\n> [!NOTE]\n> ## Reviews paused\n> \n> It looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the `reviews.auto_review.auto_pause_after_reviewed_commits` setting.\n> \n> Use the following commands to manage reviews:\n> - `@coderabbitai resume` to resume automatic reviews.\n> - `@coderabbitai review` to trigger a single review.\n> \n> Use the checkboxes below for quick actions:\n> - [ ] <!-- {\"checkboxId\": \"7f6cc2e2-2e4e-497a-8c31-c9e4573e93d1\"} --> ▶️ Resume reviews\n> - [ ] <!-- {\"checkboxId\": \"e9bb8d72-00e8-4f67-9cb2-caf3b22574fe\"} --> 🔍 Trigger review\n\n<!-- end of auto-generated comment: review paused by coderabbit.ai -->\n<!-- walkthrough_start -->\n\n<details>\n<summary>📝 Walkthrough</summary>\n\n## Walkthrough\n\nRefactored `useItemElement` hook to replace state-based element management with ref-based approach, consolidating event listener attachment into a single effect to resolve React 19 compatibility issues. Moved event handler logic from standalone hook into hook parameters, deleted `useItemEventHandlers`, and added comprehensive test coverage.\n\n## Changes\n\n|Cohort / File(s)|Summary|\n|---|---|\n|**useItemElement Hook Refactor** <br> `packages/app-desktop/gui/NoteListItem/utils/useItemElement.ts`, `packages/app-desktop/gui/NoteListItem/utils/useItemElement.test.ts`|Replaced `useState` with `useRef` for element storage, changed return type from `HTMLDivElement` to `React.MutableRefObject<HTMLDivElement \\| null>`, and made `rootElement` parameter nullable. Introduced `addItemEventListeners` helper to attach DOM event listeners. Consolidated focus-visible handling and cleanup logic into single effect. Added unit tests covering ref creation, element lifecycle, and null root handling.|\n|**Event Handler Refactoring** <br> `packages/app-desktop/gui/NoteListItem/utils/useItemEventHandlers.ts`, `packages/app-desktop/gui/NoteListItem/NoteListItem.tsx`|Removed entire `useItemEventHandlers` hook implementation and replaced with `itemEventHandlers` object passed to `useItemElement`. Updated imports to use new `ItemEventHandlers` type instead of hook utility.|\n|**Type Definitions** <br> `packages/app-desktop/gui/NoteListItem/utils/types.ts`|Added new `ItemEventHandlers` type with `onInputChange` and `onClick \\| null` properties.|\n|**Ignore File Updates** <br> `.eslintignore`, `.gitignore`|Added test file entries and removed previous utility file from ignore lists.|\n\n</details>\n\n<!-- walkthrough_end -->\n\n<!-- pre_merge_checks_walkthrough_start -->\n\n<details>\n<summary>🚥 Pre-merge checks | ✅ 6</summary>\n\n<details>\n<summary>✅ Passed checks (6 passed)</summary>\n\n| Check name | Status | Explanation |\n| :-----------------------------------: | :------- | :--------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- |\n| Title check | ✅ Passed | The pull request title clearly and accurately summarizes the main change: fixing an application crash when deleting a notebook by refactoring the useItemElement hook. |\n| Description check | ✅ Passed | The description is comprehensive and directly related to the changeset, explaining the problem, solution, changes made, and test plan for fixing the React 19 update depth issue. |\n| Linked Issues check | ✅ Passed | The pull request successfully addresses all key objectives from issue `#14621`: eliminating state updates in effects by using refs, ensuring proper event listener attachment/removal, handling nullable elements, and adding unit tests for the refactored hooks. |\n| Out of Scope Changes check | ✅ Passed | All code changes are directly scoped to the objectives of issue `#14621`. The PR modifies useItemElement, removes useItemEventHandlers, updates NoteListItem and types, and adds tests—all necessary for the crash fix. |\n| Docstring Coverage | ✅ Passed | No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. |\n| Pr Description Must Follow Guidelines | ✅ Passed | The pull request description fully adheres to guidelines with properly formatted title, clear problem description, solution explanation, comprehensive test plans, and appropriate disclosures meeting all GSoC requirements. |\n\n</details>\n\n<sub>✏️ Tip: You can configure your own custom pre-merge checks in the settings.</sub>\n\n</details>\n\n<!-- pre_merge_checks_walkthrough_end -->\n\n<!-- finishing_touch_checkbox_start -->\n\n<details>\n<summary>✨ Finishing Touches</summary>\n\n<details>\n<summary>🧪 Generate unit tests (beta)</summary>\n\n- [ ] <!-- {\"checkboxId\": \"f47ac10b-58cc-4372-a567-0e02b2c3d479\", \"radioGroupId\": \"utg-output-choice-group-unknown_comment_id\"} --> Create PR with unit tests\n- [ ] <!-- {\"checkboxId\": \"07f1e7d6-8a8e-4e23-9900-8731c2c87f58\", \"radioGroupId\": \"utg-output-choice-group-unknown_comment_id\"} --> Post copyable unit tests in a comment\n\n</details>\n\n</details>\n\n<!-- finishing_touch_checkbox_end -->\n\n<!-- pr_review_plan_action_start -->\n\n<details>\n<summary>📝 Coding Plan</summary>\n\n- [ ] <!-- {\"checkboxId\": \"6ad8a4e1-0b3a-4ea2-9b5b-d82c1f47d1f2\"} --> Generate coding plan for human review comments\n\n</details>\n\n<!-- pr_review_plan_action_end -->\n\n<!-- tips_start -->\n\n---\n\nThanks for using [CodeRabbit](https://coderabbit.ai?utm_source=oss&utm_medium=github&utm_campaign=laurent22/joplin&utm_content=14651)! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.\n\n<details>\n<summary>❤️ Share</summary>\n\n- [X](https://twitter.com/intent/tweet?text=I%20just%20used%20%40coderabbitai%20for%20my%20code%20review%2C%20and%20it%27s%20fantastic%21%20It%27s%20free%20for%20OSS%20and%20offers%20a%20free%20trial%20for%20the%20proprietary%20code.%20Check%20it%20out%3A&url=https%3A//coderabbit.ai)\n- [Mastodon](https://mastodon.social/share?text=I%20just%20used%20%40coderabbitai%20for%20my%20code%20review%2C%20and%20it%27s%20fantastic%21%20It%27s%20free%20for%20OSS%20and%20offers%20a%20free%20trial%20for%20the%20proprietary%20code.%20Check%20it%20out%3A%20https%3A%2F%2Fcoderabbit.ai)\n- [Reddit](https://www.reddit.com/submit?title=Great%20tool%20for%20code%20review%20-%20CodeRabbit&text=I%20just%20used%20CodeRabbit%20for%20my%20code%20review%2C%20and%20it%27s%20fantastic%21%20It%27s%20free%20for%20OSS%20and%20offers%20a%20free%20trial%20for%20proprietary%20code.%20Check%20it%20out%3A%20https%3A//coderabbit.ai)\n- [LinkedIn](https://www.linkedin.com/sharing/share-offsite/?url=https%3A%2F%2Fcoderabbit.ai&mini=true&title=Great%20tool%20for%20code%20review%20-%20CodeRabbit&summary=I%20just%20used%20CodeRabbit%20for%20my%20code%20review%2C%20and%20it%27s%20fantastic%21%20It%27s%20free%20for%20OSS%20and%20offers%20a%20free%20trial%20for%20proprietary%20code)\n\n</details>\n\n<sub>Comment `@coderabbitai help` to get the list of available commands and usage tips.</sub>\n\n<!-- tips_end -->\n\n<!-- internal state start -->\n\n\n<!-- DwQgtGAEAqAWCWBnSTIEMB26CuAXA9mAOYCmGJATmriQCaQDG+Ats2bgFyQAOFk+AIwBWJBrngA3EsgEBPRvlqU0AgfFwA6NPEgQAfACgjoCEYDEZyAAUASpETZWaCrKPR1AGxJcAItIDWBNxcNtL4HlLIZgCMACwAbABM0VwAgtzcHvAM1PD4WAxUiLDSkADuJVhKXuIYROiQGPg0Avj4/pAAFLaQMQkArNGQctbwAB4kHoj9AOwANJC4zqSckEoSAJQakABi46V9SdEaRlYUgl7MvpMktfVojc0kre3l6rCQzJjyTTTInQA/eIABmBAGoNow0NhEHQGqE0GJIAAiACyaDG8GYjkg2G4tGoJDWJG4uA+JDGDBIdDoyMYRVg2zgRMKaGK5TZUJhcJGD1gbQ6nW5AEkaMwAKKXdiQxAECjwOqLEr0kiE+g+ADyqMgNzYGFwKCwCKRssJ6Aw9ByHiyithuAAyksaIbEPAlA1ueKAGZe0S4ADc5Uqn2+KDFyDKlCJFBIzHwUno1H4GCpOsR5J9fqh1rhdsdZoVSqJiDQbAUrHUC14imwDAV9XIsrheIJf2GJFk+XoxtwgEwCZBZZjqE4Ge3hPB5DBcR34bj2OX1oth2OQTXa3XsQ3zs2Yej+alz9Rbh4xr32fDoRCuogYRdkokb/VrfClX5c2GQGNgGMWyiIbYAMKwJgpCIBwBhQCKYqSrG7BcNyDSni6NBoPQ+BnqazpevgfD3suzCrlqOpSvqgZsteWCPgaBCfiQXoaAw2AUD+Bq7owXiYLi3CBjGuBMVgeGnickGwqKsbilI+oABK7l4FBgbRXqInKcI0dWEhusWSwCF4kDARacn8MIfqIIGLZqriJakOeOqSaxuBLAwsB6gaqC0PkRIKq67pQeJJF9sgdG+kiMaoXeyr6bQcn/hBkAAHJPAAMkguBiQRbG4LI3ClOZND0DRp5eEieHkGUemyZQixZSQAD03LCTA0i4ApACqt7UU1yCoUo9DYbhEUCsgTBSPKipIbx/E1eR8A3i5NUMBxGB4gsS3Wp+bS4DBLnlQZ9YLEhiJUqSmCpiMkXRSt2BrUh517ea9BZE25B8AtqpLdwI7QE1PAeJgITYFgANHn8zWQH18VJSlaW4uIUz8HwCF4epmn5d9iCFPApKBl8S1oB4jAlAwHQjNUtyLg8vzPAKbxkiGGA/E8/zRKCEKQIAKARLmgGT2PyV30G+rLsmxsJFeI+Q1Y9BrFPgfNca2LI4TGYgeLII6pMKq5IAt+AODGaQa2UnLcvlF4YGgGlEDuFqKcpOEKEoI4WJAGrGWIkilF65wETa+70EgDjSEYOxeygV7YEShzJF06SZNkuT5PSbIlBGwak3cDSUy8/gbOBUCSliCpmphRK5aUXmaTqmZiMg4PZRQYDqCua47VFd4Xub+BulC6NhWNJDfmQSjyWsTGLgIV0dGnk4xVAM4xjwlAN2KhHak0SjIIWp4bxgTaofwZ49tuzpqTGdmV8FuBgLQ8qSbRA+/vJDXijvTEPmfktkH+6Dz0wzF+ir6AHLpjhGxGMcYExBjIPhYisF9RdR/qFPK+1RCILoAsO2AM4wAzyiPUa9QqDcC7pLAmIFpANXRPuPSg11rj1lIsU2V1fo6WjBtGBLkurWxolRWihRVRiywJ0Mgutox8Mcs5TcFQoHcKcqQxAWxYqpFoPQIGHVZRDXjJQRcSEBAkGAhpHCl01q3TqAsNiH9nrsTeniBqijr7SGQDGIgMYrzT0NLgc4tBaw8nkIfaIABOec8oxCVSCn6Oq+Ii6xkwOIBgMh5BgPjIubg1AaAUB3kqJMShsrWwTliNgtB4CRIdE6TyO8K6hOro7SwAEWDsPsI4L4LgjBQCsPsKYswFAYC9PACg+T7AzVvIqPCAFEqpAaq0iY7SZjoAyFkOENSlA2BUGoA0vo6ACERP4cCkBdCQFsXCRuzAADSHYbK+QlHZGSBkv5fgoADFOUDOLcNQErVBiZrbmK/s4ERYAUnANoCcHZUB0TunOM0Lam5VpMN0oWc5EKnygNjBo95uIMDkCpFeZw8h3LYGYd3VyAlOoNUSoSF6+lSBcAmZMaY0zTy2xjPQfk7Qa5MXvBQbZuy4WXIqsPJoZVqyeIxQ0U0eLBAiCROhFu0Uuj5GFBgbgeAgKkPQRgACWQiaQg0g8bkqIkX3UeJADw+RSB8Drn1ZgyBm6HXsfDc+JlAxHncq+Zo6AGBHQNFvGqYKDRURqock5qsDBAssiQNK8KDR8tde65AeEw3cuucPMVWZdwLELOoQKVdqLAQNLwwkMblRUQWN0i028FXNRquPBy+QOGJiAU5UoeFzp/kDFaut4j9TzUWnicqGQhEugrmSVAFTNBBt2QlGgyVZTQ0NsgUuJt5yzh4NNRUJ46JuIvFy9gVyopfOtk0Q1xrKrcg3mKANgLdnQGqgpfZ9A41bp5dvGG8Apg1Uytlf8zVz1QC+mo+ColoLxp3cPEGEYoz9OYLHbpcJOj7ubnbLeGxkHgPJkiSR88AbovsViqpkAakVgNGwTFoF6lOCabFe09ZdJMHw1wZEoRwEl3/eJQDF0bZiAwUxiU/ki3jCqu+ukHMFQFJyG2JgO9whunjlgSVTa+BGqINkddIaw3cdouAvGXq6L0v3kuc1lUmX+FMdbGMv0cFvqJN0iYMUIJgEMAYEwUBB46ehAQYgn8qA4Ooy5LgvAjLivEJEYY8gmBD2WcObQuhbP2fAN+hAG8OE4Fc6QZ6FkvNwU/GgMqDhSPyBGCF5Qqhws6H0EYaLpgDAaGkDacQN4cLeAMMiRrBgnbqzcylnB2XGnyElTIuogcFFKNUt9bpukaLintMlJ8Ay6s+c2WgUCU0MhXwCEEGqRBsDwBquOkgk7UpijqrDRAdVOPho0CDDQQgZ6QHo8isGIdxuTdcrVvWS6ibzekIt7gy3ECBFnGtjbW3IZTv2xOF9m7pIPou4gBYbBMCLiPGUZ9+N91Gr63wHRKBnveMgA9hUI7SstY8KkqTMaLx4SUAtZwJOdMUm4DhHBdsFU6UU+wdQ8B+vAtuPyRMg3aBcASuQWKuqySKH6Teagr85Zqn5x5WKAA1ZwhS8U/lLHQGXgug1xQ1HFcURgvhCBwgAfVLogIwk3Si9dIHzyAYJog1TANEIw4pZRYlS4oERGkSBlSCvTrguqCmOAa01mzYAjAaAUzVpoesg/Ima5YVryXlAdYaVinTluOeQDlZAcPbOsemN54sMopsvc6n1PKaQHKoDJLewtrmX316/e4P9zb23dtpQO8+o7cL/JnaalD0dVe5u16Ww31b62W9A727GDvYOTs98u7FLPOfI91aQ7dvC03559Ur69/w72jt1++435vgOJ1QxB4d47oaAP3oTf+BfUAEokL66TnguL1U1Vp/TuEFPfoeenoGPkAApvv3CNkSKhEIDCLgOwiOOYPHkTknq4jROTqIH/tTpKl/hQAzqau/izvqGzv1jBheOnnImbgqBbuSmrjbnbmAIkE7i7l8J5u7rRJ7t7j6L7pAP7vAIHo1rHiHkYNXnvsPvXitn9uPqfjtuftPqDl3nPrApoOds1OBLwXHnssKG1knrmCni4GnpQabgYLYl1I0CXioosMNs+hZnbHhN3vIVQu0EyMqCBgoCNLnJAFJNTBNGkkYUhEmqhu8A0IxH/E+NWHXJlA1O4a8Lmm2A8AUhIGwuwKYr2iWmGPQkuMjD1OtM0EZmhIDBgFgk+K9M4A4nRAxExCxPqkoGIg2gWj3rFBER0E2kYVCpkT6v5EFoaqqBpH3PRIEeUZyFCp9J1CGrRA/PUfqspHdiwJAAAAIgz1hgBZACBUAuCabKQLCujYimZEhZDBSyCvSQBaqor5G4DZEHF4ySbOjcJ/JOTbTIHKjer6qFHvTth6J5AUAjiP4XhM4f6YE4K/5U78KgbzxxgFJQZoR8DdR0CwGqGpAIH/7VqpEoGU7wnpIYFjB05YFwiM64EMCl7iDiAZ5P6/F0A1TfGKb/EolAlEggnwBgnbCdBxQl4gxgwWHPjOoGgUlEh2w0lejyChjEm0Ckk4l4kEH/gbBwGcFw6+h0J7C6SpBmwqwABelAZBjYz+VuXAYI8QMw9uwIMezS5Wgh++n2R+Y+AOreUhzAM+sh1+fk8hZ2YE+pLW6hieHmWhOWuhsizS12JIv0Qq3I+YzoiOtM3IoQXoXAhy4ajwWWKkiY3ha6nQkZPevR0oyEqoaEGEJSDULUESbYZJuJFEEuL23qUZ6evUIcUk0AqIiUPgkgUZNElZ1ZtZEgUZAAPo0IwmREoiesxrfkBsgMklQGwKkg6hBvTsgHehDnfg1HKqkgqSvHTO9t5iqHmuMUkbQEYc3NwgUkrLgCrKYi4jeCeqkUmfaSmaRIaGzuccqYFG0SEZQASTWoaM9I2YlDTLzAaLwG8UfCXLmWQrFBJJuGoBaIuCQVwKTJbG2CQGfIjngqkRTCXt1JObgLtu5sgCUB4HXEGNkB8LBTlIePKngPNITP4K0GMK+hSDmpQbZOwNDsMHgAQAUOqh0NBbRacXjIgBeEsW6MRs3AtNkP4DVLQDpPxUTKkYfLJqZGphosgDucmkooBfqKhRYlaHDAnHhFRA1DsPgIxIgGABpK6HisYvUGJpxVkPLNbrwNBXkDCMAVgA8LCIOWaMOoGJGnXnMn7BgDaOAV6KkvEQUYgpOPqnOluHhCWGWMOg1GqlYnOGQPpFSOwn+qUA8E8d2l6ADG7AnJ4eQPQLyEoshcpV/AVEilIGsLIGbEOKpXyW2nCJ8vJIGIUcPKeS5KUUEaxMZiVdUQ+G0Z7FMXhM3IWGFarpYpgN2skmSA1H4NkkoCmHycxGgPIKXFwK6HULpMOtGcSNNcgFlRtOGqmmKBRsqStE8FJNAR4MdTQMKLQEWkamUOsZlF4EWjpTCHLkgPAMwiqmqgJSqj4DLMwl9UTKcZGSxl/EQfYCSACcWKIF2HamIPIlAAAEK6LmxvF4w6jomYCuj5DLV4gYmgzXEIArpKJs75Bo1rgLGsX4ySVKjnDYBEAfDA19nSpsRCKvxDRdpzhDjMQ4RdRtotWlYwlwnU53FEgUnoFngCm2r5kimPnenC7c5i5mx8ToZ/nW4AAGNhLkatW4atRpwhppYh5pk+7eMhV+Km9pzUatO+SNfU3gkAo6OyGtchLknQJZ/kXAr5zZe1jwl11usoeCF1JAp1zAHgy17id02Eulr1hlXgXALwi091sgsdPpykGgAE9o9oZws4D57OdFhyh1dtBdn1zFIQqoYgGgqIMssIiluA26ckwAntdZ/kegP1f1XgANWyKd5dldnoINFADdVZNZTd8hLdYMt1XAaUOwt1kIAAvHoG4YPV7f5GrQPnsr5ZQByo7ZrewK7bte7QvU2cPdtO2VCoHVdWHQHT7UHWdRfRHc9YgNHe9cnfHW9IncnT2GnRnVnaEbnftbGAXdOPAEdcmB3aXanT3dXX3QPYfS2c3a3e/iQKA13ZoBAyQDXXXZQNA0PbAyPTdfgGUBPWKFPfg3/RckzX+IQ72VOf2bPfPR/aiHgCoF4GGS7P5lg0vbYSfYwnoCvbFFYMKViHjbOqrTvjeigGOVgVwGrchRg/JNrb1QRAAOQaCvpXqKOr2hCZCIgHISMGiSpO0kCBkkDa3BkfAGNhm8OP4l546UDzkYVYVi38JdBNAYBgACk5yr1iPpUphONq1IU35KUpRoWCL72vne11UKQyMPrF0CVgPd1V1oNQON04MuSj35C/UINIP0MJPoM8rsNH3sB6AbAmMIBOS0TK3pIPBq0ADeI170XAnQtDBxnc9AAAvmrSOObkNJQdblqTqQ7n4vQeIIwXCPliwezmwX1KsFJDNLAPqfwQYHrR9ofqPobRPmfsDtIZfuDrXZDkoU6fHi6e5hZJ1qnj1nod6TdhAhpfgehidn3ehdTII/5FJhkvZFUfQKxU+BEyeYRaDJ0PWkTGRclZwpRYaGWnIvqpWoxRGO8IWA8AprfM1QkQcYUh6NXZmmABsrCA9LSaIHsV4NsKKDwKfDZYgCrMtTkGigctBD1XbAqGWlCwxQnFRHRWJgUvwnjAApCZaNRT8zRAy3gHRTy+xAJYakE89K/tCwiZItJqqsxRyAOecBpD1KcU8c2HOCMAkl0fUCLh+BE0S6xG6iSDgrveCqpsi/qCqnKmWkqn1jExqvqtlXCBIC0w6qDOtVkoPFiVgJaycS0d7bKn83a6QA6/4NCYTsToCYicqI464miXjViTgczriazrLQBeid/oKdLWm/Ep1dbkoEpFdD6pm1gWDBlb4zs7I4gCY/4YWcrUSGrWa5tKE4vQU1a/hOGh7W2ykyi0G7a5QVwBqBgDa4qpQWG0O/KwJU06626NrYIrcxZqyWnHQPIqoeiLeNKQaLKUSPKXjLIMqRQKqRQaQr08CPbrMPMxAKHos0PssyPqIU3uIRaZs1aS+1PswA6WMMoU1s6RoW6fQKczoec16bFJo36aUHJUiFWw+pZO9u+RnNYwE7s3fn5lmFES62izqnqmxMkleNUagL4axPmtsWyKxBQOtrcRus7ewA1Fc11duBaHjB5Mpsh9W8297X69a8G+Ox2daJCAqK6yJpODObo8IxZakQhFE6h+ZpMQRCo6beZtvLvBmWkaS1Xax1Qyh/2U+lkJlPqsO6O7gCGyIl8F5A1MyJ220aZe4rWE46pZJzR0+JbF5Ih2VMWmjU5cOZVImWxw+pCGxPuCSIFJiC7oqF57cH+KYnCeFESLBYuJxeC3gOqUSMZaHCS4khkWSLTfTZ2w8xG/AVG0gWTrG6gQCfGxLaW9gW/imzLbnd6QLiQBKRu3izKayXu0qSqQYF0yl70/0PbrEEM67kwaCtZSXj7pI5wXQNwcwFe4aXewfg+z9maes5Ia+9aao++g6T+7Hn+66Sc9od1meCQd6YYW5zV+qnxkSNJzp4WEs4tyIct2sxIW3hfp3pt2Qp+jAMqKVFd8SMWqUP22O6Ql0DGAAI4bYMroJniyeGc8ekIBc5JTtiUwaMJMMkDQ9/fDsd2QBcP8fbBEmhfpxKe2ryaKbfKfCKC0ns4AoC2RuIEIki3EjIni3o2JvgkXd4H4n1exSXrZToC85SO/F/c3fSoz2QC1NA/GeDvOwjvw99aAHI+d3Y8Kt48eCBitP+ja2Fi60LcmmrNPtG0bMfsbdKcOm8NOwtdbu7DtcKkHtdc9dlman9dgB6kGDO7DNu6jesHnwcFcE8HB4lZlaOY5JnguaED7dMGsDpZUBZaHftH5ZUCFaaARYlYOblhDi4CG5uiICG6kte50CG6mhltB/p9+K0DRACB+LrKxCxAMCxD9AADMiQMwsQMwwIfi0QAAHGgI3zomgMCM3/0AII343yQMCFSIkESGnzFjqNENECQI3/0AwMJX3wIMCF6H4gIEkDMF6P0H4jMAwI316EkHQF38kP0CfzMNECoJFoYOny3137EIkI334sCDMC3yQIkIkLEMCE31334kSD9BaAffYEL/wEBehd+P/AAQwH6AkBpkM/CAIwGSDxAl+0QL0H/zoi0AfQAgZ/vED34zB4gDALvtEGAHoDtSjfJ/nvxUBd87+xgWfl5nUDZ8NyefMbpGFoCG4nMiAqvDGENxsAKOJAQ3IC38C59i+BoIPtU1HTIgkAtgBGkaiJh0A8MLkKwDrDyjIguASkKYBjykFIANQI0eUINgwDqCwYHFbQTsmRDuRYk4dOoDUhGjvZZytjPGEY2MHVNWmcwKQfliWTJ8AA6ucBoBnB3Ae5EgMYM0Gwh3B5g6WHzDkE6V/AtgEIaYPCEogCktAGwADF+oMBHQeCRAEBFED+BjBtnMwUkLdCpCMAgQ9uiRXyF3JChFg4oWkOkAYxSQk4HIUTEqHhxEhyIH2HQGFBhxpAmQ4wciAAA6GAIYQMNwCjDxhYwyYcAE6EcD/Y4cPQCMMmETCJhFgSwJHBSB7JZkccezgyFKCytiQNQcmFfSzgjDhhSw84aMKdhlCSApwmOOqleaCxk4kCKoDcHTgUwngJw4YRgCdh+B0Y8oRofkFOHEtHhew/wnhB0S3VAkmyRYFQFTD7Cp4K6Y4dTFMZ0wGYbYToPEGZihU4s3cEgFsFOEAAqSAESIJGnDLOcYOhFkH3AAIcgCEVAHhAJEEjfEtAwAEmEyDSAP4i4hOJUIJARkV0EYEGg1aS/VQP0Bf7AhEgatfEV8N8QBJgIyAf2tkD8o6JXidsZwDLAtAcBThxwHZN6Avjo1RAE4dSgXDqBDDEg2wSAEY2GDUAnI9YIYY33NE1IUwZRTcD+CHi2ivhlncanOR1rb19QJTKMJqK+FQB5SsNDkiSEHjbUsAmxcQJkCJAhF/gatfIB3TVoLBExGADJv9WYopjIAatWUEnWMYLBbgDADQBsCGGxYHwmabMHDFzG3BzaLtKiMU2wplMTcB6K0D+TLHfdUATAWWOHSIAmo74rooqtmnfBEgNKlY4qt+ABinCNAM4ovBQF9gslyAW4ZkfRQIwLV2wekdQOnDoDWQcgsIR9DKMUbyjw6wSBGH+QPSLpKifoScIWMQDZQ6wXLbFKPDGhcwu4xcFLgOHgCUJsIHgIeAcLJj5ANAZI5UJZnKAyxfx4rakfIEE7hBSqJAd4BvS1HmiWoK1fBHRGU4oRVO748GGrT9bzsO4boWLj+SlzOhy47oMcRfFLEYAzROySujqyXBfMDQsmQBGIio7CoIabpUMW8zZKIAMAijMMdNWTA5ikxWYmqGmIzHt0sxawHpP/FkB2jkJqEnMdyAAhcsNkRMWtrTBFS6RTwUYFMB7DtiSVThRk74U7A1B1xcgtoWQE2GYCnDvBQmfBogFOGrDIAAAKVnA2gDif4ScKcIkD2j4gGgRvk5MsD2h4+CjXDL9GwBKBgJD4bmnwB0qBFH0AAbV9EKEwIjfaIAAF1OglgAYciAkDox3cYASMAIFYIcAaoNUaIIUFgC+SiAzQBgIkHgCxA0AXfL0LQAkDv8hAQgBTNwGBD+B5xEgbgEIFiDMABAYPeIEQB/4eAZg2ACQNEDB4zBEAYPc4Oe3u769H2J+d9ibW2ZOdUpZgRKOlNymQg0MjbO0HWJ3oNjteQ0LlgcjKTugqmnoTNB02+5Ehcp6ITENiAIilxNqtMCkFSBpC0BcpOoWKYaGEyrlD4V44JCbAvA4wfgTUXMLcEtGqVH0DlSjCIgfiMACWJAICV8Ock2BWEKkhCB1yslIBopocAOOl0GrKgDGuov0NrS9a/hZq38KgNBKwAGMzpfowMUMLVrcylODQsYRgDcGQBEpbteQqQyLpX1g651K+ldTfoY8QGzFeBpmO+pj18GGUjYP6C5nczbJqcV4UcMzgChOZGAbUZwWLaYxdIatLXNAHFCG4fA4oRKOKCtna1lIriCngUjvHWiSg9AAhG6AASdAWO4MMuh8EphUSaJOOdMNJPdm4AymvYk1MUXvhDxtqMPZUO+3kk7JLOatQcQym2zzsKgOFDLiq1KBq1KYksyUVCAKDUVPEcFQdEqx0r2IhhsQc0eKHDnp4tw9MmanWAtzQh9xDE8cReFuRTiMA/Qc0ZZ3WpIycxp02lvIRCbyFGxuc6OfKD7G7pmgJQPgJnKGH+S05OIjDkYT3E5B3Qx04iS2JjmVQ4w88d8SblJl3jRA1PXEnMM8j5okwFErMK0DJCGzjZjIqahGKEnFw+RnQIuSdTOra1DiGc71lnKeCSjTR5oxkfaFuDHjCQv8msR+3DQQKvh3g4MHhH/k0AS5H4jLIQiijyBOgc8j4FSx7TZJnhV9eBESB0SLgV27yfKPcVAWjN8gNAKbOJwslEBEM+qPCNRjpzkBWF84JHJT2wQ8hkuSseCfRNNBeBwkrYJJOcG4B0Un5SIEhUFznD9y0U9YbGUMOcloL5AooI8TAAXkmpsc2lX8ZVD8CHDARqC4MJ2GwD/jnQDwH8X+JRGUwwISEnUeHMpjYjLCpivgG7PGr1ojCjlSGjmMtnWzbZ9sx2a6n4SQLN5UNUWHCEcWVQSCCwY+XBRrHxLaAJioeFdW1oXyMA9o2JVfXFZ0IWx0Ys2aOKxBlwsAEPMVg4GNYuJLFDcnZHhl4W0UQwuWKheXi9AAJM5cIZxYzG4kU9qFioWhUMKHmFLzGjC2gNnLsIdAvgHSu+DwonCKhXyYMO2C4sGXzxCw3S7AGMCCmWAVBLC8QGjT2BjA9koURyV8PfkEiUpDpRkVwBHmVjuxEEgiV7P7h3J1FiofYUAXkBpjkxtqcSW3UQZSSW5IwGEFonQlrL+oqXWDl5Ewk6YFQC0SKbFwIiFg25ZADuZctDmMjQyUy7bHcoJEe1OQICh+HQBmVppQY+DF4dNQxVzVmZXQbyhyNBCcKiFCgWWLSI/C7ze49QPJQUuJFMjtGnSDlkFWMr3LIASNByPplkiLhJU3CtkHFxXngFywrS/hZnMXAbKyQSYIZfBJGWvC6ATk3DAyAPREBTh4oWKUMLcmxxKIKYdUakhAQmClg+MSgOcAhLWxnl/MF1GJlqDhxNFpQvuSSHpwMTYpVYDiJytnDyAGRrOH+MwvYBEjZVOI5JNZFw6qCUiCcS1R5L6g4g7YAAcXUBSRcUvq4UEnJikuqF48kFKMgBsXtKYREa/1WIK3AlhfQlPJQAsFZWI41oYoDEorgARuz0e6ANaJkDprmdThuqIjDcIwBvSsQOIL6Vkh+mUhqQPUBwl2M4jAQki5ClKiwBVUepwxaoGkVy3lEIySk/ad0Dwo8j6g7J1oHMvLFtSnq+FuAWsrQCvWEhtgh8QcOmiXBLQRplUSVI2BwRHyvip8TcAqGLSNwLx8i31XLi8m5Iuk+AC1e5O2V2xfhjeU4TUm4CyAF5sAA0AAErIAiQYENEHiC0FxR8QSACSlfhPgakipbIHBqtWQA/JAUroOsAWAEgKAiODAFRKGExxvkz6SAOBWcBsaFgdwokKiFiDWBzgQw4UD4FFabguA8QbASQHQH9BgQAgFfvECakj80AXoOvv0EGAgClE8/OvrECGGQby1Ccd0PaHKpORzgvGujcZqg1VBbF1YEbDZtiB+Ihh0AWmvh2hD1IMg9OAAJc2aNQG2CTVJtoCKM4VJ0cAnCSVq0kbNYAIYRZpTCLBlgtwBpqkFrAAwqJAASY5YkAmINm9EPOPchlAvhQwmwH5oMpBUuAvfbvkQPk2MboKHGjAAjU2R4guAxwIedECGFKpcA2aqwNAEzy6N2tGgYECNqGHBxqQkUVcLH3rBcAG59o4EEMKsAeAh1O8IYYlKGE7Idk1TTbVtr225S3QuUrgAdvwDL5YABaoQJsicS4pKAGgNSf4DxC5S5gu2vbTslykVUbhxg3KS1qJiPbkQz2rAK9re3IgyA6PAGa0Ix4vbXtuUkaFjWGFfbkQHWjQF1sGGA6dkbgqHTtrR3Q7pB4O47dINO0R5ztAgDQHXE4oKkgGdAU8AvKTw4QocV8WPk9qh37bkQH2o7ZAFykTbka1sHwDNpNH/bmdW23KaDuYR46YRbQwXcDth3eSEd82sbajr22tMhhGUhLY5A6DuJtGQws1S6r9wYgp1n088bOvJDzr/pS6q6VgFXVkL9hG6scnetojZRd1wWfdeDWKQFhbpCsW3ewAvUeAn1zoO2LevYAPrfdWM9kW+tBglRHAOiOKWeF/Uat5Yr+KymfGA0KhQNRqWcP+Ch1JgVgeM5oNpQoDB7Ml71HzmATKnlSUJf4CWNCBYg1REAFABgDVANxWrSSevFZutLM6uNMNIdO7QDB3RQ4OA8QQYDMA4Dd8ZgMwfoFRL21JgyAEPPLYg3yDnla6AoYPV0BL3lSao5e+SJXvI24Aa9dehvfBowDN6a897R7sfnb1gBO9Hgbvdcj70D7ogQ+7vqQPiAT6ttmSJAP4tgAwLXdl1fUI4PxidBV9Ze2EJvoinV7a99exvTaCP1CET9BtJvOfsv3X7e9l2fvTptiAcA8NXfP/i/p2RJgJ5dpbaAAYsKl719wBo7KAfYC76IDB+6A8aVb1Pd4D2gDvWdSQOEsUDA+/oOgbw1L9YgOBwBCyVjpr6N95BqvZQfAP76m9q0+g8fmfbG03us+W0lxgtppSUggU7HUmFYZ+gNAoUMQEXzV2G4X5BAZgIbk9jDUiDghoAxXooMdpxDkBhULQf1oG8aoCBlg+PBv3sH4gqA4fZ4dH58GkwHyuyWSD2BR65UtsuXCvuINCGyDW+sA3vrsOH6pDS3M/UwYv2uGe9bBsCGppmB+Jh9WBvwzmlqR5qBQNMsQLth7pPhzD3gKI1YdEM2G4jNBxI6ftWwuGu9bh5A5kf6BP8MDmI5/vkYz5FH2gVgaaFIHKObRM0/wQA6QZqPb6qDEhqA40bgPOGUjiBtoxkdQMH9h9L/RIH0YFFDGXEIx21cO2CPF7IjlhkA7UZ322GGjLepI80eWNpH3DHRjY8kAb68Hutm6s9VLDV2nDJ9BoTacvEqMkHhDMRsQ/UckM3Gmjf2Fo1ftWNYyUDbfBIN0dH7j7M9HJSQF0GACYB8gsgLBIgCKaonpJcRToJiZcY4mq6+J9Q38cnwRGLDUx84zMauPgnj9D3RY9CdYNwmwIbfJTcPob5N8+jjo9FLgB2AVsgqgJ6o/SdiPUGmTMBlk04bZOwm+9r/PxF3wwON8x9MwPo7EQxNYn6YuJik78czw0mqjZxkQwybBPzGITrJ+460fSMcmOAtfLw83w/59H32vguvCcdpPAnrDlx80/YYWNynrTMJ2033tiBd81THAPxG32BCan0TxJnU2SZhD6nX9BoDWGKZNMgm6jUpi08ybWkMGljCoVIzaceP2mEgjfDgE6e2MEmtTcZ0k3qb6OhBXQipPtWmbpOmnJTcxv05aYDMFmVjwZ+E54ZSA6b79fRklDYo/aOilg5BPgC2a9MXHZj8Rhw7Ae7PMGiz7R+0wOdyOeGu+MZokySexN1mCTo5mWOOeYVMGPTxp1sxmZ9NZnOzOZ6Q3cZ7MPG1zCQTEZufiDbmCTDZoBujyPPJcZz0R70/OeuN3nbjUJwM+yZDMbn/E+/FE5ScJPanaz5Jvo+iAVD2heEZAFpZ8aNNAmALc5xk9mZlO5nkjj51c2sf6DakVTrm6IPyfyCCnhTPjUU5MdnNmmbzCRrs23vAsKmUDSplUy/3VM7mEL+5pCwSbigo1IKOETC3bv/PTH2zC5/0xxZItBniznRvxGWe/5JAaL1LMQPRcyoCImLuFlix2bYsgXITjBxSxBe4t+JlTqp/iwSZUEKpuA46G+VJizoFzpz+lmS6CdYuLnZTCllc0pefNP94gmx8UX0dkASANAzINgK5c0juXTjl5wC/hdvOEX7zYF8y1xc5Nt9griQNTX0bQBEhpLEpry0ZZ8tEWHz/liy5ldfOJBO+lZuCznuM4fGpLHloq5mZKvyW8z8pvs5kYov966rBpgU36B0tONCrbZ4q3JfYudXOL3V8s1Zd4tqnZgfR7XThARrqiWNBClq2NbasTWTLVp9KzNc4NxByz7fai3ZeVaxXsL4pra9efauTXiLFVjK/adBCJAeTmIprVoqdgARnRT4ZUSjRlgUBThhuJ/M4l4WwhDc+ynHOiT9A8hka+iJiIDeBvSBQbggiG4lHwBEBLlQwoG3WuRvg2zhyw84cAAlgKhfY2fHoXoH+1SDfosoZof4AbPFtEAxgxKQ7Ql4s3zBwg0S2wH6HXCCYuQym0DuRCYQYQEOxIXtpB3olfoMW/INzeVAKproJAGfXQnxJUYOIFAbltbEOhMRCQACIDpTpI4hhCw6eDQeMHJj2UthwnBOI8PIUIj7gSI14Fqy0zsYq5yoFKbMo0D83XtyIEEsENoyGw0k9YD22LaVj5Buk62GMPEK0Gi2ttyIHCDNELgeBabnNn2yiGVvBCWbbglm5IIFsc3Vc/Q34XzKCrCDA70doW4za4AFCo75g2nJLakwy3Ra9Q/4U4y7GbqYwlQV0KVTYhQc9yebLYvOm4V6FbghYiW0waInVhmEzAdYuOH4QLASCIYFtVwu+haMsA4MSzERJlEkT67pIBmj0PduV2UQ3t/oX7cGREBi75g4O10hmivwI7YQtmyiFjsKYFSid3O7RnXgF3pb6dqO1nc9s52ubtGSbAuO6EBxumfNve4LadDC3y7VQ0B9XcwC13aMnoxhLREVtSxawGKRAOlWtB8klEzibuVy0gD7husrsALB7BDh3zegcQI4FwEmDGj2Fh8v8o+mHRxJLIEKr0HRVZpwV7yfARicUpYWVQCac0bVnjAWBpcoUfa1luxWJqKhTCThcGIJEduxlZl/4U+/vfdyH3nAx95R8iHPuh2r7GghIbfZjsLz47T93+yiBmGAPw4iAVIPhyvAuRY8iuz+wY5/vJ3kQQWvRmeDQvZ1cMehXmy0NAel2RbBjmB1LaMG0ZYS+MMZrPddkyTlY8gAqdlD7vKgiO7sROWTPDjkOEgyQBwkSB6A8lc6mnJQy5DXyBYYOd+BYC2P+MrgMoV6CRxuTMJqI2YeDzDJih0KyPlQVtyzLvYMcH3fb6jgO6A+0eX3w7ejyOwY/vvGOSKSd/oceY1BehPH2UEzlY5seIA7HH9zO048mfP2UQ6QhUYqFsHKBSAmjgJ5A7aFBPh7IT/oU/m8a6Wa4a1rxb1wEMNoLw0FPGNgDNCWDdnJlDRFQFIDbB7Q/gTGIQhGU6VPnzhA5yyBIpdOBbziKe5OH6G+DQNHz6wRjahW6YRTCJIcC4iGQlACINEEsOIHQcRr2n3z+Dtl2kD8hfxULz2z05RBH3+nYzox4/c2emOLBIL5F4zbWd7av7Yt4J3A5RAIP5byD4kH8MxhOMMH3LWgIqtfzj5qg5BWFrTE4cAJzUKSVSJ4DlmNUMu494V2/YwCT2VtTjXl9PeVWt2hE7sep5+UlvCsNbGQORfKHedawjUwiZAGwDJgro1o2ascABCQeQ95CSj0B84/6FnBVwDd0V0FQYYylwgkI7NRtlleNhNHNLkHbFKOfgOy74uwoUHehoX2w7yd0Iem+jvjOmXuQqZ7Rl0pGH7HW21pqOgyntDqbuAWwPncbtwvaM8QZqTMHH5qbyLDAXAbQBmDr8W3dAL0I31oBwC0Av/Lvkomv5upL+aAHTTO9EACB9+AA4St/1oCJBWp0QId26gYAe2OhZHWwNcP6GqAF3XfBgGPtoAMAvQiQLt6pc36d9Yg/iGYGgCAH180AaAJII3zQA38GA1fbvnxboChnh+0QIAUvxBDAg79DAeILHkrfB8SWgg/gaQCEEkVc+XAqLOn3D6G5kk3IPQ4SGw8l8osBgapru9lBDHjYqQXAKEFYKKDCjjV7BOoP6DQe0PeAfABh67mCDi4nA62PoCAA -->\n\n<!-- internal state end -->\n<!-- usage_tips_start -->\n\n> [!TIP]\n> <details>\n> <summary>You can customize the high-level summary generated by CodeRabbit.</summary>\n> \n> Configure the `reviews.high_level_summary_instructions` setting to provide custom instructions for generating the high-level summary.\n> \n> </details>\n\n<!-- usage_tips_end -->",
|
||
"author_association": "CONTRIBUTOR",
|
||
"reactions": {
|
||
"url": "https://api.github.com/repos/laurent22/joplin/issues/comments/4020462232/reactions",
|
||
"total_count": 0,
|
||
"+1": 0,
|
||
"-1": 0,
|
||
"laugh": 0,
|
||
"hooray": 0,
|
||
"confused": 0,
|
||
"heart": 0,
|
||
"rocket": 0,
|
||
"eyes": 0
|
||
},
|
||
"performed_via_github_app": {
|
||
"id": 347564,
|
||
"client_id": "Iv1.6aaafe4fe882736b",
|
||
"slug": "coderabbitai",
|
||
"node_id": "A_kwHOB96YWc4ABU2s",
|
||
"owner": {
|
||
"login": "coderabbitai",
|
||
"id": 132028505,
|
||
"node_id": "O_kgDOB96YWQ",
|
||
"avatar_url": "https://avatars.githubusercontent.com/u/132028505?v=4",
|
||
"gravatar_id": "",
|
||
"url": "https://api.github.com/users/coderabbitai",
|
||
"html_url": "https://github.com/coderabbitai",
|
||
"followers_url": "https://api.github.com/users/coderabbitai/followers",
|
||
"following_url": "https://api.github.com/users/coderabbitai/following{/other_user}",
|
||
"gists_url": "https://api.github.com/users/coderabbitai/gists{/gist_id}",
|
||
"starred_url": "https://api.github.com/users/coderabbitai/starred{/owner}{/repo}",
|
||
"subscriptions_url": "https://api.github.com/users/coderabbitai/subscriptions",
|
||
"organizations_url": "https://api.github.com/users/coderabbitai/orgs",
|
||
"repos_url": "https://api.github.com/users/coderabbitai/repos",
|
||
"events_url": "https://api.github.com/users/coderabbitai/events{/privacy}",
|
||
"received_events_url": "https://api.github.com/users/coderabbitai/received_events",
|
||
"type": "Organization",
|
||
"user_view_type": "public",
|
||
"site_admin": false
|
||
},
|
||
"name": "coderabbitai",
|
||
"description": "# Transforming Code Reviews with AI\r\n\r\n## Features\r\n\r\n**Automated Reviews**: Continuous reviews of the pull requests including incremental commits. \r\n\r\n**Summarization**: Generates high-level summary and a technical walkthrough of the PR changes. \r\n\r\n**Line-by-line review**: Provides line-by-line suggestions committable with one click.\r\n\r\n**Codebase verification**: Verifies the impact on the overall codebase and identifies missing changes.\r\n\r\n**Insights into your code**: Ask any questions on your codebase within the pull request \r\n\r\n**Chat about your code** : Chat with the bot around your code. The more you chat, the smarter it gets.\r\n\r\n**Issue Validation**: Validates the PR against the linked issues and identifies other related issues \r\n\r\n\r\n\r\n",
|
||
"external_url": "https://coderabbit.ai?utm_source=cr_app&utm_medium=github",
|
||
"html_url": "https://github.com/apps/coderabbitai",
|
||
"created_at": "2023-06-14T15:47:27Z",
|
||
"updated_at": "2026-03-14T07:58:44Z",
|
||
"permissions": {
|
||
"actions": "read",
|
||
"checks": "write",
|
||
"contents": "write",
|
||
"discussions": "read",
|
||
"issues": "write",
|
||
"members": "read",
|
||
"metadata": "read",
|
||
"pull_requests": "write",
|
||
"statuses": "write"
|
||
},
|
||
"events": [
|
||
"issues",
|
||
"issue_comment",
|
||
"label",
|
||
"organization",
|
||
"pull_request",
|
||
"pull_request_review",
|
||
"pull_request_review_comment",
|
||
"pull_request_review_thread",
|
||
"release",
|
||
"repository"
|
||
]
|
||
}
|
||
},
|
||
{
|
||
"url": "https://api.github.com/repos/laurent22/joplin/issues/comments/4020469049",
|
||
"html_url": "https://github.com/laurent22/joplin/pull/14651#issuecomment-4020469049",
|
||
"issue_url": "https://api.github.com/repos/laurent22/joplin/issues/14651",
|
||
"id": 4020469049,
|
||
"node_id": "IC_kwDOBLftOs7vo305",
|
||
"user": {
|
||
"login": "Pixels57",
|
||
"id": 111073325,
|
||
"node_id": "U_kgDOBp7YLQ",
|
||
"avatar_url": "https://avatars.githubusercontent.com/u/111073325?v=4",
|
||
"gravatar_id": "",
|
||
"url": "https://api.github.com/users/Pixels57",
|
||
"html_url": "https://github.com/Pixels57",
|
||
"followers_url": "https://api.github.com/users/Pixels57/followers",
|
||
"following_url": "https://api.github.com/users/Pixels57/following{/other_user}",
|
||
"gists_url": "https://api.github.com/users/Pixels57/gists{/gist_id}",
|
||
"starred_url": "https://api.github.com/users/Pixels57/starred{/owner}{/repo}",
|
||
"subscriptions_url": "https://api.github.com/users/Pixels57/subscriptions",
|
||
"organizations_url": "https://api.github.com/users/Pixels57/orgs",
|
||
"repos_url": "https://api.github.com/users/Pixels57/repos",
|
||
"events_url": "https://api.github.com/users/Pixels57/events{/privacy}",
|
||
"received_events_url": "https://api.github.com/users/Pixels57/received_events",
|
||
"type": "User",
|
||
"user_view_type": "public",
|
||
"site_admin": false
|
||
},
|
||
"created_at": "2026-03-09T01:03:07Z",
|
||
"updated_at": "2026-03-09T01:03:07Z",
|
||
"body": "I have read the CLA Document and I hereby sign the CLA",
|
||
"author_association": "CONTRIBUTOR",
|
||
"reactions": {
|
||
"url": "https://api.github.com/repos/laurent22/joplin/issues/comments/4020469049/reactions",
|
||
"total_count": 0,
|
||
"+1": 0,
|
||
"-1": 0,
|
||
"laugh": 0,
|
||
"hooray": 0,
|
||
"confused": 0,
|
||
"heart": 0,
|
||
"rocket": 0,
|
||
"eyes": 0
|
||
},
|
||
"performed_via_github_app": null
|
||
},
|
||
{
|
||
"url": "https://api.github.com/repos/laurent22/joplin/issues/comments/4020570985",
|
||
"html_url": "https://github.com/laurent22/joplin/pull/14651#issuecomment-4020570985",
|
||
"issue_url": "https://api.github.com/repos/laurent22/joplin/issues/14651",
|
||
"id": 4020570985,
|
||
"node_id": "IC_kwDOBLftOs7vpQtp",
|
||
"user": {
|
||
"login": "Pixels57",
|
||
"id": 111073325,
|
||
"node_id": "U_kgDOBp7YLQ",
|
||
"avatar_url": "https://avatars.githubusercontent.com/u/111073325?v=4",
|
||
"gravatar_id": "",
|
||
"url": "https://api.github.com/users/Pixels57",
|
||
"html_url": "https://github.com/Pixels57",
|
||
"followers_url": "https://api.github.com/users/Pixels57/followers",
|
||
"following_url": "https://api.github.com/users/Pixels57/following{/other_user}",
|
||
"gists_url": "https://api.github.com/users/Pixels57/gists{/gist_id}",
|
||
"starred_url": "https://api.github.com/users/Pixels57/starred{/owner}{/repo}",
|
||
"subscriptions_url": "https://api.github.com/users/Pixels57/subscriptions",
|
||
"organizations_url": "https://api.github.com/users/Pixels57/orgs",
|
||
"repos_url": "https://api.github.com/users/Pixels57/repos",
|
||
"events_url": "https://api.github.com/users/Pixels57/events{/privacy}",
|
||
"received_events_url": "https://api.github.com/users/Pixels57/received_events",
|
||
"type": "User",
|
||
"user_view_type": "public",
|
||
"site_admin": false
|
||
},
|
||
"created_at": "2026-03-09T01:40:08Z",
|
||
"updated_at": "2026-03-09T01:40:08Z",
|
||
"body": "Addressed CodeRabbit feedback: (1) Added itemKey so useItemEventHandlers re-runs when the element is recreated and listeners are re-attached; (2) Made rootElement nullable in useItemElement and removed the double cast in the test.",
|
||
"author_association": "CONTRIBUTOR",
|
||
"reactions": {
|
||
"url": "https://api.github.com/repos/laurent22/joplin/issues/comments/4020570985/reactions",
|
||
"total_count": 0,
|
||
"+1": 0,
|
||
"-1": 0,
|
||
"laugh": 0,
|
||
"hooray": 0,
|
||
"confused": 0,
|
||
"heart": 0,
|
||
"rocket": 0,
|
||
"eyes": 0
|
||
},
|
||
"performed_via_github_app": null
|
||
},
|
||
{
|
||
"url": "https://api.github.com/repos/laurent22/joplin/issues/comments/4043109098",
|
||
"html_url": "https://github.com/laurent22/joplin/pull/14651#issuecomment-4043109098",
|
||
"issue_url": "https://api.github.com/repos/laurent22/joplin/issues/14651",
|
||
"id": 4043109098,
|
||
"node_id": "IC_kwDOBLftOs7w_PLq",
|
||
"user": {
|
||
"login": "Pixels57",
|
||
"id": 111073325,
|
||
"node_id": "U_kgDOBp7YLQ",
|
||
"avatar_url": "https://avatars.githubusercontent.com/u/111073325?v=4",
|
||
"gravatar_id": "",
|
||
"url": "https://api.github.com/users/Pixels57",
|
||
"html_url": "https://github.com/Pixels57",
|
||
"followers_url": "https://api.github.com/users/Pixels57/followers",
|
||
"following_url": "https://api.github.com/users/Pixels57/following{/other_user}",
|
||
"gists_url": "https://api.github.com/users/Pixels57/gists{/gist_id}",
|
||
"starred_url": "https://api.github.com/users/Pixels57/starred{/owner}{/repo}",
|
||
"subscriptions_url": "https://api.github.com/users/Pixels57/subscriptions",
|
||
"organizations_url": "https://api.github.com/users/Pixels57/orgs",
|
||
"repos_url": "https://api.github.com/users/Pixels57/repos",
|
||
"events_url": "https://api.github.com/users/Pixels57/events{/privacy}",
|
||
"received_events_url": "https://api.github.com/users/Pixels57/received_events",
|
||
"type": "User",
|
||
"user_view_type": "public",
|
||
"site_admin": false
|
||
},
|
||
"created_at": "2026-03-12T00:36:44Z",
|
||
"updated_at": "2026-03-12T00:36:44Z",
|
||
"body": "I have implemented the suggested refactor\r\n\r\n**Changes:**\r\n\r\n- `useItemEventHandlers`: Now takes only (onInputChange, onClick) and returns a stable object { onInputChange, onClick } via useMemo. It no longer takes a ref, root element, or itemKey, and has no useEffect or DOM usage, the hook only provide handlers.\r\n\r\n- `useItemElement`: Now accepts a last parameter `itemEventHandlers `(type `ItemEventHandlers`), In the same `useEffect `that creates the element, it finds inputs (checkbox/text) and buttons inside the element and attaches `itemEventHandlers.onInputChange` and `itemEventHandlers.onClick`. DOM is no longer “forced” to update by `itemKey`\r\n\r\n- `NoteListItem`: It no longer passes a ref into `useItemEventHandlers `or uses `itemKey`.\r\n\r\n- `Types`: Added `ItemEventHandlers `in utils/types.ts for the object returned by `useItemEventHandlers`.\r\n\r\n- `useItemEventHandlers.test.ts`: Since that there are No DOM or refs so I removed `act`.",
|
||
"author_association": "CONTRIBUTOR",
|
||
"reactions": {
|
||
"url": "https://api.github.com/repos/laurent22/joplin/issues/comments/4043109098/reactions",
|
||
"total_count": 0,
|
||
"+1": 0,
|
||
"-1": 0,
|
||
"laugh": 0,
|
||
"hooray": 0,
|
||
"confused": 0,
|
||
"heart": 0,
|
||
"rocket": 0,
|
||
"eyes": 0
|
||
},
|
||
"performed_via_github_app": null
|
||
},
|
||
{
|
||
"url": "https://api.github.com/repos/laurent22/joplin/issues/comments/4061443290",
|
||
"html_url": "https://github.com/laurent22/joplin/pull/14651#issuecomment-4061443290",
|
||
"issue_url": "https://api.github.com/repos/laurent22/joplin/issues/14651",
|
||
"id": 4061443290,
|
||
"node_id": "IC_kwDOBLftOs7yFLTa",
|
||
"user": {
|
||
"login": "Pixels57",
|
||
"id": 111073325,
|
||
"node_id": "U_kgDOBp7YLQ",
|
||
"avatar_url": "https://avatars.githubusercontent.com/u/111073325?v=4",
|
||
"gravatar_id": "",
|
||
"url": "https://api.github.com/users/Pixels57",
|
||
"html_url": "https://github.com/Pixels57",
|
||
"followers_url": "https://api.github.com/users/Pixels57/followers",
|
||
"following_url": "https://api.github.com/users/Pixels57/following{/other_user}",
|
||
"gists_url": "https://api.github.com/users/Pixels57/gists{/gist_id}",
|
||
"starred_url": "https://api.github.com/users/Pixels57/starred{/owner}{/repo}",
|
||
"subscriptions_url": "https://api.github.com/users/Pixels57/subscriptions",
|
||
"organizations_url": "https://api.github.com/users/Pixels57/orgs",
|
||
"repos_url": "https://api.github.com/users/Pixels57/repos",
|
||
"events_url": "https://api.github.com/users/Pixels57/events{/privacy}",
|
||
"received_events_url": "https://api.github.com/users/Pixels57/received_events",
|
||
"type": "User",
|
||
"user_view_type": "public",
|
||
"site_admin": false
|
||
},
|
||
"created_at": "2026-03-14T21:31:20Z",
|
||
"updated_at": "2026-03-16T00:45:16Z",
|
||
"body": "Hello @personalizedrefrigerator, I did the changes.\r\n- removed useItemEventHandlers.\r\n- refactored `useItemElement `with an `addItemEventListeners` helper returns **cleanup**.\r\n- fixed the ItemEventHandlers type in NoteListItem.\r\n- .md files are note ignored now.",
|
||
"author_association": "CONTRIBUTOR",
|
||
"reactions": {
|
||
"url": "https://api.github.com/repos/laurent22/joplin/issues/comments/4061443290/reactions",
|
||
"total_count": 0,
|
||
"+1": 0,
|
||
"-1": 0,
|
||
"laugh": 0,
|
||
"hooray": 0,
|
||
"confused": 0,
|
||
"heart": 0,
|
||
"rocket": 0,
|
||
"eyes": 0
|
||
},
|
||
"performed_via_github_app": null
|
||
},
|
||
{
|
||
"url": "https://api.github.com/repos/laurent22/joplin/issues/comments/4064285488",
|
||
"html_url": "https://github.com/laurent22/joplin/pull/14651#issuecomment-4064285488",
|
||
"issue_url": "https://api.github.com/repos/laurent22/joplin/issues/14651",
|
||
"id": 4064285488,
|
||
"node_id": "IC_kwDOBLftOs7yQBMw",
|
||
"user": {
|
||
"login": "Pixels57",
|
||
"id": 111073325,
|
||
"node_id": "U_kgDOBp7YLQ",
|
||
"avatar_url": "https://avatars.githubusercontent.com/u/111073325?v=4",
|
||
"gravatar_id": "",
|
||
"url": "https://api.github.com/users/Pixels57",
|
||
"html_url": "https://github.com/Pixels57",
|
||
"followers_url": "https://api.github.com/users/Pixels57/followers",
|
||
"following_url": "https://api.github.com/users/Pixels57/following{/other_user}",
|
||
"gists_url": "https://api.github.com/users/Pixels57/gists{/gist_id}",
|
||
"starred_url": "https://api.github.com/users/Pixels57/starred{/owner}{/repo}",
|
||
"subscriptions_url": "https://api.github.com/users/Pixels57/subscriptions",
|
||
"organizations_url": "https://api.github.com/users/Pixels57/orgs",
|
||
"repos_url": "https://api.github.com/users/Pixels57/repos",
|
||
"events_url": "https://api.github.com/users/Pixels57/events{/privacy}",
|
||
"received_events_url": "https://api.github.com/users/Pixels57/received_events",
|
||
"type": "User",
|
||
"user_view_type": "public",
|
||
"site_admin": false
|
||
},
|
||
"created_at": "2026-03-16T00:19:46Z",
|
||
"updated_at": "2026-03-16T00:19:46Z",
|
||
"body": "Addressed CodeRabbit: focus-visible no longer triggers a full DOM rebuild.\r\n\r\n`focusVisible` is no longer in the main effect’s dependency array. A second, ref-based effect only adds/removes the `-focus-visible` class on the existing element when `focusVisible` changes. This keeps live DOM state (e.g. text in plugin inputs, active control) intact when tabbing between notes.",
|
||
"author_association": "CONTRIBUTOR",
|
||
"reactions": {
|
||
"url": "https://api.github.com/repos/laurent22/joplin/issues/comments/4064285488/reactions",
|
||
"total_count": 1,
|
||
"+1": 1,
|
||
"-1": 0,
|
||
"laugh": 0,
|
||
"hooray": 0,
|
||
"confused": 0,
|
||
"heart": 0,
|
||
"rocket": 0,
|
||
"eyes": 0
|
||
},
|
||
"performed_via_github_app": null
|
||
},
|
||
{
|
||
"url": "https://api.github.com/repos/laurent22/joplin/issues/comments/4067391983",
|
||
"html_url": "https://github.com/laurent22/joplin/pull/14651#issuecomment-4067391983",
|
||
"issue_url": "https://api.github.com/repos/laurent22/joplin/issues/14651",
|
||
"id": 4067391983,
|
||
"node_id": "IC_kwDOBLftOs7yb3nv",
|
||
"user": {
|
||
"login": "Pixels57",
|
||
"id": 111073325,
|
||
"node_id": "U_kgDOBp7YLQ",
|
||
"avatar_url": "https://avatars.githubusercontent.com/u/111073325?v=4",
|
||
"gravatar_id": "",
|
||
"url": "https://api.github.com/users/Pixels57",
|
||
"html_url": "https://github.com/Pixels57",
|
||
"followers_url": "https://api.github.com/users/Pixels57/followers",
|
||
"following_url": "https://api.github.com/users/Pixels57/following{/other_user}",
|
||
"gists_url": "https://api.github.com/users/Pixels57/gists{/gist_id}",
|
||
"starred_url": "https://api.github.com/users/Pixels57/starred{/owner}{/repo}",
|
||
"subscriptions_url": "https://api.github.com/users/Pixels57/subscriptions",
|
||
"organizations_url": "https://api.github.com/users/Pixels57/orgs",
|
||
"repos_url": "https://api.github.com/users/Pixels57/repos",
|
||
"events_url": "https://api.github.com/users/Pixels57/events{/privacy}",
|
||
"received_events_url": "https://api.github.com/users/Pixels57/received_events",
|
||
"type": "User",
|
||
"user_view_type": "public",
|
||
"site_admin": false
|
||
},
|
||
"created_at": "2026-03-16T12:43:11Z",
|
||
"updated_at": "2026-03-16T12:43:11Z",
|
||
"body": "Hello @personalizedrefrigerator, \r\nDo you think that there are other required changes?\r\nAnd I was wondering if I could open another PR, will that be okay or not as I am a GSOC participant",
|
||
"author_association": "CONTRIBUTOR",
|
||
"reactions": {
|
||
"url": "https://api.github.com/repos/laurent22/joplin/issues/comments/4067391983/reactions",
|
||
"total_count": 0,
|
||
"+1": 0,
|
||
"-1": 0,
|
||
"laugh": 0,
|
||
"hooray": 0,
|
||
"confused": 0,
|
||
"heart": 0,
|
||
"rocket": 0,
|
||
"eyes": 0
|
||
},
|
||
"performed_via_github_app": null
|
||
},
|
||
{
|
||
"url": "https://api.github.com/repos/laurent22/joplin/issues/comments/4067555128",
|
||
"html_url": "https://github.com/laurent22/joplin/pull/14651#issuecomment-4067555128",
|
||
"issue_url": "https://api.github.com/repos/laurent22/joplin/issues/14651",
|
||
"id": 4067555128,
|
||
"node_id": "IC_kwDOBLftOs7ycfc4",
|
||
"user": {
|
||
"login": "personalizedrefrigerator",
|
||
"id": 46334387,
|
||
"node_id": "MDQ6VXNlcjQ2MzM0Mzg3",
|
||
"avatar_url": "https://avatars.githubusercontent.com/u/46334387?v=4",
|
||
"gravatar_id": "",
|
||
"url": "https://api.github.com/users/personalizedrefrigerator",
|
||
"html_url": "https://github.com/personalizedrefrigerator",
|
||
"followers_url": "https://api.github.com/users/personalizedrefrigerator/followers",
|
||
"following_url": "https://api.github.com/users/personalizedrefrigerator/following{/other_user}",
|
||
"gists_url": "https://api.github.com/users/personalizedrefrigerator/gists{/gist_id}",
|
||
"starred_url": "https://api.github.com/users/personalizedrefrigerator/starred{/owner}{/repo}",
|
||
"subscriptions_url": "https://api.github.com/users/personalizedrefrigerator/subscriptions",
|
||
"organizations_url": "https://api.github.com/users/personalizedrefrigerator/orgs",
|
||
"repos_url": "https://api.github.com/users/personalizedrefrigerator/repos",
|
||
"events_url": "https://api.github.com/users/personalizedrefrigerator/events{/privacy}",
|
||
"received_events_url": "https://api.github.com/users/personalizedrefrigerator/received_events",
|
||
"type": "User",
|
||
"user_view_type": "public",
|
||
"site_admin": false
|
||
},
|
||
"created_at": "2026-03-16T13:10:08Z",
|
||
"updated_at": "2026-03-16T13:10:08Z",
|
||
"body": "> Do you think that there are other required changes?\r\n\r\nBased on the code, this pull request looks good to me! I'm planning to test this locally in the next day or two prior to giving this change final approval.\r\n\r\n> And I was wondering if I could open another PR, will that be okay or not as I am a GSOC participant\r\n\r\nPlease do so! This change is just waiting for my final review after local testing.",
|
||
"author_association": "COLLABORATOR",
|
||
"reactions": {
|
||
"url": "https://api.github.com/repos/laurent22/joplin/issues/comments/4067555128/reactions",
|
||
"total_count": 0,
|
||
"+1": 0,
|
||
"-1": 0,
|
||
"laugh": 0,
|
||
"hooray": 0,
|
||
"confused": 0,
|
||
"heart": 0,
|
||
"rocket": 0,
|
||
"eyes": 0
|
||
},
|
||
"performed_via_github_app": null
|
||
}
|
||
]
|
||
} |