feat: support tsconfig paths #185
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This comment was marked as outdated.
This comment was marked as outdated.
Sorry, something went wrong.
There was a problem hiding this comment.
Pull Request Overview
This PR adds support for TypeScript config path resolution in Tailwind CSS configurations by introducing a tsconfig option to settings and rules. This enables the plugin to resolve path aliases defined in TypeScript configuration files when processing imports and module resolution.
- Adds
tsconfigconfiguration option to all rules and global settings - Implements TypeScript path resolution using the
tsconfig-paths-webpack-plugin - Refactors the async context system to support unified configuration and warning handling
Reviewed Changes
Copilot reviewed 62 out of 63 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| src/types/async.ts | Moves Warning interface to centralized async types |
| src/utils/options.ts | Adds tsconfig option to common options handling |
| src/async-utils/context.ts | Creates unified async context management system |
| src/async-utils/tsconfig.ts | Implements TypeScript config path resolution |
| src/async-utils/resolvers.ts | Updates resolvers to support TypeScript path mapping |
| src/tailwindcss/*.ts | Updates all Tailwind modules to use new async context system |
| src/rules/*.ts | Adds tsconfig option to all rule configurations |
| package.json | Adds tsconfig-paths-webpack-plugin dependency |
| docs/ | Updates documentation for new tsconfig option |
Comment on lines
9
to
+10
|
|
||
| runAsWorker(async ({ classes, configPath }: GetDissectedClassRequest) => { | ||
| const context = await createTailwindContext(configPath); | ||
| return getDissectedClasses(context, classes); | ||
| runAsWorker(async ({ classes, configPath, cwd, tsconfigPath }: GetDissectedClassRequest) => { |
There was a problem hiding this comment.
Missing return type annotation and response structure. The function should return GetDissectedClassResponse but only returns the raw dissected classes array.
Copilot uses AI. Check for mistakes.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
closes: #121
TODO:
tsconfigoption to settings and rules.