chore(audits): drop dead ESLint directives and redundant check:dead-code; fix the PPTX blend-group call they hid

[waleedlatif1]

Summary

• Remove all 89 eslint-disable / eslint-enable comments. Nothing runs ESLint (no eslint package or config in any workspace, every lint script is Biome, and next build 16.4 has no lint step), so they suppressed nothing and implied rules that don't exist. Where a directive carried a reason, the reason stays as a plain // why
• check:comment-hygiene now rejects ESLint directives, so they can't come back (two cases added to its existing test file; the flagging case fails against the old script)
• Fix a real bug the directives were hiding in the PPTX renderer: svg.appendChild(group)(path as any).__rectBlendGroup = undefined called the appended element as a function and threw whenever a rect gradient blend group existed. Split into the two statements it was meant to be
• Drop check:dead-code: check:unused-exports already runs knip with the same config and a superset of issue types, and check:audits already skipped the alias. The knip config stays
• Ignore the generated apps/docs/components/icons.tsx (a verbatim copy written by generate-docs.ts) in knip; its export surface is already ratcheted at the source file. The unused-exports baseline shrinks by 66 entries
• CLAUDE.md and the you-might-not-need-a-comment skill updated to match

Type of Change

• Bug fix
• Other: dead code and audit cleanup

Testing

• Ran check:comment-hygiene (passes; flags all 89 directives on staging), its test file (10/10), and check:unused-exports (passes with the shrunk baseline); biome check on every changed file
• Full lint, audits, type-check and test suites run in CI

Checklist

• Code follows project style guidelines
• Self-reviewed my changes
• Tests added/updated and passing
• No new warnings introduced
• I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment

Project	Deployment	Actions	Updated
docs	Skipped		Oct 10, 2026 2:21am UTC

[cubic-dev-ai]

All reported issues were addressed across 48 files

<sub>Reply with feedback, questions, or to request a fix.

Turn on auto-fix | Re-trigger cubic</sub>

[waleedlatif1]

@greptile

[waleedlatif1]

@cubic-dev-ai review this PR

[cubic-dev-ai]

@cubic-dev-ai review this PR

@waleedlatif1 I have started the AI code review. It will take a few minutes to complete.

[greptile-apps]

Confidence Score: 5/5

[Medium impact] The PR appears safe to merge, with the previous test-coverage concern addressed.

Summary

Removes unused ESLint directives, rejects new ones through check:comment-hygiene, and removes the redundant check:dead-code script. It also fixes the PPTX rectangular-gradient crash and excludes generated docs icons from unused-export checks.

• Rectangular gradient shapes add their blend group before the main path.
• The comment check rejects ESLint directives in source comments.
• The unused-code audit runs through one Knip check.
• Source code keeps its behavior without dead ESLint comments.

_{Reviews (3) · Last reviewed commit: "test(pptx): cover the rect gradient blen..." · Reviewed by Greptile}

[waleedlatif1]

@greptile

[waleedlatif1]

@cubic-dev-ai review this PR

[cubic-dev-ai]

@cubic-dev-ai review this PR

@waleedlatif1 I have started the AI code review. It will take a few minutes to complete.

[cubic-dev-ai]

No issues found across 49 files

Confidence score: 5/5

• Automated review surfaced no issues in the provided summaries.
• No files require special attention.

_{Turn on auto-fix | Re-trigger cubic}