Skip to content

Conversation

@coliff
Copy link
Member

@coliff coliff commented Jun 19, 2025

This pull request introduces a new auto-fix for the spec-char-escape rule, updates the extension's embedded HTMLHint version, and includes several documentation and configuration updates to enhance usability and debugging. Below are the most important changes grouped by theme:

New Feature: Auto-fix for spec-char-escape

  • Added the createSpecCharEscapeFix function to generate auto-fix actions for escaping special characters (<, >) in HTML content.
  • Integrated the new auto-fix into the createAutoFixes function to handle diagnostics for the spec-char-escape rule.

Documentation Updates

  • Updated the README.md to include spec-char-escape in the list of supported auto-fixes and updated the embedded HTMLHint version to 1.6.3. [1] [2]
  • Added a changelog entry for version 1.10.2, documenting the new auto-fix and the renamed output channel.

Debugging and Usability Improvements

  • Renamed the extension's output channel to "HTMLHint Extension" for better clarity in debugging logs.

Version and Configuration Updates

  • Bumped the extension version to 1.10.2 in package.json.
  • Updated the .htmlhintrc configuration to include attr-value-no-duplication and reordered rules for consistency.

Test Updates

  • Added a test case in test-autofixes.html to validate the new spec-char-escape auto-fix functionality.

Copy link

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request introduces an autofix feature for the spec-char-escape rule, focusing on < and > characters. The changes include the core autofix logic, updates to documentation, version bumps, and test enhancements. Key areas for potential improvement include simplifying a position calculation within the new autofix function. Overall, this is a valuable addition to the extension's capabilities.

@coliff coliff merged commit 4b237b4 into main Jun 19, 2025
10 checks passed
@coliff coliff deleted the dev/coliff/autofix-spec-char-escape branch June 19, 2025 02:59
@coliff coliff changed the title feat: Improve autofix feature (add spec-char-escape ) feat: Autofix spec-char-escape rule Jun 19, 2025
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.

2 participants