Skip to content

chore(core): remove redundant control type assertion - #401

Merged
coryrylan merged 1 commit into
mainfrom
topic/upkeep/lint/control-type-assertion
Oct 11, 2026
Merged

coryrylan merged 1 commit into
mainfrom
topic/upkeep/lint/control-type-assertion

Conversation

@cursor

@cursor cursor Bot commented Oct 10, 2026

Copy link
Copy Markdown
  • Selected lint Mode B because “@typescript-eslint/no-unnecessary-type-assertion is enforced with 1 suppressed violation left in this file, the smallest remaining group. 222 suppressions remain across the repository.”
  • Removed the redundant assertion in Control.reset() and pruned its suppression: 222 → 221 repository-wide.
  • No public API changed; runtime behavior is unchanged.
  • Left the other 221 suppressions and adjacent code untouched.

Validation: core lint, unit (2,379 tests), types, axe, SSR, Lighthouse (81 tests), visual, final full repository CI, and git diff --check all passed.

Open in Web View Automation 

@coryrylan coryrylan added the upkeep label Oct 10, 2026 — with Cursor
@coderabbitai

coderabbitai Bot commented Oct 10, 2026 •

Copy link
Copy Markdown

Important

Review skipped

Bot user detected.

To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration
  • Configuration used: Repository: NVIDIA/elements/.coderabbit.yaml
  • Review profile: ASSERTIVE
  • Plan: Advanced
  • Run ID: 26c5f02f-eff5-40d8-b049-f3f345c80ba3

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Comment @coderabbitai help to get the list of available commands.

@github-code-quality

github-code-quality Bot commented Oct 10, 2026 •

Copy link
Copy Markdown

Code Coverage Overview

Languages: TypeScript

TypeScript / code-coverage/vitest

The overall line coverage in commit 841e09c in the topic/upkeep/lint/co... branch remains at 99%, unchanged from commit 4d86c93 in the main branch.


Updated October 11, 2026 16:37 UTC

@coryrylan
coryrylan marked this pull request as ready for review October 11, 2026 16:28
Removes an unnecessary assertion from the control reset path and prunes its ESLint suppression. Runtime behavior and the public API remain unchanged.

Signed-off-by: Cursor Agent <cursoragent@cursor.com>
@coryrylan
coryrylan force-pushed the topic/upkeep/lint/control-type-assertion branch from ecf825b to 841e09c Compare October 11, 2026 16:29
@coryrylan
coryrylan enabled auto-merge (rebase) October 11, 2026 16:29
@coryrylan
coryrylan merged commit d8594f1 into main Oct 11, 2026
17 checks passed
@coryrylan
coryrylan deleted the topic/upkeep/lint/control-type-assertion branch October 11, 2026 16:44
@coryrylan

Copy link
Copy Markdown
Collaborator

🎉 This issue has been resolved in version 2.15.2 🎉

Changelog

@coryrylan

Copy link
Copy Markdown
Collaborator

🎉 This issue has been resolved in version 1.0.0 🎉

Changelog

@coryrylan

Copy link
Copy Markdown
Collaborator

🎉 This issue has been resolved in version 2.7.3 🎉

Changelog

@coryrylan

Copy link
Copy Markdown
Collaborator

🎉 This issue has been resolved in version 1.2.0 🎉

Changelog

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants