chore: Enforce Minimal Code Comments

[camielvs]

Description

Claude has been bloating our codebase with spammy code comments for far too long. This PR aims to reign in the AI and assert dominance!

Hopefully after this we will get much less long-winded and pointless code comments. Updated directives to the AI aim to remove the JSDoc standard and reduce self-explanatory code comments. The PR also does an initial high-level sweep and cleans up a few places as a proof of concept.

This should make our code more concise and readable in the long term.

Related Issue and Pull requests

Type of Change

• Improvement
• Cleanup/Refactor
• Documentation update

Checklist

• I have tested this does not break current pipelines / runs functionality
• I have tested the changes on staging

Screenshots (if applicable)

Test Instructions

Additional Comments

[github-actions]

🎩 Preview

A preview build has been created at: 08-13-chore_enforce_minimal_code_comments/8f4b718

[camielvs]

• chore: Enforce Minimal Code Comments #2630 👈 (View in Graphite)
• master

This stack of pull requests is managed by Graphite. Learn more about stacking.

[morgan-wowk]

🤖 Agent review. Verified no functional directive comments were removed anywhere (no eslint-disable/@ts-*/pragmas/etc.), the one doc value change (MIN_DOCKED_HEIGHT 100→50 in WINDOWS.md) is a correct fix vs the actual constant, and the branch passes pnpm run lint. One cross-PR concern (inline).

[camielvs]

^ inline concern has been addressed

[morgan-wowk]

🤖 Agent-assisted re-review (posted by Morgan). Approving. local/no-type-member-comments is now warn rather than error, which resolves the concern: in-flight branches that still carry an interface/object-type comment will surface a lint warning on rebase instead of a hard CI failure, so nobody gets a silent red build from a merge they didn't touch. Flipping to error later once the tree is clean is an easy follow-up. (The minor 'rename the field' message not quite fitting inline TSTypeLiteral cases is cosmetic — fine to leave.)

[camielvs]

Merge activity

• Aug 19, 12:08 AM UTC: A user started a stack merge that includes this pull request via Graphite.
• Aug 19, 12:09 AM UTC: Graphite rebased this pull request as part of a merge.
• Aug 19, 12:13 AM UTC: @camielvs merged this pull request with Graphite.