Skip to content

Migrate/monorepo - #1

Merged
Praashh merged 4 commits into
masterfrom
migrate/monorepo
Jun 18, 2026
Merged

Praashh merged 4 commits into
masterfrom
migrate/monorepo

Conversation

@Praashh

@Praashh Praashh commented Jun 16, 2026

Copy link
Copy Markdown
Owner

No description provided.

@vercel

vercel Bot commented Jun 16, 2026 •

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
typing-monster Ready Ready Preview, Comment Jun 18, 2026 5:21am

@github-actions

github-actions Bot commented Jun 16, 2026 •

Copy link
Copy Markdown

React Doctor found 6 new issues in 3 files · 6 warnings · score 72 / 100 (Needs work) · 0 fixed · vs master

6 warnings

src/Button.tsx

  • ⚠️ L34 Button missing explicit type button-has-type

src/components/Lobby.tsx

  • ⚠️ L111 Control missing accessible label control-has-associated-label
  • ⚠️ L120 Control missing accessible label control-has-associated-label

src/components/Passage.tsx

  • ⚠️ L72 Event logic handled in an effect no-event-handler
  • ⚠️ L75 Event logic handled in an effect no-event-handler
  • ⚠️ L84 Derived value copied into state no-derived-state

Reviewed by React Doctor for commit 576b4c8. See inline comments for fixes.

)}

<div className="w-full max-w-sm space-y-4">
<input

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

React Doctor · react-doctor/control-has-associated-label (warning)

Blind users can't tell what this control does because screen readers find no label, so add visible text, aria-label, or aria-labelledby.

Fix → Give every interactive control a label screen readers can read.

Docs

className="w-full bg-white/[0.04] border border-white/[0.08] rounded-xl px-4 py-3 text-txt font-mono text-sm outline-none focus:border-accent/50 transition-colors placeholder:text-txt-dim/50"
/>

<input

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

React Doctor · react-doctor/control-has-associated-label (warning)

Blind users can't tell what this control does because screen readers find no label, so add visible text, aria-label, or aria-labelledby.

Fix → Give every interactive control a label screen readers can read.

Docs

...props
}: ButtonProps) {
return (
<button

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

React Doctor · react-doctor/button-has-type (warning)

Your users can submit the form by accident because a <button> with no type defaults to submit.

Fix → Set an explicit button type so plain buttons do not submit forms by accident: type="button", "submit", or "reset".

Docs

@Praashh
Praashh merged commit db2f29f into master Jun 18, 2026
4 checks passed

This branch was successfully deployed

1 active deployment
Preview — 576b4c8e Deployed Jun 18, 2026 by vercel[bot]
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