{"$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":"D6","name":"Cohesion (LCOM4)","shortDescription":{"text":"Cohesion (LCOM4)"},"helpUri":"https://codehealth.canine.dev/dimensions/D6"},{"id":"D8","name":"Code Coverage","shortDescription":{"text":"Code Coverage"},"helpUri":"https://codehealth.canine.dev/dimensions/D8"},{"id":"D9","name":"Test Distribution","shortDescription":{"text":"Test Distribution"},"helpUri":"https://codehealth.canine.dev/dimensions/D9"},{"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":"D15","name":"Churn \u00D7 Complexity Hotspots","shortDescription":{"text":"Churn \u00D7 Complexity Hotspots"},"helpUri":"https://codehealth.canine.dev/dimensions/D15"},{"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":"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":"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":"AX10","name":"Code composition","shortDescription":{"text":"Code composition"},"helpUri":"https://codehealth.canine.dev/dimensions/AX10"},{"id":"AX9","name":"CQS / query purity","shortDescription":{"text":"CQS / query purity"},"helpUri":"https://codehealth.canine.dev/dimensions/AX9"},{"id":"DM4","name":"Rich vs anemic domain model","shortDescription":{"text":"Rich vs anemic domain model"},"helpUri":"https://codehealth.canine.dev/dimensions/DM4"},{"id":"DM5","name":"Encapsulated state","shortDescription":{"text":"Encapsulated state"},"helpUri":"https://codehealth.canine.dev/dimensions/DM5"},{"id":"DM6","name":"Domain \u2194 infrastructure boundary","shortDescription":{"text":"Domain \u2194 infrastructure boundary"},"helpUri":"https://codehealth.canine.dev/dimensions/DM6"},{"id":"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":"P3","name":"Security \u0026 performance tooling","shortDescription":{"text":"Security \u0026 performance tooling"},"helpUri":"https://codehealth.canine.dev/dimensions/P3"},{"id":"X9","name":"Subsumed condition operand","shortDescription":{"text":"Subsumed condition operand"},"helpUri":"https://codehealth.canine.dev/dimensions/X9"}]}},"results":[{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (14 lines \u00D7 2): Movies/Presentation/Scenes/Main/Favorites/Coordinator/FavoritesCoordinator.swift:19-32 | Movies/Presentation/Scenes/Main/Home/Coordinator/HomeCoordinator.swift:19-32 \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 \u0060Movies/Presentation/Scenes/Main/Favorites/Coordinator/FavoritesCoordinator.swift:19\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":"Movies/Presentation/Scenes/Main/Favorites/Coordinator/FavoritesCoordinator.swift"},"region":{"startLine":19}}}],"partialFingerprints":{"codehealthFindingId/v1":"fd675b8cea9f9313efe6128f444bde619af0f851bff3cb6ac021c8305c6af187"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (6 lines \u00D7 2): Movies/Presentation/Scenes/Auth/Login/View/LoginView.swift:59-64 | Movies/Presentation/Scenes/Auth/PersonalInfoRegistration/View/PersonalInfoRegistrationView.swift:76-81 \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. Note first that the copies are not typed on the same thing: the declarations holding them bind \u0060viewModel\u0060 to \u0060LoginViewModel\u0060 in one and \u0060PersonalInfoRegistrationViewModel\u0060 in another, and the duplicated lines use it. The extracted unit therefore needs a parameter type that fits BOTH \u2014 their common supertype where they have one, or a new abstraction over them where they do not \u2014 and settling that is the step that comes BEFORE the extraction above. Where the two types are deliberately unrelated, 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":"Movies/Presentation/Scenes/Auth/Login/View/LoginView.swift"},"region":{"startLine":59}}}],"partialFingerprints":{"codehealthFindingId/v1":"e879ac8568a7410d0e6446e231168991c75c7919af82a17d316fc457bb4ee2ba"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (6 lines \u00D7 2): Movies/Presentation/Scenes/Auth/Login/View/LoginView.swift:66-71 | Movies/Presentation/Scenes/Auth/PasswordRegistration/View/PasswordRegistrationView.swift:59-64 \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. Note first that the copies are not typed on the same thing: the declarations holding them bind \u0060viewModel\u0060 to \u0060LoginViewModel\u0060 in one and \u0060PasswordRegistrationViewModel\u0060 in another, and the duplicated lines use it. The extracted unit therefore needs a parameter type that fits BOTH \u2014 their common supertype where they have one, or a new abstraction over them where they do not \u2014 and settling that is the step that comes BEFORE the extraction above. Where the two types are deliberately unrelated, 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":"Movies/Presentation/Scenes/Auth/Login/View/LoginView.swift"},"region":{"startLine":66}}}],"partialFingerprints":{"codehealthFindingId/v1":"e4799fdf40b2d15448efbe21c6a512a02a056918fe1706c0ad5bbaaa263459cc"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (6 lines \u00D7 2): Movies/Presentation/Scenes/Auth/PersonalInfoRegistration/View/PersonalInfoRegistrationView.swift:62-67 | Movies/Presentation/Scenes/Main/Profile/View/ProfileView.swift:127-132 \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":"Movies/Presentation/Scenes/Auth/PersonalInfoRegistration/View/PersonalInfoRegistrationView.swift"},"region":{"startLine":62}}}],"partialFingerprints":{"codehealthFindingId/v1":"b39e9c1aa14aa03f854356a468490ef5c73a7dfb23c4fe8864ccba46c4946392"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (6 lines \u00D7 2): Movies/Presentation/Scenes/Auth/PersonalInfoRegistration/View/PersonalInfoRegistrationView.swift:69-74 | Movies/Presentation/Scenes/Main/Profile/View/ProfileView.swift:134-139 \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":"Movies/Presentation/Scenes/Auth/PersonalInfoRegistration/View/PersonalInfoRegistrationView.swift"},"region":{"startLine":69}}}],"partialFingerprints":{"codehealthFindingId/v1":"7a2b19009f5a3962a8484902eddadd87a3b015f9b200b7b7a37a6a9beb8adb6e"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (6 lines \u00D7 2): Movies/Presentation/Scenes/Auth/PersonalInfoRegistration/View/PersonalInfoRegistrationView.swift:83-88 | Movies/Presentation/Scenes/Main/Profile/View/ProfileView.swift:113-118 \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":"Movies/Presentation/Scenes/Auth/PersonalInfoRegistration/View/PersonalInfoRegistrationView.swift"},"region":{"startLine":83}}}],"partialFingerprints":{"codehealthFindingId/v1":"d07208b31315f2d42ca95ac5015ebc5f043d84fead31d6f5af05182b525db18e"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (6 lines \u00D7 2): Movies/Presentation/Scenes/Auth/PersonalInfoRegistration/View/PersonalInfoRegistrationView.swift:90-95 | Movies/Presentation/Scenes/Main/Profile/View/ProfileView.swift:141-146 \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":"Movies/Presentation/Scenes/Auth/PersonalInfoRegistration/View/PersonalInfoRegistrationView.swift"},"region":{"startLine":90}}}],"partialFingerprints":{"codehealthFindingId/v1":"986e3049a2ffac99276afc6229bbc6dd879f99d9b31a98d2d87117a2f928458f"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (9 lines \u00D7 2): Movies/Presentation/Scenes/Auth/PersonalInfoRegistration/ViewModel/PersonalInfoRegistrationViewModel.swift:57-65 | Movies/Presentation/Scenes/Main/Profile/ViewModel/ProfileViewModel.swift:103-112 \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":"Movies/Presentation/Scenes/Auth/PersonalInfoRegistration/ViewModel/PersonalInfoRegistrationViewModel.swift"},"region":{"startLine":57}}}],"partialFingerprints":{"codehealthFindingId/v1":"38115701e54ae473b5e11071bf087578333e1bd00cb09fc43c1961fb140d3f31"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (8 lines \u00D7 2): Movies/Data/DataSources/Local/MovieLocalDataSource.swift:27-34 | Movies/Data/DataSources/Local/ProfileLocalDataSource.swift:30-37 \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":"Movies/Data/DataSources/Local/MovieLocalDataSource.swift"},"region":{"startLine":27}}}],"partialFingerprints":{"codehealthFindingId/v1":"0eedf549c851c586e1dd2cb9e7cf8c59535add11fc4edbb9c8e0cc12283f9173"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (8 lines \u00D7 3): Movies/Presentation/Scenes/Auth/Coordinator/AuthCoordinatorView.swift:20-27 | Movies/Presentation/Scenes/Main/Favorites/Coordinator/FavoritesCoordinatorView.swift:20-27 | Movies/Presentation/Scenes/Main/Home/Coordinator/HomeCoordinatorView.swift:20-27 \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. Note first that the copies are not typed on the same thing: the declarations holding them bind \u0060factory\u0060 to \u0060AuthCoordinatorFactory\u0060 in one and \u0060FavoritesCoordinatorFactory\u0060 in another, and the duplicated lines use it. The extracted unit therefore needs a parameter type that fits BOTH \u2014 their common supertype where they have one, or a new abstraction over them where they do not \u2014 and settling that is the step that comes BEFORE the extraction above. Where the two types are deliberately unrelated, 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":"Movies/Presentation/Scenes/Auth/Coordinator/AuthCoordinatorView.swift"},"region":{"startLine":20}}}],"partialFingerprints":{"codehealthFindingId/v1":"9a7775d2ca6a24a1af61e60ca86100c5d1acaff3375dc3fca6e03ee3a36327df"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (10 lines \u00D7 2): Movies/Presentation/Scenes/Main/Favorites/Coordinator/FavoritesCoordinatorView.swift:29-38 | Movies/Presentation/Scenes/Main/Home/Coordinator/HomeCoordinatorView.swift:29-38 \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":"Movies/Presentation/Scenes/Main/Favorites/Coordinator/FavoritesCoordinatorView.swift"},"region":{"startLine":29}}}],"partialFingerprints":{"codehealthFindingId/v1":"09c5d911b96fcc815f91b204fca5e383e26f9b3e2ba463263fc532cb8ecf98f3"}},{"ruleId":"D6","level":"warning","message":{"text":"Low cohesion: AppFactory (LCOM4 5): AppFactory\u0027s methods fall into 5 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 5 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":"Movies/Application/DependencyInjection/AppFactory.swift"},"region":{"startLine":10}}}],"partialFingerprints":{"codehealthFindingId/v1":"e92afe6495a1bd7fa6ff62e75462bd63f01beb314641c62afd1a5961d2140d86"}},{"ruleId":"D8","level":"error","message":{"text":"No automated tests: No automated tests \u2014 no test code was found in this repository. Untested code is the largest single risk to changing it safely. Start with the code you change most often: add a suite in a framework a runner can collect (XCTest or Swift Testing), and run it in CI so the gap cannot reopen."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"3132564c6310e5d01a231f252a02d5d2f5a542c0a8fa29a14c89693f1e3df871"}},{"ruleId":"D9","level":"note","message":{"text":"No tests found: No test suite could be collected \u2014 no discoverable tests to count. If this repository does test, wiring the suite to a framework a runner can collect (XCTest or Swift Testing) is what makes it countable here; a pipeline step that invokes a runner is not evidence on its own, because a runner over an empty suite passes. Tests written as plain executables or shell/PowerShell harnesses are not collectible this way and are not scored here."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"c9bf64cbb5a4ae13d6bcd01fa3bc8d2879d860c73bc67f176ee3c2cecb8adce3"}},{"ruleId":"D12","level":"warning","message":{"text":"Floating branch dependency: realm-swift: Dependency \u0060realm-swift\u0060 is resolved in Movies.xcodeproj/project.xcworkspace/xcshareddata/swiftpm/Package.resolved against branch \u0060master\u0060 rather than a released version \u2014 the project pins no immutable point, so resolving the packages again moves this dependency to whatever that branch holds at the time. Change the dependency rule in Xcode to a version requirement, or to an exact commit."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"3e6cfd16fd911e622bb963feea0d89aaef45132a7a7f505668d54fb9b450335e"}},{"ruleId":"D19","level":"note","message":{"text":"Documentation: no installation or build instructions: No setup/build/run commands for Swift projects on macOS/iOS. Add a short install/dependency line plus an open-project command and build/run steps."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"README.md"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"1fa6bfcd5c6ae231d1b8ee99bced50b832d1f61cde9da6cbb533467b879a57db"}},{"ruleId":"D19","level":"note","message":{"text":"Documentation: no usage examples: No runnable example showing how to launch the app or call its API endpoints. Write a one-line \u0027how to run it\u0027 instruction with a code snippet (or link) that builds and runs the app."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"README.md"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"1355eb4e127a4bc9236772ce1c46f09510aa9286e7a06d1e47f308229df19049"}},{"ruleId":"D20","level":"note","message":{"text":"No ADRs found: No ADRs found. No recognised ADR directory (\u0060docs/adr/\u0060, \u0060docs/decisions/\u0060, \u0060adr/\u0060, \u0060docs/rfcs/\u0060, an \u0060ADR0001/\u0060 folder, or their siblings) exists anywhere in this tree. What was searched, so you can tell an empty log from a search that missed one: every directory under the tree (build output, dependencies and VCS metadata excepted), for a document that is either any non-index page inside a recognised ADR directory, whatever its name and however deeply nested (\u0060docs/adr/use-postgres.md\u0060, \u0060docs/adr/2024/0001-x.md\u0060); or a file anywhere whose name is ADR-shaped (\u00600001-use-postgres.md\u0060, \u0060adr-012-caching.md\u0060); or, when neither turned anything up, a document carrying the decision-record signature (an \u0022Architecture Decision Record\u0022 heading, or Status / Context / Decision / Consequences as section headings). A decision log that clears none of these \u2014 unnumbered files outside any recognised directory, without those headings \u2014 is not seen by this check and this row is then wrong. If that is your case, say so rather than renaming anything; otherwise, consider recording architectural decisions in \u0060docs/adr/\u0060."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"d2bea044ff79d7d275f5a91a6e2f548586178eaf480274c33960ad020c854631"}},{"ruleId":"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":"2fbd04331a6b4cf8454fb1a9f91bc3e48e484db9a420493a1bc1f1d732457374"},"properties":{"commitSha":"1ce0f0f793aedef750e9fbe3e4c8492984bf2da6"}},{"ruleId":"D28","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":"3cdfd7beb31b791ae18dcb425e21eb157c842e7bbcd522cc330573c1a6538db1"}},{"ruleId":"D34","level":"note","message":{"text":"Dormant codebase: 23 of 25 significant files have no living knowledge \u2014 the codebase as a whole is dormant, not 23 separate risks. Counted over 25 of the 183 production source files in this repository: the rest are under the ~2,400-byte size floor this dimension measures over. Re-engage owners or document before change."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"4c69f1e54b7dd6fe9029dd02df6ba8f8145b789f2f18dbdaa086c40ded49dba6"}},{"ruleId":"D35","level":"warning","message":{"text":"Change coupling: ProfileView.swift \u2194 ProfileViewState.swift: \u0060Movies/Presentation/Scenes/Main/Profile/View/ProfileView.swift\u0060 and \u0060Movies/Presentation/Scenes/Main/Profile/ViewModel/ProfileViewState.swift\u0060 change together 77% of the time (10 of the 13 commits that touched whichever of the two files changed less often, counting a file under its earlier names as well \u2014 a repo-wide or module-wide sweep is evidence about the sweep rather than about any pair inside it and is left out of BOTH sides of this ratio, while a dependency bump, a formatter/rename sweep, or a commit whose edit to one of the two files was a tool directive such as //go:generate or whitespace only is left out of the shared count ONLY, so the two sides are not taken over identical commit sets). 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. You can check this without leaving the row: of the 10 shared commits counted here, the most recent 3 are \u0060a78162ce\u0060 Embed content in the ScrollView and fix save button state; \u006043c3fd97\u0060 Add loading indicator and disableable cancel button; \u00603ab7306e\u0060 Add avatar link validation \u2014 run \u0060git show\u0060 on any of them."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Movies/Presentation/Scenes/Main/Profile/View/ProfileView.swift"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"411eca9e0bf1828c8b407ba796702e546b9754e1c45264ac3410cbdfe3999019"}},{"ruleId":"D35","level":"warning","message":{"text":"Change coupling: LoginView.swift \u2194 PasswordRegistrationView.swift: \u0060Movies/Presentation/Scenes/Auth/Login/View/LoginView.swift\u0060 and \u0060Movies/Presentation/Scenes/Auth/PasswordRegistration/View/PasswordRegistrationView.swift\u0060 change together 64% of the time (7 of the 11 commits that touched whichever of the two files changed less often, counting a file under its earlier names as well \u2014 a repo-wide or module-wide sweep is evidence about the sweep rather than about any pair inside it and is left out of BOTH sides of this ratio, while a dependency bump, a formatter/rename sweep, or a commit whose edit to one of the two files was a tool directive such as //go:generate or whitespace only is left out of the shared count ONLY, so the two sides are not taken over identical commit sets). 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. You can check this without leaving the row: of the 7 shared commits counted here, the most recent 3 are \u0060d7e4aef1\u0060 (#18827) Updat the view with the view model - added view model to vie\u2026 (at that commit the files were still \u0060Movies/Presentation/Scenes/Auth/Modules/Login/View/LoginView.swift\u0060 and \u0060Movies/Presentation/Scenes/Auth/Modules/PasswordRegistration/View/PasswordRegistrationView.swift\u0060); \u00608f988baf\u0060 (#18827) Update name of events (at that commit the files were still \u0060Movies/Presentation/Scenes/Auth/Modules/Login/View/LoginView.swift\u0060 and \u0060Movies/Presentation/Scenes/Auth/Modules/PasswordRegistration/View/PasswordRegistrationView.swift\u0060); \u0060f089dae1\u0060 (#20583) Add user login - added error display login registration fail\u2026 (at that commit the files were still \u0060Movies/Presentation/Scenes/Auth/Modules/Login/View/LoginView.swift\u0060 and \u0060Movies/Presentation/Scenes/Auth/Modules/PasswordRegistration/View/PasswordRegistrationView.swift\u0060) \u2014 run \u0060git show\u0060 on any of them."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Movies/Presentation/Scenes/Auth/Login/View/LoginView.swift"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"c0b8157f5ae43df378b270c6565fbd410d06035935596acaf1a16e6ffd0a2428"}},{"ruleId":"D35","level":"warning","message":{"text":"Change coupling: ReviewView.swift \u2194 MovieDetailsViewModel.swift: \u0060Movies/Presentation/Scenes/Main/MovieDetails/View/Components/Review/ReviewView.swift\u0060 and \u0060Movies/Presentation/Scenes/Main/MovieDetails/ViewModel/MovieDetailsViewModel.swift\u0060 change together 60% of the time (6 of the 10 commits that touched whichever of the two files changed less often, counting a file under its earlier names as well \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 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. You can check this without leaving the row: of the 6 shared commits counted here, the most recent 3 are \u00606328bc8f\u0060 Add localization of custom errors; \u00604b8f5fe2\u0060 Add anonymous review display (at that commit the files were still \u0060Movies/Presentation/Scenes/Main/Common/MovieDetails/View/Components/Review/ReviewView.swift\u0060 and \u0060Movies/Presentation/Scenes/Main/Common/MovieDetails/ViewModel/MovieDetailsViewModel.swift\u0060); \u00605d6b57e3\u0060 Add mutation state for review dialog (at that commit the files were still \u0060Movies/Presentation/Scenes/Main/Common/MovieDetails/View/Components/Review/ReviewView.swift\u0060 and \u0060Movies/Presentation/Scenes/Main/Common/MovieDetails/ViewModel/MovieDetailsViewModel.swift\u0060) \u2014 run \u0060git show\u0060 on any of them."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Movies/Presentation/Scenes/Main/MovieDetails/View/Components/Review/ReviewView.swift"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"d21db76828c618bcf5654a3654c98284d42128b3bd803ffc3122c4927b378a80"}},{"ruleId":"D35","level":"warning","message":{"text":"Change coupling: ReviewViewModel.swift \u2194 MovieDetailsView.swift: \u0060Movies/Presentation/Scenes/Main/MovieDetails/View/Components/Review/ReviewViewModel.swift\u0060 and \u0060Movies/Presentation/Scenes/Main/MovieDetails/View/MovieDetailsView.swift\u0060 change together 60% of the time (6 of the 10 commits that touched whichever of the two files changed less often, counting a file under its earlier names as well \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 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. You can check this without leaving the row: of the 6 shared commits counted here, the most recent 3 are \u00606328bc8f\u0060 Add localization of custom errors; \u00600dc49f25\u0060 Add button to add review (at that commit the files were still \u0060Movies/Presentation/Scenes/Main/Common/MovieDetails/View/Components/Review/ReviewViewModel.swift\u0060 and \u0060Movies/Presentation/Scenes/Main/Common/MovieDetails/View/MovieDetailsView.swift\u0060); \u00605d6b57e3\u0060 Add mutation state for review dialog (at that commit the files were still \u0060Movies/Presentation/Scenes/Main/Common/MovieDetails/View/Components/Review/ReviewViewModel.swift\u0060 and \u0060Movies/Presentation/Scenes/Main/Common/MovieDetails/View/MovieDetailsView.swift\u0060) \u2014 run \u0060git show\u0060 on any of them."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Movies/Presentation/Scenes/Main/MovieDetails/View/Components/Review/ReviewViewModel.swift"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"ba3c5a2c8083a6a1843484685b3155fae15b78347fc514bdb5a585cb6ea36e94"}},{"ruleId":"D35","level":"warning","message":{"text":"Change coupling: FavoritesViewModel.swift \u2194 MovieDetailsViewModel.swift: \u0060Movies/Presentation/Scenes/Main/Favorites/ViewModel/FavoritesViewModel.swift\u0060 and \u0060Movies/Presentation/Scenes/Main/MovieDetails/ViewModel/MovieDetailsViewModel.swift\u0060 change together 50% of the time (5 of the 10 commits that touched whichever of the two files changed less often, counting a file under its earlier names as well \u2014 a repo-wide or module-wide sweep is evidence about the sweep rather than about any pair inside it and is left out of BOTH sides of this ratio, while a dependency bump, a formatter/rename sweep, or a commit whose edit to one of the two files was a tool directive such as //go:generate or whitespace only is left out of the shared count ONLY, so the two sides are not taken over identical commit sets). 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. You can check this without leaving the row: of the 5 shared commits counted here, the most recent 3 are \u006042651589\u0060 Refactor; \u00601ba1b018\u0060 Update movie caching and fix a bug with updating data after logging out (at that commit the file was still \u0060Movies/Presentation/Scenes/Main/Common/MovieDetails/ViewModel/MovieDetailsViewModel.swift\u0060); \u0060b10ad5c4\u0060 Add user rating update (at that commit the file was still \u0060Movies/Presentation/Scenes/Main/Common/MovieDetails/ViewModel/MovieDetailsViewModel.swift\u0060) \u2014 run \u0060git show\u0060 on any of them."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Movies/Presentation/Scenes/Main/Favorites/ViewModel/FavoritesViewModel.swift"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"6f56fe4c0cae6135c821ffb519ce205408a72e218e324f5ea5f706e3079693a4"}},{"ruleId":"D35","level":"warning","message":{"text":"Change coupling: FavoritesViewModel.swift \u2194 MovieDetailsView.swift: \u0060Movies/Presentation/Scenes/Main/Favorites/ViewModel/FavoritesViewModel.swift\u0060 and \u0060Movies/Presentation/Scenes/Main/MovieDetails/View/MovieDetailsView.swift\u0060 change together 50% of the time (5 of the 10 commits that touched whichever of the two files changed less often, counting a file under its earlier names as well \u2014 a repo-wide or module-wide sweep is evidence about the sweep rather than about any pair inside it and is left out of BOTH sides of this ratio, while a dependency bump, a formatter/rename sweep, or a commit whose edit to one of the two files was a tool directive such as //go:generate or whitespace only is left out of the shared count ONLY, so the two sides are not taken over identical commit sets). 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. You can check this without leaving the row: of the 5 shared commits counted here, the most recent 3 are \u006067c720c6\u0060 Update header (at that commit the file was still \u0060Movies/Presentation/Scenes/Main/Common/MovieDetails/View/MovieDetailsView.swift\u0060); \u00601ba1b018\u0060 Update movie caching and fix a bug with updating data after logging out (at that commit the file was still \u0060Movies/Presentation/Scenes/Main/Common/MovieDetails/View/MovieDetailsView.swift\u0060); \u0060b10ad5c4\u0060 Add user rating update (at that commit the file was still \u0060Movies/Presentation/Scenes/Main/Common/MovieDetails/View/MovieDetailsView.swift\u0060) \u2014 run \u0060git show\u0060 on any of them."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"Movies/Presentation/Scenes/Main/Favorites/ViewModel/FavoritesViewModel.swift"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"39862ba1127d60416e22c75c9405681535321ef0c7d4b53f71dd98849b6e730c"}},{"ruleId":"M2","level":"note","message":{"text":"No ADRs: No Architecture Decision Records found \u2014 no conventional ADR directory, no numbered \u0060NNNN-title\u0060 documents in any markup this check reads, and nothing ADR-shaped by content. Design rationale recorded elsewhere (a design-notes tree, a mailing list, pull-request discussion) is not visible to this check and is not re-findable per decision, so a future maintainer cannot ask why one choice was made and get an answer."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"670b3d6e36a756d63097d0dfbf90afd5fc761308800b9354894a07c3f4e4aa14"}},{"ruleId":"M2","level":"note","message":{"text":"No architecture diagram/doc: No C4/Structurizr/PlantUML/Mermaid/Graphviz/D2 diagram, no drawn diagram named for the architecture, no file named \u0060architecture\u0060 or \u0060design\u0060 in any markup this check reads, and nothing in the README, docs or contributor guides that announces the shape \u2014 no \u0060## Architecture\u0060 heading, no \u0022architecture overview\u0022/\u0022high-level design\u0022 phrasing, no \u0022the architecture is \u2026\u0022 introduction, no guided code tour. A shape laid out in prose that never names itself as the architecture is not visible to this check, and neither is one kept outside the repository, so this row reports the absence of a re-findable shape document \u2014 not evidence that nobody wrote the shape down."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"0c190e7c159d1850ee706c3ac4486e151e8a4fe169de4bf61436b9f399654f06"}},{"ruleId":"M3","level":"note","message":{"text":"No src/ separation: Production code isn\u0027t grouped under a src/ folder \u2014 it all sits under Movies/, so the conventional src/ boundary between the product and its tooling isn\u0027t drawn."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"fdf7f5b24e313364d10170a5c2542959902fe0d26ed70edef3eaa10ec5bcdb1d"}},{"ruleId":"M3","level":"note","message":{"text":"No tests/ separation: No test surface was found \u2014 this check walked the tree for authored source in the languages it models (\u0060.cs\u0060, \u0060.vb\u0060, \u0060.fs\u0060, \u0060.java\u0060, \u0060.kt\u0060, \u0060.scala\u0060, \u0060.py\u0060, \u0060.php\u0060, \u0060.rb\u0060, \u0060.ex\u0060, \u0060.exs\u0060, \u0060.go\u0060, \u0060.erl\u0060, \u0060.hrl\u0060, \u0060.swift\u0060, \u0060.dart\u0060, \u0060.rs\u0060, \u0060.ts\u0060, \u0060.tsx\u0060, \u0060.mts\u0060, \u0060.cts\u0060) and found none of it test-shaped. \u2605 \u0060.js\u0060, \u0060.jsx\u0060, \u0060.mjs\u0060 and \u0060.cjs\u0060 are NOT in that walk, so a Jest or Mocha suite written in plain JavaScript is invisible to it and this row is then wrong. If that is your case, say so rather than moving anything. Otherwise there are no tests here to separate from production code, so the folder question hasn\u0027t been reached yet."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"999e0c784909c76b6346a705ee63961fe363ed1cd6a94e3f80e2ebe7820ccd5d"}},{"ruleId":"M4","level":"note","message":{"text":"README/code drift: README omits the Movies project entirely (the directory tree shows only Images and a Movies.xcodeproj) \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":"343bbbd51ceb8de7a9ff88d79f7c797abf9b5921f8727b67b4d0f4f8a22c5e49"}},{"ruleId":"P1","level":"warning","message":{"text":"No CI pipeline: No CI workflow found (.github/workflows, azure-pipelines.yml, .gitlab-ci.yml, \u2026) \u2014 changes aren\u0027t gated by an automated build/test."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"44f01af96e50474fba2df84d95ddf4f7e1d34c29c307830b9efc9057daa6b670"}},{"ruleId":"P3","level":"note","message":{"text":"No SAST: No static application security testing detected. For this repository\u0027s stack, add CodeQL\u0027s Swift pack (Swift/Xcode) (or \u0060semgrep --config=auto\u0060, which runs on any language) \u2014 this repository has no CI pipeline yet, so run it locally to clear the existing findings, then make it a step of the first workflow you add so a regression fails the build. What was searched, so you can tell an absence from a miss: the 0 CI workflow file(s) in this repository, and the scanner and linter configuration checked in beside them. A scan that runs outside CI, one configured in your forge\u0027s web UI rather than in a committed file, or a tool whose name is none of those this check carries, is not seen \u2014 if that is your case the row is wrong, and saying so is more useful than adding a second scanner."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"6e54424179c892f03ef2fd003130ac4bd43f39bbf3b1ca0a0143e25acb80ec87"}}],"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-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-89","guid":"6d08fdad-37eb-c150-bbf0-d7d946863407","name":"SQL Injection","shortDescription":{"text":"SQL Injection"},"helpUri":"https://cwe.mitre.org/data/definitions/89.html"},{"id":"CWE-94","guid":"75e7f50c-6c2f-dd52-bf40-bf6c52b861fd","name":"Code Injection","shortDescription":{"text":"Code Injection"},"helpUri":"https://cwe.mitre.org/data/definitions/94.html"}]}],"properties":{"codehealthPublication":{"public":true,"notice":"This is the PUBLIC form of this artifact. Findings are listed in full, but the details of SECURITY findings \u2014 which rule fired, in which file, on which line, and how to fix it \u2014 are deliberately withheld, and any secret-scanner results are excluded entirely. Where detail is absent here it was REMOVED FOR PUBLICATION; it is not missing from the analysis. The complete artifact is available from the repository owner.","securityFindingsRedacted":2,"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."]}]}