Skip to content

Modernize Phaser 4 React TypeScript template - #9

Open
scobo wants to merge 1 commit into
phaserjs:mainfrom
scobo:modernize-template
Open

Modernize Phaser 4 React TypeScript template#9
scobo wants to merge 1 commit into
phaserjs:mainfrom
scobo:modernize-template

Conversation

@scobo

@scoboscobo commented Mar 9, 2026

Copy link
Copy Markdown
  • Updates dependency versions.
  • Hardens .npmc configuration.
  • Converts all files to TypeScript.
  • Modernizes use of React 19 hooks.
  • Implements the React Compiler for automatic memoization.
  • Improves ESLint configuration and addresses all lint issues.
  • Adds Stylelint for CSS formatting and addresses all issues.
  • Creates # path alias for src to avoid relative path imports.
  • Replaces .editorconfig with Prettier for code formatting.
  • Uses CSS modules for typed class names with locally-scoped namespaces.
  • Adds Husky and lint-staged for automatic checking and fixing of formatting and lint issues.
  • Adds __DEV__ global for gating development-only code.
  • Updates log.ts to report the template type instead of the package name, as the README.md indicates it should.
  • Optimizes images.

Note

Medium Risk
Mostly tooling/configuration and documentation changes, but it alters the developer workflow (new Husky/lint-staged hooks, ESLint strict type-checked config, Stylelint/Prettier) and build scripts/logging, which can break local CI/builds if misconfigured.

Overview
Modernizes the template’s tooling and contributor workflow by replacing legacy configs (.eslintrc.cjs, .editorconfig) with ESLint flat config (eslint.config.ts), Prettier, and Stylelint, plus new Husky pre-commit/pre-push hooks and lint-staged-driven auto-fixing.

Updates project metadata/docs (README.md, index.html), tweaks repo hygiene (.gitignore, .npmrc, .nvmrc), and migrates the anonymous build logger from log.js to TypeScript (scripts/log.ts) with an updated reporting path.

Written by Cursor Bugbot for commit 479b9e7. This will update automatically on new commits. Configure here.

@cursorcursorBot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Bugbot Free Tier Details

Your team is on the Bugbot Free tier. On this plan, Bugbot will review limited PRs each billing cycle for each member of your team.

To receive Bugbot reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.

Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Comment threadeslint.config.ts Outdated
@cursor

cursorBot commented Mar 13, 2026

Copy link
Copy Markdown

You have used all of your free Bugbot PR reviews.

To receive reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.

Comment threadeslint.config.ts Outdated
Comment threadeslint.config.ts
Comment threadscripts/log.ts Outdated
Comment threadscripts/log.ts Outdated
Comment threadscripts/log.ts
Comment threadscripts/log.ts Outdated
Comment threadsrc/app/App.tsx
Comment threadsrc/app/App.tsx
Comment threadsrc/app/App.tsx
@scobo
scoboforce-pushed the modernize-template branch from 3bf2d57 to c23c0f5CompareJune 19, 2026 19:42
@scobo

Copy link
Copy Markdown
Author

Thanks for the review! I think I’ve addressed all of your feedback, but please let me know if I overlooked anything.

@scobo
scobo requested a review from shubhamkakkarJune 20, 2026 20:25
@scobo
scoboforce-pushed the modernize-template branch 2 times, most recently from c020912 to 2ebcf94CompareJuly 3, 2026 22:09
@scoboscobo changed the title Modernize React TypeScript templateModernize Phaser 4 React TypeScript templateJul 3, 2026
@scobo
scoboforce-pushed the modernize-template branch 2 times, most recently from af60b16 to 919283aCompareJuly 3, 2026 22:21
@cursor

cursorBot commented Jul 10, 2026

Copy link
Copy Markdown

Bugbot is not enabled for this team, so this pull request was not reviewed.

Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs.

Sign up for freeto 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

@scobo@shubhamkakkar