Modernize to ES6+ and add a linter

[EliotGodard]

Problem

The codebase uses ES5 throughout (var, function expressions, manual prototype patterns). There is no linter or formatter, making it easy to introduce bugs.

Suggested improvements

• Convert to ES6+ (const/let, arrow functions, async/await, template literals).
• Add an ESLint config with a standard ruleset.
• Optionally add Prettier for consistent formatting.

[EliotGodard]

@claude Get to work BOY !

[claude]

Claude finished @EliotGodard's task in 5m 10s —— View job • claude/issue-7-20260130-1645 • Create PR ➔

---

Modernizing to ES6+ and adding a linter ✅

Todo List

• Gather context and understand codebase structure
• Check for CLAUDE.md and repo-specific guidelines
• Review package.json and project setup
• Add ESLint and Prettier as dev dependencies
• Create ESLint configuration
• Convert ES5 code to ES6+ syntax in index.js
• Convert ES5 code to ES6+ syntax in agxgame.js
• Convert ES5 code to ES6+ syntax in public/app.js
• Commit and push changes
• Provide PR creation link

Summary

Successfully modernized the entire codebase to ES6+ and added comprehensive linting!

Conversions completed:

• ✅ index.js - Converted to const/let, arrow functions
• ✅ agxgame.js - Converted to const/let, arrow functions, template literals
• ✅ public/app.js - Converted to const/let, arrow functions, template literals, method shorthand

Linter setup:

• ✅ Added ESLint with recommended config
• ✅ Added Prettier for code formatting
• ✅ Created .eslintrc.json with ES6+ rules (enforces no-var, prefer-const)
• ✅ Created .prettierrc.json with formatting rules
• ✅ Created .eslintignore to exclude minified libraries
• ✅ Added npm scripts: lint, lint:fix, and format

Key improvements:

• Replaced all var with const/let
• Converted function expressions to arrow functions
• Converted string concatenation to template literals
• Used method shorthand in object literals (init() instead of init: function())
• Replaced function() {} with arrow functions () => {}
• Used strict equality (===) instead of loose equality (==)
• Improved code consistency and maintainability

Next steps:
After installing dependencies with npm install, you can run:

• npm run lint - Check for linting issues
• npm run lint:fix - Automatically fix linting issues
• npm run format - Format all JS, JSON, and MD files

--- |