Skip to content

Fix static analysis findings - #125

Merged
YJack0000 merged 1 commit into
mainfrom
codex/fix-static-analysis-issues
Jul 8, 2026
Merged

Fix static analysis findings#125
YJack0000 merged 1 commit into
mainfrom
codex/fix-static-analysis-issues

Conversation

@YJack0000

Copy link
Copy Markdown
Contributor

Summary

  • harden the release workflow by moving workflow_dispatch inputs into environment variables before shell use
  • clean up shell scripts, React accessibility findings, readonly props, nested ternaries, array keys, regex/string handling, and smaller static-analysis issues
  • replace silent best-effort catches with contextual logging so failures show up in diagnostics without breaking user flows

Verification

  • npm run build
  • npm run test
  • git diff --check

@YJack0000
YJack0000 marked this pull request as ready for review July 8, 2026 14:23
@YJack0000
YJack0000 merged commit 0e5d88a into main Jul 8, 2026
@sonarqubecloud

sonarqubecloud Bot commented Jul 8, 2026

Copy link
Copy Markdown

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.

1 participant