{"$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":"D7","name":"Architectural Integrity","shortDescription":{"text":"Architectural Integrity"},"helpUri":"https://codehealth.canine.dev/dimensions/D7"},{"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":"D14","name":"License Compliance","shortDescription":{"text":"License Compliance"},"helpUri":"https://codehealth.canine.dev/dimensions/D14"},{"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":"D25","name":"ADR Conformance","shortDescription":{"text":"ADR Conformance"},"helpUri":"https://codehealth.canine.dev/dimensions/D25"},{"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":"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":"AXB2","name":"Runtime readiness","shortDescription":{"text":"Runtime readiness"},"helpUri":"https://codehealth.canine.dev/dimensions/AXB2"},{"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":"P4","name":"Deployment \u0026 Rollback","shortDescription":{"text":"Deployment \u0026 Rollback"},"helpUri":"https://codehealth.canine.dev/dimensions/P4"},{"id":"P6","name":"Release Hygiene","shortDescription":{"text":"Release Hygiene"},"helpUri":"https://codehealth.canine.dev/dimensions/P6"},{"id":"P7","name":"Outbound HTTP resilience","shortDescription":{"text":"Outbound HTTP resilience"},"helpUri":"https://codehealth.canine.dev/dimensions/P7"},{"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":"Lexer.nextToken (cyclomatic 40): Lexer.nextToken has cyclomatic complexity 40 (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":"libraries/apollo-ast/src/commonMain/kotlin/com/apollographql/apollo/ast/internal/Lexer.kt"},"region":{"startLine":55}}}],"partialFingerprints":{"codehealthFindingId/v1":"88b31a960bebf4912482188d2c8a58421e7f271e0e0ebb7a2e478155071c1c42"}},{"ruleId":"D1","level":"warning","message":{"text":"Lexer.readNumber (cyclomatic 34): Lexer.readNumber has cyclomatic complexity 34 (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":"libraries/apollo-ast/src/commonMain/kotlin/com/apollographql/apollo/ast/internal/Lexer.kt"},"region":{"startLine":429}}}],"partialFingerprints":{"codehealthFindingId/v1":"fb4754a9baecf47f0078fd2ba4c745406627e72c31d5a0d0564899c7a0d01538"}},{"ruleId":"D1","level":"warning","message":{"text":"SchemaValidationScopeKt.validateSchema (cyclomatic 34): SchemaValidationScopeKt.validateSchema has cyclomatic complexity 34 (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":"libraries/apollo-ast/src/commonMain/kotlin/com/apollographql/apollo/ast/internal/SchemaValidationScope.kt"},"region":{"startLine":62}}}],"partialFingerprints":{"codehealthFindingId/v1":"281eb3235393e290f8823bc4d435743e17dd7dd45ffdb5ab1462fb3bcc0adbb4"}},{"ruleId":"D1","level":"warning","message":{"text":"BufferedSourceJsonReader.peekNumber (cyclomatic 31): BufferedSourceJsonReader.peekNumber has cyclomatic complexity 31 (threshold 15). This file\u0027s own header attributes it to another copyright holder, so it is code this repository carries rather than code it wrote: restructuring the body in place forks it from upstream and turns every future re-sync into a manual merge. The performable moves are to leave the body as close to its upstream form as possible and keep it behind a narrow interface of your own, and to re-sync it when upstream changes \u2014 or, if it has already diverged far enough that you maintain it here, adopt it deliberately and then split the body into named stages."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"libraries/apollo-api/src/commonMain/kotlin/com/apollographql/apollo/api/json/BufferedSourceJsonReader.kt"},"region":{"startLine":307}}}],"partialFingerprints":{"codehealthFindingId/v1":"083c3273411a85a4f9b5b3fab487cacbd0d4ee737c0f6cb9ef166bec17678fb4"}},{"ruleId":"D1","level":"warning","message":{"text":"IntrospectionKt.introspectionResolvers (cyclomatic 30): IntrospectionKt.introspectionResolvers has cyclomatic complexity 30 (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":"libraries/apollo-execution/src/commonMain/kotlin/com/apollographql/apollo/execution/internal/introspection.kt"},"region":{"startLine":42}}}],"partialFingerprints":{"codehealthFindingId/v1":"e5a1b4c280dec08f54447ca7db76892c048c1a8f2fe3bec6977ad9945b55bcc8"}},{"ruleId":"D1","level":"warning","message":{"text":"BufferedSourceJsonReader.doPeek (cyclomatic 29): BufferedSourceJsonReader.doPeek has cyclomatic complexity 29 (threshold 15). This file\u0027s own header attributes it to another copyright holder, so it is code this repository carries rather than code it wrote: restructuring the body in place forks it from upstream and turns every future re-sync into a manual merge. The performable moves are to leave the body as close to its upstream form as possible and keep it behind a narrow interface of your own, and to re-sync it when upstream changes \u2014 or, if it has already diverged far enough that you maintain it here, adopt it deliberately and then split the body into named stages."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"libraries/apollo-api/src/commonMain/kotlin/com/apollographql/apollo/api/json/BufferedSourceJsonReader.kt"},"region":{"startLine":145}}}],"partialFingerprints":{"codehealthFindingId/v1":"ca6e567ef2dc8dc73311b69ec22d315a261028a5f78019119c25810ace5ab63a"}},{"ruleId":"D1","level":"warning","message":{"text":"ValidationCommonKt.validateDirectives (cyclomatic 26): ValidationCommonKt.validateDirectives 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":"libraries/apollo-ast/src/commonMain/kotlin/com/apollographql/apollo/ast/internal/ValidationCommon.kt"},"region":{"startLine":124}}}],"partialFingerprints":{"codehealthFindingId/v1":"d5232d63929cdf1e02c3604543a2f305002c1d3bd10f6b96f0cce1c1762e498d"}},{"ruleId":"D1","level":"warning","message":{"text":"ExtensionsMergerKt.areEqual (cyclomatic 25): ExtensionsMergerKt.areEqual 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":"libraries/apollo-ast/src/commonMain/kotlin/com/apollographql/apollo/ast/internal/ExtensionsMerger.kt"},"region":{"startLine":479}}}],"partialFingerprints":{"codehealthFindingId/v1":"98f1398338904ab62b1b0f1fdb06fc7e5b334854f3831a71a0716bc30c789dd7"}},{"ruleId":"D1","level":"warning","message":{"text":"KotlinResolver.adapterInitializer (cyclomatic 22): KotlinResolver.adapterInitializer 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":"libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/kotlin/KotlinResolver.kt"},"region":{"startLine":189}}}],"partialFingerprints":{"codehealthFindingId/v1":"082adec8a747b6089424449836dda6083b786ce7617feeb01733f707735d6770"}},{"ruleId":"D1","level":"warning","message":{"text":"ScalarBuilder.prepare (cyclomatic 22): ScalarBuilder.prepare 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":"libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/kotlin/schema/ScalarBuilder.kt"},"region":{"startLine":34}}}],"partialFingerprints":{"codehealthFindingId/v1":"a7be60d066666f709b2cfbf8e3488ee071a795ee6b66589058b744d472875216"}},{"ruleId":"D1","level":"warning","message":{"text":"DefaultApolloExtension.telemetryUsedOptions (cyclomatic 21): DefaultApolloExtension.telemetryUsedOptions has cyclomatic complexity 21 (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":"libraries/apollo-gradle-plugin/src/main/kotlin/com/apollographql/apollo/gradle/internal/DefaultApolloExtension.kt"},"region":{"startLine":147}}}],"partialFingerprints":{"codehealthFindingId/v1":"cc80a0f19743c3552b491469d8f3b51cc55371fa153a5418daf771fb580c8428"}},{"ruleId":"D1","level":"warning","message":{"text":"ApolloExecutableDocumentTransform.addRequiredFields (cyclomatic 21): ApolloExecutableDocumentTransform.addRequiredFields has cyclomatic complexity 21 (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":"libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/internal/ApolloExecutableDocumentTransform.kt"},"region":{"startLine":125}}}],"partialFingerprints":{"codehealthFindingId/v1":"94de469b0d547713088089b06adb8614db6fadf9033462bee87dd9b48f6f2220"}},{"ruleId":"D1","level":"warning","message":{"text":"BufferedSourceJsonReader.readEscapeCharacter (cyclomatic 21): BufferedSourceJsonReader.readEscapeCharacter has cyclomatic complexity 21 (threshold 15). This file\u0027s own header attributes it to another copyright holder, so it is code this repository carries rather than code it wrote: restructuring the body in place forks it from upstream and turns every future re-sync into a manual merge. The performable moves are to leave the body as close to its upstream form as possible and keep it behind a narrow interface of your own, and to re-sync it when upstream changes \u2014 or, if it has already diverged far enough that you maintain it here, adopt it deliberately and then split the body into named stages."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"libraries/apollo-api/src/commonMain/kotlin/com/apollographql/apollo/api/json/BufferedSourceJsonReader.kt"},"region":{"startLine":824}}}],"partialFingerprints":{"codehealthFindingId/v1":"d540e0a64ee10671833ba4c7fa1e84d510e4f59cc3a08def7a613285460953f4"}},{"ruleId":"D1","level":"warning","message":{"text":"KmpKt.defaultTargets (cyclomatic 21): KmpKt.defaultTargets has cyclomatic complexity 21 (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":"build-logic/src/main/kotlin/kmp.kt"},"region":{"startLine":36}}}],"partialFingerprints":{"codehealthFindingId/v1":"1fb8059c5c5ce17558002d309192bc6cd34ee3bc3685c773cf11a78be8e4159b"}},{"ruleId":"D1","level":"warning","message":{"text":"DefaultApolloExtension.registerService (cyclomatic 20): DefaultApolloExtension.registerService 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":"libraries/apollo-gradle-plugin/src/main/kotlin/com/apollographql/apollo/gradle/internal/DefaultApolloExtension.kt"},"region":{"startLine":407}}}],"partialFingerprints":{"codehealthFindingId/v1":"641146e9007ae6ed687db7d06f370abde84756282bb11b75f390a29ba7e13c49"}},{"ruleId":"D1","level":"warning","message":{"text":"ScalarBuilder.prepare (cyclomatic 20): ScalarBuilder.prepare has cyclomatic complexity 20 (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":"libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/java/schema/ScalarBuilder.kt"},"region":{"startLine":35}}}],"partialFingerprints":{"codehealthFindingId/v1":"b308d4d5738ffef73705f06591b6481979b8f752a123e4b321ac3b18e2911e8e"}},{"ruleId":"D1","level":"warning","message":{"text":"Parser.parseDefinition (cyclomatic 20): Parser.parseDefinition has cyclomatic complexity 20 (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":"libraries/apollo-ast/src/commonMain/kotlin/com/apollographql/apollo/ast/internal/Parser.kt"},"region":{"startLine":824}}}],"partialFingerprints":{"codehealthFindingId/v1":"b9d6d34f793afaf1c514f8a17543553d1541951533fa1e336e8b06d65f7ac676"}},{"ruleId":"D1","level":"warning","message":{"text":"IrDataBuildersKt.buildIrDataBuilders (cyclomatic 19): IrDataBuildersKt.buildIrDataBuilders has cyclomatic complexity 19 (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":"libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/ir/IrDataBuilders.kt"},"region":{"startLine":105}}}],"partialFingerprints":{"codehealthFindingId/v1":"2fb0d8faa0248168a8bab8f5714bc1345e02082aa34b35bdd11c330ff5d57e52"}},{"ruleId":"D1","level":"warning","message":{"text":"MultipartReader.nextPart (cyclomatic 17): MultipartReader.nextPart has cyclomatic complexity 17 (threshold 15). This file\u0027s own header attributes it to another copyright holder, so it is code this repository carries rather than code it wrote: restructuring the body in place forks it from upstream and turns every future re-sync into a manual merge. The performable moves are to leave the body as close to its upstream form as possible and keep it behind a narrow interface of your own, and to re-sync it when upstream changes \u2014 or, if it has already diverged far enough that you maintain it here, adopt it deliberately and then split the body into named stages."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"libraries/apollo-runtime/src/commonMain/kotlin/com/apollographql/apollo/internal/MultipartReader.kt"},"region":{"startLine":88}}}],"partialFingerprints":{"codehealthFindingId/v1":"8dadc5e6976c3ee2fabed83185d3fecec0bb9b841f32659a5d79693ca4b389ca"}},{"ruleId":"D1","level":"warning","message":{"text":"WebSocketNetworkTransport.supervise (cyclomatic 17): WebSocketNetworkTransport.supervise 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":"libraries/apollo-runtime/src/commonMain/kotlin/com/apollographql/apollo/network/ws/WebSocketNetworkTransport.kt"},"region":{"startLine":130}}}],"partialFingerprints":{"codehealthFindingId/v1":"d0bc74be31d209cd5b129ce2fdbb1d7f431eb9e719d4858e81e40d76587f1378"}},{"ruleId":"D1","level":"warning","message":{"text":"OperationContext.completeValueOrThrow (cyclomatic 17): OperationContext.completeValueOrThrow 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":"libraries/apollo-execution/src/commonMain/kotlin/com/apollographql/apollo/execution/internal/OperationContext.kt"},"region":{"startLine":378}}}],"partialFingerprints":{"codehealthFindingId/v1":"a4d941fe31a472c945fcbcf6fbc0d243629c323cbbe93ddcfb6029255cf3c1e8"}},{"ruleId":"D1","level":"warning","message":{"text":"CoerceArgumentValuesKt.coerceInputLiteralToInternal (cyclomatic 17): CoerceArgumentValuesKt.coerceInputLiteralToInternal 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":"libraries/apollo-execution/src/commonMain/kotlin/com/apollographql/apollo/execution/internal/coerceArgumentValues.kt"},"region":{"startLine":122}}}],"partialFingerprints":{"codehealthFindingId/v1":"b772bc5da00083ee919c2396311a34acd2320bf7151bc681f8baeb154c921469"}},{"ruleId":"D1","level":"warning","message":{"text":"ApolloCompiler.buildCodegenSchema (cyclomatic 16): ApolloCompiler.buildCodegenSchema 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":"libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/ApolloCompiler.kt"},"region":{"startLine":99}}}],"partialFingerprints":{"codehealthFindingId/v1":"d5bacbd9222dfdc7d6cf22e97884730894bf89ff5edc88f0f6fa57dd50a9af90"}},{"ruleId":"D1","level":"warning","message":{"text":"IrOperationsBuilder.mergeWith (cyclomatic 16): IrOperationsBuilder.mergeWith 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":"libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/ir/IrOperationsBuilder.kt"},"region":{"startLine":331}}}],"partialFingerprints":{"codehealthFindingId/v1":"92663a882913c4303b064fa2c30800fbc28a8c2020ce7d9d7a5bb4d9751be83a"}},{"ruleId":"D1","level":"warning","message":{"text":"ExecutableValidationScope.areValuesEqual (cyclomatic 16): ExecutableValidationScope.areValuesEqual 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":"libraries/apollo-ast/src/commonMain/kotlin/com/apollographql/apollo/ast/internal/ExecutableValidationScope.kt"},"region":{"startLine":604}}}],"partialFingerprints":{"codehealthFindingId/v1":"a03cacd90f8d548e6d972d277de57d172dbde63161f8d17e484b2414fa19a14e"}},{"ruleId":"D1","level":"warning","message":{"text":"Fields_mergingKt.requireSameOutputTypeShape (cyclomatic 16): Fields_mergingKt.requireSameOutputTypeShape 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":"libraries/apollo-ast/src/commonMain/kotlin/com/apollographql/apollo/ast/internal/fields_merging.kt"},"region":{"startLine":300}}}],"partialFingerprints":{"codehealthFindingId/v1":"52388e2e2c0fa128ed0826e4d6a08880274399443ac801fc667d1e37cf811079"}},{"ruleId":"D2","level":"warning","message":{"text":"Lexer.readNumber (cognitive 69): Lexer.readNumber has cognitive complexity 69 (threshold 15). Drivers by points: if/else 18 (48 pts), match/switch 6 (17 pts), boolean chains 3, loops 1 (nesting depth added 41). 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":"libraries/apollo-ast/src/commonMain/kotlin/com/apollographql/apollo/ast/internal/Lexer.kt"},"region":{"startLine":429}}}],"partialFingerprints":{"codehealthFindingId/v1":"4e2fd07247b7b89064e6b0d10404e9bc01a2a8fbe514d3f5bee7cb710ec037fd"}},{"ruleId":"D2","level":"warning","message":{"text":"SchemaValidationScopeKt.validateSchema (cognitive 46): SchemaValidationScopeKt.validateSchema has cognitive complexity 46 (threshold 15). Drivers by points: if/else 23 (41 pts), match/switch 3, boolean chains 2 (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":"libraries/apollo-ast/src/commonMain/kotlin/com/apollographql/apollo/ast/internal/SchemaValidationScope.kt"},"region":{"startLine":62}}}],"partialFingerprints":{"codehealthFindingId/v1":"d70fceabf9cc4442ffb1cbb697cbfd123cc90f2b429dfca64e191f483561c1e2"}},{"ruleId":"D2","level":"warning","message":{"text":"BufferedSourceJsonReader.peekNumber (cognitive 42): BufferedSourceJsonReader.peekNumber has cognitive complexity 42 (threshold 15). Drivers by points: if/else 11 (26 pts), match/switch 3 (8 pts), boolean chains 7, loops 1 (nesting depth added 20). This file\u0027s own header attributes it to another copyright holder, so it is code this repository carries rather than code it wrote: restructuring the body in place forks it from upstream and turns every future re-sync into a manual merge. The performable moves are to leave the body as close to its upstream form as possible and keep it behind a narrow interface of your own, and to re-sync it when upstream changes \u2014 or, if it has already diverged far enough that you maintain it here, adopt it deliberately and then split the body into named stages."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"libraries/apollo-api/src/commonMain/kotlin/com/apollographql/apollo/api/json/BufferedSourceJsonReader.kt"},"region":{"startLine":307}}}],"partialFingerprints":{"codehealthFindingId/v1":"1d53acd42a80ab2d19e7e937c2ee4ea4c2060db328c64ed6a355266ed259ec04"}},{"ruleId":"D2","level":"warning","message":{"text":"ExtensionsMergerKt.areEqual (cognitive 41): ExtensionsMergerKt.areEqual has cognitive complexity 41 (threshold 15). Drivers by points: if/else 12 (32 pts), loops 2 (8 pts), match/switch 1 (nesting depth added 26). 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":"libraries/apollo-ast/src/commonMain/kotlin/com/apollographql/apollo/ast/internal/ExtensionsMerger.kt"},"region":{"startLine":479}}}],"partialFingerprints":{"codehealthFindingId/v1":"22a2d6070c2f21c4351fb52c7874e713b31e7c16a51aa0ae925bfd46520d33e2"}},{"ruleId":"D2","level":"warning","message":{"text":"WebSocketNetworkTransport.supervise (cognitive 40): WebSocketNetworkTransport.supervise has cognitive complexity 40 (threshold 15). Drivers by points: if/else 9 (26 pts), error handling 2 (8 pts), match/switch 2 (5 pts), loops 1 (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":"libraries/apollo-runtime/src/commonMain/kotlin/com/apollographql/apollo/network/ws/WebSocketNetworkTransport.kt"},"region":{"startLine":130}}}],"partialFingerprints":{"codehealthFindingId/v1":"07f605dc8f32c283f18be4e0ae2d063f6221d62a80d255951220b7364bda1a38"}},{"ruleId":"D2","level":"warning","message":{"text":"Lexer.nextToken (cognitive 36): Lexer.nextToken has cognitive complexity 36 (threshold 15). Drivers by points: if/else 10 (23 pts), loops 2 (6 pts), boolean chains 5, match/switch 1 (2 pts) (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":"libraries/apollo-ast/src/commonMain/kotlin/com/apollographql/apollo/ast/internal/Lexer.kt"},"region":{"startLine":55}}}],"partialFingerprints":{"codehealthFindingId/v1":"0c79a96abab7917db6ddf5e8e3e2a7575bb16979f3a9cde71d63d89a1dfdd0ef"}},{"ruleId":"D2","level":"warning","message":{"text":"IrDataBuildersKt.buildIrDataBuilders (cognitive 32): IrDataBuildersKt.buildIrDataBuilders has cognitive complexity 32 (threshold 15). Drivers by points: if/else 10 (22 pts), match/switch 4 (9 pts), loops 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":"libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/ir/IrDataBuilders.kt"},"region":{"startLine":105}}}],"partialFingerprints":{"codehealthFindingId/v1":"1ebf17dea535879f84f6a4c243d8fba00cad6665bd393cf566727a2d358c9362"}},{"ruleId":"D2","level":"warning","message":{"text":"Fields_mergingKt.requireSameOutputTypeShape (cognitive 30): Fields_mergingKt.requireSameOutputTypeShape has cognitive complexity 30 (threshold 15). Drivers by points: if/else 9 (19 pts), match/switch 2 (6 pts), loops 2 (3 pts), boolean chains 2 (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":"libraries/apollo-ast/src/commonMain/kotlin/com/apollographql/apollo/ast/internal/fields_merging.kt"},"region":{"startLine":300}}}],"partialFingerprints":{"codehealthFindingId/v1":"70788da109890b5f790286f149c21069a3e16db67cfb59d6e0cb661415772813"}},{"ruleId":"D2","level":"warning","message":{"text":"BufferedSourceJsonReader.selectName (cognitive 30): BufferedSourceJsonReader.selectName has cognitive complexity 30 (threshold 15). Drivers by points: if/else 8 (26 pts), loops 2 (4 pts) (nesting depth added 20). This file\u0027s own header attributes it to another copyright holder, so it is code this repository carries rather than code it wrote: restructuring the body in place forks it from upstream and turns every future re-sync into a manual merge. The performable moves are to leave the body as close to its upstream form as possible and keep it behind a narrow interface of your own, and to re-sync it when upstream changes \u2014 or, if it has already diverged far enough that you maintain it here, adopt it deliberately and then split the body into named stages."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"libraries/apollo-api/src/commonMain/kotlin/com/apollographql/apollo/api/json/BufferedSourceJsonReader.kt"},"region":{"startLine":708}}}],"partialFingerprints":{"codehealthFindingId/v1":"d48e656ef9261e1c88d0bf4f90a09e5d5b73397a36b2098d800cd3026f631313"}},{"ruleId":"D2","level":"warning","message":{"text":"Fields_mergingKt.collectFields (cognitive 28): Fields_mergingKt.collectFields has cognitive complexity 28 (threshold 15). Drivers by points: if/else 7 (24 pts), match/switch 1 (2 pts), boolean chains 1, loops 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":"libraries/apollo-ast/src/commonMain/kotlin/com/apollographql/apollo/ast/internal/fields_merging.kt"},"region":{"startLine":101}}}],"partialFingerprints":{"codehealthFindingId/v1":"266746fed4c5e3751abbfd864bf46169fd74b0191b0cdae9344c420d169ad88e"}},{"ruleId":"D2","level":"warning","message":{"text":"BufferedSourceJsonReader.doPeek (cognitive 27): BufferedSourceJsonReader.doPeek has cognitive complexity 27 (threshold 15). Drivers by points: if/else 12 (18 pts), match/switch 5 (9 pts) (nesting depth added 10). This file\u0027s own header attributes it to another copyright holder, so it is code this repository carries rather than code it wrote: restructuring the body in place forks it from upstream and turns every future re-sync into a manual merge. The performable moves are to leave the body as close to its upstream form as possible and keep it behind a narrow interface of your own, and to re-sync it when upstream changes \u2014 or, if it has already diverged far enough that you maintain it here, adopt it deliberately and then split the body into named stages."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"libraries/apollo-api/src/commonMain/kotlin/com/apollographql/apollo/api/json/BufferedSourceJsonReader.kt"},"region":{"startLine":145}}}],"partialFingerprints":{"codehealthFindingId/v1":"1a8dbd25e72c9ca9186863ad51abd13b5107dd6b15b34e6ff8507cbae7aabd62"}},{"ruleId":"D2","level":"warning","message":{"text":"DefaultApolloExtension.registerService (cognitive 25): DefaultApolloExtension.registerService has cognitive complexity 25 (threshold 15). Drivers by points: if/else 17 (23 pts), boolean chains 2 (nesting depth added 6). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"libraries/apollo-gradle-plugin/src/main/kotlin/com/apollographql/apollo/gradle/internal/DefaultApolloExtension.kt"},"region":{"startLine":407}}}],"partialFingerprints":{"codehealthFindingId/v1":"83c262d557770ee53ea46208775d53d13d15bbce18b78c7cd6940041ea75cd93"}},{"ruleId":"D2","level":"warning","message":{"text":"MultipartReader.nextPart (cognitive 23): MultipartReader.nextPart has cognitive complexity 23 (threshold 15). Drivers by points: if/else 7 (17 pts), loops 2 (3 pts), match/switch 1 (2 pts), boolean chains 1 (nesting depth added 12). This file\u0027s own header attributes it to another copyright holder, so it is code this repository carries rather than code it wrote: restructuring the body in place forks it from upstream and turns every future re-sync into a manual merge. The performable moves are to leave the body as close to its upstream form as possible and keep it behind a narrow interface of your own, and to re-sync it when upstream changes \u2014 or, if it has already diverged far enough that you maintain it here, adopt it deliberately and then split the body into named stages."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"libraries/apollo-runtime/src/commonMain/kotlin/com/apollographql/apollo/internal/MultipartReader.kt"},"region":{"startLine":88}}}],"partialFingerprints":{"codehealthFindingId/v1":"26aac08be009afc442d26052ee85dc336c852ac82d183abd647c2f0620ffd586"}},{"ruleId":"D2","level":"warning","message":{"text":"OptionsKt.validate (cognitive 23): OptionsKt.validate has cognitive complexity 23 (threshold 15). Drivers by points: if/else 12 (23 pts) (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":"libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/Options.kt"},"region":{"startLine":635}}}],"partialFingerprints":{"codehealthFindingId/v1":"79866a6ceb1400c4f781ef1be65247e850b3334073159f5db61dda73231d6cb6"}},{"ruleId":"D2","level":"warning","message":{"text":"HttpNetworkTransport.multipleResponses (cognitive 22): HttpNetworkTransport.multipleResponses has cognitive complexity 22 (threshold 15). Drivers by points: if/else 7 (15 pts), match/switch 2 (5 pts), loops 1 (2 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":"libraries/apollo-runtime/src/commonMain/kotlin/com/apollographql/apollo/network/http/HttpNetworkTransport.kt"},"region":{"startLine":195}}}],"partialFingerprints":{"codehealthFindingId/v1":"67e393dde95e973d6b936fb165a3af0e2c1bee4a5b5cc668097d6d530bba78ff"}},{"ruleId":"D2","level":"warning","message":{"text":"DiskLruCache.completeEdit (cognitive 22): DiskLruCache.completeEdit has cognitive complexity 22 (threshold 15). Drivers by points: if/else 8 (16 pts), boolean chains 3, loops 2 (3 pts) (nesting depth added 9). This file\u0027s own header attributes it to another copyright holder, so it is code this repository carries rather than code it wrote: restructuring the body in place forks it from upstream and turns every future re-sync into a manual merge. The performable moves are to leave the body as close to its upstream form as possible and keep it behind a narrow interface of your own, and to re-sync it when upstream changes \u2014 or, if it has already diverged far enough that you maintain it here, adopt it deliberately and then split the body into named stages."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"libraries/apollo-http-cache/src/main/kotlin/com/apollographql/apollo/cache/http/internal/DiskLruCache.kt"},"region":{"startLine":470}}}],"partialFingerprints":{"codehealthFindingId/v1":"4e6339222e4fa76c10d2b7901d2351824d483e02750c460b84b3edeed2d5ff8e"}},{"ruleId":"D2","level":"warning","message":{"text":"IntrospectionKt.introspectionResolvers (cognitive 22): IntrospectionKt.introspectionResolvers has cognitive complexity 22 (threshold 15). Drivers by points: if/else 10, match/switch 6 (7 pts), boolean chains 5 (nesting depth added 1). To reduce it, split the body: this score is breadth rather than depth \u2014 many checks laid out side by side rather than nested inside one another, so inverting conditions into early returns has nothing left to flatten. Group the statements between the checks into named steps and move each step into its own function, so the body reads as a short sequence of named stages."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"libraries/apollo-execution/src/commonMain/kotlin/com/apollographql/apollo/execution/internal/introspection.kt"},"region":{"startLine":42}}}],"partialFingerprints":{"codehealthFindingId/v1":"0b2dedf7b4526ac4aa560c9bc8599a50dcbc942edb4e7cd481dd8b7d750f4541"}},{"ruleId":"D2","level":"warning","message":{"text":"IrOperationsBuilder.mergeWith (cognitive 22): IrOperationsBuilder.mergeWith has cognitive complexity 22 (threshold 15). Drivers by points: if/else 8 (15 pts), boolean chains 7 (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":"libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/ir/IrOperationsBuilder.kt"},"region":{"startLine":331}}}],"partialFingerprints":{"codehealthFindingId/v1":"7649983e40629029d1a0785f036c65aa1c4d821f4ea3060bd7e8b30bc67766a0"}},{"ruleId":"D2","level":"warning","message":{"text":"CoerceArgumentValuesKt.coerceInputLiteralToInternal (cognitive 21): CoerceArgumentValuesKt.coerceInputLiteralToInternal has cognitive complexity 21 (threshold 15). Drivers by points: if/else 7 (18 pts), match/switch 2 (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":"libraries/apollo-execution/src/commonMain/kotlin/com/apollographql/apollo/execution/internal/coerceArgumentValues.kt"},"region":{"startLine":122}}}],"partialFingerprints":{"codehealthFindingId/v1":"9906798b07128cb954c6e3a04775e14fa77a40345658ce402874a69ae3322071"}},{"ruleId":"D2","level":"warning","message":{"text":"DefaultApolloExtension.telemetryUsedOptions (cognitive 20): DefaultApolloExtension.telemetryUsedOptions has cognitive complexity 20 (threshold 15). Drivers by points: if/else 20. 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":"libraries/apollo-gradle-plugin/src/main/kotlin/com/apollographql/apollo/gradle/internal/DefaultApolloExtension.kt"},"region":{"startLine":147}}}],"partialFingerprints":{"codehealthFindingId/v1":"b046a5c7a8289fd9d10233cd5402a0d35e46c8488c6cfd37a11e7656f4dad5ec"}},{"ruleId":"D2","level":"warning","message":{"text":"ApolloCompiler.buildCodegenSchema (cognitive 18): ApolloCompiler.buildCodegenSchema has cognitive complexity 18 (threshold 15). Drivers by points: if/else 11 (14 pts), boolean chains 4 (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."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/ApolloCompiler.kt"},"region":{"startLine":99}}}],"partialFingerprints":{"codehealthFindingId/v1":"863b41e67ccd4a3e5a21a8d58e3e835ce72ac3460869577fc89f2fea326b438f"}},{"ruleId":"D2","level":"warning","message":{"text":"ApolloExecutableDocumentTransform.addRequiredFields (cognitive 18): ApolloExecutableDocumentTransform.addRequiredFields has cognitive complexity 18 (threshold 15). Drivers by points: if/else 9 (11 pts), boolean chains 5, match/switch 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":"libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/internal/ApolloExecutableDocumentTransform.kt"},"region":{"startLine":125}}}],"partialFingerprints":{"codehealthFindingId/v1":"6cd7b03a34a1f10e7e344034cd867172f3d30d5b1e6f0711ae869a2cebeaae54"}},{"ruleId":"D2","level":"warning","message":{"text":"Lexer.readBlockString (cognitive 18): Lexer.readBlockString has cognitive complexity 18 (threshold 15). Drivers by points: if/else 5 (12 pts), boolean chains 3, match/switch 1 (2 pts), loops 1 (nesting depth added 8). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"libraries/apollo-ast/src/commonMain/kotlin/com/apollographql/apollo/ast/internal/Lexer.kt"},"region":{"startLine":340}}}],"partialFingerprints":{"codehealthFindingId/v1":"0e06520473244bca27b9bb7f3e53ff65d92ff089283be6fb14afae9141c89e29"}},{"ruleId":"D2","level":"warning","message":{"text":"OperationContext.completeValueOrThrow (cognitive 17): OperationContext.completeValueOrThrow has cognitive complexity 17 (threshold 15). Drivers by points: if/else 9 (15 pts), boolean chains 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":"libraries/apollo-execution/src/commonMain/kotlin/com/apollographql/apollo/execution/internal/OperationContext.kt"},"region":{"startLine":378}}}],"partialFingerprints":{"codehealthFindingId/v1":"29e1a8b92711dcd537443a08fa42dd64a142d360ccfffce014cafd328000286d"}},{"ruleId":"D2","level":"warning","message":{"text":"DefaultHttpRequestComposer.compose (cognitive 17): DefaultHttpRequestComposer.compose has cognitive complexity 17 (threshold 15). Drivers by points: if/else 8 (14 pts), boolean chains 2, 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":"libraries/apollo-api/src/commonMain/kotlin/com/apollographql/apollo/api/http/DefaultHttpRequestComposer.kt"},"region":{"startLine":50}}}],"partialFingerprints":{"codehealthFindingId/v1":"1ce6e219f25a92f4ef2b9aedd21ba20c9701cf8a98de16043545b2d7b96b0e51"}},{"ruleId":"D2","level":"warning","message":{"text":"NormalizedCache.prettifyDump (cognitive 16): NormalizedCache.prettifyDump has cognitive complexity 16 (threshold 15). Drivers by points: loops 4 (11 pts), match/switch 1 (4 pts), if/else 1 (nesting depth added 10). 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":"libraries/apollo-normalized-cache-api/src/commonMain/kotlin/com/apollographql/apollo/cache/normalized/api/NormalizedCache.kt"},"region":{"startLine":85}}}],"partialFingerprints":{"codehealthFindingId/v1":"a8825a3eb19df5a9177e27ed974bb1dda0d9e1e18fa20d46fc74a679c440d975"}},{"ruleId":"D2","level":"warning","message":{"text":"CoerceArgumentValuesKt.coerceArgumentValues (cognitive 16): CoerceArgumentValuesKt.coerceArgumentValues has cognitive complexity 16 (threshold 15). Drivers by points: if/else 9 (15 pts), boolean chains 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":"libraries/apollo-execution/src/commonMain/kotlin/com/apollographql/apollo/execution/internal/coerceArgumentValues.kt"},"region":{"startLine":60}}}],"partialFingerprints":{"codehealthFindingId/v1":"f4ae2436b8802f3a93189db5fa544f409b55716f382e6769b6f42c726c18965f"}},{"ruleId":"D2","level":"warning","message":{"text":"AdapterCommonKt.readFromResponseCodeBlock (cognitive 16): AdapterCommonKt.readFromResponseCodeBlock has cognitive complexity 16 (threshold 15). Drivers by points: if/else 9 (12 pts), boolean chains 3, match/switch 1 (nesting depth added 3). 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":"libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/kotlin/helpers/AdapterCommon.kt"},"region":{"startLine":37}}}],"partialFingerprints":{"codehealthFindingId/v1":"a773c096b66a7eae60926dd514f7ef19c7939a60c571e57b582ab06e5176bf14"}},{"ruleId":"D2","level":"warning","message":{"text":"ScalarBuilder.prepare (cognitive 16): ScalarBuilder.prepare has cognitive complexity 16 (threshold 15). Drivers by points: if/else 6 (9 pts), match/switch 4 (7 pts) (nesting depth added 6). 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":"libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/kotlin/schema/ScalarBuilder.kt"},"region":{"startLine":34}}}],"partialFingerprints":{"codehealthFindingId/v1":"9d91c4af5545695ad8bdd4b0c1c85fbde65ceda2fc4b28da866622b4eac33c92"}},{"ruleId":"D2","level":"warning","message":{"text":"IrOperationsBuilder.merge (cognitive 16): IrOperationsBuilder.merge has cognitive complexity 16 (threshold 15). Drivers by points: if/else 11 (14 pts), loops 2 (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."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/ir/IrOperationsBuilder.kt"},"region":{"startLine":466}}}],"partialFingerprints":{"codehealthFindingId/v1":"e1511e9716cd0286f48eb7affa3d96b2a46fe88fe6cc5b1b3a29bcd1580ed16a"}},{"ruleId":"D2","level":"warning","message":{"text":"SchemaValidationScopeKt.getLinkedSchemas (cognitive 16): SchemaValidationScopeKt.getLinkedSchemas has cognitive complexity 16 (threshold 15). Drivers by points: if/else 7 (14 pts), boolean chains 2 (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":"libraries/apollo-ast/src/commonMain/kotlin/com/apollographql/apollo/ast/internal/SchemaValidationScope.kt"},"region":{"startLine":402}}}],"partialFingerprints":{"codehealthFindingId/v1":"88004bc050949c3b3442729a43f3afc6c5a6c89c9c96d3a8899569442eab7e94"}},{"ruleId":"D3","level":"warning","message":{"text":"FileTooLong: ast/gql.kt: FileTooLong \u2014 1817 significant lines (blank, comment-only and punctuation-only lines excluded). The bar is 500 significant lines; this is 1317 over it, 3.63\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":"libraries/apollo-ast/src/commonMain/kotlin/com/apollographql/apollo/ast/gql.kt"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"14968ec10e269c12b9920b9c7f2b8f4663c8963b12c71103487bf67c8a55aa0a"}},{"ruleId":"D3","level":"warning","message":{"text":"MethodTooLong: DefaultApolloExtension.registerService: MethodTooLong \u2014 registerService runs 285 significant lines (blank, comment-only and punctuation-only lines excluded) in one body. The bar is 100 significant lines; this is 185 over it, 2.85\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":"libraries/apollo-gradle-plugin/src/main/kotlin/com/apollographql/apollo/gradle/internal/DefaultApolloExtension.kt"},"region":{"startLine":407}}}],"partialFingerprints":{"codehealthFindingId/v1":"7ca2bc66cbef5c9c654c1daef68e77b866082c749bbe3235437af51518999ff8"}},{"ruleId":"D3","level":"warning","message":{"text":"TooManyMethods: Parser: TooManyMethods \u2014 80 methods. The bar is 30 methods; this is 50 over it, 2.67\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":"libraries/apollo-ast/src/commonMain/kotlin/com/apollographql/apollo/ast/internal/Parser.kt"},"region":{"startLine":6}}}],"partialFingerprints":{"codehealthFindingId/v1":"a0857222bfbd2bd7b1561a9ebc15c383a1a5200075d77a163dfb919d5bfc8791"}},{"ruleId":"D3","level":"warning","message":{"text":"ClassTooLong: Parser: ClassTooLong \u2014 782 significant lines (blank, comment-only and punctuation-only lines excluded), 80 methods. The bar is 400 significant lines; this is 382 over it, 1.96\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":"libraries/apollo-ast/src/commonMain/kotlin/com/apollographql/apollo/ast/internal/Parser.kt"},"region":{"startLine":6}}}],"partialFingerprints":{"codehealthFindingId/v1":"a262d83c358f5b8276934ac2d096cf780ba098ad0e03b76d1b6dcd2573b2f3c6"}},{"ruleId":"D3","level":"warning","message":{"text":"MethodTooLong: IntrospectionKt.introspectionResolvers: MethodTooLong \u2014 introspectionResolvers runs 184 significant lines (blank, comment-only and punctuation-only lines excluded) in one body. The bar is 100 significant lines; this is 84 over it, 1.84\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":"libraries/apollo-execution/src/commonMain/kotlin/com/apollographql/apollo/execution/internal/introspection.kt"},"region":{"startLine":42}}}],"partialFingerprints":{"codehealthFindingId/v1":"3f73fb011548ca4fcebc568db54774b6abf5078026774aa392e9cb7612190bfb"}},{"ruleId":"D3","level":"warning","message":{"text":"FileTooLong: internal/Parser.kt: FileTooLong \u2014 785 significant lines (blank, comment-only and punctuation-only lines excluded), about 100% of them inside a single declaration: Parser (6-1119). The bar is 500 significant lines; this is 285 over it, 1.57\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":"libraries/apollo-ast/src/commonMain/kotlin/com/apollographql/apollo/ast/internal/Parser.kt"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"96d9d956d89b5bc39739f38594cc0e4b7db4c19ba7ee571736e32b2005dca793"}},{"ruleId":"D3","level":"warning","message":{"text":"TooManyMethods: Builder: TooManyMethods \u2014 44 methods. The bar is 30 methods; this is 14 over it, 1.47\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":"libraries/apollo-runtime/src/commonMain/kotlin/com/apollographql/apollo/ApolloClient.kt"},"region":{"startLine":355}}}],"partialFingerprints":{"codehealthFindingId/v1":"66be31792c2714f37af1313b653cafd3c0c2a8b66a98733d9c61fea01b0f589c"}},{"ruleId":"D3","level":"warning","message":{"text":"ClassTooLong: DefaultApolloExtension: ClassTooLong \u2014 581 significant lines (blank, comment-only and punctuation-only lines excluded), 16 methods. The bar is 400 significant lines; this is 181 over it, 1.45\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":"libraries/apollo-gradle-plugin/src/main/kotlin/com/apollographql/apollo/gradle/internal/DefaultApolloExtension.kt"},"region":{"startLine":57}}}],"partialFingerprints":{"codehealthFindingId/v1":"1f3d199a87a0f1324455efa416301a708e8337c8fdcad822dc23f353b0d4574f"}},{"ruleId":"D3","level":"warning","message":{"text":"MethodTooLong: SchemaValidationScopeKt.validateSchema: MethodTooLong \u2014 validateSchema runs 144 significant lines (blank, comment-only and punctuation-only lines excluded) in one body. The bar is 100 significant lines; this is 44 over it, 1.44\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":"libraries/apollo-ast/src/commonMain/kotlin/com/apollographql/apollo/ast/internal/SchemaValidationScope.kt"},"region":{"startLine":62}}}],"partialFingerprints":{"codehealthFindingId/v1":"ff320129f4f9d9241c72e115f35a7516a33bfeab583a154e286ab4f76a499454"}},{"ruleId":"D3","level":"warning","message":{"text":"ClassTooLong: BufferedSourceJsonReader: ClassTooLong \u2014 567 significant lines (blank, comment-only and punctuation-only lines excluded), 36 methods. The bar is 400 significant lines; this is 167 over it, 1.42\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":"libraries/apollo-api/src/commonMain/kotlin/com/apollographql/apollo/api/json/BufferedSourceJsonReader.kt"},"region":{"startLine":35}}}],"partialFingerprints":{"codehealthFindingId/v1":"4de10b924fb084b78f9b8788e2dd4acd8f594dc4a9516e52c9dc68aa9a34e2db"}},{"ruleId":"D3","level":"warning","message":{"text":"FileTooLong: internal/SchemaValidationScope.kt: FileTooLong \u2014 696 significant lines (blank, comment-only and punctuation-only lines excluded). The bar is 500 significant lines; this is 196 over it, 1.39\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":"libraries/apollo-ast/src/commonMain/kotlin/com/apollographql/apollo/ast/internal/SchemaValidationScope.kt"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"f86d217130ba4c6c5d1fb8dc26b2dcb28af12f78dfc6f45c417361ea633c724a"}},{"ruleId":"D3","level":"warning","message":{"text":"ClassTooLong: DiskLruCache: ClassTooLong \u2014 529 significant lines (blank, comment-only and punctuation-only lines excluded), 25 methods. The bar is 400 significant lines; this is 129 over it, 1.32\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":"libraries/apollo-http-cache/src/main/kotlin/com/apollographql/apollo/cache/http/internal/DiskLruCache.kt"},"region":{"startLine":87}}}],"partialFingerprints":{"codehealthFindingId/v1":"90e9f9c58278dffbba596ec1c26075193c15209897d05d863cc8b0d526c4db79"}},{"ruleId":"D3","level":"warning","message":{"text":"FileTooLong: internal/DefaultApolloExtension.kt: FileTooLong \u2014 650 significant lines (blank, comment-only and punctuation-only lines excluded), about 89% of them inside a single declaration: DefaultApolloExtension (57-988). The bar is 500 significant lines; this is 150 over it, 1.30\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":"libraries/apollo-gradle-plugin/src/main/kotlin/com/apollographql/apollo/gradle/internal/DefaultApolloExtension.kt"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"2fb6b662cb188972285b16c00e9d2c0caed8000903d3993c55b5e035dd592bc4"}},{"ruleId":"D3","level":"warning","message":{"text":"TooManyMethods: DefaultService: TooManyMethods \u2014 37 methods. The bar is 30 methods; this is 7 over it, 1.23\u00D7 the bar. Most of these members implement api.Service (31 of 34 members), so moving them onto a smaller type would remove them from that contract rather than reduce it. To reduce it, split the contract instead: give each cohesive group of operations its own smaller interface and its own implementing type. Where the contract has to stay whole, move the work behind these members into collaborator types, so what is left here is a forward per member rather than a responsibility per member."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"libraries/apollo-gradle-plugin/src/main/kotlin/com/apollographql/apollo/gradle/internal/DefaultService.kt"},"region":{"startLine":21}}}],"partialFingerprints":{"codehealthFindingId/v1":"a671bdf1eb532c204aa831550d8b9cce0bbfe1b01c8bd15ed293fe64455c9914"}},{"ruleId":"D3","level":"warning","message":{"text":"MethodTooLong: OperationBasedModelGroupBuilder.buildField: MethodTooLong \u2014 buildField runs 121 significant lines (blank, comment-only and punctuation-only lines excluded) in one body. The bar is 100 significant lines; this is 21 over it, 1.21\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":"libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/ir/OperationBasedModelGroupBuilder.kt"},"region":{"startLine":124}}}],"partialFingerprints":{"codehealthFindingId/v1":"ebe456a10bb2383f1185c72ef65ee804876e1e711c322db22308df55add25621"}},{"ruleId":"D3","level":"warning","message":{"text":"TooManyMethods: BufferedSourceJsonReader: TooManyMethods \u2014 36 methods. The bar is 30 methods; this is 6 over it, 1.20\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":"libraries/apollo-api/src/commonMain/kotlin/com/apollographql/apollo/api/json/BufferedSourceJsonReader.kt"},"region":{"startLine":35}}}],"partialFingerprints":{"codehealthFindingId/v1":"4e782b6a8c95edddba14059fb2d47d45f24ff96aa6951bc59e8aca21e85ea526"}},{"ruleId":"D3","level":"warning","message":{"text":"FileTooLong: json/BufferedSourceJsonReader.kt: FileTooLong \u2014 576 significant lines (blank, comment-only and punctuation-only lines excluded), about 98% of them inside a single declaration: BufferedSourceJsonReader (35-911). The bar is 500 significant lines; this is 76 over it, 1.15\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":"libraries/apollo-api/src/commonMain/kotlin/com/apollographql/apollo/api/json/BufferedSourceJsonReader.kt"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"9c63f72cec4b7d682a2ff19d0e2e1467f0f6b8c65b0b35da73fbada85a3a0d20"}},{"ruleId":"D3","level":"warning","message":{"text":"FileTooLong: internal/DiskLruCache.kt: FileTooLong \u2014 560 significant lines (blank, comment-only and punctuation-only lines excluded), about 94% of them inside a single declaration: DiskLruCache (87-1014). The bar is 500 significant lines; this is 60 over it, 1.12\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":"libraries/apollo-http-cache/src/main/kotlin/com/apollographql/apollo/cache/http/internal/DiskLruCache.kt"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"3f7529146e03d8be76244a56ed7f7879af102ab9b8e931f267205ea1cd3d9504"}},{"ruleId":"D3","level":"warning","message":{"text":"FileTooLong: internal/definitions.kt: FileTooLong \u2014 552 significant lines (blank, comment-only and punctuation-only lines excluded). The bar is 500 significant lines; this is 52 over it, 1.10\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":"libraries/apollo-ast/src/commonMain/kotlin/com/apollographql/apollo/ast/internal/definitions.kt"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"c49df7752fb2146f556f332d3771f1c5899e358c60ce6d336f61e29f0df8eea4"}},{"ruleId":"D3","level":"warning","message":{"text":"MethodTooLong: Lexer.readNumber: MethodTooLong \u2014 readNumber runs 101 significant lines (blank, comment-only and punctuation-only lines excluded) in one body. The bar is 100 significant lines; this is 1 over it, 1.01\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":"libraries/apollo-ast/src/commonMain/kotlin/com/apollographql/apollo/ast/internal/Lexer.kt"},"region":{"startLine":429}}}],"partialFingerprints":{"codehealthFindingId/v1":"1a86e184396c57b9a35a5a020f1d0a984c79918a96be9692399bad800d6d986b"}},{"ruleId":"D3","level":"warning","message":{"text":"MethodTooLong: OperationBasedWithInterfacesModelGroupBuilder.buildNode: MethodTooLong \u2014 buildNode runs 101 significant lines (blank, comment-only and punctuation-only lines excluded) in one body. The bar is 100 significant lines; this is 1 over it, 1.01\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":"libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/ir/OperationBasedWithInterfacesModelGroupBuilder.kt"},"region":{"startLine":125}}}],"partialFingerprints":{"codehealthFindingId/v1":"206e1878dec8a358a6220bc7d7bb2a0001a1c24e158ed91731ca07726358823b"}},{"ruleId":"D4","level":"warning","message":{"text":"Near-duplicate member pair (42 shared lines): libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/java/helpers/AdapterCommon.kt:65-233 | libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/kotlin/helpers/AdapterCommon.kt:41-198 \u2014 These two members are variants of one another: 42 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":"libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/java/helpers/AdapterCommon.kt"},"region":{"startLine":65}}}],"partialFingerprints":{"codehealthFindingId/v1":"cde7404e85d6b12cebe3c8e8054e8c30097669a4cdfd5e231e7c8bd41de94503"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (21 lines \u00D7 2): libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/ir/buckets.kt:36-56 | libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/ir/shapes.kt:59-79 \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. Read the line range as the matched WINDOW rather than a finished unit: at \u0060libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/ir/buckets.kt:36\u0060 it runs out through the closing brace of the declaration holding it and carries on into the declaration that follows \u2014 the window is the tail of one member plus the head of the next, so no call can be substituted for those exact lines, and the smallest declaration that contains all of them is the type they sit in. The repeated unit is the member each site sits in: where those members\u0027 bodies are the same, move one whole member to the shared location and have the others delegate to it; where the copies are a run of near-identical overloads or wrappers that differ only in their signatures, the repetition IS the run \u2014 a one-line delegation has no helper inside it to lift \u2014 so generate the run from the set it enumerates, or accept it and keep each member\u0027s own documentation with it."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/ir/buckets.kt"},"region":{"startLine":36}}}],"partialFingerprints":{"codehealthFindingId/v1":"2e97475e0bf27667a926b4a37d913d43d021ddb419999f85312e0bac2b32fdca"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (21 lines \u00D7 2): libraries/apollo-api/src/commonMain/kotlin/com/apollographql/apollo/api/json/MapJsonReader.kt:227-247 | libraries/apollo-api/src/jsMain/kotlin/com/apollographql/apollo/api/json/DynamicJsJsonReader.kt:271-291 \u2014 before extracting anything, compare \u0060libraries/apollo-api/src/commonMain/kotlin/com/apollographql/apollo/api/json/MapJsonReader.kt\u0060 and \u0060libraries/apollo-api/src/jsMain/kotlin/com/apollographql/apollo/api/json/DynamicJsJsonReader.kt\u0060 as WHOLE FILES: this scan already matched 12 separate duplicated blocks between them, totalling at least 168 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. Read the line range as the matched WINDOW rather than a finished unit: at \u0060libraries/apollo-api/src/commonMain/kotlin/com/apollographql/apollo/api/json/MapJsonReader.kt:227\u0060 it begins part-way through the construct above it, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"libraries/apollo-api/src/commonMain/kotlin/com/apollographql/apollo/api/json/MapJsonReader.kt"},"region":{"startLine":227}}}],"partialFingerprints":{"codehealthFindingId/v1":"fb9ad041f3e27ac85be9d0e1e4d30e8526e30c7023458373ae152e8a53a4f9ea"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (21 lines \u00D7 2): libraries/apollo-api/src/commonMain/kotlin/com/apollographql/apollo/api/json/MapJsonReader.kt:250-270 | libraries/apollo-api/src/jsMain/kotlin/com/apollographql/apollo/api/json/DynamicJsJsonReader.kt:294-314 \u2014 before extracting anything, compare \u0060libraries/apollo-api/src/commonMain/kotlin/com/apollographql/apollo/api/json/MapJsonReader.kt\u0060 and \u0060libraries/apollo-api/src/jsMain/kotlin/com/apollographql/apollo/api/json/DynamicJsJsonReader.kt\u0060 as WHOLE FILES: this scan already matched 12 separate duplicated blocks between them, totalling at least 168 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. Read the line range as the matched WINDOW rather than a finished unit: at \u0060libraries/apollo-api/src/commonMain/kotlin/com/apollographql/apollo/api/json/MapJsonReader.kt:250\u0060 it begins part-way through the construct above it, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"libraries/apollo-api/src/commonMain/kotlin/com/apollographql/apollo/api/json/MapJsonReader.kt"},"region":{"startLine":250}}}],"partialFingerprints":{"codehealthFindingId/v1":"57adb463ac53bf94220ea0402912756d5a1e480f1276dfb1a5931b3bde96d6fa"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (21 lines \u00D7 2): libraries/apollo-api/src/commonMain/kotlin/com/apollographql/apollo/api/json/MapJsonReader.kt:273-293 | libraries/apollo-api/src/jsMain/kotlin/com/apollographql/apollo/api/json/DynamicJsJsonReader.kt:317-337 \u2014 before extracting anything, compare \u0060libraries/apollo-api/src/commonMain/kotlin/com/apollographql/apollo/api/json/MapJsonReader.kt\u0060 and \u0060libraries/apollo-api/src/jsMain/kotlin/com/apollographql/apollo/api/json/DynamicJsJsonReader.kt\u0060 as WHOLE FILES: this scan already matched 12 separate duplicated blocks between them, totalling at least 168 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. Read the line range as the matched WINDOW rather than a finished unit: at \u0060libraries/apollo-api/src/commonMain/kotlin/com/apollographql/apollo/api/json/MapJsonReader.kt:273\u0060 it begins part-way through the construct above it, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"libraries/apollo-api/src/commonMain/kotlin/com/apollographql/apollo/api/json/MapJsonReader.kt"},"region":{"startLine":273}}}],"partialFingerprints":{"codehealthFindingId/v1":"0e018147428c974b3d87bad5d9bb9007495edbd4c1aa304cffcc1aebd659396e"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (21 lines \u00D7 2): libraries/apollo-api/src/commonMain/kotlin/com/apollographql/apollo/api/json/MapJsonReader.kt:361-381 | libraries/apollo-api/src/jsMain/kotlin/com/apollographql/apollo/api/json/DynamicJsJsonReader.kt:376-396 \u2014 before extracting anything, compare \u0060libraries/apollo-api/src/commonMain/kotlin/com/apollographql/apollo/api/json/MapJsonReader.kt\u0060 and \u0060libraries/apollo-api/src/jsMain/kotlin/com/apollographql/apollo/api/json/DynamicJsJsonReader.kt\u0060 as WHOLE FILES: this scan already matched 12 separate duplicated blocks between them, totalling at least 168 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. Each matched range is the entire body of the declaration above it, so the region is already a complete unit: move that whole declaration to the shared location and have each site call it, rather than lifting the lines out of their bodies. Any \u0060return\u0060 inside it is the body\u0027s own exit and keeps its meaning in the moved unit."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"libraries/apollo-api/src/commonMain/kotlin/com/apollographql/apollo/api/json/MapJsonReader.kt"},"region":{"startLine":361}}}],"partialFingerprints":{"codehealthFindingId/v1":"46fe884c63eae4a625d63f09bfea3e9a683dc8d9a63ebc9e55531eb8c080df08"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (20 lines \u00D7 2): libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/java/schema/EnumAsClassBuilder.kt:80-99 | libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/java/schema/EnumAsEnumBuilder.kt:79-98 \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. Read the line range as the matched WINDOW rather than a finished unit: at \u0060libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/java/schema/EnumAsClassBuilder.kt:80\u0060 it begins part-way through the construct above it, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/java/schema/EnumAsClassBuilder.kt"},"region":{"startLine":80}}}],"partialFingerprints":{"codehealthFindingId/v1":"e3c721817878a4c15c16c10af8ad13637f6e93dfb68ca9f1864bfc39b706d562"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (19 lines \u00D7 2): libraries/apollo-execution/src/commonMain/kotlin/com/apollographql/apollo/execution/internal/LruCache.kt:93-111 | libraries/apollo-normalized-cache-api/src/commonMain/kotlin/com/apollographql/apollo/cache/normalized/api/internal/LruCache.kt:93-111 \u2014 before extracting anything, compare \u0060libraries/apollo-execution/src/commonMain/kotlin/com/apollographql/apollo/execution/internal/LruCache.kt\u0060 and \u0060libraries/apollo-normalized-cache-api/src/commonMain/kotlin/com/apollographql/apollo/cache/normalized/api/internal/LruCache.kt\u0060 as WHOLE FILES: this scan already matched 6 separate duplicated blocks between them, totalling at least 83 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. Each matched range is the entire body of the declaration above it, so the region is already a complete unit: move that whole declaration to the shared location and have each site call it, rather than lifting the lines out of their bodies. Any \u0060return\u0060 inside it is the body\u0027s own exit and keeps its meaning in the moved unit."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"libraries/apollo-execution/src/commonMain/kotlin/com/apollographql/apollo/execution/internal/LruCache.kt"},"region":{"startLine":93}}}],"partialFingerprints":{"codehealthFindingId/v1":"180861423a76800850be3262f652c800c9bf22eb6de1a54b3579eccaa3a37a41"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (19 lines \u00D7 2): libraries/apollo-execution/src/commonMain/kotlin/com/apollographql/apollo/execution/internal/LruCache.kt:134-152 | libraries/apollo-normalized-cache-api/src/commonMain/kotlin/com/apollographql/apollo/cache/normalized/api/internal/LruCache.kt:134-152 \u2014 before extracting anything, compare \u0060libraries/apollo-execution/src/commonMain/kotlin/com/apollographql/apollo/execution/internal/LruCache.kt\u0060 and \u0060libraries/apollo-normalized-cache-api/src/commonMain/kotlin/com/apollographql/apollo/cache/normalized/api/internal/LruCache.kt\u0060 as WHOLE FILES: this scan already matched 6 separate duplicated blocks between them, totalling at least 83 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. Each matched range is the entire body of the declaration above it, so the region is already a complete unit: move that whole declaration to the shared location and have each site call it, rather than lifting the lines out of their bodies. Any \u0060return\u0060 inside it is the body\u0027s own exit and keeps its meaning in the moved unit."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"libraries/apollo-execution/src/commonMain/kotlin/com/apollographql/apollo/execution/internal/LruCache.kt"},"region":{"startLine":134}}}],"partialFingerprints":{"codehealthFindingId/v1":"9e90fc4ad4f4c975344c0c2f1f5f5a4d2e882ae8ab9b3675457c49dcc474d1d7"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (18\u201319 lines \u00D7 2): libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/java/helpers/AdapterCommon.kt:114-131 | libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/kotlin/helpers/AdapterCommon.kt:88-106 \u2014 before extracting anything, compare \u0060libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/java/helpers/AdapterCommon.kt\u0060 and \u0060libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/kotlin/helpers/AdapterCommon.kt\u0060 as WHOLE FILES: this scan already matched 3 separate duplicated blocks between them, totalling at least 43 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. Read the line range as the matched WINDOW rather than a finished unit: at \u0060libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/java/helpers/AdapterCommon.kt:114\u0060 it begins part-way through the construct above it, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that. Note first that the copies are not typed on the same thing: \u0060CodeBlock\u0060 names \u0060com.squareup.javapoet.CodeBlock\u0060 in one and \u0060com.squareup.kotlinpoet.CodeBlock\u0060 in another \u2014 different types that share a simple name, which is why the text matched. A single extracted unit cannot be given a parameter type that fits both, so unifying those types (or introducing a shared abstraction over them) is the step that has to come BEFORE the extraction above; if they are deliberately separate, the duplication is the price of that separation and the honest resolution is to record the decision rather than to extract."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/java/helpers/AdapterCommon.kt"},"region":{"startLine":114}}}],"partialFingerprints":{"codehealthFindingId/v1":"adcd2d76059e023bd5639dcde694f29cae27a18ce5b84edc0749d6b47617c50b"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (19 lines \u00D7 2): libraries/apollo-api/src/commonMain/kotlin/com/apollographql/apollo/api/json/MapJsonReader.kt:296-314 | libraries/apollo-api/src/jsMain/kotlin/com/apollographql/apollo/api/json/DynamicJsJsonReader.kt:340-358 \u2014 before extracting anything, compare \u0060libraries/apollo-api/src/commonMain/kotlin/com/apollographql/apollo/api/json/MapJsonReader.kt\u0060 and \u0060libraries/apollo-api/src/jsMain/kotlin/com/apollographql/apollo/api/json/DynamicJsJsonReader.kt\u0060 as WHOLE FILES: this scan already matched 12 separate duplicated blocks between them, totalling at least 168 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. Read the line range as the matched WINDOW rather than a finished unit: at \u0060libraries/apollo-api/src/commonMain/kotlin/com/apollographql/apollo/api/json/MapJsonReader.kt:296\u0060 it begins part-way through the construct above it, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"libraries/apollo-api/src/commonMain/kotlin/com/apollographql/apollo/api/json/MapJsonReader.kt"},"region":{"startLine":296}}}],"partialFingerprints":{"codehealthFindingId/v1":"3c9be2c34ad8b615a857694088bae8c5790a29ff17f7b1d097ec09f08957076e"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (18 lines \u00D7 3): libraries/apollo-ast/src/commonMain/kotlin/com/apollographql/apollo/ast/gql.kt:675-692 | libraries/apollo-ast/src/commonMain/kotlin/com/apollographql/apollo/ast/gql.kt:1126-1143 | libraries/apollo-ast/src/commonMain/kotlin/com/apollographql/apollo/ast/gql.kt:1266-1283 \u2014 all 3 copies are in the same file, so extract the block into one function there and call it from every one of those sites \u2014 resolving only two of them leaves the rest to drift apart the first time one is edited. Read the line range as the matched WINDOW rather than a finished unit: at \u0060libraries/apollo-ast/src/commonMain/kotlin/com/apollographql/apollo/ast/gql.kt:675\u0060 it runs out through the closing brace of the declaration holding it \u2014 the window is that declaration\u0027s tail, not a fragment that begins part-way through something, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"libraries/apollo-ast/src/commonMain/kotlin/com/apollographql/apollo/ast/gql.kt"},"region":{"startLine":675}}}],"partialFingerprints":{"codehealthFindingId/v1":"58e16c2303082ecd441be141c558c5b802f89d7227f2500c2d3452e3ae7ea6dc"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (18 lines \u00D7 2): libraries/apollo-execution/src/commonMain/kotlin/com/apollographql/apollo/execution/internal/LruCache.kt:114-131 | libraries/apollo-normalized-cache-api/src/commonMain/kotlin/com/apollographql/apollo/cache/normalized/api/internal/LruCache.kt:114-131 \u2014 before extracting anything, compare \u0060libraries/apollo-execution/src/commonMain/kotlin/com/apollographql/apollo/execution/internal/LruCache.kt\u0060 and \u0060libraries/apollo-normalized-cache-api/src/commonMain/kotlin/com/apollographql/apollo/cache/normalized/api/internal/LruCache.kt\u0060 as WHOLE FILES: this scan already matched 6 separate duplicated blocks between them, totalling at least 83 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. Each matched range is the entire body of the declaration above it, so the region is already a complete unit: move that whole declaration to the shared location and have each site call it, rather than lifting the lines out of their bodies. Any \u0060return\u0060 inside it is the body\u0027s own exit and keeps its meaning in the moved unit."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"libraries/apollo-execution/src/commonMain/kotlin/com/apollographql/apollo/execution/internal/LruCache.kt"},"region":{"startLine":114}}}],"partialFingerprints":{"codehealthFindingId/v1":"6831f03720b3d2ee33b4e786c66d897f505360e447b0256e544cbf1882a87476"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (18 lines \u00D7 2): libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/java/JavaCodegen.kt:259-276 | libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/kotlin/KotlinCodegen.kt:296-313 \u2014 the copies span different directories, so extracting a shared function means choosing where it lives: put it somewhere both call sites can already reach \u2014 a location they all depend on today, or a new shared one if there is none \u2014 and call it from each site; until then, every change has to be made twice. Read the line range as the matched WINDOW rather than a finished unit: at \u0060libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/java/JavaCodegen.kt:259\u0060 it begins part-way through the construct above it, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that. Note first that the copies are not typed on the same thing: \u0060FragmentVariablesAdapterBuilder\u0060 names \u0060com.apollographql.apollo.compiler.codegen.java.operations.FragmentVariablesAdapterBuilder\u0060 in one and \u0060com.apollographql.apollo.compiler.codegen.kotlin.operations.FragmentVariablesAdapterBuilder\u0060 in another \u2014 different types that share a simple name, which is why the text matched. A single extracted unit cannot be given a parameter type that fits both, so unifying those types (or introducing a shared abstraction over them) is the step that has to come BEFORE the extraction above; if they are deliberately separate, the duplication is the price of that separation and the honest resolution is to record the decision rather than to extract."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/java/JavaCodegen.kt"},"region":{"startLine":259}}}],"partialFingerprints":{"codehealthFindingId/v1":"3656ea584e48b58de56335b72188670085fcff762f901021ee58d4896486f2b1"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (18 lines \u00D7 2): libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/kotlin/schema/EnumAsApolloEnumBuilder.kt:206-223 | libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/kotlin/schema/EnumAsSealedInterfaceBuilder.kt:159-176 \u2014 before extracting anything, compare \u0060libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/kotlin/schema/EnumAsApolloEnumBuilder.kt\u0060 and \u0060libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/kotlin/schema/EnumAsSealedInterfaceBuilder.kt\u0060 as WHOLE FILES: this scan already matched 3 separate duplicated blocks between them, totalling at least 37 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. Read the line range as the matched WINDOW rather than a finished unit: at \u0060libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/kotlin/schema/EnumAsApolloEnumBuilder.kt:206\u0060 it begins part-way through the construct above it, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/kotlin/schema/EnumAsApolloEnumBuilder.kt"},"region":{"startLine":206}}}],"partialFingerprints":{"codehealthFindingId/v1":"87f1b40ed2c941460235a7f85e8f3bb734d77436be486ea180be17c858178440"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (17 lines \u00D7 2): libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/java/JavaCodegen.kt:232-248 | libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/kotlin/KotlinCodegen.kt:266-282 \u2014 the copies span different directories, so extracting a shared function means choosing where it lives: put it somewhere both call sites can already reach \u2014 a location they all depend on today, or a new shared one if there is none \u2014 and call it from each site; until then, every change has to be made twice. Read the line range as the matched WINDOW rather than a finished unit: at \u0060libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/java/JavaCodegen.kt:232\u0060 it begins part-way through the construct above it, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that. Note first that the copies are not typed on the same thing: \u0060FragmentModelsBuilder\u0060 names \u0060com.apollographql.apollo.compiler.codegen.java.operations.FragmentModelsBuilder\u0060 in one and \u0060com.apollographql.apollo.compiler.codegen.kotlin.operations.FragmentModelsBuilder\u0060 in another \u2014 different types that share a simple name, which is why the text matched. A single extracted unit cannot be given a parameter type that fits both, so unifying those types (or introducing a shared abstraction over them) is the step that has to come BEFORE the extraction above; if they are deliberately separate, the duplication is the price of that separation and the honest resolution is to record the decision rather than to extract. \u2605 These copies have DRIFTED, and that is worth reading before extracting anything: just before the matched lines, \u0060libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/java/JavaCodegen.kt:229\u0060 calls \u0060generateMethodsJava\u0060 and \u0060libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/kotlin/KotlinCodegen.kt:263\u0060 does not \u2014 after which the two agree again for 2 more lines. One of those two behaviours is the intended one and the other is what a copy-paste left behind, so decide which BEFORE unifying them: extracting the shared part will silently settle it, and if the copy that skips the call is the wrong one, that bug is already live."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/java/JavaCodegen.kt"},"region":{"startLine":232}}}],"partialFingerprints":{"codehealthFindingId/v1":"710e896d19291ccbcae72e1d43308ec50bde2ae4148faf6207e06029a18ebd04"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (15\u201316 lines \u00D7 2): libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/ir/IrSchema.kt:302-317 | libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/ir/IrSchema.kt:321-335 \u2014 both copies are in the same file, so extract the block into one function there and call it from each site \u2014 the copies drift apart the first time only one of them is edited. Read the line range as the matched WINDOW rather than a finished unit: at \u0060libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/ir/IrSchema.kt:302\u0060 it runs out through the closing brace of the declaration holding it \u2014 the window is that declaration\u0027s tail, not a fragment that begins part-way through something, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/ir/IrSchema.kt"},"region":{"startLine":302}}}],"partialFingerprints":{"codehealthFindingId/v1":"53b3a91b2d7de26ad16221d05108a0ed954a3a3f84a0c144b95dae8125990efd"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (15 lines \u00D7 2): libraries/apollo-runtime/src/commonMain/kotlin/com/apollographql/apollo/network/ws/GraphQLWsProtocol.kt:54-68 | libraries/apollo-runtime/src/commonMain/kotlin/com/apollographql/apollo/network/ws/SubscriptionWsProtocol.kt:34-48 \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. Read the line range as the matched WINDOW rather than a finished unit: at \u0060libraries/apollo-runtime/src/commonMain/kotlin/com/apollographql/apollo/network/ws/GraphQLWsProtocol.kt:54\u0060 it does not close everything it opens, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that. The matched lines also transfer control out of the body holding them, which cannot survive a move into a called unit unchanged: have the extracted unit return that decision and let each site act on it."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"libraries/apollo-runtime/src/commonMain/kotlin/com/apollographql/apollo/network/ws/GraphQLWsProtocol.kt"},"region":{"startLine":54}}}],"partialFingerprints":{"codehealthFindingId/v1":"214e2bb42bf4a3b995aee8791f14b4a85a1af638a0b54a7e3dcbb5704f968083"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (15 lines \u00D7 2): libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/java/schema/EnumAsClassBuilder.kt:56-70 | libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/java/schema/EnumAsEnumBuilder.kt:56-70 \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. Read the line range as the matched WINDOW rather than a finished unit: at \u0060libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/java/schema/EnumAsClassBuilder.kt:56\u0060 it begins part-way through the construct above it, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/java/schema/EnumAsClassBuilder.kt"},"region":{"startLine":56}}}],"partialFingerprints":{"codehealthFindingId/v1":"5bbf9008055088747ed25038c22e6e0cb01ef99c7e7a336fe0da154022b10bfc"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (15 lines \u00D7 2): libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/ir/OperationBasedModelGroupBuilder.kt:287-301 | libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/ir/OperationBasedWithInterfacesModelGroupBuilder.kt:221-235 \u2014 before extracting anything, compare \u0060libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/ir/OperationBasedModelGroupBuilder.kt\u0060 and \u0060libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/ir/OperationBasedWithInterfacesModelGroupBuilder.kt\u0060 as WHOLE FILES: this scan already matched 4 separate duplicated blocks between them, totalling at least 49 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":"libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/ir/OperationBasedModelGroupBuilder.kt"},"region":{"startLine":287}}}],"partialFingerprints":{"codehealthFindingId/v1":"5af20ca1d4cdcfbd6c45669c73d371aa0099d72f86d3a0aed207f674f57901f0"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (14 lines \u00D7 2): libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/kotlin/builders/FragmentExtensionBuilder.kt:35-48 | libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/kotlin/builders/ResolverBuilder.kt:35-48 \u2014 before extracting anything, compare \u0060libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/kotlin/builders/FragmentExtensionBuilder.kt\u0060 and \u0060libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/kotlin/builders/ResolverBuilder.kt\u0060 as WHOLE FILES: 97% of the shorter file\u0027s lines also appear in the other, so this reads as one file having been copied from the other rather than as a helper waiting to be extracted. The 3 duplicated block(s) this scan matched between them are fragments of that copy, not the extent of it \u2014 treat the file pair as the unit. 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. Each matched range is the entire body of the declaration above it, so the region is already a complete unit: move that whole declaration to the shared location and have each site call it, rather than lifting the lines out of their bodies. Any \u0060return\u0060 inside it is the body\u0027s own exit and keeps its meaning in the moved unit."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/kotlin/builders/FragmentExtensionBuilder.kt"},"region":{"startLine":35}}}],"partialFingerprints":{"codehealthFindingId/v1":"e99800a957b4245b3ea7f3b6ce7aa32eedda559eb42d59a2eaf8412da46e7824"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (14 lines \u00D7 2): libraries/apollo-ast/src/commonMain/kotlin/com/apollographql/apollo/ast/gql.kt:735-748 | libraries/apollo-ast/src/commonMain/kotlin/com/apollographql/apollo/ast/gql.kt:1180-1193 \u2014 both copies are in the same file, so extract the block into one function there and call it from each site \u2014 the copies drift apart the first time only one of them is edited. Read the line range as the matched WINDOW rather than a finished unit: at \u0060libraries/apollo-ast/src/commonMain/kotlin/com/apollographql/apollo/ast/gql.kt:735\u0060 it runs out through the closing brace of the declaration holding it \u2014 the window is that declaration\u0027s tail, not a fragment that begins part-way through something, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"libraries/apollo-ast/src/commonMain/kotlin/com/apollographql/apollo/ast/gql.kt"},"region":{"startLine":735}}}],"partialFingerprints":{"codehealthFindingId/v1":"198014d8260d3a24e11b888ba6476fba475aa581ca3a5c1264579d450d8543fb"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (13 lines \u00D7 2): libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/java/helpers/AdapterCommon.kt:216-228 | libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/kotlin/helpers/AdapterCommon.kt:183-195 \u2014 before extracting anything, compare \u0060libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/java/helpers/AdapterCommon.kt\u0060 and \u0060libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/kotlin/helpers/AdapterCommon.kt\u0060 as WHOLE FILES: this scan already matched 3 separate duplicated blocks between them, totalling at least 43 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. Read the line range as the matched WINDOW rather than a finished unit: at \u0060libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/java/helpers/AdapterCommon.kt:216\u0060 it begins part-way through the construct above it, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that. The matched lines also transfer control out of the body holding them, which cannot survive a move into a called unit unchanged: have the extracted unit return that decision and let each site act on it. Note first that the copies are not typed on the same thing: \u0060CodeBlock\u0060 names \u0060com.squareup.javapoet.CodeBlock\u0060 in one and \u0060com.squareup.kotlinpoet.CodeBlock\u0060 in another \u2014 different types that share a simple name, which is why the text matched. A single extracted unit cannot be given a parameter type that fits both, so unifying those types (or introducing a shared abstraction over them) is the step that has to come BEFORE the extraction above; if they are deliberately separate, the duplication is the price of that separation and the honest resolution is to record the decision rather than to extract. \u2605 These copies have DRIFTED, and that is worth reading before extracting anything: just after the matched lines, \u0060libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/java/helpers/AdapterCommon.kt:230\u0060 calls \u0060applyIf\u0060, \u0060isNotEmpty\u0060, \u0060add\u0060 and \u0060libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/kotlin/helpers/AdapterCommon.kt:197\u0060 does not \u2014 after which the two agree again for 2 more lines. One of those two behaviours is the intended one and the other is what a copy-paste left behind, so decide which BEFORE unifying them: extracting the shared part will silently settle it, and if the copy that skips the call is the wrong one, that bug is already live."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/java/helpers/AdapterCommon.kt"},"region":{"startLine":216}}}],"partialFingerprints":{"codehealthFindingId/v1":"1d9f53ce99e128d783d83a761bd589f4dc8d70eaaab42facfa18c0798718d721"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (13 lines \u00D7 2): libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/ir/OperationBasedModelGroupBuilder.kt:349-361 | libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/ir/OperationBasedWithInterfacesModelGroupBuilder.kt:272-284 \u2014 before extracting anything, compare \u0060libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/ir/OperationBasedModelGroupBuilder.kt\u0060 and \u0060libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/ir/OperationBasedWithInterfacesModelGroupBuilder.kt\u0060 as WHOLE FILES: this scan already matched 4 separate duplicated blocks between them, totalling at least 49 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. Read the line range as the matched WINDOW rather than a finished unit: at \u0060libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/ir/OperationBasedModelGroupBuilder.kt:349\u0060 it runs out through the closing brace of the declaration holding it and carries on into the declaration that follows \u2014 the window is the tail of one member plus the head of the next, so no call can be substituted for those exact lines, and the smallest declaration that contains all of them is the type they sit in. The repeated unit is the member each site sits in: where those members\u0027 bodies are the same, move one whole member to the shared location and have the others delegate to it; where the copies are a run of near-identical overloads or wrappers that differ only in their signatures, the repetition IS the run \u2014 a one-line delegation has no helper inside it to lift \u2014 so generate the run from the set it enumerates, or accept it and keep each member\u0027s own documentation with it."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/ir/OperationBasedModelGroupBuilder.kt"},"region":{"startLine":349}}}],"partialFingerprints":{"codehealthFindingId/v1":"f619e39dc98a2552f822c50c309a97ffdac3888410ba376a7ef6326ed4f4b724"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (13 lines \u00D7 2): libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/ir/OperationBasedModelGroupBuilder.kt:400-412 | libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/ir/OperationBasedWithInterfacesModelGroupBuilder.kt:347-359 \u2014 before extracting anything, compare \u0060libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/ir/OperationBasedModelGroupBuilder.kt\u0060 and \u0060libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/ir/OperationBasedWithInterfacesModelGroupBuilder.kt\u0060 as WHOLE FILES: this scan already matched 4 separate duplicated blocks between them, totalling at least 49 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. Each matched range is the entire body of the declaration above it, so the region is already a complete unit: move that whole declaration to the shared location and have each site call it, rather than lifting the lines out of their bodies. Any \u0060return\u0060 inside it is the body\u0027s own exit and keeps its meaning in the moved unit."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/ir/OperationBasedModelGroupBuilder.kt"},"region":{"startLine":400}}}],"partialFingerprints":{"codehealthFindingId/v1":"daa4759a5c9713efd54bdfefc2e1b950351d3a35372aac145df94251addb5dfd"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (12 lines \u00D7 2): libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/kotlin/schema/EnumAsEnumBuilder.kt:137-148 | libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/kotlin/schema/EnumAsSealedInterfaceBuilder.kt:136-147 \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. Read the line range as the matched WINDOW rather than a finished unit: at \u0060libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/kotlin/schema/EnumAsEnumBuilder.kt:137\u0060 it begins part-way through the construct above it, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that. Note that the copies do not run to the end of the range shown: their LAST lines are different code, not the same code under different names \u2014 the matched region ends inside that line. Extract the lines above it, and read the last line of each site separately."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/kotlin/schema/EnumAsEnumBuilder.kt"},"region":{"startLine":137}}}],"partialFingerprints":{"codehealthFindingId/v1":"2eef9c19769eed8d668960b80c752c7df7f9090cd903688822fcc36790094c49"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (10\u201311 lines \u00D7 2): libraries/apollo-normalized-cache-sqlite/src/commonMain/kotlin/com/apollographql/apollo/cache/normalized/sql/SqlNormalizedCache.kt:163-172 | libraries/apollo-normalized-cache-sqlite/src/commonMain/kotlin/com/apollographql/apollo/cache/normalized/sql/SqlNormalizedCache.kt:198-208 \u2014 both copies are in the same file, so extract the block into one function there and call it from each site \u2014 the copies drift apart the first time only one of them is edited. Read the line range as the matched WINDOW rather than a finished unit: at \u0060libraries/apollo-normalized-cache-sqlite/src/commonMain/kotlin/com/apollographql/apollo/cache/normalized/sql/SqlNormalizedCache.kt:198\u0060 it runs out through the closing brace of the declaration holding it \u2014 the window is that declaration\u0027s tail, not a fragment that begins part-way through something, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"libraries/apollo-normalized-cache-sqlite/src/commonMain/kotlin/com/apollographql/apollo/cache/normalized/sql/SqlNormalizedCache.kt"},"region":{"startLine":163}}}],"partialFingerprints":{"codehealthFindingId/v1":"3daeafb30e731033bb7c345d0820e75fd675b44391acbfa572d41b4ad400874e"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (9\u201310 lines \u00D7 2): libraries/apollo-tooling/src/main/kotlin/com/apollographql/apollo/tooling/SchemaUploader.kt:63-72 | libraries/apollo-tooling/src/main/kotlin/com/apollographql/apollo/tooling/SchemaUploader.kt:100-108 \u2014 both copies are in the same file, so extract the block into one function there and call it from each site \u2014 the copies drift apart the first time only one of them is edited. Read the line range as the matched WINDOW rather than a finished unit: at \u0060libraries/apollo-tooling/src/main/kotlin/com/apollographql/apollo/tooling/SchemaUploader.kt:63\u0060 it begins part-way through the construct above it, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"libraries/apollo-tooling/src/main/kotlin/com/apollographql/apollo/tooling/SchemaUploader.kt"},"region":{"startLine":63}}}],"partialFingerprints":{"codehealthFindingId/v1":"3698c03eb4a7655ec3a89d59f6b93079c2ad099132b6282a9e81cae9d0539b49"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (10 lines \u00D7 2): libraries/apollo-runtime/src/commonMain/kotlin/com/apollographql/apollo/network/ws/AppSyncWsProtocol.kt:91-100 | libraries/apollo-runtime/src/commonMain/kotlin/com/apollographql/apollo/network/ws/SubscriptionWsProtocol.kt:59-68 \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":"libraries/apollo-runtime/src/commonMain/kotlin/com/apollographql/apollo/network/ws/AppSyncWsProtocol.kt"},"region":{"startLine":91}}}],"partialFingerprints":{"codehealthFindingId/v1":"098c48df9bb18a9a0f5ffd1469a87308ee8dc55fc2841b074a0eb779e67f773d"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (10 lines \u00D7 2): libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/java/JavaCodegen.kt:179-188 | libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/kotlin/KotlinCodegen.kt:214-223 \u2014 the copies span different directories, so extracting a shared function means choosing where it lives: put it somewhere both call sites can already reach \u2014 a location they all depend on today, or a new shared one if there is none \u2014 and call it from each site; until then, every change has to be made twice. Read the line range as the matched WINDOW rather than a finished unit: at \u0060libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/java/JavaCodegen.kt:179\u0060 it does not close everything it opens, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that. Note first that the copies are not typed on the same thing: \u0060InterfaceBuilder\u0060 names \u0060com.apollographql.apollo.compiler.codegen.java.schema.InterfaceBuilder\u0060 in one and \u0060com.apollographql.apollo.compiler.codegen.kotlin.schema.InterfaceBuilder\u0060 in another \u2014 different types that share a simple name, which is why the text matched. A single extracted unit cannot be given a parameter type that fits both, so unifying those types (or introducing a shared abstraction over them) is the step that has to come BEFORE the extraction above; if they are deliberately separate, the duplication is the price of that separation and the honest resolution is to record the decision rather than to extract."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/java/JavaCodegen.kt"},"region":{"startLine":179}}}],"partialFingerprints":{"codehealthFindingId/v1":"a4fe2bbfbbac06ad08c56b02929f1fee7b083f17f5dac36e11c1ffb898f7ce51"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (9 lines \u00D7 2): libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/java/schema/TypesBuilder.kt:58-66 | libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/java/schema/TypesBuilder.kt:75-83 \u2014 both copies are in the same file, so extract the block into one function there and call it from each site \u2014 the copies drift apart the first time only one of them is edited. The \u0060return\u0060 at the foot of the matched lines is the enclosing body\u0027s own terminal exit, not an early one: it moves with them unchanged, and each site calls the extracted unit from the position that \u0060return\u0060 occupied \u2014 no decision has to be handed back and re-acted on."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/java/schema/TypesBuilder.kt"},"region":{"startLine":58}}}],"partialFingerprints":{"codehealthFindingId/v1":"9eef0ece2f3363949d0e4cb3720890dbe415a67935394510ad146d539be1b824"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (9 lines \u00D7 2): libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/kotlin/schema/util/TypesBuilder.kt:56-64 | libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/kotlin/schema/util/TypesBuilder.kt:73-81 \u2014 both copies are in the same file, so extract the block into one function there and call it from each site \u2014 the copies drift apart the first time only one of them is edited. The \u0060return\u0060 at the foot of the matched lines is the enclosing body\u0027s own terminal exit, not an early one: it moves with them unchanged, and each site calls the extracted unit from the position that \u0060return\u0060 occupied \u2014 no decision has to be handed back and re-acted on."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/kotlin/schema/util/TypesBuilder.kt"},"region":{"startLine":56}}}],"partialFingerprints":{"codehealthFindingId/v1":"c02983db58ab78b7c13bda34a5a10a219a05f0b04b938bd197a52fb45f8b3c27"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (9 lines \u00D7 2): libraries/apollo-ast/src/commonMain/kotlin/com/apollographql/apollo/ast/internal/Parser.kt:334-342 | libraries/apollo-ast/src/commonMain/kotlin/com/apollographql/apollo/ast/internal/Parser.kt:894-902 \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":"libraries/apollo-ast/src/commonMain/kotlin/com/apollographql/apollo/ast/internal/Parser.kt"},"region":{"startLine":334}}}],"partialFingerprints":{"codehealthFindingId/v1":"6b571c89581eea17e4e1962c9276914769c5b8cffe2d348b7732f01472c3873a"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (8 lines \u00D7 2): libraries/apollo-api/src/commonMain/kotlin/com/apollographql/apollo/api/json/MapJsonReader.kt:109-116 | libraries/apollo-api/src/jsMain/kotlin/com/apollographql/apollo/api/json/DynamicJsJsonReader.kt:149-156 \u2014 before extracting anything, compare \u0060libraries/apollo-api/src/commonMain/kotlin/com/apollographql/apollo/api/json/MapJsonReader.kt\u0060 and \u0060libraries/apollo-api/src/jsMain/kotlin/com/apollographql/apollo/api/json/DynamicJsJsonReader.kt\u0060 as WHOLE FILES: this scan already matched 12 separate duplicated blocks between them, totalling at least 168 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. Each matched range is the entire body of the declaration above it, so the region is already a complete unit: move that whole declaration to the shared location and have each site call it, rather than lifting the lines out of their bodies. Any \u0060return\u0060 inside it is the body\u0027s own exit and keeps its meaning in the moved unit."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"libraries/apollo-api/src/commonMain/kotlin/com/apollographql/apollo/api/json/MapJsonReader.kt"},"region":{"startLine":109}}}],"partialFingerprints":{"codehealthFindingId/v1":"b95526717275a3c34ac98edf904f4d6dbd78d1d4c3349aa2285cbe5707248740"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (7 lines \u00D7 2): libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/internal/serialization.kt:56-62 | libraries/apollo-tooling/src/main/kotlin/com/apollographql/apollo/tooling/serialization.kt:12-18 \u2014 the copies span different directories, so extracting a shared function means choosing where it lives: put it somewhere both call sites can already reach \u2014 a location they all depend on today, or a new shared one if there is none \u2014 and call it from each site; until then, every change has to be made twice. Read the line range as the matched WINDOW rather than a finished unit: at \u0060libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/internal/serialization.kt:56\u0060 it runs out through the closing brace of the declaration holding it \u2014 the window is that declaration\u0027s tail, not a fragment that begins part-way through something, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/internal/serialization.kt"},"region":{"startLine":56}}}],"partialFingerprints":{"codehealthFindingId/v1":"9cbb6f0cbae6ca118d1006a095a7283f0d8954ee5e6e743570a01517e06b6af4"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (7 lines \u00D7 2): libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/java/helpers/bestGuess.kt:12-18 | libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/kotlin/helpers/bestGuess.kt:14-20 \u2014 the copies span different directories, so extracting a shared function means choosing where it lives: put it somewhere both call sites can already reach \u2014 a location they all depend on today, or a new shared one if there is none \u2014 and call it from each site; until then, every change has to be made twice. Read the line range as the matched WINDOW rather than a finished unit: at \u0060libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/java/helpers/bestGuess.kt:12\u0060 it does not close everything it opens, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that. The matched lines also transfer control out of the body holding them, which cannot survive a move into a called unit unchanged: have the extracted unit return that decision and let each site act on it."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/java/helpers/bestGuess.kt"},"region":{"startLine":12}}}],"partialFingerprints":{"codehealthFindingId/v1":"1b17e91420984b0ed2048164df13fb84efe2496b7dd676aecee57fa12dd5eaa5"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (7 lines \u00D7 2): libraries/apollo-api/src/commonMain/kotlin/com/apollographql/apollo/api/json/MapJsonReader.kt:64-70 | libraries/apollo-api/src/jsMain/kotlin/com/apollographql/apollo/api/json/DynamicJsJsonReader.kt:98-104 \u2014 before extracting anything, compare \u0060libraries/apollo-api/src/commonMain/kotlin/com/apollographql/apollo/api/json/MapJsonReader.kt\u0060 and \u0060libraries/apollo-api/src/jsMain/kotlin/com/apollographql/apollo/api/json/DynamicJsJsonReader.kt\u0060 as WHOLE FILES: this scan already matched 12 separate duplicated blocks between them, totalling at least 168 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":"libraries/apollo-api/src/commonMain/kotlin/com/apollographql/apollo/api/json/MapJsonReader.kt"},"region":{"startLine":64}}}],"partialFingerprints":{"codehealthFindingId/v1":"c2045ba684d3e5d5327d368cb1365a1553c5fb8b1d303146f5c7eec2068ed99a"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (6 lines \u00D7 2): libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/java/schema/ScalarBuilder.kt:38-43 | libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/kotlin/schema/ScalarBuilder.kt:37-42 \u2014 the copies span different directories, so extracting a shared function means choosing where it lives: put it somewhere both call sites can already reach \u2014 a location they all depend on today, or a new shared one if there is none \u2014 and call it from each site; until then, every change has to be made twice. Read the line range as the matched WINDOW rather than a finished unit: at \u0060libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/java/schema/ScalarBuilder.kt:38\u0060 it begins part-way through the construct above it, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that. \u2605 These copies have DRIFTED, and that is worth reading before extracting anything: just after the matched lines, \u0060libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/kotlin/schema/ScalarBuilder.kt:44\u0060 calls \u0060registerScalarInlineProperty\u0060 and \u0060libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/java/schema/ScalarBuilder.kt:44\u0060 does not \u2014 after which the two agree again for 2 more lines. One of those two behaviours is the intended one and the other is what a copy-paste left behind, so decide which BEFORE unifying them: extracting the shared part will silently settle it, and if the copy that skips the call is the wrong one, that bug is already live."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/java/schema/ScalarBuilder.kt"},"region":{"startLine":38}}}],"partialFingerprints":{"codehealthFindingId/v1":"f6c14d59b697dd23498dc3e8caa9ccd99eceaaaa6993b8018e5c50cef22f3ff0"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (6 lines \u00D7 2): libraries/apollo-api/src/commonMain/kotlin/com/apollographql/apollo/api/json/BufferedSourceJsonReader.kt:515-520 | libraries/apollo-api/src/commonMain/kotlin/com/apollographql/apollo/api/json/BufferedSourceJsonReader.kt:624-629 \u2014 both copies are in the same file, so extract the block into one function there and call it from each site \u2014 the copies drift apart the first time only one of them is edited. Read the line range as the matched WINDOW rather than a finished unit: at \u0060libraries/apollo-api/src/commonMain/kotlin/com/apollographql/apollo/api/json/BufferedSourceJsonReader.kt:515\u0060 it does not close everything it opens, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"libraries/apollo-api/src/commonMain/kotlin/com/apollographql/apollo/api/json/BufferedSourceJsonReader.kt"},"region":{"startLine":515}}}],"partialFingerprints":{"codehealthFindingId/v1":"0483814a8518960cfeadb5f166dd9d561a89b4a8bbba1ffeb2ce4f706d16425e"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (9 lines \u00D7 3): libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/model/CompilationUnitModel.kt:41-49 | libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/model/ProjectModel.kt:20-28 | libraries/apollo-tooling/src/main/kotlin/com/apollographql/apollo/tooling/model/TelemetryData.kt:24-32 \u2014 the copies span different directories, so extracting a shared function means choosing where it lives: put it somewhere all 3 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 3 times. The matched lines also transfer control out of the body holding them, which cannot survive a move into a called unit unchanged: have the extracted unit return that decision and let each site act on it."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/model/CompilationUnitModel.kt"},"region":{"startLine":41}}}],"partialFingerprints":{"codehealthFindingId/v1":"300eeb242a6a1cd215a1ff346bdd383b9bbfa0a90052618d7244be2f6e3a75b1"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (10 lines \u00D7 2): libraries/apollo-ast/src/commonMain/kotlin/com/apollographql/apollo/ast/gql.kt:2475-2484 | libraries/apollo-tooling/src/main/kotlin/com/apollographql/apollo/tooling/RegisterOperations.kt:67-76 \u2014 the copies span different directories, so extracting a shared function means choosing where it lives: put it somewhere both call sites can already reach \u2014 a location they all depend on today, or a new shared one if there is none \u2014 and call it from each site; until then, every change has to be made twice. Read the line range as the matched WINDOW rather than a finished unit: at \u0060libraries/apollo-ast/src/commonMain/kotlin/com/apollographql/apollo/ast/gql.kt:2475\u0060 it begins part-way through the construct above it, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"libraries/apollo-ast/src/commonMain/kotlin/com/apollographql/apollo/ast/gql.kt"},"region":{"startLine":2475}}}],"partialFingerprints":{"codehealthFindingId/v1":"e61cd92b5d0dfef93310ae5ee8d45fd4726d791ec5272531d1fe70660dae199e"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (10 lines \u00D7 2): libraries/apollo-normalized-cache/src/commonMain/kotlin/com/apollographql/apollo/cache/normalized/ClientCacheExtensions.kt:131-140 | libraries/apollo-runtime/src/commonMain/kotlin/com/apollographql/apollo/interceptor/ApolloInterceptor.kt:68-77 \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. The matched lines also transfer control out of the body holding them, which cannot survive a move into a called unit unchanged: have the extracted unit return that decision and let each site act on it."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"libraries/apollo-normalized-cache/src/commonMain/kotlin/com/apollographql/apollo/cache/normalized/ClientCacheExtensions.kt"},"region":{"startLine":131}}}],"partialFingerprints":{"codehealthFindingId/v1":"f5a8690d636032ed6c47b83a3f8032fe50627984462650d57e51aa94f437f10c"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (10 lines \u00D7 2): libraries/apollo-runtime/src/commonMain/kotlin/com/apollographql/apollo/network/websocket/GraphQLWsProtocol.kt:24-33 | libraries/apollo-runtime/src/commonMain/kotlin/com/apollographql/apollo/network/websocket/SubscriptionWsProtocol.kt:26-35 \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. The matched lines also transfer control out of the body holding them, which cannot survive a move into a called unit unchanged: have the extracted unit return that decision and let each site act on it."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"libraries/apollo-runtime/src/commonMain/kotlin/com/apollographql/apollo/network/websocket/GraphQLWsProtocol.kt"},"region":{"startLine":24}}}],"partialFingerprints":{"codehealthFindingId/v1":"2896eb136f107ade250d2dd3befbad0d1b860cd7507e8517d0b381dc91d9257f"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (5 lines \u00D7 2): libraries/apollo-runtime/src/jvmCommonMain/kotlin/com/apollographql/apollo/network/OkHttpExtensions.kt:74-78 | libraries/apollo-runtime/src/jvmCommonMain/kotlin/com/apollographql/apollo/network/websocket/WebSocketEngine.jvm.kt:98-102 \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":"libraries/apollo-runtime/src/jvmCommonMain/kotlin/com/apollographql/apollo/network/OkHttpExtensions.kt"},"region":{"startLine":74}}}],"partialFingerprints":{"codehealthFindingId/v1":"6b51eaa40ba5250baea13359f2b534675379031bca995e27d9f99cb4abb86046"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (11 lines \u00D7 2): libraries/apollo-execution/src/commonMain/kotlin/com/apollographql/apollo/execution/internal/LruCache.kt:34-44 | libraries/apollo-normalized-cache-api/src/commonMain/kotlin/com/apollographql/apollo/cache/normalized/api/internal/LruCache.kt:34-44 \u2014 before extracting anything, compare \u0060libraries/apollo-execution/src/commonMain/kotlin/com/apollographql/apollo/execution/internal/LruCache.kt\u0060 and \u0060libraries/apollo-normalized-cache-api/src/commonMain/kotlin/com/apollographql/apollo/cache/normalized/api/internal/LruCache.kt\u0060 as WHOLE FILES: this scan already matched 6 separate duplicated blocks between them, totalling at least 83 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":"libraries/apollo-execution/src/commonMain/kotlin/com/apollographql/apollo/execution/internal/LruCache.kt"},"region":{"startLine":34}}}],"partialFingerprints":{"codehealthFindingId/v1":"2202e17271396a43681a82040456a8696bdb6cdc300e952d637538a3afbcdd4a"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (8 lines \u00D7 2): libraries/apollo-execution/src/commonMain/kotlin/com/apollographql/apollo/execution/internal/LruCache.kt:52-59 | libraries/apollo-normalized-cache-api/src/commonMain/kotlin/com/apollographql/apollo/cache/normalized/api/internal/LruCache.kt:52-59 \u2014 before extracting anything, compare \u0060libraries/apollo-execution/src/commonMain/kotlin/com/apollographql/apollo/execution/internal/LruCache.kt\u0060 and \u0060libraries/apollo-normalized-cache-api/src/commonMain/kotlin/com/apollographql/apollo/cache/normalized/api/internal/LruCache.kt\u0060 as WHOLE FILES: this scan already matched 6 separate duplicated blocks between them, totalling at least 83 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. The matched lines also transfer control out of the body holding them, which cannot survive a move into a called unit unchanged: have the extracted unit return that decision and let each site act on it."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"libraries/apollo-execution/src/commonMain/kotlin/com/apollographql/apollo/execution/internal/LruCache.kt"},"region":{"startLine":52}}}],"partialFingerprints":{"codehealthFindingId/v1":"bc0392e9ac859e733be4aef76b8cc18691ef1a84701e56473333e8c3748b6466"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (8 lines \u00D7 2): libraries/apollo-execution/src/commonMain/kotlin/com/apollographql/apollo/execution/internal/LruCache.kt:83-90 | libraries/apollo-normalized-cache-api/src/commonMain/kotlin/com/apollographql/apollo/cache/normalized/api/internal/LruCache.kt:83-90 \u2014 before extracting anything, compare \u0060libraries/apollo-execution/src/commonMain/kotlin/com/apollographql/apollo/execution/internal/LruCache.kt\u0060 and \u0060libraries/apollo-normalized-cache-api/src/commonMain/kotlin/com/apollographql/apollo/cache/normalized/api/internal/LruCache.kt\u0060 as WHOLE FILES: this scan already matched 6 separate duplicated blocks between them, totalling at least 83 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":"libraries/apollo-execution/src/commonMain/kotlin/com/apollographql/apollo/execution/internal/LruCache.kt"},"region":{"startLine":83}}}],"partialFingerprints":{"codehealthFindingId/v1":"91633ec55c8bc6831258168b4332a3c906176f592cbecf85db079fb10508010c"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (10 lines \u00D7 2): libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/Flatten.kt:5-14 | libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/ir/SelectionSetsBuilder.kt:24-33 \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. The matched lines also transfer control out of the body holding them, which cannot survive a move into a called unit unchanged: have the extracted unit return that decision and let each site act on it."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/Flatten.kt"},"region":{"startLine":5}}}],"partialFingerprints":{"codehealthFindingId/v1":"889af10fd87733d832421844cf7287d0d9997ba806c18e46de37b6e3bddc67c8"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (7 lines \u00D7 4): libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/java/helpers/InputAdapter.kt:72-78 | libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/java/operations/util/VariablesAdapter.kt:55-61 | libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/kotlin/helpers/InputAdapter.kt:67-73 | libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/kotlin/operations/util/VariablesAdapter.kt:43-50 \u2014 the copies span different directories, so extracting a shared function means choosing where it lives: put it somewhere all 4 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 4 times. The matched lines also transfer control out of the body holding them, which cannot survive a move into a called unit unchanged: have the extracted unit return that decision and let each site act on it. Note first that the copies are not typed on the same thing: \u0060CodeBlock\u0060 names \u0060com.squareup.javapoet.CodeBlock\u0060 in one and \u0060com.squareup.kotlinpoet.CodeBlock\u0060 in another \u2014 different types that share a simple name, which is why the text matched. A single extracted unit cannot be given a parameter type that fits both, so unifying those types (or introducing a shared abstraction over them) is the step that has to come BEFORE the extraction above; if they are deliberately separate, the duplication is the price of that separation and the honest resolution is to record the decision rather than to extract."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/java/helpers/InputAdapter.kt"},"region":{"startLine":72}}}],"partialFingerprints":{"codehealthFindingId/v1":"6caf1277cd5d2c0518b05a1b55b96178a7b166575550af52b22e3ccd455b064d"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (8 lines \u00D7 2): libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/java/operations/FragmentDataAdapterBuilder.kt:43-50 | libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/java/operations/OperationResponseAdapterBuilder.kt:43-50 \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. The matched lines also transfer control out of the body holding them, which cannot survive a move into a called unit unchanged: have the extracted unit return that decision and let each site act on it."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/java/operations/FragmentDataAdapterBuilder.kt"},"region":{"startLine":43}}}],"partialFingerprints":{"codehealthFindingId/v1":"2adb5612fdedc79d6c5af6dd4f7e1673d8a1e75dc1616bb650592c139b51c339"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (9 lines \u00D7 2): libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/java/schema/InputObjectAdapterBuilder.kt:35-43 | libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/kotlin/schema/InputObjectAdapterBuilder.kt:36-44 \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. The matched lines also transfer control out of the body holding them, which cannot survive a move into a called unit unchanged: have the extracted unit return that decision and let each site act on it. Note first that the copies are not typed on the same thing: \u0060TypeSpec\u0060 names \u0060com.squareup.javapoet.TypeSpec\u0060 in one and \u0060com.squareup.kotlinpoet.TypeSpec\u0060 in another \u2014 different types that share a simple name, which is why the text matched. A single extracted unit cannot be given a parameter type that fits both, so unifying those types (or introducing a shared abstraction over them) is the step that has to come BEFORE the extraction above; if they are deliberately separate, the duplication is the price of that separation and the honest resolution is to record the decision rather than to extract."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/java/schema/InputObjectAdapterBuilder.kt"},"region":{"startLine":35}}}],"partialFingerprints":{"codehealthFindingId/v1":"e38d570320c536914f084bc27dbae47e387d9d8004f8b9efcd26d75dee479acd"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (10 lines \u00D7 2): libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/java/schema/InterfaceBuilder.kt:38-47 | libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/java/schema/ObjectBuilder.kt:45-54 \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. The matched lines also transfer control out of the body holding them, which cannot survive a move into a called unit unchanged: have the extracted unit return that decision and let each site act on it."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/java/schema/InterfaceBuilder.kt"},"region":{"startLine":38}}}],"partialFingerprints":{"codehealthFindingId/v1":"6a0e60218e5907e9133eab8f78bb891589925bbe7e56ef7159f4bcd582608f6b"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (8 lines \u00D7 2): libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/kotlin/builders/FragmentExtensionBuilder.kt:26-33 | libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/kotlin/builders/ResolverBuilder.kt:26-33 \u2014 before extracting anything, compare \u0060libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/kotlin/builders/FragmentExtensionBuilder.kt\u0060 and \u0060libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/kotlin/builders/ResolverBuilder.kt\u0060 as WHOLE FILES: 97% of the shorter file\u0027s lines also appear in the other, so this reads as one file having been copied from the other rather than as a helper waiting to be extracted. The 3 duplicated block(s) this scan matched between them are fragments of that copy, not the extent of it \u2014 treat the file pair as the unit. 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. The matched lines also transfer control out of the body holding them, which cannot survive a move into a called unit unchanged: have the extracted unit return that decision and let each site act on it."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/kotlin/builders/FragmentExtensionBuilder.kt"},"region":{"startLine":26}}}],"partialFingerprints":{"codehealthFindingId/v1":"996ef9a7bb345a166f852a0a3b4d7eeea2ff9e3eda192e43132b7d5235e7273d"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (7 lines \u00D7 2): libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/kotlin/builders/FragmentExtensionBuilder.kt:50-56 | libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/kotlin/builders/ResolverBuilder.kt:50-56 \u2014 before extracting anything, compare \u0060libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/kotlin/builders/FragmentExtensionBuilder.kt\u0060 and \u0060libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/kotlin/builders/ResolverBuilder.kt\u0060 as WHOLE FILES: 97% of the shorter file\u0027s lines also appear in the other, so this reads as one file having been copied from the other rather than as a helper waiting to be extracted. The 3 duplicated block(s) this scan matched between them are fragments of that copy, not the extent of it \u2014 treat the file pair as the unit. 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. The matched lines also transfer control out of the body holding them, which cannot survive a move into a called unit unchanged: have the extracted unit return that decision and let each site act on it."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/kotlin/builders/FragmentExtensionBuilder.kt"},"region":{"startLine":50}}}],"partialFingerprints":{"codehealthFindingId/v1":"39508588583bfd24b332c60b5a20c185dc2731f38a4021469e51b049d63a6cfb"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (7 lines \u00D7 2): libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/kotlin/schema/EnumAsApolloEnumBuilder.kt:88-94 | libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/kotlin/schema/EnumAsSealedInterfaceBuilder.kt:71-77 \u2014 before extracting anything, compare \u0060libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/kotlin/schema/EnumAsApolloEnumBuilder.kt\u0060 and \u0060libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/kotlin/schema/EnumAsSealedInterfaceBuilder.kt\u0060 as WHOLE FILES: this scan already matched 3 separate duplicated blocks between them, totalling at least 37 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. The matched lines also transfer control out of the body holding them, which cannot survive a move into a called unit unchanged: have the extracted unit return that decision and let each site act on it."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/kotlin/schema/EnumAsApolloEnumBuilder.kt"},"region":{"startLine":88}}}],"partialFingerprints":{"codehealthFindingId/v1":"bbc84f00db0fd4eac1db187a85a5e918fc0fce539a649d11a37bd027ccaa28f5"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (9 lines \u00D7 3): libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/kotlin/schema/InterfaceBuilder.kt:46-54 | libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/kotlin/schema/ObjectBuilder.kt:46-54 | libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/kotlin/schema/UnionBuilder.kt:41-49 \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 all 3 call sites, so a change lands once. The matched lines also transfer control out of the body holding them, which cannot survive a move into a called unit unchanged: have the extracted unit return that decision and let each site act on it. \u2605 These copies have DRIFTED, and that is worth reading before extracting anything: just after the matched lines, \u0060libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/kotlin/schema/InterfaceBuilder.kt:58\u0060 calls \u0060addProperties\u0060, \u0060argumentsPropertySpecs\u0060 and \u0060libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/kotlin/schema/UnionBuilder.kt:53\u0060 does not \u2014 after which the two agree again for 2 more lines. One of those two behaviours is the intended one and the other is what a copy-paste left behind, so decide which BEFORE unifying them: extracting the shared part will silently settle it, and if the copy that skips the call is the wrong one, that bug is already live."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/kotlin/schema/InterfaceBuilder.kt"},"region":{"startLine":46}}}],"partialFingerprints":{"codehealthFindingId/v1":"460896253f642a0cb5ef2e52afc9945b8858f1284ad1c6ef1c0aed9d4c525cb1"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (6 lines \u00D7 2): libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/kotlin/schema/InterfaceBuilder.kt:56-61 | libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/kotlin/schema/ObjectBuilder.kt:56-61 \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. The matched lines also transfer control out of the body holding them, which cannot survive a move into a called unit unchanged: have the extracted unit return that decision and let each site act on it."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/kotlin/schema/InterfaceBuilder.kt"},"region":{"startLine":56}}}],"partialFingerprints":{"codehealthFindingId/v1":"aeb45e6fde439de538eef78184fa7925f119f30842c572eda2742f8d83ada5e1"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (8 lines \u00D7 2): libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/ir/OperationBasedModelGroupBuilder.kt:414-421 | libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/ir/OperationBasedWithInterfacesModelGroupBuilder.kt:393-400 \u2014 before extracting anything, compare \u0060libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/ir/OperationBasedModelGroupBuilder.kt\u0060 and \u0060libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/ir/OperationBasedWithInterfacesModelGroupBuilder.kt\u0060 as WHOLE FILES: this scan already matched 4 separate duplicated blocks between them, totalling at least 49 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. The matched lines also transfer control out of the body holding them, which cannot survive a move into a called unit unchanged: have the extracted unit return that decision and let each site act on it."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/ir/OperationBasedModelGroupBuilder.kt"},"region":{"startLine":414}}}],"partialFingerprints":{"codehealthFindingId/v1":"e58a416cb3e5d1265440486ba8b662ac32f6c68a9efb6138285ac2098fd20f66"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (6 lines \u00D7 4): libraries/apollo-api/src/appleMain/kotlin/com/apollographql/apollo/api/toJson.kt:15-20 | libraries/apollo-api/src/jsMain/kotlin/com/apollographql/apollo/api/toJson.kt:12-17 | libraries/apollo-api/src/linuxMain/kotlin/com/apollographql/apollo/api/toJson.kt:12-17 | libraries/apollo-api/src/wasmJsMain/kotlin/com/apollographql/apollo/api/toJson.wasmJs.kt:12-17 \u2014 before extracting anything, compare \u0060libraries/apollo-api/src/appleMain/kotlin/com/apollographql/apollo/api/toJson.kt\u0060 and \u0060libraries/apollo-api/src/jsMain/kotlin/com/apollographql/apollo/api/toJson.kt\u0060 as WHOLE FILES: 100% of the shorter file\u0027s lines also appear in the other, so this reads as one file having been copied from the other rather than as a helper waiting to be extracted. The 1 duplicated block(s) this scan matched between them are fragments of that copy, not the extent of it \u2014 treat the file pair as the unit. 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. Read the line range as the matched WINDOW rather than a finished unit: at \u0060libraries/apollo-api/src/appleMain/kotlin/com/apollographql/apollo/api/toJson.kt:15\u0060 it begins part-way through the construct above it, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"libraries/apollo-api/src/appleMain/kotlin/com/apollographql/apollo/api/toJson.kt"},"region":{"startLine":15}}}],"partialFingerprints":{"codehealthFindingId/v1":"d4e635b9f0f900ab8ce01bd9c565cba02bba500623362d96bb6ee59a8f8f260f"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (9 lines \u00D7 2): libraries/apollo-api/src/commonMain/kotlin/com/apollographql/apollo/api/json/MapJsonReader.kt:132-140 | libraries/apollo-api/src/jsMain/kotlin/com/apollographql/apollo/api/json/DynamicJsJsonReader.kt:173-181 \u2014 before extracting anything, compare \u0060libraries/apollo-api/src/commonMain/kotlin/com/apollographql/apollo/api/json/MapJsonReader.kt\u0060 and \u0060libraries/apollo-api/src/jsMain/kotlin/com/apollographql/apollo/api/json/DynamicJsJsonReader.kt\u0060 as WHOLE FILES: this scan already matched 12 separate duplicated blocks between them, totalling at least 168 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":"libraries/apollo-api/src/commonMain/kotlin/com/apollographql/apollo/api/json/MapJsonReader.kt"},"region":{"startLine":132}}}],"partialFingerprints":{"codehealthFindingId/v1":"6485089c0b5335d1574fe9e95a0b42ae6cd825723b35f67a5c984b6eded9a82b"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (7 lines \u00D7 2): libraries/apollo-api/src/commonMain/kotlin/com/apollographql/apollo/api/json/MapJsonReader.kt:168-174 | libraries/apollo-api/src/jsMain/kotlin/com/apollographql/apollo/api/json/DynamicJsJsonReader.kt:207-213 \u2014 before extracting anything, compare \u0060libraries/apollo-api/src/commonMain/kotlin/com/apollographql/apollo/api/json/MapJsonReader.kt\u0060 and \u0060libraries/apollo-api/src/jsMain/kotlin/com/apollographql/apollo/api/json/DynamicJsJsonReader.kt\u0060 as WHOLE FILES: this scan already matched 12 separate duplicated blocks between them, totalling at least 168 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":"libraries/apollo-api/src/commonMain/kotlin/com/apollographql/apollo/api/json/MapJsonReader.kt"},"region":{"startLine":168}}}],"partialFingerprints":{"codehealthFindingId/v1":"a2383b9fc5009d803bf57c13a35e11f361a1bbf30f96e8df750d4b71c769226f"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (9 lines \u00D7 2): libraries/apollo-api/src/commonMain/kotlin/com/apollographql/apollo/api/json/MapJsonReader.kt:206-214 | libraries/apollo-api/src/jsMain/kotlin/com/apollographql/apollo/api/json/DynamicJsJsonReader.kt:250-258 \u2014 before extracting anything, compare \u0060libraries/apollo-api/src/commonMain/kotlin/com/apollographql/apollo/api/json/MapJsonReader.kt\u0060 and \u0060libraries/apollo-api/src/jsMain/kotlin/com/apollographql/apollo/api/json/DynamicJsJsonReader.kt\u0060 as WHOLE FILES: this scan already matched 12 separate duplicated blocks between them, totalling at least 168 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. The matched lines also transfer control out of the body holding them, which cannot survive a move into a called unit unchanged: have the extracted unit return that decision and let each site act on it."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"libraries/apollo-api/src/commonMain/kotlin/com/apollographql/apollo/api/json/MapJsonReader.kt"},"region":{"startLine":206}}}],"partialFingerprints":{"codehealthFindingId/v1":"94e6c494b03587a50129afdb8f8ec369947986b19d8d13ec8ba0556820a561ce"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (17 lines \u00D7 2): libraries/apollo-api/src/commonMain/kotlin/com/apollographql/apollo/api/json/MapJsonReader.kt:383-399 | libraries/apollo-api/src/jsMain/kotlin/com/apollographql/apollo/api/json/DynamicJsJsonReader.kt:398-414 \u2014 before extracting anything, compare \u0060libraries/apollo-api/src/commonMain/kotlin/com/apollographql/apollo/api/json/MapJsonReader.kt\u0060 and \u0060libraries/apollo-api/src/jsMain/kotlin/com/apollographql/apollo/api/json/DynamicJsJsonReader.kt\u0060 as WHOLE FILES: this scan already matched 12 separate duplicated blocks between them, totalling at least 168 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. The matched lines also transfer control out of the body holding them, which cannot survive a move into a called unit unchanged: have the extracted unit return that decision and let each site act on it."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"libraries/apollo-api/src/commonMain/kotlin/com/apollographql/apollo/api/json/MapJsonReader.kt"},"region":{"startLine":383}}}],"partialFingerprints":{"codehealthFindingId/v1":"f193d402a69fdea91711e10655322c56d362cf0366493d55b1d3bf68dabc668d"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (8 lines \u00D7 2): libraries/apollo-api/src/commonMain/kotlin/com/apollographql/apollo/api/json/MapJsonReader.kt:412-419 | libraries/apollo-api/src/jsMain/kotlin/com/apollographql/apollo/api/json/DynamicJsJsonReader.kt:427-434 \u2014 before extracting anything, compare \u0060libraries/apollo-api/src/commonMain/kotlin/com/apollographql/apollo/api/json/MapJsonReader.kt\u0060 and \u0060libraries/apollo-api/src/jsMain/kotlin/com/apollographql/apollo/api/json/DynamicJsJsonReader.kt\u0060 as WHOLE FILES: this scan already matched 12 separate duplicated blocks between them, totalling at least 168 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. The matched lines also transfer control out of the body holding them, which cannot survive a move into a called unit unchanged: have the extracted unit return that decision and let each site act on it."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"libraries/apollo-api/src/commonMain/kotlin/com/apollographql/apollo/api/json/MapJsonReader.kt"},"region":{"startLine":412}}}],"partialFingerprints":{"codehealthFindingId/v1":"2b56c758d3d8de3ccca50533dd44ee11baf52de79c5c565de72a6d8351965794"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (7 lines \u00D7 2): libraries/apollo-api-java/src/main/kotlin/com/apollographql/apollo/api/java/adapter/GuavaOptionalAdapter.kt:30-36 | libraries/apollo-api-java/src/main/kotlin/com/apollographql/apollo/api/java/adapter/JavaOptionalAdapter.kt:29-35 \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. Note first that the copies are not typed on the same thing: \u0060Optional\u0060 names \u0060com.google.common.base.Optional\u0060 in one and \u0060java.util.Optional\u0060 in another \u2014 different types that share a simple name, which is why the text matched. A single extracted unit cannot be given a parameter type that fits both, so unifying those types (or introducing a shared abstraction over them) is the step that has to come BEFORE the extraction above; if they are deliberately separate, the duplication is the price of that separation and the honest resolution is to record the decision rather than to extract."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"libraries/apollo-api-java/src/main/kotlin/com/apollographql/apollo/api/java/adapter/GuavaOptionalAdapter.kt"},"region":{"startLine":30}}}],"partialFingerprints":{"codehealthFindingId/v1":"85ec54dae632030f2694b49691c3a8323014676e6eaf9f4d591e25e3fa39c89b"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (12 lines \u00D7 2): libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/kotlin/schema/EnumAsApolloEnumBuilder.kt:152-163 | libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/kotlin/schema/EnumAsSealedInterfaceBuilder.kt:116-127 \u2014 before extracting anything, compare \u0060libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/kotlin/schema/EnumAsApolloEnumBuilder.kt\u0060 and \u0060libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/kotlin/schema/EnumAsSealedInterfaceBuilder.kt\u0060 as WHOLE FILES: this scan already matched 3 separate duplicated blocks between them, totalling at least 37 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. Read the line range as the matched WINDOW rather than a finished unit: at \u0060libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/kotlin/schema/EnumAsApolloEnumBuilder.kt:152\u0060 it begins part-way through the construct above it, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/kotlin/schema/EnumAsApolloEnumBuilder.kt"},"region":{"startLine":152}}}],"partialFingerprints":{"codehealthFindingId/v1":"e79ee5c20bf8c95e4156f56dd09f5e12c2dbdbeb83e18196cdee513d19493068"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (12 lines \u00D7 2): libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/kotlin/helpers/Collections.kt:16-27 | libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/kotlin/helpers/Collections.kt:37-48 \u2014 both copies are in the same file, so extract the block into one function there and call it from each site \u2014 the copies drift apart the first time only one of them is edited. Read the line range as the matched WINDOW rather than a finished unit: at \u0060libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/kotlin/helpers/Collections.kt:16\u0060 it begins part-way through the construct above it, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/kotlin/helpers/Collections.kt"},"region":{"startLine":16}}}],"partialFingerprints":{"codehealthFindingId/v1":"54fd53d0c7cade01cfa693427e3c86f90e7729d8a66eeef342490747f4be75e5"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (11 lines \u00D7 2): libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/java/helpers/AdapterCommon.kt:86-96 | libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/kotlin/helpers/AdapterCommon.kt:58-68 \u2014 before extracting anything, compare \u0060libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/java/helpers/AdapterCommon.kt\u0060 and \u0060libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/kotlin/helpers/AdapterCommon.kt\u0060 as WHOLE FILES: this scan already matched 3 separate duplicated blocks between them, totalling at least 43 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. Read the line range as the matched WINDOW rather than a finished unit: at \u0060libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/java/helpers/AdapterCommon.kt:86\u0060 it does not close everything it opens, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that. Note first that the copies are not typed on the same thing: \u0060CodeBlock\u0060 names \u0060com.squareup.javapoet.CodeBlock\u0060 in one and \u0060com.squareup.kotlinpoet.CodeBlock\u0060 in another \u2014 different types that share a simple name, which is why the text matched. A single extracted unit cannot be given a parameter type that fits both, so unifying those types (or introducing a shared abstraction over them) is the step that has to come BEFORE the extraction above; if they are deliberately separate, the duplication is the price of that separation and the honest resolution is to record the decision rather than to extract. \u2605 These copies have DRIFTED, and that is worth reading before extracting anything: just after the matched lines, \u0060libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/java/helpers/AdapterCommon.kt:97\u0060 calls \u0060add\u0060 and \u0060libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/kotlin/helpers/AdapterCommon.kt:69\u0060 does not \u2014 after which the two agree again for 4 more lines. One of those two behaviours is the intended one and the other is what a copy-paste left behind, so decide which BEFORE unifying them: extracting the shared part will silently settle it, and if the copy that skips the call is the wrong one, that bug is already live."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/java/helpers/AdapterCommon.kt"},"region":{"startLine":86}}}],"partialFingerprints":{"codehealthFindingId/v1":"67e76daa4e4fe3ddb8876e736ae7c9d5e69324203ecced81f62a150445e2d2a7"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (8 lines \u00D7 2): libraries/apollo-gradle-plugin/src/main/kotlin/com/apollographql/apollo/gradle/internal/DefaultApolloExtension.kt:641-648 | libraries/apollo-gradle-plugin/src/main/kotlin/com/apollographql/apollo/gradle/internal/DefaultApolloExtension.kt:665-672 \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":"libraries/apollo-gradle-plugin/src/main/kotlin/com/apollographql/apollo/gradle/internal/DefaultApolloExtension.kt"},"region":{"startLine":641}}}],"partialFingerprints":{"codehealthFindingId/v1":"c9d367f071bcd499af5d3e8e0fb22551784efe9d65142cba6ca540c0f2429baa"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (5 lines \u00D7 2): libraries/apollo-ast/src/commonMain/kotlin/com/apollographql/apollo/ast/internal/SchemaValidationScope.kt:187-191 | libraries/apollo-ast/src/commonMain/kotlin/com/apollographql/apollo/ast/internal/SchemaValidationScope.kt:200-204 \u2014 both copies are in the same file, so extract the block into one function there and call it from each site \u2014 the copies drift apart the first time only one of them is edited. Read the line range as the matched WINDOW rather than a finished unit: at \u0060libraries/apollo-ast/src/commonMain/kotlin/com/apollographql/apollo/ast/internal/SchemaValidationScope.kt:187\u0060 it does not close everything it opens, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"libraries/apollo-ast/src/commonMain/kotlin/com/apollographql/apollo/ast/internal/SchemaValidationScope.kt"},"region":{"startLine":187}}}],"partialFingerprints":{"codehealthFindingId/v1":"3d9fff1eb9e80d2c2e46f288e4f326d20c229775b28df266d2d7ed1603d7b028"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (13 lines \u00D7 2): libraries/apollo-testing-support/src/commonMain/kotlin/com/apollographql/apollo/testing/TestNetworkTransport.kt:38-50 | libraries/apollo-testing-support/src/commonMain/kotlin/com/apollographql/apollo/testing/TestNetworkTransport.kt:83-95 \u2014 both copies are in the same file, so extract the block into one function there and call it from each site \u2014 the copies drift apart the first time only one of them is edited. Read the line range as the matched WINDOW rather than a finished unit: at \u0060libraries/apollo-testing-support/src/commonMain/kotlin/com/apollographql/apollo/testing/TestNetworkTransport.kt:38\u0060 it begins part-way through the construct above it, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"libraries/apollo-testing-support/src/commonMain/kotlin/com/apollographql/apollo/testing/TestNetworkTransport.kt"},"region":{"startLine":38}}}],"partialFingerprints":{"codehealthFindingId/v1":"ce2ef2b993b01753934288fd16bfb5ae4c694e74a3dda2201ff4dc4ad06ebdfb"}},{"ruleId":"D5","level":"warning","message":{"text":"Off the main sequence: multi-modules-custom-scalar:root: multi-modules-custom-scalar:root: abstractness 0.00, instability 0.00, distance 1.00 \u2014 zone of pain \u2014 concrete and depended on by 1 project(s), so it\u0027s rigid to change."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"a8d10615bcb2d9a268b6b63fa9a41197e2100ac6390d79774ac89a7db2dbecb3"}},{"ruleId":"D6","level":"warning","message":{"text":"Low cohesion: DefaultApolloCompilerRegistry (LCOM4 8): DefaultApolloCompilerRegistry\u0027s methods fall into 8 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 8 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":"libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/internal/DefaultApolloCompilerRegistry.kt"},"region":{"startLine":66}}}],"partialFingerprints":{"codehealthFindingId/v1":"fd3bfb9c996f92ff5b3b9c11cac1d34e02e1f2a1130089eaa3a32d4e6ee79d45"}},{"ruleId":"D13","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":"60db318291d38d4bfbb98d84bf8e6440d2d858eeb04919f3025ec0ae97d313e4"},"taxa":[{"id":"CWE-259","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-798","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D15","level":"warning","message":{"text":"Hotspot: libraries/apollo-gradle-plugin/src/main/kotlin/com/apollographql/apollo/gradle/internal/DefaultApolloExtension.kt: libraries/apollo-gradle-plugin/src/main/kotlin/com/apollographql/apollo/gradle/internal/DefaultApolloExtension.kt changed 2 times in last 90 days, and the most complex body those changes touched has cyclomatic complexity 20 in DefaultApolloExtension.registerService at line 407. Frequent change and high complexity in one file compound: schedule the next change to it to include carving out the part being edited, with the area under test before it moves. Counted over 2026-06-27..2026-09-25, the 90 days ending at the analysed commit. Reproduce with \u0060git log --since=\u00272026-06-27 11:17:15 \u002B02:00\u0027 --until=\u00272026-09-25 11:17:15 \u002B02:00\u0027 --full-history --no-merges -- libraries/apollo-gradle-plugin/src/main/kotlin/com/apollographql/apollo/gradle/internal/DefaultApolloExtension.kt\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":"libraries/apollo-gradle-plugin/src/main/kotlin/com/apollographql/apollo/gradle/internal/DefaultApolloExtension.kt"},"region":{"startLine":407}}}],"partialFingerprints":{"codehealthFindingId/v1":"bb0e23a5905f3b662ce91ffd27acf5e3e2ae1def69001b1badd748b2bf47c748"}},{"ruleId":"D16","level":"note","message":{"text":"Off-boarding risk: anonymized user #1: If anonymized user #1 becomes unavailable, 25 significant file(s) lose their only recent owner: libraries/apollo-gradle-plugin/src/main/kotlin/com/apollographql/apollo/gradle/api/Service.kt, libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/kotlin/KotlinCodegen.kt, libraries/apollo-execution/src/commonMain/kotlin/com/apollographql/apollo/execution/ExecutableSchema.kt, libraries/apollo-runtime/src/commonMain/kotlin/com/apollographql/apollo/network/websocket/internal/SubscribableWebSocket.kt, libraries/apollo-tooling/src/main/kotlin/com/apollographql/apollo/tooling/SchemaDownloader.kt, libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/internal/ApolloExecutableDocumentTransform.kt, libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/entrypoints.kt, libraries/apollo-gradle-plugin/build.gradle.kts (\u002B17 more). Pair on, review, or document these before any departure."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"fa65889e8a64a6bb4bddf7a5144e24d8ef1bd353bbc33ec7e86407984b5abf0f"}},{"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: libraries/apollo-ast/src/commonMain/kotlin/com/apollographql/apollo/ast/internal/validation/defer_labels.kt, libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/internal/serialization.kt, libraries/apollo-api/src/commonMain/kotlin/com/apollographql/apollo/api/Executables.kt, libraries/apollo-api/src/commonMain/kotlin/com/apollographql/apollo/api/json/JsonReaders.kt. Pair on, review, or document these before any departure."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"2fd9a785372266b1e76beadf0aba95cfd6922acadb0d65fcc2063e658f86b872"}},{"ruleId":"D16","level":"note","message":{"text":"Further sole-owners (lower concentration): 1 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 (30 single-owned of 254 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; 254 of the 735 production source files in this repository met that bar). They are anonymized user #3 (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":"XxxComment: // XXX: this potentially writes null instead of an error \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":"libraries/apollo-api/src/commonMain/kotlin/com/apollographql/apollo/api/Adapters.kt"},"region":{"startLine":461}}}],"partialFingerprints":{"codehealthFindingId/v1":"540a82fe7d14d5da8499c984e77f69b607c560ec5efbff634816fbd65a4f7461"}},{"ruleId":"D17","level":"warning","message":{"text":"XxxComment: // XXX probably wrong if there is a surrogate pair \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":"libraries/apollo-api/src/commonMain/kotlin/com/apollographql/apollo/api/http/internal/UrlEncode.kt"},"region":{"startLine":70}}}],"partialFingerprints":{"codehealthFindingId/v1":"c7a63694cbb7e77029f659533318926ba0a330c189fccd2899407c21b057fbaf"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO(\u0022support variable directives\u0022) \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":"libraries/apollo-ast/src/commonMain/kotlin/com/apollographql/apollo/ast/gql.kt"},"region":{"startLine":1618}}}],"partialFingerprints":{"codehealthFindingId/v1":"63dad62037a319862cc9111e5b541a732ee8a0ee71a359371e357b034d16cba5"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: support lists of sourceLocation for redefinitions \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":"libraries/apollo-ast/src/commonMain/kotlin/com/apollographql/apollo/ast/exception.kt"},"region":{"startLine":10}}}],"partialFingerprints":{"codehealthFindingId/v1":"1a2ac014c573a7ae4167b20d7641ad643abfcfa006243c81ec5bf951254539da"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: support multiple sourceLocations. A single issue like a redefinition for an example, might have several impacted sourceLocations. \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":"libraries/apollo-ast/src/commonMain/kotlin/com/apollographql/apollo/ast/Issue.kt"},"region":{"startLine":6}}}],"partialFingerprints":{"codehealthFindingId/v1":"de1dc8e19b985ab4af3456a5a5adc1d0d3f171f9d54191abae3ec00605331738"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO only register a single issue once we have issues that can support several source locations \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":"libraries/apollo-ast/src/commonMain/kotlin/com/apollographql/apollo/ast/internal/fields_merging.kt"},"region":{"startLine":36}}}],"partialFingerprints":{"codehealthFindingId/v1":"6a918d57fc33b8e6f6c466c6b239eb3bb1c6732e75abee84ace35ebbef18e733"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO only register a single issue once we have issues that can support several source locations \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":"libraries/apollo-ast/src/commonMain/kotlin/com/apollographql/apollo/ast/internal/fields_merging.kt"},"region":{"startLine":46}}}],"partialFingerprints":{"codehealthFindingId/v1":"eddf1b6ebb1525db0c3acdd74567df6f38df05cb0e593867ac66035fa337c144"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: any validation to do 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":"libraries/apollo-ast/src/commonMain/kotlin/com/apollographql/apollo/ast/internal/SchemaValidationScope.kt"},"region":{"startLine":153}}}],"partialFingerprints":{"codehealthFindingId/v1":"c809de337e9992f478bd375eee71ed20452831467c424b9487ad6601e0eaa47e"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: we are lenient here and allow potentially invalid chars like invalid surrogate pairs \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":"libraries/apollo-ast/src/commonMain/kotlin/com/apollographql/apollo/ast/internal/Lexer.kt"},"region":{"startLine":333}}}],"partialFingerprints":{"codehealthFindingId/v1":"71e8d841dc55c1632a04f3f0e02a181f906b91b82a8004d5100a289ac54a9fb0"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: we are lenient here and allow potentially invalid chars like invalid surrogate pairs \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":"libraries/apollo-ast/src/commonMain/kotlin/com/apollographql/apollo/ast/internal/Lexer.kt"},"region":{"startLine":406}}}],"partialFingerprints":{"codehealthFindingId/v1":"4f2cca642fae813d65931a0caab732e19c31dce65f1511beda5bfd9f9f60c353"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Can object literals have duplicate fields? \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":"libraries/apollo-ast/src/commonMain/kotlin/com/apollographql/apollo/ast/internal/ExtensionsMerger.kt"},"region":{"startLine":533}}}],"partialFingerprints":{"codehealthFindingId/v1":"f249481e38bf84ab066723156b0bf5b363f9e9980be2af0550c4f69fc98bd709"}},{"ruleId":"D17","level":"warning","message":{"text":"XxxComment: // XXX: we could relax the validation here as different inline fragments on different type \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":"libraries/apollo-ast/src/commonMain/kotlin/com/apollographql/apollo/ast/internal/ExecutableValidationScope.kt"},"region":{"startLine":659}}}],"partialFingerprints":{"codehealthFindingId/v1":"56c5ecdeff33fced8d7059d7b20f3fba70a22d025a88baebcfd01a104ce858e1"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO \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":"libraries/apollo-ast/src/commonTest/kotlin/com/apollographql/apollo/graphql/ast/test/LexerTest.kt"},"region":{"startLine":238}}}],"partialFingerprints":{"codehealthFindingId/v1":"5ed0fac12530600d862d9d5276024e80bd2f11f1ab962f9739284778b04392cd"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO \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":"libraries/apollo-ast/src/commonTest/kotlin/com/apollographql/apollo/graphql/ast/test/LexerTest.kt"},"region":{"startLine":455}}}],"partialFingerprints":{"codehealthFindingId/v1":"9176c58bb00c496e002c016def3c91a6c95c1d5f80e9f458e6f679bf4e618145"}},{"ruleId":"D17","level":"warning","message":{"text":"XxxComment: // XXX: use a streaming API when they are not experimental anymore \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":"libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/json.kt"},"region":{"startLine":32}}}],"partialFingerprints":{"codehealthFindingId/v1":"7597eeda55d7ca31279d8d9e4f69d2eb9f7d2297fde4f7869f118e6388ddec8e"}},{"ruleId":"D17","level":"warning","message":{"text":"XxxComment: // XXX: use a streaming API when they are not experimental anymore \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":"libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/json.kt"},"region":{"startLine":37}}}],"partialFingerprints":{"codehealthFindingId/v1":"84541f6465c2a4f8b123d4232be4ca195dfea48d71fd791bec2692ceeb733090"}},{"ruleId":"D17","level":"warning","message":{"text":"XxxComment: // XXX: use a streaming API when they are not experimental anymore \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":"libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/json.kt"},"region":{"startLine":91}}}],"partialFingerprints":{"codehealthFindingId/v1":"48909dd9cca9b178daf972b6706a63aee29a9f5b51dcf78474ad4420db58583f"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: allow the user to override the severities for schema validation \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":"libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/ApolloCompiler.kt"},"region":{"startLine":214}}}],"partialFingerprints":{"codehealthFindingId/v1":"269c273a55254938af7e13e2234817cf9172fa08cd57fdec2a4a3bbb71b95c27"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Is Nothing correct 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":"libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/kotlin/helpers/Value.kt"},"region":{"startLine":18}}}],"partialFingerprints":{"codehealthFindingId/v1":"5e77af90aad6cd1b287322bb398439da499b81ed961c717ffd8c0f7581c00bad"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Is Nothing correct 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":"libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/kotlin/helpers/Value.kt"},"region":{"startLine":27}}}],"partialFingerprints":{"codehealthFindingId/v1":"fa6cb929ed6127b6e1c04369c21ed0257aea15586cd8d0ba97e854853eec7ed1"}},{"ruleId":"D17","level":"warning","message":{"text":"XxxComment: // XXX: can an enum be made deprecated (and not only its values) ? \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":"libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/codegen/kotlin/schema/EnumAsSealedInterfaceBuilder.kt"},"region":{"startLine":60}}}],"partialFingerprints":{"codehealthFindingId/v1":"b3f74f60fe10df58b86897d4b67da92b903cb9bb79de9a997194f4aa47796d4e"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: restrict the possible types to the possible types in the context of this selection \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":"libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/ir/SelectionSetsBuilder.kt"},"region":{"startLine":115}}}],"partialFingerprints":{"codehealthFindingId/v1":"935a5f3e41e0320f2d78b421e9b07d650f7d9c767ef3bdf8bedf556de7f0e6a7"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: restrict the possible types to the possible types in the context of this selection \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":"libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/ir/SelectionSetsBuilder.kt"},"region":{"startLine":136}}}],"partialFingerprints":{"codehealthFindingId/v1":"97468e762e77cf3e1e0f247ea1729cc72f36bdd7fdb6a0fd6a1f5d41cdaec089"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: cache this computation \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":"libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/ir/OperationBasedWithInterfacesModelGroupBuilder.kt"},"region":{"startLine":320}}}],"partialFingerprints":{"codehealthFindingId/v1":"45aedb66e7b947a9d994968b07dca5e3467c9a4b159cd56706a4c4c5c751bf2a"}},{"ruleId":"D17","level":"warning","message":{"text":"XxxComment: // XXX: this is not spec-compliant. Directive cannot be on union definitions \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":"libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/ir/IrSchema.kt"},"region":{"startLine":175}}}],"partialFingerprints":{"codehealthFindingId/v1":"278988e45cadfd021895342f394e5af25adc8908f5257f7e0f4bccbdee97d277"}},{"ruleId":"D17","level":"warning","message":{"text":"XxxComment: // XXX: this is not spec-compliant. Scalar definitions do not have deprecation \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":"libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/ir/IrSchema.kt"},"region":{"startLine":191}}}],"partialFingerprints":{"codehealthFindingId/v1":"5504e46b6bea1223c6bdfad3390ebe175d7e8b70b26d3857b6ea86021503dd2e"}},{"ruleId":"D17","level":"warning","message":{"text":"XxxComment: // XXX: this is not spec-compliant. Directive cannot be on input objects definitions \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":"libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/ir/IrSchema.kt"},"region":{"startLine":293}}}],"partialFingerprints":{"codehealthFindingId/v1":"fec7e8fe56631503ce2794c24698dce774c729bf18839b7ba83b591520c9ea3d"}},{"ruleId":"D17","level":"warning","message":{"text":"XxxComment: // XXX: this is not spec-compliant. Deprecation directives cannot be on interfaces \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":"libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/ir/IrSchema.kt"},"region":{"startLine":306}}}],"partialFingerprints":{"codehealthFindingId/v1":"859c0b86f99b9e636241bb5a16da093fa6ed9959070f5fe5a7a2fd99a8954e3a"}},{"ruleId":"D17","level":"warning","message":{"text":"XxxComment: // XXX: this is not spec-compliant. Directive cannot be on object definitions \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":"libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/ir/IrSchema.kt"},"region":{"startLine":325}}}],"partialFingerprints":{"codehealthFindingId/v1":"adfa3ff53c60e40a5065997e42a5bfb5fc0bf50befdb6188a3ac101a7cb89727"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: move to a validation rule \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":"libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/ir/IrOperationsBuilder.kt"},"region":{"startLine":522}}}],"partialFingerprints":{"codehealthFindingId/v1":"efb75fcb8de30b43ede599d7723ecd36cb77a0d409a6ba3693b94427475eee9d"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: should this fail instead? \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":"libraries/apollo-execution/src/commonMain/kotlin/com/apollographql/apollo/execution/internal/coerceArgumentValues.kt"},"region":{"startLine":184}}}],"partialFingerprints":{"codehealthFindingId/v1":"3305258060efe89d9401f1d7db9b2b6f5a7443fb31137bd4e9b230b9f1f670f6"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: allow implementers to terminate the stream with an exception \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":"libraries/apollo-execution/src/commonMain/kotlin/com/apollographql/apollo/execution/internal/OperationContext.kt"},"region":{"startLine":257}}}],"partialFingerprints":{"codehealthFindingId/v1":"381077a8304755e6452f3b8e796c3ca24ada3875df8171bac9a7234717a08fd9"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: remove dispose \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":"libraries/apollo-runtime/src/commonMain/kotlin/com/apollographql/apollo/network/http/HttpInterceptor.kt"},"region":{"startLine":29}}}],"partialFingerprints":{"codehealthFindingId/v1":"f2c460ec1cd694854aa5e6b3e605534d2afe6e824244a3b29c5b45fbc0ffde81"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: this is most likely going to transform BigNumbers into strings, not sure how much of an issue that is \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":"libraries/apollo-runtime/src/commonMain/kotlin/com/apollographql/apollo/network/http/BatchingHttpInterceptor.kt"},"region":{"startLine":201}}}],"partialFingerprints":{"codehealthFindingId/v1":"c9989ed9d176aa025cca55e3bffc7c65335f966c809a45d238ba7cd3fd916e90"}},{"ruleId":"D17","level":"warning","message":{"text":"XxxComment: // XXX: we could be smarter about this (the parsers are \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":"tests/data-builders-kotlin/src/test/kotlin/test/FragmentTest.kt"},"region":{"startLine":56}}}],"partialFingerprints":{"codehealthFindingId/v1":"62531666e7829498957cfec66de52c033aadcbd3a6537bec28489066715d366c"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO Ignored for now, Router doesn\u0027t return the error - see https://github.com/apollographql/router/issues/2329"},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"tests/defer/src/commonTest/kotlin/test/DeferWithRouterTest.kt"},"region":{"startLine":313}}}],"partialFingerprints":{"codehealthFindingId/v1":"4f275d20e13aa14d7342b03b25e5d7fce032e5f5011dd97f7749b8731695e74e"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO For now chunked is not supported on JS - remove this check when it is \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":"tests/defer/src/commonTest/kotlin/test/DeferV0_2Test.kt"},"region":{"startLine":190}}}],"partialFingerprints":{"codehealthFindingId/v1":"4c27bf5a9aa2591a10acaf42da0218d09f204e14549adcb6f54969d56f0d3890"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO For now chunked is not supported on JS - remove this check when it is \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":"tests/defer/src/commonTest/kotlin/test/DeferV0_2NormalizedCacheTest.kt"},"region":{"startLine":567}}}],"partialFingerprints":{"codehealthFindingId/v1":"855245a489ee453dea25f17da35c59cf6a3754f5bfffe4a28836067417d0a7a3"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO For now chunked is not supported on JS - remove this check when it is \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":"tests/defer/src/commonTest/kotlin/test/DeferV0_1Test.kt"},"region":{"startLine":258}}}],"partialFingerprints":{"codehealthFindingId/v1":"f8deed97125d0305305ea022bdb5653af2aae8a1b4ce449ea19191c26a017696"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO For now chunked is not supported on JS - remove this check when it is \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":"tests/defer/src/commonTest/kotlin/test/DeferV0_1NormalizedCacheTest.kt"},"region":{"startLine":469}}}],"partialFingerprints":{"codehealthFindingId/v1":"9c0e8725f9fcc28cb115e325d0cc96ddfa59e05606caaedbbffac73032180f7d"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Error doesn\u0027t use @JsExport \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":"tests/jsexport/src/jsTest/kotlin/JsExportTest.js.kt"},"region":{"startLine":29}}}],"partialFingerprints":{"codehealthFindingId/v1":"b746abd865bb46b3d48fe89f1157cf73057b10dd08bd32be268d38bd5984e1e9"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: not sure why this needs to be nullable \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":"tests/sample-server/src/main/kotlin/com/apollographql/apollo/sample/server/graphql/SubscriptionRoot.kt"},"region":{"startLine":78}}}],"partialFingerprints":{"codehealthFindingId/v1":"02912c9b362c3430e22b13e128fc3b94fe591e7a1ba92569731c67379be03c7c"}},{"ruleId":"D19","level":"note","message":{"text":"Documentation: no installation or build instructions: The README describes a standalone Android Studio benchmark project but provides no build/run commands and no link to the deployed documentation. Add installation steps (gradle tasks) and a link to the Apollo docs site so readers can run it locally."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"benchmark/README.md"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"f2f23cf174bd40779ddd2c7d31213652239f83c037841e26c60b88f0c624b499"}},{"ruleId":"D19","level":"note","message":{"text":"Documentation: no usage examples: The README states how to run tests but does not show any actual commands or results. Add a shell block showing both IDE and command-line builds with output, then link the Apollo docs for further usage."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"benchmark/README.md"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"1efff367ea2f29a72a6ff8537a8c2b9b63a59ee83553c88e0b2a2c0e8a0ec108"}},{"ruleId":"D20","level":"warning","message":{"text":"Consequences/trade-offs (e.g. higher context-switch cost for plumbing) and whether coroutines are actually used beyond the interceptor rationale are not visible before the clip: "},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"design-docs/Threading.md"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"95a7f9c5aa3adb75be629019cf15b7da1cb1080fad751141349336a2e1fdf1fe"}},{"ruleId":"D26","level":"note","message":{"text":"Split libraries/apollo-api: A huge Apollo API project whose namespaces are all sub-apis (http/json/apple/wasm/js) and the public types sit overwhelmingly in one namespace. Suggested: split into Apollo HTTP, JSON, JS, WASM, Apple, and wasmJs namespaces"},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"b032677615dda3337e7b3bf2883d758a528c5400c4a8ba59555a301d4e0c10bf"}},{"ruleId":"D26","level":"note","message":{"text":"Split libraries/apollo-ast: A massive Apollo AST project whose namespaces are all sub-packages of the same core (ast/introspection/codegen/ir) and the public types sit overwhelmingly in one namespace. Suggested: split into Apollo AST IR, introspection, codegen/java/helpers/kotlin/schema/operations/util"},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"8de1e2fa92665b993db0f0c2c86e9ec2926ed725b70c706482e9ab0306fd3bbc"}},{"ruleId":"D26","level":"note","message":{"text":"Split libraries/apollo-compiler: A sprawling Apollo compiler project whose namespaces are all sub-packages of the same core (ir/compiler/codegen) and the public types sit overwhelmingly in one namespace. Suggested: split into Apollo IR, compiler codegen/java/kotlin/schema/operations/util"},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"7eb622c58fa1788f7690879924e455271b50739907ad71a582e07f7186095902"}},{"ruleId":"D26","level":"note","message":{"text":"Split libraries/apollo-runtime: A huge Apollo runtime project whose namespaces are all sub-areas of the same core (network/websocket/http) and the public types sit overwhelmingly in one namespace. Suggested: split into Apollo network HTTP/WebSocket/WS internals"},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"d458e4f591396e203b0db68c174dc16532c49b6ec0d6835bd0ad93b8115bd57b"}},{"ruleId":"D28","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":"a0a5112dd350305af95ec567f3119df525f9eb4d20457e50840df556ccc074ea"},"taxa":[{"id":"CWE-321","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-798","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":"ff8f8334e77776aefaaba8bd1abda5dc87556c0b4b68f4899b59caa7d9e8f366"},"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":"f43a874423f4768c9141fc9071510fc2e6090bd48b9f3bdb3aff6e0806e5578b"},"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":"fd7f5fa5af8b505cb9aafdb80c336c5926a94e1ea3084545fecdb2717f5c3bb5"},"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":"b497d855908d513555a3bc2702d5164b73bb334e7d93af7982ea3fe8abfef3b6"},"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":"b068fd2a344cf2b6f4239c878326e29df8d40802e8202086293fafd909c57881"},"taxa":[{"id":"CWE-732","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":"51216ce9338c6f4d403d758a88d4da4ae33ab97bbe7d891d86319bde8952ad71"},"properties":{"dependency":{"package":"minimatch","version":"9.0.5","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":"3a79b801a13832b293c122e885b65c4f136c1c80c6655442c529373df4ff56bf"},"properties":{"dependency":{"package":"js-yaml","version":"4.1.1","advisory":"[GHSA redacted]","aliases":["[CVE redacted]","[CVE redacted]","[CVE redacted]","[GHSA 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":"2745a10f288dc5ecbb986da0de4ecfc31ddfd20a9c1c21b0c79a0d67a8bdcc3e"},"properties":{"dependency":{"package":"brace-expansion","version":"2.0.2","advisory":"[GHSA redacted]","aliases":["[CVE redacted]","[CVE redacted]","[CVE redacted]","[CVE redacted]","[GHSA 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":"341dfd795eb53f4b988b890644e88af47150d151c75c930a87f588b3196f1396"},"properties":{"dependency":{"package":"ws","version":"8.18.3","advisory":"[GHSA redacted]","aliases":["[CVE redacted]","[CVE 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":"653299697e002141b748581b859bdd4945a7cedea06c426af9198625e7cbea26"},"properties":{"dependency":{"package":"serialize-javascript","version":"6.0.2","advisory":"[GHSA redacted]","aliases":["[CVE redacted]","[GHSA redacted]"],"reachability":{"kind":"unknown"}}}},{"ruleId":"D30","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":"eee6dfa6c47e8a6889935551e7db7d12f08f58a65cc8135cd129b7cf5fbba7cf"},"properties":{"dependency":{"package":"diff","version":"7.0.0","advisory":"[GHSA redacted]","aliases":["[CVE redacted]"],"reachability":{"kind":"unknown"}}}},{"ruleId":"D34","level":"note","message":{"text":"Orphaned files with no living knowledge: 7 of 254 analysed file(s) have no living knowledge left \u2014 their last meaningful change has decayed away, so if one breaks, no one currently understands it (counted over production source files of roughly 2,400 bytes or more, excluding vendored, generated and example/demo trees and test files identified by path convention, largest first; 254 of the 735 production source files in this repository met that bar). None is large enough to earn a read-through of its own, so this row stands in for the per-file rows rather than raising one each \u2014 most significant first: libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/ir/OperationBasedWithInterfacesModelGroupBuilder.kt, libraries/apollo-api/src/jsMain/kotlin/com/apollographql/apollo/api/json/DynamicJsJsonReader.kt, libraries/apollo-runtime/src/wasmJsMain/kotlin/com/apollographql/apollo/network/websocket/WebSocketEngine.wasmJs.kt, libraries/apollo-runtime/src/jsMain/kotlin/com/apollographql/apollo/network/ws/JsWebSocketEngine.kt, libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/ir/buckets.kt, libraries/apollo-api/src/commonMain/kotlin/com/apollographql/apollo/api/ExecutionContext.kt, libraries/apollo-api/src/commonMain/kotlin/com/apollographql/apollo/api/json/internal/JsonScope.kt. 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: DefaultApolloExtension.kt \u2194 ModelNames.kt: \u0060libraries/apollo-gradle-plugin/src/main/kotlin/com/apollographql/apollo/gradle/internal/DefaultApolloExtension.kt\u0060 and \u0060libraries/apollo-gradle-plugin/src/main/kotlin/com/apollographql/apollo/gradle/internal/ModelNames.kt\u0060 change together 90% of the time (9 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). They sit in the same directory, and in this ecosystem sibling files there normally share one namespace/package \u2014 so a direct reference between them needs no import and this pass cannot see whether one exists. Read the pair before acting: if one file only DECLARES what the other consumes (a constants/types file beside its user), the co-change is definitional and the question is whether the split earns its keep; if they duplicate structure, extract the common part into a shared function or type they both call; if neither holds, the coupling is hidden and worth breaking. You can check this without leaving the row: of the 9 shared commits counted here, the most recent 3 are \u00601c58f413\u0060 Introduce Service.pluginsArguments and relax the check for multiple p\u2026; \u00609628c004\u0060 [gradle-plugin] Remove checkApolloVersion (#6569); \u0060a00dc1b9\u0060 expose regular configurations for multi-module projects (#6408) (at that commit the files were still \u0060libraries/apollo-gradle-plugin-external/src/main/kotlin/com/apollographql/apollo/gradle/internal/DefaultApolloExtension.kt\u0060 and \u0060libraries/apollo-gradle-plugin-external/src/main/kotlin/com/apollographql/apollo/gradle/internal/ModelNames.kt\u0060) \u2014 run \u0060git show\u0060 on any of them."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"libraries/apollo-gradle-plugin/src/main/kotlin/com/apollographql/apollo/gradle/internal/DefaultApolloExtension.kt"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"926f3b6d55736fd435cbf01489b683d6beda47b791736000c503ac0e49e65f30"}},{"ruleId":"D35","level":"warning","message":{"text":"Change coupling: SchemaDownloader.kt \u2194 SchemaHelper.kt: \u0060libraries/apollo-tooling/src/main/kotlin/com/apollographql/apollo/tooling/SchemaDownloader.kt\u0060 and \u0060libraries/apollo-tooling/src/main/kotlin/com/apollographql/apollo/tooling/SchemaHelper.kt\u0060 change together 58% of the time (7 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). They sit in the same directory, and in this ecosystem sibling files there normally share one namespace/package \u2014 so a direct reference between them needs no import and this pass cannot see whether one exists. Read the pair before acting: if one file only DECLARES what the other consumes (a constants/types file beside its user), the co-change is definitional and the question is whether the split earns its keep; if they duplicate structure, extract the common part into a shared function or type they both call; if neither holds, the coupling is hidden and worth breaking. You can check this without leaving the row: of the 7 shared commits counted here, the most recent 3 are \u0060dad74b0c\u0060 Workaround introspection for servers that do not support isOneOf desp\u2026; \u006039a2669f\u0060 Allow to shutdown SchemaDownloader (#6091); \u0060c40bcf87\u0060 Implement 2-step introspection (#5371) (at that commit the files were still \u0060libraries/apollo-tooling/src/main/kotlin/com/apollographql/apollo3/tooling/SchemaDownloader.kt\u0060 and \u0060libraries/apollo-tooling/src/main/kotlin/com/apollographql/apollo3/tooling/SchemaHelper.kt\u0060) \u2014 run \u0060git show\u0060 on any of them."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"libraries/apollo-tooling/src/main/kotlin/com/apollographql/apollo/tooling/SchemaDownloader.kt"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"f9b18dd4d71c39ebe3027fcdc80743959afc533eff293abe083ad7e386e95122"}},{"ruleId":"D35","level":"warning","message":{"text":"Change coupling: gqldocument.kt \u2194 SchemaValidationScope.kt: \u0060libraries/apollo-ast/src/commonMain/kotlin/com/apollographql/apollo/ast/gqldocument.kt\u0060 and \u0060libraries/apollo-ast/src/commonMain/kotlin/com/apollographql/apollo/ast/internal/SchemaValidationScope.kt\u0060 change together 52% of the time (16 of the 31 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 16 shared commits counted here, the most recent 3 are \u00601c54e7c8\u0060 Implement field extensions (#6856); \u00606a300660\u0060 Exclude auto-imported cache related directives when the Apollo cache \u2026; \u00609822b20d\u0060 Add allowAddingDirectivesToExistingFieldDefinitions (#6470) \u2014 run \u0060git show\u0060 on any of them."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"libraries/apollo-ast/src/commonMain/kotlin/com/apollographql/apollo/ast/gqldocument.kt"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"501e87c1231ab832c30e21a0aac07bb5e4591d8a66f9ced4c1fbec1e569a0404"}},{"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":"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":"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":"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":"M4","level":"note","message":{"text":"README/code drift: README omits the Apollo Server project (subgraph-computers) \u2014 reported by the model that read the README against this repository; no term search was run for this one, so nothing here has been checked against the tree. Treat it as a reading to confirm, not as a measured contradiction: verify it against the code before acting on it, and if the footprint it describes does exist, this row is wrong."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"8e812c9ab3084d2a72553fcafb2279fd3c4b2b363d1802f8312d9642a140ae50"}},{"ruleId":"M4","level":"note","message":{"text":"README/code drift: README omits the Apollo Router project (subgraph-computers) \u2014 reported by the model that read the README against this repository; no term search was run for this one, so nothing here has been checked against the tree. Treat it as a reading to confirm, not as a measured contradiction: verify it against the code before acting on it, and if the footprint it describes does exist, this row is wrong."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"a5a519f21f27f5eb7ee6b250abf32e4900457c45115acdfaedd1021ebdd08823"}},{"ruleId":"P2","level":"note","message":{"text":"Logging is not universal: Only 1/2 runnable modules use logging (modules with no entry point or server are excluded \u2014 they are libraries a runnable module hosts). Silent: \u0060build-logic\u0060."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"91a94e21d6d4d0e6a2003f94505b0b02b157176f0b24c4820f3f82b4447a97fe"}},{"ruleId":"P7","level":"warning","message":{"text":"Outbound HTTP without resilience: \u0060HttpClient(\u0060 makes an outbound HTTP call, and nothing bounds it: no timeout, deadline, retry or circuit breaker is set for it here, and the client has no process-wide default. A slow or failing dependency will hold this service\u0027s request, thread or connection until the call gives up on its own \u2014 or never, for a client with no default timeout."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"libraries/apollo-runtime/src/linuxMain/kotlin/com/apollographql/apollo/network/websocket/WebSocketEngine.linux.kt"},"region":{"startLine":33}}}],"partialFingerprints":{"codehealthFindingId/v1":"0bc22df354b18b1861f406faed6c65f83b3e951f59e2186baffa2610491dce0a"}}],"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-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-321","guid":"0ce564a5-a44c-3358-b3db-317712e6e019","name":"CWE-321","shortDescription":{"text":"CWE-321"},"helpUri":"https://cwe.mitre.org/data/definitions/321.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-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-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":18,"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."]}]}