Skip to content

chore: make code comments concise and preserve essential contracts - #5

Merged
fraa2a merged 1 commit into
mainfrom
chore/comment-cleanup
Oct 8, 2026
Merged

fraa2a merged 1 commit into
mainfrom
chore/comment-cleanup

Conversation

@fraa2a

@fraa2a fraa2a commented Oct 8, 2026

Copy link
Copy Markdown
Owner

Remove decorative section dividers, obsolete task/migration narratives, obvious descriptions and unsupported claims of correctness or safety. Keep short English comments for nonobvious ownership, callback/teardown constraints, units, timeline rules and format limits. Correct stale field annotations and synchronize checked-in Stream Deck JavaScript.

326 comment groups were changed in 65 source files. Including CSS and generated counterparts, the source-preservation check covers 69 files: all surviving source lines match the audit parent, with three named inline field annotations checked separately. License notices and tool directives are retained. No functional code changes are included.

Validation: both TypeScript checks and Vite builds pass with identical production asset hashes to the audit parent; plugin build/socket integration passes and checked-in JS matches compiler output; repository text/diff checks pass. The complete core CI on #4 is green; this branch also runs the workflows. Report evidence is updated in docs/audit/2026-10-08.md.

Stacked on #4, which is based on packaging PR #3. Review/merge order: #3, #4, then this PR. Full Windows build and clean-install runtime limits remain explicit in the audit report.

@fraa2a
fraa2a changed the base branch from fix/code-audit to main October 8, 2026 01:45
@fraa2a
fraa2a merged commit 3428527 into main Oct 8, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant