diff options
| author | Tommaso Negri <me@tommasongr.com> | 2026-10-14 21:58:57 +0200 |
|---|---|---|
| committer | Tommaso Negri <me@tommasongr.com> | 2026-10-14 21:58:57 +0200 |
| commit | f802039757d5da01a2b7cb7d1b15c692f137d91f (patch) | |
| tree | 86c328bda63b0d566eb56c8d0c83409fcc9b8b8c /tasks/20261014-192305/TASK.md | |
| parent | e6b4d0cb69809ea3d742ba03f4a0586cbbb3d1b3 (diff) | |
| download | bar-f802039757d5da01a2b7cb7d1b15c692f137d91f.tar.gz bar-f802039757d5da01a2b7cb7d1b15c692f137d91f.zip | |
add tasks
Diffstat (limited to 'tasks/20261014-192305/TASK.md')
| -rw-r--r-- | tasks/20261014-192305/TASK.md | 15 |
1 files changed, 15 insertions, 0 deletions
diff --git a/tasks/20261014-192305/TASK.md b/tasks/20261014-192305/TASK.md new file mode 100644 index 0000000..6442a5a --- /dev/null +++ b/tasks/20261014-192305/TASK.md @@ -0,0 +1,15 @@ +# Review and remove unused HEX conversion surface + +- STATUS: OPEN +- PRIORITY: 170 +- TAGS: review,cleanup,core + +File: Sources/bar/Models/HEX.swift; references across Sources, Tests, documentation, and patches. + +Observation: The core uses Color.hex(String), while the HEX wrapper, Color.init(hex:), and Color.Resolved.hex reverse conversion appear unnecessary for current runtime behavior. They add concepts a reader must understand in a deliberately small project. + +Approach: Search all references first, including documentation and published patch expectations. Decide whether these helpers are intentionally useful to users customizing Swift source. If they have a concrete use, keep them and document it briefly; otherwise remove the unused wrapper/reverse-conversion surface and update Config comments/DocC references to the remaining helper. Treat removal as an optional design decision, not a correctness requirement. Preserve direct .hex("#...") convenience. + +Completion checks: swift build and swift test pass; no dangling HEX symbol links or references remain. Supported hex syntax and Config appearance are unchanged. Explain the kept/removed API choice in the change description so it can be reviewed separately from strict input validation. + +Design constraint: Follow README.md: keep source-based configuration, no dependencies or plugin framework, and use the smallest readable change that makes the existing behavior reliable. This is a suggestion for individual review, not a deadline. Findings were recorded on 2026-10-14; re-check the current code before implementing. |
