# Changelog

## Score

- CAI 63 → 59 (-3.4)
- Rubric changed (rubric-2026.10.1 → rubric-2026.10.5) — scores are not directly comparable.

## Lenses

- Code Health 100 → 76 (-24.3)
- Architecture 69 → 69 (+0.0)
- Maturity 47 → 44 (-3.0)
- Readiness 71 → 71 (+0.0)
- Security 100 → 100 (+0.0)

## Resolved (4)

- Ambiguous distinction between `logger` and `default_logger`. It is unclear if `logger` returns the configured logger or the default, and how it differs from `default_logger`. This creates confusion for users trying to access the logging instance.
- Duplicate method definition for `find_or_create` with different parameter names. This suggests copy-paste errors or inconsistent refactoring within the same class.
- Redundant and conflicting job submission methods. `Job.async()` appears to be a legacy or instance-level wrapper, while `ClassMethods` exposes the standard `perform_async` and `perform_in` interface. Furthermore, `ClassMethods` defines `perform_in` twice with different signatures (one with `interval`, one with `_`), which is a clear duplication error.
- Workflow token permissions not restricted

## New (6)

- Ambiguous distinction between `logger` and `default_logger`. It is unclear if `logger` returns the configured logger or the default, and how `logger=` interacts with `default_logger`.
- Documentation: no architecture or design documentation (README.md)
- Duplicate method definition for `find_or_create` with different parameter names, indicating a copy-paste error or incomplete refactoring.
- Orphaned file (lib/sucker_punch/queue.rb)
- Redundant and conflicting job submission methods. `Job.async()` appears to be a legacy or instance-level wrapper, while `ClassMethods` exposes `perform_async` and `perform_in`. Furthermore, `perform_in` is defined twice with different signatures (`interval` vs `_`), suggesting an incomplete refactoring or copy-paste error.
- Workflow token permissions not restricted (.github/workflows/build.yml)