...
- There are passing tests for new functionality
- The code is internationalized - at a minimum all human readable text are in strings in the options block
- The public API is commented
- Core logic is available through the public API (hint: your event handlers often contain core logic)
- ARIA roles are used where appropriate
- All functionality is accessible via the keyboard
- Code is broken up into well named things
- All javascript files pass jsLint
- All html files pass the w3c validator
- All json files pass json lint
...