camc314
ff2a1d4937
fix(linter): move exhaustive-deps to react ( #7251 )
...
@Boshen lemme know if this is incorrect, but i noticed that `rules-of-hooks` lives inside `react` not `react-hooks`
1f2a6c666f/crates/oxc_linter/src/config/rules.rs (L291-L292)1f2a6c666f/crates/oxc_linter/src/options/filter.rs (L254)
2024-11-14 08:02:26 +00:00
Alexander S.
216d533b01
fix(language_server): revalidate files when oxlint config is changing ( #7259 )
...
---------
Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com>
2024-11-14 10:45:17 +08:00
overlookmotel
10cdce9b67
style(ast): add line break ( #7271 )
2024-11-13 15:45:44 +00:00
ottomated
897d3b1567
feat(ast): serialize StringLiterals to ESTree without raw ( #7263 )
...
#7211 alternative. Makes the `raw` field on estree literals optional.
2024-11-13 15:37:30 +00:00
overlookmotel
e84ea2c3c6
refactor(traverse)!: remove TraverseCtx::clone_identifier_reference ( #7266 )
...
Remove `TraverseCtx::clone_identifier_reference`.
This API encourages unnecessary repeated lookups of the `SymbolId` for a reference. It is preferable to use `MaybeBoundIdentifier` which only looks up the `SymbolId` once.
2024-11-13 15:12:35 +00:00
overlookmotel
8c754b1b43
feat(traverse): introduce MaybeBoundIdentifier ( #7265 )
...
`MaybeBoundIdentifier` is similar to `BoundIdentifier`, but can be used where the identifier may or may not have have a `SymbolId` (may or may not be bound).
Typical usage:
```rs
// Create `MaybeBoundIdentifier` from an existing `IdentifierReference`
let binding = MaybeBoundIdentifier::from_identifier_reference(ident, ctx);
// Generate `IdentifierReference`s and insert them into AST
assign_expr.left = binding.create_write_target(ctx);
assign_expr.right = binding.create_read_expression(ctx);
```
2024-11-13 15:12:33 +00:00
overlookmotel
de472ca7c0
refactor(ast): move StringLiteral definition higher up ( #7270 )
...
Pure refactor. `StringLiteral` definition was sandwiched in the middle of RegExp-related code. Move it higher up in `literal.rs`.
All the rest of the diff is just re-ordering generated code.
2024-11-13 13:37:37 +00:00
overlookmotel
834c94d9d2
docs(traverse): tidy doc comments for TraverseCtx::is_static ( #7267 )
...
Better describe what this method does in its doc comment, and tidy up wording a little.
2024-11-13 11:40:07 +00:00
overlookmotel
7a4872818e
refactor(traverse): reorder imports ( #7264 )
2024-11-13 11:02:37 +00:00
Orenbek
9c91151603
feat(linter): implement typescript/no-empty-object-type ( #6977 )
2024-11-13 09:31:16 +00:00
DonIsaac
2268a0ef90
feat(linter): support overrides config field ( #6974 )
...
Part of #5653
2024-11-13 05:40:59 +00:00
camchenry
c6a48684c3
refactor(linter): temporarily remove unknown rules checking ( #7260 )
...
I had intended to leave this in until we could rework some of the rule/plugin name resolution code, however this is causing issues with merging https://github.com/oxc-project/oxc/pull/6974 . The mutability of `self` is difficult to resolve in that PR without more drastic changes. Since this isn't currently helping us out, I am simply removing the code for now until we can revisit this.
2024-11-13 03:41:03 +00:00
Cameron A McHenry
df5c535dd0
fix(linter): revert unmatched rule error ( #7257 )
...
- related to https://github.com/oxc-project/oxc/issues/6988
This is causing a lot of errors currently such as
https://github.com/oxc-project/oxc/issues/7233 . I think we need to
handle rule names more robustly first:
- https://github.com/oxc-project/oxc/issues/7240
- https://github.com/oxc-project/oxc/issues/7082
- https://github.com/oxc-project/oxc/discussions/7242
then, I think we can revisit this and maybe implement it as an actual
lint plugin too? https://github.com/oxc-project/oxc/discussions/7086
2024-11-13 09:06:20 +08:00
camc314
c4ed230f8a
fix(linter): fix false positive in eslint/no-cond-assign ( #7241 )
...
fixes #7238
2024-11-12 14:45:38 +00:00
overlookmotel
740ba4bcbd
docs(ast): correct doc comment for StringLiteral ( #7255 )
...
Correct doc comment for `StringLiteral`'s `value` field. The doc comment was wrong.
2024-11-12 13:47:16 +00:00
dalaoshu
ef847dab9c
fix(linter): false positive in jsx-a11y/iframe-has-title ( #7253 )
...
closes #7243
2024-11-12 13:14:50 +00:00
overlookmotel
44375a5662
refactor(ast)!: rename TSEnumMemberName enum variants ( #7250 )
...
Variants of `TSEnumMemberName` were previously only prefixed with `Static` to avoid naming conflicts with the variants inherited from `Expression` for non-static member names. #7219 removed the variants inherited from `Expression`, so all variants are now static. So we can shorten the names again (back to what they were before `inherit_variants!` was introduced).
2024-11-12 12:22:21 +00:00
overlookmotel
d3d58f8ace
refactor(ast): remove inherit_variants! from TSEnumMemberName ( #7248 )
...
#7219 removed all variants of `TSEnumMemberName` except `IdentifierName` and `StringLiteral`. It no longer inherits variants from `Expression`, so we can remove the `inherit_variants!` macro wrapper.
The discriminants no longer need to avoid clashes with `Expression`'s, so they can start at 0.
2024-11-12 12:22:20 +00:00
camc314
62b6327879
fix(linter): react/exhaustive-deps update span for unknown deps diagnostic ( #7249 )
...
Relates to #7246
2024-11-12 11:55:50 +00:00
camc314
3dcac1ae0b
feat(linter): react/exhaustive-deps ( #7151 )
...
Firstly, a massive thanks to @alisnic for starting this (incredibly complicated) lint rule in https://github.com/oxc-project/oxc/pull/2637 !
still a draft. current state:
3x false positives (all todo with refs)
3x false negatives (TS will catch these
13x false negatvies todo with refs
1x false negative TODO
closes #2637
relates to #2174
2024-11-12 11:42:47 +00:00
overlookmotel
c5f4ee7a82
fix(transformer): correct code comments ( #7247 )
...
Follow-on after #7199 . Just correct typos in a couple of comments.
2024-11-12 11:09:22 +00:00
Boshen
20fe3cd6e7
chore(semantic): fix doc
2024-11-12 12:00:16 +08:00
Boshen
4740642926
chore(semantic): rename examples/simple.rs to examples/semantic.rs
2024-11-12 11:55:58 +08:00
Boshen
f54d330eca
feat(tasks/lint_rules): mark eslint/no-undef-init as done ( #7244 )
...
closes #6456
2024-11-12 11:55:37 +08:00
Boshen
1282221e36
fix(codegen): print comments when block is empty ( #7236 )
...
part of #7190
2024-11-11 11:20:25 +00:00
7086cmd
0d6a66aa9e
test(minifier): fix minimize condition tests ( #7222 )
2024-11-11 08:36:19 +00:00
Boshen
080a67be16
test(semantic): semantic checker store conformance pass/fail info in a separate file ( #7235 )
...
closes #6069
2024-11-11 08:13:13 +00:00
Boshen
2e0364e21a
chore(deps): update crate-ci/typos action to v1.27.3 ( #7228 )
...
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
2024-11-10 00:41:21 +08:00
oxc-bot
de107246c8
release(crates): v0.36.0 ( #7227 )
...
## [0.36.0] - 2024-11-09
- b11ed2c#7220 ) (Boshen)
- 0e4adc1#7219 ) (Boshen)
- 846711c#7213 ) (Boshen)
- 092de67#7212 ) (ottomated)
- d1d1874#7154 ) (Boshen)
- 843bce4#7126 ) (overlookmotel)
### Features
- cc8a191#7127 )
(overlookmotel)
- dc0215c#7149 ) (ottomated)
- 9d6cc9d#7152 )
(ottomated)
- b74686c#7204 ) (Dunqing)
- ad3a2f5#7176 )
(Boshen)
- b4258ee#7225 )
(Boshen)
- 324c3fe#7188 )
(Boshen)
- a166a4a#7210 ) (Boshen)
- 3a20b90#7193 ) (Boshen)
- 22898c8#7184 )
(Boshen)
- a579011#7182 ) (Boshen)
- 8573f79#7135 ) (Dunqing)
- df77241#7113 )
(Dunqing)
- b6a5750#7125 ) (Dunqing)
- 1910227#7201 ) (Dunqing)
- ffa8604#7138 ) (Dunqing)
- e536d47#7218 ) (Dunqing)
- 5cfdc05#7206 ) (Dunqing)
### Bug Fixes
- c82b273#7200 ) (Dunqing)
- b2a888d#7148 )
(Dunqing)
- 19892ed#7147 ) (Dunqing)
- ede10dc#7171 ) (Dunqing)
- 293d072#7199 ) (Dunqing)
- ae692d7#7117 ) (overlookmotel)
- eea4ab8#7226 ) (Dunqing)
### Refactor
- d27e14f#5546 )
(overlookmotel)
- fac5042#7130 ) (overlookmotel)
- a297765#7178 )
(7086cmd)
- fc86703#7221 )
(Boshen)
- c5485ae#7217 )
(camchenry)
- abf1602#7216 )
(camchenry)
- 42171eb#7215 )
(camchenry)
- de56083#7191 ) (Boshen)
- 0a43c64#7189 ) (Boshen)
- 0e1f12c#7162 ) (Boshen)
- a981caf#7161 )
(Boshen)
- 8340243#7143 )
(Boshen)
- 481f7e6#7142 )
(Boshen)
- 55e6989#7139 ) (Boshen)
- fdfd9a4#7128 )
(overlookmotel)
- ff8bd50#7107 ) (Dunqing)
- 4a515be#7203 ) (Dunqing)
- c307e1b#7169 ) (overlookmotel)
- 217d433#7165 ) (overlookmotel)
- 426df71#7164 )
(overlookmotel)
- 11c5e12#7163 )
(overlookmotel)
- 1238506#7167 ) (overlookmotel)
- 84ee581#7170 ) (overlookmotel)
- 1b12328#7172 ) (overlookmotel)
- cd1006f#7134 ) (Dunqing)
- 2c5734d#7132 ) (Dunqing)
- 5ce83bd#7131 ) (Dunqing)
- e04ee97#7106 ) (Dunqing)
- b57d5a5#7166 ) (overlookmotel)
- f80085c#7105 ) (Dunqing)
- e2241e6#7159 ) (overlookmotel)
- 1dfd241#7158 ) (overlookmotel)
- fd9b44c#7160 ) (overlookmotel)
- cacfb9b#7129 ) (overlookmotel)
### Styling
- 38a6df6#7168 ) (overlookmotel)
- 64b7e3a#7173 ) (overlookmotel)
### Testing
- be819dd
2024-11-10 00:31:14 +08:00
Boshen
be819dded1
test(napi/transform): add test for not default es transform
2024-11-10 00:21:26 +08:00
Dunqing
eea4ab830a
fix(transformer/helper-loader): incorrect SymbolFlags for default import when SourceType is script ( #7226 )
2024-11-09 14:32:44 +00:00
Boshen
b4258ee58e
feat(transformer): add defaulted Module::Preserve option ( #7225 )
2024-11-09 12:55:13 +00:00
oxc-bot
68dc899887
release(oxlint): v0.11.1 ( #7224 )
...
## [0.11.1] - 2024-11-09
### Features
- 4dd9b60#7196 ) (Nicholas Rayburn)
- 1fcd709#7120 )
(no-yan)
- 4d577cf#7180 ) (Dmitry Zakharov)
- 9b8973f#7187 ) (Dmitry
Zakharov)
- 5ab1ff6#6989 ) (Orenbek)
### Bug Fixes
- eea8879#7181 ) (Nicholas Rayburn)
- b73cfd9#7095 ) (tbashiyy)
### Refactor
- 8c0a362#7155 ) (Boshen)
- c5485ae#7217 )
(camchenry)
- abf1602#7216 )
(camchenry)
- 42171eb#7215 )
(camchenry)
Co-authored-by: Boshen <1430279+Boshen@users.noreply.github.com>
2024-11-09 19:57:08 +08:00
Boshen
c62e192a68
ci: fix oxlint-ecosystem-ci
2024-11-09 19:56:07 +08:00
7086cmd
a2977655c8
refactor(minifier): use map and and_then instead of let else ( #7178 )
...
For the test case, Closure Compiler doesn't handle this at all in the REPL! If it's necessary, I will turn it back.
This PR uses builtin `and_then` and `map` method, which is better instead of a lot of `if let Some`.
2024-11-09 11:48:48 +00:00
overlookmotel
d27e14f065
refactor(ast): AstKind::as_* methods take self ( #5546 )
...
Make `AstKind::as_*` methods (e.g. `AstKind::as_boolean_literal`) take `self`, not `&self`.
`AstKind` is `Copy`.
See: https://github.com/oxc-project/oxc/issues/5506#issuecomment-2332912976
2024-11-09 09:39:13 +00:00
Dunqing
5cfdc05d06
feat(transformer/typescript): support transform export = and import = require(...) when module is commonjs ( #7206 )
...
close : #7141
2024-11-09 09:32:28 +00:00
Boshen
fc86703933
refactor(napi/transform): change test files to TypeScript ( #7221 )
2024-11-09 17:32:15 +08:00
BitterGourd
94865a9390
feat(rulegen): support outdent and dedent tagged template expressions ( #6866 )
...
close #6645
---------
Co-authored-by: Don Isaac <donald.isaac@gmail.com>
2024-11-09 17:26:27 +08:00
Boshen
24c585a20a
feat(napi/parser): reexport @oxc-project/types ( #7214 )
...
closes #7207
2024-11-09 17:24:07 +08:00
no-yan
1fcd70932d
feat(linter): add jsx support for only-used-in-recursion ( #7120 )
...
close #5530
This PR adds support for handling recursive JSX components, allowing
detect functions like the one below:
```javascript
function ListItem({ depth }) {
return <ListItem depth={depth} />
}
```
Some logic is duplicated with the non-JSX case. But the refactor will be
complicated and affects over-all, so we can make it in another PR.
cc: @DonIsaac @camc314
---------
Co-authored-by: Cam McHenry <camchenry@users.noreply.github.com>
Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com>
2024-11-09 17:11:27 +08:00
Dmitry Zakharov
4d577cf69f
feat(linter): add import/first rule ( #7180 )
2024-11-09 17:10:59 +08:00
Boshen
b11ed2cf7b
refactor(ast)!: remove useless ObjectProperty::init field ( #7220 )
...
closes https://github.com/oxc-project/backlog/issues/143
2024-11-09 09:00:29 +00:00
Boshen
0e4adc15dd
feat(ast)!: remove invalid expressions from TSEnumMemberName ( #7219 )
2024-11-09 08:48:14 +00:00
Dunqing
e536d47e56
feat(transformer/babel): add support for trying to get the Module from BabelPlugins ( #7218 )
...
When configuring a `transform-modules-commonjs` plugin in `options.json`, we should treat it as a `CommonJS` module
2024-11-09 08:38:15 +00:00
Boshen
324c3fe184
feat(transformer): add TransformOptions::module option ( #7188 )
2024-11-09 08:38:15 +00:00
camchenry
c5485ae7ee
refactor(semantic): add ancestor_kinds iterator function ( #7217 )
...
for convenience, I've added a new function called `ancestor_kinds` which loops overall `ancestors` and gets their `AstKind`. this is a common pattern in a couple of places. I also did some somewhat related refactoring to remove places where we were manually calling `AstNode::kind` instead of using `ancestor_kinds` or calling `parent_kind`.
2024-11-09 08:04:38 +00:00
camchenry
abf1602f33
refactor(semantic): rename iter_parents to ancestors ( #7216 )
...
for consistency with the `ancestor_ids` function, this changes it to use the same terminology but make it clear it actually returns the node, not just the ID.
this is pure refactor, no functional changes here.
2024-11-09 07:07:26 +00:00
camchenry
42171eba94
refactor(semantic): rename ancestors to ancestor_ids ( #7215 )
...
pure refactor, renaming this method to make it more clear that it returns only IDs, not nodes. in preparation for the next PR.
2024-11-09 07:00:15 +00:00
Boshen
846711cf41
feat(transformer)!: change API to take a &TransformOptions instead of TransformOptions ( #7213 )
...
closes #7185
`TransformOption`s has an initialization cost, it should be initialized once and shared across files.
2024-11-09 06:01:13 +00:00
