Conversation
- Fixed STATE.md with accurate pylint score (4.87/10) - Fixed branch reference from 'c8ef07b' to 'f48afe7' - Created TODOs.md with prioritized task list - Added known issues section - Added quality metrics with actual values - Added duplicate code issues to technical debt Updates reflect: - Pylint score: 4.87 (not 10.0) - Missing 'crow' module in tests - Duplicate code in cells.py and shapes.py - Empty docstrings needing completion
…code - Added missing __init__.py files: scripts/, testconftest/, charts/, items/, skills/ - Fixed CLI import errors for cam and spreadsheet modules - Added subtrix>=0.0.1 to pyproject.toml dependencies - Refactored duplicate code in cells.py, shapes.py, json.py, yaml.py - Created data module stubs: csv.py, json.py, xml.py, yaml.py - Removed crow module from codebase and tests - Created TODOs.md and STATE.md documentation
- pyffice/items/__init__.py: Item base classes - pyffice/skills/__init__.py: Skill infrastructure - pyffice/media/media.py: Media handling base
solubrew
pushed a commit
that referenced
this pull request
Aug 10, 2026
Consolidated 1,283-line TODO into 304-line focused list. What was added: - Sprint 19-20 dedup achievements (12 commits, 5,000+ lines consolidated): 7 duplicate classes deleted, 4 new helpers, 1 new io_helpers module, ArchiveHandler base class. - 4 helper methods documented: _set_with_change, _add_to_collection, _del_from_collection, _del_from_dict. - Updated metrics: 0 nontrivial duplicate class methods, 0 pass-only bodies, 0 raise Exception, 0 TODO/FIXME markers, 0 empty/missing docstrings, 0 stubs, 0 broad except handlers. What was removed: - Stale Sprint 16/17/18 handoff blocks. - The 24-list 'pass-only CLI stubs' (T-NEW-049 closed). - The '72+ commands' claim (T-NEW-050 closed). - The '185 stub methods' (T-NEW-061 closed). - The 'broad except Exception handlers' (already closed). - The 'Raise Exception(...)' sites (T-NEW-055 closed). - The TODO/FIXME marker sweep (T-NEW-057 closed). What remains OPEN (with explicit migration plans): - T-NEW-044: 4 pre-existing import bugs in email/reports/ video/skills that crash CLI subpackage imports. - T-NEW-053: Re-audit contacts.py for # TODO stubs. - T-NEW-054: workflow/formulas.py protocol design. - T-NEW-058: Fill 13 missing test files. - T-NEW-059: __version__ docstring snippet fix. - T-NEW-060: squirl print() (out of scope, lives in squirl). Updated 4 User NEW TODOs to actual status: - #1 data/ refactor: PARTIAL (Mixmin alias done, full port migration deferred). - #2 consistent shape: CLOSED (additive _canonicalize). - #3 stubbed document types: PARTIAL (audit 0 stubs, but functional coverage audit still pending). - #4 e2e conversion: CLOSED (8/8 fixtures verified).
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.