# Changelog

## Score

- CAI 48 → 49 (+1.7)
- Rubric changed (rubric-2026.09.11 → rubric-2026.09.18) — scores are not directly comparable.

## Lenses

- Code Health 91 → 91 (+0.0)
- Architecture 72 → 72 (+0.0)
- Maturity 47 → 47 (+0.0)
- Readiness 35 → 35 (+0.0)
- Security 63 → 76 (+13.5)

## Resolved (10)

- Documentation: no installation or build instructions (docs/help/import_database.md)
- Documentation: written for insiders (docs/help/settings_other.md)
- Documentation: written for insiders (docs/help/settings_security.md)
- Inconsistent naming for what appears to be the same or related report functionality. One uses 'v2' suffix while the other does not, which is ambiguous without context.
- Inconsistent prefix 'm' in method names (ClickOnm...) and a typo 'Repor' instead of 'Report'. The 'm' prefix appears to be a typo or inconsistent naming convention for 'Insurance'.
- Inconsistent spelling of 'Origin' (PlaceOfOrgin vs PlaceofOrigin) and inconsistent spacing/capitalization in method names (AddPlaceofOrigin vs AddtoWishlist vs AddAmmunitiontomyCollection). 'Orgin' is a typo for 'Origin'. 'Addto' and 'tomy' lack proper spacing or capitalization compared to standard PascalCase conventions seen elsewhere.
- Namespace typo 'Ammuntion' vs correct spelling 'Ammunition' implied by context and other parts of the codebase (e.g., 'AmmunitionInventory').
- Typo in 'ColletorDetails' (ViewWindow) vs 'CollectorDetails' (AddWindow). The concept is the same (Collector Details), but the spelling differs.
- Typo in 'Controler' vs 'Control' (implied by 'ToolBar' and standard English). 'Controler' is misspelled.
- Typo in 'SendTExt' (capital T instead of lowercase t).

## New (6)

- Several report menu click methods have typos in their names. 'ClickOnmInsurance...' has an extra 'm' after 'On'. 'ClickOnmInsuranceReporWithTotaltAppraisedValue' has both 'Repor' (missing 't') and 'Totalt' (extra 't'). These are clearly the same concept (clicking a specific insurance report) but are named with distinct typos.
- The concept of 'Collector Details' is misspelled as 'ColletorDetails' in the ViewWindow helper, while correctly spelled as 'CollectorDetails' in the AddWindow helper. This is a direct inconsistency in spelling the same concept.
- The concept of 'Place of Origin' is spelled inconsistently. Some methods use 'PlaceOfOrgin' (missing the second 'i'), while one uses 'PlaceofOrigin' (lowercase 'of', correct spelling). This creates confusion between a typo and a different naming convention.
- The method 'SendTExt' contains a typo ('TExt' instead of 'Text'). While other helper methods like 'ClickOnElement' are spelled correctly, this specific method breaks the pattern and is likely a typo.
- The word 'Ammunition' is misspelled as 'Ammuntion' in the namespace `BSMyGunCollection.UnitTest.Command.Helpers.UI.Ammuntion` and as 'Ammunitiontomy' (merged words, missing space) in the method `ClickOnAddAmmunitiontomyCollection`. While the namespace is anchored by its folder structure, the method name shows a lack of consistency in handling the word 'Ammunition' and spacing.
- Two methods exist for clicking on a 'Bound Book' report, one with a 'v2' suffix and one without. This suggests versioning or duplication that is not clearly distinguished by a consistent naming convention (e.g., 'ClickOnBoundBookV2' vs 'ClickOnBoundBook'). The 'v2' is attached directly to the noun, which is inconsistent with standard versioning practices (usually separated or capitalized).