[fix] Remove the redundant boolean annotation that fails TypeScript lint - #5462
Conversation
…dget `no-inferrable-types` fails the TypeScript lint job on release/v0.105.9. Claude-Session: https://claude.ai/code/session_01LMcS17kn4KEdoWucKttuzD
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Context
The TypeScript lint job is red on
release/v0.105.9. It has been red since the commit that hid the context-window meter from the agent composer (535f6a2), so it is not caused by PR #5461. That commit wrote the feature flag asconst showContextBudget: boolean = false, and ESLint's@typescript-eslint/no-inferrable-typesrejects an annotation the initialiser already implies.Changes
The annotation is gone. The line is now
const showContextBudget = false, which keeps the flag exactly as it was for anyone flipping it back on.Tests / notes
pnpm lintinweb/passes, all 11 tasks.tsc --noEmitonweb/ossreports nothing for this file. The narrowerfalsetype does not upset the one place the flag is read.DriveExplorer.tsxandVirtualTileGrid.tsx. Those do not fail the job and are untouched here.