{"$schema":"https://json.schemastore.org/sarif-2.1.0.json","version":"2.1.0","runs":[{"tool":{"driver":{"name":"codehealth","informationUri":"https://codehealth.canine.dev","rules":[{"id":"D1","name":"Cyclomatic Complexity","shortDescription":{"text":"Cyclomatic Complexity"},"helpUri":"https://codehealth.canine.dev/dimensions/D1"},{"id":"D2","name":"Cognitive Complexity","shortDescription":{"text":"Cognitive Complexity"},"helpUri":"https://codehealth.canine.dev/dimensions/D2"},{"id":"D3","name":"God Classes","shortDescription":{"text":"God Classes"},"helpUri":"https://codehealth.canine.dev/dimensions/D3"},{"id":"D4","name":"Code Duplication","shortDescription":{"text":"Code Duplication"},"helpUri":"https://codehealth.canine.dev/dimensions/D4"},{"id":"D13","name":"Secret Scanning","shortDescription":{"text":"Secret Scanning"},"helpUri":"https://codehealth.canine.dev/dimensions/D13","relationships":[{"target":{"id":"CWE-798","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]},{"target":{"id":"CWE-259","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]}],"properties":{"cwe":["CWE-798","CWE-259"]}},{"id":"D15","name":"Churn \u00D7 Complexity Hotspots","shortDescription":{"text":"Churn \u00D7 Complexity Hotspots"},"helpUri":"https://codehealth.canine.dev/dimensions/D15"},{"id":"D16","name":"Bus Factor","shortDescription":{"text":"Bus Factor"},"helpUri":"https://codehealth.canine.dev/dimensions/D16"},{"id":"D19","name":"Documentation Quality","shortDescription":{"text":"Documentation Quality"},"helpUri":"https://codehealth.canine.dev/dimensions/D19"},{"id":"D21","name":"Naming Consistency","shortDescription":{"text":"Naming Consistency"},"helpUri":"https://codehealth.canine.dev/dimensions/D21"},{"id":"D28","name":"Secrets (history)","shortDescription":{"text":"Secrets (history)"},"helpUri":"https://codehealth.canine.dev/dimensions/D28","relationships":[{"target":{"id":"CWE-798","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]},{"target":{"id":"CWE-259","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]}],"properties":{"cwe":["CWE-798","CWE-259"]}},{"id":"D29","name":"Static Analysis (SAST)","shortDescription":{"text":"Static Analysis (SAST)"},"helpUri":"https://codehealth.canine.dev/dimensions/D29","relationships":[{"target":{"id":"CWE-79","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]},{"target":{"id":"CWE-89","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]},{"target":{"id":"CWE-78","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]},{"target":{"id":"CWE-94","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]},{"target":{"id":"CWE-77","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]}],"properties":{"cwe":["CWE-79","CWE-89","CWE-78","CWE-94","CWE-77"]}},{"id":"D31","name":"IaC \u0026 Container Security","shortDescription":{"text":"IaC \u0026 Container Security"},"helpUri":"https://codehealth.canine.dev/dimensions/D31","relationships":[{"target":{"id":"CWE-1032","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]},{"target":{"id":"CWE-732","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]},{"target":{"id":"CWE-16","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]}],"properties":{"cwe":["CWE-1032","CWE-732","CWE-16"]}},{"id":"D34","name":"Knowledge Freshness","shortDescription":{"text":"Knowledge Freshness"},"helpUri":"https://codehealth.canine.dev/dimensions/D34"},{"id":"D35","name":"Change Coupling","shortDescription":{"text":"Change Coupling"},"helpUri":"https://codehealth.canine.dev/dimensions/D35"},{"id":"D36","name":"Supply-chain Provenance \u0026 Signing","shortDescription":{"text":"Supply-chain Provenance \u0026 Signing"},"helpUri":"https://codehealth.canine.dev/dimensions/D36","relationships":[{"target":{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]},{"target":{"id":"CWE-494","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]}],"properties":{"cwe":["CWE-1357","CWE-494"]}}]}},"results":[{"ruleId":"D1","level":"warning","message":{"text":"escapeInterpreter.parseOne (cyclomatic 69): escapeInterpreter.parseOne has cyclomatic complexity 69 (threshold 15). This file\u0027s own header attributes it to another copyright holder, so it is code this repository carries rather than code it wrote: restructuring the body in place forks it from upstream and turns every future re-sync into a manual merge. The performable moves are to leave the body as close to its upstream form as possible and keep it behind a narrow interface of your own, and to re-sync it when upstream changes \u2014 or, if it has already diverged far enough that you maintain it here, adopt it deliberately and then split the body into named stages."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gocui/escape.go"},"region":{"startLine":237}}}],"partialFingerprints":{"codehealthFindingId/v1":"c671c1e365911b4e273215b2932233dbfdacf8faad9d969bef97d4c4f2b1906f"}},{"ruleId":"D1","level":"warning","message":{"text":"graph.getBoxDrawingChars (cyclomatic 65): graph.getBoxDrawingChars has cyclomatic complexity 65 (threshold 15). To reduce it, name the conditions: bind each compound test to a well-named local or a small predicate function, so the body reads as a sequence of named decisions rather than a chain of operators."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gui/presentation/graph/cell.go"},"region":{"startLine":147}}}],"partialFingerprints":{"codehealthFindingId/v1":"c27a89060ee17f99209d79af09d12d8ea65039f8627793f3b4505dbce020fdd7"}},{"ruleId":"D1","level":"warning","message":{"text":"Gui.onKey (cyclomatic 49): Gui.onKey has cyclomatic complexity 49 (threshold 15). This file\u0027s own header attributes it to another copyright holder, so it is code this repository carries rather than code it wrote: restructuring the body in place forks it from upstream and turns every future re-sync into a manual merge. The performable moves are to leave the body as close to its upstream form as possible and keep it behind a narrow interface of your own, and to re-sync it when upstream changes \u2014 or, if it has already diverged far enough that you maintain it here, adopt it deliberately and then split the body into named stages."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gocui/gui.go"},"region":{"startLine":1663}}}],"partialFingerprints":{"codehealthFindingId/v1":"d99243ad4dc371cf5d81ed085a8c7e75485636d5e90db53598b561ecf1f8ded7"}},{"ruleId":"D1","level":"warning","message":{"text":"Gui.layout (cyclomatic 42): Gui.layout has cyclomatic complexity 42 (threshold 15). Of this number, 30 points are the body\u0027s own statements and 12 belong to one function literal inside it that branches. To reduce it, separate the cases: extract each independent branch into its own named function, and where the body has guards that only reject input, fold those into early returns at the top."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gui/layout.go"},"region":{"startLine":13}}}],"partialFingerprints":{"codehealthFindingId/v1":"2fc3597f7582859ff9c619803d61f33e29a1bdf18074f7c608e7409f6393d05b"}},{"ruleId":"D1","level":"warning","message":{"text":"RefreshHelper.performRefresh (cyclomatic 40): RefreshHelper.performRefresh has cyclomatic complexity 40 (threshold 15). Of this number, 36 points are the body\u0027s own statements and 4 belong to one function literal inside it that branches. To reduce it, split the body: these branches sit side by side rather than nested inside one another, so extracting each one on its own would leave a function per branch. Group the statements between the checks into named steps and move each step into its own function, so the body reads as a short sequence of named stages."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gui/controllers/helpers/refresh_helper.go"},"region":{"startLine":168}}}],"partialFingerprints":{"codehealthFindingId/v1":"9cf18f8fd7993edea3b315b229b513eadf28ab1a600e13e0e67ac6dfcc713ed4"}},{"ruleId":"D1","level":"warning","message":{"text":"gocui.gocuiEventFromTcellEvent (cyclomatic 35): gocui.gocuiEventFromTcellEvent has cyclomatic complexity 35 (threshold 15). This file\u0027s own header attributes it to another copyright holder, so it is code this repository carries rather than code it wrote: restructuring the body in place forks it from upstream and turns every future re-sync into a manual merge. The performable moves are to leave the body as close to its upstream form as possible and keep it behind a narrow interface of your own, and to re-sync it when upstream changes \u2014 or, if it has already diverged far enough that you maintain it here, adopt it deliberately and then split the body into named stages."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gocui/tcell_driver.go"},"region":{"startLine":323}}}],"partialFingerprints":{"codehealthFindingId/v1":"903eb23495aaf5311daff3ef6cb0418c347906105d53be419c3ff87d622a89e2"}},{"ruleId":"D1","level":"warning","message":{"text":"ViewBufferManager.NewCmdTask (cyclomatic 33): ViewBufferManager.NewCmdTask has cyclomatic complexity 33 (threshold 15). Most of this is not in the body itself: 1 of the 33 points is its own statement and the rest belongs to one function literal inside it that branches (line 168). The decisions are inside the literal, which nothing outside this body can call, review or test on its own, so splitting the enclosing body is not the move available here. To reduce it, lift the literal\u0027s work into a named function or method at the enclosing scope and have the literal call it, then reduce whichever part then reads as the largest."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/tasks/tasks.go"},"region":{"startLine":167}}}],"partialFingerprints":{"codehealthFindingId/v1":"ba121c8a80c2ebaf00fad3634d6030d87bea9c44a53312829d13bf0a0ac336d9"}},{"ruleId":"D1","level":"warning","message":{"text":"graph.renderPipeSet (cyclomatic 31): graph.renderPipeSet has cyclomatic complexity 31 (threshold 15). Of this number, 24 points are the body\u0027s own statements and 7 belong to 2 function literals inside it that branch. To reduce it, separate the cases: extract each independent branch into its own named function, and where the body has guards that only reject input, fold those into early returns at the top."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gui/presentation/graph/graph.go"},"region":{"startLine":275}}}],"partialFingerprints":{"codehealthFindingId/v1":"7d6074afb0191ee02bf7ca954df2656e95044e01dd25588eece5b4a83dd4a3bd"}},{"ruleId":"D1","level":"warning","message":{"text":"config.KeyFromLabel (cyclomatic 29): config.KeyFromLabel has cyclomatic complexity 29 (threshold 15). To reduce it, separate the cases: extract each independent branch into its own named function, and where the body has guards that only reject input, fold those into early returns at the top."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/config/keynames.go"},"region":{"startLine":95}}}],"partialFingerprints":{"codehealthFindingId/v1":"7110e8f11860839cba7709c3f69d4140cef50843b123287e24ae0deb287c0865"}},{"ruleId":"D1","level":"warning","message":{"text":"presentation.getBranchDisplayStrings (cyclomatic 29): presentation.getBranchDisplayStrings has cyclomatic complexity 29 (threshold 15). To reduce it, separate the cases: extract each independent branch into its own named function, and where the body has guards that only reject input, fold those into early returns at the top."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gui/presentation/branches.go"},"region":{"startLine":47}}}],"partialFingerprints":{"codehealthFindingId/v1":"f3cad920146484c5a5ed5146c12fbd7d384468a1e18613b23d90db276cebf40e"}},{"ruleId":"D1","level":"warning","message":{"text":"app.Start (cyclomatic 27): app.Start has cyclomatic complexity 27 (threshold 15). Of this number, 26 points are the body\u0027s own statements and 1 belongs to one function literal inside it that branches. To reduce it, separate the cases: extract each independent branch into its own named function, and where the body has guards that only reject input, fold those into early returns at the top."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/app/entry_point.go"},"region":{"startLine":53}}}],"partialFingerprints":{"codehealthFindingId/v1":"8346ce284e77ff149432aa6c171c3ed31352a002d0e990060bc7f52bc18a003f"}},{"ruleId":"D1","level":"warning","message":{"text":"Gui.execKeybindings (cyclomatic 27): Gui.execKeybindings has cyclomatic complexity 27 (threshold 15). This file\u0027s own header attributes it to another copyright holder, so it is code this repository carries rather than code it wrote: restructuring the body in place forks it from upstream and turns every future re-sync into a manual merge. The performable moves are to leave the body as close to its upstream form as possible and keep it behind a narrow interface of your own, and to re-sync it when upstream changes \u2014 or, if it has already diverged far enough that you maintain it here, adopt it deliberately and then split the body into named stages."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gocui/gui.go"},"region":{"startLine":1915}}}],"partialFingerprints":{"codehealthFindingId/v1":"78132e00ae09b7735db16fc07a5241e2e5cfd8bf21af5484c3770bda124a7f56"}},{"ruleId":"D1","level":"warning","message":{"text":"graph.getNextPipes (cyclomatic 27): graph.getNextPipes has cyclomatic complexity 27 (threshold 15). Of this number, 19 points are the body\u0027s own statements and 8 belong to 4 function literals inside it that branch. To reduce it, separate the cases: extract each independent branch into its own named function, and where the body has guards that only reject input, fold those into early returns at the top."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gui/presentation/graph/graph.go"},"region":{"startLine":109}}}],"partialFingerprints":{"codehealthFindingId/v1":"7db532d3562ccd49c5d617bc76dab2d3bdf4c7e21d47449f896f797315c7e041"}},{"ruleId":"D1","level":"warning","message":{"text":"helpers.shrinkToContentSidePanelBoxes (cyclomatic 26): helpers.shrinkToContentSidePanelBoxes has cyclomatic complexity 26 (threshold 15). To reduce it, separate the cases: extract each independent branch into its own named function, and where the body has guards that only reject input, fold those into early returns at the top."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gui/controllers/helpers/window_arrangement_helper.go"},"region":{"startLine":547}}}],"partialFingerprints":{"codehealthFindingId/v1":"5e2721f051157aea32986577e5ca6491c17f564faa130cf4b2e1ee321b8ce57e"}},{"ruleId":"D1","level":"warning","message":{"text":"Gui.draw (cyclomatic 25): Gui.draw has cyclomatic complexity 25 (threshold 15). This file\u0027s own header attributes it to another copyright holder, so it is code this repository carries rather than code it wrote: restructuring the body in place forks it from upstream and turns every future re-sync into a manual merge. The performable moves are to leave the body as close to its upstream form as possible and keep it behind a narrow interface of your own, and to re-sync it when upstream changes \u2014 or, if it has already diverged far enough that you maintain it here, adopt it deliberately and then split the body into named stages."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gocui/gui.go"},"region":{"startLine":1593}}}],"partialFingerprints":{"codehealthFindingId/v1":"6bbd6129b6d7b061ae61aa397cbcd65cf3f47ff034bbd6ac55edebf62d4cdbec"}},{"ruleId":"D1","level":"warning","message":{"text":"presentation.displayCommit (cyclomatic 23): presentation.displayCommit has cyclomatic complexity 23 (threshold 15). To reduce it, split the body: these branches sit side by side rather than nested inside one another, so extracting each one on its own would leave a function per branch. Group the statements between the checks into named steps and move each step into its own function, so the body reads as a short sequence of named stages."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gui/presentation/commits.go"},"region":{"startLine":341}}}],"partialFingerprints":{"codehealthFindingId/v1":"2779ec65d773e4baf4f251739eb00ff74cbc4780473799fc7b0e73f4987f1e3d"}},{"ruleId":"D1","level":"warning","message":{"text":"escapeInterpreter.outputCSI (cyclomatic 22): escapeInterpreter.outputCSI has cyclomatic complexity 22 (threshold 15). This file\u0027s own header attributes it to another copyright holder, so it is code this repository carries rather than code it wrote: restructuring the body in place forks it from upstream and turns every future re-sync into a manual merge. The performable moves are to leave the body as close to its upstream form as possible and keep it behind a narrow interface of your own, and to re-sync it when upstream changes \u2014 or, if it has already diverged far enough that you maintain it here, adopt it deliberately and then split the body into named stages."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gocui/escape.go"},"region":{"startLine":481}}}],"partialFingerprints":{"codehealthFindingId/v1":"8fe840f806615612e07d2a5142df993c9b66cbc1aaa12be63457863d0b5d92b6"}},{"ruleId":"D1","level":"warning","message":{"text":"RefreshHelper.refreshStateFiles (cyclomatic 22): RefreshHelper.refreshStateFiles has cyclomatic complexity 22 (threshold 15). Of this number, 16 points are the body\u0027s own statements and 6 belong to 2 function literals inside it that branch. To reduce it, separate the cases: extract each independent branch into its own named function, and where the body has guards that only reject input, fold those into early returns at the top."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gui/controllers/helpers/refresh_helper.go"},"region":{"startLine":1286}}}],"partialFingerprints":{"codehealthFindingId/v1":"5fbb20e732d0107b54a120d3e3834be840b064d949769191fda0d99c603ab280"}},{"ruleId":"D1","level":"warning","message":{"text":"Gui.drawTitle (cyclomatic 21): Gui.drawTitle has cyclomatic complexity 21 (threshold 15). This file\u0027s own header attributes it to another copyright holder, so it is code this repository carries rather than code it wrote: restructuring the body in place forks it from upstream and turns every future re-sync into a manual merge. The performable moves are to leave the body as close to its upstream form as possible and keep it behind a narrow interface of your own, and to re-sync it when upstream changes \u2014 or, if it has already diverged far enough that you maintain it here, adopt it deliberately and then split the body into named stages."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gocui/gui.go"},"region":{"startLine":1374}}}],"partialFingerprints":{"codehealthFindingId/v1":"3c8b11c62526da3fc947a10a6596faf9d1a74c6d8d9df01726ee108c733123a3"}},{"ruleId":"D1","level":"warning","message":{"text":"WorktreeLoader.GetWorktrees (cyclomatic 20): WorktreeLoader.GetWorktrees has cyclomatic complexity 20 (threshold 15). Of this number, 18 points are the body\u0027s own statements and 2 belong to one function literal inside it that branches. To reduce it, separate the cases: extract each independent branch into its own named function, and where the body has guards that only reject input, fold those into early returns at the top."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/commands/git_commands/worktree_loader.go"},"region":{"startLine":24}}}],"partialFingerprints":{"codehealthFindingId/v1":"eafeeb98aa03938f590927319db0285b425598556c509bfa0cf47dad51d32390"}},{"ruleId":"D1","level":"warning","message":{"text":"Gui.drawFrameEdges (cyclomatic 20): Gui.drawFrameEdges has cyclomatic complexity 20 (threshold 15). This file\u0027s own header attributes it to another copyright holder, so it is code this repository carries rather than code it wrote: restructuring the body in place forks it from upstream and turns every future re-sync into a manual merge. The performable moves are to leave the body as close to its upstream form as possible and keep it behind a narrow interface of your own, and to re-sync it when upstream changes \u2014 or, if it has already diverged far enough that you maintain it here, adopt it deliberately and then split the body into named stages."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gocui/gui.go"},"region":{"startLine":1194}}}],"partialFingerprints":{"codehealthFindingId/v1":"45830d781ed1aaf2f41fc79bbeef1b71bd1b815a534b53a5829a6ed644fee43c"}},{"ruleId":"D1","level":"warning","message":{"text":"Gui.onNewRepo (cyclomatic 20): Gui.onNewRepo has cyclomatic complexity 20 (threshold 15). Most of this is not in the body itself: 7 of the 20 points are its own statements and the rest belongs to 4 function literals inside it that branch (lines 373, 400, 427, \u2026). The decisions are inside those literals, which nothing outside this body can call, review or test on its own, so splitting the enclosing body is not the move available here. To reduce it, lift the literals\u0027 work into a named function or method at the enclosing scope and have each literal call it, then reduce whichever part then reads as the largest."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gui/gui.go"},"region":{"startLine":342}}}],"partialFingerprints":{"codehealthFindingId/v1":"66d74838e401fc19edc7ee49ee94d3edb5fed7d42adfaa6d65f18e6c1b49e5e4"}},{"ruleId":"D1","level":"warning","message":{"text":"Gui.drawFrameCorners (cyclomatic 19): Gui.drawFrameCorners has cyclomatic complexity 19 (threshold 15). This file\u0027s own header attributes it to another copyright holder, so it is code this repository carries rather than code it wrote: restructuring the body in place forks it from upstream and turns every future re-sync into a manual merge. The performable moves are to leave the body as close to its upstream form as possible and keep it behind a narrow interface of your own, and to re-sync it when upstream changes \u2014 or, if it has already diverged far enough that you maintain it here, adopt it deliberately and then split the body into named stages."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gocui/gui.go"},"region":{"startLine":1334}}}],"partialFingerprints":{"codehealthFindingId/v1":"1f62a543a233e011fb61acff569f18254326392ca29298840ee4eecc5f9453d8"}},{"ruleId":"D1","level":"warning","message":{"text":"View.draw (cyclomatic 19): View.draw has cyclomatic complexity 19 (threshold 15). This file\u0027s own header attributes it to another copyright holder, so it is code this repository carries rather than code it wrote: restructuring the body in place forks it from upstream and turns every future re-sync into a manual merge. The performable moves are to leave the body as close to its upstream form as possible and keep it behind a narrow interface of your own, and to re-sync it when upstream changes \u2014 or, if it has already diverged far enough that you maintain it here, adopt it deliberately and then split the body into named stages."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gocui/view.go"},"region":{"startLine":1322}}}],"partialFingerprints":{"codehealthFindingId/v1":"2e3e5e26109eab8102047001b61c27158e6bcc7fdfa83f865a0a659bae9dbad3"}},{"ruleId":"D1","level":"warning","message":{"text":"presentation.GetCommitListDisplayStrings (cyclomatic 19): presentation.GetCommitListDisplayStrings has cyclomatic complexity 19 (threshold 15). Of this number, 14 points are the body\u0027s own statements and 5 belong to 2 function literals inside it that branch. To reduce it, separate the cases: extract each independent branch into its own named function, and where the body has guards that only reject input, fold those into early returns at the top."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gui/presentation/commits.go"},"region":{"startLine":40}}}],"partialFingerprints":{"codehealthFindingId/v1":"ff7adedd80a7d013cb7c5d3b57b09484a38e5ac6f3b6c2bccb025c34e9c17d76"}},{"ruleId":"D1","level":"warning","message":{"text":"clients.RunTUI (cyclomatic 19): clients.RunTUI has cyclomatic complexity 19 (threshold 15). Most of this is not in the body itself: 3 of the 19 points are its own statements and the rest belongs to 11 function literals inside it that branch (lines 46, 58, 119, \u2026). The decisions are inside those literals, which nothing outside this body can call, review or test on its own, so splitting the enclosing body is not the move available here. To reduce it, lift the literals\u0027 work into a named function or method at the enclosing scope and have each literal call it, then reduce whichever part then reads as the largest."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/integration/clients/tui.go"},"region":{"startLine":25}}}],"partialFingerprints":{"codehealthFindingId/v1":"32c1169b6506425d986235485fcd0ac329147527d622ff884b84c0feadaa7d86"}},{"ruleId":"D1","level":"warning","message":{"text":"App.setupRepo (cyclomatic 18): App.setupRepo has cyclomatic complexity 18 (threshold 15). To reduce it, separate the cases: extract each independent branch into its own named function, and where the body has guards that only reject input, fold those into early returns at the top."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/app/app.go"},"region":{"startLine":194}}}],"partialFingerprints":{"codehealthFindingId/v1":"552fabab7b44bd2b5b59fd0a4daa1126a274e003b1322e8cf89fe69887181762"}},{"ruleId":"D1","level":"warning","message":{"text":"config.computeMigratedConfig (cyclomatic 18): config.computeMigratedConfig has cyclomatic complexity 18 (threshold 15). To reduce it, split the body: these branches sit side by side rather than nested inside one another, so extracting each one on its own would leave a function per branch. Group the statements between the checks into named steps and move each step into its own function, so the body reads as a short sequence of named stages."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/config/app_config.go"},"region":{"startLine":277}}}],"partialFingerprints":{"codehealthFindingId/v1":"6675a2c3e2f3997e5a13ef746c371545eaa2091c03f240ecce4f7a7c32550373"}},{"ruleId":"D1","level":"warning","message":{"text":"patch_exploring.NewState (cyclomatic 18): patch_exploring.NewState has cyclomatic complexity 18 (threshold 15). To reduce it, split the body: these branches sit side by side rather than nested inside one another, so extracting each one on its own would leave a function per branch. Group the statements between the checks into named steps and move each step into its own function, so the body reads as a short sequence of named stages."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gui/patch_exploring/state.go"},"region":{"startLine":48}}}],"partialFingerprints":{"codehealthFindingId/v1":"efafbcce712db1870c63dff0e252e3be3661270d1597b78d7fa248dc60e1c695"}},{"ruleId":"D1","level":"warning","message":{"text":"WorkingTreeCommands.DiscardAllDirChanges (cyclomatic 17): WorkingTreeCommands.DiscardAllDirChanges has cyclomatic complexity 17 (threshold 15). Most of this is not in the body itself: 6 of the 17 points are its own statements and the rest belongs to one function literal inside it that branches (line 198). The decisions are inside the literal, which nothing outside this body can call, review or test on its own, so splitting the enclosing body is not the move available here. To reduce it, lift the literal\u0027s work into a named function or method at the enclosing scope and have the literal call it, then reduce whichever part then reads as the largest."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/commands/git_commands/working_tree.go"},"region":{"startLine":190}}}],"partialFingerprints":{"codehealthFindingId/v1":"257acee658effaaea539b39da029752e832dac0960ca50e1e964c2dca774cc69"}},{"ruleId":"D1","level":"warning","message":{"text":"gocui.cornerCustomRune (cyclomatic 17): gocui.cornerCustomRune has cyclomatic complexity 17 (threshold 15). This file\u0027s own header attributes it to another copyright holder, so it is code this repository carries rather than code it wrote: restructuring the body in place forks it from upstream and turns every future re-sync into a manual merge. The performable moves are to leave the body as close to its upstream form as possible and keep it behind a narrow interface of your own, and to re-sync it when upstream changes \u2014 or, if it has already diverged far enough that you maintain it here, adopt it deliberately and then split the body into named stages."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gocui/gui.go"},"region":{"startLine":1274}}}],"partialFingerprints":{"codehealthFindingId/v1":"c81316228e201b1643b1ad1f0455cb6a668d7394dcdb4a185b66ec21d209e4fc"}},{"ruleId":"D1","level":"warning","message":{"text":"View.setCharacter (cyclomatic 17): View.setCharacter has cyclomatic complexity 17 (threshold 15). This file\u0027s own header attributes it to another copyright holder, so it is code this repository carries rather than code it wrote: restructuring the body in place forks it from upstream and turns every future re-sync into a manual merge. The performable moves are to leave the body as close to its upstream form as possible and keep it behind a narrow interface of your own, and to re-sync it when upstream changes \u2014 or, if it has already diverged far enough that you maintain it here, adopt it deliberately and then split the body into named stages."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gocui/view.go"},"region":{"startLine":619}}}],"partialFingerprints":{"codehealthFindingId/v1":"c347ed9ee33d8c29335802f8f97b548f44fc8b5f7932453949b2ae46e7c4cae5"}},{"ruleId":"D1","level":"warning","message":{"text":"View.write (cyclomatic 17): View.write has cyclomatic complexity 17 (threshold 15). Of this number, 16 points are the body\u0027s own statements and 1 belongs to one function literal inside it that branches. This file\u0027s own header attributes it to another copyright holder, so it is code this repository carries rather than code it wrote: restructuring the body in place forks it from upstream and turns every future re-sync into a manual merge. The performable moves are to leave the body as close to its upstream form as possible and keep it behind a narrow interface of your own, and to re-sync it when upstream changes \u2014 or, if it has already diverged far enough that you maintain it here, adopt it deliberately and then split the body into named stages."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gocui/view.go"},"region":{"startLine":856}}}],"partialFingerprints":{"codehealthFindingId/v1":"faaedaff66f1b9a17d24383620059434131d502c14f8cefe3916a94143082888"}},{"ruleId":"D1","level":"warning","message":{"text":"helpers.infoSectionChildren (cyclomatic 17): helpers.infoSectionChildren has cyclomatic complexity 17 (threshold 15). Of this number, 13 points are the body\u0027s own statements and 4 belong to 3 function literals inside it that branch. To reduce it, split the body: these branches sit side by side rather than nested inside one another, so extracting each one on its own would leave a function per branch. Group the statements between the checks into named steps and move each step into its own function, so the body reads as a short sequence of named stages."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gui/controllers/helpers/window_arrangement_helper.go"},"region":{"startLine":280}}}],"partialFingerprints":{"codehealthFindingId/v1":"24d8caaaf8414fb0794b155a49c055b9cb10a53347028e415cd5103e0daf8076"}},{"ruleId":"D1","level":"warning","message":{"text":"CommitLoader.getConflictedCommitImpl (cyclomatic 16): CommitLoader.getConflictedCommitImpl has cyclomatic complexity 16 (threshold 15). To reduce it, split the body: these branches sit side by side rather than nested inside one another, so extracting each one on its own would leave a function per branch. Group the statements between the checks into named steps and move each step into its own function, so the body reads as a short sequence of named stages."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/commands/git_commands/commit_loader.go"},"region":{"startLine":405}}}],"partialFingerprints":{"codehealthFindingId/v1":"441c1ae5aaa3dc1365619195373181ca298dd9037477925fcbc4e82ccbc25e6b"}},{"ruleId":"D1","level":"warning","message":{"text":"config.migratePagersToDiffRenderers (cyclomatic 16): config.migratePagersToDiffRenderers has cyclomatic complexity 16 (threshold 15). Most of this is not in the body itself: 1 of the 16 points is its own statement and the rest belongs to one function literal inside it that branches (line 563). The decisions are inside the literal, which nothing outside this body can call, review or test on its own, so splitting the enclosing body is not the move available here. To reduce it, lift the literal\u0027s work into a named function or method at the enclosing scope and have the literal call it, then reduce whichever part then reads as the largest."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/config/app_config.go"},"region":{"startLine":562}}}],"partialFingerprints":{"codehealthFindingId/v1":"f79f8fb1c65722ab8770387e701c78f18bc63cb232847b1e8e974793849c88e2"}},{"ruleId":"D1","level":"warning","message":{"text":"config.validateCustomCommands (cyclomatic 16): config.validateCustomCommands has cyclomatic complexity 16 (threshold 15). To reduce it, separate the cases: extract each independent branch into its own named function, and where the body has guards that only reject input, fold those into early returns at the top."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/config/user_config_validation.go"},"region":{"startLine":194}}}],"partialFingerprints":{"codehealthFindingId/v1":"477af2ca2c72c47b15603ea1c54beabbb100e29fcf4292ba8dc1297abc30f9fa"}},{"ruleId":"D1","level":"warning","message":{"text":"StagingHelper.RefreshStagingPanel (cyclomatic 16): StagingHelper.RefreshStagingPanel has cyclomatic complexity 16 (threshold 15). To reduce it, split the body: these branches sit side by side rather than nested inside one another, so extracting each one on its own would leave a function per branch. Group the statements between the checks into named steps and move each step into its own function, so the body reads as a short sequence of named stages."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gui/controllers/helpers/staging_helper.go"},"region":{"startLine":22}}}],"partialFingerprints":{"codehealthFindingId/v1":"8a14a1791df87e48724ab2aed9d8563e93c7c0179d58b32d5280091f179463f6"}},{"ruleId":"D1","level":"warning","message":{"text":"Gui.configureViewProperties (cyclomatic 16): Gui.configureViewProperties has cyclomatic complexity 16 (threshold 15). Of this number, 13 points are the body\u0027s own statements and 3 belong to 2 function literals inside it that branch. To reduce it, break up the iteration: give each loop body a named function, and split a multi-phase loop into one function per phase so no single body carries the whole pipeline."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gui/views.go"},"region":{"startLine":158}}}],"partialFingerprints":{"codehealthFindingId/v1":"7524ed64510e95d99618e423f7f9886d617a0e76859fad30aacfd9d1385c5bf1"}},{"ruleId":"D1","level":"warning","message":{"text":"utils.WrapViewLinesToWidth (cyclomatic 16): utils.WrapViewLinesToWidth has cyclomatic complexity 16 (threshold 15). To reduce it, separate the cases: extract each independent branch into its own named function, and where the body has guards that only reject input, fold those into early returns at the top."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/utils/lines.go"},"region":{"startLine":112}}}],"partialFingerprints":{"codehealthFindingId/v1":"6a3d2b81aa47066eb05b629b30bc901a6f2f379d54036324afab9d7aa51a88c7"}},{"ruleId":"D2","level":"warning","message":{"text":"Gui.onKey (cognitive 94): Gui.onKey has cognitive complexity 94 (threshold 15). Drivers by points: if/else 78, loops 8, boolean chains 7, match/switch 1 (nesting depth added 54). This file\u0027s own header attributes it to another copyright holder, so it is code this repository carries rather than code it wrote: restructuring the body in place forks it from upstream and turns every future re-sync into a manual merge. The performable moves are to leave the body as close to its upstream form as possible and keep it behind a narrow interface of your own, and to re-sync it when upstream changes \u2014 or, if it has already diverged far enough that you maintain it here, adopt it deliberately and then split the body into named stages."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gocui/gui.go"},"region":{"startLine":1663}}}],"partialFingerprints":{"codehealthFindingId/v1":"4537ab35a17a15e31964f33bade1818c933f0901a4aaf159e5a02ab577d53502"}},{"ruleId":"D2","level":"warning","message":{"text":"ViewBufferManager.NewCmdTask (cognitive 61): ViewBufferManager.NewCmdTask has cognitive complexity 61 (threshold 15). Drivers by points: if/else 36, match/switch 13, jumps 5, loops 5, boolean chains 2 (nesting depth added 29). Most of this is not in the body itself: 0 of the 61 points are its own statements and the rest belongs to one function literal inside it that branches (line 168). The decisions are inside the literal, which nothing outside this body can call, review or test on its own, so splitting the enclosing body is not the move available here. To reduce it, lift the literal\u0027s work into a named function or method at the enclosing scope and have the literal call it, then reduce whichever part then reads as the largest."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/tasks/tasks.go"},"region":{"startLine":167}}}],"partialFingerprints":{"codehealthFindingId/v1":"c17a957aaaffb1de05a62794d53d0e093b5a628005798fd2a9d4f5e0f1133585"}},{"ruleId":"D2","level":"warning","message":{"text":"Gui.layout (cognitive 60): Gui.layout has cognitive complexity 60 (threshold 15). Drivers by points: if/else 46, boolean chains 7, loops 4, match/switch 2, jumps 1 (nesting depth added 18). Of this number, 45 points are the body\u0027s own statements and 15 belong to one function literal inside it that branches. To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gui/layout.go"},"region":{"startLine":13}}}],"partialFingerprints":{"codehealthFindingId/v1":"20fa9885add7122732ade417a5e2294b1f65ae2e316ba858f5770f03e585fdb0"}},{"ruleId":"D2","level":"warning","message":{"text":"escapeInterpreter.parseOne (cognitive 52): escapeInterpreter.parseOne has cognitive complexity 52 (threshold 15). Drivers by points: if/else 29, boolean chains 12, match/switch 11 (nesting depth added 21). This file\u0027s own header attributes it to another copyright holder, so it is code this repository carries rather than code it wrote: restructuring the body in place forks it from upstream and turns every future re-sync into a manual merge. The performable moves are to leave the body as close to its upstream form as possible and keep it behind a narrow interface of your own, and to re-sync it when upstream changes \u2014 or, if it has already diverged far enough that you maintain it here, adopt it deliberately and then split the body into named stages."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gocui/escape.go"},"region":{"startLine":237}}}],"partialFingerprints":{"codehealthFindingId/v1":"6280ecf5b2a9bdee0fde5c7390106728b02dfb1b09de8590065d653d5292b71b"}},{"ruleId":"D2","level":"warning","message":{"text":"presentation.getBranchDisplayStrings (cognitive 50): presentation.getBranchDisplayStrings has cognitive complexity 50 (threshold 15). Drivers by points: if/else 47, boolean chains 3 (nesting depth added 16). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gui/presentation/branches.go"},"region":{"startLine":47}}}],"partialFingerprints":{"codehealthFindingId/v1":"c3e59133f497de78dc68db5268942d50404d1d6ed12f41af5c8f902928bd0479"}},{"ruleId":"D2","level":"warning","message":{"text":"gocui.gocuiEventFromTcellEvent (cognitive 45): gocui.gocuiEventFromTcellEvent has cognitive complexity 45 (threshold 15). Drivers by points: if/else 27, match/switch 13, boolean chains 5 (nesting depth added 24). This file\u0027s own header attributes it to another copyright holder, so it is code this repository carries rather than code it wrote: restructuring the body in place forks it from upstream and turns every future re-sync into a manual merge. The performable moves are to leave the body as close to its upstream form as possible and keep it behind a narrow interface of your own, and to re-sync it when upstream changes \u2014 or, if it has already diverged far enough that you maintain it here, adopt it deliberately and then split the body into named stages."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gocui/tcell_driver.go"},"region":{"startLine":323}}}],"partialFingerprints":{"codehealthFindingId/v1":"b39758193d4fc6a49a8b29d4193496735bdfe76dcde0568fc44d87f02d36e3a7"}},{"ruleId":"D2","level":"warning","message":{"text":"Gui.draw (cognitive 45): Gui.draw has cognitive complexity 45 (threshold 15). Drivers by points: if/else 40, boolean chains 5 (nesting depth added 19). This file\u0027s own header attributes it to another copyright holder, so it is code this repository carries rather than code it wrote: restructuring the body in place forks it from upstream and turns every future re-sync into a manual merge. The performable moves are to leave the body as close to its upstream form as possible and keep it behind a narrow interface of your own, and to re-sync it when upstream changes \u2014 or, if it has already diverged far enough that you maintain it here, adopt it deliberately and then split the body into named stages."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gocui/gui.go"},"region":{"startLine":1593}}}],"partialFingerprints":{"codehealthFindingId/v1":"90f042fdce5a24c6edfe2db0b65d75180688ae847046fdec8a06245eb060a200"}},{"ruleId":"D2","level":"warning","message":{"text":"RefreshHelper.performRefresh (cognitive 45): RefreshHelper.performRefresh has cognitive complexity 45 (threshold 15). Drivers by points: if/else 34, boolean chains 9, loops 2 (nesting depth added 2). Of this number, 40 points are the body\u0027s own statements and 5 belong to one function literal inside it that branches. To reduce it, split the body: this score is breadth rather than depth \u2014 many checks laid out side by side rather than nested inside one another, so inverting conditions into early returns has nothing left to flatten. Group the statements between the checks into named steps and move each step into its own function, so the body reads as a short sequence of named stages."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gui/controllers/helpers/refresh_helper.go"},"region":{"startLine":168}}}],"partialFingerprints":{"codehealthFindingId/v1":"ac89775bcededacc2b4171d21d4b28184a99983c3c3e07748de92014a244baac"}},{"ruleId":"D2","level":"warning","message":{"text":"View.updateSearchPositions (cognitive 44): View.updateSearchPositions has cognitive complexity 44 (threshold 15). Drivers by points: if/else 33, loops 11 (nesting depth added 27). Of this number, 28 points are the body\u0027s own statements and 16 belong to one function literal inside it that branches. This file\u0027s own header attributes it to another copyright holder, so it is code this repository carries rather than code it wrote: restructuring the body in place forks it from upstream and turns every future re-sync into a manual merge. The performable moves are to leave the body as close to its upstream form as possible and keep it behind a narrow interface of your own, and to re-sync it when upstream changes \u2014 or, if it has already diverged far enough that you maintain it here, adopt it deliberately and then split the body into named stages."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gocui/view.go"},"region":{"startLine":1233}}}],"partialFingerprints":{"codehealthFindingId/v1":"d0f0e2b3b3d1ed0bf23d668121c2a34dc961ab1923ae3f1d5d57f40d0bd19816"}},{"ruleId":"D2","level":"warning","message":{"text":"helpers.shrinkToContentSidePanelBoxes (cognitive 42): helpers.shrinkToContentSidePanelBoxes has cognitive complexity 42 (threshold 15). Drivers by points: if/else 24, loops 13, boolean chains 5 (nesting depth added 16). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gui/controllers/helpers/window_arrangement_helper.go"},"region":{"startLine":547}}}],"partialFingerprints":{"codehealthFindingId/v1":"19bff15bed474f7285a172cd59a672e329d343c64cc57f95f95ed46fa6efe233"}},{"ruleId":"D2","level":"warning","message":{"text":"graph.getNextPipes (cognitive 40): graph.getNextPipes has cognitive complexity 40 (threshold 15). Drivers by points: if/else 24, loops 13, boolean chains 3 (nesting depth added 13). Of this number, 30 points are the body\u0027s own statements and 10 belong to 4 function literals inside it that branch. To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gui/presentation/graph/graph.go"},"region":{"startLine":109}}}],"partialFingerprints":{"codehealthFindingId/v1":"35a133ddc1777477712ade2af506c2f2ab14580789710a70ebd6fafe56a6c778"}},{"ruleId":"D2","level":"warning","message":{"text":"Gui.execKeybindings (cognitive 39): Gui.execKeybindings has cognitive complexity 39 (threshold 15). Drivers by points: if/else 33, boolean chains 5, loops 1 (nesting depth added 15). This file\u0027s own header attributes it to another copyright holder, so it is code this repository carries rather than code it wrote: restructuring the body in place forks it from upstream and turns every future re-sync into a manual merge. The performable moves are to leave the body as close to its upstream form as possible and keep it behind a narrow interface of your own, and to re-sync it when upstream changes \u2014 or, if it has already diverged far enough that you maintain it here, adopt it deliberately and then split the body into named stages."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gocui/gui.go"},"region":{"startLine":1915}}}],"partialFingerprints":{"codehealthFindingId/v1":"0bc071ce23b336acbfdf5b65d5f9373fbbb8a7572f6b0351ef0495606fe08481"}},{"ruleId":"D2","level":"warning","message":{"text":"graph.renderPipeSet (cognitive 39): graph.renderPipeSet has cognitive complexity 39 (threshold 15). Drivers by points: if/else 19, loops 12, boolean chains 8 (nesting depth added 10). Of this number, 31 points are the body\u0027s own statements and 8 belong to 2 function literals inside it that branch. To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gui/presentation/graph/graph.go"},"region":{"startLine":275}}}],"partialFingerprints":{"codehealthFindingId/v1":"4ef7ade32641a04ee69642bad02d29926f498e27f4b3cdaebc53f9364e2eec33"}},{"ruleId":"D2","level":"warning","message":{"text":"RefreshHelper.refreshStateFiles (cognitive 38): RefreshHelper.refreshStateFiles has cognitive complexity 38 (threshold 15). Drivers by points: if/else 31, boolean chains 4, loops 3 (nesting depth added 16). Of this number, 29 points are the body\u0027s own statements and 9 belong to 2 function literals inside it that branch. To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gui/controllers/helpers/refresh_helper.go"},"region":{"startLine":1286}}}],"partialFingerprints":{"codehealthFindingId/v1":"315030df44a61820e89a61548136a00d2fa1be35206b7d75a86e8dbdb2a5a6d4"}},{"ruleId":"D2","level":"warning","message":{"text":"config.KeyFromLabel (cognitive 36): config.KeyFromLabel has cognitive complexity 36 (threshold 15). Drivers by points: if/else 29, boolean chains 4, match/switch 2, loops 1 (nesting depth added 13). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/config/keynames.go"},"region":{"startLine":95}}}],"partialFingerprints":{"codehealthFindingId/v1":"dcec601bef65009bd8b05da01f7627e4b2c90c1aff310cb8b896412e5c5bf1ae"}},{"ruleId":"D2","level":"warning","message":{"text":"View.draw (cognitive 36): View.draw has cognitive complexity 36 (threshold 15). Drivers by points: if/else 31, loops 3, boolean chains 2 (nesting depth added 17). This file\u0027s own header attributes it to another copyright holder, so it is code this repository carries rather than code it wrote: restructuring the body in place forks it from upstream and turns every future re-sync into a manual merge. The performable moves are to leave the body as close to its upstream form as possible and keep it behind a narrow interface of your own, and to re-sync it when upstream changes \u2014 or, if it has already diverged far enough that you maintain it here, adopt it deliberately and then split the body into named stages."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gocui/view.go"},"region":{"startLine":1322}}}],"partialFingerprints":{"codehealthFindingId/v1":"5aa4d311a4d926d7fe7ed8deffc5874be5b8bcc4896825d2ecfb3f6a8e27176e"}},{"ruleId":"D2","level":"warning","message":{"text":"app.Start (cognitive 35): app.Start has cognitive complexity 35 (threshold 15). Drivers by points: if/else 33, boolean chains 2 (nesting depth added 9). Of this number, 33 points are the body\u0027s own statements and 2 belong to one function literal inside it that branches. To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/app/entry_point.go"},"region":{"startLine":53}}}],"partialFingerprints":{"codehealthFindingId/v1":"261baf75fb0b31d83c20a5853c0e3c9876ff22774063abfdf26e0cb169e3815d"}},{"ruleId":"D2","level":"warning","message":{"text":"Gui.drawTitle (cognitive 35): Gui.drawTitle has cognitive complexity 35 (threshold 15). Drivers by points: if/else 27, boolean chains 4, loops 4 (nesting depth added 13). This file\u0027s own header attributes it to another copyright holder, so it is code this repository carries rather than code it wrote: restructuring the body in place forks it from upstream and turns every future re-sync into a manual merge. The performable moves are to leave the body as close to its upstream form as possible and keep it behind a narrow interface of your own, and to re-sync it when upstream changes \u2014 or, if it has already diverged far enough that you maintain it here, adopt it deliberately and then split the body into named stages."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gocui/gui.go"},"region":{"startLine":1374}}}],"partialFingerprints":{"codehealthFindingId/v1":"da41061ae88dd2892b9b558e46ac87210eda946bf4ed0ba5ea9bd6819c10931c"}},{"ruleId":"D2","level":"warning","message":{"text":"presentation.GetCommitListDisplayStrings (cognitive 34): presentation.GetCommitListDisplayStrings has cognitive complexity 34 (threshold 15). Drivers by points: if/else 29, boolean chains 4, loops 1 (nesting depth added 16). Of this number, 29 points are the body\u0027s own statements and 5 belong to 2 function literals inside it that branch. To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gui/presentation/commits.go"},"region":{"startLine":40}}}],"partialFingerprints":{"codehealthFindingId/v1":"0f0c2065bb072774b2da050d3e439eecab9b1b043ada80e4ea813653db890f7b"}},{"ruleId":"D2","level":"warning","message":{"text":"Gui.drawFrameEdges (cognitive 33): Gui.drawFrameEdges has cognitive complexity 33 (threshold 15). Drivers by points: if/else 25, boolean chains 6, loops 2 (nesting depth added 14). This file\u0027s own header attributes it to another copyright holder, so it is code this repository carries rather than code it wrote: restructuring the body in place forks it from upstream and turns every future re-sync into a manual merge. The performable moves are to leave the body as close to its upstream form as possible and keep it behind a narrow interface of your own, and to re-sync it when upstream changes \u2014 or, if it has already diverged far enough that you maintain it here, adopt it deliberately and then split the body into named stages."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gocui/gui.go"},"region":{"startLine":1194}}}],"partialFingerprints":{"codehealthFindingId/v1":"57155960f7ba8483931bfb80690c23447c96e72cc6782e3fe88c37d823d210d7"}},{"ruleId":"D2","level":"warning","message":{"text":"graph.getBoxDrawingChars (cognitive 32): graph.getBoxDrawingChars has cognitive complexity 32 (threshold 15). Drivers by points: boolean chains 16, if/else 16. To reduce it, name the conditions: bind each compound test to a well-named local or a small predicate function, so the body reads as a sequence of named decisions rather than a chain of operators."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gui/presentation/graph/cell.go"},"region":{"startLine":147}}}],"partialFingerprints":{"codehealthFindingId/v1":"fd9f32ba79cdf7e361d57ad4bc6baf50fa967640d18287b0e6a853b58dff41d4"}},{"ruleId":"D2","level":"warning","message":{"text":"config.loadUserConfig (cognitive 31): config.loadUserConfig has cognitive complexity 31 (threshold 15). Drivers by points: if/else 27, match/switch 3, loops 1 (nesting depth added 19). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/config/app_config.go"},"region":{"startLine":163}}}],"partialFingerprints":{"codehealthFindingId/v1":"79a1286d0f13e8cf5b50275a2c28ccd3981da01b1e5be7895c2c31010e9255bb"}},{"ruleId":"D2","level":"warning","message":{"text":"utils.WrapViewLinesToWidth (cognitive 31): utils.WrapViewLinesToWidth has cognitive complexity 31 (threshold 15). Drivers by points: if/else 23, loops 7, boolean chains 1 (nesting depth added 14). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/utils/lines.go"},"region":{"startLine":112}}}],"partialFingerprints":{"codehealthFindingId/v1":"0b9698795c655d1f17f4f37b855518708f942e13aa60876b0ae92e4d586b281e"}},{"ruleId":"D2","level":"warning","message":{"text":"App.setupRepo (cognitive 30): App.setupRepo has cognitive complexity 30 (threshold 15). Drivers by points: if/else 28, match/switch 2 (nesting depth added 16). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/app/app.go"},"region":{"startLine":194}}}],"partialFingerprints":{"codehealthFindingId/v1":"e806bb37a2c332ef9d3802dbccd735e751ea60c6f6c7e92bf6fe2528dca1be5e"}},{"ruleId":"D2","level":"warning","message":{"text":"View.write (cognitive 30): View.write has cognitive complexity 30 (threshold 15). Drivers by points: if/else 17, loops 9, boolean chains 2, match/switch 2 (nesting depth added 15). Of this number, 29 points are the body\u0027s own statements and 1 belongs to one function literal inside it that branches. This file\u0027s own header attributes it to another copyright holder, so it is code this repository carries rather than code it wrote: restructuring the body in place forks it from upstream and turns every future re-sync into a manual merge. The performable moves are to leave the body as close to its upstream form as possible and keep it behind a narrow interface of your own, and to re-sync it when upstream changes \u2014 or, if it has already diverged far enough that you maintain it here, adopt it deliberately and then split the body into named stages."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gocui/view.go"},"region":{"startLine":856}}}],"partialFingerprints":{"codehealthFindingId/v1":"e6be9dee0c1671842b625cdb9d33811fbd8bb2ac4f95a39bba8077bae1dbaaf9"}},{"ruleId":"D2","level":"warning","message":{"text":"FixupHelper.blameAddedLines (cognitive 30): FixupHelper.blameAddedLines has cognitive complexity 30 (threshold 15). Drivers by points: if/else 27, loops 2, boolean chains 1 (nesting depth added 13). Of this number, 17 points are the body\u0027s own statements and 13 belong to one function literal inside it that branches. To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gui/controllers/helpers/fixup_helper.go"},"region":{"startLine":300}}}],"partialFingerprints":{"codehealthFindingId/v1":"abdaa907f00210a8203170cf8491b46ead0b75848ad4a95400f418b620dd1433"}},{"ruleId":"D2","level":"warning","message":{"text":"WorktreeLoader.GetWorktrees (cognitive 29): WorktreeLoader.GetWorktrees has cognitive complexity 29 (threshold 15). Drivers by points: if/else 23, loops 5, boolean chains 1 (nesting depth added 10). Of this number, 25 points are the body\u0027s own statements and 4 belong to one function literal inside it that branches. To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/commands/git_commands/worktree_loader.go"},"region":{"startLine":24}}}],"partialFingerprints":{"codehealthFindingId/v1":"b457895b871719c855765f71d15a101d1fffe8f8736f351bde63c569b7d2fe45"}},{"ruleId":"D2","level":"warning","message":{"text":"Gui.onNewRepo (cognitive 29): Gui.onNewRepo has cognitive complexity 29 (threshold 15). Drivers by points: if/else 28, boolean chains 1 (nesting depth added 10). Most of this is not in the body itself: 7 of the 29 points are its own statements and the rest belongs to 4 function literals inside it that branch (lines 373, 400, 427, \u2026). The decisions are inside those literals, which nothing outside this body can call, review or test on its own, so splitting the enclosing body is not the move available here. To reduce it, lift the literals\u0027 work into a named function or method at the enclosing scope and have each literal call it, then reduce whichever part then reads as the largest."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gui/gui.go"},"region":{"startLine":342}}}],"partialFingerprints":{"codehealthFindingId/v1":"ded16880b08e326125055df0894c2616df0ecde00a70fdfaab40fc41e02abb2f"}},{"ruleId":"D2","level":"warning","message":{"text":"BranchLoader.Load (cognitive 28): BranchLoader.Load has cognitive complexity 28 (threshold 15). Drivers by points: if/else 19, loops 7, boolean chains 1, jumps 1 (nesting depth added 13). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/commands/git_commands/branch_loader.go"},"region":{"startLine":67}}}],"partialFingerprints":{"codehealthFindingId/v1":"3b3507bed6d34f5cee45b6ab4958da6c66e4f51b9d6e9ecd62b61a2ef570eb95"}},{"ruleId":"D2","level":"warning","message":{"text":"config.validateCustomCommands (cognitive 27): config.validateCustomCommands has cognitive complexity 27 (threshold 15). Drivers by points: if/else 22, loops 4, boolean chains 1 (nesting depth added 16). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/config/user_config_validation.go"},"region":{"startLine":194}}}],"partialFingerprints":{"codehealthFindingId/v1":"a198451b2e1aa3bdeef7a948b30098aa2c07d3b6eddfeab8120d0487f475b3a8"}},{"ruleId":"D2","level":"warning","message":{"text":"presentation.displayCommit (cognitive 26): presentation.displayCommit has cognitive complexity 26 (threshold 15). Drivers by points: if/else 23, boolean chains 3 (nesting depth added 4). To reduce it, split the body: most of this score is breadth rather than depth \u2014 checks laid out side by side rather than stacked \u2014 so group the statements between the checks into named steps and move each step into its own function. Some of it IS depth: where a check sits inside another whose only job is to reach it, merge the two into one condition, and where an else follows a branch that already returns, drop the trailing else and let the rest of the body continue at one level."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gui/presentation/commits.go"},"region":{"startLine":341}}}],"partialFingerprints":{"codehealthFindingId/v1":"d69a99309d32b4e449a51e55c429da8c61e371fd8fb465ce9212a83720c0f49d"}},{"ruleId":"D2","level":"warning","message":{"text":"HandlerCreator.call (cognitive 26): HandlerCreator.call has cognitive complexity 26 (threshold 15). Drivers by points: if/else 23, match/switch 2, loops 1 (nesting depth added 16). Most of this is not in the body itself: 0 of the 26 points are its own statements and the rest belongs to one function literal inside it that branches (line 48). The decisions are inside the literal, which nothing outside this body can call, review or test on its own, so splitting the enclosing body is not the move available here. To reduce it, lift the literal\u0027s work into a named function or method at the enclosing scope and have the literal call it, then reduce whichever part then reads as the largest."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gui/services/custom_commands/handler_creator.go"},"region":{"startLine":47}}}],"partialFingerprints":{"codehealthFindingId/v1":"a8964be81eef17f59ee00cd673bd6e9e7bfddbfeab1839438e9934ba3e8475c3"}},{"ruleId":"D2","level":"warning","message":{"text":"WorkingTreeCommands.DiscardAllDirChanges (cognitive 25): WorkingTreeCommands.DiscardAllDirChanges has cognitive complexity 25 (threshold 15). Drivers by points: if/else 19, boolean chains 4, loops 2 (nesting depth added 9). Most of this is not in the body itself: 6 of the 25 points are its own statements and the rest belongs to one function literal inside it that branches (line 198). The decisions are inside the literal, which nothing outside this body can call, review or test on its own, so splitting the enclosing body is not the move available here. To reduce it, lift the literal\u0027s work into a named function or method at the enclosing scope and have the literal call it, then reduce whichever part then reads as the largest."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/commands/git_commands/working_tree.go"},"region":{"startLine":190}}}],"partialFingerprints":{"codehealthFindingId/v1":"c5343dfc3fc3a328d57c001389e4260dd0352f33c6dbdca912b4115cd2663f43"}},{"ruleId":"D2","level":"warning","message":{"text":"oscommands.CopyDir (cognitive 25): oscommands.CopyDir has cognitive complexity 25 (threshold 15). Drivers by points: if/else 23, boolean chains 1, loops 1 (nesting depth added 10). This file\u0027s own header attributes it to another copyright holder, so it is code this repository carries rather than code it wrote: restructuring the body in place forks it from upstream and turns every future re-sync into a manual merge. The performable moves are to leave the body as close to its upstream form as possible and keep it behind a narrow interface of your own, and to re-sync it when upstream changes \u2014 or, if it has already diverged far enough that you maintain it here, adopt it deliberately and then split the body into named stages."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/commands/oscommands/copy.go"},"region":{"startLine":80}}}],"partialFingerprints":{"codehealthFindingId/v1":"4109c3ff9efaea423ff64e4f7aaab1fbe5037b78f54589a5399f3ced2f1c1d71"}},{"ruleId":"D2","level":"warning","message":{"text":"config.changeCustomCommandStreamAndOutputToOutputEnum (cognitive 25): config.changeCustomCommandStreamAndOutputToOutputEnum has cognitive complexity 25 (threshold 15). Drivers by points: if/else 21, boolean chains 4 (nesting depth added 10). Most of this is not in the body itself: 0 of the 25 points are its own statements and the rest belongs to one function literal inside it that branches (line 432). The decisions are inside the literal, which nothing outside this body can call, review or test on its own, so splitting the enclosing body is not the move available here. To reduce it, lift the literal\u0027s work into a named function or method at the enclosing scope and have the literal call it, then reduce whichever part then reads as the largest."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/config/app_config.go"},"region":{"startLine":431}}}],"partialFingerprints":{"codehealthFindingId/v1":"459ed7cee6c5bfa502a0ddb797911ba0e06ad36be547ca56ceb311e42b4006ca"}},{"ruleId":"D2","level":"warning","message":{"text":"config.migratePagersToDiffRenderers (cognitive 24): config.migratePagersToDiffRenderers has cognitive complexity 24 (threshold 15). Drivers by points: if/else 19, boolean chains 4, loops 1 (nesting depth added 7). Most of this is not in the body itself: 0 of the 24 points are its own statements and the rest belongs to one function literal inside it that branches (line 563). The decisions are inside the literal, which nothing outside this body can call, review or test on its own, so splitting the enclosing body is not the move available here. To reduce it, lift the literal\u0027s work into a named function or method at the enclosing scope and have the literal call it, then reduce whichever part then reads as the largest."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/config/app_config.go"},"region":{"startLine":562}}}],"partialFingerprints":{"codehealthFindingId/v1":"94b9399d5eff579574019db4d945a4458911f8e9d6e0a1228a8f6870743f4473"}},{"ruleId":"D2","level":"warning","message":{"text":"patchTransformer.transformHunkLines (cognitive 23): patchTransformer.transformHunkLines has cognitive complexity 23 (threshold 15). Drivers by points: if/else 18, boolean chains 4, loops 1 (nesting depth added 10). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/commands/patch/transform.go"},"region":{"startLine":172}}}],"partialFingerprints":{"codehealthFindingId/v1":"eae9b1556096cfe24b2d9e9a83d5895e16232006e40d7722a92eadb10eac6f14"}},{"ruleId":"D2","level":"warning","message":{"text":"View.setCharacter (cognitive 23): View.setCharacter has cognitive complexity 23 (threshold 15). Drivers by points: if/else 20, boolean chains 3 (nesting depth added 7). This file\u0027s own header attributes it to another copyright holder, so it is code this repository carries rather than code it wrote: restructuring the body in place forks it from upstream and turns every future re-sync into a manual merge. The performable moves are to leave the body as close to its upstream form as possible and keep it behind a narrow interface of your own, and to re-sync it when upstream changes \u2014 or, if it has already diverged far enough that you maintain it here, adopt it deliberately and then split the body into named stages."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gocui/view.go"},"region":{"startLine":619}}}],"partialFingerprints":{"codehealthFindingId/v1":"59eb40bd8157a5db1ba58cd2a79292e1a55bad354f4847e006d4d80702a5a7b3"}},{"ruleId":"D2","level":"warning","message":{"text":"UndoController.parseReflogForActions (cognitive 23): UndoController.parseReflogForActions has cognitive complexity 23 (threshold 15). Drivers by points: if/else 21, boolean chains 1, loops 1 (nesting depth added 9). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gui/controllers/undo_controller.go"},"region":{"startLine":200}}}],"partialFingerprints":{"codehealthFindingId/v1":"5b60313bda79cae702de4006b5f2bb178e14ed8e6d8266d027145141e52973a6"}},{"ruleId":"D2","level":"warning","message":{"text":"WorkingTreeCommands.removeEmptyDirs (cognitive 22): WorkingTreeCommands.removeEmptyDirs has cognitive complexity 22 (threshold 15). Drivers by points: if/else 15, loops 5, boolean chains 2 (nesting depth added 12). Of this number, 21 points are the body\u0027s own statements and 1 belongs to one function literal inside it that branches. To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/commands/git_commands/working_tree.go"},"region":{"startLine":294}}}],"partialFingerprints":{"codehealthFindingId/v1":"dd594c03481dc4974f45fabca21d80366e1b8757223b1b152ded7fae8a1715eb"}},{"ruleId":"D2","level":"warning","message":{"text":"config.computeMigratedConfig (cognitive 21): config.computeMigratedConfig has cognitive complexity 21 (threshold 15). Drivers by points: if/else 19, loops 2 (nesting depth added 4). To reduce it, split the body: most of this score is breadth rather than depth \u2014 checks laid out side by side rather than stacked \u2014 so group the statements between the checks into named steps and move each step into its own function. Some of it IS depth: where a check sits inside another whose only job is to reach it, merge the two into one condition."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/config/app_config.go"},"region":{"startLine":277}}}],"partialFingerprints":{"codehealthFindingId/v1":"d6cf2102869212309736bb613ce61c818470bdfc6a97577bb41c4d5469c51ebd"}},{"ruleId":"D2","level":"warning","message":{"text":"gocui.lineWrap (cognitive 21): gocui.lineWrap has cognitive complexity 21 (threshold 15). Drivers by points: if/else 15, loops 5, boolean chains 1 (nesting depth added 9). This file\u0027s own header attributes it to another copyright holder, so it is code this repository carries rather than code it wrote: restructuring the body in place forks it from upstream and turns every future re-sync into a manual merge. The performable moves are to leave the body as close to its upstream form as possible and keep it behind a narrow interface of your own, and to re-sync it when upstream changes \u2014 or, if it has already diverged far enough that you maintain it here, adopt it deliberately and then split the body into named stages."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gocui/view.go"},"region":{"startLine":1686}}}],"partialFingerprints":{"codehealthFindingId/v1":"9a72f238c9114aff19fc64deb46fac2211a7b919ec7f58d3f321141e4a25f390"}},{"ruleId":"D2","level":"warning","message":{"text":"app.layout (cognitive 21): app.layout has cognitive complexity 21 (threshold 15). Drivers by points: if/else 21 (nesting depth added 7). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/integration/clients/tui.go"},"region":{"startLine":275}}}],"partialFingerprints":{"codehealthFindingId/v1":"2cdc5f1085f538e6a032905c4dbb24fdc90029d54ab1ba29f4f527a6618e46ff"}},{"ruleId":"D2","level":"warning","message":{"text":"FileLoader.GetStatusFiles (cognitive 20): FileLoader.GetStatusFiles has cognitive complexity 20 (threshold 15). Drivers by points: if/else 15, loops 4, boolean chains 1 (nesting depth added 8). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/commands/git_commands/file_loader.go"},"region":{"startLine":47}}}],"partialFingerprints":{"codehealthFindingId/v1":"8ae98a3f17a3e06b3410b4bff696c6d7e881b040dfc84fb2d7fd4bfeb97508f4"}},{"ruleId":"D2","level":"warning","message":{"text":"WorkingTreeCommands.DiscardAllFileChanges (cognitive 20): WorkingTreeCommands.DiscardAllFileChanges has cognitive complexity 20 (threshold 15). Drivers by points: if/else 18, boolean chains 2 (nesting depth added 6). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/commands/git_commands/working_tree.go"},"region":{"startLine":123}}}],"partialFingerprints":{"codehealthFindingId/v1":"4f8d24216e1883ba08d9cd5e230bb19f7dbc67d9c8d32048b514ccb8e3945755"}},{"ruleId":"D2","level":"warning","message":{"text":"cmdObjRunner.processOutput (cognitive 20): cmdObjRunner.processOutput has cognitive complexity 20 (threshold 15). Drivers by points: if/else 19, loops 1 (nesting depth added 12). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/commands/oscommands/cmd_obj_runner.go"},"region":{"startLine":382}}}],"partialFingerprints":{"codehealthFindingId/v1":"d36d342d47ff7dd09150be27be31391cf518da3a5f502044a5c851d155d6c7b6"}},{"ruleId":"D2","level":"warning","message":{"text":"config.validateKeybindingsRecurse (cognitive 20): config.validateKeybindingsRecurse has cognitive complexity 20 (threshold 15). Drivers by points: if/else 16, loops 4 (nesting depth added 9). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/config/user_config_validation.go"},"region":{"startLine":146}}}],"partialFingerprints":{"codehealthFindingId/v1":"39f6fc0b49754279fc24593b60f75edb54533b995348aa1b5727b0f78b39e3a3"}},{"ruleId":"D2","level":"warning","message":{"text":"gocui.contentToCells (cognitive 20): gocui.contentToCells has cognitive complexity 20 (threshold 15). Drivers by points: if/else 11, loops 6, boolean chains 3 (nesting depth added 8). Of this number, 19 points are the body\u0027s own statements and 1 belongs to one function literal inside it that branches. To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gocui/text_area.go"},"region":{"startLine":63}}}],"partialFingerprints":{"codehealthFindingId/v1":"0f71738fe0d4292d9f5e7cccbd7230ade8dbb2bdf3aac774ed023f3ddc1f9d7c"}},{"ruleId":"D2","level":"warning","message":{"text":"View.refreshViewLinesIfNeeded (cognitive 20): View.refreshViewLinesIfNeeded has cognitive complexity 20 (threshold 15). Drivers by points: if/else 16, loops 3, boolean chains 1 (nesting depth added 9). This file\u0027s own header attributes it to another copyright holder, so it is code this repository carries rather than code it wrote: restructuring the body in place forks it from upstream and turns every future re-sync into a manual merge. The performable moves are to leave the body as close to its upstream form as possible and keep it behind a narrow interface of your own, and to re-sync it when upstream changes \u2014 or, if it has already diverged far enough that you maintain it here, adopt it deliberately and then split the body into named stages."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gocui/view.go"},"region":{"startLine":1420}}}],"partialFingerprints":{"codehealthFindingId/v1":"af5b05bf257ae80cde5b918ee338c4cca989936141cf7d05364df57d04d991ee"}},{"ruleId":"D2","level":"warning","message":{"text":"ContextMgr.pushToContextStack (cognitive 20): ContextMgr.pushToContextStack has cognitive complexity 20 (threshold 15). Drivers by points: if/else 14, boolean chains 4, loops 2 (nesting depth added 6). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gui/context.go"},"region":{"startLine":76}}}],"partialFingerprints":{"codehealthFindingId/v1":"ce0921820a0478fdbd5968df15e14a9dd415c13b9e496fcbd5a7f35b9287cb3e"}},{"ruleId":"D2","level":"warning","message":{"text":"helpers.infoSectionChildren (cognitive 20): helpers.infoSectionChildren has cognitive complexity 20 (threshold 15). Drivers by points: if/else 16, boolean chains 3, loops 1 (nesting depth added 3). Of this number, 15 points are the body\u0027s own statements and 5 belong to 3 function literals inside it that branch. To reduce it, split the body: most of this score is breadth rather than depth \u2014 checks laid out side by side rather than stacked \u2014 so group the statements between the checks into named steps and move each step into its own function. Some of it IS depth: where a check sits inside another whose only job is to reach it, merge the two into one condition, and where an else follows a branch that already returns, drop the trailing else and let the rest of the body continue at one level."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gui/controllers/helpers/window_arrangement_helper.go"},"region":{"startLine":280}}}],"partialFingerprints":{"codehealthFindingId/v1":"55ded62efa80256acf557e1ae724d3bac251432b90177483b01cd40f9a80f018"}},{"ruleId":"D2","level":"warning","message":{"text":"ViewDriver.assertLines (cognitive 20): ViewDriver.assertLines has cognitive complexity 20 (threshold 15). Drivers by points: if/else 14, boolean chains 4, loops 2 (nesting depth added 8). Of this number, 13 points are the body\u0027s own statements and 7 belong to one function literal inside it that branches. To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/integration/components/view_driver.go"},"region":{"startLine":230}}}],"partialFingerprints":{"codehealthFindingId/v1":"d45debf35e44c28ab716d1df22bac55cf4c058d018ccf60188eb38882b35384d"}},{"ruleId":"D2","level":"warning","message":{"text":"SubmoduleCommands.GetConfigs (cognitive 19): SubmoduleCommands.GetConfigs has cognitive complexity 19 (threshold 15). Drivers by points: if/else 18, loops 1 (nesting depth added 8). Of this number, 18 points are the body\u0027s own statements and 1 belongs to one function literal inside it that branches. To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/commands/git_commands/submodule.go"},"region":{"startLine":30}}}],"partialFingerprints":{"codehealthFindingId/v1":"ad3afb4ff31302c4033b1ff844d8276431d9925804a050eebbd9009d5d22a8ab"}},{"ruleId":"D2","level":"warning","message":{"text":"cmdObjRunner.runAndStreamAux (cognitive 19): cmdObjRunner.runAndStreamAux has cognitive complexity 19 (threshold 15). Drivers by points: if/else 19 (nesting depth added 5). Of this number, 18 points are the body\u0027s own statements and 1 belongs to one function literal inside it that branches. To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/commands/oscommands/cmd_obj_runner.go"},"region":{"startLine":234}}}],"partialFingerprints":{"codehealthFindingId/v1":"dcc781ab6abdad654bafaec7ef8dd906ca494a6b1fa6f5d0471d483fc9d82d2d"}},{"ruleId":"D2","level":"warning","message":{"text":"Gui.drawFrameCorners (cognitive 19): Gui.drawFrameCorners has cognitive complexity 19 (threshold 15). Drivers by points: if/else 16, boolean chains 2, loops 1 (nesting depth added 8). This file\u0027s own header attributes it to another copyright holder, so it is code this repository carries rather than code it wrote: restructuring the body in place forks it from upstream and turns every future re-sync into a manual merge. The performable moves are to leave the body as close to its upstream form as possible and keep it behind a narrow interface of your own, and to re-sync it when upstream changes \u2014 or, if it has already diverged far enough that you maintain it here, adopt it deliberately and then split the body into named stages."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gocui/gui.go"},"region":{"startLine":1334}}}],"partialFingerprints":{"codehealthFindingId/v1":"94181bf1f5a8a15946de01085553ceb753f9a0cafb984f4db4da41a498c531eb"}},{"ruleId":"D2","level":"warning","message":{"text":"filetree.BuildTreeFromCommitFiles (cognitive 19): filetree.BuildTreeFromCommitFiles has cognitive complexity 19 (threshold 15). Drivers by points: if/else 10, loops 6, jumps 2, boolean chains 1 (nesting depth added 10). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gui/filetree/build_tree.go"},"region":{"startLine":81}}}],"partialFingerprints":{"codehealthFindingId/v1":"8f3765e1f06202b93a35e62b4c438d1bc0767a610056e2b5480570f27c3ddd0b"}},{"ruleId":"D2","level":"warning","message":{"text":"patch_exploring.NewState (cognitive 19): patch_exploring.NewState has cognitive complexity 19 (threshold 15). Drivers by points: if/else 12, boolean chains 4, loops 3 (nesting depth added 4). To reduce it, split the body: most of this score is breadth rather than depth \u2014 checks laid out side by side rather than stacked \u2014 so group the statements between the checks into named steps and move each step into its own function. Some of it IS depth: where a check sits inside another whose only job is to reach it, merge the two into one condition, and where an else follows a branch that already returns, drop the trailing else and let the rest of the body continue at one level."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gui/patch_exploring/state.go"},"region":{"startLine":48}}}],"partialFingerprints":{"codehealthFindingId/v1":"212af2f7302e8400a168e859bb931dce7687f5f13154cb0bb42f7e7ae5715b7d"}},{"ruleId":"D2","level":"warning","message":{"text":"config.LabelForKey (cognitive 18): config.LabelForKey has cognitive complexity 18 (threshold 15). Drivers by points: if/else 16, boolean chains 2 (nesting depth added 2). To reduce it, split the body: most of this score is breadth rather than depth \u2014 checks laid out side by side rather than stacked \u2014 so group the statements between the checks into named steps and move each step into its own function. Some of it IS depth: where a check sits inside another whose only job is to reach it, merge the two into one condition, and where an else follows a branch that already returns, drop the trailing else and let the rest of the body continue at one level."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/config/keynames.go"},"region":{"startLine":50}}}],"partialFingerprints":{"codehealthFindingId/v1":"bbd1edfa9d8991570bdf47d5235931ca976e8d898bf6f6041262bb6dc53f18c9"}},{"ruleId":"D2","level":"warning","message":{"text":"escapeInterpreter.csiColor (cognitive 18): escapeInterpreter.csiColor has cognitive complexity 18 (threshold 15). Drivers by points: if/else 17, match/switch 1 (nesting depth added 8). This file\u0027s own header attributes it to another copyright holder, so it is code this repository carries rather than code it wrote: restructuring the body in place forks it from upstream and turns every future re-sync into a manual merge. The performable moves are to leave the body as close to its upstream form as possible and keep it behind a narrow interface of your own, and to re-sync it when upstream changes \u2014 or, if it has already diverged far enough that you maintain it here, adopt it deliberately and then split the body into named stages."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gocui/escape.go"},"region":{"startLine":542}}}],"partialFingerprints":{"codehealthFindingId/v1":"e7f70b4c18164b1b6f224b253de8c42c9bf82f8fca214897aa7f864a5bb01742"}},{"ruleId":"D2","level":"warning","message":{"text":"helpers.sidePanelChildren (cognitive 18): helpers.sidePanelChildren has cognitive complexity 18 (threshold 15). Drivers by points: if/else 13, boolean chains 2, match/switch 2, loops 1 (nesting depth added 6). Most of this is not in the body itself: 0 of the 18 points are its own statements and the rest belongs to one function literal inside it that branches (line 436). The decisions are inside the literal, which nothing outside this body can call, review or test on its own, so splitting the enclosing body is not the move available here. To reduce it, lift the literal\u0027s work into a named function or method at the enclosing scope and have the literal call it, then reduce whichever part then reads as the largest."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gui/controllers/helpers/window_arrangement_helper.go"},"region":{"startLine":435}}}],"partialFingerprints":{"codehealthFindingId/v1":"8ff19a2442a7a7d47ae5f0fa97d2ffda2c4d8993d2a15f619098dc9b17fafc6e"}},{"ruleId":"D2","level":"warning","message":{"text":"ReposHelper.getCurrentBranch (cognitive 18): ReposHelper.getCurrentBranch has cognitive complexity 18 (threshold 15). Drivers by points: if/else 18 (nesting depth added 9). Of this number, 14 points are the body\u0027s own statements and 4 belong to one function literal inside it that branches. To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gui/controllers/helpers/repos_helper.go"},"region":{"startLine":62}}}],"partialFingerprints":{"codehealthFindingId/v1":"f60d26c5555fb57f356c909934ae7093d80208f0e6e10377f691bea84a2314f9"}},{"ruleId":"D2","level":"warning","message":{"text":"StagingHelper.RefreshStagingPanel (cognitive 18): StagingHelper.RefreshStagingPanel has cognitive complexity 18 (threshold 15). Drivers by points: if/else 12, boolean chains 6 (nesting depth added 1). To reduce it, split the body: this score is breadth rather than depth \u2014 many checks laid out side by side rather than nested inside one another, so inverting conditions into early returns has nothing left to flatten. Group the statements between the checks into named steps and move each step into its own function, so the body reads as a short sequence of named stages."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gui/controllers/helpers/staging_helper.go"},"region":{"startLine":22}}}],"partialFingerprints":{"codehealthFindingId/v1":"f9051093b4ef995f85208d15cc11a1dc992d1f259d552d75b12fcdedbcd64563"}},{"ruleId":"D2","level":"warning","message":{"text":"SuggestionsHelper.GetFilePathSuggestionsFunc (cognitive 18): SuggestionsHelper.GetFilePathSuggestionsFunc has cognitive complexity 18 (threshold 15). Drivers by points: if/else 13, loops 5 (nesting depth added 9). Most of this is not in the body itself: 0 of the 18 points are its own statements and the rest belongs to 2 function literals inside it that branch (lines 122, 157). The decisions are inside those literals, which nothing outside this body can call, review or test on its own, so splitting the enclosing body is not the move available here. To reduce it, lift the literals\u0027 work into a named function or method at the enclosing scope and have each literal call it, then reduce whichever part then reads as the largest."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gui/controllers/helpers/suggestions_helper.go"},"region":{"startLine":121}}}],"partialFingerprints":{"codehealthFindingId/v1":"f449bc8ac99e426598a9df35ab15a892bf0d85f7d5556717b24d1945b4b672e4"}},{"ruleId":"D2","level":"warning","message":{"text":"filetree.BuildTreeFromFiles (cognitive 18): filetree.BuildTreeFromFiles has cognitive complexity 18 (threshold 15). Drivers by points: if/else 12, loops 3, jumps 2, boolean chains 1 (nesting depth added 9). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gui/filetree/build_tree.go"},"region":{"startLine":10}}}],"partialFingerprints":{"codehealthFindingId/v1":"ef5593edacde42fe481f18cb7beec771685ff562f4b6516a76e86eb21067fba0"}},{"ruleId":"D2","level":"warning","message":{"text":"Client.showCustomCommandsMenu (cognitive 18): Client.showCustomCommandsMenu has cognitive complexity 18 (threshold 15). Drivers by points: if/else 15, boolean chains 2, loops 1 (nesting depth added 9). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gui/services/custom_commands/client.go"},"region":{"startLine":66}}}],"partialFingerprints":{"codehealthFindingId/v1":"58425ea696e1c79bc95744952222abe99ac646a58df923cc8221aa485ddb7603"}},{"ruleId":"D2","level":"warning","message":{"text":"HandlerCreator.finalHandler (cognitive 18): HandlerCreator.finalHandler has cognitive complexity 18 (threshold 15). Drivers by points: if/else 16, boolean chains 2 (nesting depth added 5). Most of this is not in the body itself: 3 of the 18 points are its own statements and the rest belongs to one function literal inside it that branches (line 306). The decisions are inside the literal, which nothing outside this body can call, review or test on its own, so splitting the enclosing body is not the move available here. To reduce it, lift the literal\u0027s work into a named function or method at the enclosing scope and have the literal call it, then reduce whichever part then reads as the largest."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gui/services/custom_commands/handler_creator.go"},"region":{"startLine":288}}}],"partialFingerprints":{"codehealthFindingId/v1":"e5e96dffbb7e7c1a20524915f6d88945f65bd1e2cc9915704db4977206affe2e"}},{"ruleId":"D2","level":"warning","message":{"text":"clients.RunTUI (cognitive 18): clients.RunTUI has cognitive complexity 18 (threshold 15). Drivers by points: if/else 18. Most of this is not in the body itself: 2 of the 18 points are its own statements and the rest belongs to 11 function literals inside it that branch (lines 46, 58, 119, \u2026). The decisions are inside those literals, which nothing outside this body can call, review or test on its own, so splitting the enclosing body is not the move available here. To reduce it, lift the literals\u0027 work into a named function or method at the enclosing scope and have each literal call it, then reduce whichever part then reads as the largest."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/integration/clients/tui.go"},"region":{"startLine":25}}}],"partialFingerprints":{"codehealthFindingId/v1":"4b8284dda5fac96e42177c240c19dd820ad597568b2f4666eed7bb70e5ffde0b"}},{"ruleId":"D2","level":"warning","message":{"text":"RebaseCommands.DiscardOldFileChanges (cognitive 17): RebaseCommands.DiscardOldFileChanges has cognitive complexity 17 (threshold 15). Drivers by points: if/else 16, loops 1 (nesting depth added 8). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/commands/git_commands/rebase.go"},"region":{"startLine":520}}}],"partialFingerprints":{"codehealthFindingId/v1":"ac6fd48ba68d209552ed481b5da7494d9a3cbff5921a536a8dddfa41adb31b1c"}},{"ruleId":"D2","level":"warning","message":{"text":"escapeInterpreter.outputCSI (cognitive 17): escapeInterpreter.outputCSI has cognitive complexity 17 (threshold 15). Drivers by points: if/else 8, boolean chains 6, match/switch 2, loops 1 (nesting depth added 6). This file\u0027s own header attributes it to another copyright holder, so it is code this repository carries rather than code it wrote: restructuring the body in place forks it from upstream and turns every future re-sync into a manual merge. The performable moves are to leave the body as close to its upstream form as possible and keep it behind a narrow interface of your own, and to re-sync it when upstream changes \u2014 or, if it has already diverged far enough that you maintain it here, adopt it deliberately and then split the body into named stages."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gocui/escape.go"},"region":{"startLine":481}}}],"partialFingerprints":{"codehealthFindingId/v1":"646ca0bda6a1fd0f2a08c2d976e160476a9bde6deef72961508f9eea82706533"}},{"ruleId":"D2","level":"warning","message":{"text":"BackgroundRoutineMgr.startBackgroundRoutines (cognitive 17): BackgroundRoutineMgr.startBackgroundRoutines has cognitive complexity 17 (threshold 15). Drivers by points: if/else 15, loops 2 (nesting depth added 5). Of this number, 13 points are the body\u0027s own statements and 4 belong to one function literal inside it that branches. To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gui/background.go"},"region":{"startLine":42}}}],"partialFingerprints":{"codehealthFindingId/v1":"4297d6f14c4c48896ee2725614a0fe1083c2a0fe94e0a6e4cd1071aa0af149ab"}},{"ruleId":"D2","level":"warning","message":{"text":"Gui.RunAndHandleError (cognitive 17): Gui.RunAndHandleError has cognitive complexity 17 (threshold 15). Drivers by points: if/else 15, loops 2 (nesting depth added 10). Most of this is not in the body itself: 0 of the 17 points are its own statements and the rest belongs to one function literal inside it that branches (line 1002). The decisions are inside the literal, which nothing outside this body can call, review or test on its own, so splitting the enclosing body is not the move available here. To reduce it, lift the literal\u0027s work into a named function or method at the enclosing scope and have the literal call it, then reduce whichever part then reads as the largest."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gui/gui.go"},"region":{"startLine":1000}}}],"partialFingerprints":{"codehealthFindingId/v1":"d55b516f1015694f4c8d454d19bdb0b26067778e57e4b5cf3cbf601a19bddd4d"}},{"ruleId":"D2","level":"warning","message":{"text":"theme.GetTextStyle (cognitive 17): theme.GetTextStyle has cognitive complexity 17 (threshold 15). Drivers by points: if/else 14, match/switch 2, loops 1 (nesting depth added 9). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/theme/style.go"},"region":{"startLine":9}}}],"partialFingerprints":{"codehealthFindingId/v1":"da53f7c28aa0aca52d66cbe0a08cc609853dab4997ca3af822d2b56138922632"}},{"ruleId":"D2","level":"warning","message":{"text":"PatchCommands.MovePatchToSelectedCommit (cognitive 16): PatchCommands.MovePatchToSelectedCommit has cognitive complexity 16 (threshold 15). Drivers by points: if/else 16 (nesting depth added 3). Of this number, 14 points are the body\u0027s own statements and 2 belong to one function literal inside it that branches. To reduce it, split the body: most of this score is breadth rather than depth \u2014 checks laid out side by side rather than stacked \u2014 so group the statements between the checks into named steps and move each step into its own function. Some of it IS depth: where a check sits inside another whose only job is to reach it, merge the two into one condition."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/commands/git_commands/patch.go"},"region":{"startLine":116}}}],"partialFingerprints":{"codehealthFindingId/v1":"273a6cdd8aff39372b8eb4c250ee5c2490ab320c2ec56f6fe7e88d5999e8f80e"}},{"ruleId":"D2","level":"warning","message":{"text":"PatchCommands.MovePatchIntoIndex (cognitive 16): PatchCommands.MovePatchIntoIndex has cognitive complexity 16 (threshold 15). Drivers by points: if/else 16 (nesting depth added 4). Of this number, 10 points are the body\u0027s own statements and 6 belong to one function literal inside it that branches. To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/commands/git_commands/patch.go"},"region":{"startLine":220}}}],"partialFingerprints":{"codehealthFindingId/v1":"67751035f2bb35690ac6a1c9d98801caeaf19e8b16e62eb77dd30fd95ccae266"}},{"ruleId":"D2","level":"warning","message":{"text":"OSCommand.PipeCommands (cognitive 16): OSCommand.PipeCommands has cognitive complexity 16 (threshold 15). Drivers by points: if/else 14, loops 2 (nesting depth added 7). Most of this is not in the body itself: 5 of the 16 points are its own statements and the rest belongs to one function literal inside it that branches (line 237). The decisions are inside the literal, which nothing outside this body can call, review or test on its own, so splitting the enclosing body is not the move available here. To reduce it, lift the literal\u0027s work into a named function or method at the enclosing scope and have the literal call it, then reduce whichever part then reads as the largest."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/commands/oscommands/os.go"},"region":{"startLine":205}}}],"partialFingerprints":{"codehealthFindingId/v1":"f41ed61eab01ddfc84f3dae8c4a7bd8f3752bbf821e38a75eefce22e9981d019"}},{"ruleId":"D2","level":"warning","message":{"text":"Gui.processRemainingEvents (cognitive 16): Gui.processRemainingEvents has cognitive complexity 16 (threshold 15). Drivers by points: if/else 12, match/switch 2, boolean chains 1, loops 1 (nesting depth added 9). This file\u0027s own header attributes it to another copyright holder, so it is code this repository carries rather than code it wrote: restructuring the body in place forks it from upstream and turns every future re-sync into a manual merge. The performable moves are to leave the body as close to its upstream form as possible and keep it behind a narrow interface of your own, and to re-sync it when upstream changes \u2014 or, if it has already diverged far enough that you maintain it here, adopt it deliberately and then split the body into named stages."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gocui/gui.go"},"region":{"startLine":1107}}}],"partialFingerprints":{"codehealthFindingId/v1":"6dbe432f2bd5f143b6c006d4088115af29b4071e270565037fa14d24722af257"}},{"ruleId":"D2","level":"warning","message":{"text":"TextArea.SetCursor2D (cognitive 16): TextArea.SetCursor2D has cognitive complexity 16 (threshold 15). Drivers by points: if/else 14, boolean chains 1, loops 1 (nesting depth added 6). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gocui/text_area.go"},"region":{"startLine":622}}}],"partialFingerprints":{"codehealthFindingId/v1":"ff996920c2b120beed2bba7988fc0787aeabaa63d90b0d4c4aa53d1bbbd50d96"}},{"ruleId":"D2","level":"warning","message":{"text":"View.Read (cognitive 16): View.Read has cognitive complexity 16 (threshold 15). Drivers by points: if/else 13, loops 3 (nesting depth added 9). This file\u0027s own header attributes it to another copyright holder, so it is code this repository carries rather than code it wrote: restructuring the body in place forks it from upstream and turns every future re-sync into a manual merge. The performable moves are to leave the body as close to its upstream form as possible and keep it behind a narrow interface of your own, and to re-sync it when upstream changes \u2014 or, if it has already diverged far enough that you maintain it here, adopt it deliberately and then split the body into named stages."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gocui/view.go"},"region":{"startLine":1088}}}],"partialFingerprints":{"codehealthFindingId/v1":"42f1f64b4f0ce4f7286860c896ba0140a2af06a9a42eef54cbbcdd1471bd35ea"}},{"ruleId":"D2","level":"warning","message":{"text":"context.NewLocalCommitsContext (cognitive 16): context.NewLocalCommitsContext has cognitive complexity 16 (threshold 15). Drivers by points: if/else 14, boolean chains 2 (nesting depth added 6). Most of this is not in the body itself: 0 of the 16 points are its own statements and the rest belongs to 2 function literals inside it that branch (lines 80, 46). The decisions are inside those literals, which nothing outside this body can call, review or test on its own, so splitting the enclosing body is not the move available here. To reduce it, lift the literals\u0027 work into a named function or method at the enclosing scope and have each literal call it, then reduce whichever part then reads as the largest."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gui/context/local_commits_context.go"},"region":{"startLine":39}}}],"partialFingerprints":{"codehealthFindingId/v1":"eb22c790d4b15cc03dd3b749615d88fcd239ceba1d7eb6796d17a99dde3072e0"}},{"ruleId":"D2","level":"warning","message":{"text":"controllers.getExplodeImage (cognitive 16): controllers.getExplodeImage has cognitive complexity 16 (threshold 15). Drivers by points: if/else 12, loops 3, boolean chains 1 (nesting depth added 7). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gui/controllers/workspace_reset_controller.go"},"region":{"startLine":227}}}],"partialFingerprints":{"codehealthFindingId/v1":"3bb7ce35f177982f3c6eb7b582e2cffa57392b1316f02075c759b41603a981c4"}},{"ruleId":"D2","level":"warning","message":{"text":"MergeAndRebaseHelper.genericMergeCommandImpl (cognitive 16): MergeAndRebaseHelper.genericMergeCommandImpl has cognitive complexity 16 (threshold 15). Drivers by points: if/else 10, boolean chains 6 (nesting depth added 3). Of this number, 15 points are the body\u0027s own statements and 1 belongs to one function literal inside it that branches. To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gui/controllers/helpers/merge_and_rebase_helper.go"},"region":{"startLine":97}}}],"partialFingerprints":{"codehealthFindingId/v1":"5754c91b9f07530148f160e44856e5d10729edc6c5ec266936056dc7ca80297b"}},{"ruleId":"D2","level":"warning","message":{"text":"presentation.getFileLine (cognitive 16): presentation.getFileLine has cognitive complexity 16 (threshold 15). Drivers by points: if/else 12, boolean chains 4 (nesting depth added 2). To reduce it, split the body: most of this score is breadth rather than depth \u2014 checks laid out side by side rather than stacked \u2014 so group the statements between the checks into named steps and move each step into its own function. Some of it IS depth: where a check sits inside another whose only job is to reach it, merge the two into one condition, and where an else follows a branch that already returns, drop the trailing else and let the rest of the body continue at one level."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gui/presentation/files.go"},"region":{"startLine":112}}}],"partialFingerprints":{"codehealthFindingId/v1":"ed35b349c8e2b2ef2237554344caf9a69c03a5b822186bd745e1e0593f8b0fcc"}},{"ruleId":"D2","level":"warning","message":{"text":"ViewDriver.ContainsLines (cognitive 16): ViewDriver.ContainsLines has cognitive complexity 16 (threshold 15). Drivers by points: if/else 12, loops 3, boolean chains 1 (nesting depth added 9). Most of this is not in the body itself: 0 of the 16 points are its own statements and the rest belongs to one function literal inside it that branches (line 102). The decisions are inside the literal, which nothing outside this body can call, review or test on its own, so splitting the enclosing body is not the move available here. To reduce it, lift the literal\u0027s work into a named function or method at the enclosing scope and have the literal call it, then reduce whichever part then reads as the largest."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/integration/components/view_driver.go"},"region":{"startLine":98}}}],"partialFingerprints":{"codehealthFindingId/v1":"c5502f02ce010f407f3b5e867877a341968470abf065b42cfbeb9f11176227c0"}},{"ruleId":"D2","level":"warning","message":{"text":"utils.excludeBlankColumns (cognitive 16): utils.excludeBlankColumns has cognitive complexity 16 (threshold 15). Drivers by points: if/else 7, loops 7, boolean chains 1, jumps 1 (nesting depth added 5). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/utils/formatting.go"},"region":{"startLine":90}}}],"partialFingerprints":{"codehealthFindingId/v1":"68de3f4429f9cfb864c2a0daa020efa78fe6926cdfbb0b516e47fee83ed5515b"}},{"ruleId":"D3","level":"warning","message":{"text":"TooManyMethods: Gui: TooManyMethods \u2014 139 methods, declared across 20 files: gui/gui.go (32), gui/global_handlers.go (20), gui/keybindings.go (11), gui/main_panels.go (11), \u002B16 more file(s). That list is where to read them, not a suggestion to split the file: the members belong to the type wherever they are declared, so moving them between files leaves the count unchanged. To reduce it, group the members that share the same data into a smaller type of their own and delegate to it, so no single type carries every responsibility."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gui/gui.go"},"region":{"startLine":62}}}],"partialFingerprints":{"codehealthFindingId/v1":"a0332749c8108e0c058de2511db6a38410a08df1d954513172e8b3b2b2dc6f1b"}},{"ruleId":"D3","level":"warning","message":{"text":"FileTooLong: i18n/english.go: FileTooLong \u2014 2233 significant lines (blank, comment-only and punctuation-only lines excluded). To reduce it, split the file along the responsibilities already in it: move each cohesive group of declarations into its own sibling file in the same module or package, so no one file has to be read whole to change one of them."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/i18n/english.go"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"56565e11932c0231c999167a8d0eb72581fbc66e68e14ccf9e840c2c0c4261bd"}},{"ruleId":"D3","level":"warning","message":{"text":"TooManyMethods: View: TooManyMethods \u2014 104 methods. To reduce it, group the members that share the same data into a smaller type of their own and delegate to it, so no single type carries every responsibility."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gocui/view.go"},"region":{"startLine":30}}}],"partialFingerprints":{"codehealthFindingId/v1":"99116414439e3c196367f3ad82036bc5ddfff3f92ef5cdecfd2fcacb1d921f51"}},{"ruleId":"D3","level":"warning","message":{"text":"TooManyMethods: Gui: TooManyMethods \u2014 89 methods, declared across 3 files: gocui/gui.go (85), gocui/tcell_driver.go (3), gocui/gui_others.go (1). That list is where to read them, not a suggestion to split the file: the members belong to the type wherever they are declared, so moving them between files leaves the count unchanged. To reduce it, group the members that share the same data into a smaller type of their own and delegate to it, so no single type carries every responsibility."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gocui/gui.go"},"region":{"startLine":126}}}],"partialFingerprints":{"codehealthFindingId/v1":"50fa74dc4b6d229b0a53fb279d05376d2ce15a247836bc4722155ae00006fb11"}},{"ruleId":"D3","level":"warning","message":{"text":"TooManyMethods: LocalCommitsController: TooManyMethods \u2014 83 methods. To reduce it, group the members that share the same data into a smaller type of their own and delegate to it, so no single type carries every responsibility."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gui/controllers/local_commits_controller.go"},"region":{"startLine":32}}}],"partialFingerprints":{"codehealthFindingId/v1":"50b126c090661e2673065edda9ca39b541df9c73987e4beeb52fddda95f35937"}},{"ruleId":"D3","level":"warning","message":{"text":"TooManyMethods: FilesController: TooManyMethods \u2014 75 methods, declared across 2 files: controllers/files_controller.go (72), controllers/workspace_reset_controller.go (3). That list is where to read them, not a suggestion to split the file: the members belong to the type wherever they are declared, so moving them between files leaves the count unchanged. To reduce it, group the members that share the same data into a smaller type of their own and delegate to it, so no single type carries every responsibility."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gui/controllers/files_controller.go"},"region":{"startLine":20}}}],"partialFingerprints":{"codehealthFindingId/v1":"fca15ec86281a5569fa8a85283ea52aef622cd8837759e1711c8f016c8651e3a"}},{"ruleId":"D3","level":"warning","message":{"text":"FileTooLong: controllers/local_commits_controller.go: FileTooLong \u2014 1215 significant lines (blank, comment-only and punctuation-only lines excluded). To reduce it, split the file along the responsibilities already in it: move each cohesive group of declarations into its own sibling file in the same module or package, so no one file has to be read whole to change one of them."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gui/controllers/local_commits_controller.go"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"b114a0daf2ffb98be3adf321a3c07c7ee581a5bd19ab04571de268c06d1d7817"}},{"ruleId":"D3","level":"warning","message":{"text":"FileTooLong: gocui/view.go: FileTooLong \u2014 1144 significant lines (blank, comment-only and punctuation-only lines excluded). To reduce it, split the file along the responsibilities already in it: move each cohesive group of declarations into its own sibling file in the same module or package, so no one file has to be read whole to change one of them."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gocui/view.go"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"42ae2154782f2e6bedd2311f67f8efc9ca232a9c16e0f90b1cfcb8ab6da61aeb"}},{"ruleId":"D3","level":"warning","message":{"text":"FileTooLong: gocui/gui.go: FileTooLong \u2014 1099 significant lines (blank, comment-only and punctuation-only lines excluded). To reduce it, split the file along the responsibilities already in it: move each cohesive group of declarations into its own sibling file in the same module or package, so no one file has to be read whole to change one of them."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gocui/gui.go"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"ce46ec580b27c5a7fc4f2a01028774969d0a8b918cc7c5a93a0b05afbcbd52fa"}},{"ruleId":"D3","level":"warning","message":{"text":"FileTooLong: controllers/files_controller.go: FileTooLong \u2014 1046 significant lines (blank, comment-only and punctuation-only lines excluded). To reduce it, split the file along the responsibilities already in it: move each cohesive group of declarations into its own sibling file in the same module or package, so no one file has to be read whole to change one of them."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gui/controllers/files_controller.go"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"4e360374bc3ddf453a9b2cdaf0a678eb9359a49050478a843f911cea3c733415"}},{"ruleId":"D3","level":"warning","message":{"text":"TooManyMethods: Shell: TooManyMethods \u2014 60 methods. To reduce it, group the members that share the same data into a smaller type of their own and delegate to it, so no single type carries every responsibility."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/integration/components/shell.go"},"region":{"startLine":17}}}],"partialFingerprints":{"codehealthFindingId/v1":"9784fe626eb2abd0b621ca6e0548123c0b40a01898e76e58dce11a22ef909504"}},{"ruleId":"D3","level":"warning","message":{"text":"TooManyMethods: ViewDriver: TooManyMethods \u2014 56 methods. To reduce it, group the members that share the same data into a smaller type of their own and delegate to it, so no single type carries every responsibility."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/integration/components/view_driver.go"},"region":{"startLine":12}}}],"partialFingerprints":{"codehealthFindingId/v1":"739da28cdf0320e7915165c1d6ce3acb709eb7758f403eeea60e62fa760f3a25"}},{"ruleId":"D3","level":"warning","message":{"text":"FileTooLong: helpers/refresh_helper.go: FileTooLong \u2014 893 significant lines (blank, comment-only and punctuation-only lines excluded). To reduce it, split the file along the responsibilities already in it: move each cohesive group of declarations into its own sibling file in the same module or package, so no one file has to be read whole to change one of them."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gui/controllers/helpers/refresh_helper.go"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"e24a731782a21525ee4409d12dc924642c50e047c784044bef5983cc0ac4cf72"}},{"ruleId":"D3","level":"warning","message":{"text":"TooManyMethods: guiCommon: TooManyMethods \u2014 48 methods. Most of these members implement types.IGuiCommon (46 of 48 members), so moving them onto a smaller type would remove them from that contract rather than reduce it. To reduce it, split the contract instead: give each cohesive group of operations its own smaller interface and its own implementing type. Where the contract has to stay whole, move the work behind these members into collaborator types, so what is left here is a forward per member rather than a responsibility per member."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gui/gui_common.go"},"region":{"startLine":14}}}],"partialFingerprints":{"codehealthFindingId/v1":"10afc6589a2190ded9fd8c35cd47b8c8bece9790c018d10eed5df29bbec53386"}},{"ruleId":"D3","level":"warning","message":{"text":"FileTooLong: icons/file_icons.go: FileTooLong \u2014 769 significant lines (blank, comment-only and punctuation-only lines excluded), about 69% of them inside a single declaration: extIconMap (223-757). Moving the declarations that sit BESIDE it into sibling files will not shorten this file. Extract from INSIDE that declaration instead: lift each cohesive group of its body \u2014 the parts that share the same inputs and are named together \u2014 into its own unit in a sibling file, and have the original call them."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gui/presentation/icons/file_icons.go"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"3b82ef00e29d32f0e2edad9db2155f40f04a2900fbdfc06bd3b8c210c6a1ecca"}},{"ruleId":"D3","level":"warning","message":{"text":"TooManyMethods: RefreshHelper: TooManyMethods \u2014 45 methods. To reduce it, group the members that share the same data into a smaller type of their own and delegate to it, so no single type carries every responsibility."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gui/controllers/helpers/refresh_helper.go"},"region":{"startLine":29}}}],"partialFingerprints":{"codehealthFindingId/v1":"b0430be997eab7292cebb3719bb9aa614be95bc0ff71bbbacd3782c41b76bc4c"}},{"ruleId":"D3","level":"warning","message":{"text":"FileTooLong: config/user_config.go: FileTooLong \u2014 749 significant lines (blank, comment-only and punctuation-only lines excluded). To reduce it, split the file along the responsibilities already in it: move each cohesive group of declarations into its own sibling file in the same module or package, so no one file has to be read whole to change one of them."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/config/user_config.go"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"64a5bd77e1e15d0ff4b5d98c9ff2b4698b6284b2f2ee9fe58493fb3a63dfb3cd"}},{"ruleId":"D3","level":"warning","message":{"text":"FileTooLong: gui/gui.go: FileTooLong \u2014 725 significant lines (blank, comment-only and punctuation-only lines excluded). To reduce it, split the file along the responsibilities already in it: move each cohesive group of declarations into its own sibling file in the same module or package, so no one file has to be read whole to change one of them."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gui/gui.go"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"5acd4adb83cd46feab9a8de3fc74adc2533a65bff0125abc9d61ae14f3fa0dd2"}},{"ruleId":"D3","level":"warning","message":{"text":"TooManyMethods: TextArea: TooManyMethods \u2014 38 methods. To reduce it, group the members that share the same data into a smaller type of their own and delegate to it, so no single type carries every responsibility."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gocui/text_area.go"},"region":{"startLine":31}}}],"partialFingerprints":{"codehealthFindingId/v1":"7248be00f63a1e154b1078f8b2f1f9743da4108bfc8159927a20c3cf2406ac25"}},{"ruleId":"D3","level":"warning","message":{"text":"TooManyMethods: BranchesController: TooManyMethods \u2014 37 methods. To reduce it, group the members that share the same data into a smaller type of their own and delegate to it, so no single type carries every responsibility."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gui/controllers/branches_controller.go"},"region":{"startLine":19}}}],"partialFingerprints":{"codehealthFindingId/v1":"f8516c6121f12afc27d04e838de5367fc7105fea06c03a7c426c20351ffed3d9"}},{"ruleId":"D3","level":"warning","message":{"text":"TooManyMethods: State: TooManyMethods \u2014 37 methods. To reduce it, group the members that share the same data into a smaller type of their own and delegate to it, so no single type carries every responsibility."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gui/patch_exploring/state.go"},"region":{"startLine":16}}}],"partialFingerprints":{"codehealthFindingId/v1":"610fbe785c368dd9e0aa796be3199d5e996759c425e46a209ed0ec8141141ec9"}},{"ruleId":"D3","level":"warning","message":{"text":"TooManyMethods: WorkingTreeCommands: TooManyMethods \u2014 36 methods. To reduce it, group the members that share the same data into a smaller type of their own and delegate to it, so no single type carries every responsibility."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/commands/git_commands/working_tree.go"},"region":{"startLine":18}}}],"partialFingerprints":{"codehealthFindingId/v1":"a700fad3e673b367aaa4d8afba626df4d1ac414dadf73874fe5ebf8bd641d71b"}},{"ruleId":"D3","level":"warning","message":{"text":"TooManyMethods: RebaseCommands: TooManyMethods \u2014 35 methods. To reduce it, group the members that share the same data into a smaller type of their own and delegate to it, so no single type carries every responsibility."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/commands/git_commands/rebase.go"},"region":{"startLine":17}}}],"partialFingerprints":{"codehealthFindingId/v1":"f8ccf840ec98b69cb1e075b3d09f0935ba560c70f5451420a29c58c379141843"}},{"ruleId":"D3","level":"warning","message":{"text":"FileTooLong: controllers/branches_controller.go: FileTooLong \u2014 581 significant lines (blank, comment-only and punctuation-only lines excluded). To reduce it, split the file along the responsibilities already in it: move each cohesive group of declarations into its own sibling file in the same module or package, so no one file has to be read whole to change one of them."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gui/controllers/branches_controller.go"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"0206c88a5c4d578bdb1857ab588885df7e6739ee342700ea466ae4a79183fce1"}},{"ruleId":"D3","level":"warning","message":{"text":"TooManyMethods: Views: TooManyMethods \u2014 33 methods. To reduce it, group the members that share the same data into a smaller type of their own and delegate to it, so no single type carries every responsibility."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/integration/components/views.go"},"region":{"startLine":9}}}],"partialFingerprints":{"codehealthFindingId/v1":"2610b87cc5fff3c39ea4f140c2cc0a09e5acfe6541a752e03fcd5276027e8569"}},{"ruleId":"D3","level":"warning","message":{"text":"FileTooLong: config/app_config.go: FileTooLong \u2014 520 significant lines (blank, comment-only and punctuation-only lines excluded). To reduce it, split the file along the responsibilities already in it: move each cohesive group of declarations into its own sibling file in the same module or package, so no one file has to be read whole to change one of them."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/config/app_config.go"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"91c4c9eab38879e05e4a3a3cee3fec902413d66271a187434e6e0d06dd4d7dd1"}},{"ruleId":"D3","level":"warning","message":{"text":"TooManyMethods: GuiDriver: TooManyMethods \u2014 31 methods. To reduce it, group the members that share the same data into a smaller type of their own and delegate to it, so no single type carries every responsibility."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gui/gui_driver.go"},"region":{"startLine":19}}}],"partialFingerprints":{"codehealthFindingId/v1":"25ba9548465831b6e0f0465e81b4cb3fb1a7584c6001d26d980889a2df4bded7"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (15 lines \u00D7 2): pkg/commands/git_commands/patch.go:177-191 | pkg/commands/git_commands/patch.go:287-301 \u2014 both copies are in the same file, so extract the block into one function there and call it from each site \u2014 the copies drift apart the first time only one of them is edited. Read the line range as the matched WINDOW rather than a finished unit: at \u0060pkg/commands/git_commands/patch.go:177\u0060 it begins part-way through the construct above it, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that. Note that the copies do not run to the end of the range shown: their LAST lines are different code, not the same code under different names \u2014 the matched region ends inside that line. Extract the lines above it, and read the last line of each site separately."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/commands/git_commands/patch.go"},"region":{"startLine":177}}}],"partialFingerprints":{"codehealthFindingId/v1":"70ae64be62cc1b17df5108d550c1753aaaefc4fc913f67bd4ab5478b3a50d5be"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (15 lines \u00D7 2): pkg/gui/controllers/helpers/merge_and_rebase_helper.go:441-455 | pkg/gui/controllers/helpers/merge_and_rebase_helper.go:472-486 \u2014 both copies are in the same file, so extract the block into one function there and call it from each site \u2014 the copies drift apart the first time only one of them is edited. Read the line range as the matched WINDOW rather than a finished unit: at \u0060pkg/gui/controllers/helpers/merge_and_rebase_helper.go:441\u0060 it begins part-way through the construct above it, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gui/controllers/helpers/merge_and_rebase_helper.go"},"region":{"startLine":441}}}],"partialFingerprints":{"codehealthFindingId/v1":"13f5e717e877d021cc96478ed871811544d290222a0a31e852a4ed19aa8ac24d"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (14 lines \u00D7 3): pkg/gui/controllers/submodules_controller.go:229-242 | pkg/gui/controllers/submodules_controller.go:244-257 | pkg/gui/controllers/submodules_controller.go:259-272 \u2014 all 3 copies are in the same file, so extract the block into one function there and call it from every one of those sites \u2014 resolving only two of them leaves the rest to drift apart the first time one is edited. Read the line range as the matched WINDOW rather than a finished unit: at \u0060pkg/gui/controllers/submodules_controller.go:229\u0060 it begins part-way through the construct above it, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gui/controllers/submodules_controller.go"},"region":{"startLine":229}}}],"partialFingerprints":{"codehealthFindingId/v1":"77345f855fe909da65dabb8bf6cf6fb50a17b24c1e846c802a8c7346009f7178"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (14 lines \u00D7 2): pkg/commands/git_commands/patch.go:93-107 | pkg/commands/git_commands/patch.go:282-295 \u2014 both copies are in the same file, so extract the block into one function there and call it from each site \u2014 the copies drift apart the first time only one of them is edited. Read the line range as the matched WINDOW rather than a finished unit: at \u0060pkg/commands/git_commands/patch.go:93\u0060 it begins part-way through the construct above it, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that. Note that the copies do not run to the end of the range shown: their LAST lines are different code, not the same code under different names \u2014 the matched region ends inside that line. Extract the lines above it, and read the last line of each site separately."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/commands/git_commands/patch.go"},"region":{"startLine":93}}}],"partialFingerprints":{"codehealthFindingId/v1":"f5638935100178d022866f11ec8e69a9faa9939fb40ba7a25b0bc415c4f5a811"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (14 lines \u00D7 2): pkg/gui/controllers/custom_patch_options_menu_action.go:196-209 | pkg/gui/controllers/custom_patch_options_menu_action.go:231-244 \u2014 both copies are in the same file, so extract the block into one function there and call it from each site \u2014 the copies drift apart the first time only one of them is edited. Read the line range as the matched WINDOW rather than a finished unit: at \u0060pkg/gui/controllers/custom_patch_options_menu_action.go:196\u0060 it begins part-way through the construct above it, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gui/controllers/custom_patch_options_menu_action.go"},"region":{"startLine":196}}}],"partialFingerprints":{"codehealthFindingId/v1":"8857284935d887aae4d7a70d345c3de754ee75865f5a87023be2a8c05ecdf3ea"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (14 lines \u00D7 2): pkg/gui/controllers/helpers/refs_helper.go:547-560 | pkg/gui/controllers/helpers/refs_helper.go:583-596 \u2014 both copies are in the same file, so extract the block into one function there and call it from each site \u2014 the copies drift apart the first time only one of them is edited. Read the line range as the matched WINDOW rather than a finished unit: at \u0060pkg/gui/controllers/helpers/refs_helper.go:547\u0060 it begins part-way through the construct above it, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gui/controllers/helpers/refs_helper.go"},"region":{"startLine":547}}}],"partialFingerprints":{"codehealthFindingId/v1":"8b2b1e8c7c7846b1dfb10c9f6e422bcb75cb18823b1ea2b7ccb5db4371474b5b"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (13 lines \u00D7 2): pkg/gui/filetree/commit_file_tree_view_model.go:168-180 | pkg/gui/filetree/file_tree_view_model.go:179-191 \u2014 the copies sit in sibling files of one directory: extract the block into a single shared function in that directory and call it from each site, so a change lands once. Read the line range as the matched WINDOW rather than a finished unit: at \u0060pkg/gui/filetree/commit_file_tree_view_model.go:168\u0060 it begins part-way through the construct above it, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gui/filetree/commit_file_tree_view_model.go"},"region":{"startLine":168}}}],"partialFingerprints":{"codehealthFindingId/v1":"659a3046f2a4254cb65364fc9ced7f74c4410256e23eb52a611ed6bd6e55a597"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (13 lines \u00D7 2): pkg/gui/presentation/files.go:306-318 | pkg/gui/presentation/files.go:334-346 \u2014 both copies are in the same file, so extract the block into one function there and call it from each site \u2014 the copies drift apart the first time only one of them is edited. Read the line range as the matched WINDOW rather than a finished unit: at \u0060pkg/gui/presentation/files.go:306\u0060 it begins part-way through the construct above it, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gui/presentation/files.go"},"region":{"startLine":306}}}],"partialFingerprints":{"codehealthFindingId/v1":"c807837ca9606fdc69f00a334a667ef342a34148792bb5ce08f736b22c2433c2"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (12 lines \u00D7 2): pkg/gui/controllers/custom_patch_options_menu_action.go:177-188 | pkg/gui/controllers/custom_patch_options_menu_action.go:212-223 \u2014 both copies are in the same file, so extract the block into one function there and call it from each site \u2014 the copies drift apart the first time only one of them is edited. Read the line range as the matched WINDOW rather than a finished unit: at \u0060pkg/gui/controllers/custom_patch_options_menu_action.go:177\u0060 it does not close everything it opens, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gui/controllers/custom_patch_options_menu_action.go"},"region":{"startLine":177}}}],"partialFingerprints":{"codehealthFindingId/v1":"fbc7d5a4552486618631c3e7aeb477b497bda15fbe7cf155ce0c6404b98a059d"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (11 lines \u00D7 2): pkg/config/keybinding.go:51-61 | pkg/config/side_panel.go:33-43 \u2014 the copies sit in sibling files of one directory: extract the block into a single shared function in that directory and call it from each site, so a change lands once. The matched lines also transfer control out of the body holding them, which cannot survive a move into a called unit unchanged: have the extracted unit return that decision and let each site act on it."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/config/keybinding.go"},"region":{"startLine":51}}}],"partialFingerprints":{"codehealthFindingId/v1":"402bde95e498e631d9e50d5cd00095e54ac1619c2cbdf280507da9f73c76f53a"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (11 lines \u00D7 2): pkg/gocui/escape.go:313-323 | pkg/gocui/escape.go:408-419 \u2014 both copies are in the same file, so extract the block into one function there and call it from each site \u2014 the copies drift apart the first time only one of them is edited. Read the line range as the matched WINDOW rather than a finished unit: at \u0060pkg/gocui/escape.go:313\u0060 it begins part-way through the construct above it, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that. The matched lines also transfer control out of the body holding them, which cannot survive a move into a called unit unchanged: have the extracted unit return that decision and let each site act on it."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gocui/escape.go"},"region":{"startLine":313}}}],"partialFingerprints":{"codehealthFindingId/v1":"43765d20aa68f8eb6aeb24673c94c63e58446a7340e403f3c8dc1b64445d9a1f"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (11 lines \u00D7 2): pkg/gui/context/local_commits_context.go:382-392 | pkg/gui/context/sub_commits_context.go:232-242 \u2014 the copies sit in sibling files of one directory: extract the block into a single shared function in that directory and call it from each site, so a change lands once. Read the line range as the matched WINDOW rather than a finished unit: at \u0060pkg/gui/context/local_commits_context.go:382\u0060 it begins part-way through the construct above it, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gui/context/local_commits_context.go"},"region":{"startLine":382}}}],"partialFingerprints":{"codehealthFindingId/v1":"9040103513697b316c9072a5cc53eeaa1a7772ce5179bd804f9806aacad6394f"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (11 lines \u00D7 2): pkg/gui/controllers/tags_controller.go:177-187 | pkg/gui/controllers/tags_controller.go:227-237 \u2014 both copies are in the same file, so extract the block into one function there and call it from each site \u2014 the copies drift apart the first time only one of them is edited. Read the line range as the matched WINDOW rather than a finished unit: at \u0060pkg/gui/controllers/tags_controller.go:177\u0060 it does not close everything it opens, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gui/controllers/tags_controller.go"},"region":{"startLine":177}}}],"partialFingerprints":{"codehealthFindingId/v1":"aa6b96fb738f87ecb04f44d7552da60398ddd2c5b176f1b2db792089350aa231"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (11 lines \u00D7 2): pkg/gui/controllers/tags_controller.go:197-207 | pkg/gui/controllers/tags_controller.go:247-257 \u2014 both copies are in the same file, so extract the block into one function there and call it from each site \u2014 the copies drift apart the first time only one of them is edited. Read the line range as the matched WINDOW rather than a finished unit: at \u0060pkg/gui/controllers/tags_controller.go:197\u0060 it begins part-way through the construct above it, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gui/controllers/tags_controller.go"},"region":{"startLine":197}}}],"partialFingerprints":{"codehealthFindingId/v1":"58bbec62d4e9ff09a20746ba60faa5274f6602214225f161ac4fc9565336205c"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (11 lines \u00D7 2): pkg/gui/global_handlers.go:26-40 | pkg/gui/global_handlers.go:47-57 \u2014 both copies are in the same file, so extract the block into one function there and call it from each site \u2014 the copies drift apart the first time only one of them is edited."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gui/global_handlers.go"},"region":{"startLine":26}}}],"partialFingerprints":{"codehealthFindingId/v1":"012d766d6cf74f3d7a40b3b8ced3a00ce43613e0846d09b881d38ff4c976795f"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (10 lines \u00D7 2): pkg/config/app_config.go:392-402 | pkg/config/app_config.go:414-423 \u2014 both copies are in the same file, so extract the block into one function there and call it from each site \u2014 the copies drift apart the first time only one of them is edited. Read the line range as the matched WINDOW rather than a finished unit: at \u0060pkg/config/app_config.go:392\u0060 it does not close everything it opens, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that. Note that the copies do not run to the end of the range shown: their LAST lines are different code, not the same code under different names \u2014 the matched region ends inside that line. Extract the lines above it, and read the last line of each site separately."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/config/app_config.go"},"region":{"startLine":392}}}],"partialFingerprints":{"codehealthFindingId/v1":"9450471ac4f788696b7cd73f7df424f1b8e9d76ebeab547bce75e4afed787c3a"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (10 lines \u00D7 2): pkg/gui/controllers/commits_files_controller.go:227-236 | pkg/gui/controllers/files_controller.go:1366-1375 \u2014 the copies sit in sibling files of one directory: extract the block into a single shared function in that directory and call it from each site, so a change lands once. Read the line range as the matched WINDOW rather than a finished unit: at \u0060pkg/gui/controllers/commits_files_controller.go:227\u0060 it begins part-way through the construct above it, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gui/controllers/commits_files_controller.go"},"region":{"startLine":227}}}],"partialFingerprints":{"codehealthFindingId/v1":"47c448a6cc5394113431c6e674e9de724efd83ab403b2df47a0152d9151dc9f8"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (10 lines \u00D7 2): pkg/integration/clients/tui.go:116-125 | pkg/integration/clients/tui.go:130-139 \u2014 both copies are in the same file, so extract the block into one function there and call it from each site \u2014 the copies drift apart the first time only one of them is edited. Read the line range as the matched WINDOW rather than a finished unit: at \u0060pkg/integration/clients/tui.go:116\u0060 it begins part-way through the construct above it, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/integration/clients/tui.go"},"region":{"startLine":116}}}],"partialFingerprints":{"codehealthFindingId/v1":"42a41a0692fb8bd0a92c4cee0270ad62f0976e76b4829633b63de8b188452eee"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (10 lines \u00D7 2): pkg/gui/view_helpers.go:78-87 | pkg/gui/view_helpers.go:96-105 \u2014 both copies are in the same file, so extract the block into one function there and call it from each site \u2014 the copies drift apart the first time only one of them is edited. Read the line range as the matched WINDOW rather than a finished unit: at \u0060pkg/gui/view_helpers.go:78\u0060 it does not close everything it opens, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that. The matched lines also transfer control out of the body holding them, which cannot survive a move into a called unit unchanged: have the extracted unit return that decision and let each site act on it. Note that the copies do not run to the end of the range shown: their LAST lines are different code, not the same code under different names \u2014 the matched region ends inside that line. Extract the lines above it, and read the last line of each site separately."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gui/view_helpers.go"},"region":{"startLine":78}}}],"partialFingerprints":{"codehealthFindingId/v1":"0308eece789d5e64788eb6fa0a660b26cbfa69fcc4766190d5190080cbe2c0f1"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (9 lines \u00D7 2): pkg/gui/controllers/branches_controller.go:237-245 | pkg/gui/controllers/branches_controller.go:270-278 \u2014 both copies are in the same file, so extract the block into one function there and call it from each site \u2014 the copies drift apart the first time only one of them is edited. Read the line range as the matched WINDOW rather than a finished unit: at \u0060pkg/gui/controllers/branches_controller.go:237\u0060 it does not close everything it opens, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gui/controllers/branches_controller.go"},"region":{"startLine":237}}}],"partialFingerprints":{"codehealthFindingId/v1":"f040cbe3332b47f17346b86e43f44d6975d27ea3136c9a44f8f09235a2d907f2"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (9 lines \u00D7 2): pkg/gui/controllers/commits_files_controller.go:497-505 | pkg/gui/controllers/commits_files_controller.go:555-563 \u2014 both copies are in the same file, so extract the block into one function there and call it from each site \u2014 the copies drift apart the first time only one of them is edited. Read the line range as the matched WINDOW rather than a finished unit: at \u0060pkg/gui/controllers/commits_files_controller.go:497\u0060 it does not close everything it opens, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gui/controllers/commits_files_controller.go"},"region":{"startLine":497}}}],"partialFingerprints":{"codehealthFindingId/v1":"b4c3fc526fdffdc1748482e6163ac544291fc7bec11acb210869bcf89fed099c"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (9 lines \u00D7 2): pkg/gui/controllers/helpers/branches_helper.go:44-52 | pkg/gui/controllers/helpers/branches_helper.go:319-327 \u2014 both copies are in the same file, so extract the block into one function there and call it from each site \u2014 the copies drift apart the first time only one of them is edited. Read the line range as the matched WINDOW rather than a finished unit: at \u0060pkg/gui/controllers/helpers/branches_helper.go:319\u0060 it does not close everything it opens, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that. Note that the copies do not run to the end of the range shown: their LAST lines are different code, not the same code under different names \u2014 the matched region ends inside that line. Extract the lines above it, and read the last line of each site separately."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gui/controllers/helpers/branches_helper.go"},"region":{"startLine":44}}}],"partialFingerprints":{"codehealthFindingId/v1":"af682450aa954e13f6435f9a5750ba345f4a1b7464f517056cf289631f850796"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (9 lines \u00D7 2): pkg/gui/controllers/helpers/branches_helper.go:156-164 | pkg/gui/controllers/helpers/branches_helper.go:339-347 \u2014 both copies are in the same file, so extract the block into one function there and call it from each site \u2014 the copies drift apart the first time only one of them is edited. Read the line range as the matched WINDOW rather than a finished unit: at \u0060pkg/gui/controllers/helpers/branches_helper.go:339\u0060 it does not close everything it opens, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that. Note that the copies do not run to the end of the range shown: their LAST lines are different code, not the same code under different names \u2014 the matched region ends inside that line. Extract the lines above it, and read the last line of each site separately."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gui/controllers/helpers/branches_helper.go"},"region":{"startLine":156}}}],"partialFingerprints":{"codehealthFindingId/v1":"f8ac5cb66d5f8f47e0e133b6abdcb2d8b6e6ab83d685057641c0eae8c4ca383d"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (9 lines \u00D7 2): pkg/gui/controllers/helpers/branches_helper.go:320-328 | pkg/gui/controllers/helpers/branches_helper.go:340-348 \u2014 both copies are in the same file, so extract the block into one function there and call it from each site \u2014 the copies drift apart the first time only one of them is edited. Read the line range as the matched WINDOW rather than a finished unit: at \u0060pkg/gui/controllers/helpers/branches_helper.go:320\u0060 it begins part-way through the construct above it, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that. Note that the copies do not run to the end of the range shown: their LAST lines are different code, not the same code under different names \u2014 the matched region ends inside that line. Extract the lines above it, and read the last line of each site separately."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gui/controllers/helpers/branches_helper.go"},"region":{"startLine":320}}}],"partialFingerprints":{"codehealthFindingId/v1":"5ba91309f8bb1309ba13be027872d8544ef858b733f7d7d6cfc4c8ba1437909d"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (9 lines \u00D7 2): pkg/gui/controllers/helpers/refs_helper.go:538-546 | pkg/gui/controllers/helpers/refs_helper.go:568-576 \u2014 both copies are in the same file, so extract the block into one function there and call it from each site \u2014 the copies drift apart the first time only one of them is edited. Read the line range as the matched WINDOW rather than a finished unit: at \u0060pkg/gui/controllers/helpers/refs_helper.go:538\u0060 it begins part-way through the construct above it, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that. Note that the copies do not run to the end of the range shown: their LAST lines are different code, not the same code under different names \u2014 the matched region ends inside that line. Extract the lines above it, and read the last line of each site separately."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gui/controllers/helpers/refs_helper.go"},"region":{"startLine":538}}}],"partialFingerprints":{"codehealthFindingId/v1":"a993b67312a12ec8128d98229dddd0ffd617c5df7e6fe17db2a6d39368841ba6"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (8 lines \u00D7 2): pkg/gui/controllers/branches_controller.go:255-262 | pkg/gui/controllers/helpers/merge_and_rebase_helper.go:409-417 \u2014 the copies span different directories, so extracting a shared function means choosing where it lives: put it somewhere both call sites can already reach \u2014 a location they all depend on today, or a new shared one if there is none \u2014 and call it from each site; until then, every change has to be made twice. Read the line range as the matched WINDOW rather than a finished unit: at \u0060pkg/gui/controllers/helpers/merge_and_rebase_helper.go:409\u0060 it does not close everything it opens, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that. Note that the copies do not run to the end of the range shown: their LAST lines are different code, not the same code under different names \u2014 the matched region ends inside that line. Extract the lines above it, and read the last line of each site separately."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gui/controllers/branches_controller.go"},"region":{"startLine":255}}}],"partialFingerprints":{"codehealthFindingId/v1":"1860404920d23b075706917019112219b35ce59e581daf707abcc1da5e6d0d3e"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (8 lines \u00D7 2): pkg/gui/controllers/local_commits_controller.go:704-711 | pkg/gui/controllers/sub_commits_controller.go:47-54 \u2014 the copies sit in sibling files of one directory: extract the block into a single shared function in that directory and call it from each site, so a change lands once. Read the line range as the matched WINDOW rather than a finished unit: at \u0060pkg/gui/controllers/local_commits_controller.go:704\u0060 it begins part-way through the construct above it, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gui/controllers/local_commits_controller.go"},"region":{"startLine":704}}}],"partialFingerprints":{"codehealthFindingId/v1":"3d79660413c023a86f5f3ee84e1999c58df67ce24a64c44df5e56720c3633fbd"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (7 lines \u00D7 2): pkg/commands/git_commands/file.go:48-54 | pkg/commands/git_commands/file.go:60-66 \u2014 both copies are in the same file, so extract the block into one function there and call it from each site \u2014 the copies drift apart the first time only one of them is edited. The matched lines also transfer control out of the body holding them, which cannot survive a move into a called unit unchanged: have the extracted unit return that decision and let each site act on it. Note that the copies do not run to the end of the range shown: their LAST lines are different code, not the same code under different names \u2014 the matched region ends inside that line. Extract the lines above it, and read the last line of each site separately."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/commands/git_commands/file.go"},"region":{"startLine":48}}}],"partialFingerprints":{"codehealthFindingId/v1":"a8172fcb34cb0ada2e4346e40a54ee1211c24db086c6ba7abb12c0d80f5d54cd"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (7 lines \u00D7 2): pkg/gocui/gui.go:462-468 | pkg/gocui/gui.go:477-483 \u2014 both copies are in the same file, so extract the block into one function there and call it from each site \u2014 the copies drift apart the first time only one of them is edited. Read the line range as the matched WINDOW rather than a finished unit: at \u0060pkg/gocui/gui.go:462\u0060 it does not close everything it opens, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that. The matched lines also register a scope-exit action (a \u0060defer\u0060-style statement) that runs when the function holding them returns: moved into a called unit it would run when THAT unit returns instead \u2014 before the caller uses what it releases \u2014 so keep the registration at the call site and extract only the work around it, or have the extracted unit hand the resource back for the caller to register. Note that the copies do not run to the end of the range shown: their LAST lines are different code, not the same code under different names \u2014 the matched region ends inside that line. Extract the lines above it, and read the last line of each site separately."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gocui/gui.go"},"region":{"startLine":462}}}],"partialFingerprints":{"codehealthFindingId/v1":"f7b73486f2295a3f419ceceb4ec7767d2b91c1eb0878ea6de535e5182f3c0ee3"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (7 lines \u00D7 2): pkg/gui/controllers/local_commits_controller.go:771-777 | pkg/gui/controllers/local_commits_controller.go:787-793 \u2014 both copies are in the same file, so extract the block into one function there and call it from each site \u2014 the copies drift apart the first time only one of them is edited. Read the line range as the matched WINDOW rather than a finished unit: at \u0060pkg/gui/controllers/local_commits_controller.go:771\u0060 it does not close everything it opens, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gui/controllers/local_commits_controller.go"},"region":{"startLine":771}}}],"partialFingerprints":{"codehealthFindingId/v1":"87bc4ea2db6a707d28c71f06f218c8eb365d91e66753616b9b2d431e1c6e5b71"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (7 lines \u00D7 2): pkg/gui/controllers/local_commits_controller.go:845-851 | pkg/gui/controllers/local_commits_controller.go:1526-1532 \u2014 both copies are in the same file, so extract the block into one function there and call it from each site \u2014 the copies drift apart the first time only one of them is edited. Read the line range as the matched WINDOW rather than a finished unit: at \u0060pkg/gui/controllers/local_commits_controller.go:845\u0060 it does not close everything it opens, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that. Note that the copies do not run to the end of the range shown: their LAST lines are different code, not the same code under different names \u2014 the matched region ends inside that line. Extract the lines above it, and read the last line of each site separately."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gui/controllers/local_commits_controller.go"},"region":{"startLine":845}}}],"partialFingerprints":{"codehealthFindingId/v1":"53bb64b40d627e4e927a5daa56e40d60076d67ec773b067a81aafe5b5822f341"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (7 lines \u00D7 2): pkg/gui/controllers/local_commits_controller.go:1431-1437 | pkg/gui/controllers/local_commits_controller.go:1542-1548 \u2014 both copies are in the same file, so extract the block into one function there and call it from each site \u2014 the copies drift apart the first time only one of them is edited. Read the line range as the matched WINDOW rather than a finished unit: at \u0060pkg/gui/controllers/local_commits_controller.go:1431\u0060 it does not close everything it opens, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gui/controllers/local_commits_controller.go"},"region":{"startLine":1431}}}],"partialFingerprints":{"codehealthFindingId/v1":"14af93ccc04a87297e7aab36ae151ec377094d5148ed208a67036b9d3a4c4f06"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (6 lines \u00D7 2): pkg/gui/controllers/bisect_controller.go:169-174 | pkg/gui/controllers/bisect_controller.go:187-192 \u2014 both copies are in the same file, so extract the block into one function there and call it from each site \u2014 the copies drift apart the first time only one of them is edited. Read the line range as the matched WINDOW rather than a finished unit: at \u0060pkg/gui/controllers/bisect_controller.go:169\u0060 it does not close everything it opens, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gui/controllers/bisect_controller.go"},"region":{"startLine":169}}}],"partialFingerprints":{"codehealthFindingId/v1":"58a1f060fe03540e2fde62754c9a76c6fcd079b7b1b3576c7a2eb8374dacdf9f"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (6 lines \u00D7 2): pkg/gui/controllers/files_controller.go:1737-1742 | pkg/gui/controllers/files_controller.go:1763-1768 \u2014 both copies are in the same file, so extract the block into one function there and call it from each site \u2014 the copies drift apart the first time only one of them is edited. Read the line range as the matched WINDOW rather than a finished unit: at \u0060pkg/gui/controllers/files_controller.go:1737\u0060 it does not close everything it opens, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that. The matched lines also register a scope-exit action (a \u0060defer\u0060-style statement) that runs when the function holding them returns: moved into a called unit it would run when THAT unit returns instead \u2014 before the caller uses what it releases \u2014 so keep the registration at the call site and extract only the work around it, or have the extracted unit hand the resource back for the caller to register."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gui/controllers/files_controller.go"},"region":{"startLine":1737}}}],"partialFingerprints":{"codehealthFindingId/v1":"dd328b55079f0fe585f2e6704e293fbfe493fbcc8144a025b0dcba87652cfd46"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (6 lines \u00D7 2): pkg/gui/controllers/helpers/branches_helper.go:30-35 | pkg/gui/controllers/helpers/branches_helper.go:107-112 \u2014 both copies are in the same file, so extract the block into one function there and call it from each site \u2014 the copies drift apart the first time only one of them is edited. Read the line range as the matched WINDOW rather than a finished unit: at \u0060pkg/gui/controllers/helpers/branches_helper.go:30\u0060 it begins part-way through the construct above it, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gui/controllers/helpers/branches_helper.go"},"region":{"startLine":30}}}],"partialFingerprints":{"codehealthFindingId/v1":"cf7d1d55ae5a96d8a0effc6e69299324e0d9f8b05d8d3635dc862bdf2f6cd06f"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (6 lines \u00D7 2): pkg/gui/controllers/local_commits_controller.go:1318-1323 | pkg/gui/controllers/local_commits_controller.go:1341-1346 \u2014 both copies are in the same file, so extract the block into one function there and call it from each site \u2014 the copies drift apart the first time only one of them is edited. Read the line range as the matched WINDOW rather than a finished unit: at \u0060pkg/gui/controllers/local_commits_controller.go:1318\u0060 it does not close everything it opens, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gui/controllers/local_commits_controller.go"},"region":{"startLine":1318}}}],"partialFingerprints":{"codehealthFindingId/v1":"9bce995d1266d232286bce3b4240e131ec8ef35df76ac27de317bb2adb73cb8a"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (5 lines \u00D7 2): pkg/gui/controllers/custom_patch_options_menu_action.go:135-139 | pkg/gui/controllers/patch_building_controller.go:226-230 \u2014 the copies sit in sibling files of one directory: extract the block into a single shared function in that directory and call it from each site, so a change lands once. Read the line range as the matched WINDOW rather than a finished unit: at \u0060pkg/gui/controllers/custom_patch_options_menu_action.go:135\u0060 it does not close everything it opens, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gui/controllers/custom_patch_options_menu_action.go"},"region":{"startLine":135}}}],"partialFingerprints":{"codehealthFindingId/v1":"bfa888cd51ddc1beac0bfe6e16aacec5cca5cdc5402954795c9a58045f06f2b9"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (5 lines \u00D7 2): pkg/gui/controllers/helpers/refs_helper.go:282-286 | pkg/gui/controllers/workspace_reset_controller.go:163-167 \u2014 the copies span different directories, so extracting a shared function means choosing where it lives: put it somewhere both call sites can already reach \u2014 a location they all depend on today, or a new shared one if there is none \u2014 and call it from each site; until then, every change has to be made twice. Read the line range as the matched WINDOW rather than a finished unit: at \u0060pkg/gui/controllers/helpers/refs_helper.go:282\u0060 it does not close everything it opens, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that. Note that the copies do not run to the end of the range shown: their LAST lines are different code, not the same code under different names \u2014 the matched region ends inside that line. Extract the lines above it, and read the last line of each site separately."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gui/controllers/helpers/refs_helper.go"},"region":{"startLine":282}}}],"partialFingerprints":{"codehealthFindingId/v1":"1f75fb596e7deab9a87eaf81f39d851421597b6f5d4442cd038997f68160e0e0"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (5 lines \u00D7 2): pkg/gui/controllers/helpers/working_tree_helper.go:236-241 | pkg/gui/controllers/helpers/working_tree_helper.go:258-262 \u2014 both copies are in the same file, so extract the block into one function there and call it from each site \u2014 the copies drift apart the first time only one of them is edited. Read the line range as the matched WINDOW rather than a finished unit: at \u0060pkg/gui/controllers/helpers/working_tree_helper.go:236\u0060 it does not close everything it opens, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that. Note that the copies do not run to the end of the range shown: their LAST lines are different code, not the same code under different names \u2014 the matched region ends inside that line. Extract the lines above it, and read the last line of each site separately."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gui/controllers/helpers/working_tree_helper.go"},"region":{"startLine":236}}}],"partialFingerprints":{"codehealthFindingId/v1":"4a18a6ed0bcb0eb46511f852355402cf777eae15190eadd2d43905df1530ebd6"}},{"ruleId":"D15","level":"warning","message":{"text":"Hotspot: pkg/gui/controllers/helpers/refresh_helper.go: pkg/gui/controllers/helpers/refresh_helper.go changed 72 times in last 90 days, max complexity 40. Frequent change and high complexity in one file compound: schedule the next change to it to include carving out the part being edited, behind tests written first."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gui/controllers/helpers/refresh_helper.go"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"e337e610fad070e8b4ae54efcfcf5e59a19bc785358a31528f437146aa1b3d6c"}},{"ruleId":"D15","level":"warning","message":{"text":"Hotspot: pkg/gocui/gui.go: pkg/gocui/gui.go changed 21 times in last 90 days, max complexity 49. 1 of those changes was a fix/bug commit, and the other 20 changed it for other reasons \u2014 this file is under both repair and feature pressure. Before the next change lands here, cover the area it touches with tests, then split that area out of the file so the following change is smaller than this one \u2014 a file this often edited pays the complexity back every time."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gocui/gui.go"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"8d121b1b302cd328a4c84973225a329fa5d812626ef469fc506d4795a24f6490"}},{"ruleId":"D15","level":"warning","message":{"text":"Hotspot: pkg/gui/gui.go: pkg/gui/gui.go changed 25 times in last 90 days, max complexity 20. 1 of those changes was a fix/bug commit, and the other 24 changed it for other reasons \u2014 this file is under both repair and feature pressure. Before the next change lands here, cover the area it touches with tests, then split that area out of the file so the following change is smaller than this one \u2014 a file this often edited pays the complexity back every time."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gui/gui.go"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"6823aefc08b8e77c5fe8133f3d076be989c60695e8a5957f54fbaa397408c061"}},{"ruleId":"D15","level":"warning","message":{"text":"Hotspot: pkg/gocui/escape.go: pkg/gocui/escape.go changed 5 times in last 90 days, max complexity 69. Frequent change and high complexity in one file compound: schedule the next change to it to include carving out the part being edited, behind tests written first."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gocui/escape.go"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"1f0eea2f4314bdd42b2e5de34285dd7794139708f0232897deef244b2273b90c"}},{"ruleId":"D15","level":"warning","message":{"text":"Hotspot: pkg/tasks/tasks.go: pkg/tasks/tasks.go changed 10 times in last 90 days, max complexity 33. 1 of those changes was a fix/bug commit, and the other 9 changed it for other reasons \u2014 this file is under both repair and feature pressure. Before the next change lands here, cover the area it touches with tests, then split that area out of the file so the following change is smaller than this one \u2014 a file this often edited pays the complexity back every time."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/tasks/tasks.go"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"33f123ca545d5fc930ec0ba2b5012e0886078e86c9ad8078749547120395014c"}},{"ruleId":"D15","level":"warning","message":{"text":"Hotspot: pkg/gui/controllers/helpers/merge_and_rebase_helper.go: pkg/gui/controllers/helpers/merge_and_rebase_helper.go changed 19 times in last 90 days, max complexity 15. Frequent change and high complexity in one file compound: schedule the next change to it to include carving out the part being edited, behind tests written first."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gui/controllers/helpers/merge_and_rebase_helper.go"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"72926efdaedd89ef3073e76ce80d6ebd7a10b077d3192f6e3f36ae61f9367acc"}},{"ruleId":"D15","level":"warning","message":{"text":"Hotspot: pkg/gui/layout.go: pkg/gui/layout.go changed 5 times in last 90 days, max complexity 42. 2 of those changes were fix/bug commits, and the other 3 changed it for other reasons \u2014 this file is under both repair and feature pressure. Before the next change lands here, cover the area it touches with tests, then split that area out of the file so the following change is smaller than this one \u2014 a file this often edited pays the complexity back every time."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gui/layout.go"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"23aefa9911004d75bcca0b148e6ce1dd42c26447bd68eca8ab4307cebc3bf097"}},{"ruleId":"D15","level":"warning","message":{"text":"Hotspot: pkg/gocui/view.go: pkg/gocui/view.go changed 11 times in last 90 days, max complexity 19. Frequent change and high complexity in one file compound: schedule the next change to it to include carving out the part being edited, behind tests written first."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gocui/view.go"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"d980335f79bf6d0a54e1f281d412d5bba5c2af8352606dd60e1d7fef071b6117"}},{"ruleId":"D15","level":"warning","message":{"text":"Hotspot: pkg/gocui/tcell_driver.go: pkg/gocui/tcell_driver.go changed 5 times in last 90 days, max complexity 35. Frequent change and high complexity in one file compound: schedule the next change to it to include carving out the part being edited, behind tests written first."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gocui/tcell_driver.go"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"25963b2e8e4f1b2a82197404ba84ef5676cb792f8f73cbf8b472264c54934995"}},{"ruleId":"D15","level":"warning","message":{"text":"Hotspot: pkg/config/app_config.go: pkg/config/app_config.go changed 6 times in last 90 days, max complexity 18. Frequent change and high complexity in one file compound: schedule the next change to it to include carving out the part being edited, behind tests written first."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/config/app_config.go"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"7381fbe8178569938a26357d5323779f66560c48def40da2719605249b745893"}},{"ruleId":"D16","level":"note","message":{"text":"Off-boarding risk: anonymized user #1: If anonymized user #1 becomes unavailable, 120 significant file(s) lose their only recent owner: pkg/gui/controllers/helpers/refresh_helper.go, pkg/gui/controllers/local_commits_controller.go, pkg/gocui/gui.go, pkg/gocui/view.go, pkg/gui/controllers/branches_controller.go, pkg/gui/controllers/helpers/merge_and_rebase_helper.go, pkg/integration/components/view_driver.go, pkg/gui/controllers/helpers/refs_helper.go (\u002B112 more). Pair on, review, or document these before any departure."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"2e4a7a216ac36b12d188d6ae4f02f09bbb104a0a7cfc83dca95fa8b2a85c1b93"}},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"d5b8f11381d61a58459ffc7c595fd52d09c03c711894a63a9578043d2338412b"},"taxa":[{"id":"CWE-829","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"7ceee73d76663034321cbd3b5bbecae8b673163039b70b12462f979a331e73d3"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"e7a9139ada736d07ff392922c41056ab290cbfa563beec669fc516345f13b2e4"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"5b8626211601d6a7ba6f641f410f4535c648612ca4595244d22117ca4fe9bc16"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"8eb50784ab958284eb3ecd7050384e4061ddeae0c76cf46d9d242ff3f816fc6b"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"c4824e50983c727b5b022c31156fa276f550f89dba1cc06eba7a7ebefc650eda"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"a4385f6fba218ba1d8f89cd3e243e90456fccd20a2af785b5f4f9849ba23f325"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"77765882d55462152a369fef948de5215108ca7f45d82e78ff505271ac66abac"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"c50de480766a7b165a1a642a67af95d5649f6621ffe29fc52e4448f771e31ddb"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"4e5ba20d569d258ae6d0a24953bb1bf2096a834bc4a53403da7c12ca10d2411b"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"2f9837116a08b8e6d1c399be63869d7e00828011b478e4dec43ad56ec6d8fca0"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"229f33f5083a1fefa5ebd7f3b2a50f455445c022072eadf8ca675541407f621d"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"63f6fe19604e9bbc201805ab413127670d457ca05c4ff3d9f121266d5a97d87d"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"ec18c7e38200a3ea39c3a5970f9d5b0810fd43fe6d132e294dfa65a87ec165c2"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"50854385222ed7f5d1948613cf1ac36ac8082a917ec3d410cf0276c5ba2053ff"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"154e79376669773c5d0b9fdf78d6c851f5f1957a6b54b003e1633160453b1407"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"18ffbbe72349beeca02b813e7a731cf97c1989f541de37e2ac5589abbb02bc76"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"38d9c95fee026a62ceadc6993f4a4c210676e4e6c83cecde108b0f9c4b162a6a"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"8035557874aea6d453f1ac510e51d138a9e685ca80c8ac1b5f00213db3f50a23"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"a616f6c11800bd17fafcb5c16d00f0732af9c44ca05635ea5dd728fcce94161c"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"52074a871461e75be9d433631f4c80cb30cc3706531f5dd745d446efb4a086e4"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"c15e03ac1fa47e87e1db1facbff939b0109157aa1a15ea9510ba7aa38b6e07cb"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"160f8e4f73e01d60a8b0f3a55ca62cade4ad4cf4c5bac6b888bcc7b6a7289d91"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"cab79e38e0e5a59b49f54c6bbfc802ec8726628d462a1ac1e757bffbc787a530"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"warning","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"aadc8cdf36a04788a512954d67d9e72775b6a7aa939c7af57ab4cf1e1f958797"},"taxa":[{"id":"CWE-319","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"warning","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"655243207986e551272912095882d8d3135d1ae9078b377ff6a87a0413db28a0"},"taxa":[{"id":"CWE-338","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"warning","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"788a989324b62e72dd5eb04ccb3bc0a04b1665083dffc3f004967e34ed87f15d"},"taxa":[{"id":"CWE-338","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"warning","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"db5149ff2d08db67d19dcc44d023b534b10cfeef91167c97651bc8aabd4c3ba9"},"taxa":[{"id":"CWE-328","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"warning","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"35e2565dde0ef2bdb8f57038895025b822faaad64ef3fc731b706dd282b6d436"},"taxa":[{"id":"CWE-338","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"warning","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"b1dedbba8ae6671a5e2c1adbfe61d80736a4e7449b9bc925d47b705c09a3aef5"},"taxa":[{"id":"CWE-338","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"note","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"9fd349e2532ba426412d4ba51b31d8df73b16338e5d65047acd79e3c03aac3b9"},"taxa":[{"id":"CWE-242","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"note","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"77ac1dfbb96de8f73e0fcd7905b2b0294981e2efc6ab9a6ebce39b0dfcecd7d6"},"taxa":[{"id":"CWE-242","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"note","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"e748d87c0a2ede7bedce7a84d5e0bfde1f23280b172248b49c1f172a227e94b5"},"taxa":[{"id":"CWE-242","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"note","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"24b681fc5c35506a51cec8c7a9dda0b3d889c103484c0c80ed5dfcd9696c5319"},"taxa":[{"id":"CWE-242","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"note","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"ae8ca07999ed26b25d597092cadcdfb5c14b7acac5f18a68d02cd20d4c8b74ec"},"taxa":[{"id":"CWE-242","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"note","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"cf20786a405c424c9e985cd0efec0b1ef2883a70b51b40974d18604e2c4d795a"},"taxa":[{"id":"CWE-242","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"note","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"62d2d337af1353f7c3668913f5c1eb258fe2284b979acfb2d9dca2408cd04519"},"taxa":[{"id":"CWE-242","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D31","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"fe2d688610b51e0a5a02c7c5dd1724a4986e40f2689fb84510e2ec0164c27d6a"}},{"ruleId":"D31","level":"warning","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"b67787f58bbdcde79a43cc08af597219f8bc8ccd5a82b8588a8f7af6b62ec402"}},{"ruleId":"D34","level":"note","message":{"text":"Further orphaned files (smaller): 19 of 229 analysed file(s) have no living knowledge left \u2014 their last meaningful change has decayed away, so if one breaks, no one currently understands it (counted over production source files of roughly 100 lines or more, excluding tests, vendored, generated and example/demo trees, largest first). None is large enough to earn a read-through of its own, so this row stands in for the per-file rows rather than raising one each \u2014 largest first: pkg/integration/components/random.go, pkg/updates/updates.go, pkg/gui/context/context.go (and 16 more). Attach the read to the next change that touches one of them: have a second person review that change, and leave behind a short comment or test recording what the file is for, so the knowledge comes back at the cost of a change you were making anyway."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"ce861fd78f6935114d3a342cb90ad25870f984e1042954fcbbe27c0e23be3658"}},{"ruleId":"D35","level":"warning","message":{"text":"Change coupling: status_controller.go \u2194 status.go: \u0060pkg/gui/controllers/status_controller.go\u0060 and \u0060pkg/gui/presentation/status.go\u0060 change together 70% of the time (7 of the 10 commits that touched whichever of the two files changed less often, counting a file under its earlier names as well) with no explicit dependency \u2014 the edge is real but nothing declares it. Read the pair before acting: if one registers itself into the other through a hook or an initialiser, the missing dependency is DELIBERATE \u2014 the registration is the link, and it is meant not to be an import \u2014 and the thing to add is a comment on each side naming the other, not a merge; if they simply belong together, co-locate them; if neither holds, the coupling is hidden and worth breaking."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gui/controllers/status_controller.go"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"f763406ca45a8d8871331f02cd30857d4d8aa9fb0d859d2f729eccd0fe9ed4c5"}},{"ruleId":"D35","level":"warning","message":{"text":"Change coupling: app.go \u2194 dummies.go: \u0060pkg/app/app.go\u0060 and \u0060pkg/gui/dummies.go\u0060 change together 62% of the time (8 of the 13 commits that touched whichever of the two files changed less often, counting a file under its earlier names as well) with no explicit dependency \u2014 the edge is real but nothing declares it. Read the pair before acting: if one registers itself into the other through a hook or an initialiser, the missing dependency is DELIBERATE \u2014 the registration is the link, and it is meant not to be an import \u2014 and the thing to add is a comment on each side naming the other, not a merge; if they simply belong together, co-locate them; if neither holds, the coupling is hidden and worth breaking."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/app/app.go"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"609f34a17364e4781783eccefa0d8c2ac5eaa4cc4872589500dec3d63083ae60"}},{"ruleId":"D35","level":"warning","message":{"text":"Change coupling: cli.go \u2194 main.go: \u0060pkg/integration/clients/cli.go\u0060 and \u0060pkg/integration/clients/injector/main.go\u0060 change together 60% of the time (6 of the 10 commits that touched whichever of the two files changed less often, counting a file under its earlier names as well) with no explicit dependency \u2014 the edge is real but nothing declares it. Read the pair before acting: if one registers itself into the other through a hook or an initialiser, the missing dependency is DELIBERATE \u2014 the registration is the link, and it is meant not to be an import \u2014 and the thing to add is a comment on each side naming the other, not a merge; if they simply belong together, co-locate them; if neither holds, the coupling is hidden and worth breaking."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/integration/clients/cli.go"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"4b436ee196bcc6472d6b1714e870f78f47eaea66d469f5052aca94aad5b1edc3"}},{"ruleId":"D35","level":"warning","message":{"text":"Change coupling: git.go \u2194 commit_file.go: \u0060pkg/commands/git.go\u0060 and \u0060pkg/commands/models/commit_file.go\u0060 change together 57% of the time (8 of the 14 commits that touched whichever of the two files changed less often, counting a file under its earlier names as well) with no explicit dependency \u2014 the edge is real but nothing declares it. Read the pair before acting: if one registers itself into the other through a hook or an initialiser, the missing dependency is DELIBERATE \u2014 the registration is the link, and it is meant not to be an import \u2014 and the thing to add is a comment on each side naming the other, not a merge; if they simply belong together, co-locate them; if neither holds, the coupling is hidden and worth breaking."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/commands/git.go"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"3930e9b79a849f8d2f4be80acb1a39b2aa20a5796591483de21d51a4d44050de"}},{"ruleId":"D35","level":"warning","message":{"text":"Change coupling: commit.go \u2194 diff.go: \u0060pkg/commands/git_commands/commit.go\u0060 and \u0060pkg/commands/git_commands/diff.go\u0060 change together 50% of the time (9 of the 18 commits that touched whichever of the two files changed less often, counting a file under its earlier names as well). They sit in the same directory, and in this ecosystem sibling files there normally share one namespace/package \u2014 so a direct reference between them needs no import and this pass cannot see whether one exists. Read the pair before acting: if one file only DECLARES what the other consumes (a constants/types file beside its user), the co-change is definitional and the question is whether the split earns its keep; if they duplicate structure, extract the common part into a shared function or type they both call; if neither holds, the coupling is hidden and worth breaking."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/commands/git_commands/commit.go"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"01d154a17f6fa1b9ed88f2f03b3bbce253e8482d02cc51739d14e24fa4e9f290"}},{"ruleId":"D35","level":"warning","message":{"text":"Change coupling: commits.go \u2194 reflog_commits.go: \u0060pkg/gui/presentation/commits.go\u0060 and \u0060pkg/gui/presentation/reflog_commits.go\u0060 change together 50% of the time (7 of the 14 commits that touched whichever of the two files changed less often, counting a file under its earlier names as well). They sit in the same directory, and in this ecosystem sibling files there normally share one namespace/package \u2014 so a direct reference between them needs no import and this pass cannot see whether one exists. Read the pair before acting: if one file only DECLARES what the other consumes (a constants/types file beside its user), the co-change is definitional and the question is whether the split earns its keep; if they duplicate structure, extract the common part into a shared function or type they both call; if neither holds, the coupling is hidden and worth breaking."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/gui/presentation/commits.go"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"09598310f184d981cf5ad0113520546eeada2ef909db45fb021f68fb712ad4b7"}},{"ruleId":"D36","level":"note","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"1b213f6eedd4b140d0bc37bdf1496f72811a643f34064f12518b32e9e83bcfc7"}},{"ruleId":"D36","level":"note","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"0e17f71490e4d120a48b2b2881ab866c93b272bef2febb673a3e8e42b2c288ab"}},{"ruleId":"D36","level":"note","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"eb00976a698a5d68999b7ee6d27206fd374e916853e7ff1ead5eed42386f043f"}},{"ruleId":"D36","level":"warning","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"90f83b4fa27db32740afe9540c905f3c0499caed87f61049a8a903ecc95b41c3"}},{"ruleId":"D36","level":"warning","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"0752d0df7434000fcabf1048e2de30a7a1a8b982b3db9a19898c4dec199856b4"}}],"taxonomies":[{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d","organization":"MITRE","informationUri":"https://cwe.mitre.org/","isComprehensive":false,"shortDescription":{"text":"The MITRE Common Weakness Enumeration (CWE)."},"taxa":[{"id":"CWE-1032","guid":"5f21e517-68aa-a650-9a25-5771ef024637","name":"OWASP Top Ten \u2014 Security Misconfiguration category","shortDescription":{"text":"OWASP Top Ten \u2014 Security Misconfiguration category"},"helpUri":"https://cwe.mitre.org/data/definitions/1032.html"},{"id":"CWE-1357","guid":"e4d2e772-757e-0a5c-bd7d-77052949d866","name":"Reliance on Insufficiently Trustworthy Component","shortDescription":{"text":"Reliance on Insufficiently Trustworthy Component"},"helpUri":"https://cwe.mitre.org/data/definitions/1357.html"},{"id":"CWE-16","guid":"659db3ea-affc-8453-8add-c1218fbfcb92","name":"Configuration","shortDescription":{"text":"Configuration"},"helpUri":"https://cwe.mitre.org/data/definitions/16.html"},{"id":"CWE-242","guid":"5c147e5c-be37-b051-9928-764ef5a8f3d3","name":"CWE-242","shortDescription":{"text":"CWE-242"},"helpUri":"https://cwe.mitre.org/data/definitions/242.html"},{"id":"CWE-259","guid":"ae9ad959-fbb6-9d5e-892d-3dca66da0b69","name":"Use of Hard-coded Password","shortDescription":{"text":"Use of Hard-coded Password"},"helpUri":"https://cwe.mitre.org/data/definitions/259.html"},{"id":"CWE-319","guid":"7af97476-9f0d-4458-9163-3d2043918a1a","name":"CWE-319","shortDescription":{"text":"CWE-319"},"helpUri":"https://cwe.mitre.org/data/definitions/319.html"},{"id":"CWE-328","guid":"f0ffc869-97c3-dc5c-8825-be1159f1fc5f","name":"CWE-328","shortDescription":{"text":"CWE-328"},"helpUri":"https://cwe.mitre.org/data/definitions/328.html"},{"id":"CWE-338","guid":"e89b7604-c3f7-8b53-bc8c-4d4718c28617","name":"CWE-338","shortDescription":{"text":"CWE-338"},"helpUri":"https://cwe.mitre.org/data/definitions/338.html"},{"id":"CWE-353","guid":"09d7e902-d4ee-f05d-ae6c-0a1554d0c18f","name":"CWE-353","shortDescription":{"text":"CWE-353"},"helpUri":"https://cwe.mitre.org/data/definitions/353.html"},{"id":"CWE-494","guid":"b8a65e0d-e459-4a55-a931-fc1136482375","name":"Download of Code Without Integrity Check","shortDescription":{"text":"Download of Code Without Integrity Check"},"helpUri":"https://cwe.mitre.org/data/definitions/494.html"},{"id":"CWE-732","guid":"1da27e8f-b330-7650-ab63-bd61953eae5d","name":"Incorrect Permission Assignment for Critical Resource","shortDescription":{"text":"Incorrect Permission Assignment for Critical Resource"},"helpUri":"https://cwe.mitre.org/data/definitions/732.html"},{"id":"CWE-77","guid":"332c8ade-6612-9f56-a06b-d8d90b1a8750","name":"Command Injection","shortDescription":{"text":"Command Injection"},"helpUri":"https://cwe.mitre.org/data/definitions/77.html"},{"id":"CWE-78","guid":"2e31ceaf-c7ae-2e5e-9661-cfb1362789cf","name":"OS Command Injection","shortDescription":{"text":"OS Command Injection"},"helpUri":"https://cwe.mitre.org/data/definitions/78.html"},{"id":"CWE-79","guid":"fd45580b-e8c4-fc5e-8c2f-aa8fab0b4dbf","name":"Cross-site Scripting (XSS)","shortDescription":{"text":"Cross-site Scripting (XSS)"},"helpUri":"https://cwe.mitre.org/data/definitions/79.html"},{"id":"CWE-798","guid":"5e8f057d-fee3-995a-a0cb-9fc5b0d174d1","name":"Use of Hard-coded Credentials","shortDescription":{"text":"Use of Hard-coded Credentials"},"helpUri":"https://cwe.mitre.org/data/definitions/798.html"},{"id":"CWE-829","guid":"13c33925-97fb-5a5e-b40c-56d328b8a4d7","name":"CWE-829","shortDescription":{"text":"CWE-829"},"helpUri":"https://cwe.mitre.org/data/definitions/829.html"},{"id":"CWE-89","guid":"6d08fdad-37eb-c150-bbf0-d7d946863407","name":"SQL Injection","shortDescription":{"text":"SQL Injection"},"helpUri":"https://cwe.mitre.org/data/definitions/89.html"},{"id":"CWE-94","guid":"75e7f50c-6c2f-dd52-bf40-bf6c52b861fd","name":"Code Injection","shortDescription":{"text":"Code Injection"},"helpUri":"https://cwe.mitre.org/data/definitions/94.html"}]}],"properties":{"codehealthPublication":{"public":true,"notice":"This is the PUBLIC form of this artifact. Findings are listed in full, but the details of SECURITY findings \u2014 which rule fired, in which file, on which line, and how to fix it \u2014 are deliberately withheld, and any secret-scanner results are excluded entirely. Where detail is absent here it was REMOVED FOR PUBLICATION; it is not missing from the analysis. The complete artifact is available from the repository owner.","securityFindingsRedacted":44,"secretScannerRunsExcluded":0}},"redactionTokens":["A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."]}]}