refactor(*): simplify and cleanup logics

[lumirlumir]

This pull request refactors the textlint-rule-allowed-uris module to simplify its structure, improve maintainability, and transition type definitions from JavaScript to TypeScript. The changes include consolidating imports, removing redundant code, and updating the implementation to use simpler data structures.

Refactoring and Simplification:

• src/textlint-rule-allowed-uris.js: Consolidated imports and replaced the use of getUriTypesLink and getUriTypesImage with inline objects for Link and Image nodes, simplifying the logic. Adjusted Definition and Html node handling to directly use arrays instead of UriTypes class. [1] [2]

• src/utils/get-uri-types/get-uri-types.js: Removed the UriTypes class and replaced it with plain arrays for URI type handling. Simplified getUriTypesDefinition and getUriTypesHtml functions to directly return arrays.

• src/utils/get-uri-types/get-uri-types.test.js: Updated tests to reflect the removal of the UriTypes class, ensuring direct comparison with arrays. Removed tests for getUriTypesLink and getUriTypesImage as they are no longer used. [1] [2] [3]

Transition to TypeScript:

• src/types/index.js: Removed JavaScript type definitions.

• src/utils/types.ts: Introduced TypeScript interfaces (UriType and Options) to define common types for the module, improving type safety and clarity.

File Structure Updates:

• src/utils/theme.js: Moved and renamed from src/utils/theme/theme.js, simplifying the directory structure. Updated imports accordingly. [1] [2]

• src/utils/uri-types/index.js: Removed redundant export for the deleted UriTypes class.

[github-actions]

Labels have been automatically applied based on the Conventional Commits specification.🏷️

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 98.34%. Comparing base (f36fbeb) to head (bf5ab98).
⚠️ Report is 1 commits behind head on main.

@@            Coverage Diff             @@
##             main     #303      +/-   ##
==========================================
- Coverage   98.90%   98.34%   -0.56%     
==========================================
  Files          10        7       -3     
  Lines         365      242     -123     
  Branches        1        1              
==========================================
- Hits          361      238     -123     
  Misses          4        4              

Files with missing lines	Coverage Δ
src/textlint-rule-allowed-uris.js	100.00% <100.00%> (ø)
...tion-node-uri-type/get-definition-node-uri-type.js	92.72% <100.00%> (ø)
src/utils/get-uri-types/get-uri-types.js	100.00% <100.00%> (ø)
src/utils/theme.js	100.00% <100.00%> (ø)

---

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f36fbeb...bf5ab98. Read the comment docs.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[Copilot]

Pull Request Overview

This PR refactors the textlint-rule-allowed-uris module to simplify its architecture by removing the UriTypes class and transitioning from JavaScript type definitions to TypeScript interfaces. The changes aim to improve maintainability and reduce complexity while preserving the same functionality.

Key changes include:

• Removal of the UriTypes class wrapper in favor of plain arrays
• Consolidation of utility functions and elimination of redundant exports
• Migration from JSDoc typedefs to TypeScript interfaces for better type safety

Reviewed Changes

Copilot reviewed 11 out of 12 changed files in this pull request and generated 1 comment.

Show a summary per file

File	Description
src/utils/uri-types/uri-types.js	Removed the UriTypes class entirely
src/utils/uri-types/uri-types.test.js	Removed all tests for the deleted UriTypes class
src/utils/uri-types/index.js	Removed export of deleted UriTypes class
src/utils/types.ts	Added TypeScript interfaces to replace JSDoc type definitions
src/utils/theme.js	Moved from nested directory and added whitespace formatting
src/utils/get-uri-types/get-uri-types.js	Simplified functions to return arrays instead of UriTypes instances
src/utils/get-uri-types/get-uri-types.test.js	Updated tests to work with arrays instead of UriTypes objects
src/textlint-rule-allowed-uris.js	Updated to use inline objects and direct array returns