{"$schema":"https://json.schemastore.org/sarif-2.1.0.json","version":"2.1.0","runs":[{"tool":{"driver":{"name":"codehealth","informationUri":"https://codehealth.canine.dev","rules":[{"id":"D1","name":"Cyclomatic Complexity","shortDescription":{"text":"Cyclomatic Complexity"},"helpUri":"https://codehealth.canine.dev/dimensions/D1"},{"id":"D2","name":"Cognitive Complexity","shortDescription":{"text":"Cognitive Complexity"},"helpUri":"https://codehealth.canine.dev/dimensions/D2"},{"id":"D3","name":"God Classes","shortDescription":{"text":"God Classes"},"helpUri":"https://codehealth.canine.dev/dimensions/D3"},{"id":"D4","name":"Code Duplication","shortDescription":{"text":"Code Duplication"},"helpUri":"https://codehealth.canine.dev/dimensions/D4"},{"id":"D5","name":"Coupling","shortDescription":{"text":"Coupling"},"helpUri":"https://codehealth.canine.dev/dimensions/D5"},{"id":"D6","name":"Cohesion (LCOM4)","shortDescription":{"text":"Cohesion (LCOM4)"},"helpUri":"https://codehealth.canine.dev/dimensions/D6"},{"id":"D9","name":"Test Distribution","shortDescription":{"text":"Test Distribution"},"helpUri":"https://codehealth.canine.dev/dimensions/D9"},{"id":"D11","name":"Test Reliability","shortDescription":{"text":"Test Reliability"},"helpUri":"https://codehealth.canine.dev/dimensions/D11"},{"id":"D12","name":"Dependency Hygiene","shortDescription":{"text":"Dependency Hygiene"},"helpUri":"https://codehealth.canine.dev/dimensions/D12"},{"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":"D18","name":"Solution Shape","shortDescription":{"text":"Solution Shape"},"helpUri":"https://codehealth.canine.dev/dimensions/D18"},{"id":"D21","name":"Naming Consistency","shortDescription":{"text":"Naming Consistency"},"helpUri":"https://codehealth.canine.dev/dimensions/D21"},{"id":"D23","name":"Boundary Type-Coupling","shortDescription":{"text":"Boundary Type-Coupling"},"helpUri":"https://codehealth.canine.dev/dimensions/D23"},{"id":"D26","name":"Project Cohesion","shortDescription":{"text":"Project Cohesion"},"helpUri":"https://codehealth.canine.dev/dimensions/D26"},{"id":"D27","name":"Navigability","shortDescription":{"text":"Navigability"},"helpUri":"https://codehealth.canine.dev/dimensions/D27"},{"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"}]}},"results":[{"ruleId":"D1","level":"warning","message":{"text":"RebuildProjection_Job.RunJobAsync (cyclomatic 18): RebuildProjection_Job.RunJobAsync has cyclomatic complexity 18 (threshold 15). Of this number, 15 points are the body\u0027s own statements and 3 belong to 2 function literals inside it that branch. To reduce it, separate the cases: extract each independent branch into its own named function, and where the body has guards that only reject input, fold those into early returns at the top."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Elders.Cronus/Projections/Rebuilding/RebuildProjection_Job.cs"},"region":{"startLine":60}}}],"partialFingerprints":{"codehealthFindingId/v1":"42391a26242194d56d0f9e93ac907fbe6481d98827a7f93b6dc77cb01e6de92e"}},{"ruleId":"D1","level":"warning","message":{"text":"ProjectionVersions.Add (cyclomatic 17): ProjectionVersions.Add has cyclomatic complexity 17 (threshold 15). To reduce it, split the body: these branches sit side by side rather than nested inside one another, so extracting each one on its own would leave a function per branch. Group the statements between the checks into named steps and move each step into its own function, so the body reads as a short sequence of named stages."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Elders.Cronus/Projections/ProjectionVersions.cs"},"region":{"startLine":58}}}],"partialFingerprints":{"codehealthFindingId/v1":"3a3e8bbbef3f61b166378dea6d66d36442bdabbc499b1be89d2790bef878ffdc"}},{"ruleId":"D1","level":"warning","message":{"text":"RebuildProjectionSequentially_Job.RunJobAsync (cyclomatic 17): RebuildProjectionSequentially_Job.RunJobAsync has cyclomatic complexity 17 (threshold 15). Of this number, 11 points are the body\u0027s own statements and 6 belong to one function literal inside it that branches. To reduce it, separate the cases: extract each independent branch into its own named function, and where the body has guards that only reject input, fold those into early returns at the top."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Elders.Cronus/Projections/Rebuilding/RebuildProjectionSequentially_Job.cs"},"region":{"startLine":64}}}],"partialFingerprints":{"codehealthFindingId/v1":"f648e6fea85752938cceac7eeb0ada1710d8c5804453047de85770d5be468e4b"}},{"ruleId":"D1","level":"warning","message":{"text":"CronusHost.Start (cyclomatic 16): CronusHost.Start has cyclomatic complexity 16 (threshold 15). To reduce it, name the conditions: bind each compound test to a well-named local or a small predicate function, so the body reads as a sequence of named decisions rather than a chain of operators."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Elders.Cronus/Hosting/CronusHost.cs"},"region":{"startLine":187}}}],"partialFingerprints":{"codehealthFindingId/v1":"595740379cee5baf3627e81c6fc2845d90110b97725bdf35ae7e0663af2ce773"}},{"ruleId":"D1","level":"warning","message":{"text":"CronusHost.Stop (cyclomatic 16): CronusHost.Stop has cyclomatic complexity 16 (threshold 15). To reduce it, name the conditions: bind each compound test to a well-named local or a small predicate function, so the body reads as a sequence of named decisions rather than a chain of operators."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Elders.Cronus/Hosting/CronusHost.cs"},"region":{"startLine":206}}}],"partialFingerprints":{"codehealthFindingId/v1":"5b59f61d4695c91da6938e39bb798f68bb8314e9602bd1c6a84a8f3f974314ff"}},{"ruleId":"D2","level":"warning","message":{"text":"CronusOptionsProviderBase.TryValidateObjectRecursive (cognitive 36): CronusOptionsProviderBase.TryValidateObjectRecursive has cognitive complexity 36 (threshold 15). Of this number, 35 points are the body\u0027s own statements and 1 belongs to one function literal inside it that branches. To reduce it, flatten the nesting: invert conditions into early returns or guard clauses so the happy path stays at one level, and lift the deepest nested block into its own named function."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Elders.Cronus/Hosting/CronusOptionsProviderBase.cs"},"region":{"startLine":69}}}],"partialFingerprints":{"codehealthFindingId/v1":"ec9abf7bfec8110a4c8ebb52cbc19f943f0cbe6ffb440c1834df7a0a945c89a1"}},{"ruleId":"D2","level":"warning","message":{"text":"RebuildProjectionSequentially_Job.RunJobAsync (cognitive 32): RebuildProjectionSequentially_Job.RunJobAsync has cognitive complexity 32 (threshold 15). Most of this is not in the body itself: 14 of the 32 points are its own statements and the rest belongs to one function literal inside it that branches (line 99). The decisions are inside the literal, which nothing outside this body can call, review or test on its own, so splitting the enclosing body is not the move available here. To reduce it, lift the literal\u0027s work into a named function or method at the enclosing scope and have the literal call it, then reduce whichever part then reads as the largest."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Elders.Cronus/Projections/Rebuilding/RebuildProjectionSequentially_Job.cs"},"region":{"startLine":64}}}],"partialFingerprints":{"codehealthFindingId/v1":"30d9e9b2461781ff059bdd00b63284517547781f598c07376edf4763d69beb41"}},{"ruleId":"D2","level":"warning","message":{"text":"ProjectionRepository.SaveAsync (cognitive 29): ProjectionRepository.SaveAsync has cognitive complexity 29 (threshold 15). To reduce it, flatten the nesting: invert conditions into early returns or guard clauses so the happy path stays at one level, and lift the deepest nested block into its own named function."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Elders.Cronus/Projections/ProjectionRepository.cs"},"region":{"startLine":38}}}],"partialFingerprints":{"codehealthFindingId/v1":"afd368cffe9e71b884ed4fa64eca01f165727ddd344dd843202ff247a42d4555"}},{"ruleId":"D2","level":"warning","message":{"text":"RebuildProjection_Job.RunJobAsync (cognitive 27): RebuildProjection_Job.RunJobAsync has cognitive complexity 27 (threshold 15). Of this number, 21 points are the body\u0027s own statements and 6 belong to 2 function literals inside it that branch. To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Elders.Cronus/Projections/Rebuilding/RebuildProjection_Job.cs"},"region":{"startLine":60}}}],"partialFingerprints":{"codehealthFindingId/v1":"62e403d74463cbe1b088e0c0601d7507da60b76af386bffcfba500d41740b952"}},{"ruleId":"D2","level":"warning","message":{"text":"RetryableOperation.TryExecute (cognitive 21): RetryableOperation.TryExecute has cognitive complexity 21 (threshold 15). To reduce it, flatten the nesting: invert conditions into early returns or guard clauses so the happy path stays at one level, and lift the deepest nested block into its own named function."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Elders.Cronus/Userfull/RetryableOperation.cs"},"region":{"startLine":32}}}],"partialFingerprints":{"codehealthFindingId/v1":"ba3c5f51df5ad20e0e69559045b67e5cb171d22620a0ba81da2cebc2c1d4ea88"}},{"ruleId":"D2","level":"warning","message":{"text":"RetryPolicy.ExecuteActionAsync (cognitive 20): RetryPolicy.ExecuteActionAsync has cognitive complexity 20 (threshold 15). To reduce it, flatten the nesting: invert conditions into early returns or guard clauses so the happy path stays at one level, and lift the deepest nested block into its own named function."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Elders.Cronus/FaultHandling/RetryPolicy.cs"},"region":{"startLine":149}}}],"partialFingerprints":{"codehealthFindingId/v1":"51391416324e83b047b07a6113a4c778e252f8d42b954fe532100e880c2f6f6e"}},{"ruleId":"D2","level":"warning","message":{"text":"ProjectionRepository.SaveAsync (cognitive 20): ProjectionRepository.SaveAsync has cognitive complexity 20 (threshold 15). To reduce it, flatten the nesting: invert conditions into early returns or guard clauses so the happy path stays at one level, and lift the deepest nested block into its own named function."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Elders.Cronus/Projections/ProjectionRepository.cs"},"region":{"startLine":104}}}],"partialFingerprints":{"codehealthFindingId/v1":"90d6586116e3552da5d067fbfa3f972e59bd6bf577fb0d91d12cbf07da0437ec"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (11 lines \u00D7 2): src/Elders.Cronus/Discoveries/ProjectionsDiscovery.cs:19-30 | src/Elders.Cronus/Discoveries/ProjectionsDiscovery.cs:66-76 \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 \u0060src/Elders.Cronus/Discoveries/ProjectionsDiscovery.cs:19\u0060 it begins part-way through the construct above it, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that. The matched lines also transfer control out of the body holding them, which cannot survive a move into a called unit unchanged: have the extracted unit return that decision and let each site act on it."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Elders.Cronus/Discoveries/ProjectionsDiscovery.cs"},"region":{"startLine":19}}}],"partialFingerprints":{"codehealthFindingId/v1":"f568a3e81491ff650dba31d5e38c0fd6cf91edad274ae424d0667718fdff659e"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (11 lines \u00D7 2): src/Elders.Cronus/Projections/Rebuilding/RebuildProjectionSequentially_Job.cs:74-84 | src/Elders.Cronus/Projections/Rebuilding/RebuildProjection_Job.cs:77-87 \u2014 the copies sit in sibling files of one directory: extract the block into a single shared function in that directory and call it from each site, so a change lands once. The matched lines also transfer control out of the body holding them, which cannot survive a move into a called unit unchanged: have the extracted unit return that decision and let each site act on it."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Elders.Cronus/Projections/Rebuilding/RebuildProjectionSequentially_Job.cs"},"region":{"startLine":74}}}],"partialFingerprints":{"codehealthFindingId/v1":"e2bd24b037f1d5e50bb8f1142d1db8d73b51194383043ab434dbacb33ef47801"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (11 lines \u00D7 2): src/Elders.Cronus/Projections/Rebuilding/RebuildProjectionSequentially_Job.cs:193-203 | src/Elders.Cronus/Projections/Rebuilding/RebuildProjection_Job.cs:176-186 \u2014 the copies sit in sibling files of one directory: extract the block into a single shared function in that directory and call it from each site, so a change lands once. 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":"src/Elders.Cronus/Projections/Rebuilding/RebuildProjectionSequentially_Job.cs"},"region":{"startLine":193}}}],"partialFingerprints":{"codehealthFindingId/v1":"a7a6d95169579f85d99321b20af46b29661ffdc8c1b5b95b2eef234057a2aada"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (7 lines \u00D7 2): src/Elders.Cronus/EventStore/Index/RebuildIndex_MessageCounter_Job.cs:87-93 | src/Elders.Cronus/Projections/Rebuilding/ProjectionVersionHelper.cs:119-125 \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 \u0060src/Elders.Cronus/EventStore/Index/RebuildIndex_MessageCounter_Job.cs:87\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":"src/Elders.Cronus/EventStore/Index/RebuildIndex_MessageCounter_Job.cs"},"region":{"startLine":87}}}],"partialFingerprints":{"codehealthFindingId/v1":"4e5b7899440ce7da31d97eb7fc5913b0457c9dbd7f83b3437b8ea688ed00b2a1"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (6 lines \u00D7 2): src/Elders.Cronus/MessageProcessing/ApplicationServiceSubscriberWorkflow.cs:25-30 | src/Elders.Cronus/MessageProcessing/EventStoreIndexSubscriberWorkflow.cs:31-36 \u2014 the copies sit in sibling files of one directory: extract the block into a single shared function in that directory and call it from each site, so a change lands once. Read the line range as the matched WINDOW rather than a finished unit: at \u0060src/Elders.Cronus/MessageProcessing/ApplicationServiceSubscriberWorkflow.cs:25\u0060 it begins part-way through the construct above it, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that. The matched lines also transfer control out of the body holding them, which cannot survive a move into a called unit unchanged: have the extracted unit return that decision and let each site act on it."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Elders.Cronus/MessageProcessing/ApplicationServiceSubscriberWorkflow.cs"},"region":{"startLine":25}}}],"partialFingerprints":{"codehealthFindingId/v1":"aec517360399250d60ea304bc6e19bbaa26496e640ca50db14e0fe12a5503dc5"}},{"ruleId":"D5","level":"warning","message":{"text":"Off the main sequence: Elders.Cronus(net8.0): Elders.Cronus(net8.0): abstractness 0.26, instability 0.00, distance 0.74 \u2014 zone of pain \u2014 concrete and depended on by 1 project(s), so it\u0027s rigid to change."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"68491415ad1ba90d6d0a5d5906155d50c096dc7ae3ec99fb2c8d1dc032c7879b"}},{"ruleId":"D9","level":"warning","message":{"text":"Tests outside the analyzed solution: 1 .NET test project(s) exist on disk but aren\u0027t in the .NET solution that was analyzed (e.g. Elders.Cronus.Tests.Sanity.csproj), so they aren\u0027t built or gated with the production code and can\u0027t be assessed here. If this .NET code is part of the product, add those projects to the solution or point CodeHealth at a solution that includes them; if it is a client/compatibility harness beside a product written in another language, nothing is wrong here and this row can be ignored."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"7a0b53e0d18583a1361cd04775cd53329dedef67393349e0c99678c3c468f894"}},{"ruleId":"D17","level":"warning","message":{"text":"CommentedOutCode: 5 consecutive commented-code lines"},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Elders.Cronus.Tests/InMemory/InMemoryIndexStore.cs"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"67b6610e3e7d03997a860ceed4184901300066836b0ee4a97fb45ebe339e7f5a"}},{"ruleId":"D17","level":"warning","message":{"text":"CommentedOutCode: 5 consecutive commented-code lines"},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Elders.Cronus.Tests/InMemoryEventStoreSuite/When_loading_aggregate_root_with_entity_from_event_store.cs"},"region":{"startLine":30}}}],"partialFingerprints":{"codehealthFindingId/v1":"7c79b1399006d26cebce2431a53ff397cbf0e62247cc8e57e50236662f8042e6"}},{"ruleId":"D17","level":"warning","message":{"text":"CommentedOutCode: 5 consecutive commented-code lines"},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Elders.Cronus/FaultHandling/Strategies/ExponentialBackoff.cs"},"region":{"startLine":63}}}],"partialFingerprints":{"codehealthFindingId/v1":"2b65ca4e5b5f9cbea2ae16e359e561da217f2ce0cc359c13635993c8b673a1cb"}},{"ruleId":"D17","level":"warning","message":{"text":"CommentedOutCode: 5 consecutive commented-code lines"},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Elders.Cronus/MessageProcessing/CronusPublisher.cs"},"region":{"startLine":11}}}],"partialFingerprints":{"codehealthFindingId/v1":"b09e7ee16da9cd6baef7f4d20b9e05e4878399eb0355a1fe1196b6b1ddbdb649"}},{"ruleId":"D17","level":"warning","message":{"text":"CommentedOutCode: 5 consecutive commented-code lines"},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Elders.Cronus/Projections/InMemory/InMemoryProjectionStore.cs"},"region":{"startLine":42}}}],"partialFingerprints":{"codehealthFindingId/v1":"0368749504a1753e32838138db40bac4ebeefc74f9b9590a359b1d39c5738b29"}},{"ruleId":"D17","level":"warning","message":{"text":"CommentedOutCode: 7 consecutive commented-code lines"},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Elders.Cronus.Tests/InMemory/InMemoryPublisher.cs"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"e681b1a914c466fd45e40f7855351a24211fe19dc009976b0317886be744eff5"}},{"ruleId":"D17","level":"warning","message":{"text":"CommentedOutCode: 7 consecutive commented-code lines"},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Elders.Cronus.Tests/InMemory/InMemoryPublisher.cs"},"region":{"startLine":15}}}],"partialFingerprints":{"codehealthFindingId/v1":"996a7ff08af76d15f8d0880db27b5e417d88424cb51a2ba9983381836297d40b"}},{"ruleId":"D17","level":"warning","message":{"text":"CommentedOutCode: 7 consecutive commented-code lines"},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Elders.Cronus.Tests/InMemory/InMemoryPublisher.cs"},"region":{"startLine":72}}}],"partialFingerprints":{"codehealthFindingId/v1":"ffca1a2ab2a8d36cf6e83d52011ccce7c97e72459500cd9fb1a3b6bcbdcb2ee3"}},{"ruleId":"D17","level":"warning","message":{"text":"CommentedOutCode: 7 consecutive commented-code lines"},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Elders.Cronus.Tests/InMemoryEventStoreSuite/When_loading_aggregate_root_with_entity_from_event_store.cs"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"5a5e83660b1c54d71e36d5e5011c2496a5a959564c02e149b5e078108854ca2d"}},{"ruleId":"D17","level":"warning","message":{"text":"CommentedOutCode: 7 consecutive commented-code lines"},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Elders.Cronus.Tests/Projections/When_projection_version_with_status_building_is_outside_of_the_timebox.cs"},"region":{"startLine":74}}}],"partialFingerprints":{"codehealthFindingId/v1":"b0cd91fc5565fe3825459fbc13d5277aaeb3f31e1ed60b0c4d0b90fb9b525770"}},{"ruleId":"D17","level":"warning","message":{"text":"CommentedOutCode: 4 consecutive commented-code lines"},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Elders.Cronus.Tests/InMemory/InMemoryPublisher.cs"},"region":{"startLine":41}}}],"partialFingerprints":{"codehealthFindingId/v1":"4cf52bfc884fa18cf5e8022bfec4bea95f1531b3c94c52aa13c74b86d6c81ec0"}},{"ruleId":"D17","level":"warning","message":{"text":"CommentedOutCode: 4 consecutive commented-code lines"},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Elders.Cronus.Tests/InMemoryLock/When_checking_a_locked_resource.cs"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"d88cf005efbfdb08d05bb1bbbc77e31a0f1e7d6b035a69c1070e6f2d4a3ccca5"}},{"ruleId":"D17","level":"warning","message":{"text":"CommentedOutCode: 4 consecutive commented-code lines"},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Elders.Cronus.Tests/InMemoryLock/When_checking_for_expired_locked_resource.cs"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"d3ffaeae09397c291eccf608ef36e4017c3fce532b20bfce5a5d24bef0b0f0b1"}},{"ruleId":"D17","level":"warning","message":{"text":"CommentedOutCode: 4 consecutive commented-code lines"},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Elders.Cronus.Tests/InMemoryLock/When_locking_resource.cs"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"1aa8bea2b23e729454c6a5492423b7d6284c141054ec1bb8dbbb72eba33fb8ca"}},{"ruleId":"D17","level":"warning","message":{"text":"CommentedOutCode: 4 consecutive commented-code lines"},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Elders.Cronus.Tests/InMemoryLock/When_unlocking_a_locked_resource.cs"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"dee3e15d1308c6ee9ce9630b006eca490cdfa480ea4509b51dbcd8b90af8bf92"}},{"ruleId":"D17","level":"warning","message":{"text":"CommentedOutCode: 4 consecutive commented-code lines"},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Elders.Cronus.Tests/InMemoryLock/When_unlocking_a_specific_locked_resource.cs"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"61bc03892337db3827338eddbab6d865f7f081affb73ee5a5b29fcee7b48cda4"}},{"ruleId":"D17","level":"warning","message":{"text":"CommentedOutCode: 4 consecutive commented-code lines"},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Elders.Cronus.Tests/When_resolve_scoped_component_out_of_scope.cs"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"ddee9819da475d0c3a29cc993209243a610d8b3782dcff5245743c4eb5750902"}},{"ruleId":"D17","level":"warning","message":{"text":"CommentedOutCode: 4 consecutive commented-code lines"},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Elders.Cronus/EventStore/Integrity/EventStreamIntegrityPolicy.cs"},"region":{"startLine":44}}}],"partialFingerprints":{"codehealthFindingId/v1":"ce67aef0160d9442f099a9849fc64d7c632e11618bf4aa9d32769f55c2748868"}},{"ruleId":"D17","level":"warning","message":{"text":"CommentedOutCode: 4 consecutive commented-code lines"},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Elders.Cronus/EventStore/Integrity/EventStreamIntegrityPolicy.cs"},"region":{"startLine":51}}}],"partialFingerprints":{"codehealthFindingId/v1":"4d51bc8c709029fe17710295a820f4e07e42b5677e0f0de51df14333a55fd610"}},{"ruleId":"D17","level":"warning","message":{"text":"CommentedOutCode: 4 consecutive commented-code lines"},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Elders.Cronus/Projections/InMemory/InMemoryProjectionStore.cs"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"693b03f6b4394e6bcb7b86fdb04591e9ae195094e8423174c555312904b9ca8a"}},{"ruleId":"D17","level":"warning","message":{"text":"CommentedOutCode: 4 consecutive commented-code lines"},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Elders.Cronus/Projections/ProjectionVersion.cs"},"region":{"startLine":36}}}],"partialFingerprints":{"codehealthFindingId/v1":"6ba4de8c07d7c6dd03ad4007071d95b72c74a78be3c7b5b8ec55abfa14d7eba3"}},{"ruleId":"D17","level":"warning","message":{"text":"CommentedOutCode: 4 consecutive commented-code lines"},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Elders.Cronus/Projections/Versioning/InMemoryProjectionVersionStore.cs"},"region":{"startLine":10}}}],"partialFingerprints":{"codehealthFindingId/v1":"538b4143bcdffebed5185fe2a322571816d71f6ff38978deea4155c56ffdfab8"}},{"ruleId":"D17","level":"warning","message":{"text":"CommentedOutCode: 3 consecutive commented-code lines"},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Elders.Cronus.Tests/InMemoryEventStoreSuite/When_loading_aggregate_root_with_entity_from_event_store.cs"},"region":{"startLine":26}}}],"partialFingerprints":{"codehealthFindingId/v1":"b122fda3d1b5cf5f7c0be30a6cd40f51904b2cbf32707be2f1b9e65ad2b4b8d5"}},{"ruleId":"D17","level":"warning","message":{"text":"CommentedOutCode: 3 consecutive commented-code lines"},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Elders.Cronus.Tests/InMemoryEventStoreSuite/When_loading_events_for_replay.cs"},"region":{"startLine":33}}}],"partialFingerprints":{"codehealthFindingId/v1":"66f150023eb4bfabc0961c848d3f033d05cae81526aa7344a9ed3bcd810badaf"}},{"ruleId":"D17","level":"warning","message":{"text":"CommentedOutCode: 3 consecutive commented-code lines"},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Elders.Cronus.Tests/InMemoryLock/When_checking_a_not_locked_resource.cs"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"d12be55614b2001dabc8c05a1bdb10caa5068900be7e7b96fba0574a87e6e5f9"}},{"ruleId":"D17","level":"warning","message":{"text":"CommentedOutCode: 3 consecutive commented-code lines"},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Elders.Cronus.Tests/When_resolve_scoped_component_out_of_scope.cs"},"region":{"startLine":23}}}],"partialFingerprints":{"codehealthFindingId/v1":"fc26bae8a605df1deca77e32ec9bbc8fb14158b2c619109b1337798a2f97ae57"}},{"ruleId":"D17","level":"warning","message":{"text":"CommentedOutCode: 3 consecutive commented-code lines"},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Elders.Cronus.Tests/When_resolve_scoped_component_out_of_scope.cs"},"region":{"startLine":27}}}],"partialFingerprints":{"codehealthFindingId/v1":"bac464f2778f2a48710f515eea97e255959241e46bf6edbb53205ec404f6f7c7"}},{"ruleId":"D17","level":"warning","message":{"text":"CommentedOutCode: 3 consecutive commented-code lines"},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Elders.Cronus.Tests/When_resolve_scoped_component_out_of_scope.cs"},"region":{"startLine":31}}}],"partialFingerprints":{"codehealthFindingId/v1":"6c6574072277045e4da9e48ba15d1b63db902659839e4346c46c4f6661cc97bb"}},{"ruleId":"D17","level":"warning","message":{"text":"CommentedOutCode: 3 consecutive commented-code lines"},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Elders.Cronus/Discoveries/ProjectionPlayerDiscovery.cs"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"7af5619434ad918a12271989c748c568c629393beeedadef74ccd96ea3810c43"}},{"ruleId":"D17","level":"warning","message":{"text":"CommentedOutCode: 3 consecutive commented-code lines"},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Elders.Cronus/FaultHandling/Strategies/Incremental.cs"},"region":{"startLine":56}}}],"partialFingerprints":{"codehealthFindingId/v1":"df4c04d765a13567975f89422d0be49f5675e60b843058e63665344e226eb981"}},{"ruleId":"D17","level":"warning","message":{"text":"CommentedOutCode: 3 consecutive commented-code lines"},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Elders.Cronus/Hosting/AssemblyLoader.cs"},"region":{"startLine":53}}}],"partialFingerprints":{"codehealthFindingId/v1":"24440b00a350de99eaabfc6e8183828b8014f53cb47056252239641027894db6"}},{"ruleId":"D17","level":"warning","message":{"text":"CommentedOutCode: 3 consecutive commented-code lines"},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Elders.Cronus/Multitenancy/DefaultTenantResolver.cs"},"region":{"startLine":19}}}],"partialFingerprints":{"codehealthFindingId/v1":"306184cb5f9924b09ec908da6e188fd469e87165605abbd45132d06dd638f7c1"}},{"ruleId":"D17","level":"warning","message":{"text":"CommentedOutCode: 9 consecutive commented-code lines"},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Elders.Cronus.Tests/InMemoryEventStoreSuite/When_loading_events_for_replay.cs"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"e4bbd47a53c98db19ec644bb94ee53f3089b7d1f103d48eeeece2b1b9e308912"}},{"ruleId":"D17","level":"warning","message":{"text":"CommentedOutCode: 9 consecutive commented-code lines"},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Elders.Cronus.Tests/When_resolve_scoped_component_out_of_scope.cs"},"region":{"startLine":41}}}],"partialFingerprints":{"codehealthFindingId/v1":"afaa516918cbd5c54b2cf18995253bb12a38d20f63e358cff6e78a22a9d83ed0"}},{"ruleId":"D17","level":"warning","message":{"text":"HackComment: // hack \u2014 a workaround marked in source: record what it is compensating for and what would allow its removal (the upstream fix, the API it is waiting on, the invariant it restores), so the next reader can judge whether it is still needed rather than rediscovering why it is there."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Elders.Cronus.Tests/Migrations/TestModel/TestEventStorePlayer.cs"},"region":{"startLine":30}}}],"partialFingerprints":{"codehealthFindingId/v1":"009305f4f3cae9133a9a05f49f5f20d8ba85a74675c836ea15796042f1c7a93b"}},{"ruleId":"D17","level":"warning","message":{"text":"CommentedOutCode: 10 consecutive commented-code lines"},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Elders.Cronus.Tests/When_resolve_scoped_component_out_of_scope.cs"},"region":{"startLine":51}}}],"partialFingerprints":{"codehealthFindingId/v1":"b9a05f6b5209f7b4d30ecc10998a0677cc9e74351c0c7ae2e7d2b1cdb5ea9871"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: in future we can have scheduled messages \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":"src/Elders.Cronus/AutoUpdates/AutoUpdateSaga.cs"},"region":{"startLine":11}}}],"partialFingerprints":{"codehealthFindingId/v1":"3f0824c2d01cb161b7e60768d2299799311afa5baffa4a50cb1e752c69462004"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Check if this is alrady registered in the foreach above. If yes we can remove this line. Elase we have to figure out what is going on \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":"src/Elders.Cronus/Discoveries/CronusHostDiscovery.cs"},"region":{"startLine":42}}}],"partialFingerprints":{"codehealthFindingId/v1":"b0bb4a1b2ebedbda76ac0c70ce1458a014bd27fe9386f27c2c2f0967020e9f1e"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: failed status? \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":"src/Elders.Cronus/EventStore/AutoUpdater/AutoUpdater.cs"},"region":{"startLine":56}}}],"partialFingerprints":{"codehealthFindingId/v1":"e58439bb8af8dad27ea7ff6846abc1d3019b26315624b416acd9bc11160d7693"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: think for retries? \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":"src/Elders.Cronus/EventStore/AutoUpdater/AutoUpdaterState.cs"},"region":{"startLine":72}}}],"partialFingerprints":{"codehealthFindingId/v1":"6e0f7db04ec6f771786ec1a230713473e015f766429277791662119a08d96e35"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: in future we might execute in parallel \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":"src/Elders.Cronus/EventStore/AutoUpdater/AutoUpdaterState.cs"},"region":{"startLine":80}}}],"partialFingerprints":{"codehealthFindingId/v1":"499fdfc91428d9e97ebab0d9b5db0677ee2227882ea221502ef2a9dffd73c679"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: //TODO: Document which headers are essential or make another ctor for CronusMessage with byte[] \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":"src/Elders.Cronus/EventStore/Players/ReplayPublicEvents_Job.cs"},"region":{"startLine":48}}}],"partialFingerprints":{"codehealthFindingId/v1":"8f628b2fbb937e3fd4ed74c454c42b5d530432321ca6b5cc573b28ddd3ee6d50"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: Make this async \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":"src/Elders.Cronus/Hosting/ICronusStartup.cs"},"region":{"startLine":8}}}],"partialFingerprints":{"codehealthFindingId/v1":"47f5d881b5cf3e0afa2e7fb964286e96ff80d0036e3a57efde6ee0448ab5fc81"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: //TODO: also make this async :) kali \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":"src/Elders.Cronus/Hosting/ICronusStartup.cs"},"region":{"startLine":15}}}],"partialFingerprints":{"codehealthFindingId/v1":"ff48964cab426449917bc524b9fb19f752fc4a6923a97fe81a36bb927f55738a"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: mynkow \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":"src/Elders.Cronus/Hosting/SingletonPerTenantContainer.cs"},"region":{"startLine":29}}}],"partialFingerprints":{"codehealthFindingId/v1":"ddb343a6d6eeebd8b4f9a49a660cf471f1a4cf470fff61b833fc04624cd31fe9"}},{"ruleId":"D17","level":"warning","message":{"text":"CommentedOutCode: 6 consecutive commented-code lines"},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Elders.Cronus/MessageProcessing/CronusPublisher.cs"},"region":{"startLine":17}}}],"partialFingerprints":{"codehealthFindingId/v1":"336050bf5bb7a96ab7960b1c122108f236b348fc89d5806a7f30f7b17161104b"}},{"ruleId":"D17","level":"warning","message":{"text":"CommentedOutCode: 6 consecutive commented-code lines"},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Elders.Cronus/MessageProcessing/CronusPublisher.cs"},"region":{"startLine":24}}}],"partialFingerprints":{"codehealthFindingId/v1":"8c6b1fbabf3d00f6886a01c3d3a4634bcc03a64dec61fefa7acabf2fbd5cdebb"}},{"ruleId":"D17","level":"warning","message":{"text":"CommentedOutCode: 6 consecutive commented-code lines"},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Elders.Cronus/MessageProcessing/CronusPublisher.cs"},"region":{"startLine":31}}}],"partialFingerprints":{"codehealthFindingId/v1":"f779a987e86f3a74598286cd789d7276a24598238ea8a959df46e1e15491b3f2"}},{"ruleId":"D17","level":"error","message":{"text":"EmptyCatchBlock: empty catch block \u2014 the error is discarded with nothing recorded, so a failure here leaves no trace anywhere. Narrow the catch to the exception you actually expect, record it through whatever this codebase already uses to report problems, or \u2014 where swallowing really is correct, as it often is on a teardown/dispose path where throwing would mask the original failure \u2014 write down WHY in a comment on the catch. The comment has to give the reason: a note that only restates the swallow (\u0022ignored\u0022, \u0022do nothing\u0022) is read as no explanation at all and leaves this row in place. Any of the three makes the decision reviewable; all three clear this row."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Elders.Cronus/MessageProcessing/ErrorContext.cs"},"region":{"startLine":45}}}],"partialFingerprints":{"codehealthFindingId/v1":"f5efff014f25bb619dec97051bdab26f4dce791a1e24e71bfa2af14b8ad54106"}},{"ruleId":"D17","level":"error","message":{"text":"EmptyCatchBlock: empty catch block \u2014 the error is discarded with nothing recorded, so a failure here leaves no trace anywhere. Narrow the catch to the exception you actually expect, record it through whatever this codebase already uses to report problems, or \u2014 where swallowing really is correct, as it often is on a teardown/dispose path where throwing would mask the original failure \u2014 write down WHY in a comment on the catch. The comment has to give the reason: a note that only restates the swallow (\u0022ignored\u0022, \u0022do nothing\u0022) is read as no explanation at all and leaves this row in place. Any of the three makes the decision reviewable; all three clear this row."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Elders.Cronus/Projections/Rebuilding/ProgressTracker.cs"},"region":{"startLine":170}}}],"partialFingerprints":{"codehealthFindingId/v1":"c5aad40183e0bed1f13db0209d2e460c8bac77da39f4f7e66c08461c4470f6f1"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: inspect the type if it implements IProjectionDefinition \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":"src/Elders.Cronus/Projections/ProjectionRepository.cs"},"region":{"startLine":44}}}],"partialFingerprints":{"codehealthFindingId/v1":"e44d8d2983c736310e33dfe28e1796a9192b07ba1b954ce938d79bb6c7797797"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: This will crash with null ref \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":"src/Elders.Cronus/Projections/ProjectionVersions.cs"},"region":{"startLine":152}}}],"partialFingerprints":{"codehealthFindingId/v1":"e7772224b368251901a9e6c66874b30d9b48e089d8d94c22a3deb7d7298e37bf"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: This will crash with null ref \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":"src/Elders.Cronus/Projections/ProjectionVersions.cs"},"region":{"startLine":164}}}],"partialFingerprints":{"codehealthFindingId/v1":"f3a7a5480f54efd56f2095cfd8971ff8c12d94e3a79f096770f97c1697620a68"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: check if the timebox really has expired LOL :), Believe me, do it \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":"src/Elders.Cronus/Projections/Versioning/ProjectionVersionManager.cs"},"region":{"startLine":94}}}],"partialFingerprints":{"codehealthFindingId/v1":"a7b6f93a3369138a8a7ffc7562abbb2964c945b1e4b4384ae586dd8daf50f65d"}},{"ruleId":"D17","level":"warning","message":{"text":"Dead code: Class1: NamedType Class1 \u2014 no references found in solution."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Elders.Cronus.Tests/Performance/Class1.cs"},"region":{"startLine":3}}}],"partialFingerprints":{"codehealthFindingId/v1":"3d22ab32bcce1c2487edce75e69210b322b6cb704c4ae32687679f23d4032588"}},{"ruleId":"D23","level":"note","message":{"text":"Bounded contexts not declared: At 12337 LoC spread over 6 projects the codebase is large and multi-module, so explicit bounded contexts are needed. Name this codebase\u0027s bounded contexts (\u22652 module groups, e.g. per subsystem) so cross-boundary type coupling can be assessed. Declare them in \u0060.codehealth/config.yaml\u0060 at the repository root (create it if absent), mapping each context name to the module-path or namespace prefixes that belong to it \u2014 e.g. \u0060architecture:\u0060 \u2192 \u0060contexts:\u0060 \u2192 \u0060Billing: [\u0022src/billing\u0022, \u0022Acme.Billing\u0022]\u0060, \u0060Catalog: [\u0022src/catalog\u0022, \u0022Acme.Catalog\u0022]\u0060."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"1c7e276c9c682731f01819ed5378b90ca320cde1b583c90920172911598362b3"}},{"ruleId":"D26","level":"note","message":{"text":"Split Elders.Cronus: A 12k-LoC library whose 32 namespaces are all cron job and scheduling code scattered over unrelated types. Suggested: by namespace: CronJobEngine, SchedulerRegistry, TimeWindowParser"},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"86a341d4ae4404fab6cbea6cded650f34b0c55b49997fafa0a77b1513949612c"}},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"d5b8f11381d61a58459ffc7c595fd52d09c03c711894a63a9578043d2338412b"},"taxa":[{"id":"CWE-829","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"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":"7cf85abffe732f2645ef95a624d6badc26f0263616284d6a4cc46ca8e983031d"}},{"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":"b96b7173c4377d5bea4d6fd7b52323631c631c44172b743bec6f266c10a4e538"}},{"ruleId":"D34","level":"note","message":{"text":"Dormant codebase: 54 of 64 significant files have no living knowledge \u2014 the codebase as a whole is dormant, not 54 separate risks. Re-engage owners or document before change."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"4c69f1e54b7dd6fe9029dd02df6ba8f8145b789f2f18dbdaa086c40ded49dba6"}},{"ruleId":"D35","level":"warning","message":{"text":"Change coupling: NewProjectionVersionIsNowLive.cs \u2194 ProjectionVersionRequestTimedout.cs: \u0060src/Elders.Cronus/Projections/Versioning/Events/NewProjectionVersionIsNowLive.cs\u0060 and \u0060src/Elders.Cronus/Projections/Versioning/Events/ProjectionVersionRequestTimedout.cs\u0060 change together 70% of the time (7 of the 10 commits that touched the less-changed of the two, renames followed). They sit in the same directory, and in this ecosystem sibling files there normally share one namespace/package \u2014 so a direct reference between them needs no import and this pass cannot see whether one exists. Read the pair before acting: if one file only DECLARES what the other consumes (a constants/types file beside its user), the co-change is definitional and the question is whether the split earns its keep; if they duplicate structure, extract the common part into a shared function or type they both call; if neither holds, the coupling is hidden and worth breaking."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Elders.Cronus/Projections/Versioning/Events/NewProjectionVersionIsNowLive.cs"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"5738cefebcea480c43ebee0e25c65b4b7f99ce78981dc0c2e5aa643b258f9cdd"}},{"ruleId":"D35","level":"warning","message":{"text":"Change coupling: NewProjectionVersionIsNowLive.cs \u2194 ProjectionVersionRequested.cs: \u0060src/Elders.Cronus/Projections/Versioning/Events/NewProjectionVersionIsNowLive.cs\u0060 and \u0060src/Elders.Cronus/Projections/Versioning/Events/ProjectionVersionRequested.cs\u0060 change together 70% of the time (7 of the 10 commits that touched the less-changed of the two, renames followed). They sit in the same directory, and in this ecosystem sibling files there normally share one namespace/package \u2014 so a direct reference between them needs no import and this pass cannot see whether one exists. Read the pair before acting: if one file only DECLARES what the other consumes (a constants/types file beside its user), the co-change is definitional and the question is whether the split earns its keep; if they duplicate structure, extract the common part into a shared function or type they both call; if neither holds, the coupling is hidden and worth breaking."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Elders.Cronus/Projections/Versioning/Events/NewProjectionVersionIsNowLive.cs"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"6a1861062640feb01db509095089fa112d09efc7faeac92cda87740fed38725f"}},{"ruleId":"D35","level":"warning","message":{"text":"Change coupling: ProjectionVersionRequestTimedout.cs \u2194 ProjectionVersionRequested.cs: \u0060src/Elders.Cronus/Projections/Versioning/Events/ProjectionVersionRequestTimedout.cs\u0060 and \u0060src/Elders.Cronus/Projections/Versioning/Events/ProjectionVersionRequested.cs\u0060 change together 69% of the time (9 of the 13 commits that touched the less-changed of the two, renames followed). They sit in the same directory, and in this ecosystem sibling files there normally share one namespace/package \u2014 so a direct reference between them needs no import and this pass cannot see whether one exists. Read the pair before acting: if one file only DECLARES what the other consumes (a constants/types file beside its user), the co-change is definitional and the question is whether the split earns its keep; if they duplicate structure, extract the common part into a shared function or type they both call; if neither holds, the coupling is hidden and worth breaking."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Elders.Cronus/Projections/Versioning/Events/ProjectionVersionRequestTimedout.cs"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"ff8376d1ab3018df865553a047297f190faa9b286aa5f0a6e4fd5bf5ca6a300e"}},{"ruleId":"D35","level":"warning","message":{"text":"Change coupling: FixProjectionVersion.cs \u2194 RegisterProjection.cs: \u0060src/Elders.Cronus/Projections/Versioning/Commands/FixProjectionVersion.cs\u0060 and \u0060src/Elders.Cronus/Projections/Versioning/Commands/RegisterProjection.cs\u0060 change together 58% of the time (7 of the 12 commits that touched the less-changed of the two, renames followed). They sit in the same directory, and in this ecosystem sibling files there normally share one namespace/package \u2014 so a direct reference between them needs no import and this pass cannot see whether one exists. Read the pair before acting: if one file only DECLARES what the other consumes (a constants/types file beside its user), the co-change is definitional and the question is whether the split earns its keep; if they duplicate structure, extract the common part into a shared function or type they both call; if neither holds, the coupling is hidden and worth breaking."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Elders.Cronus/Projections/Versioning/Commands/FixProjectionVersion.cs"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"2023f896fe12085849c7985c65cc12ec52f3227059657908648b7fc654a1aa00"}},{"ruleId":"D35","level":"warning","message":{"text":"Change coupling: RegisterProjection.cs \u2194 ProjectionVersionRequested.cs: \u0060src/Elders.Cronus/Projections/Versioning/Commands/RegisterProjection.cs\u0060 and \u0060src/Elders.Cronus/Projections/Versioning/Events/ProjectionVersionRequested.cs\u0060 change together 58% of the time (7 of the 12 commits that touched the less-changed of the two, renames followed). They sit in different directories, but in this ecosystem the namespace is declared in the FILE, not by the folder \u2014 so the two may well share one namespace and reference each other with no import for this pass to see. Read the pair before acting: if one derives from or overrides the other, the dependency is explicit in the type declaration and the co-change is definitional; if one registers itself into the other through a hook or an initialiser, the missing dependency is DELIBERATE and the thing to add is a comment saying so; if they simply belong together, co-locate them; if none of these holds, the coupling is hidden and worth breaking."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Elders.Cronus/Projections/Versioning/Commands/RegisterProjection.cs"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"6b213bb02a82d197b5a59aedd67a55c9bba05826de12a368c46d5b53c3208155"}},{"ruleId":"D35","level":"warning","message":{"text":"Change coupling: RegisterProjection.cs \u2194 ProjectionVersionRequestTimedout.cs: \u0060src/Elders.Cronus/Projections/Versioning/Commands/RegisterProjection.cs\u0060 and \u0060src/Elders.Cronus/Projections/Versioning/Events/ProjectionVersionRequestTimedout.cs\u0060 change together 50% of the time (6 of the 12 commits that touched the less-changed of the two, renames followed). They sit in different directories, but in this ecosystem the namespace is declared in the FILE, not by the folder \u2014 so the two may well share one namespace and reference each other with no import for this pass to see. Read the pair before acting: if one derives from or overrides the other, the dependency is explicit in the type declaration and the co-change is definitional; if one registers itself into the other through a hook or an initialiser, the missing dependency is DELIBERATE and the thing to add is a comment saying so; if they simply belong together, co-locate them; if none of these holds, the coupling is hidden and worth breaking."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Elders.Cronus/Projections/Versioning/Commands/RegisterProjection.cs"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"622ab041141bbcba5031d249459aa643735f226a4726434be1bcd129247d3c8e"}},{"ruleId":"D35","level":"warning","message":{"text":"Change coupling: RegisterProjection.cs \u2194 ProjectionBuilder.cs: \u0060src/Elders.Cronus/Projections/Versioning/Commands/RegisterProjection.cs\u0060 and \u0060src/Elders.Cronus/Projections/Versioning/Handlers/ProjectionBuilder.cs\u0060 change together 50% of the time (6 of the 12 commits that touched the less-changed of the two, renames followed). They sit in different directories, but in this ecosystem the namespace is declared in the FILE, not by the folder \u2014 so the two may well share one namespace and reference each other with no import for this pass to see. Read the pair before acting: if one derives from or overrides the other, the dependency is explicit in the type declaration and the co-change is definitional; if one registers itself into the other through a hook or an initialiser, the missing dependency is DELIBERATE and the thing to add is a comment saying so; if they simply belong together, co-locate them; if none of these holds, the coupling is hidden and worth breaking."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Elders.Cronus/Projections/Versioning/Commands/RegisterProjection.cs"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"fac50fe2c68fe941d37c41928a902ab9b8e697e56d22ce0268f1550830493530"}},{"ruleId":"D35","level":"warning","message":{"text":"Change coupling: ProjectionBuilder.cs \u2194 ProjectionVersionManagerState.cs: \u0060src/Elders.Cronus/Projections/Versioning/Handlers/ProjectionBuilder.cs\u0060 and \u0060src/Elders.Cronus/Projections/Versioning/ProjectionVersionManagerState.cs\u0060 change together 50% of the time (6 of the 12 commits that touched the less-changed of the two, renames followed). They sit in different directories, but in this ecosystem the namespace is declared in the FILE, not by the folder \u2014 so the two may well share one namespace and reference each other with no import for this pass to see. Read the pair before acting: if one derives from or overrides the other, the dependency is explicit in the type declaration and the co-change is definitional; if one registers itself into the other through a hook or an initialiser, the missing dependency is DELIBERATE and the thing to add is a comment saying so; if they simply belong together, co-locate them; if none of these holds, the coupling is hidden and worth breaking."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Elders.Cronus/Projections/Versioning/Handlers/ProjectionBuilder.cs"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"deba400a5823563c5ad93734c9ab1b81edeba2061b0ef4ee6416dea4d4686985"}},{"ruleId":"D35","level":"warning","message":{"text":"Change coupling: NewProjectionVersionIsNowLive.cs \u2194 ProjectionBuilder.cs: \u0060src/Elders.Cronus/Projections/Versioning/Events/NewProjectionVersionIsNowLive.cs\u0060 and \u0060src/Elders.Cronus/Projections/Versioning/Handlers/ProjectionBuilder.cs\u0060 change together 50% of the time (5 of the 10 commits that touched the less-changed of the two, renames followed). They sit in different directories, but in this ecosystem the namespace is declared in the FILE, not by the folder \u2014 so the two may well share one namespace and reference each other with no import for this pass to see. Read the pair before acting: if one derives from or overrides the other, the dependency is explicit in the type declaration and the co-change is definitional; if one registers itself into the other through a hook or an initialiser, the missing dependency is DELIBERATE and the thing to add is a comment saying so; if they simply belong together, co-locate them; if none of these holds, the coupling is hidden and worth breaking."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Elders.Cronus/Projections/Versioning/Events/NewProjectionVersionIsNowLive.cs"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"3621355bc499c13faa3271c95a85a28d278d8a9e39141749335b23253f6df0b0"}},{"ruleId":"D35","level":"warning","message":{"text":"Change coupling: RegisterProjection.cs \u2194 NewProjectionVersionIsNowLive.cs: \u0060src/Elders.Cronus/Projections/Versioning/Commands/RegisterProjection.cs\u0060 and \u0060src/Elders.Cronus/Projections/Versioning/Events/NewProjectionVersionIsNowLive.cs\u0060 change together 50% of the time (5 of the 10 commits that touched the less-changed of the two, renames followed). They sit in different directories, but in this ecosystem the namespace is declared in the FILE, not by the folder \u2014 so the two may well share one namespace and reference each other with no import for this pass to see. Read the pair before acting: if one derives from or overrides the other, the dependency is explicit in the type declaration and the co-change is definitional; if one registers itself into the other through a hook or an initialiser, the missing dependency is DELIBERATE and the thing to add is a comment saying so; if they simply belong together, co-locate them; if none of these holds, the coupling is hidden and worth breaking."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"src/Elders.Cronus/Projections/Versioning/Commands/RegisterProjection.cs"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"1213ae4ff6d9162902ec8e25342fa195646c9f89c18d38896ea01afe3c5ada7c"}}],"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-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-77","guid":"332c8ade-6612-9f56-a06b-d8d90b1a8750","name":"Command Injection","shortDescription":{"text":"Command Injection"},"helpUri":"https://cwe.mitre.org/data/definitions/77.html"},{"id":"CWE-78","guid":"2e31ceaf-c7ae-2e5e-9661-cfb1362789cf","name":"OS Command Injection","shortDescription":{"text":"OS Command Injection"},"helpUri":"https://cwe.mitre.org/data/definitions/78.html"},{"id":"CWE-79","guid":"fd45580b-e8c4-fc5e-8c2f-aa8fab0b4dbf","name":"Cross-site Scripting (XSS)","shortDescription":{"text":"Cross-site Scripting (XSS)"},"helpUri":"https://cwe.mitre.org/data/definitions/79.html"},{"id":"CWE-798","guid":"5e8f057d-fee3-995a-a0cb-9fc5b0d174d1","name":"Use of Hard-coded Credentials","shortDescription":{"text":"Use of Hard-coded Credentials"},"helpUri":"https://cwe.mitre.org/data/definitions/798.html"},{"id":"CWE-829","guid":"13c33925-97fb-5a5e-b40c-56d328b8a4d7","name":"CWE-829","shortDescription":{"text":"CWE-829"},"helpUri":"https://cwe.mitre.org/data/definitions/829.html"},{"id":"CWE-89","guid":"6d08fdad-37eb-c150-bbf0-d7d946863407","name":"SQL Injection","shortDescription":{"text":"SQL Injection"},"helpUri":"https://cwe.mitre.org/data/definitions/89.html"},{"id":"CWE-937","guid":"16f316ae-415c-b354-a59b-1f7905f756e9","name":"Using Components with Known Vulnerabilities","shortDescription":{"text":"Using Components with Known Vulnerabilities"},"helpUri":"https://cwe.mitre.org/data/definitions/937.html"},{"id":"CWE-94","guid":"75e7f50c-6c2f-dd52-bf40-bf6c52b861fd","name":"Code Injection","shortDescription":{"text":"Code Injection"},"helpUri":"https://cwe.mitre.org/data/definitions/94.html"}]}],"properties":{"codehealthPublication":{"public":true,"notice":"This is the PUBLIC form of this artifact. Findings are listed in full, but the details of SECURITY findings \u2014 which rule fired, in which file, on which line, and how to fix it \u2014 are deliberately withheld, and any secret-scanner results are excluded entirely. Where detail is absent here it was REMOVED FOR PUBLICATION; it is not missing from the analysis. The complete artifact is available from the repository owner.","securityFindingsRedacted":3,"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."]}]}