resyntax
resyntax copied to clipboard
jackfirth
A Racket refactoring engine
Resyntax should be able to remove unused local `define-syntax` bindings, like it can `define` bindings: ```scheme #lang resyntax/private/refactoring-test -------------------- #lang racket (define (foo) (define-syntax m (syntax-rules () [(m) (void)])) (+...
When libraries deprecate functions, those functions can often still be trivially expressed in terms of the library's other public APIs. Migrating users away from those functions is a trivial refactoring:...
This test should pass: ```scheme #lang resyntax/private/refactoring-test require: resyntax/default-recommendations let-binding-suggestions header: - #lang racket/base test: "shadowed bindings in parent definition contexts don't prevent let-to-define refactoring" -------------------- (define (f) (define x...
It's incredibly difficult to test refactoring rules without `#lang resyntax/testing/refactoring-test`. The testing language ought to be documented and exposed to users, so custom rules created by users can be tested....
This ought to be possible: ``` #lang resyntax/testing/refactoring-test header: - #lang racket/base ; See issue #12345, where this caused a bug in Resyntax because of x, y, and z. test:...
If Resyntax suggests replacing this code: ```scheme (define (foo) (define (bar) 42) ... hundreds of lines of code ...) ``` With this code: ```scheme (define (foo) ... hundreds of lines...
When test cases for complex rules fail, it can be tricky to tell why. Logging helps since the log messages show up in the test output. Going one step further,...
The `define-syntax-parser-to-define-syntax-parse-rule-simple` rule was incorrectly matching and stripping `syntax` wrappers from all body forms, including cases where the body used `syntax/loc`, had no wrapper, or used `quasisyntax`. **Changes:** - Match...
The Resyntax CLI had no test coverage for argument parsing or file refactoring functionality. ## Changes - **Added 21 test cases** in `cli.rkt` covering: - Argument parsing for `analyze` and...
A few tests that check that it parses arguments correctly and refactors some temp files would go a long way.