{"$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":"D5","name":"Coupling","shortDescription":{"text":"Coupling"},"helpUri":"https://codehealth.canine.dev/dimensions/D5"},{"id":"D6","name":"Cohesion (LCOM4)","shortDescription":{"text":"Cohesion (LCOM4)"},"helpUri":"https://codehealth.canine.dev/dimensions/D6"},{"id":"D9","name":"Test Distribution","shortDescription":{"text":"Test Distribution"},"helpUri":"https://codehealth.canine.dev/dimensions/D9"},{"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":"D17","name":"Explicit Debt","shortDescription":{"text":"Explicit Debt"},"helpUri":"https://codehealth.canine.dev/dimensions/D17"},{"id":"D19","name":"Documentation Quality","shortDescription":{"text":"Documentation Quality"},"helpUri":"https://codehealth.canine.dev/dimensions/D19"},{"id":"D20","name":"ADR Quality","shortDescription":{"text":"ADR Quality"},"helpUri":"https://codehealth.canine.dev/dimensions/D20"},{"id":"D21","name":"Naming Consistency","shortDescription":{"text":"Naming Consistency"},"helpUri":"https://codehealth.canine.dev/dimensions/D21"},{"id":"D22","name":"Internal API Consistency","shortDescription":{"text":"Internal API Consistency"},"helpUri":"https://codehealth.canine.dev/dimensions/D22"},{"id":"D26","name":"Project Cohesion","shortDescription":{"text":"Project Cohesion"},"helpUri":"https://codehealth.canine.dev/dimensions/D26"},{"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":"D30","name":"Dependency Vulnerabilities","shortDescription":{"text":"Dependency Vulnerabilities"},"helpUri":"https://codehealth.canine.dev/dimensions/D30","relationships":[{"target":{"id":"CWE-1395","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]},{"target":{"id":"CWE-937","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]}],"properties":{"cwe":["CWE-1395","CWE-937"]}},{"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"]}},{"id":"D43","name":"Malicious Dependencies","shortDescription":{"text":"Malicious Dependencies"},"helpUri":"https://codehealth.canine.dev/dimensions/D43","relationships":[{"target":{"id":"CWE-506","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]}],"properties":{"cwe":["CWE-506"]}},{"id":"D44","name":"Platform End-of-Life","shortDescription":{"text":"Platform End-of-Life"},"helpUri":"https://codehealth.canine.dev/dimensions/D44"},{"id":"AC3","name":"Page structure","shortDescription":{"text":"Page structure"},"helpUri":"https://codehealth.canine.dev/dimensions/AC3"},{"id":"AC6","name":"Visual \u0026 motion safety","shortDescription":{"text":"Visual \u0026 motion safety"},"helpUri":"https://codehealth.canine.dev/dimensions/AC6"},{"id":"AC7","name":"A11y enforcement","shortDescription":{"text":"A11y enforcement"},"helpUri":"https://codehealth.canine.dev/dimensions/AC7"},{"id":"AX10","name":"Code composition","shortDescription":{"text":"Code composition"},"helpUri":"https://codehealth.canine.dev/dimensions/AX10"},{"id":"AX3","name":"Project dependency cycles","shortDescription":{"text":"Project dependency cycles"},"helpUri":"https://codehealth.canine.dev/dimensions/AX3"},{"id":"AX4","name":"Dependency direction","shortDescription":{"text":"Dependency direction"},"helpUri":"https://codehealth.canine.dev/dimensions/AX4"},{"id":"AX8","name":"Test isolation","shortDescription":{"text":"Test isolation"},"helpUri":"https://codehealth.canine.dev/dimensions/AX8"},{"id":"AX9","name":"CQS / query purity","shortDescription":{"text":"CQS / query purity"},"helpUri":"https://codehealth.canine.dev/dimensions/AX9"},{"id":"DM4","name":"Rich vs anemic domain model","shortDescription":{"text":"Rich vs anemic domain model"},"helpUri":"https://codehealth.canine.dev/dimensions/DM4"},{"id":"DM5","name":"Encapsulated state","shortDescription":{"text":"Encapsulated state"},"helpUri":"https://codehealth.canine.dev/dimensions/DM5"},{"id":"DM6","name":"Domain \u2194 infrastructure boundary","shortDescription":{"text":"Domain \u2194 infrastructure boundary"},"helpUri":"https://codehealth.canine.dev/dimensions/DM6"},{"id":"ES1","name":"Fold determinism","shortDescription":{"text":"Fold determinism"},"helpUri":"https://codehealth.canine.dev/dimensions/ES1"},{"id":"ES2","name":"Immutable events","shortDescription":{"text":"Immutable events"},"helpUri":"https://codehealth.canine.dev/dimensions/ES2"},{"id":"M1","name":"Documentation (README)","shortDescription":{"text":"Documentation (README)"},"helpUri":"https://codehealth.canine.dev/dimensions/M1"},{"id":"M2","name":"Architecture documentation","shortDescription":{"text":"Architecture documentation"},"helpUri":"https://codehealth.canine.dev/dimensions/M2"},{"id":"M3","name":"Folder \u0026 project structure","shortDescription":{"text":"Folder \u0026 project structure"},"helpUri":"https://codehealth.canine.dev/dimensions/M3"},{"id":"M4","name":"Documentation accuracy","shortDescription":{"text":"Documentation accuracy"},"helpUri":"https://codehealth.canine.dev/dimensions/M4"},{"id":"P1","name":"CI/CD gates","shortDescription":{"text":"CI/CD gates"},"helpUri":"https://codehealth.canine.dev/dimensions/P1"},{"id":"P12","name":"CI test-gate honesty","shortDescription":{"text":"CI test-gate honesty"},"helpUri":"https://codehealth.canine.dev/dimensions/P12"},{"id":"P2","name":"Observability","shortDescription":{"text":"Observability"},"helpUri":"https://codehealth.canine.dev/dimensions/P2"},{"id":"P3","name":"Security \u0026 performance tooling","shortDescription":{"text":"Security \u0026 performance tooling"},"helpUri":"https://codehealth.canine.dev/dimensions/P3"},{"id":"P6","name":"Release Hygiene","shortDescription":{"text":"Release Hygiene"},"helpUri":"https://codehealth.canine.dev/dimensions/P6"},{"id":"P8","name":"Schema migrations","shortDescription":{"text":"Schema migrations"},"helpUri":"https://codehealth.canine.dev/dimensions/P8"},{"id":"PF3","name":"Async \u0026 latency hygiene","shortDescription":{"text":"Async \u0026 latency hygiene"},"helpUri":"https://codehealth.canine.dev/dimensions/PF3"},{"id":"SC1","name":"Supply-chain hygiene","shortDescription":{"text":"Supply-chain hygiene"},"helpUri":"https://codehealth.canine.dev/dimensions/SC1"},{"id":"X9","name":"Subsumed condition operand","shortDescription":{"text":"Subsumed condition operand"},"helpUri":"https://codehealth.canine.dev/dimensions/X9"}]}},"results":[{"ruleId":"D1","level":"warning","message":{"text":"UI::handle_subcommands (cyclomatic 83): UI::handle_subcommands has cyclomatic complexity 83 (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. This file is where this pass\u0027s cyclomatic complexity CONCENTRATES: crates/forge_main/src/ui.rs holds 4 of the 38 functions over the threshold \u2014 including the worst \u2014 and 123 of the 436 points over it (28%), 2\u00D7 the next-largest file (crates/forge_select/src/preview.rs at 61). No single row can show this, because each is measured only against the threshold: reducing this one file moves this dimension further than any other file in the repository."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_main/src/ui.rs"},"region":{"startLine":466}}}],"partialFingerprints":{"codehealthFindingId/v1":"2b7d4290247185d374c9b41ddc428997cab7197862fd281423cfab05ac4bf74d"}},{"ruleId":"D1","level":"warning","message":{"text":"UI::on_command (cyclomatic 56): UI::on_command has cyclomatic complexity 56 (threshold 15). To reduce it, keep the dispatch but shrink the arms: move each non-trivial case body into its own named function (or onto the value being matched) so the dispatch reads one line per case, and group related cases into a sub-dispatch. Where every arm is uniform \u2014 the same kind of value, with no behaviour of its own \u2014 a table keyed by the case is the shorter form; wherever the arms carry different data or different behaviour, keep them as cases, because collapsing those trades an explicit, reviewable set of cases for nothing. This file is where this pass\u0027s cyclomatic complexity CONCENTRATES: crates/forge_main/src/ui.rs holds 4 of the 38 functions over the threshold \u2014 including the worst \u2014 and 123 of the 436 points over it (28%), 2\u00D7 the next-largest file (crates/forge_select/src/preview.rs at 61). No single row can show this, because each is measured only against the threshold: reducing this one file moves this dimension further than any other file in the repository."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_main/src/ui.rs"},"region":{"startLine":2167}}}],"partialFingerprints":{"codehealthFindingId/v1":"42c31e46b752305c389c47f2b1fa0c2b3c03020211f9045be37d0365d7dbc127"}},{"ruleId":"D1","level":"warning","message":{"text":"ToolOperation::into_tool_output (cyclomatic 51): ToolOperation::into_tool_output has cyclomatic complexity 51 (threshold 15). To reduce it, separate the branches: extract each independent case into its own named function so the top-level body reads as a short sequence of named decisions."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_app/src/operation.rs"},"region":{"startLine":235}}}],"partialFingerprints":{"codehealthFindingId/v1":"d689c4bde303ef83361db19c9c1f7be6836e931d64aef3b9d57894a756e940d4"}},{"ruleId":"D1","level":"warning","message":{"text":"forge_app::utils::sanitize_gemini_schema (cyclomatic 45): forge_app::utils::sanitize_gemini_schema has cyclomatic complexity 45 (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":"crates/forge_app/src/utils.rs"},"region":{"startLine":626}}}],"partialFingerprints":{"codehealthFindingId/v1":"55f52d8bff121ea82b5e42ee9fd635ecab62c64dd0d821f1a0862fc3cdc68787"}},{"ruleId":"D1","level":"warning","message":{"text":"forge_select::preview::run_select_ui_values (cyclomatic 43): forge_select::preview::run_select_ui_values has cyclomatic complexity 43 (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":"crates/forge_select/src/preview.rs"},"region":{"startLine":289}}}],"partialFingerprints":{"codehealthFindingId/v1":"bbbcfa4c2dbe809fe01767ed58e222fe7f86d33eba753bd743159a6d1f509364"}},{"ruleId":"D1","level":"warning","message":{"text":"JsonRepairParser::parse_string (cyclomatic 37): JsonRepairParser::parse_string has cyclomatic complexity 37 (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":"crates/forge_json_repair/src/parser.rs"},"region":{"startLine":348}}}],"partialFingerprints":{"codehealthFindingId/v1":"a49828177d0730cd2bd04225e7906c6abff632243855dbe91d065e0b80d0747b"}},{"ruleId":"D1","level":"warning","message":{"text":"forge_json_repair::schema_coercion::coerce_value_with_schema_object (cyclomatic 36): forge_json_repair::schema_coercion::coerce_value_with_schema_object has cyclomatic complexity 36 (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":"crates/forge_json_repair/src/schema_coercion.rs"},"region":{"startLine":38}}}],"partialFingerprints":{"codehealthFindingId/v1":"063588ac92c6a56620ca3cb9611dfe31a2629b62267a81e47ff22227136c66e4"}},{"ruleId":"D1","level":"warning","message":{"text":"Renderer::render_event (cyclomatic 36): Renderer::render_event has cyclomatic complexity 36 (threshold 15). To reduce it, keep the dispatch but shrink the arms: move each non-trivial case body into its own named function (or onto the value being matched) so the dispatch reads one line per case, and group related cases into a sub-dispatch. Where every arm is uniform \u2014 the same kind of value, with no behaviour of its own \u2014 a table keyed by the case is the shorter form; wherever the arms carry different data or different behaviour, keep them as cases, because collapsing those trades an explicit, reviewable set of cases for nothing."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_markdown_stream/src/renderer.rs"},"region":{"startLine":126}}}],"partialFingerprints":{"codehealthFindingId/v1":"a89fa3ab9068dfcac09fc5d63dce769f91f83d423668c0d772cfe055a6675d8a"}},{"ruleId":"D1","level":"warning","message":{"text":"forge_select::preview::handle_key_event (cyclomatic 35): forge_select::preview::handle_key_event has cyclomatic complexity 35 (threshold 15). To reduce it, keep the dispatch but shrink the arms: move each non-trivial case body into its own named function (or onto the value being matched) so the dispatch reads one line per case, and group related cases into a sub-dispatch. Where every arm is uniform \u2014 the same kind of value, with no behaviour of its own \u2014 a table keyed by the case is the shorter form; wherever the arms carry different data or different behaviour, keep them as cases, because collapsing those trades an explicit, reviewable set of cases for nothing."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_select/src/preview.rs"},"region":{"startLine":622}}}],"partialFingerprints":{"codehealthFindingId/v1":"f9c6cab8ac80f6cf831e593d1254d6c48ec643a9197d1d5beffa856a494ad1ea"}},{"ruleId":"D1","level":"warning","message":{"text":"forge_markdown_stream::table::wrap (cyclomatic 33): forge_markdown_stream::table::wrap has cyclomatic complexity 33 (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":"crates/forge_markdown_stream/src/table.rs"},"region":{"startLine":135}}}],"partialFingerprints":{"codehealthFindingId/v1":"4f95456916dafada1d5e2e135c2cafd8b2c9d7e9012e0b2d5e3d282295b763df"}},{"ruleId":"D1","level":"warning","message":{"text":"forge_markdown_stream::utils::collapse_ansi_codes (cyclomatic 29): forge_markdown_stream::utils::collapse_ansi_codes has cyclomatic complexity 29 (threshold 15). To reduce it, keep the dispatch but shrink the arms: move each non-trivial case body into its own named function (or onto the value being matched) so the dispatch reads one line per case, and group related cases into a sub-dispatch. Where every arm is uniform \u2014 the same kind of value, with no behaviour of its own \u2014 a table keyed by the case is the shorter form; wherever the arms carry different data or different behaviour, keep them as cases, because collapsing those trades an explicit, reviewable set of cases for nothing."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_markdown_stream/src/utils.rs"},"region":{"startLine":340}}}],"partialFingerprints":{"codehealthFindingId/v1":"e25f675af9e3d235ccd5d3b0a0788add2073e278f639911bc51cd2c6736ac511"}},{"ruleId":"D1","level":"warning","message":{"text":"ChatCompletionMessage::try_from (cyclomatic 28): ChatCompletionMessage::try_from has cyclomatic complexity 28 (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":"crates/forge_app/src/dto/openai/response.rs"},"region":{"startLine":324}}}],"partialFingerprints":{"codehealthFindingId/v1":"6901e13a50365d93ded3bc0aa5c54bf7d5367f3aeabaf02531789248ccea3aaf"}},{"ruleId":"D1","level":"warning","message":{"text":"forge_select::preview::draw_preview_ui (cyclomatic 28): forge_select::preview::draw_preview_ui has cyclomatic complexity 28 (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":"crates/forge_select/src/preview.rs"},"region":{"startLine":875}}}],"partialFingerprints":{"codehealthFindingId/v1":"72f7d99002c1ec837641feb910e1c0f27d948d412ea2c8dfd5b524c53c741fe3"}},{"ruleId":"D1","level":"warning","message":{"text":"ToolCatalog::to_content (cyclomatic 26): ToolCatalog::to_content has cyclomatic complexity 26 (threshold 15). To reduce it, keep the dispatch but shrink the arms: move each non-trivial case body into its own named function (or onto the value being matched) so the dispatch reads one line per case, and group related cases into a sub-dispatch. Where every arm is uniform \u2014 the same kind of value, with no behaviour of its own \u2014 a table keyed by the case is the shorter form; wherever the arms carry different data or different behaviour, keep them as cases, because collapsing those trades an explicit, reviewable set of cases for nothing."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_app/src/fmt/fmt_input.rs"},"region":{"startLine":9}}}],"partialFingerprints":{"codehealthFindingId/v1":"51af3d5d136a2fa4e1769e426ac29a3ddbe5d3a3bc6deffc1442afc301cb41e1"}},{"ruleId":"D1","level":"warning","message":{"text":"JsonRepairParser::parse_unquoted_string (cyclomatic 25): JsonRepairParser::parse_unquoted_string has cyclomatic complexity 25 (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":"crates/forge_json_repair/src/parser.rs"},"region":{"startLine":694}}}],"partialFingerprints":{"codehealthFindingId/v1":"0a8c53d616cd77ad15815f65a1887fe52a12b34ce2ea2cfbbb80d7c621ed4ef9"}},{"ruleId":"D1","level":"warning","message":{"text":"UI::select_model (cyclomatic 24): UI::select_model has cyclomatic complexity 24 (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. This file is where this pass\u0027s cyclomatic complexity CONCENTRATES: crates/forge_main/src/ui.rs holds 4 of the 38 functions over the threshold \u2014 including the worst \u2014 and 123 of the 436 points over it (28%), 2\u00D7 the next-largest file (crates/forge_select/src/preview.rs at 61). No single row can show this, because each is measured only against the threshold: reducing this one file moves this dimension further than any other file in the repository."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_main/src/ui.rs"},"region":{"startLine":2904}}}],"partialFingerprints":{"codehealthFindingId/v1":"0bfc2b856a062d9821d626531d3d2b72d5ccb86da054aec1011f930ff6922459"}},{"ruleId":"D1","level":"warning","message":{"text":"forge_domain::compact::summary::extract_tool_info (cyclomatic 22): forge_domain::compact::summary::extract_tool_info has cyclomatic complexity 22 (threshold 15). To reduce it, keep the dispatch but shrink the arms: move each non-trivial case body into its own named function (or onto the value being matched) so the dispatch reads one line per case, and group related cases into a sub-dispatch. Where every arm is uniform \u2014 the same kind of value, with no behaviour of its own \u2014 a table keyed by the case is the shorter form; wherever the arms carry different data or different behaviour, keep them as cases, because collapsing those trades an explicit, reviewable set of cases for nothing."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_domain/src/compact/summary.rs"},"region":{"startLine":339}}}],"partialFingerprints":{"codehealthFindingId/v1":"f65f63b8720d3efc1c9775154aa0eb99f19766df0af372c62179a118b21e3097"}},{"ruleId":"D1","level":"warning","message":{"text":"forge_json_repair::schema_coercion::try_coerce_string (cyclomatic 21): forge_json_repair::schema_coercion::try_coerce_string has cyclomatic complexity 21 (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":"crates/forge_json_repair/src/schema_coercion.rs"},"region":{"startLine":243}}}],"partialFingerprints":{"codehealthFindingId/v1":"b29695de2f8be8da8ec4c3ad9c8f87db28451ab9d1193f4b8e4da6ee1edcbc16"}},{"ruleId":"D1","level":"warning","message":{"text":"forge_domain::conversation_html::create_conversation_context_section (cyclomatic 20): forge_domain::conversation_html::create_conversation_context_section has cyclomatic complexity 20 (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":"crates/forge_domain/src/conversation_html.rs"},"region":{"startLine":288}}}],"partialFingerprints":{"codehealthFindingId/v1":"bbb228c288c3d78db7907b7d3b6114a1900e2f3076cb2f4798fc921444c68d0e"}},{"ruleId":"D1","level":"warning","message":{"text":"UI::handle_chat_response (cyclomatic 20): UI::handle_chat_response has cyclomatic complexity 20 (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. This file is where this pass\u0027s cyclomatic complexity CONCENTRATES: crates/forge_main/src/ui.rs holds 4 of the 38 functions over the threshold \u2014 including the worst \u2014 and 123 of the 436 points over it (28%), 2\u00D7 the next-largest file (crates/forge_select/src/preview.rs at 61). No single row can show this, because each is measured only against the threshold: reducing this one file moves this dimension further than any other file in the repository."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_main/src/ui.rs"},"region":{"startLine":4140}}}],"partialFingerprints":{"codehealthFindingId/v1":"c9623e24c5476657aa5fa84a20daf95067ef3b3ce74861a31c9f2a9fd6b768b2"}},{"ruleId":"D1","level":"warning","message":{"text":"forge_fs::binary_detection::is_binary_internal (cyclomatic 19): forge_fs::binary_detection::is_binary_internal has cyclomatic complexity 19 (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":"crates/forge_fs/src/binary_detection.rs"},"region":{"startLine":43}}}],"partialFingerprints":{"codehealthFindingId/v1":"eaac86eb42483bf520c7ac7d8e92a7a6a05e3e720799f97adae69f7b95ce7dfd"}},{"ruleId":"D1","level":"warning","message":{"text":"Walker::get_blocking (cyclomatic 19): Walker::get_blocking has cyclomatic complexity 19 (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":"crates/forge_walker/src/walker.rs"},"region":{"startLine":118}}}],"partialFingerprints":{"codehealthFindingId/v1":"5ac07b3ee77d702bd9cc3c627ead55ab78568c150be7282d9cc4bb02684c6abb"}},{"ruleId":"D1","level":"warning","message":{"text":"Agent::apply_config (cyclomatic 18): Agent::apply_config 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":"crates/forge_app/src/agent.rs"},"region":{"startLine":95}}}],"partialFingerprints":{"codehealthFindingId/v1":"9da8c4bd1ea186f1106a7ffecfef1c4dd16e1bf8364dbaacb3d612c13cddef49"}},{"ruleId":"D1","level":"warning","message":{"text":"JsonRepairParser::parse_number (cyclomatic 18): JsonRepairParser::parse_number 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":"crates/forge_json_repair/src/parser.rs"},"region":{"startLine":595}}}],"partialFingerprints":{"codehealthFindingId/v1":"5cd577750ac8b4af1a826214d0be855c50c874d9b081848b600c5694de651d33"}},{"ruleId":"D1","level":"warning","message":{"text":"forge_app::utils::enforce_strict_schema (cyclomatic 17): forge_app::utils::enforce_strict_schema has cyclomatic complexity 17 (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":"crates/forge_app/src/utils.rs"},"region":{"startLine":448}}}],"partialFingerprints":{"codehealthFindingId/v1":"1f36c5db10a0956195133ffc60e0470e71b9b178176b7d6d6aee279ea217f2d6"}},{"ruleId":"D1","level":"warning","message":{"text":"forge_app::dto::anthropic::response::get_context_length (cyclomatic 17): forge_app::dto::anthropic::response::get_context_length has cyclomatic complexity 17 (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":"crates/forge_app/src/dto/anthropic/response.rs"},"region":{"startLine":81}}}],"partialFingerprints":{"codehealthFindingId/v1":"a215eee023dc8462bceb90dcccf2b12baf4f9a8a9fe12350747c6c964e1be4f7"}},{"ruleId":"D1","level":"warning","message":{"text":"ContextSummary::from (cyclomatic 17): ContextSummary::from has cyclomatic complexity 17 (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":"crates/forge_domain/src/compact/summary.rs"},"region":{"startLine":218}}}],"partialFingerprints":{"codehealthFindingId/v1":"afd6edfa80754748fe62f651c4ede1cf9447ab6b1349c61a99551eede14866b2"}},{"ruleId":"D1","level":"warning","message":{"text":"ChatCompletionMessage::try_from (cyclomatic 17): ChatCompletionMessage::try_from has cyclomatic complexity 17 (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":"crates/forge_app/src/dto/google/response.rs"},"region":{"startLine":451}}}],"partialFingerprints":{"codehealthFindingId/v1":"a2a1f98b329c26f72fb4d2a96be29b49170061b609712c7d8591ce242f629f4d"}},{"ruleId":"D1","level":"warning","message":{"text":"ToolCallFull::try_from_parts (cyclomatic 17): ToolCallFull::try_from_parts has cyclomatic complexity 17 (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":"crates/forge_domain/src/tools/call/tool_call.rs"},"region":{"startLine":106}}}],"partialFingerprints":{"codehealthFindingId/v1":"9eace86d644646b79a2f1e9579262d59e23d95b852b4e00c3a4050fb707723af"}},{"ruleId":"D1","level":"warning","message":{"text":"JsonRepairParser::parse_object (cyclomatic 17): JsonRepairParser::parse_object has cyclomatic complexity 17 (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":"crates/forge_json_repair/src/parser.rs"},"region":{"startLine":195}}}],"partialFingerprints":{"codehealthFindingId/v1":"82e3110944a757f83225cd19cb5ab65f8525d434a82bdb75922a4aa43b4ab089"}},{"ruleId":"D1","level":"warning","message":{"text":"ForgeCommandManager::parse (cyclomatic 17): ForgeCommandManager::parse has cyclomatic complexity 17 (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. This is NOT this file\u0027s highest cyclomatic complexity: AppCommand::name (cyclomatic 46) is higher and carries no row of its own \u2014 it was excluded as a flat dispatcher (a long switch/match over independent cases: many branches, almost no nesting), which this dimension does not treat as a refactor obligation. It is named here so the ranking you see in this file is not mistaken for the whole of it; the excluded function is counted neither in this dimension\u0027s figures nor in its score."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_main/src/model.rs"},"region":{"startLine":288}}}],"partialFingerprints":{"codehealthFindingId/v1":"5ff7db334034ea6f6d58ee98e54b311840ba8fd1949c8c43e0676824bf3f923f"}},{"ruleId":"D1","level":"warning","message":{"text":"forge_markdown_stream::table::split_word_at_width (cyclomatic 17): forge_markdown_stream::table::split_word_at_width has cyclomatic complexity 17 (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":"crates/forge_markdown_stream/src/table.rs"},"region":{"startLine":304}}}],"partialFingerprints":{"codehealthFindingId/v1":"9277afe0b785546ee7abd4879dfd19a365dfeee7ed9201613578889d9b993ca8"}},{"ruleId":"D1","level":"warning","message":{"text":"OpenAIResponsesProvider::get_headers_for_conversation (cyclomatic 17): OpenAIResponsesProvider::get_headers_for_conversation has cyclomatic complexity 17 (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":"crates/forge_repo/src/provider/openai_responses/repository.rs"},"region":{"startLine":76}}}],"partialFingerprints":{"codehealthFindingId/v1":"862b0b6fafee603fe5dc94b746d7e5c58db9ece1601d0a82a308773330a804b0"}},{"ruleId":"D1","level":"warning","message":{"text":"forge_services::tool_services::fs_patch::apply_replacement (cyclomatic 17): forge_services::tool_services::fs_patch::apply_replacement has cyclomatic complexity 17 (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":"crates/forge_services/src/tool_services/fs_patch.rs"},"region":{"startLine":202}}}],"partialFingerprints":{"codehealthFindingId/v1":"a8e33a9d883d475661ebfcbdd4887dbdf13697fa18acad1fb21959921bc21d82"}},{"ruleId":"D1","level":"warning","message":{"text":"Response::into_domain (cyclomatic 16): Response::into_domain 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":"crates/forge_repo/src/provider/openai_responses/response.rs"},"region":{"startLine":156}}}],"partialFingerprints":{"codehealthFindingId/v1":"4a91bb72189f3a21ba79f99a4fffbe213f5ca658f282cdb328ff612510c366db"}},{"ruleId":"D1","level":"warning","message":{"text":"DiffFormat::format (cyclomatic 16): DiffFormat::format 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":"crates/forge_display/src/diff.rs"},"region":{"startLine":50}}}],"partialFingerprints":{"codehealthFindingId/v1":"4496f0f9a958cd119bb655b32fc3f7cd4354697e5eabff5c1b7ee930b988db8e"}},{"ruleId":"D1","level":"warning","message":{"text":"OpenAIProvider::get_headers (cyclomatic 16): OpenAIProvider::get_headers has cyclomatic complexity 16 (threshold 15). To reduce it, keep the dispatch but shrink the arms: move each non-trivial case body into its own named function (or onto the value being matched) so the dispatch reads one line per case, and group related cases into a sub-dispatch. Where every arm is uniform \u2014 the same kind of value, with no behaviour of its own \u2014 a table keyed by the case is the shorter form; wherever the arms carry different data or different behaviour, keep them as cases, because collapsing those trades an explicit, reviewable set of cases for nothing."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_repo/src/provider/openai.rs"},"region":{"startLine":72}}}],"partialFingerprints":{"codehealthFindingId/v1":"316c14c003a66bc223776e365e001886cef1ef2280fbf1a9f93deccd20baa986"}},{"ruleId":"D1","level":"warning","message":{"text":"ForgeProviderRepository::migrate_env_to_file (cyclomatic 16): ForgeProviderRepository::migrate_env_to_file 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":"crates/forge_repo/src/provider/provider_repo.rs"},"region":{"startLine":337}}}],"partialFingerprints":{"codehealthFindingId/v1":"7a0990dd368403a69a90dd882e1931599a983e562a20bfc8acbdb054d86944e4"}},{"ruleId":"D2","level":"warning","message":{"text":"forge_app::utils::sanitize_gemini_schema (cognitive 107): forge_app::utils::sanitize_gemini_schema has cognitive complexity 107 (threshold 15). Drivers by points: if/else 23 (64 pts), loops 8 (26 pts), boolean chains 10, match/switch 3 (7 pts) (nesting depth added 63). The drivers above price the dispatch low by construction \u2014 a dispatch is charged once however many cases it lists, while each branch inside an arm is charged in full \u2014 so most of this count is what the case bodies hold, and the arms are where it can be reduced. To reduce it, keep the dispatch but shrink the arms: move each non-trivial case body into its own named function (or onto the value being matched) so the dispatch reads one line per case, and group related cases into a sub-dispatch. Keep every case explicit, and make the behaviour for cases you do not list a deliberate choice rather than an accident."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_app/src/utils.rs"},"region":{"startLine":626}}}],"partialFingerprints":{"codehealthFindingId/v1":"9ef2537f6abc8c84e15a7ae28fe1a7b7bf804b6993f85f889f1066c19b06092b"}},{"ruleId":"D2","level":"warning","message":{"text":"UI::handle_subcommands (cognitive 104): UI::handle_subcommands has cognitive complexity 104 (threshold 15). Drivers by points: if/else 31 (73 pts), match/switch 9 (17 pts), loops 4 (13 pts), boolean chains 1 (nesting depth added 59). 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":"crates/forge_main/src/ui.rs"},"region":{"startLine":466}}}],"partialFingerprints":{"codehealthFindingId/v1":"0f8ef4772e9d5d731765a0403d39f2df6a5dac3bd439d6507d2361dfb4d14623"}},{"ruleId":"D2","level":"warning","message":{"text":"forge_markdown_stream::table::wrap (cognitive 99): forge_markdown_stream::table::wrap has cognitive complexity 99 (threshold 15). Drivers by points: if/else 27 (78 pts), loops 4 (15 pts), boolean chains 6 (nesting depth added 62). To reduce it, flatten the nesting: this score is depth rather than breadth \u2014 most of its points come from checks stacked inside one another, so the work sits several levels in. Invert each enclosing check into an early exit (a return, or the language\u0027s equivalent) so the happy path stays at one level, and where a level cannot be exited early, lift the block it encloses into its own named function."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_markdown_stream/src/table.rs"},"region":{"startLine":135}}}],"partialFingerprints":{"codehealthFindingId/v1":"d38409a252610f802560a7bf9f6d95bdfc81782d8bf5f3645008cca9e556daee"}},{"ruleId":"D2","level":"warning","message":{"text":"JsonRepairParser::parse_string (cognitive 83): JsonRepairParser::parse_string has cognitive complexity 83 (threshold 15). Drivers by points: if/else 27 (61 pts), loops 3 (10 pts), boolean chains 8, match/switch 1 (4 pts) (nesting depth added 44). 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":"crates/forge_json_repair/src/parser.rs"},"region":{"startLine":348}}}],"partialFingerprints":{"codehealthFindingId/v1":"262fdb8fb4408b8a10766b3ef454f266b5ccfcae8c405e770847b0af15534ddc"}},{"ruleId":"D2","level":"warning","message":{"text":"ToolOperation::into_tool_output (cognitive 82): ToolOperation::into_tool_output has cognitive complexity 82 (threshold 15). Drivers by points: if/else 19 (39 pts), loops 9 (26 pts), match/switch 7 (15 pts), boolean chains 2 (nesting depth added 45). The drivers above price the dispatch low by construction \u2014 a dispatch is charged once however many cases it lists, while each branch inside an arm is charged in full \u2014 so most of this count is what the case bodies hold, and the arms are where it can be reduced. To reduce it, keep the dispatch but shrink the arms: move each non-trivial case body into its own named function (or onto the value being matched) so the dispatch reads one line per case, and group related cases into a sub-dispatch. Keep every case explicit, and make the behaviour for cases you do not list a deliberate choice rather than an accident."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_app/src/operation.rs"},"region":{"startLine":235}}}],"partialFingerprints":{"codehealthFindingId/v1":"871282140189c82bea202eb1c36879ef72e203975b379dc84748a3a5f45e5ee9"}},{"ruleId":"D2","level":"warning","message":{"text":"forge_select::preview::run_select_ui_values (cognitive 78): forge_select::preview::run_select_ui_values has cognitive complexity 78 (threshold 15). Drivers by points: if/else 23 (58 pts), match/switch 4 (13 pts), boolean chains 5, loops 2 (nesting depth added 44). 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":"crates/forge_select/src/preview.rs"},"region":{"startLine":289}}}],"partialFingerprints":{"codehealthFindingId/v1":"5fb0e87419355e5a35ec1cc30430d9c0151e8ba84a0d450f6e133b52ca99ad17"}},{"ruleId":"D2","level":"warning","message":{"text":"ChatCompletionMessage::try_from (cognitive 76): ChatCompletionMessage::try_from has cognitive complexity 76 (threshold 15). Drivers by points: if/else 18 (55 pts), loops 2 (10 pts), match/switch 4 (10 pts), boolean chains 1 (nesting depth added 51). The drivers above price the dispatch low by construction \u2014 a dispatch is charged once however many cases it lists, while each branch inside an arm is charged in full \u2014 so most of this count is what the case bodies hold, and the arms are where it can be reduced. To reduce it, keep the dispatch but shrink the arms: move each non-trivial case body into its own named function (or onto the value being matched) so the dispatch reads one line per case, and group related cases into a sub-dispatch. Keep every case explicit, and make the behaviour for cases you do not list a deliberate choice rather than an accident."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_app/src/dto/openai/response.rs"},"region":{"startLine":324}}}],"partialFingerprints":{"codehealthFindingId/v1":"a69432d3e5dbfcacfb96fa6792ece01f956d17412de0ea2e7c25a6d20792d194"}},{"ruleId":"D2","level":"warning","message":{"text":"forge_json_repair::schema_coercion::coerce_value_with_schema_object (cognitive 71): forge_json_repair::schema_coercion::coerce_value_with_schema_object has cognitive complexity 71 (threshold 15). Drivers by points: if/else 24 (55 pts), loops 4 (9 pts), boolean chains 7 (nesting depth added 36). To reduce it, flatten the nesting: this score is depth rather than breadth \u2014 most of its points come from checks stacked inside one another, so the work sits several levels in. Invert each enclosing check into an early exit (a return, or the language\u0027s equivalent) so the happy path stays at one level, and where a level cannot be exited early, lift the block it encloses into its own named function."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_json_repair/src/schema_coercion.rs"},"region":{"startLine":38}}}],"partialFingerprints":{"codehealthFindingId/v1":"c2c78504727021eff6764518c5921c9c90b2f146ec4d13eeb0f224375eaa8ac7"}},{"ruleId":"D2","level":"warning","message":{"text":"forge_markdown_stream::table::split_word_at_width (cognitive 59): forge_markdown_stream::table::split_word_at_width has cognitive complexity 59 (threshold 15). Drivers by points: if/else 14 (46 pts), loops 3 (11 pts), boolean chains 2 (nesting depth added 40). To reduce it, flatten the nesting: this score is depth rather than breadth \u2014 most of its points come from checks stacked inside one another, so the work sits several levels in. Invert each enclosing check into an early exit (a return, or the language\u0027s equivalent) so the happy path stays at one level, and where a level cannot be exited early, lift the block it encloses into its own named function."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_markdown_stream/src/table.rs"},"region":{"startLine":304}}}],"partialFingerprints":{"codehealthFindingId/v1":"dc17391412995542b0a0bfdae7ef4ba5cf450f693844f9ef434c5209330072f1"}},{"ruleId":"D2","level":"warning","message":{"text":"forge_select::preview::draw_preview_ui (cognitive 57): forge_select::preview::draw_preview_ui has cognitive complexity 57 (threshold 15). Drivers by points: if/else 19 (39 pts), loops 5 (8 pts), match/switch 4 (7 pts), boolean chains 3 (nesting depth added 26). 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":"crates/forge_select/src/preview.rs"},"region":{"startLine":875}}}],"partialFingerprints":{"codehealthFindingId/v1":"179155c840c432f85306129be2c3690783b5cd6beeb4e22607d5acdfb8903856"}},{"ruleId":"D2","level":"warning","message":{"text":"forge_domain::conversation_html::create_conversation_context_section (cognitive 55): forge_domain::conversation_html::create_conversation_context_section has cognitive complexity 55 (threshold 15). Drivers by points: if/else 23 (49 pts), match/switch 2 (5 pts), boolean chains 1 (nesting depth added 29). To reduce it, flatten the nesting: this score is depth rather than breadth \u2014 most of its points come from checks stacked inside one another, so the work sits several levels in. Invert each enclosing check into an early exit (a return, or the language\u0027s equivalent) so the happy path stays at one level, and where a level cannot be exited early, lift the block it encloses into its own named function."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_domain/src/conversation_html.rs"},"region":{"startLine":288}}}],"partialFingerprints":{"codehealthFindingId/v1":"ecc07bb886cf62da2a05dfc4c3eea661aaf6bb4c30a6cf0281517737ed86d49d"}},{"ruleId":"D2","level":"warning","message":{"text":"JsonRepairParser::parse_unquoted_string (cognitive 47): JsonRepairParser::parse_unquoted_string has cognitive complexity 47 (threshold 15). Drivers by points: if/else 15 (32 pts), loops 5 (9 pts), boolean chains 6 (nesting depth added 21). To reduce it, flatten the nesting: this score is depth rather than breadth \u2014 most of its points come from checks stacked inside one another, so the work sits several levels in. Invert each enclosing check into an early exit (a return, or the language\u0027s equivalent) so the happy path stays at one level, and where a level cannot be exited early, lift the block it encloses into its own named function."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_json_repair/src/parser.rs"},"region":{"startLine":694}}}],"partialFingerprints":{"codehealthFindingId/v1":"42073713227bd0dddfa5a823ecacb2abf4b6c55f34a263ed1ec0aeb904a633f5"}},{"ruleId":"D2","level":"warning","message":{"text":"ContextSummary::from (cognitive 46): ContextSummary::from has cognitive complexity 46 (threshold 15). Drivers by points: if/else 11 (32 pts), loops 3 (11 pts), match/switch 1 (2 pts), boolean chains 1 (nesting depth added 30). 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":"crates/forge_domain/src/compact/summary.rs"},"region":{"startLine":218}}}],"partialFingerprints":{"codehealthFindingId/v1":"34c59b67ba8ab1dd8faa3f01ec842a897698fbde4ab5325953f7fb0258f5d437"}},{"ruleId":"D2","level":"warning","message":{"text":"UI::select_model (cognitive 45): UI::select_model has cognitive complexity 45 (threshold 15). Drivers by points: if/else 20 (35 pts), loops 5 (7 pts), boolean chains 2, match/switch 1 (nesting depth added 17). 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":"crates/forge_main/src/ui.rs"},"region":{"startLine":2904}}}],"partialFingerprints":{"codehealthFindingId/v1":"46a82d7d8f593ec59466c5281c7948fdb84122447bc30633d8c6f3452b20de0d"}},{"ruleId":"D2","level":"warning","message":{"text":"Response::into_domain (cognitive 43): Response::into_domain has cognitive complexity 43 (threshold 15). Drivers by points: if/else 11 (27 pts), match/switch 3 (11 pts), loops 2 (5 pts) (nesting depth added 27). 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":"crates/forge_repo/src/provider/openai_responses/response.rs"},"region":{"startLine":156}}}],"partialFingerprints":{"codehealthFindingId/v1":"119644bbea02d9c06eeb441979288f912036a6b3cf375b04864a75a2f7b08037"}},{"ruleId":"D2","level":"warning","message":{"text":"DiffFormat::format (cognitive 37): DiffFormat::format has cognitive complexity 37 (threshold 15). Drivers by points: if/else 7 (17 pts), loops 7 (16 pts), match/switch 1 (4 pts) (nesting depth added 22). 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":"crates/forge_display/src/diff.rs"},"region":{"startLine":50}}}],"partialFingerprints":{"codehealthFindingId/v1":"6f5e94efc8b670303c4fe74eec7c7640a76a3cdb4eb4332f898d494729d52611"}},{"ruleId":"D2","level":"warning","message":{"text":"ChatCompletionMessage::try_from (cognitive 35): ChatCompletionMessage::try_from has cognitive complexity 35 (threshold 15). Drivers by points: if/else 15 (32 pts), loops 1 (2 pts), boolean chains 1 (nesting depth added 18). 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":"crates/forge_app/src/dto/google/response.rs"},"region":{"startLine":451}}}],"partialFingerprints":{"codehealthFindingId/v1":"511d964e5ff8cb95e2202ee37e57ae2bdbc549939a0e1e42e468520cf0adbdf0"}},{"ruleId":"D2","level":"warning","message":{"text":"ForgeProviderAuthService::refresh_provider_credential (cognitive 34): ForgeProviderAuthService::refresh_provider_credential has cognitive complexity 34 (threshold 15). Drivers by points: if/else 6 (21 pts), match/switch 2 (10 pts), loops 1 (3 pts) (nesting depth added 25). To reduce it, flatten the nesting: this score is depth rather than breadth \u2014 most of its points come from checks stacked inside one another, so the work sits several levels in. Invert each enclosing check into an early exit (a return, or the language\u0027s equivalent) so the happy path stays at one level, and where a level cannot be exited early, lift the block it encloses into its own named function."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_services/src/provider_auth.rs"},"region":{"startLine":149}}}],"partialFingerprints":{"codehealthFindingId/v1":"648654b629c0add6f49a9973ba48f1d74b0e4157fb8a455580038f8b0743eb1b"}},{"ruleId":"D2","level":"warning","message":{"text":"ToolCallFull::try_from_parts (cognitive 33): ToolCallFull::try_from_parts has cognitive complexity 33 (threshold 15). Drivers by points: if/else 13 (28 pts), boolean chains 4, loops 1 (nesting depth added 15). To reduce it, flatten the nesting: this score is depth rather than breadth \u2014 most of its points come from checks stacked inside one another, so the work sits several levels in. Invert each enclosing check into an early exit (a return, or the language\u0027s equivalent) so the happy path stays at one level, and where a level cannot be exited early, lift the block it encloses into its own named function."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_domain/src/tools/call/tool_call.rs"},"region":{"startLine":106}}}],"partialFingerprints":{"codehealthFindingId/v1":"81cb635258bbb30821320917820773997705085219f17041da942a0b80b76859"}},{"ruleId":"D2","level":"warning","message":{"text":"JsonRepairParser::parse_object (cognitive 33): JsonRepairParser::parse_object has cognitive complexity 33 (threshold 15). Drivers by points: if/else 16 (28 pts), boolean chains 4, 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":"crates/forge_json_repair/src/parser.rs"},"region":{"startLine":195}}}],"partialFingerprints":{"codehealthFindingId/v1":"1f8ecec5b930109b05dcdea3457184ed1a9ce7b683fc2ef7431114046d92e98d"}},{"ruleId":"D2","level":"warning","message":{"text":"UI::on_dump (cognitive 32): UI::on_dump has cognitive complexity 32 (threshold 15). Drivers by points: if/else 14 (32 pts) (nesting depth added 18). To reduce it, flatten the nesting: this score is depth rather than breadth \u2014 most of its points come from checks stacked inside one another, so the work sits several levels in. Invert each enclosing check into an early exit (a return, or the language\u0027s equivalent) so the happy path stays at one level, and where a level cannot be exited early, lift the block it encloses into its own named function."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_main/src/ui.rs"},"region":{"startLine":4067}}}],"partialFingerprints":{"codehealthFindingId/v1":"8db7142d83b96a8b6cb2370c0f9bac909ddb18dbe903346885ab8c92175fa30e"}},{"ruleId":"D2","level":"warning","message":{"text":"UI::on_show_models (cognitive 30): UI::on_show_models has cognitive complexity 30 (threshold 15). Drivers by points: if/else 14 (26 pts), loops 2 (3 pts), match/switch 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":"crates/forge_main/src/ui.rs"},"region":{"startLine":1426}}}],"partialFingerprints":{"codehealthFindingId/v1":"9bb43e3764b26d34d845947c6f5799b8a8844aa41a8645517a5b1d0c34570e4e"}},{"ruleId":"D2","level":"warning","message":{"text":"UI::run_inner (cognitive 29): UI::run_inner has cognitive complexity 29 (threshold 15). Drivers by points: match/switch 5 (14 pts), if/else 5 (13 pts), boolean chains 1, loops 1 (nesting depth added 17). To reduce it, keep the dispatch but shrink the arms: move each non-trivial case body into its own named function (or onto the value being matched) so the dispatch reads one line per case, and group related cases into a sub-dispatch. Keep every case explicit, and make the behaviour for cases you do not list a deliberate choice rather than an accident."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_main/src/ui.rs"},"region":{"startLine":358}}}],"partialFingerprints":{"codehealthFindingId/v1":"9ba60535662442988869c8020b02a2e793da20e310a023bce50f573f0decc0de"}},{"ruleId":"D2","level":"warning","message":{"text":"Orchestrator::run (cognitive 28): Orchestrator::run has cognitive complexity 28 (threshold 15). Drivers by points: if/else 9 (22 pts), boolean chains 3, loops 2 (3 pts) (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":"crates/forge_app/src/orch.rs"},"region":{"startLine":245}}}],"partialFingerprints":{"codehealthFindingId/v1":"d05dc21bf936102ba50c3050e9de72ed39d20c26d933242a6883caf29e2b1da1"}},{"ruleId":"D2","level":"warning","message":{"text":"forge_app::utils::enforce_strict_schema (cognitive 28): forge_app::utils::enforce_strict_schema has cognitive complexity 28 (threshold 15). Drivers by points: if/else 9 (21 pts), boolean chains 4, loops 1 (2 pts), match/switch 1 (nesting depth added 13). The drivers above price the dispatch low by construction \u2014 a dispatch is charged once however many cases it lists, while each branch inside an arm is charged in full \u2014 so most of this count is what the case bodies hold, and the arms are where it can be reduced. To reduce it, keep the dispatch but shrink the arms: move each non-trivial case body into its own named function (or onto the value being matched) so the dispatch reads one line per case, and group related cases into a sub-dispatch. Keep every case explicit, and make the behaviour for cases you do not list a deliberate choice rather than an accident."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_app/src/utils.rs"},"region":{"startLine":448}}}],"partialFingerprints":{"codehealthFindingId/v1":"67930967ee29272697013bda940b90c125578a156064af010b7bf15db1a1994c"}},{"ruleId":"D2","level":"warning","message":{"text":"forge_infra::auth::strategy::poll_for_tokens (cognitive 28): forge_infra::auth::strategy::poll_for_tokens has cognitive complexity 28 (threshold 15). Drivers by points: if/else 10 (25 pts), boolean chains 2, loops 1 (nesting depth added 15). 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":"crates/forge_infra/src/auth/strategy.rs"},"region":{"startLine":794}}}],"partialFingerprints":{"codehealthFindingId/v1":"3236d167b20e5d4c4963adf27ac859b753b5afcc44eba8434a6eb381b7459694"}},{"ruleId":"D2","level":"warning","message":{"text":"JsonRepairParser::parse_number (cognitive 28): JsonRepairParser::parse_number has cognitive complexity 28 (threshold 15). Drivers by points: if/else 14 (22 pts), loops 3 (5 pts), 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":"crates/forge_json_repair/src/parser.rs"},"region":{"startLine":595}}}],"partialFingerprints":{"codehealthFindingId/v1":"a2cb3fb07eeb94fa9088011f8a36f336f63d80a09cc6f260f887ce827deb3aae"}},{"ruleId":"D2","level":"warning","message":{"text":"UI::on_command (cognitive 27): UI::on_command has cognitive complexity 27 (threshold 15). Drivers by points: if/else 17 (26 pts), match/switch 1 (nesting depth added 9). The drivers above price the dispatch low by construction \u2014 a dispatch is charged once however many cases it lists, while each branch inside an arm is charged in full \u2014 so most of this count is what the case bodies hold, and the arms are where it can be reduced. To reduce it, keep the dispatch but shrink the arms: move each non-trivial case body into its own named function (or onto the value being matched) so the dispatch reads one line per case, and group related cases into a sub-dispatch. Keep every case explicit, and make the behaviour for cases you do not list a deliberate choice rather than an accident."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_main/src/ui.rs"},"region":{"startLine":2167}}}],"partialFingerprints":{"codehealthFindingId/v1":"1018276c8d0607ee33c42896aa64aef13ce5a69acb964765c8ed9b1540dd8cc8"}},{"ruleId":"D2","level":"warning","message":{"text":"UI::handle_api_key_input (cognitive 27): UI::handle_api_key_input has cognitive complexity 27 (threshold 15). Drivers by points: if/else 17 (25 pts), boolean chains 2 (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":"crates/forge_main/src/ui.rs"},"region":{"startLine":3085}}}],"partialFingerprints":{"codehealthFindingId/v1":"fb8d36e213132689093fc298367a4bcdf2ac4e06d170a98e7cf5a78bb4c53071"}},{"ruleId":"D2","level":"warning","message":{"text":"forge_select::preview::wrap_ansi_line (cognitive 27): forge_select::preview::wrap_ansi_line has cognitive complexity 27 (threshold 15). Drivers by points: if/else 13 (22 pts), loops 2 (4 pts), boolean chains 1 (nesting depth added 11). 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":"crates/forge_select/src/preview.rs"},"region":{"startLine":1193}}}],"partialFingerprints":{"codehealthFindingId/v1":"285c927255f089de50c2116d137248547b668e08b8458dbee57bbe2a5c05c51d"}},{"ruleId":"D2","level":"warning","message":{"text":"ZshRPrompt::fmt (cognitive 26): ZshRPrompt::fmt has cognitive complexity 26 (threshold 15). Drivers by points: if/else 17 (22 pts), boolean chains 2, match/switch 1 (2 pts) (nesting depth added 6). 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":"crates/forge_main/src/zsh/rprompt.rs"},"region":{"startLine":98}}}],"partialFingerprints":{"codehealthFindingId/v1":"b0df5af4e6ab409629030f85d71bd822b8a0e6c46cfd279a87d93400a895c285"}},{"ruleId":"D2","level":"warning","message":{"text":"forge_json_repair::schema_coercion::coerce_array_value (cognitive 25): forge_json_repair::schema_coercion::coerce_array_value has cognitive complexity 25 (threshold 15). Drivers by points: if/else 9 (21 pts), boolean chains 4 (nesting depth added 12). To reduce it, flatten the nesting: this score is depth rather than breadth \u2014 most of its points come from checks stacked inside one another, so the work sits several levels in. Invert each enclosing check into an early exit (a return, or the language\u0027s equivalent) so the happy path stays at one level, and where a level cannot be exited early, lift the block it encloses into its own named function."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_json_repair/src/schema_coercion.rs"},"region":{"startLine":323}}}],"partialFingerprints":{"codehealthFindingId/v1":"e515215d1a988a6e65b0190deae03779f74d856efc5148ad3298ba70935808b8"}},{"ruleId":"D2","level":"warning","message":{"text":"UI::handle_chat_response (cognitive 25): UI::handle_chat_response has cognitive complexity 25 (threshold 15). Drivers by points: if/else 10 (19 pts), match/switch 3 (5 pts), boolean chains 1 (nesting depth added 11). 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":"crates/forge_main/src/ui.rs"},"region":{"startLine":4140}}}],"partialFingerprints":{"codehealthFindingId/v1":"b4c25e90a3bd7a93afd92afafa5fced0a962174ce33fb4a4d4afd278df584d8f"}},{"ruleId":"D2","level":"warning","message":{"text":"forge_markdown_stream::table::render_table (cognitive 25): forge_markdown_stream::table::render_table has cognitive complexity 25 (threshold 15). Drivers by points: if/else 7 (14 pts), loops 5 (8 pts), boolean chains 3 (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":"crates/forge_markdown_stream/src/table.rs"},"region":{"startLine":10}}}],"partialFingerprints":{"codehealthFindingId/v1":"f944c4c20c21a990b933a792d0bb24807c4cf4ef4b1741bb5baa7c081ac369e4"}},{"ruleId":"D2","level":"warning","message":{"text":"forge_services::tool_services::fs_patch::apply_replacement (cognitive 25): forge_services::tool_services::fs_patch::apply_replacement has cognitive complexity 25 (threshold 15). Drivers by points: if/else 7 (15 pts), match/switch 2 (4 pts), boolean chains 3, loops 1 (3 pts) (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":"crates/forge_services/src/tool_services/fs_patch.rs"},"region":{"startLine":202}}}],"partialFingerprints":{"codehealthFindingId/v1":"6f6814c56ca041576fe5cb8b74ab2fe72f86918ba9822f164e1adeceffbf6a5a"}},{"ruleId":"D2","level":"warning","message":{"text":"Message::from_domain (cognitive 24): Message::from_domain has cognitive complexity 24 (threshold 15). Drivers by points: if/else 7 (14 pts), loops 2 (6 pts), match/switch 2 (3 pts), boolean chains 1 (nesting depth added 12). The drivers above price the dispatch low by construction \u2014 a dispatch is charged once however many cases it lists, while each branch inside an arm is charged in full \u2014 so most of this count is what the case bodies hold, and the arms are where it can be reduced. To reduce it, keep the dispatch but shrink the arms: move each non-trivial case body into its own named function (or onto the value being matched) so the dispatch reads one line per case, and group related cases into a sub-dispatch. Keep every case explicit, and make the behaviour for cases you do not list a deliberate choice rather than an accident."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_repo/src/provider/bedrock.rs"},"region":{"startLine":762}}}],"partialFingerprints":{"codehealthFindingId/v1":"9b1f0c3aeffaff9ccce0fc5294f86332e82a9f8e07e4641a3310b4bfea00ba3c"}},{"ruleId":"D2","level":"warning","message":{"text":"forge_domain::conversation_html::create_info_table (cognitive 24): forge_domain::conversation_html::create_info_table has cognitive complexity 24 (threshold 15). Drivers by points: if/else 10 (21 pts), match/switch 1 (3 pts) (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":"crates/forge_domain/src/conversation_html.rs"},"region":{"startLine":136}}}],"partialFingerprints":{"codehealthFindingId/v1":"eb917633969c4c3883c3d0cc8c91d2f4299fd9522e2adbb14d6568ab55eec6a4"}},{"ruleId":"D2","level":"warning","message":{"text":"ForgeCommandManager::parse (cognitive 24): ForgeCommandManager::parse has cognitive complexity 24 (threshold 15). Drivers by points: if/else 8 (16 pts), boolean chains 5, match/switch 2 (3 pts) (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":"crates/forge_main/src/model.rs"},"region":{"startLine":288}}}],"partialFingerprints":{"codehealthFindingId/v1":"ea80afd20bd6e73838e638616f8cadaa28c28181e8283b8223fa4b185a82775e"}},{"ruleId":"D2","level":"warning","message":{"text":"forge_json_repair::schema_coercion::try_coerce_string (cognitive 23): forge_json_repair::schema_coercion::try_coerce_string has cognitive complexity 23 (threshold 15). Drivers by points: if/else 9 (17 pts), boolean chains 3, match/switch 2 (3 pts) (nesting depth added 9). The drivers above price the dispatch low by construction \u2014 a dispatch is charged once however many cases it lists, while each branch inside an arm is charged in full \u2014 so most of this count is what the case bodies hold, and the arms are where it can be reduced. To reduce it, keep the dispatch but shrink the arms: move each non-trivial case body into its own named function (or onto the value being matched) so the dispatch reads one line per case, and group related cases into a sub-dispatch. Keep every case explicit, and make the behaviour for cases you do not list a deliberate choice rather than an accident."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_json_repair/src/schema_coercion.rs"},"region":{"startLine":243}}}],"partialFingerprints":{"codehealthFindingId/v1":"2172554fd6a5ced6bbd848a1ec7cf83038fcd713f4ad8b9453c36342120fcb3a"}},{"ruleId":"D2","level":"warning","message":{"text":"DoomLoopDetector::count_recent_pattern_repetitions (cognitive 22): DoomLoopDetector::count_recent_pattern_repetitions has cognitive complexity 22 (threshold 15). Drivers by points: if/else 13 (20 pts), boolean chains 1, loops 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":"crates/forge_app/src/hooks/doom_loop.rs"},"region":{"startLine":117}}}],"partialFingerprints":{"codehealthFindingId/v1":"92e263d91bb9aaaa0a8aebdbeaa2d3c7a2f1607db63ae78e55051442f975330c"}},{"ruleId":"D2","level":"warning","message":{"text":"forge_fs::binary_detection::is_binary_internal (cognitive 22): forge_fs::binary_detection::is_binary_internal has cognitive complexity 22 (threshold 15). Drivers by points: boolean chains 11, if/else 6 (9 pts), loops 1 (2 pts) (nesting depth added 4). 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":"crates/forge_fs/src/binary_detection.rs"},"region":{"startLine":43}}}],"partialFingerprints":{"codehealthFindingId/v1":"a4ef86b5cb3c0577834d175fcbbbff819c7cd6725a88535e23ea0e29d4bfa25c"}},{"ruleId":"D2","level":"warning","message":{"text":"forge_markdown_stream::utils::collapse_ansi_codes (cognitive 22): forge_markdown_stream::utils::collapse_ansi_codes has cognitive complexity 22 (threshold 15). Drivers by points: if/else 10 (16 pts), loops 2 (3 pts), match/switch 1 (3 pts) (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":"crates/forge_markdown_stream/src/utils.rs"},"region":{"startLine":340}}}],"partialFingerprints":{"codehealthFindingId/v1":"8a2ac2b37fc0df3e11337423c7a6ca6a0bdc0ea55bd0bbcd6bb462bde457f6f2"}},{"ruleId":"D2","level":"warning","message":{"text":"forge_domain::compact::summary::extract_tool_info (cognitive 21): forge_domain::compact::summary::extract_tool_info has cognitive complexity 21 (threshold 15). Drivers by points: if/else 4 (11 pts), match/switch 2 (7 pts), loops 1 (3 pts) (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":"crates/forge_domain/src/compact/summary.rs"},"region":{"startLine":339}}}],"partialFingerprints":{"codehealthFindingId/v1":"f57fef206625b3fe0e15e7536438483d25286a97b353c3acf91a135c578a99ee"}},{"ruleId":"D2","level":"warning","message":{"text":"ForgeMcpClient::create_connection (cognitive 21): ForgeMcpClient::create_connection has cognitive complexity 21 (threshold 15). Drivers by points: if/else 6 (11 pts), loops 2 (5 pts), match/switch 2 (4 pts), boolean chains 1 (nesting depth added 10). The drivers above price the dispatch low by construction \u2014 a dispatch is charged once however many cases it lists, while each branch inside an arm is charged in full \u2014 so most of this count is what the case bodies hold, and the arms are where it can be reduced. To reduce it, keep the dispatch but shrink the arms: move each non-trivial case body into its own named function (or onto the value being matched) so the dispatch reads one line per case, and group related cases into a sub-dispatch. Keep every case explicit, and make the behaviour for cases you do not list a deliberate choice rather than an accident."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_infra/src/mcp_client.rs"},"region":{"startLine":95}}}],"partialFingerprints":{"codehealthFindingId/v1":"901108c4057351bce23a613c2768362349a5f470439c41af79ab89fc860837cf"}},{"ruleId":"D2","level":"warning","message":{"text":"UI::on_index (cognitive 21): UI::on_index has cognitive complexity 21 (threshold 15). Drivers by points: if/else 9 (18 pts), match/switch 1 (2 pts), 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":"crates/forge_main/src/ui.rs"},"region":{"startLine":4752}}}],"partialFingerprints":{"codehealthFindingId/v1":"e66a7f3d979161048ca3b52675b792f731bdfc9dab8363a03178d1fbe8ca2a50"}},{"ruleId":"D2","level":"warning","message":{"text":"ForgeSkillRepository::load_skills_from_dir (cognitive 21): ForgeSkillRepository::load_skills_from_dir has cognitive complexity 21 (threshold 15). Drivers by points: if/else 11 (18 pts), match/switch 1 (2 pts), 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":"crates/forge_repo/src/skill.rs"},"region":{"startLine":119}}}],"partialFingerprints":{"codehealthFindingId/v1":"fdc13c2d025dfe231f463c94142c3d6f5937395d7119268518b50bda7a89bd25"}},{"ruleId":"D2","level":"warning","message":{"text":"ForgeFetch::check_robots_txt (cognitive 21): ForgeFetch::check_robots_txt has cognitive complexity 21 (threshold 15). Drivers by points: if/else 7 (18 pts), loops 1 (2 pts), boolean chains 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":"crates/forge_services/src/tool_services/fetch.rs"},"region":{"startLine":31}}}],"partialFingerprints":{"codehealthFindingId/v1":"841a9025527165352c3412647b7a50772feca1f24e58e57b68caa4145cefe2ce"}},{"ruleId":"D2","level":"warning","message":{"text":"Walker::get_blocking (cognitive 21): Walker::get_blocking has cognitive complexity 21 (threshold 15). Drivers by points: if/else 14 (16 pts), match/switch 3, 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":"crates/forge_walker/src/walker.rs"},"region":{"startLine":118}}}],"partialFingerprints":{"codehealthFindingId/v1":"3f655b5d2c6ea0a0ef302c49b2fb13c28681675b9a8d201b8571be9dd3d8d9af"}},{"ruleId":"D2","level":"warning","message":{"text":"forge_app::fmt::todo_fmt::format_todos_diff (cognitive 20): forge_app::fmt::todo_fmt::format_todos_diff has cognitive complexity 20 (threshold 15). Drivers by points: if/else 7 (13 pts), match/switch 1 (4 pts), loops 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":"crates/forge_app/src/fmt/todo_fmt.rs"},"region":{"startLine":56}}}],"partialFingerprints":{"codehealthFindingId/v1":"97fd8386236a168460803bd798e117862030d8b5fd898e2fd4abe28cd7d9fe1d"}},{"ruleId":"D2","level":"warning","message":{"text":"forge_main::zsh::paste::wrap_tokens (cognitive 20): forge_main::zsh::paste::wrap_tokens has cognitive complexity 20 (threshold 15). Drivers by points: if/else 8 (17 pts), 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":"crates/forge_main/src/zsh/paste.rs"},"region":{"startLine":141}}}],"partialFingerprints":{"codehealthFindingId/v1":"13d98459d4f92e05b7f259273ffb931d947388836def3bfb60a5456472a4ac2b"}},{"ruleId":"D2","level":"warning","message":{"text":"SanitizeToolIds::transform (cognitive 20): SanitizeToolIds::transform has cognitive complexity 20 (threshold 15). Drivers by points: if/else 3 (11 pts), loops 2 (5 pts), match/switch 1 (4 pts) (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":"crates/forge_repo/src/provider/bedrock_sanitize_ids.rs"},"region":{"startLine":29}}}],"partialFingerprints":{"codehealthFindingId/v1":"496679e14fbb3b8adc5f593cfd29ec50863c37d41549cf54f8f1ce7b34b02f16"}},{"ruleId":"D2","level":"warning","message":{"text":"UI::on_show_mcp_servers (cognitive 19): UI::on_show_mcp_servers has cognitive complexity 19 (threshold 15). Drivers by points: if/else 6 (9 pts), loops 3 (6 pts), boolean chains 2, match/switch 1 (2 pts) (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":"crates/forge_main/src/ui.rs"},"region":{"startLine":1730}}}],"partialFingerprints":{"codehealthFindingId/v1":"9232d33f5aeb40696d0809f9c2d533a1cfbdb667d6c32afd7fbf27ba72983822"}},{"ruleId":"D2","level":"warning","message":{"text":"forge_tool_macros::derive_description (cognitive 19): forge_tool_macros::derive_description has cognitive complexity 19 (threshold 15). Drivers by points: if/else 8 (15 pts), loops 2 (3 pts), 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":"crates/forge_tool_macros/src/lib.rs"},"region":{"startLine":16}}}],"partialFingerprints":{"codehealthFindingId/v1":"128decb6de4bdaf01974fe3fdbeabeaa49c6cf9f382b40e91f528ac0d87ef472"}},{"ruleId":"D2","level":"warning","message":{"text":"Request::from (cognitive 18): Request::from has cognitive complexity 18 (threshold 15). Drivers by points: if/else 9 (13 pts), match/switch 3 (4 pts), loops 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":"crates/forge_app/src/dto/google/request.rs"},"region":{"startLine":302}}}],"partialFingerprints":{"codehealthFindingId/v1":"1fbd8daac87e792eec4b596ae8743154e7d595df9a8c7f49166d5941ae7cde58"}},{"ruleId":"D2","level":"warning","message":{"text":"TrimToolCallIds::transform (cognitive 18): TrimToolCallIds::transform has cognitive complexity 18 (threshold 15). Drivers by points: if/else 4 (12 pts), loops 2 (6 pts) (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":"crates/forge_app/src/dto/openai/transformers/trim_tool_call_ids.rs"},"region":{"startLine":12}}}],"partialFingerprints":{"codehealthFindingId/v1":"316b3ebbe652b1a44603aaac0b36f414309fe6e62891ce12866b9b6c758a6e05"}},{"ruleId":"D2","level":"warning","message":{"text":"ForgeProviderRepository::migrate_env_to_file (cognitive 18): ForgeProviderRepository::migrate_env_to_file has cognitive complexity 18 (threshold 15). Drivers by points: if/else 9 (11 pts), boolean chains 6, loops 1 (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":"crates/forge_repo/src/provider/provider_repo.rs"},"region":{"startLine":337}}}],"partialFingerprints":{"codehealthFindingId/v1":"11a38e1093c1b93da8c159aa02a2f6935c928905771d22fa6e6aabc5c90b660a"}},{"ruleId":"D2","level":"warning","message":{"text":"forge_select::preview::handle_key_event (cognitive 18): forge_select::preview::handle_key_event has cognitive complexity 18 (threshold 15). Drivers by points: boolean chains 9, if/else 4 (8 pts), match/switch 1 (nesting depth added 4). The drivers above price the dispatch low by construction \u2014 a dispatch is charged once however many cases it lists, while each branch inside an arm is charged in full \u2014 so most of this count is what the case bodies hold, and the arms are where it can be reduced. To reduce it, keep the dispatch but shrink the arms: move each non-trivial case body into its own named function (or onto the value being matched) so the dispatch reads one line per case, and group related cases into a sub-dispatch. Keep every case explicit, and make the behaviour for cases you do not list a deliberate choice rather than an accident."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_select/src/preview.rs"},"region":{"startLine":622}}}],"partialFingerprints":{"codehealthFindingId/v1":"db5f1b5e9bc83375d19d42f1a35d7cc352758b503c7011cb932246a9304c2552"}},{"ruleId":"D2","level":"warning","message":{"text":"AgentExecutor::execute (cognitive 17): AgentExecutor::execute has cognitive complexity 17 (threshold 15). Drivers by points: if/else 7 (11 pts), match/switch 2 (5 pts), loops 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":"crates/forge_app/src/agent_executor.rs"},"region":{"startLine":42}}}],"partialFingerprints":{"codehealthFindingId/v1":"6bcfdfe0b46ad425ff061f86959a8b9f1b102da46202f095cf8d031c189e2bf4"}},{"ruleId":"D2","level":"warning","message":{"text":"ContextMessage::to_text (cognitive 17): ContextMessage::to_text has cognitive complexity 17 (threshold 15). Drivers by points: if/else 4 (10 pts), loops 2 (6 pts), match/switch 1 (nesting depth added 10). The drivers above price the dispatch low by construction \u2014 a dispatch is charged once however many cases it lists, while each branch inside an arm is charged in full \u2014 so most of this count is what the case bodies hold, and the arms are where it can be reduced. To reduce it, keep the dispatch but shrink the arms: move each non-trivial case body into its own named function (or onto the value being matched) so the dispatch reads one line per case, and group related cases into a sub-dispatch. Keep every case explicit, and make the behaviour for cases you do not list a deliberate choice rather than an accident."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_domain/src/context.rs"},"region":{"startLine":113}}}],"partialFingerprints":{"codehealthFindingId/v1":"e0ecceec46a9c15386ad8ad8cb50b95fe96642d2f20fa8f94defb0cd6a9e986e"}},{"ruleId":"D2","level":"warning","message":{"text":"ForgeDirectoryReaderService::read_directory_files (cognitive 17): ForgeDirectoryReaderService::read_directory_files has cognitive complexity 17 (threshold 15). Drivers by points: if/else 7 (13 pts), boolean chains 2, loops 1, match/switch 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":"crates/forge_infra/src/fs_read_dir.rs"},"region":{"startLine":68}}}],"partialFingerprints":{"codehealthFindingId/v1":"cf43144632e728af42e754b92efd5cba2485eab6a0fb1d395ac4d5b2756aa5a0"}},{"ruleId":"D2","level":"warning","message":{"text":"ForgePrompt::render_prompt_right (cognitive 17): ForgePrompt::render_prompt_right has cognitive complexity 17 (threshold 15). Drivers by points: if/else 10 (12 pts), boolean chains 3, match/switch 1 (2 pts) (nesting depth added 3). 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":"crates/forge_main/src/prompt.rs"},"region":{"startLine":116}}}],"partialFingerprints":{"codehealthFindingId/v1":"c69021dcda3fc2ff91ded47607c5aab416c902dc061c18ea2791fff79cd2622c"}},{"ruleId":"D2","level":"warning","message":{"text":"forge_main::highlighter::highlight_mentions (cognitive 17): forge_main::highlighter::highlight_mentions has cognitive complexity 17 (threshold 15). Drivers by points: match/switch 4 (12 pts), if/else 1 (3 pts), boolean chains 1, loops 1 (nesting depth added 10). To reduce it, keep the dispatch but shrink the arms: move each non-trivial case body into its own named function (or onto the value being matched) so the dispatch reads one line per case, and group related cases into a sub-dispatch. Keep every case explicit, and make the behaviour for cases you do not list a deliberate choice rather than an accident."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_main/src/highlighter.rs"},"region":{"startLine":74}}}],"partialFingerprints":{"codehealthFindingId/v1":"d1750a8d20bc907a94a955231d2cd0db34e68313f4a6a05dc1d6317e0aabf501"}},{"ruleId":"D2","level":"warning","message":{"text":"ForgeFsSearch::search_content (cognitive 17): ForgeFsSearch::search_content has cognitive complexity 17 (threshold 15). Drivers by points: if/else 9 (15 pts), 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":"crates/forge_services/src/tool_services/fs_search.rs"},"region":{"startLine":421}}}],"partialFingerprints":{"codehealthFindingId/v1":"c88b5407cb4ebe906d4b75f57888e8dff7c0b099a3eaa64e5d5875a655c749b0"}},{"ruleId":"D2","level":"warning","message":{"text":"forge_app::dto::anthropic::response::get_context_length (cognitive 16): forge_app::dto::anthropic::response::get_context_length has cognitive complexity 16 (threshold 15). Drivers by points: if/else 10, boolean chains 6. 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":"crates/forge_app/src/dto/anthropic/response.rs"},"region":{"startLine":81}}}],"partialFingerprints":{"codehealthFindingId/v1":"cb584afbaa888cd3680efd84cf81f3acc7dacd085c39156b630f64210f85bed5"}},{"ruleId":"D2","level":"warning","message":{"text":"forge_domain::compact::strategy::find_sequence_preserving_last_n (cognitive 16): forge_domain::compact::strategy::find_sequence_preserving_last_n has cognitive complexity 16 (threshold 15). Drivers by points: if/else 10 (11 pts), boolean chains 3, loops 1 (2 pts) (nesting depth added 2). 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":"crates/forge_domain/src/compact/strategy.rs"},"region":{"startLine":81}}}],"partialFingerprints":{"codehealthFindingId/v1":"1184b71d5b02c3fd3ba149a0b511c9333d271c1533621b7b9641d0b2bb7c7ad9"}},{"ruleId":"D2","level":"warning","message":{"text":"ForgeHttpInfra::new (cognitive 16): ForgeHttpInfra::new has cognitive complexity 16 (threshold 15). Drivers by points: if/else 7 (10 pts), match/switch 2 (4 pts), loops 1 (2 pts) (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":"crates/forge_infra/src/http.rs"},"region":{"startLine":39}}}],"partialFingerprints":{"codehealthFindingId/v1":"bb76b1685b36176b15ae1db15989a7f4ea8e97821593f69e2a677d5e538819d8"}},{"ruleId":"D2","level":"warning","message":{"text":"Info::from (cognitive 16): Info::from has cognitive complexity 16 (threshold 15). Drivers by points: if/else 8 (12 pts), boolean chains 2, loops 1 (2 pts) (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":"crates/forge_main/src/info.rs"},"region":{"startLine":418}}}],"partialFingerprints":{"codehealthFindingId/v1":"804f428c286ed8c8df9b8b2ceb5ed1b40848585097aae393ddf18500c05d9770"}},{"ruleId":"D2","level":"warning","message":{"text":"Renderer::render_event (cognitive 16): Renderer::render_event has cognitive complexity 16 (threshold 15). Drivers by points: loops 4 (9 pts), if/else 4 (6 pts), match/switch 1 (nesting depth added 7). 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":"crates/forge_markdown_stream/src/renderer.rs"},"region":{"startLine":126}}}],"partialFingerprints":{"codehealthFindingId/v1":"edda38056cb3353859fcfabd0ff64f20f37e99b798cd7527f885b30c3309ec03"}},{"ruleId":"D2","level":"warning","message":{"text":"forge_markdown_stream::utils::parse_atoms (cognitive 16): forge_markdown_stream::utils::parse_atoms has cognitive complexity 16 (threshold 15). Drivers by points: if/else 4 (9 pts), loops 2 (5 pts), match/switch 1 (2 pts) (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":"crates/forge_markdown_stream/src/utils.rs"},"region":{"startLine":261}}}],"partialFingerprints":{"codehealthFindingId/v1":"aef2dc1d2bdd1ee534f7db06b72e70b7f1ddbae83f54962148c981fdba89ccd1"}},{"ruleId":"D3","level":"warning","message":{"text":"FileTooLong: src/ui.rs: FileTooLong \u2014 3313 significant lines (blank, comment-only and punctuation-only lines excluded, and inline test code \u2014 #[cfg(test)] modules and bare #[test] functions \u2014 not counted), about 97% of them inside a single declaration: A (119-5211). The bar is 500 significant lines; this is 2813 over it, 6.63\u00D7 the bar. 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":"crates/forge_main/src/ui.rs"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"5e4cce9fb19f9a9b1275f075f2268b66cbcfb9ac32ca4f5b52e5cb32891e1682"}},{"ruleId":"D3","level":"warning","message":{"text":"TooManyMethods: UI: TooManyMethods \u2014 116 methods. The bar is 30 methods; this is 86 over it, 3.87\u00D7 the bar. 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":"crates/forge_main/src/ui.rs"},"region":{"startLine":105}}}],"partialFingerprints":{"codehealthFindingId/v1":"458da1c6ff395ffeb24757b15749b5a14c6b25a27af76318eda8545e02adb836"}},{"ruleId":"D3","level":"warning","message":{"text":"MethodTooLong: ToolOperation.into_tool_output: MethodTooLong \u2014 into_tool_output runs 343 significant lines (blank, comment-only and punctuation-only lines excluded, and inline test code \u2014 #[cfg(test)] modules and bare #[test] functions \u2014 not counted) in one body. The bar is 100 significant lines; this is 243 over it, 3.43\u00D7 the bar. This is length, not branching: a long straight-line body scores low on complexity and is still read whole to change any part of it, so the complexity numbers beside this row neither confirm nor excuse it. To reduce it, extract each cohesive step of the body \u2014 the runs of statements that work on the same values and would earn the same name \u2014 into its own named unit, and have this one call them in order."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_app/src/operation.rs"},"region":{"startLine":235}}}],"partialFingerprints":{"codehealthFindingId/v1":"87f655758397b583f3321c3ff70f2dabb1865d213ff2e000f3544197f3541f09"}},{"ruleId":"D3","level":"warning","message":{"text":"MethodTooLong: UI.handle_subcommands: MethodTooLong \u2014 handle_subcommands runs 295 significant lines (blank, comment-only and punctuation-only lines excluded, and inline test code \u2014 #[cfg(test)] modules and bare #[test] functions \u2014 not counted) in one body. The bar is 100 significant lines; this is 195 over it, 2.95\u00D7 the bar. This is length, not branching: a long straight-line body scores low on complexity and is still read whole to change any part of it, so the complexity numbers beside this row neither confirm nor excuse it. To reduce it, extract each cohesive step of the body \u2014 the runs of statements that work on the same values and would earn the same name \u2014 into its own named unit, and have this one call them in order."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_main/src/ui.rs"},"region":{"startLine":466}}}],"partialFingerprints":{"codehealthFindingId/v1":"27a0b1be4127665aec198671c9e3a12189e1d52e375870575bcf59765e13e588"}},{"ruleId":"D3","level":"warning","message":{"text":"FunctionTooLong: forge_select::preview::draw_preview_ui: FunctionTooLong \u2014 forge_select::preview::draw_preview_ui runs 247 significant lines (blank, comment-only and punctuation-only lines excluded, and inline test code \u2014 #[cfg(test)] modules and bare #[test] functions \u2014 not counted) in one body. The bar is 100 significant lines; this is 147 over it, 2.47\u00D7 the bar. This is length, not branching: a long straight-line body scores low on complexity and is still read whole to change any part of it, so the complexity numbers beside this row neither confirm nor excuse it. To reduce it, extract each cohesive step of the body \u2014 the runs of statements that work on the same values and would earn the same name \u2014 into its own named unit, and have this one call them in order."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_select/src/preview.rs"},"region":{"startLine":875}}}],"partialFingerprints":{"codehealthFindingId/v1":"edb0a311d4c8e2f3f51a6bbdb96209622c9ce8fc7c52a72b13b628924943c9fb"}},{"ruleId":"D3","level":"warning","message":{"text":"FunctionTooLong: forge_select::preview::run_select_ui_values: FunctionTooLong \u2014 forge_select::preview::run_select_ui_values runs 235 significant lines (blank, comment-only and punctuation-only lines excluded, and inline test code \u2014 #[cfg(test)] modules and bare #[test] functions \u2014 not counted) in one body. The bar is 100 significant lines; this is 135 over it, 2.35\u00D7 the bar. This is length, not branching: a long straight-line body scores low on complexity and is still read whole to change any part of it, so the complexity numbers beside this row neither confirm nor excuse it. To reduce it, extract each cohesive step of the body \u2014 the runs of statements that work on the same values and would earn the same name \u2014 into its own named unit, and have this one call them in order."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_select/src/preview.rs"},"region":{"startLine":289}}}],"partialFingerprints":{"codehealthFindingId/v1":"324a3cde7aea113b1698cc662fbd63ef6654466de1dddad4bf0ed66f68fd1ef6"}},{"ruleId":"D3","level":"warning","message":{"text":"FileTooLong: src/preview.rs: FileTooLong \u2014 942 significant lines (blank, comment-only and punctuation-only lines excluded, and inline test code \u2014 #[cfg(test)] modules and bare #[test] functions \u2014 not counted). The bar is 500 significant lines; this is 442 over it, 1.88\u00D7 the bar. 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":"crates/forge_select/src/preview.rs"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"a9df101abee0e230c16ea16c5b74a4c8350471fb61ac580f65d34160f2636c28"}},{"ruleId":"D3","level":"warning","message":{"text":"MethodTooLong: UI.on_command: MethodTooLong \u2014 on_command runs 177 significant lines (blank, comment-only and punctuation-only lines excluded, and inline test code \u2014 #[cfg(test)] modules and bare #[test] functions \u2014 not counted) in one body. The bar is 100 significant lines; this is 77 over it, 1.77\u00D7 the bar. This is length, not branching: a long straight-line body scores low on complexity and is still read whole to change any part of it, so the complexity numbers beside this row neither confirm nor excuse it. To reduce it, extract each cohesive step of the body \u2014 the runs of statements that work on the same values and would earn the same name \u2014 into its own named unit, and have this one call them in order."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_main/src/ui.rs"},"region":{"startLine":2167}}}],"partialFingerprints":{"codehealthFindingId/v1":"8666456ea0961bdb39db265bd62d33ab8ca8613f0f51fed88735d95287e6a187"}},{"ruleId":"D3","level":"warning","message":{"text":"ClassTooLong: JsonRepairParser: ClassTooLong \u2014 676 significant lines (blank, comment-only and punctuation-only lines excluded, and inline test code \u2014 #[cfg(test)] modules and bare #[test] functions \u2014 not counted), 48 methods, 2 blocks, lines 5-1073. The bar is 400 significant lines; this is 276 over it, 1.69\u00D7 the bar. 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":"crates/forge_json_repair/src/parser.rs"},"region":{"startLine":5}}}],"partialFingerprints":{"codehealthFindingId/v1":"c575a4d8f07a5fc08b7b71427ca2b9eb42e6b4a655c68e6af041ed7aac37e6f8"}},{"ruleId":"D3","level":"warning","message":{"text":"MethodTooLong: ChatCompletionMessage.try_from: MethodTooLong \u2014 try_from runs 168 significant lines (blank, comment-only and punctuation-only lines excluded, and inline test code \u2014 #[cfg(test)] modules and bare #[test] functions \u2014 not counted) in one body. The bar is 100 significant lines; this is 68 over it, 1.68\u00D7 the bar. This is length, not branching: a long straight-line body scores low on complexity and is still read whole to change any part of it, so the complexity numbers beside this row neither confirm nor excuse it. To reduce it, extract each cohesive step of the body \u2014 the runs of statements that work on the same values and would earn the same name \u2014 into its own named unit, and have this one call them in order."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_app/src/dto/openai/response.rs"},"region":{"startLine":324}}}],"partialFingerprints":{"codehealthFindingId/v1":"cfd774bdfa3bba90879f4107c3ec30243abd989dea4266243863712797aedb57"}},{"ruleId":"D3","level":"warning","message":{"text":"TooManyFields: ForgeConfig: TooManyFields \u2014 50 stored fields beside 2 methods. The bar is 30 stored fields; this is 20 over it, 1.67\u00D7 the bar. This is width in DATA, not behaviour: every reader that takes the whole type couples to all of its fields, so a change to any one of them is a change every reader has to be checked against. To reduce it, group the fields that are read together by the same callers into a smaller type of their own, and have this one hold that type as a single member \u2014 each reader then names only the group it uses."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_config/src/config.rs"},"region":{"startLine":121}}}],"partialFingerprints":{"codehealthFindingId/v1":"ebfd4c4fdd5265fd1b5cd80d75f7bd9fa08b7f0edd654ecfbae91555b99581dd"}},{"ruleId":"D3","level":"warning","message":{"text":"TooManyMethods: JsonRepairParser: TooManyMethods \u2014 48 methods. The bar is 30 methods; this is 18 over it, 1.60\u00D7 the bar. 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":"crates/forge_json_repair/src/parser.rs"},"region":{"startLine":5}}}],"partialFingerprints":{"codehealthFindingId/v1":"a14242b8b79c4708d56094d82cb0b8b456a468b2524ddb2c52a82b527bb39231"}},{"ruleId":"D3","level":"warning","message":{"text":"FileTooLong: auth/strategy.rs: FileTooLong \u2014 770 significant lines (blank, comment-only and punctuation-only lines excluded, and inline test code \u2014 #[cfg(test)] modules and bare #[test] functions \u2014 not counted). The bar is 500 significant lines; this is 270 over it, 1.54\u00D7 the bar. 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":"crates/forge_infra/src/auth/strategy.rs"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"3f26872c1e1d93c2bc52b65b821a9d89d1c1b4f35adf616f2e028c969e82129d"}},{"ruleId":"D3","level":"warning","message":{"text":"MethodTooLong: ToolExecutor.call_internal: MethodTooLong \u2014 call_internal runs 154 significant lines (blank, comment-only and punctuation-only lines excluded, and inline test code \u2014 #[cfg(test)] modules and bare #[test] functions \u2014 not counted) in one body. The bar is 100 significant lines; this is 54 over it, 1.54\u00D7 the bar. This is length, not branching: a long straight-line body scores low on complexity and is still read whole to change any part of it, so the complexity numbers beside this row neither confirm nor excuse it. To reduce it, extract each cohesive step of the body \u2014 the runs of statements that work on the same values and would earn the same name \u2014 into its own named unit, and have this one call them in order."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_app/src/tool_executor.rs"},"region":{"startLine":151}}}],"partialFingerprints":{"codehealthFindingId/v1":"2d31312e1595a6bee49e91a099e042bdd7aa343d9565cdce84d41fdba0971063"}},{"ruleId":"D3","level":"warning","message":{"text":"MethodTooLong: JsonRepairParser.parse_string: MethodTooLong \u2014 parse_string runs 143 significant lines (blank, comment-only and punctuation-only lines excluded, and inline test code \u2014 #[cfg(test)] modules and bare #[test] functions \u2014 not counted) in one body. The bar is 100 significant lines; this is 43 over it, 1.43\u00D7 the bar. This is length, not branching: a long straight-line body scores low on complexity and is still read whole to change any part of it, so the complexity numbers beside this row neither confirm nor excuse it. To reduce it, extract each cohesive step of the body \u2014 the runs of statements that work on the same values and would earn the same name \u2014 into its own named unit, and have this one call them in order."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_json_repair/src/parser.rs"},"region":{"startLine":348}}}],"partialFingerprints":{"codehealthFindingId/v1":"baae8bf7572e2bc586753e62b5f6af9598ce395fadb471a280de90917a93f29e"}},{"ruleId":"D3","level":"warning","message":{"text":"FunctionTooLong: forge_app::utils::sanitize_gemini_schema: FunctionTooLong \u2014 forge_app::utils::sanitize_gemini_schema runs 142 significant lines (blank, comment-only and punctuation-only lines excluded, and inline test code \u2014 #[cfg(test)] modules and bare #[test] functions \u2014 not counted) in one body. The bar is 100 significant lines; this is 42 over it, 1.42\u00D7 the bar. This is length, not branching: a long straight-line body scores low on complexity and is still read whole to change any part of it, so the complexity numbers beside this row neither confirm nor excuse it. To reduce it, extract each cohesive step of the body \u2014 the runs of statements that work on the same values and would earn the same name \u2014 into its own named unit, and have this one call them in order."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_app/src/utils.rs"},"region":{"startLine":626}}}],"partialFingerprints":{"codehealthFindingId/v1":"cf33292a6d4bf4ac7514ae4f6c4120e40e9d199e3de7ee5988b2284f6197309b"}},{"ruleId":"D3","level":"warning","message":{"text":"FileTooLong: src/services.rs: FileTooLong \u2014 681 significant lines (blank, comment-only and punctuation-only lines excluded, and inline test code \u2014 #[cfg(test)] modules and bare #[test] functions \u2014 not counted). The bar is 500 significant lines; this is 181 over it, 1.36\u00D7 the bar. 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":"crates/forge_app/src/services.rs"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"efe88382372d435a76d748ba2524e8e13016c7dfc315cfeb8e52a65354e90d6c"}},{"ruleId":"D3","level":"warning","message":{"text":"FileTooLong: src/parser.rs: FileTooLong \u2014 681 significant lines (blank, comment-only and punctuation-only lines excluded, and inline test code \u2014 #[cfg(test)] modules and bare #[test] functions \u2014 not counted), about 99% of them inside a single declaration: JsonRepairParser (2 blocks, 5-1073). The bar is 500 significant lines; this is 181 over it, 1.36\u00D7 the bar. 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":"crates/forge_json_repair/src/parser.rs"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"849d754e8f1b727555c5f182ac608b0d53719b0d9370b514cf8b317d17e7a754"}},{"ruleId":"D3","level":"warning","message":{"text":"FileTooLong: conversation/conversation_record.rs: FileTooLong \u2014 675 significant lines (blank, comment-only and punctuation-only lines excluded, and inline test code \u2014 #[cfg(test)] modules and bare #[test] functions \u2014 not counted). The bar is 500 significant lines; this is 175 over it, 1.35\u00D7 the bar. 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":"crates/forge_repo/src/conversation/conversation_record.rs"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"2de9ba8345a654121ca01d901763d4c1da8657390e38bc49e47838dbae139ac0"}},{"ruleId":"D3","level":"warning","message":{"text":"MethodTooLong: Orchestrator.run: MethodTooLong \u2014 run runs 133 significant lines (blank, comment-only and punctuation-only lines excluded, and inline test code \u2014 #[cfg(test)] modules and bare #[test] functions \u2014 not counted) in one body. The bar is 100 significant lines; this is 33 over it, 1.33\u00D7 the bar. This is length, not branching: a long straight-line body scores low on complexity and is still read whole to change any part of it, so the complexity numbers beside this row neither confirm nor excuse it. To reduce it, extract each cohesive step of the body \u2014 the runs of statements that work on the same values and would earn the same name \u2014 into its own named unit, and have this one call them in order."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_app/src/orch.rs"},"region":{"startLine":245}}}],"partialFingerprints":{"codehealthFindingId/v1":"0df28fd81bd12c03ce87389e24e4222de926dcf16d0e4c47d24ee67d37991595"}},{"ruleId":"D3","level":"warning","message":{"text":"FileTooLong: provider/bedrock.rs: FileTooLong \u2014 661 significant lines (blank, comment-only and punctuation-only lines excluded, and inline test code \u2014 #[cfg(test)] modules and bare #[test] functions \u2014 not counted). The bar is 500 significant lines; this is 161 over it, 1.32\u00D7 the bar. 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":"crates/forge_repo/src/provider/bedrock.rs"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"82f034ead76ce75fbae254c5e91046defeae5f1d31911da698723a55fe7c32ff"}},{"ruleId":"D3","level":"warning","message":{"text":"FunctionTooLong: forge_domain::conversation_html::create_conversation_context_section: FunctionTooLong \u2014 forge_domain::conversation_html::create_conversation_context_section runs 127 significant lines (blank, comment-only and punctuation-only lines excluded, and inline test code \u2014 #[cfg(test)] modules and bare #[test] functions \u2014 not counted) in one body. The bar is 100 significant lines; this is 27 over it, 1.27\u00D7 the bar. This is length, not branching: a long straight-line body scores low on complexity and is still read whole to change any part of it, so the complexity numbers beside this row neither confirm nor excuse it. To reduce it, extract each cohesive step of the body \u2014 the runs of statements that work on the same values and would earn the same name \u2014 into its own named unit, and have this one call them in order."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_domain/src/conversation_html.rs"},"region":{"startLine":288}}}],"partialFingerprints":{"codehealthFindingId/v1":"cdeb419e47d621b7946e77f79c5a3a25582dbdca5232e341920b3068b2534a72"}},{"ruleId":"D3","level":"warning","message":{"text":"FileTooLong: tools/catalog.rs: FileTooLong \u2014 586 significant lines (blank, comment-only and punctuation-only lines excluded, and inline test code \u2014 #[cfg(test)] modules and bare #[test] functions \u2014 not counted). The bar is 500 significant lines; this is 86 over it, 1.17\u00D7 the bar. 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":"crates/forge_domain/src/tools/catalog.rs"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"a009a01185cf619e73045f0cf7fde1ee9ea22255f512f48f36a8343eba0e284a"}},{"ruleId":"D3","level":"warning","message":{"text":"MethodTooLong: UI.select_model: MethodTooLong \u2014 select_model runs 114 significant lines (blank, comment-only and punctuation-only lines excluded, and inline test code \u2014 #[cfg(test)] modules and bare #[test] functions \u2014 not counted) in one body. The bar is 100 significant lines; this is 14 over it, 1.14\u00D7 the bar. This is length, not branching: a long straight-line body scores low on complexity and is still read whole to change any part of it, so the complexity numbers beside this row neither confirm nor excuse it. To reduce it, extract each cohesive step of the body \u2014 the runs of statements that work on the same values and would earn the same name \u2014 into its own named unit, and have this one call them in order."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_main/src/ui.rs"},"region":{"startLine":2904}}}],"partialFingerprints":{"codehealthFindingId/v1":"d2555568f0fce3f75174006e273e978490dd1e0392d6ec5350c6bb7a1925576e"}},{"ruleId":"D3","level":"warning","message":{"text":"MethodTooLong: Renderer.render_event: MethodTooLong \u2014 render_event runs 114 significant lines (blank, comment-only and punctuation-only lines excluded, and inline test code \u2014 #[cfg(test)] modules and bare #[test] functions \u2014 not counted) in one body. The bar is 100 significant lines; this is 14 over it, 1.14\u00D7 the bar. This is length, not branching: a long straight-line body scores low on complexity and is still read whole to change any part of it, so the complexity numbers beside this row neither confirm nor excuse it. To reduce it, extract each cohesive step of the body \u2014 the runs of statements that work on the same values and would earn the same name \u2014 into its own named unit, and have this one call them in order."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_markdown_stream/src/renderer.rs"},"region":{"startLine":126}}}],"partialFingerprints":{"codehealthFindingId/v1":"aaf40080483d23420a283d6cd380ac47a222d5763642ff8705a9f0a23f4278ec"}},{"ruleId":"D3","level":"warning","message":{"text":"FunctionTooLong: forge_services::tool_services::fs_patch::apply_replacement: FunctionTooLong \u2014 forge_services::tool_services::fs_patch::apply_replacement runs 114 significant lines (blank, comment-only and punctuation-only lines excluded, and inline test code \u2014 #[cfg(test)] modules and bare #[test] functions \u2014 not counted) in one body. The bar is 100 significant lines; this is 14 over it, 1.14\u00D7 the bar. This is length, not branching: a long straight-line body scores low on complexity and is still read whole to change any part of it, so the complexity numbers beside this row neither confirm nor excuse it. To reduce it, extract each cohesive step of the body \u2014 the runs of statements that work on the same values and would earn the same name \u2014 into its own named unit, and have this one call them in order."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_services/src/tool_services/fs_patch.rs"},"region":{"startLine":202}}}],"partialFingerprints":{"codehealthFindingId/v1":"02bd8e1509294a2a779192e360f17d4d86d032140cd4d5d919409148ae82766c"}},{"ruleId":"D3","level":"warning","message":{"text":"FunctionTooLong: forge_markdown_stream::table::wrap: FunctionTooLong \u2014 forge_markdown_stream::table::wrap runs 110 significant lines (blank, comment-only and punctuation-only lines excluded, and inline test code \u2014 #[cfg(test)] modules and bare #[test] functions \u2014 not counted) in one body. The bar is 100 significant lines; this is 10 over it, 1.10\u00D7 the bar. This is length, not branching: a long straight-line body scores low on complexity and is still read whole to change any part of it, so the complexity numbers beside this row neither confirm nor excuse it. To reduce it, extract each cohesive step of the body \u2014 the runs of statements that work on the same values and would earn the same name \u2014 into its own named unit, and have this one call them in order."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_markdown_stream/src/table.rs"},"region":{"startLine":135}}}],"partialFingerprints":{"codehealthFindingId/v1":"6a3f4e707ea7406fe041d52de280e2641104193ff28265362b5978e9155a42df"}},{"ruleId":"D3","level":"warning","message":{"text":"ClassTooLong: ForgeRepo: ClassTooLong \u2014 416 significant lines (blank, comment-only and punctuation-only lines excluded, and inline test code \u2014 #[cfg(test)] modules and bare #[test] functions \u2014 not counted), 1 methods, 28 blocks, lines 41-680. The bar is 400 significant lines; this is 16 over it, 1.04\u00D7 the bar. 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":"crates/forge_repo/src/forge_repo.rs"},"region":{"startLine":41}}}],"partialFingerprints":{"codehealthFindingId/v1":"7044e8366c7095fe2b76ffa818837a037f07b2f5cc69c9431ddfc95835fa7c87"}},{"ruleId":"D3","level":"warning","message":{"text":"MethodTooLong: ForgeApp.chat: MethodTooLong \u2014 chat runs 103 significant lines (blank, comment-only and punctuation-only lines excluded, and inline test code \u2014 #[cfg(test)] modules and bare #[test] functions \u2014 not counted) in one body. The bar is 100 significant lines; this is 3 over it, 1.03\u00D7 the bar. This is length, not branching: a long straight-line body scores low on complexity and is still read whole to change any part of it, so the complexity numbers beside this row neither confirm nor excuse it. To reduce it, extract each cohesive step of the body \u2014 the runs of statements that work on the same values and would earn the same name \u2014 into its own named unit, and have this one call them in order."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_app/src/app.rs"},"region":{"startLine":60}}}],"partialFingerprints":{"codehealthFindingId/v1":"3b0494350e5a3a5ebf1817ce1619af812832a9cae0bb716326037c08e2283101"}},{"ruleId":"D3","level":"warning","message":{"text":"FileTooLong: src/context.rs: FileTooLong \u2014 505 significant lines (blank, comment-only and punctuation-only lines excluded, and inline test code \u2014 #[cfg(test)] modules and bare #[test] functions \u2014 not counted). The bar is 500 significant lines; this is 5 over it, 1.01\u00D7 the bar. 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":"crates/forge_domain/src/context.rs"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"2b199054e68eb61ab0514d50504c316dbe8f66b349920b62dc7476053f9f5ad7"}},{"ruleId":"D3","level":"warning","message":{"text":"FileTooLong: src/utils.rs: FileTooLong \u2014 502 significant lines (blank, comment-only and punctuation-only lines excluded, and inline test code \u2014 #[cfg(test)] modules and bare #[test] functions \u2014 not counted). The bar is 500 significant lines; this is 2 over it, 1.00\u00D7 the bar. 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":"crates/forge_app/src/utils.rs"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"f79370c73a274f933dc9fcd262465d14702cd583f00c0d856c3ffefdb5f85f66"}},{"ruleId":"D4","level":"warning","message":{"text":"Near-duplicate member pair (33 shared lines): crates/forge_repo/src/provider/openai.rs:72-128 | crates/forge_repo/src/provider/openai_responses/repository.rs:76-151 \u2014 These two members are variants of one another: 33 of their lines are already reported as duplicated blocks below, spread through both bodies rather than gathered into one. Read them as a single construct written twice. The repair is at the members\u0027 grain \u2014 factor the shared pipeline into one implementation the two call with their differences as parameters or as an injected step, or, where the difference is systematic (sync against async, one transport against another), generate one from the other. Extracting the individual blocks below is not the same fix: it leaves the two bodies in place and the next edit still has to be made twice."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_repo/src/provider/openai.rs"},"region":{"startLine":72}}}],"partialFingerprints":{"codehealthFindingId/v1":"75e8752982cbc67fa1ec597923e6d494d335d51673fda1b43ff90178b03cf4be"}},{"ruleId":"D4","level":"warning","message":{"text":"Members sharing a duplicated core (4 members, 50\u002B identical tokens): crates/forge_repo/src/provider/anthropic.rs:368-380 | crates/forge_repo/src/provider/bedrock.rs:1074-1087 | crates/forge_repo/src/provider/google.rs:200-212 | crates/forge_repo/src/provider/openai_responses/repository.rs:687-699 \u2014 These 4 members share a duplicated core: a run of at least 50 identical tokens appears in every one of them. That run is NOT broken out as duplicated-block rows below \u2014 it is what admitted this row, and the blocks below cover only the part of it that clears the block floor, so they understate the correspondence. Read the members as one construct written 4 times. The repair is at the members\u0027 grain \u2014 factor the shared implementation out once and have all of them call it with their differences as parameters or as an injected step, or, where the difference is systematic, generate them from one template. Extracting the individual blocks below is not the same fix: it leaves every body in place and the next edit still has to be made 4 times."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_repo/src/provider/anthropic.rs"},"region":{"startLine":368}}}],"partialFingerprints":{"codehealthFindingId/v1":"3d1354ba029d720de79c726b2ce5646fa2004abc6cd34558bf403b27f2082f75"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (26\u201327 lines \u00D7 2): crates/forge_markdown_stream/src/table.rs:149-175 | crates/forge_markdown_stream/src/table.rs:311-336 \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":"crates/forge_markdown_stream/src/table.rs"},"region":{"startLine":149}}}],"partialFingerprints":{"codehealthFindingId/v1":"ff9806390923aeb6033d663be77cf6a482c7f292db71a7e0d59b4104a445db6f"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (24 lines \u00D7 2): crates/forge_services/src/tool_services/fs_patch.rs:514-537 | crates/forge_services/src/tool_services/fs_patch.rs:576-599 \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":"crates/forge_services/src/tool_services/fs_patch.rs"},"region":{"startLine":514}}}],"partialFingerprints":{"codehealthFindingId/v1":"e9286deedce7b1012381de08028e5ef650a459899920a3a6d301fa8bae5dd290"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (22 lines \u00D7 2): crates/forge_app/src/operation.rs:463-484 | crates/forge_app/src/operation.rs:486-507 \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":"crates/forge_app/src/operation.rs"},"region":{"startLine":463}}}],"partialFingerprints":{"codehealthFindingId/v1":"0cfb6858e358a4112eddf043a0f88b7f68236b262b38ad4e0ab66134495e1476"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (22 lines \u00D7 2): crates/forge_json_repair/src/schema_coercion.rs:135-156 | crates/forge_json_repair/src/schema_coercion.rs:338-359 \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":"crates/forge_json_repair/src/schema_coercion.rs"},"region":{"startLine":135}}}],"partialFingerprints":{"codehealthFindingId/v1":"3a80b12098fbefed2b5dce696f84d888d9e04d219c405dc04c2a3b37bc7a14b7"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (19\u201320 lines \u00D7 2): crates/forge_infra/src/mcp_client.rs:407-425 | crates/forge_infra/src/mcp_client.rs:639-658 \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":"crates/forge_infra/src/mcp_client.rs"},"region":{"startLine":407}}}],"partialFingerprints":{"codehealthFindingId/v1":"bd2041debd31632c525fc6edd590184fb61efcaed2d593a2f2b6a4ccdd171428"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (17\u201318 lines \u00D7 2): crates/forge_infra/src/auth/strategy.rs:243-260 | crates/forge_infra/src/auth/strategy.rs:346-362 \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":"crates/forge_infra/src/auth/strategy.rs"},"region":{"startLine":243}}}],"partialFingerprints":{"codehealthFindingId/v1":"28a2628dc4f76e513bd961b1174560655d8322b7a81b723c58f20858a7dfd9b9"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (16\u201318 lines \u00D7 2): crates/forge_main/src/conversation_selector.rs:58-75 | crates/forge_main/src/ui.rs:530-545 \u2014 before extracting anything, compare \u0060crates/forge_main/src/conversation_selector.rs\u0060 and \u0060crates/forge_main/src/ui.rs\u0060 as WHOLE FILES: this scan already matched 4 separate duplicated blocks between them, totalling at least 58 lines, which is the signature of one file having been copied from the other rather than of a helper waiting to be extracted. If that is what happened, the fix is to keep one copy and have the other call it (or delete it), which resolves this row and its siblings together \u2014 extracting one helper per block leaves the fork in place."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_main/src/conversation_selector.rs"},"region":{"startLine":58}}}],"partialFingerprints":{"codehealthFindingId/v1":"6762a6aa293e3bd5ac7ef7c635514a9233cc5faf01931de979670cf54e69a9c8"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (14\u201317 lines \u00D7 2): crates/forge_main/src/ui.rs:3163-3179 | crates/forge_main/src/ui.rs:3186-3199 \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":"crates/forge_main/src/ui.rs"},"region":{"startLine":3163}}}],"partialFingerprints":{"codehealthFindingId/v1":"adee8db985e5fe7e4ae0b84be94f01dd9dfb19145a5650872eee0e5348254772"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (14\u201316 lines \u00D7 2): crates/forge_main/src/conversation_selector.rs:54-67 | crates/forge_main/src/ui.rs:2116-2131 \u2014 before extracting anything, compare \u0060crates/forge_main/src/conversation_selector.rs\u0060 and \u0060crates/forge_main/src/ui.rs\u0060 as WHOLE FILES: this scan already matched 4 separate duplicated blocks between them, totalling at least 58 lines, which is the signature of one file having been copied from the other rather than of a helper waiting to be extracted. If that is what happened, the fix is to keep one copy and have the other call it (or delete it), which resolves this row and its siblings together \u2014 extracting one helper per block leaves the fork in place."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_main/src/conversation_selector.rs"},"region":{"startLine":54}}}],"partialFingerprints":{"codehealthFindingId/v1":"8aafac32db53c3612d83747019cf766a24bb4b874e8e6e8039c7ba9bfc98f482"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (16 lines \u00D7 2): crates/forge_repo/src/provider/openai.rs:83-98 | crates/forge_repo/src/provider/openai_responses/repository.rs:87-102 \u2014 before extracting anything, compare \u0060crates/forge_repo/src/provider/openai.rs\u0060 and \u0060crates/forge_repo/src/provider/openai_responses/repository.rs\u0060 as WHOLE FILES: this scan already matched 5 separate duplicated blocks between them, totalling at least 48 lines, which is the signature of one file having been copied from the other rather than of a helper waiting to be extracted. The two sit in different directories, so one cannot simply be deleted in favour of the other while both are reached separately: hoist the shared part into a location both already depend on and have each file call it, and retire whichever file turns out to have no caller of its own left. Extracting one helper per block leaves the fork in place."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_repo/src/provider/openai.rs"},"region":{"startLine":83}}}],"partialFingerprints":{"codehealthFindingId/v1":"22da44fae11c68f6ce68d136dff45792a4487493657f8c4a354eb7dacd8650cc"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (13\u201316 lines \u00D7 2): crates/forge_services/src/provider_auth.rs:37-52 | crates/forge_services/src/provider_auth.rs:124-136 \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":"crates/forge_services/src/provider_auth.rs"},"region":{"startLine":37}}}],"partialFingerprints":{"codehealthFindingId/v1":"567fcac262f61f471962ad38cf8581645aeefb67098b938c0eca1d9e21e2de34"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (15 lines \u00D7 2): crates/forge_app/src/utils.rs:19-33 | crates/forge_domain/src/tools/catalog.rs:1136-1150 \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."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_app/src/utils.rs"},"region":{"startLine":19}}}],"partialFingerprints":{"codehealthFindingId/v1":"416f8755a7589c3f20cfba61c63fe9b4b9177c9ec8265d73aaa70f299f3c0a78"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (15 lines \u00D7 2): crates/forge_json_repair/src/parser.rs:196-210 | crates/forge_json_repair/src/parser.rs:274-288 \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":"crates/forge_json_repair/src/parser.rs"},"region":{"startLine":196}}}],"partialFingerprints":{"codehealthFindingId/v1":"39a645248899054cf47ecb369ef3af2118964f98e3def63d8d1e8a6b821c9cd0"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (14 lines \u00D7 2): crates/forge_services/src/auth.rs:25-38 | crates/forge_services/src/auth.rs:43-56 \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":"crates/forge_services/src/auth.rs"},"region":{"startLine":25}}}],"partialFingerprints":{"codehealthFindingId/v1":"743f060b2299d92176845ae0cbd88efb19776a6c58fc88f8c9439691a41ddf8b"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (13 lines \u00D7 2): crates/forge_app/src/dto/openai/response.rs:384-396 | crates/forge_app/src/dto/openai/response.rs:451-463 \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":"crates/forge_app/src/dto/openai/response.rs"},"region":{"startLine":384}}}],"partialFingerprints":{"codehealthFindingId/v1":"266088ad9f6b6aef6229fd0c32d7cb85d80583846b88f904a88e0d2e4e4c6c0b"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (13 lines \u00D7 2): crates/forge_app/src/user_prompt.rs:76-88 | crates/forge_app/src/user_prompt.rs:122-134 \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":"crates/forge_app/src/user_prompt.rs"},"region":{"startLine":76}}}],"partialFingerprints":{"codehealthFindingId/v1":"7095d51ae1a82078e1ebf4eb3b5be449a73ef3725ce6dd4a4d16dfe4d48a8473"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (13 lines \u00D7 2): crates/forge_repo/src/provider/anthropic.rs:267-279 | crates/forge_repo/src/provider/google.rs:101-113 \u2014 before extracting anything, compare \u0060crates/forge_repo/src/provider/anthropic.rs\u0060 and \u0060crates/forge_repo/src/provider/google.rs\u0060 as WHOLE FILES: this scan already matched 3 separate duplicated blocks between them, totalling at least 35 lines, which is the signature of one file having been copied from the other rather than of a helper waiting to be extracted. If that is what happened, the fix is to keep one copy and have the other call it (or delete it), which resolves this row and its siblings together \u2014 extracting one helper per block leaves the fork in place."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_repo/src/provider/anthropic.rs"},"region":{"startLine":267}}}],"partialFingerprints":{"codehealthFindingId/v1":"ae03866d2c5da82c175ef20f4b30df126ef2c47680ade2cc871d3b18d0f3c67d"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (13 lines \u00D7 2): crates/forge_repo/src/provider/anthropic.rs:284-296 | crates/forge_repo/src/provider/google.rs:124-136 \u2014 before extracting anything, compare \u0060crates/forge_repo/src/provider/anthropic.rs\u0060 and \u0060crates/forge_repo/src/provider/google.rs\u0060 as WHOLE FILES: this scan already matched 3 separate duplicated blocks between them, totalling at least 35 lines, which is the signature of one file having been copied from the other rather than of a helper waiting to be extracted. If that is what happened, the fix is to keep one copy and have the other call it (or delete it), which resolves this row and its siblings together \u2014 extracting one helper per block leaves the fork in place."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_repo/src/provider/anthropic.rs"},"region":{"startLine":284}}}],"partialFingerprints":{"codehealthFindingId/v1":"f33468882534c6dad1adfef6de48c856ffd8085d6f1fcdf7139c1c90eda65643"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (13 lines \u00D7 2): crates/forge_repo/src/provider/bedrock.rs:722-734 | crates/forge_repo/src/provider/bedrock.rs:856-868 \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":"crates/forge_repo/src/provider/bedrock.rs"},"region":{"startLine":722}}}],"partialFingerprints":{"codehealthFindingId/v1":"722cc0179b8054fd0d1d7a3c11bd0757d26214514c0f3637d7fd9e4126af2cce"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (13 lines \u00D7 2): crates/forge_services/src/tool_services/fs_search.rs:330-342 | crates/forge_services/src/tool_services/fs_search.rs:368-380 \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":"crates/forge_services/src/tool_services/fs_search.rs"},"region":{"startLine":330}}}],"partialFingerprints":{"codehealthFindingId/v1":"77842cff550eca80657c1fb0e51dd7127564e891dcfb281f153be04e49ef321c"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (12 lines \u00D7 2): crates/forge_json_repair/src/parser.rs:616-627 | crates/forge_json_repair/src/parser.rs:634-645 \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":"crates/forge_json_repair/src/parser.rs"},"region":{"startLine":616}}}],"partialFingerprints":{"codehealthFindingId/v1":"f38895f7da8dc355b5a71d1d9d1ee6cd2147b4b13054e7d45f1758ca87405f39"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (12 lines \u00D7 2): crates/forge_json_repair/src/schema_coercion.rs:167-178 | crates/forge_json_repair/src/schema_coercion.rs:370-381 \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":"crates/forge_json_repair/src/schema_coercion.rs"},"region":{"startLine":167}}}],"partialFingerprints":{"codehealthFindingId/v1":"9443b446bf503c12c3c35478c1fc43c0a015762758d05440174f94162b29de34"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (12 lines \u00D7 2): crates/forge_main/src/ui.rs:516-527 | crates/forge_main/src/ui.rs:2107-2118 \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":"crates/forge_main/src/ui.rs"},"region":{"startLine":516}}}],"partialFingerprints":{"codehealthFindingId/v1":"41dcf5cf052b7f8b97607cb31ce6f65e1f32ccd220c3d1a62f5a4518b8e3dee0"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (11 lines \u00D7 4): crates/forge_json_repair/src/schema_coercion.rs:135-145 | crates/forge_json_repair/src/schema_coercion.rs:167-177 | crates/forge_json_repair/src/schema_coercion.rs:338-348 | crates/forge_json_repair/src/schema_coercion.rs:370-380 \u2014 all 4 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."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_json_repair/src/schema_coercion.rs"},"region":{"startLine":135}}}],"partialFingerprints":{"codehealthFindingId/v1":"5c859a0b3944ea001285c73a3c1aa84cb76c3a9a2a1e978180d35d906d4b0996"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (11 lines \u00D7 3): crates/forge_main/src/conversation_selector.rs:58-68 | crates/forge_main/src/ui.rs:529-540 | crates/forge_main/src/ui.rs:2120-2132 \u2014 before extracting anything, compare \u0060crates/forge_main/src/conversation_selector.rs\u0060 and \u0060crates/forge_main/src/ui.rs\u0060 as WHOLE FILES: this scan already matched 4 separate duplicated blocks between them, totalling at least 58 lines, which is the signature of one file having been copied from the other rather than of a helper waiting to be extracted. If that is what happened, the fix is to keep one copy and have the other call it (or delete it), which resolves this row and its siblings together \u2014 extracting one helper per block leaves the fork in place."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_main/src/conversation_selector.rs"},"region":{"startLine":58}}}],"partialFingerprints":{"codehealthFindingId/v1":"37a54222fa538995cf22a60aac38b66fc0173d93ca16f5c4777aac604eac95b4"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (11 lines \u00D7 2): crates/forge_domain/src/conversation_html.rs:181-191 | crates/forge_domain/src/conversation_html.rs:250-260 \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":"crates/forge_domain/src/conversation_html.rs"},"region":{"startLine":181}}}],"partialFingerprints":{"codehealthFindingId/v1":"7dabc809fd13ae64557e7ffb0bc05336d924798fa680e898c8b53fc6afb944fd"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (10\u201311 lines \u00D7 2): crates/forge_main/src/conversation_selector.rs:72-81 | crates/forge_main/src/ui.rs:2996-3006 \u2014 before extracting anything, compare \u0060crates/forge_main/src/conversation_selector.rs\u0060 and \u0060crates/forge_main/src/ui.rs\u0060 as WHOLE FILES: this scan already matched 4 separate duplicated blocks between them, totalling at least 58 lines, which is the signature of one file having been copied from the other rather than of a helper waiting to be extracted. If that is what happened, the fix is to keep one copy and have the other call it (or delete it), which resolves this row and its siblings together \u2014 extracting one helper per block leaves the fork in place."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_main/src/conversation_selector.rs"},"region":{"startLine":72}}}],"partialFingerprints":{"codehealthFindingId/v1":"a487efb1b5396af740553d3baba26d36499b3186d0c100ae336c9b19b7a3d8ff"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (10\u201311 lines \u00D7 2): crates/forge_main/src/ui.rs:1460-1470 | crates/forge_main/src/ui.rs:2959-2968 \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":"crates/forge_main/src/ui.rs"},"region":{"startLine":1460}}}],"partialFingerprints":{"codehealthFindingId/v1":"f0930f2e0e320501ef128fece63e7b9fe78afe3ab9b3acb6ddf2c5f253ff16bf"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (6\u201310 lines \u00D7 3): crates/forge_app/src/dto/anthropic/response.rs:362-367 | crates/forge_app/src/dto/anthropic/response.rs:375-384 | crates/forge_app/src/dto/anthropic/response.rs:386-395 \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."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_app/src/dto/anthropic/response.rs"},"region":{"startLine":362}}}],"partialFingerprints":{"codehealthFindingId/v1":"fae23bacc5e422258cbca739577f8cb1e9510ff3e0d8bfdf12d2b6b233645c45"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (10 lines \u00D7 2): crates/forge_json_repair/src/parser.rs:598-607 | crates/forge_json_repair/src/parser.rs:614-623 \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":"crates/forge_json_repair/src/parser.rs"},"region":{"startLine":598}}}],"partialFingerprints":{"codehealthFindingId/v1":"26ab17a073c041389c71d9ae2f1edbfc663d4a9d9e01d547915f4ad6a9d357c8"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (10 lines \u00D7 2): crates/forge_repo/src/provider/bedrock.rs:735-744 | crates/forge_repo/src/provider/bedrock.rs:869-878 \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":"crates/forge_repo/src/provider/bedrock.rs"},"region":{"startLine":735}}}],"partialFingerprints":{"codehealthFindingId/v1":"cc8aec3add71fb2438361ae7948d19374686af56fe67885ad8a97ea667bb372c"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (10 lines \u00D7 2): crates/forge_select/src/multi.rs:33-42 | crates/forge_select/src/select.rs:87-96 \u2014 the copies sit in sibling files of one directory, so a shared home is within easy reach: extract the block into a single shared function the call sites can all reach \u2014 a file they already depend on, or a new one alongside them \u2014 and call it from both call sites, so a change lands once."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_select/src/multi.rs"},"region":{"startLine":33}}}],"partialFingerprints":{"codehealthFindingId/v1":"264ccacdabb04415016c2782b85894b2f66936412d2d53031debe5322743c465"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (9 lines \u00D7 4): crates/forge_repo/src/provider/anthropic.rs:372-380 | crates/forge_repo/src/provider/bedrock.rs:1079-1087 | crates/forge_repo/src/provider/google.rs:204-212 | crates/forge_repo/src/provider/openai_responses/repository.rs:691-699 \u2014 before extracting anything, compare \u0060crates/forge_repo/src/provider/anthropic.rs\u0060 and \u0060crates/forge_repo/src/provider/google.rs\u0060 as WHOLE FILES: this scan already matched 3 separate duplicated blocks between them, totalling at least 35 lines, which is the signature of one file having been copied from the other rather than of a helper waiting to be extracted. If that is what happened, the fix is to keep one copy and have the other call it (or delete it), which resolves this row and its siblings together \u2014 extracting one helper per block leaves the fork in place."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_repo/src/provider/anthropic.rs"},"region":{"startLine":372}}}],"partialFingerprints":{"codehealthFindingId/v1":"fc257e003eb49a719ccf5d65cf9475086c133edbe730530b6b9c64a6e0a96161"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (9 lines \u00D7 2): crates/forge_app/src/dto/openai/response.rs:360-368 | crates/forge_app/src/dto/openai/response.rs:428-436 \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":"crates/forge_app/src/dto/openai/response.rs"},"region":{"startLine":360}}}],"partialFingerprints":{"codehealthFindingId/v1":"9c0330cbd74e524e7a42c0629eb107ae7c2197c49bceef4509ba1f57a95128fb"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (9 lines \u00D7 2): crates/forge_json_repair/src/schema_coercion.rs:77-85 | crates/forge_json_repair/src/schema_coercion.rs:88-96 \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":"crates/forge_json_repair/src/schema_coercion.rs"},"region":{"startLine":77}}}],"partialFingerprints":{"codehealthFindingId/v1":"e2e9d1744bfd1864f1322d4abbafe040a47ce7bf7c5b426be862386090257c1a"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (8\u20139 lines \u00D7 2): crates/forge_services/src/tool_services/fs_patch.rs:320-327 | crates/forge_services/src/tool_services/fs_patch.rs: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."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_services/src/tool_services/fs_patch.rs"},"region":{"startLine":320}}}],"partialFingerprints":{"codehealthFindingId/v1":"04bb45843320eb20ecee8abf50a91520df726700aefcf270a686e2b0c5472af1"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (8 lines \u00D7 2): crates/forge_app/src/dto/openai/response.rs:349-356 | crates/forge_app/src/dto/openai/response.rs:416-424 \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":"crates/forge_app/src/dto/openai/response.rs"},"region":{"startLine":349}}}],"partialFingerprints":{"codehealthFindingId/v1":"034ab6e02d985ccb19a0e54eaeed0b3751e735f29ba1451f7928053f381e3196"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (8 lines \u00D7 2): crates/forge_app/src/dto/openai/transformers/github_copilot_reasoning.rs:20-27 | crates/forge_app/src/dto/openai/transformers/reasoning_content.rs:21-28 \u2014 the copies sit in sibling files of one directory, so a shared home is within easy reach: extract the block into a single shared function the call sites can all reach \u2014 a file they already depend on, or a new one alongside them \u2014 and call it from both call sites, so a change lands once."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_app/src/dto/openai/transformers/github_copilot_reasoning.rs"},"region":{"startLine":20}}}],"partialFingerprints":{"codehealthFindingId/v1":"5aa974051e76788028160f6df5a89c789500b8c80f2f7e8f9ebd90b223a1a916"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (8 lines \u00D7 2): crates/forge_domain/src/conversation_html.rs:29-36 | crates/forge_domain/src/conversation_html.rs:110-117 \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":"crates/forge_domain/src/conversation_html.rs"},"region":{"startLine":29}}}],"partialFingerprints":{"codehealthFindingId/v1":"fbb9cec572caabd011fc31dd9a32743de7752d3785892023d69faac3a2b92ab8"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (8 lines \u00D7 2): crates/forge_eventsource_stream/src/event_stream.rs:246-253 | crates/forge_eventsource_stream/src/event_stream.rs:274-281 \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":"crates/forge_eventsource_stream/src/event_stream.rs"},"region":{"startLine":246}}}],"partialFingerprints":{"codehealthFindingId/v1":"e7ece9de5910bab5c472cb7b5406e147a9791706c44f6a05dd13c9a5e1ef5d2b"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (7\u20138 lines \u00D7 2): crates/forge_main/src/ui.rs:2174-2180 | crates/forge_main/src/ui.rs:2203-2210 \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":"crates/forge_main/src/ui.rs"},"region":{"startLine":2174}}}],"partialFingerprints":{"codehealthFindingId/v1":"3d0a58e1582ac9b93dd51935edd71a2e7598bcb8a500150abdb02b39e18ffcc9"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (8 lines \u00D7 2): crates/forge_repo/src/provider/openai.rs:73-80 | crates/forge_repo/src/provider/openai_responses/repository.rs:77-84 \u2014 before extracting anything, compare \u0060crates/forge_repo/src/provider/openai.rs\u0060 and \u0060crates/forge_repo/src/provider/openai_responses/repository.rs\u0060 as WHOLE FILES: this scan already matched 5 separate duplicated blocks between them, totalling at least 48 lines, which is the signature of one file having been copied from the other rather than of a helper waiting to be extracted. The two sit in different directories, so one cannot simply be deleted in favour of the other while both are reached separately: hoist the shared part into a location both already depend on and have each file call it, and retire whichever file turns out to have no caller of its own left. Extracting one helper per block leaves the fork in place."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_repo/src/provider/openai.rs"},"region":{"startLine":73}}}],"partialFingerprints":{"codehealthFindingId/v1":"da4fbd1aba5614ab39f5492d7e7919f1d02a4ccacc5310e14a94537191aad65c"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (8 lines \u00D7 2): crates/forge_repo/src/provider/openai.rs:113-120 | crates/forge_repo/src/provider/openai_responses/repository.rs:117-124 \u2014 before extracting anything, compare \u0060crates/forge_repo/src/provider/openai.rs\u0060 and \u0060crates/forge_repo/src/provider/openai_responses/repository.rs\u0060 as WHOLE FILES: this scan already matched 5 separate duplicated blocks between them, totalling at least 48 lines, which is the signature of one file having been copied from the other rather than of a helper waiting to be extracted. The two sit in different directories, so one cannot simply be deleted in favour of the other while both are reached separately: hoist the shared part into a location both already depend on and have each file call it, and retire whichever file turns out to have no caller of its own left. Extracting one helper per block leaves the fork in place."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_repo/src/provider/openai.rs"},"region":{"startLine":113}}}],"partialFingerprints":{"codehealthFindingId/v1":"a9e258bdda721dcc9041b539fb81262571986d9846df98ed7eafba9fff8f2e05"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (8 lines \u00D7 2): crates/forge_services/src/tool_services/fs_patch.rs:271-278 | crates/forge_services/src/tool_services/fs_patch.rs:294-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."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_services/src/tool_services/fs_patch.rs"},"region":{"startLine":271}}}],"partialFingerprints":{"codehealthFindingId/v1":"16ec51435fb13df7542b99d93d9ba56fb2a063cf71b3132ee6f5ab3db0090bf3"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (7 lines \u00D7 3): crates/forge_app/src/operation.rs:302-308 | crates/forge_app/src/operation.rs:475-481 | crates/forge_app/src/operation.rs:498-504 \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."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_app/src/operation.rs"},"region":{"startLine":302}}}],"partialFingerprints":{"codehealthFindingId/v1":"72b1b66ee0ccd8473e99f926face604df2330a25cd1516f73343c2ec55f3bf5e"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (7 lines \u00D7 3): crates/forge_repo/src/provider/anthropic.rs:46-52 | crates/forge_repo/src/provider/openai.rs:75-81 | crates/forge_repo/src/provider/openai_responses/repository.rs:79-85 \u2014 before extracting anything, compare \u0060crates/forge_repo/src/provider/openai.rs\u0060 and \u0060crates/forge_repo/src/provider/openai_responses/repository.rs\u0060 as WHOLE FILES: this scan already matched 5 separate duplicated blocks between them, totalling at least 48 lines, which is the signature of one file having been copied from the other rather than of a helper waiting to be extracted. The two sit in different directories, so one cannot simply be deleted in favour of the other while both are reached separately: hoist the shared part into a location both already depend on and have each file call it, and retire whichever file turns out to have no caller of its own left. Extracting one helper per block leaves the fork in place."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_repo/src/provider/anthropic.rs"},"region":{"startLine":46}}}],"partialFingerprints":{"codehealthFindingId/v1":"b3b38d941ce9f8325b361910d9bf757d8b0e38feefb764bbd7a20253f364b992"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (7 lines \u00D7 2): crates/forge_domain/src/message.rs:44-50 | crates/forge_domain/src/message.rs:72-78 \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":"crates/forge_domain/src/message.rs"},"region":{"startLine":44}}}],"partialFingerprints":{"codehealthFindingId/v1":"61465902c006e3608c1b84da36f275e445fa840216da7f8cbc768d13f3e9915c"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (7 lines \u00D7 2): crates/forge_domain/src/policies/policy.rs:39-46 | crates/forge_domain/src/policies/policy.rs:48-54 \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":"crates/forge_domain/src/policies/policy.rs"},"region":{"startLine":39}}}],"partialFingerprints":{"codehealthFindingId/v1":"9ccbc2196d1f27d623a7782203d30779490027130b4f48a12804292077cb96a4"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (6\u20137 lines \u00D7 2): crates/forge_main/src/ui.rs:1450-1456 | crates/forge_main/src/ui.rs:2951-2956 \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":"crates/forge_main/src/ui.rs"},"region":{"startLine":1450}}}],"partialFingerprints":{"codehealthFindingId/v1":"abd42eb629d6cc6937d31e4b6b2edeab42fd74d98e03ae28efdf2473843664ff"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (7 lines \u00D7 2): crates/forge_spinner/src/lib.rs:151-157 | crates/forge_spinner/src/lib.rs:166-172 \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":"crates/forge_spinner/src/lib.rs"},"region":{"startLine":151}}}],"partialFingerprints":{"codehealthFindingId/v1":"15a3eda6bd451a562bcdec2716595447a39c2094ed82ecbc8cd7c6e96c6ac4e2"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (6 lines \u00D7 2): crates/forge_infra/src/auth/strategy.rs:464-469 | crates/forge_repo/src/provider/provider_repo.rs:516-524 \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."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_infra/src/auth/strategy.rs"},"region":{"startLine":464}}}],"partialFingerprints":{"codehealthFindingId/v1":"e254cfec42489f48be09eb4a9dafab9e645a6301569a57b7d129b06521e5ea71"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (6 lines \u00D7 2): crates/forge_main/src/ui.rs:1043-1048 | crates/forge_main/src/ui.rs:1116-1121 \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":"crates/forge_main/src/ui.rs"},"region":{"startLine":1043}}}],"partialFingerprints":{"codehealthFindingId/v1":"5fa521bdb9662387f39cc1e731286cef79f1d81e7486d76438c83457ad5dc3b8"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (6 lines \u00D7 2): crates/forge_main/src/ui.rs:1398-1403 | crates/forge_main/src/ui.rs:3558-3563 \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":"crates/forge_main/src/ui.rs"},"region":{"startLine":1398}}}],"partialFingerprints":{"codehealthFindingId/v1":"fd64c18f7e99b59631786983b3d67389ccfd4a51bb85ed76259ce612bceecf11"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (5 lines \u00D7 2): crates/forge_domain/src/tool_order.rs:94-98 | crates/forge_domain/src/tool_order.rs:108-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."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_domain/src/tool_order.rs"},"region":{"startLine":94}}}],"partialFingerprints":{"codehealthFindingId/v1":"6293fde16aa21b46ecb7491d1378d3140d211d9153d892108efcb73df51e49c5"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (6 lines \u00D7 2): crates/forge_app/src/utils.rs:103-108 | crates/forge_fs/src/lib.rs:30-35 \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."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_app/src/utils.rs"},"region":{"startLine":103}}}],"partialFingerprints":{"codehealthFindingId/v1":"b0d665b709dea4c37e87f262d34fb09154287a521f57bb0e1aba358f608e3560"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (7 lines \u00D7 2): crates/forge_infra/src/http.rs:273-279 | crates/forge_repo/src/provider/utils.rs:11-17 \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."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_infra/src/http.rs"},"region":{"startLine":273}}}],"partialFingerprints":{"codehealthFindingId/v1":"bc6b988e004433d5115fde72838bb4662f0b00ced4fca31dc9b30e80c779bfd3"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (13 lines \u00D7 2): crates/forge_repo/src/provider/anthropic.rs:302-314 | crates/forge_repo/src/provider/openai_responses/repository.rs:382-394 \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."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_repo/src/provider/anthropic.rs"},"region":{"startLine":302}}}],"partialFingerprints":{"codehealthFindingId/v1":"b2f1bb444a5b57483e9943130ba8bf309e838ee031b842c74dc2d5927d05ce1d"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (12 lines \u00D7 2): crates/forge_infra/src/auth/strategy.rs:185-196 | crates/forge_infra/src/auth/strategy.rs:710-725 \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":"crates/forge_infra/src/auth/strategy.rs"},"region":{"startLine":185}}}],"partialFingerprints":{"codehealthFindingId/v1":"3a160e139a3084febe42b3ea709dc38c38ee4b83fa5b385094dd271f59385261"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (9 lines \u00D7 2): crates/forge_app/src/template_engine.rs:27-35 | crates/forge_app/src/template_engine.rs:43-54 \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":"crates/forge_app/src/template_engine.rs"},"region":{"startLine":27}}}],"partialFingerprints":{"codehealthFindingId/v1":"e6e69f7f07889e465fda22242e7a9cc73fd24ecdc43409c6c8df3cfdc4c79b07"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (11 lines \u00D7 2): crates/forge_main/src/ui.rs:4091-4101 | crates/forge_main/src/ui.rs:4115-4125 \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":"crates/forge_main/src/ui.rs"},"region":{"startLine":4091}}}],"partialFingerprints":{"codehealthFindingId/v1":"d29ccf910eb69780d8fb7fc82ab8c09d14614b9c199ebf952977bd5b0b681dd7"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (10 lines \u00D7 2): crates/forge_repo/src/provider/openai_responses/response.rs:206-215 | crates/forge_repo/src/provider/openai_responses/response.rs:234-243 \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":"crates/forge_repo/src/provider/openai_responses/response.rs"},"region":{"startLine":206}}}],"partialFingerprints":{"codehealthFindingId/v1":"cc07236b7a48caaa553754cbd67e1ba20442012968e7d970b33f91dff7946a3a"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (9 lines \u00D7 2): crates/forge_repo/src/agent.rs:89-97 | crates/forge_services/src/command.rs:77-85 \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."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_repo/src/agent.rs"},"region":{"startLine":89}}}],"partialFingerprints":{"codehealthFindingId/v1":"fd245ad574413cfcbde2881f16cd367fa9205ae0db088b265047187fa6981a03"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (9 lines \u00D7 2): crates/forge_app/src/operation.rs:250-258 | crates/forge_app/src/operation.rs:283-291 \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":"crates/forge_app/src/operation.rs"},"region":{"startLine":250}}}],"partialFingerprints":{"codehealthFindingId/v1":"5609d7f38d879a1f6210146fe16b4c4b22ada67a9166cdde623179b52bfabf3e"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (8 lines \u00D7 2): crates/forge_repo/src/provider/openai.rs:331-339 | crates/forge_repo/src/provider/openai_responses/repository.rs:712-719 \u2014 before extracting anything, compare \u0060crates/forge_repo/src/provider/openai.rs\u0060 and \u0060crates/forge_repo/src/provider/openai_responses/repository.rs\u0060 as WHOLE FILES: this scan already matched 5 separate duplicated blocks between them, totalling at least 48 lines, which is the signature of one file having been copied from the other rather than of a helper waiting to be extracted. The two sit in different directories, so one cannot simply be deleted in favour of the other while both are reached separately: hoist the shared part into a location both already depend on and have each file call it, and retire whichever file turns out to have no caller of its own left. Extracting one helper per block leaves the fork in place."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_repo/src/provider/openai.rs"},"region":{"startLine":331}}}],"partialFingerprints":{"codehealthFindingId/v1":"06c7fb89634b3f1fcb5588421c8767fcc92adb4d2c52205704b415257eb03466"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (8 lines \u00D7 2): crates/forge_main/src/ui.rs:4580-4587 | crates/forge_main/src/ui.rs:4590-4597 \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":"crates/forge_main/src/ui.rs"},"region":{"startLine":4580}}}],"partialFingerprints":{"codehealthFindingId/v1":"82f091b8f193a8e5bb22de0c623c0f75cba0ea9946929a6dd1efe9df23db71ef"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (7 lines \u00D7 2): crates/forge_app/src/operation.rs:533-539 | crates/forge_app/src/operation.rs:541-547 \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":"crates/forge_app/src/operation.rs"},"region":{"startLine":533}}}],"partialFingerprints":{"codehealthFindingId/v1":"ba1717e195c65028989b3a976b104877d0c101e05d383bf450be100cce7a21fc"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (5 lines \u00D7 3): crates/forge_services/src/tool_services/fs_patch.rs:249-254 | crates/forge_services/src/tool_services/fs_patch.rs:321-325 | crates/forge_services/src/tool_services/fs_patch.rs:341-345 \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."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_services/src/tool_services/fs_patch.rs"},"region":{"startLine":249}}}],"partialFingerprints":{"codehealthFindingId/v1":"7877995f60c2d4a70e421dd5f253763b6f7d3fbc8c8b66bfa6c7b0c7b8beacd5"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (5 lines \u00D7 2): crates/forge_main/src/ui.rs:2495-2499 | crates/forge_main/src/ui.rs:4516-4520 \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":"crates/forge_main/src/ui.rs"},"region":{"startLine":2495}}}],"partialFingerprints":{"codehealthFindingId/v1":"013468e6f9a93a6a6f9c6e438dbfbc41f4d29b04d3ae37b677b599da943b42b8"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (5 lines \u00D7 2): crates/forge_markdown_stream/src/theme.rs:294-298 | crates/forge_markdown_stream/src/theme.rs:341-345 \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":"crates/forge_markdown_stream/src/theme.rs"},"region":{"startLine":294}}}],"partialFingerprints":{"codehealthFindingId/v1":"fb8202856abb991160146160c2cc158f048466c18256af25fde1a05c45251345"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (5 lines \u00D7 2): crates/forge_select/src/preview.rs:757-761 | crates/forge_select/src/preview.rs:784-788 \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":"crates/forge_select/src/preview.rs"},"region":{"startLine":757}}}],"partialFingerprints":{"codehealthFindingId/v1":"cf292edca1639cd6f828d8d893800b89f18bfd82837171c5f99983877bc59f7b"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (5 lines \u00D7 2): crates/forge_select/src/preview.rs:790-794 | crates/forge_select/src/preview.rs:917-921 \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":"crates/forge_select/src/preview.rs"},"region":{"startLine":790}}}],"partialFingerprints":{"codehealthFindingId/v1":"cd91bd2bbe206217542f2d7ddb86388dd4553f358aa573df2f3a0f4938ad2bba"}},{"ruleId":"D5","level":"warning","message":{"text":"Unstable project forge_api: forge_api has instability 0.88 with 1 dependents."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"b0a961bb4f9b4e44edc6a400aa96677d6a23849039c745f2adb9dcfb24cefd6b"}},{"ruleId":"D5","level":"warning","message":{"text":"Unstable project forge_infra: forge_infra has instability 0.82 with 2 dependents."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"f05c39d2e3db3b9d75d518c7fbecfaed25729775bf6c0957742d94e2587a3dcc"}},{"ruleId":"D5","level":"warning","message":{"text":"Unstable project forge_repo: forge_repo has instability 0.91 with 1 dependents."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"b126c93f987a3d95f6a05ce574c60bd63a48bdcc429bf043baf23dc626717b85"}},{"ruleId":"D5","level":"warning","message":{"text":"Off the main sequence: forge_display: forge_display: abstractness 0.00, instability 0.00, distance 1.00 \u2014 zone of pain \u2014 concrete and depended on by 2 project(s), so it\u0027s rigid to change."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"afd44c69e3e2feef6a65f3b133b7c831cd1321fae43eb70ffdf8a98277d4e146"}},{"ruleId":"D5","level":"warning","message":{"text":"Off the main sequence: forge_json_repair: forge_json_repair: abstractness 0.00, instability 0.00, distance 1.00 \u2014 zone of pain \u2014 concrete and depended on by 3 project(s), so it\u0027s rigid to change."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"155ae81c5a4a3a7d25f5904a41ab8a056cb777388df1a7dc4873cc4b09b48f8d"}},{"ruleId":"D5","level":"warning","message":{"text":"Off the main sequence: forge_select: forge_select: abstractness 0.00, instability 0.00, distance 1.00 \u2014 zone of pain \u2014 concrete and depended on by 2 project(s), so it\u0027s rigid to change."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"9f3e66b00e5e78c1bbad22e0d3095d670484c59f0f5aa79584ab0456bbe97ff0"}},{"ruleId":"D5","level":"warning","message":{"text":"Off the main sequence: forge_stream: forge_stream: abstractness 0.00, instability 0.00, distance 1.00 \u2014 zone of pain \u2014 concrete and depended on by 3 project(s), so it\u0027s rigid to change."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"54091c48821c9868991293877b107f959ccf77913406b1073c6ce3369e2192b1"}},{"ruleId":"D5","level":"warning","message":{"text":"Off the main sequence: forge_walker: forge_walker: abstractness 0.00, instability 0.00, distance 1.00 \u2014 zone of pain \u2014 concrete and depended on by 2 project(s), so it\u0027s rigid to change."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"53a2aa6cc530537614b5200ba2c52ce4187a858d401d3d8c54689b32c115b29e"}},{"ruleId":"D5","level":"warning","message":{"text":"Off the main sequence: forge_config: forge_config: abstractness 0.00, instability 0.14, distance 0.86 \u2014 zone of pain \u2014 concrete and depended on by 6 project(s), so it\u0027s rigid to change."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"f7957895282c10e2ac12b8bec2d7bd81a3af7f05f26a0a7e85ad1d727f102d88"}},{"ruleId":"D5","level":"warning","message":{"text":"Off the main sequence: forge_fs: forge_fs: abstractness 0.00, instability 0.17, distance 0.83 \u2014 zone of pain \u2014 concrete and depended on by 5 project(s), so it\u0027s rigid to change."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"b52c48d6a8bc9fb75abf59c58292d97f5013fd8154d03661040f4d0ced90f3fa"}},{"ruleId":"D5","level":"warning","message":{"text":"Off the main sequence: forge_eventsource_stream: forge_eventsource_stream: abstractness 0.20, instability 0.00, distance 0.80 \u2014 zone of pain \u2014 concrete and depended on by 2 project(s), so it\u0027s rigid to change."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"3c93095bbd5b7e50a9749f47a07e37b89669a8870484bf995277edf5f07060a3"}},{"ruleId":"D6","level":"warning","message":{"text":"Low cohesion: Context (LCOM4 4): Context\u0027s methods fall into 4 groups that share no field and call none of each other, against a bar of more than 3 for this run (LCOM4, configurable \u2014 your repository\u0027s bar is the one quoted here). Each group is a set of methods reachable from one another through shared fields or direct calls, so 4 groups means the type has that many internally-connected clusters with nothing tying them together. Types whose shape makes a high count expected \u2014 and which would otherwise dominate this list \u2014 are excluded before this row is raised, so this is a genuine split candidate rather than a metric reading. It is still a shape, not a defect: confirm the groups match responsibilities you can name before splitting."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_domain/src/context.rs"},"region":{"startLine":401}}}],"partialFingerprints":{"codehealthFindingId/v1":"61d4184f64941f6dc4ecb788183f79b2c3ffdaf7800568cb52ce9b5b7add758d"}},{"ruleId":"D15","level":"warning","message":{"text":"Hotspot: crates/forge_domain/src/provider.rs: crates/forge_domain/src/provider.rs changed 6 times in last 90 days, and the most complex body those changes touched has cyclomatic complexity 42 in ProviderId::from_str at line 194. 5 of those changes were fix/bug commits, so the churn is repair rather than feature work. Before the next change lands here, make sure the area it touches is under test, 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. Counted over 2026-07-01..2026-09-29, the 90 days ending at the analysed commit. Reproduce with \u0060git log --since=\u00272026-07-01 15:42:13 \u002B05:30\u0027 --until=\u00272026-09-29 15:42:13 \u002B05:30\u0027 --full-history --no-merges -- crates/forge_domain/src/provider.rs\u0060: merges are excluded because a merge re-states changes already counted at their own commits, and history is NOT path-simplified because a change that reached the file through a merged branch is still a change to it. That command counts raw commits and can read HIGHER than this row, which counts a cherry-picked re-land, and a revert together with the commit it undoes, once each \u2014 a difference of several commits on a file whose history was re-landed or reverted inside the window."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_domain/src/provider.rs"},"region":{"startLine":194}}}],"partialFingerprints":{"codehealthFindingId/v1":"3535bda5605654c8db461ce1178a3ef7cf3945c698255faa1c3a0e1487d07f19"}},{"ruleId":"D15","level":"warning","message":{"text":"Hotspot: crates/forge_app/src/dto/anthropic/response.rs: crates/forge_app/src/dto/anthropic/response.rs changed 5 times in last 90 days, and the most complex body those changes touched has cyclomatic complexity 17 in forge_app::dto::anthropic::response::get_context_length at line 81. 3 of those changes were fix/bug commits, so the churn is repair rather than feature work. Before the next change lands here, make sure the area it touches is under test, 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. Counted over 2026-07-01..2026-09-29, the 90 days ending at the analysed commit. Reproduce with \u0060git log --since=\u00272026-07-01 15:42:13 \u002B05:30\u0027 --until=\u00272026-09-29 15:42:13 \u002B05:30\u0027 --full-history --no-merges -- crates/forge_app/src/dto/anthropic/response.rs\u0060: merges are excluded because a merge re-states changes already counted at their own commits, and history is NOT path-simplified because a change that reached the file through a merged branch is still a change to it. That command counts raw commits and can read HIGHER than this row, which counts a cherry-picked re-land, and a revert together with the commit it undoes, once each \u2014 a difference of several commits on a file whose history was re-landed or reverted inside the window."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_app/src/dto/anthropic/response.rs"},"region":{"startLine":81}}}],"partialFingerprints":{"codehealthFindingId/v1":"ec687ebbdc3b559344a3cc96c1e5b77ce1ecc9021b1fc4800cbad512f4d1cca0"}},{"ruleId":"D15","level":"warning","message":{"text":"Hotspot: crates/forge_repo/src/provider/openai_responses/repository.rs: crates/forge_repo/src/provider/openai_responses/repository.rs changed 2 times in last 90 days, and the most complex body those changes touched has cyclomatic complexity 16 in OpenAIResponsesProvider::chat at line 155. 1 of those changes was a fix/bug commit, and the other 1 changed it for other reasons \u2014 this file is under both repair and feature pressure. Before the next change lands here, make sure the area it touches is under test, 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. Counted over 2026-07-01..2026-09-29, the 90 days ending at the analysed commit. Reproduce with \u0060git log --since=\u00272026-07-01 15:42:13 \u002B05:30\u0027 --until=\u00272026-09-29 15:42:13 \u002B05:30\u0027 --full-history --no-merges -- crates/forge_repo/src/provider/openai_responses/repository.rs\u0060: merges are excluded because a merge re-states changes already counted at their own commits, and history is NOT path-simplified because a change that reached the file through a merged branch is still a change to it. That command counts raw commits and can read HIGHER than this row, which counts a cherry-picked re-land, and a revert together with the commit it undoes, once each \u2014 a difference of several commits on a file whose history was re-landed or reverted inside the window."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_repo/src/provider/openai_responses/repository.rs"},"region":{"startLine":155}}}],"partialFingerprints":{"codehealthFindingId/v1":"b4afae394bc159a8700981130c71ed243ab1dc2eb779f38eb54b15fa8ab1906f"}},{"ruleId":"D16","level":"note","message":{"text":"Off-boarding risk: anonymized user #1: If anonymized user #1 becomes unavailable, 8 significant file(s) lose their only recent owner: crates/forge_json_repair/src/schema_coercion.rs, crates/forge_app/src/dto/openai/copilot_model.rs, crates/forge_repo/src/provider/bedrock_sanitize_ids.rs, crates/forge_app/src/dto/openai/transformers/trim_tool_call_ids.rs, crates/forge_app/src/dto/anthropic/transforms/sanitize_tool_ids.rs, crates/forge_infra/src/auth/mcp_credentials.rs, crates/forge_repo/src/provider/bedrock_cache.rs, crates/forge_app/src/dto/openai/transformers/default_reasoning_content.rs. Pair on, review, or document these before any departure."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"e0474d2396f3fcfdb2d55682376a129877faba506df48ca4e24c815fd8b29ccd"}},{"ruleId":"D16","level":"note","message":{"text":"Off-boarding risk: anonymized user #2: If anonymized user #2 becomes unavailable, 4 significant file(s) lose their only recent owner: crates/forge_main/src/sync_display.rs, crates/forge_app/src/dto/anthropic/transforms/drop_invalid_toolcalls.rs, crates/forge_infra/src/walker.rs, crates/forge_main/src/logs.rs. Pair on, review, or document these before any departure."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"8f169c4e4b7757ecfd7f44231aa22f0d7847d612ab65765d6963acf65b1072de"}},{"ruleId":"D16","level":"note","message":{"text":"Further sole-owners (lower concentration): 2 other contributor(s) are each the sole owner of a small amount of code below the off-boarding threshold \u2014 folded into the bus-factor score and metrics (14 single-owned of 288 analysed files in total, counted over production source files of roughly 2,400 bytes or more, excluding vendored, generated and example/demo trees and test files identified by path convention, largest first; 288 of the 462 production source files in this repository met that bar). They are anonymized user #3 (1 file(s)), anonymized user #4 (1 file(s)) \u2014 spread or document their files in the same way, at lower priority than the named off-boarding risks above."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"6eac528f2429e724e1a97ed868f79eefbcf1888dab4af9a9e673429369bb5b2f"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // todo list \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_app/src/user_prompt.rs"},"region":{"startLine":595}}}],"partialFingerprints":{"codehealthFindingId/v1":"f736dce3e9d8868f29cf564eba0e2318b6e85c84f522bbad350a127fcbb89e9f"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: We should move Snapshot service to Services from infra \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_app/src/services.rs"},"region":{"startLine":429}}}],"partialFingerprints":{"codehealthFindingId/v1":"1dbab72c02b6764de430558946ef112fd265134627ccae1ae5419133a6267174"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Consider using \u0060eviction.max(retention)\u0060 \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_app/src/compact.rs"},"region":{"startLine":46}}}],"partialFingerprints":{"codehealthFindingId/v1":"885de486e6ae7361041b1e2477d200dde9b3104f68d47f9f3fbd7cac04a102ac"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Needs review, should we throw an err here? \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_app/src/agent_provider_resolver.rs"},"region":{"startLine":35}}}],"partialFingerprints":{"codehealthFindingId/v1":"f4543549b1b064d1669870faa651d1fc1898a4f3e3c4ef2aa1773d178d3d1220"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Needs review, should we throw an err here? \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_app/src/agent_provider_resolver.rs"},"region":{"startLine":62}}}],"partialFingerprints":{"codehealthFindingId/v1":"38bf7aab63d5bcddc4f3d84e5fa6bcd4511ff999c79566a86d9159c0d008ffb1"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: verify this Thinking variants don\u0027t support cache control \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_app/src/dto/anthropic/request.rs"},"region":{"startLine":407}}}],"partialFingerprints":{"codehealthFindingId/v1":"db0392dbc8419bd712e4f12e0a1afcfbf508fbd3cf602e77c509b1225e950896"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: should be an enum \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_app/src/dto/openai/request.rs"},"region":{"startLine":161}}}],"partialFingerprints":{"codehealthFindingId/v1":"146e432a3b8574cd92d0ba9fd8363e1658f088dbe69686dc4ac18e67592d4f55"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: /// Todo operation - each todo_write is unique and won\u0027t be deduplicated \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_app/src/transformers/trim_context_summary.rs"},"region":{"startLine":40}}}],"partialFingerprints":{"codehealthFindingId/v1":"65638182d1f3573c4d24ac39a059da7132a45d8182999db0b166da487763f54b"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: add provider information to the model \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_domain/src/model.rs"},"region":{"startLine":33}}}],"partialFingerprints":{"codehealthFindingId/v1":"606ebf52c88448f992f754ce8b95b4ac7aa63e7bda29ceceb1f5e9b47a064cad"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: //TODO: Rename to TextMessage \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_domain/src/context.rs"},"region":{"startLine":294}}}],"partialFingerprints":{"codehealthFindingId/v1":"dffd847fe40fec42b7698173180759ad21b67eb1a06791e012ac4426aeb6fa70"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: This is an extremely slow operation \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_domain/src/tools/catalog.rs"},"region":{"startLine":1188}}}],"partialFingerprints":{"codehealthFindingId/v1":"2f9b6e0c3f055b766f299b830afbe7a2a9002809cffa8f694893f713bbd3d0e6"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Drop the \u0022Service\u0022 suffix. Use names like ForgeFileReader, ForgeFileWriter, \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_infra/src/forge_infra.rs"},"region":{"startLine":38}}}],"partialFingerprints":{"codehealthFindingId/v1":"b179be9caa2cd31be910e7adf560efa6281e1e2c8d740fc3e54c3e6fdd231c33"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: //TODO: UIState and ForgePrompt seem like the same thing and can be merged \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_main/src/state.rs"},"region":{"startLine":6}}}],"partialFingerprints":{"codehealthFindingId/v1":"a55ed4069a561f75a38bd8c58286773f39c94af80a3aa20ff541b4526b615edc"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Move this deserialization logic into Conversation repo \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_repo/src/conversation/conversation_record.rs"},"region":{"startLine":535}}}],"partialFingerprints":{"codehealthFindingId/v1":"8a327125ec88bf78bfb87100a994fb599083c1b88f8d3fa47f6f3724816a067f"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Implement it on Context or Conversation instead of ConverseStreamInput \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_repo/src/provider/bedrock_cache.rs"},"region":{"startLine":16}}}],"partialFingerprints":{"codehealthFindingId/v1":"0b76893c78787ae59af99d6c04ca4d94b91340e260ba350ec714344f971c11c7"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Can return a diff later \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_services/src/policy.rs"},"region":{"startLine":69}}}],"partialFingerprints":{"codehealthFindingId/v1":"5c508d0194e59528c55b26baaf5ecf962be025bf72e542312d6be048ab817111"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: use forge_fs \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_services/src/tool_services/fs_patch.rs"},"region":{"startLine":494}}}],"partialFingerprints":{"codehealthFindingId/v1":"cb5126f7736344711189eb570ceca244d77c642efb943257474e2437958082ba"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: move http request to a dispatch \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_tracker/src/collect/posthog.rs"},"region":{"startLine":97}}}],"partialFingerprints":{"codehealthFindingId/v1":"beaae27c49392cf1832c2fd122cc5ed6b73f5a75d1a821c30c4b2753726fc77f"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Convert to async and return a stream \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_walker/src/walker.rs"},"region":{"startLine":136}}}],"partialFingerprints":{"codehealthFindingId/v1":"4bd175487881f859ada7282ea5d374b2f9b069f667174a9582a1ab6316d95f3f"}},{"ruleId":"D20","level":"note","message":{"text":"No ADRs found: No ADRs found. No recognised ADR directory (\u0060docs/adr/\u0060, \u0060docs/decisions/\u0060, \u0060adr/\u0060, \u0060docs/rfcs/\u0060, an \u0060ADR0001/\u0060 folder, or their siblings) exists anywhere in this tree. What was searched, so you can tell an empty log from a search that missed one: every directory under the tree (build output, dependencies and VCS metadata excepted), for a document that is either any non-index page inside a recognised ADR directory, whatever its name and however deeply nested (\u0060docs/adr/use-postgres.md\u0060, \u0060docs/adr/2024/0001-x.md\u0060); or a file anywhere whose name is ADR-shaped (\u00600001-use-postgres.md\u0060, \u0060adr-012-caching.md\u0060); or, when neither turned anything up, a document carrying the decision-record signature (an \u0022Architecture Decision Record\u0022 heading, or Status / Context / Decision / Consequences as section headings). A decision log that clears none of these \u2014 unnumbered files outside any recognised directory, without those headings \u2014 is not seen by this check and this row is then wrong. If that is your case, say so rather than renaming anything; otherwise, consider recording architectural decisions in \u0060docs/adr/\u0060."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"d2bea044ff79d7d275f5a91a6e2f548586178eaf480274c33960ad020c854631"}},{"ruleId":"D22","level":"warning","message":{"text":"Duplicate intent: Both methods appear to retrieve the list of available tools. The naming convention differs (\u0027get\u0027 vs \u0027list\u0027) and they reside in different service layers (API vs App), suggesting a lack of clear separation of concerns or a redundant wrapper.: Unify to a single source of truth. If \u0060ForgeApp\u0060 is the core domain service, expose \u0060list_tools\u0060 there and have \u0060API.get_tools\u0060 delegate to it. Standardize on \u0027list\u0027 for collections. (signatures: API.get_tools(): Result | ForgeApp.list_tools(): Result)"},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"e315a269997cd66b48f0fd727e7acfedb75b45067a79510dcfe9885a340b706c"}},{"ruleId":"D22","level":"warning","message":{"text":"Duplicate intent: Both methods retrieve the list of available models. Similar to the tools issue, this duplication across API and App layers without clear distinction in signature or naming is inconsistent.: Unify to \u0060ForgeApp.get_models\u0060 and have the API layer delegate. Ensure the return type is consistent (e.g., a specific Model list type rather than generic Result). (signatures: API.get_models(): Result | ForgeApp.get_models(): Result)"},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"12a955335da86e2f6673d8ae5fff69cf5bea595a06a2ce1450840edcc5467586"}},{"ruleId":"D22","level":"warning","message":{"text":"Duplicate intent: Both methods retrieve all provider-specific models. The identical naming across different types suggests a copy-paste pattern or lack of abstraction, but the existence of both implies potential redundancy or unclear ownership.: Consolidate into \u0060ForgeApp.get_all_provider_models\u0060. The API layer should not duplicate this logic unless it adds specific filtering not present in the app layer, which is not indicated by the signature. (signatures: API.get_all_provider_models(): Result | ForgeApp.get_all_provider_models(): Result)"},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"6c8db425efb6cda8e9598c17c1d6f4e6723bac08cd00d47c43a979ec72483a6b"}},{"ruleId":"D22","level":"warning","message":{"text":"Inconsistent naming and signature for shell execution: The API exposes \u0060execute_shell_command\u0060 and \u0060execute_shell_command_raw\u0060, while the underlying infrastructure exposes \u0060execute_command\u0060 and \u0060execute_command_raw\u0060. The \u0027shell\u0027 prefix is inconsistent between layers. Additionally, the parameters differ significantly (API uses \u0060working_dir\u0060 and no \u0060env_vars\u0060 explicitly in the raw version, while Infra uses \u0060env_vars\u0060 and \u0060silent\u0060), making the mapping unclear.: Standardize on \u0060execute_command\u0060 across layers. If \u0027shell\u0027 is a specific domain concept, use it consistently. Clarify the relationship between \u0027raw\u0027 and standard execution (e.g., does raw imply no shell wrapper or no env var handling?). Align parameter names. (signatures: API.execute_shell_command(command: str, working_dir: PathBuf): Result | API.execute_shell_command_raw(command: str): Result | CommandInfra.execute_command(command: String, working_dir: PathBuf, silent: bool, env_vars: String): Result | CommandInfra.execute_command_raw(command: str, working_dir: PathBuf, env_vars: String): Result)"},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"0b95d01cbd9ff7dbbfcc3183d8594457e521d4bd810a5232890d935fb6648bc9"}},{"ruleId":"D22","level":"warning","message":{"text":"Duplicate intent: Both methods retrieve agent information. The API layer duplicates the repository method without a clear differentiator in the signature (both return generic Result).: Remove \u0060API.get_agent_infos\u0060 if it simply delegates to the repository. If it adds transformation, rename to \u0060API.get_agent_details\u0060 or similar to indicate the added value. (signatures: API.get_agent_infos(): Result | AgentRepository.get_agent_infos(): Result)"},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"a02310f08f55cf7b72455d9fbee705d96a3656b718742a50d12ac7dca3140ffc"}},{"ruleId":"D22","level":"warning","message":{"text":"Duplicate intent: Both methods resolve the provider for a given agent. The API layer duplicates the resolver\u0027s functionality.: Have \u0060API.get_agent_provider\u0060 delegate to \u0060AgentProviderResolver.get_provider\u0060. Ensure the return type is consistent. (signatures: API.get_agent_provider(agent_id: AgentId): Result | AgentProviderResolver.get_provider(agent_id: AgentId): Result)"},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"8e7550a442c40e87e242014e1a5de07b8b5a7594198f692ffca256e9d851e641"}},{"ruleId":"D22","level":"warning","message":{"text":"Inconsistent return types for similar operations: \u0060API.get_agent_model\u0060 returns \u0060ModelId\u0060 directly, while \u0060AgentProviderResolver.get_model\u0060 returns \u0060Result\u0060. This inconsistency in error handling (one throws/panics on failure, the other returns a Result) is confusing for API consumers.: Standardize on returning \u0060Result\u003CModelId\u003E\u0060 for both to handle potential resolution failures gracefully. (signatures: API.get_agent_model(agent_id: AgentId): ModelId | AgentProviderResolver.get_model(agent_id: AgentId): Result)"},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"b040bcca2d340269855e8c7ff4adb83f1723e817077e41ef31c3a67595bd67da"}},{"ruleId":"D26","level":"note","message":{"text":"Projects may be oversized for their cohesion: 2 of 25 project(s) overshoot their size bounds, lowering Project Cohesion to 8.4/10. The most over is \u0060crates/forge_app\u0060 (29835 LoC, 311 public types across 12 directories). Review these for cohesion \u2014 draw the boundary inside the module first (group each responsibility into its own package or directory and keep the cross-boundary members non-public), since splitting a published package moves types between packages and breaks consumers."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"d5a94650dc74886a0f1f08eb9fb6775f1fc395279ef7e901c970c9d26f767a72"}},{"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":"0285cc52d398ed77d6db3dd6685ed3a88e2463f16d3c215842507180d2c55286"},"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":"a824d8c1469ec4a0eb449d2cd827feeaab938689168ac00562ee167080ba4b69"},"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":"7c811a64ac346c52b1940436711a44aa300ce3b781154c4bf140bd0ff63135fc"},"taxa":[{"id":"CWE-522","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"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":"8605ea17501f26ff1aa32db104943e13f34a39789a2ced923bee79ceed27aae8"},"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":"e8be707755755685b77f500972396bbfc78bcc132311d92cffd4d3e25ce4429d"},"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":"2a0be52f1beb0fd0a9a3e3139f1c54b73074b469c4ec500fa9bc31cf20087faf"},"taxa":[{"id":"CWE-214","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-532","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":"954fc072740588a038db1a22d036f1e5c2ca6a8854b646d35a7e2265ff1f4ae0"},"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":"b47689ebad5a8f2671a544fcf3e610579838bfea9eae0544609e433aa08446d1"},"taxa":[{"id":"CWE-214","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-532","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":"28847bfb68128ad6c95eb6a19a3df3a0787d1bdadcf090baba24802b8a233364"},"taxa":[{"id":"CWE-732","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":"703b990f6d9a60db46de372b4bc3713bc769740b61be7dfed2b23b523664ac12"},"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":"76b1aee0684dd72e51e0aefef8b3759450aca1c37a61b14bf7923519ee602e7b"},"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":"c81e985a7dd07de975c7ef57495d875a147b20911a75c10b16927ab819ffaf8a"},"taxa":[{"id":"CWE-522","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"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":"af124d1319ae313fb19f1e202367a85bc4a75eb673f6288a3384afca9f0abceb"},"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":"482e3b19efd040fb2215efbc5e65f5b9996234817368c5a9198eb89910e46231"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"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":"bc26593caf509b487cb3dff9d1a587737a02c72fc5e13e59c1a151f7d6412ad4"},"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":"6bb9cf1195ec12a40db09939947f98fc2e7c8621a77e1a791a5eb53da2fbb34d"},"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":"f5eeab96d2b61d744f1f7de393206e4fb02e6fc024316146d95fd7483478d2fe"},"taxa":[{"id":"CWE-522","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"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":"cc1fa63a7fda7200fccfafce8e41075841b3b266b1f262f93d706132d313e16e"},"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":"6bc3b539f953f180f304780b633bbb65e6d2911ce57503f2f0e4a05dfd504060"},"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":"000b811b23da1065238bc2fcfbcc72acd8cceb53fc8a964dd67340076d07cd09"},"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":"78636907e3bfeb465c6a82fc3785f3f815502fe5a4e4daba7fa37b578165b6f1"},"taxa":[{"id":"CWE-522","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"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":"28fb95c74cf7e51a3f6e58d6d5d9aaba698bdc10ef21fbd45c72dc90736c26c6"},"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":"a5ff6c36cc77c52281fa395ac6b9e1e18754b22fb93caffb8d30a7715d1e22e8"},"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":"4c04097bfa4cef1c891aff0825baf09174c7f7c9e501de2f936698ce48bbb776"},"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":"d16b56b9264f1bbaec0a3deebd4fd0f32d1767d5b19dfa7381bd7929af1a0e26"},"taxa":[{"id":"CWE-522","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"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":"7cbc34ace1f085d4c3ca2ab5d3e7fc9e3615c46ac9576b4817a4229fc612ed25"},"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":"d13dd8426a8a32655cde684b9dca20298e11c8fed618b7e1e6c3e28b3d639687"},"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":"bbf5c3ec40f4aa1855e8d400e6942127028fa007757477c9ce1de82a6c4e9547"},"taxa":[{"id":"CWE-522","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"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":"697b121d2ca84985bf1bdcec53408e627ae1fdf25cb480d680687ebdd03152e3"},"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":"1375048b9ca8ef192c23faf83af425d2e99a11eb2f6e7951f3d837d8c63c990e"},"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":"9e9ced78c6a92f6c31fd89d86642cb70bb77073d44bb62db27b61a507511397e"},"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":"2a32d75f1b5db579865f2ae7a4ebb596cc235388463afb7a283db5a86cfcf607"},"taxa":[{"id":"CWE-522","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"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":"056d03c5b3fbd554325e8d45e1ff456b8851ce7c0c33ce8c14fe6a632f840240"},"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":"9106879afc7874b7d64a3fb75f7aaa72622ed16a633b7b6825ebbcda840bb393"},"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":"00f6c3712098a59250b858f7b01e291b5afa4793b78a951e4d5a62908c8afca6"},"taxa":[{"id":"CWE-522","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"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":"2bab1df41990b251e9a3086a90ed5b2410e0450cc6851a08ecbdb352d596bfa0"},"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":"53a2916440c1e13c853fb75eb5e15cec59ca542fd5f7852166998c1b52502927"},"taxa":[{"id":"CWE-214","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-532","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":"91521117a20472542f0521253904e9f9c97e37ea7668a367a7710fcbfdbf497f"},"taxa":[{"id":"CWE-269","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"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":"a5ad627f28730e24904f000f79fd2bdcc43b74655ee59bc98e207bbecc80f43c"},"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":"5c1b27f444ee9d48ae76463cfd1bb763c2d0286df94968144ab53792bf5c9449"},"taxa":[{"id":"CWE-522","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"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":"918e87ed446d72f76f309d3bfca113544f498227508f646edffff92970336aab"},"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":"dd7ebd8eea6873c2ce7f6f0e4b75806ed89760b5c2831810d528fcb1a20bc68f"},"taxa":[{"id":"CWE-522","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"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":"5be03ab325ce0dd57044d4e411f1428508ecf7870bea56774afff406be2e7ce3"},"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":"d29688f45b2bc3fb841fa1991841d9f617b13b9371f598be920f6051cba6d6ed"},"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":"08cfa45b295d9a05394b748cfb9b711ec5b47c6d005178d01f592ca75627e270"},"taxa":[{"id":"CWE-522","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"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":"ac291b612c659d1f3b777111252bf86a0300d78247e1652776528ed274bbeb82"},"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":"11d364b1cd33824e644fa644e3564aa06a8754b7874bb31ca34872e7fe34d94b"},"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":"0eb17d088671dedf675a5aeaf5981794e687c461f9990585fbb14ba3e543cdfa"},"taxa":[{"id":"CWE-522","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"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":"4c2149720e1c3fc03cf479374b81f857e6a3e6836a39766f5df26f3f3b4f986f"},"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":"75369c09bb288495c1ec96550eb2aa9585bb73f2d0abcdd876376a94900b52cc"},"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":"dfcdb4b780a860a9975856f693a257545be3ff23c5a3c2e15292c545a485c71e"},"taxa":[{"id":"CWE-78","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":"3643ff687baaf09e4fa86f24721b381bb8ec583e4fb1bc9174a893ca376bdfb1"},"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":"2c1c5cfb5ffb698000cf0be3f001db88940557e01122c111cff604ad0b2df75d"},"taxa":[{"id":"CWE-78","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":"b045f06583c9b465f808666ab807bb301b75e163408a0670ffe050a381f461f6"},"taxa":[{"id":"CWE-214","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-532","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":"ab8d83539c5ebe567cb6deae98d392d6617f9ea4001a34a9a21fb3f393c0be8f"},"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":"662c0ae4f372a49a57979908fcf460b3db827fc68a7631f4070481950ef58412"},"taxa":[{"id":"CWE-78","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D30","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":"5ed384055dc716a282d5e05d367f817795de1f0324f3b20b12b91701c74092f0"},"properties":{"dependency":{"package":"rmcp","version":"1.8.0","advisory":"[GHSA redacted]","aliases":["[CVE redacted]","[CVE redacted]","[CVE redacted]","[GHSA redacted]","[GHSA redacted]"],"reachability":{"kind":"unknown"}}}},{"ruleId":"D30","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":"b5308f9ed219ff7d361ed2b01ab7b5fa7a33b0013fcd35575e7068ccda1d2997"},"properties":{"dependency":{"package":"quinn-proto","version":"0.11.14","advisory":"[GHSA redacted]","aliases":["[CVE redacted]","RUSTSEC-2026-0185"],"reachability":{"kind":"unknown"}}}},{"ruleId":"D30","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":"0d3d2c2346bccaaa14c1bd0f0713c18ae4de849fffdc7282dbd533ab105c9398"},"properties":{"dependency":{"package":"rustls-webpki","version":"0.101.7","advisory":"[GHSA redacted]","aliases":["[CVE redacted]","[CVE redacted]","[CVE redacted]","[GHSA redacted]","[GHSA redacted]","RUSTSEC-2026-0098","RUSTSEC-2026-0099","RUSTSEC-2026-0104"],"reachability":{"kind":"unknown"}}}},{"ruleId":"D30","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":"84478e16a12858ab2ebd7037ad3bc3d1975f9cebe46b89975eb341aba1217962"},"properties":{"dependency":{"package":"hickory-proto","version":"0.25.2","advisory":"[GHSA redacted]","aliases":["[GHSA redacted]","RUSTSEC-2026-0118","RUSTSEC-2026-0119","RUSTSEC-2026-0120"],"reachability":{"kind":"unknown"}}}},{"ruleId":"D30","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":"b873cf768c77dcd3130dcabea8e403d85fedf544ec95e711983ac77e0dcd7758"},"properties":{"dependency":{"package":"libyml","version":"0.0.5","advisory":"[GHSA redacted]","aliases":["RUSTSEC-2025-0067"],"reachability":{"kind":"unknown"}}}},{"ruleId":"D30","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":"b995e75272b61cf5f3649600c4af60acb108488f8113b0f25a04ec6792e70778"},"properties":{"dependency":{"package":"cmov","version":"0.5.3","advisory":"[GHSA redacted]","aliases":["[CVE redacted]"],"reachability":{"kind":"unknown"}}}},{"ruleId":"D30","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":"5ceb9040e13548b3df718b94867858f8e5659519eb178fbf2fd0e4e23343e0da"},"properties":{"dependency":{"package":"yaml-rust","version":"0.4.5","advisory":"RUSTSEC-2024-0320","reachability":{"kind":"unknown"}}}},{"ruleId":"D30","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":"5922a5beeab5ac077ffb485a783ad545036e79b56239f83da38f3fb84336b113"},"properties":{"dependency":{"package":"paste","version":"1.0.15","advisory":"RUSTSEC-2024-0436","reachability":{"kind":"unknown"}}}},{"ruleId":"D30","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":"b5524f25221308ec94b7206b58afaa59996104c20abaa242285d451f5344e44b"},"properties":{"dependency":{"package":"rustls-pemfile","version":"1.0.4","advisory":"RUSTSEC-2025-0134","reachability":{"kind":"unknown"}}}},{"ruleId":"D30","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":"4975822701b3b96654c82dada9bad969e348a6adefa52cd3bba1b6cfd71c3bb7"},"properties":{"dependency":{"package":"bincode","version":"1.3.3","advisory":"RUSTSEC-2025-0141","reachability":{"kind":"unknown"}}}},{"ruleId":"D30","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":"d469c4cdd0b2892278c4ee90c01a5a8ea4872a98733c483777e04c904cfebdfc"},"properties":{"dependency":{"package":"proc-macro-error2","version":"2.0.1","advisory":"RUSTSEC-2026-0173","reachability":{"kind":"unknown"}}}},{"ruleId":"D30","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":"8f033184701e1b0bb0a41dc94d91bb8f68c0336d8f7a6d1b9083b901dc9f1df6"},"properties":{"dependency":{"package":"serde_yml","version":"0.0.13","advisory":"RUSTSEC-2025-0068","aliases":["[GHSA redacted]"],"reachability":{"kind":"unknown"}}}},{"ruleId":"D30","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":"d44a25813d8a9e18eb42fa5350a972c298292d454155c6472f091bfb670363df"},"properties":{"dependency":{"package":"rand","version":"0.8.5","advisory":"RUSTSEC-2026-0097","aliases":["[GHSA redacted]"],"reachability":{"kind":"unknown"}}}},{"ruleId":"D30","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":"a7ac1525ee21f8396482ed6caafb5092cc4f1f938990a606ace871da2f8aa6f6"},"properties":{"dependency":{"package":"serde_with","version":"3.18.0","advisory":"[GHSA redacted]","reachability":{"kind":"unknown"}}}},{"ruleId":"D30","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":"91d43a93b3706ec87efad5b997b467aa58e0ea37e710457e42ec82a30c7c70e5"},"properties":{"dependency":{"package":"serde_yml","version":"0.0.12","advisory":"[GHSA redacted]","aliases":["RUSTSEC-2025-0068"],"reachability":{"kind":"unknown"}}}},{"ruleId":"D30","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":"5585c48a8c4ed670247ebaa40a533d385248c4e49079fee700e7b93cbab137a9"},"properties":{"dependency":{"package":"quick-xml","version":"0.38.4","advisory":"RUSTSEC-2026-0194","aliases":["RUSTSEC-2026-0195"],"reachability":{"kind":"unknown"}}}},{"ruleId":"D30","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":"c4a0c58aca13bc848c37d4c3d5bcb90f8e046f8df5b386f5a3743d0625ad8dae"},"properties":{"dependency":{"package":"crossbeam-epoch","version":"0.9.18","advisory":"RUSTSEC-2026-0204","reachability":{"kind":"unknown"}}}},{"ruleId":"D30","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":"a96829008b8bd26424b0dbf2d446f682c023dea58a16e6d8ece060f154efaf0f"},"properties":{"dependency":{"package":"h2","version":"0.3.27","advisory":"RUSTSEC-2026-0258","aliases":["[GHSA redacted]"],"reachability":{"kind":"unknown"}}}},{"ruleId":"D30","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":"1aeda6bb9ec20b9417e4f543809e49662aa43dedcf4fdc584b04edfa54a2e8a4"},"properties":{"dependency":{"package":"h2","version":"0.4.13","advisory":"RUSTSEC-2026-0258","aliases":["[GHSA redacted]"],"reachability":{"kind":"unknown"}}}},{"ruleId":"D30","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":"4e552103ab1a3aa40047b41afa8510ba2a32878c4c786dcfca60ff29a670f18b"},"properties":{"dependency":{"package":"undici","version":"7.29.0","advisory":"[GHSA redacted]","aliases":["[CVE redacted]"],"reachability":{"kind":"unknown"}}}},{"ruleId":"D34","level":"error","message":{"text":"Orphaned knowledge: No living knowledge remains for this large file \u2014 its last meaningful change has decayed away, so if it breaks, no one currently understands it. It does carry its own tests, so the behaviour is pinned even though the understanding is gone: schedule a read-through, using those tests as the specification, before the next change lands here."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_main/src/oauth_callback.rs"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"f01502e050de0f42252c72a01373dff42b881fa9c6ba9bde77ea332c6f2095b5"}},{"ruleId":"D34","level":"note","message":{"text":"Further orphaned files (smaller): 35 smaller file(s) also have no living knowledge \u2014 folded into the freshness score and metrics rather than raised one row each \u2014 most significant first: crates/forge_eventsource_stream/src/event_stream.rs, crates/forge_domain/src/tool_order.rs, crates/forge_app/src/terminal_context.rs, crates/forge_domain/src/policies/policy.rs, crates/forge_eventsource/src/event_source.rs, crates/forge_domain/src/message_pattern.rs, crates/forge_template/src/element.rs, crates/forge_app/src/transformers/dedupe_role.rs (and 27 more) (36 orphaned of 288 analysed files in total, counted over production source files of roughly 2,400 bytes or more, excluding vendored, generated and example/demo trees and test files identified by path convention, largest first; 288 of the 462 production source files in this repository met that bar). 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: drop_tool_call.rs \u2194 set_cache.rs: \u0060crates/forge_app/src/dto/openai/transformers/drop_tool_call.rs\u0060 and \u0060crates/forge_app/src/dto/openai/transformers/set_cache.rs\u0060 change together 92% of the time (12 of the 13 commits that touched whichever of the two files changed less often, counting a file under its earlier names as well \u2014 a repo-wide or module-wide sweep is evidence about the sweep rather than about any pair inside it and is left out of BOTH sides of this ratio, while a dependency bump, a formatter/rename sweep, or a commit whose edit to one of the two files was a tool directive such as //go:generate or whitespace only is left out of the shared count ONLY, so the two sides are not taken over identical commit sets) with no explicit dependency between them. They sit in the same directory, but in this ecosystem each file is its own module \u2014 a sibling reference still needs an import \u2014 so the missing import edge is real: the coupling runs through shared behaviour, not a declared dependency. If they duplicate structure, extract the common part into one unit; otherwise the coupling is hidden and worth breaking. You can check this without leaving the row: of the 12 shared commits counted here, the most recent 3 are \u00608c0af00b\u0060 fix: add GitHub Copilot optimization headers for billing. (#2813); \u00603c27238e\u0060 refactor(commit): request commit messages with json schema structured\u2026; \u0060d529c960\u0060 refactor: streaming control via context (#2066) \u2014 run \u0060git show\u0060 on any of them."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_app/src/dto/openai/transformers/drop_tool_call.rs"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"9a3faa9421320339689a7d6247676fa7386c8b24433dabae5b3ac52eb7d3e3d0"}},{"ruleId":"D35","level":"warning","message":{"text":"Change-coupling hub: orch.rs \u2192 agent_executor.rs, compact_config.rs, event.rs, system_context.rs, conversation.rs, template.rs: \u0060crates/forge_app/src/orch.rs\u0060 changes together with 6 other files \u2014 \u0060crates/forge_app/src/agent_executor.rs\u0060, \u0060crates/forge_domain/src/compact/compact_config.rs\u0060, \u0060crates/forge_domain/src/event.rs\u0060, \u0060crates/forge_domain/src/system_context.rs\u0060, \u0060crates/forge_services/src/conversation.rs\u0060, \u0060crates/forge_services/src/template.rs\u0060 \u2014 none of which declares a dependency on it: one file is the hub of 6 separate couplings, not 6 unrelated pairs. Read the hub first: if the others each duplicate a part of what it does, the shared concern belongs in ONE unit and extracting it clears every edge at once; if the hub is a registry, dispatcher or barrel that must name each of them, the coupling is structural and the question is whether that list can be discovered instead of enumerated. Fixing the hub is one change; breaking the couplings one pair at a time is 6."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_app/src/orch.rs"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"82cb2005567516e4d498e2fc1a8f0b83a86817ccac09939770cc0b6d07b2cdc9"}},{"ruleId":"D35","level":"warning","message":{"text":"Change coupling: tool_executor.rs \u2194 summary.rs: \u0060crates/forge_app/src/tool_executor.rs\u0060 and \u0060crates/forge_domain/src/compact/summary.rs\u0060 change together 77% of the time (10 of the 13 commits that touched whichever of the two files changed less often, counting a file under its earlier names as well \u2014 a repo-wide or module-wide sweep is evidence about the sweep rather than about any pair inside it and is left out of BOTH sides of this ratio, while a dependency bump, a formatter/rename sweep, or a commit whose edit to one of the two files was a tool directive such as //go:generate or whitespace only is left out of the shared count ONLY, so the two sides are not taken over identical commit sets) 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. You can check this without leaving the row: of the 10 shared commits counted here, the most recent 3 are \u00609d5094fb\u0060 feat: subagent as task (#2508); \u006098732649\u0060 feat(tools): add multi_patch tool for batch file edits (#2840); \u0060a7a77f7f\u0060 fix(semantic_search): remove extensions field (#2339) \u2014 run \u0060git show\u0060 on any of them."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_app/src/tool_executor.rs"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"5b3b5e3f2eb047f817a9a40987d92c394af0b7b7c6f37f49f9f68b258d2bc9cd"}},{"ruleId":"D35","level":"warning","message":{"text":"Change-coupling hub: fmt_input.rs \u2192 operation.rs, tool_executor.rs, summary.rs: \u0060crates/forge_app/src/fmt/fmt_input.rs\u0060 changes together with 3 other files \u2014 \u0060crates/forge_app/src/operation.rs\u0060, \u0060crates/forge_app/src/tool_executor.rs\u0060, \u0060crates/forge_domain/src/compact/summary.rs\u0060 \u2014 none of which declares a dependency on it: one file is the hub of 3 separate couplings, not 3 unrelated pairs. Read the hub first: if the others each duplicate a part of what it does, the shared concern belongs in ONE unit and extracting it clears every edge at once; if the hub is a registry, dispatcher or barrel that must name each of them, the coupling is structural and the question is whether that list can be discovered instead of enumerated. Fixing the hub is one change; breaking the couplings one pair at a time is 3."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_app/src/fmt/fmt_input.rs"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"19f135cb4f3de62c653c2f01eed956d76bd337998be26d41679c6734b8ee12f0"}},{"ruleId":"D35","level":"warning","message":{"text":"Change-coupling hub: executor.rs \u2192 operation.rs, attachment.rs, shell.rs: \u0060crates/forge_infra/src/executor.rs\u0060 changes together with 3 other files \u2014 \u0060crates/forge_app/src/operation.rs\u0060, \u0060crates/forge_services/src/attachment.rs\u0060, \u0060crates/forge_services/src/tool_services/shell.rs\u0060 \u2014 none of which declares a dependency on it: one file is the hub of 3 separate couplings, not 3 unrelated pairs. Read the hub first: if the others each duplicate a part of what it does, the shared concern belongs in ONE unit and extracting it clears every edge at once; if the hub is a registry, dispatcher or barrel that must name each of them, the coupling is structural and the question is whether that list can be discovered instead of enumerated. Fixing the hub is one change; breaking the couplings one pair at a time is 3."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_infra/src/executor.rs"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"90544382b075b8bcccd13e2279fabc9dddc08852812b25680980c1b77866b374"}},{"ruleId":"D35","level":"warning","message":{"text":"Change coupling: attachment.rs \u2194 shell.rs: \u0060crates/forge_services/src/attachment.rs\u0060 and \u0060crates/forge_services/src/tool_services/shell.rs\u0060 change together 67% of the time (8 of the 12 commits that touched whichever of the two files changed less often, counting a file under its earlier names as well \u2014 a repo-wide or module-wide sweep is evidence about the sweep rather than about any pair inside it and is left out of BOTH sides of this ratio, while a dependency bump, a formatter/rename sweep, or a commit whose edit to one of the two files was a tool directive such as //go:generate or whitespace only is left out of the shared count ONLY, so the two sides are not taken over identical commit sets) 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. You can check this without leaving the row: of the 8 shared commits counted here, the most recent 3 are \u00604dd37a8b\u0060 feat(permissions): enable permission checks only in restricted mode (\u2026; \u0060038552d7\u0060 feat: support Streaming HTTP (#1864); \u00604c4c35e4\u0060 refactor: use \u0060faker\u0060 to generate env in tests (#1767) \u2014 run \u0060git show\u0060 on any of them."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_services/src/attachment.rs"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"1c338876d83f4ea59ed4ea98514fdb698d23ed94368aeb025aba6944a2889ad2"}},{"ruleId":"D35","level":"warning","message":{"text":"Change-coupling hub: api.rs \u2192 command_generator.rs, services.rs, repo.rs, app_config.rs, conversation.rs: \u0060crates/forge_api/src/api.rs\u0060 changes together with 5 other files \u2014 \u0060crates/forge_app/src/command_generator.rs\u0060, \u0060crates/forge_app/src/services.rs\u0060, \u0060crates/forge_domain/src/repo.rs\u0060, \u0060crates/forge_services/src/app_config.rs\u0060, \u0060crates/forge_services/src/conversation.rs\u0060 \u2014 none of which declares a dependency on it: one file is the hub of 5 separate couplings, not 5 unrelated pairs. Read the hub first: if the others each duplicate a part of what it does, the shared concern belongs in ONE unit and extracting it clears every edge at once; if the hub is a registry, dispatcher or barrel that must name each of them, the coupling is structural and the question is whether that list can be discovered instead of enumerated. Fixing the hub is one change; breaking the couplings one pair at a time is 5."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_api/src/api.rs"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"b188351c295f282be24d3d4cf0dd1fc7341a91e371e764e189280375d4135ee9"}},{"ruleId":"D35","level":"warning","message":{"text":"Change coupling: services.rs \u2194 repo.rs: \u0060crates/forge_app/src/services.rs\u0060 and \u0060crates/forge_domain/src/repo.rs\u0060 change together 64% of the time (7 of the 11 commits that touched whichever of the two files changed less often, counting a file under its earlier names as well \u2014 a repo-wide or module-wide sweep is evidence about the sweep rather than about any pair inside it and is left out of BOTH sides of this ratio, while a dependency bump, a formatter/rename sweep, or a commit whose edit to one of the two files was a tool directive such as //go:generate or whitespace only is left out of the shared count ONLY, so the two sides are not taken over identical commit sets) 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. You can check this without leaving the row: of the 7 shared commits counted here, the most recent 3 are \u00606481375e\u0060 refactor(workspace): rename codebase to workspace in types and servic\u2026; \u0060e86205fc\u0060 fix(operation): use cdata for warning messages to prevent html escapi\u2026; \u00605a805e25\u0060 feat: add conversation delete command (#2077) \u2014 run \u0060git show\u0060 on any of them."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_app/src/services.rs"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"81a4b98727dbca1341aa705881af17be5307357589d09cc9b66c52990cf799b7"}},{"ruleId":"D35","level":"warning","message":{"text":"Change coupling: forge_api.rs \u2194 app_config.rs: \u0060crates/forge_api/src/forge_api.rs\u0060 and \u0060crates/forge_services/src/app_config.rs\u0060 change together 62% of the time (10 of the 16 commits that touched whichever of the two files changed less often, counting a file under its earlier names as well \u2014 a repo-wide or module-wide sweep is evidence about the sweep rather than about any pair inside it and is left out of BOTH sides of this ratio, while a dependency bump, a formatter/rename sweep, or a commit whose edit to one of the two files was a tool directive such as //go:generate or whitespace only is left out of the shared count ONLY, so the two sides are not taken over identical commit sets) 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. You can check this without leaving the row: of the 10 shared commits counted here, the most recent 3 are \u0060a387bfd0\u0060 refactor(config): replace get_default_provider/get_provider_model wit\u2026; \u0060bfd9f7fc\u0060 fix: atomic provider and model selection with proper cancellation han\u2026; \u0060da37b431\u0060 Feat(suggest-model): add suggest model command (#2511) \u2014 run \u0060git show\u0060 on any of them."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_api/src/forge_api.rs"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"cd7ce4e3fbc4885cfca66884af1c38ed8cef81467a34da9783511ffd8cb2d48d"}},{"ruleId":"D35","level":"warning","message":{"text":"Change coupling: services.rs \u2194 lib.rs: \u0060crates/forge_app/src/services.rs\u0060 and \u0060crates/forge_services/src/lib.rs\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 \u2014 a repo-wide or module-wide sweep is evidence about the sweep rather than about any pair inside it and is left out of BOTH sides of this ratio, while a dependency bump, a formatter/rename sweep, or a commit whose edit to one of the two files was a tool directive such as //go:generate or whitespace only is left out of the shared count ONLY, so the two sides are not taken over identical commit sets) 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. You can check this without leaving the row: of the 8 shared commits counted here, the most recent 3 are \u006096509376\u0060 fix(sync): use env-based batch size for consistent file-read concurre\u2026; \u00606e20da27\u0060 feat: ability to define provider per agent (#1705); \u00601a22db8b\u0060 feat: ability to define commands in MD (#1708) \u2014 run \u0060git show\u0060 on any of them."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_app/src/services.rs"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"759ee9d385995ce9b5038a378a61ee9aaca03c780caee7aab8039073522b82d6"}},{"ruleId":"D35","level":"warning","message":{"text":"Change coupling: orch_setup.rs \u2194 env.rs: \u0060crates/forge_app/src/orch_spec/orch_setup.rs\u0060 and \u0060crates/forge_infra/src/env.rs\u0060 change together 59% of the time (20 of the 34 commits that touched whichever of the two files changed less often, counting a file under its earlier names as well \u2014 a repo-wide or module-wide sweep is evidence about the sweep rather than about any pair inside it and is left out of BOTH sides of this ratio, while a dependency bump, a formatter/rename sweep, or a commit whose edit to one of the two files was a tool directive such as //go:generate or whitespace only is left out of the shared count ONLY, so the two sides are not taken over identical commit sets) 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. You can check this without leaving the row: of the 20 shared commits counted here, the most recent 3 are \u0060279ec190\u0060 feat(config): allow configuration of the base_path (#2864); \u0060e75c7837\u0060 chore(provider): increase model cache TTL to one week (#2588); \u0060e25c1c08\u0060 fix: add FIXME comments for EMFILE risk in unbounded parallel file re\u2026 \u2014 run \u0060git show\u0060 on any of them."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_app/src/orch_spec/orch_setup.rs"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"266770f899a5048863dbf192f8b837d9bdeb706312e8e3428516e63b53118bc6"}},{"ruleId":"D35","level":"warning","message":{"text":"Change coupling: forge_infra.rs \u2194 attachment.rs: \u0060crates/forge_infra/src/forge_infra.rs\u0060 and \u0060crates/forge_services/src/attachment.rs\u0060 change together 50% of the time (15 of the 30 commits that touched whichever of the two files changed less often, counting a file under its earlier names as well \u2014 a repo-wide or module-wide sweep is evidence about the sweep rather than about any pair inside it and is left out of BOTH sides of this ratio, while a dependency bump, a formatter/rename sweep, or a commit whose edit to one of the two files was a tool directive such as //go:generate or whitespace only is left out of the shared count ONLY, so the two sides are not taken over identical commit sets) 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. You can check this without leaving the row: of the 15 shared commits counted here, the most recent 3 are \u006027124d74\u0060 fix(http): append debug request body chunks instead of overwriting (#\u2026; \u00601b114a43\u0060 fix(sync): handle error conditions in sync gracefully (#2520); \u0060b93403a8\u0060 refactor(context_engine): remove retry logic and simplify error handl\u2026 \u2014 run \u0060git show\u0060 on any of them."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_infra/src/forge_infra.rs"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"0582473b8efdced76f23b5fddcfd05144e8968b0d385a9780c7d94758e3440ef"}},{"ruleId":"D35","level":"warning","message":{"text":"Change coupling: prompt.rs \u2194 state.rs: \u0060crates/forge_main/src/prompt.rs\u0060 and \u0060crates/forge_main/src/state.rs\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 \u2014 a repo-wide or module-wide sweep is evidence about the sweep rather than about any pair inside it and is left out of BOTH sides of this ratio, while a dependency bump, a formatter/rename sweep, or a commit whose edit to one of the two files was a tool directive such as //go:generate or whitespace only is left out of the shared count ONLY, so the two sides are not taken over identical commit sets) with no explicit dependency between them. They sit in the same directory, but in this ecosystem each file is its own module \u2014 a sibling reference still needs an import \u2014 so the missing import edge is real: the coupling runs through shared behaviour, not a declared dependency. If they duplicate structure, extract the common part into one unit; otherwise the coupling is hidden and worth breaking. You can check this without leaving the row: of the 9 shared commits counted here, the most recent 3 are \u006079cae5d7\u0060 feat: support for current working directory (#1174); \u006086901d15\u0060 refactor: update agent names to Forge and Fuse (#930); \u0060577fb935\u0060 feat: allow user to select agents interactively (#919) \u2014 run \u0060git show\u0060 on any of them."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_main/src/prompt.rs"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"dd799eded635545654b78f4602fa70d5d4e344ad27a6c6c4a27e94c8579f00e7"}},{"ruleId":"D35","level":"warning","message":{"text":"Change coupling: banner.rs \u2194 model.rs: \u0060crates/forge_main/src/banner.rs\u0060 and \u0060crates/forge_main/src/model.rs\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 \u2014 a repo-wide or module-wide sweep is evidence about the sweep rather than about any pair inside it and is left out of BOTH sides of this ratio, while a dependency bump, a formatter/rename sweep, or a commit whose edit to one of the two files was a tool directive such as //go:generate or whitespace only is left out of the shared count ONLY, so the two sides are not taken over identical commit sets) with no explicit dependency between them. They sit in the same directory, but in this ecosystem each file is its own module \u2014 a sibling reference still needs an import \u2014 so the missing import edge is real: the coupling runs through shared behaviour, not a declared dependency. If they duplicate structure, extract the common part into one unit; otherwise the coupling is hidden and worth breaking. You can check this without leaving the row: of the 7 shared commits counted here, the most recent 3 are \u006016f54bbc\u0060 feat(repl): bring REPL to feature parity with ZSH shell plugin (#2984); \u0060c6dfcfb9\u0060 fix: correct command references from \u0022/conversations\u0022 to \u0022/conversati\u2026; \u006075950bfb\u0060 feat: add \u0060/usage\u0060 command (#1365) \u2014 run \u0060git show\u0060 on any of them."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_main/src/banner.rs"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"a3fbcd9acd95097d6b794d45dc9aa07783dad1f4c9f45552a24e5d00e5c7bf4d"}},{"ruleId":"D35","level":"warning","message":{"text":"Change coupling: env.rs \u2194 shell.rs: \u0060crates/forge_infra/src/env.rs\u0060 and \u0060crates/forge_services/src/tool_services/shell.rs\u0060 change together 50% of the time (6 of the 12 commits that touched whichever of the two files changed less often, counting a file under its earlier names as well \u2014 a repo-wide or module-wide sweep is evidence about the sweep rather than about any pair inside it and is left out of BOTH sides of this ratio, while a dependency bump, a formatter/rename sweep, or a commit whose edit to one of the two files was a tool directive such as //go:generate or whitespace only is left out of the shared count ONLY, so the two sides are not taken over identical commit sets) 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. You can check this without leaving the row: of the 6 shared commits counted here, the most recent 3 are \u0060d96f8f91\u0060 fix: add fallback from rbash to bash/zsh/sh for command execution (#2\u2026; \u00604dd37a8b\u0060 feat(permissions): enable permission checks only in restricted mode (\u2026; \u0060038552d7\u0060 feat: support Streaming HTTP (#1864) \u2014 run \u0060git show\u0060 on any of them."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_infra/src/env.rs"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"106a76baadfba67fdd5ab0f655b390518db2385bc8099ccaaef898952d18b6fd"}},{"ruleId":"D35","level":"warning","message":{"text":"Change coupling: operation.rs \u2194 fs_read.rs: \u0060crates/forge_app/src/operation.rs\u0060 and \u0060crates/forge_services/src/tool_services/fs_read.rs\u0060 change together 50% of the time (5 of the 10 commits that touched whichever of the two files changed less often, counting a file under its earlier names as well \u2014 a repo-wide or module-wide sweep is evidence about the sweep rather than about any pair inside it and is left out of BOTH sides of this ratio, while a dependency bump, a formatter/rename sweep, or a commit whose edit to one of the two files was a tool directive such as //go:generate or whitespace only is left out of the shared count ONLY, so the two sides are not taken over identical commit sets) 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. You can check this without leaving the row: of the 5 shared commits counted here, the most recent 3 are \u006029db91a4\u0060 fix(attachment): hash full file content instead of range content and \u2026; \u00605ed3660d\u0060 feat(tools): enhance read tool description with detailed usage guidel\u2026; \u00609298b626\u0060 fix: hash computation for externally modified files (#1979) \u2014 run \u0060git show\u0060 on any of them."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_app/src/operation.rs"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"c0ed7c4eb3726fc4684f2408a3575f239b8eaa4fb6f2e0889dcdab0d4ea663e4"}},{"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":"d068c977b62d9a977df1bb6f53e8b00b586c3abee955f91d715f92b8e019d624"}},{"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":"fe274ba5adebde25ca0b3db842e44cbb1aaeb7cfecfbb7215cd452596ba1f4c2"}},{"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":"d657599f6f99da1927d3377533dfc0888b34c4d84aa7d5579841fd72d0e39bce"}},{"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":"4f1e7684f8281bd255ddef97105c7055c2ebae06964c8745716b83e174debe83"}},{"ruleId":"AC3","level":"warning","message":{"text":"Page without a main landmark: No \u003Cmain\u003E (or role=\u0022main\u0022) means no \u0022skip to content\u0022 target and a weaker landmark map. Wrap the primary content in \u003Cmain\u003E."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_domain/src/snapshots/forge_domain__conversation_html__tests__conversation.snap.html"},"region":{"startLine":2}}}],"partialFingerprints":{"codehealthFindingId/v1":"0b519dca81f3de54d9536645001149469ecbd4d45b2447eedbd5c7acdfefdf46"}},{"ruleId":"AC6","level":"warning","message":{"text":"Low contrast colour pair in CSS (4.2:1): \u0060.agent-conversation a\u0060 sets color: #2563eb on background-color: #dbeafe \u2014 4.2:1, below the 4.5:1 WCAG AA minimum for normal text. Darken or lighten one of them."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_domain/src/snapshots/forge_domain__conversation_html__tests__conversation.snap.html"},"region":{"startLine":16}}}],"partialFingerprints":{"codehealthFindingId/v1":"2c71ecef778569b70010de2d198292d77c05cf2c9b3213f133c331c0066e88fc"}},{"ruleId":"AC6","level":"warning","message":{"text":"Low contrast colour pair in CSS (4.2:1): \u0060.agent-conversation a\u0060 sets color: #2563eb on background-color: #dbeafe \u2014 4.2:1, below the 4.5:1 WCAG AA minimum for normal text. Darken or lighten one of them."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_domain/src/conversation_style.css"},"region":{"startLine":256}}}],"partialFingerprints":{"codehealthFindingId/v1":"5d3902848a39d4cf67698df362a592f5da5c5a6018e772b53132d675d341ea73"}},{"ruleId":"AC7","level":"warning","message":{"text":"Accessibility enforcement below the top rung: No accessibility enforcement found \u2014 no a11y linter (an accessibility check that can read your UI \u2014 no component framework was detected and your pages are rendered by server-side templates, which neither the JSX/Vue ESLint plugins nor the HTML-template linters can parse; run axe/pa11y over the rendered pages, or assert the accessibility invariants over that rendered HTML in the test suite you already have) and no axe/pa11y/Lighthouse in tests or CI. Start by running that check over your rendered pages in CI. What was searched, so you can tell an absence from a miss: the 1 markup file(s) this pass actually assessed, the linter configuration checked in beside them, and this repository\u0027s test and CI files \u2014 matched by name against the accessibility checkers this dimension carries. An audit run outside the repository, a hosted scanner, or a check whose name is not one of those, is not seen here."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"d46019b86eff5ddad9db289e6802ac158899e980df54c34f67722442787ead62"}},{"ruleId":"DM4","level":"error","message":{"text":"Anemic domain model: Walker: \u0060Walker\u0060 is a data-only domain entity with \u0060pub\u0060 mutable state and no invariant-enforcing behaviour of its own, while the repo holds the logic in a foreign service/manager/use-case \u2014 the anemic domain model. Move the behaviour onto the entity so it enforces its own invariants (or make it an immutable value object with private fields) rather than being driven from outside."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_app/src/walker.rs"},"region":{"startLine":8}}}],"partialFingerprints":{"codehealthFindingId/v1":"be48df2ab804a81b553644dc9cd8a9497531e4ef9c032926e37b832856619acc"}},{"ruleId":"M2","level":"note","message":{"text":"No ADRs: No Architecture Decision Records found \u2014 no conventional ADR directory, no numbered \u0060NNNN-title\u0060 documents in any markup this check reads, and nothing ADR-shaped by content. Design rationale recorded elsewhere (a design-notes tree, a mailing list, pull-request discussion) is not visible to this check and is not re-findable per decision, so a future maintainer cannot ask why one choice was made and get an answer."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"670b3d6e36a756d63097d0dfbf90afd5fc761308800b9354894a07c3f4e4aa14"}},{"ruleId":"M4","level":"note","message":{"text":"README/code drift: README claims a ZSH \u0060:\u0060 prefix plugin mode but the evidence shows no such feature \u2014 searched for: \u0060ZSH \u0060:\u0060 prefix\u0060. Each was matched case- and separator-insensitively against file and directory NAMES anywhere in the tree, and against the CONTENTS of manifest files (package.json, *.csproj, *.props, *.slnx, *.yml, Dockerfile); the README\u0027s own prose never counts, so a claim is never refuted by merely being made. Nothing outside that search was read \u2014 a footprint living only in a submodule, in a file type not listed here, or under a name none of those terms matches is not seen, and this row is then wrong."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"343bbbd51ceb8de7a9ff88d79f7c797abf9b5921f8727b67b4d0f4f8a22c5e49"}},{"ruleId":"PF3","level":"warning","message":{"text":"Sync-over-async blocking: \u0060main\u0060 is async and calls readFileSync \u2014 a blocking call inside async code stalls the event loop \u2014 every other request waits for as long as it runs."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"benchmarks/evals/semantic_search_quality/llm_judge.ts"},"region":{"startLine":361}}}],"partialFingerprints":{"codehealthFindingId/v1":"2f6049cdcd22380e54431ca880fbdeb6b8bc910843d065acbe5860d3a190958b"}},{"ruleId":"X9","level":"note","message":{"text":"Subsumed condition operand: \u0060evalName.endsWith(\u0022task.yml\u0022)\u0060 can never decide this \u0060||\u0060 \u2014 every value satisfying \u0060evalName.endsWith(\u0022task.yml\u0022)\u0060 also satisfies \u0060evalName.endsWith(\u0022.yml\u0022)\u0060, so the \u0060||\u0060 chain is already decided by the latter. The expression is equivalent to the chain without it, which means it is wider than it reads. Delete the dead operand, or narrow the surviving one if IT is the accident."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"benchmarks/parse.ts"},"region":{"startLine":37}}}],"partialFingerprints":{"codehealthFindingId/v1":"970232136de24901950e966da10811dd5be8bf76510178256cad89dbc1849688"}},{"ruleId":"X9","level":"note","message":{"text":"Subsumed condition operand: \u0060actual.contains(\u0022HIGH\u0022)\u0060 can never decide this \u0060||\u0060 \u2014 every value satisfying \u0060actual.contains(\u0022HIGH\u0022)\u0060 also satisfies \u0060actual.contains(\u0022HIG\u0022)\u0060, so the \u0060||\u0060 chain is already decided by the latter. The expression is equivalent to the chain without it, which means it is wider than it reads. Delete the dead operand, or narrow the surviving one if IT is the accident."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"crates/forge_main/src/prompt.rs"},"region":{"startLine":433}}}],"partialFingerprints":{"codehealthFindingId/v1":"ec99c18b7054ebc183f2ab9cf4206432e7cd9f1b118fd65038c8c8ee236abd64"}}],"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-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-1395","guid":"800e09e7-c11a-8654-9fa6-86f398995fed","name":"Dependency on Vulnerable Third-Party Component","shortDescription":{"text":"Dependency on Vulnerable Third-Party Component"},"helpUri":"https://cwe.mitre.org/data/definitions/1395.html"},{"id":"CWE-214","guid":"b10ad120-fb22-1351-9348-aa23b453e815","name":"CWE-214","shortDescription":{"text":"CWE-214"},"helpUri":"https://cwe.mitre.org/data/definitions/214.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-269","guid":"70e1f5f6-81e5-4e5a-ba40-b541c3a346e2","name":"CWE-269","shortDescription":{"text":"CWE-269"},"helpUri":"https://cwe.mitre.org/data/definitions/269.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-506","guid":"401d6455-56e3-0552-9a39-f77461673e3f","name":"CWE-506","shortDescription":{"text":"CWE-506"},"helpUri":"https://cwe.mitre.org/data/definitions/506.html"},{"id":"CWE-522","guid":"71fb233e-ce6a-ae57-9419-ef8373540b09","name":"CWE-522","shortDescription":{"text":"CWE-522"},"helpUri":"https://cwe.mitre.org/data/definitions/522.html"},{"id":"CWE-532","guid":"1cd8877a-76e8-ce54-b40b-779af23364a0","name":"CWE-532","shortDescription":{"text":"CWE-532"},"helpUri":"https://cwe.mitre.org/data/definitions/532.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-937","guid":"16f316ae-415c-b354-a59b-1f7905f756e9","name":"Using Components with Known Vulnerabilities","shortDescription":{"text":"Using Components with Known Vulnerabilities"},"helpUri":"https://cwe.mitre.org/data/definitions/937.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":85,"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."]}]}