Skip to content

chore(deps): update dependency @vue/tsconfig to v0.8.1#340

Merged
00o-sh merged 4 commits intomainfrom
claude/review-pr-333-01LcYP4uqDt5C3vWFPR34eaF
Dec 10, 2025
Merged

chore(deps): update dependency @vue/tsconfig to v0.8.1#340
00o-sh merged 4 commits intomainfrom
claude/review-pr-333-01LcYP4uqDt5C3vWFPR34eaF

Conversation

@00o-sh
Copy link
Copy Markdown

@00o-sh 00o-sh commented Dec 10, 2025

No description provided.

renovate bot and others added 3 commits December 10, 2025 04:50
This commit fixes all TypeScript errors introduced by upgrading @vue/tsconfig
from 0.4.0 to 0.8.1. The new version enables stricter type checking with
noUncheckedIndexedAccess, which requires proper null/undefined handling.

Key changes:
- Add noEmit: false to composite tsconfig files to fix build errors
- Add proper null/undefined checks for array/object access throughout codebase
- Fix potentially undefined values in VueUse hooks (keys.escape, keys.ctrl_k, etc.)
- Add null guards for array element access in utility functions
- Improve type safety in temperature converter, benchmark builder, and other tools

All 138 unit tests pass and the production build succeeds.
@vercel
Copy link
Copy Markdown

vercel bot commented Dec 10, 2025

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

Project Deployment Preview Comments Updated (UTC)
it-tools Ready Ready Preview Comment Dec 10, 2025 1:28pm

Fix linting issues found in CI:
- Convert randFromArray to function declaration (top-level functions require function keyword)
- Fix ISO8601 regex backreference issue by replacing \17 with :? and removing contradictory lookahead
- Preserve IPv4 regex empty alternative with eslint-disable comment (intentional for 0-9 matching)
- Add null check for temperature converter input value

All tests passing and linter satisfied.
@cloudflare-workers-and-pages
Copy link
Copy Markdown

cloudflare-workers-and-pages bot commented Dec 10, 2025

Deploying it-tools with  Cloudflare Pages  Cloudflare Pages

Latest commit: fc7d39e
Status: ✅  Deploy successful!
Preview URL: https://3ab6bd17.it-tools-bsv.pages.dev
Branch Preview URL: https://claude-review-pr-333-01lcyp4.it-tools-bsv.pages.dev

View logs

@00o-sh 00o-sh merged commit 02bf708 into main Dec 10, 2025
13 checks passed
@00o-sh 00o-sh deleted the claude/review-pr-333-01LcYP4uqDt5C3vWFPR34eaF branch December 10, 2025 13:32
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.

2 participants