New rule: `Shared else-value for if/case`

オープン

#784 2026年3月21日に作成コメント 0 件リアクション 1 件担当者 0 名 GitHub で見る

まだ誰も着手していません。

評価

難易度: 3/5
見積もり時間: 1〜2日
初心者へのやさしさ: 55/100

issue の種類: 機能追加
明瞭さ: 明確に書かれている
活発さ: 停滞
領域: tooling

調査の方向性

まず、既存のリファクタリングルールの実装とそのテストスイートを探します。issue には Racket の肯定的なテストケースが示されており、観察箇所として webracket リポジトリの renderer.rkt が示されています。示された if/case 形式が case 形式に変換され、無関係なコードが変更されない状態になれば完了です。

索引モデルが issue の本文から書いたものです。

説明

new lint

Rule summary

From:

      (if (symbol? level)
          (case level
            [(warning)                                'warning]
            [(danger)                                 'danger]
            [(info success warning danger
                   primary secondary light dark)      level]
            [else                                     'info])
          'info)

To:

          (case level
            [(warning)                                'warning]
            [(danger)                                 'danger]
            [(info success warning danger
                   primary secondary light dark)      level]
            [else                                     'info])

The observation is that the value return by the else-arm of the if-expression,
is the same as the expression returned in the else-expression of the case.

Test case

#lang resyntax/test

test: "original code should be refactorable to new code"
--------------------
#lang racket
(define level 'foo)
(if (symbol? level)
    (case level
      [(warning) 'warning]
      [else      'info])
    'info)

====================
#lang racket
(define level 'foo)
(case level
      [(warning) 'warning]
      [else      'info])
--------------------

No-change test case

Additional context

Spotted here:

https://github.com/soegaard/webracket/blob/main/lib/web-easy/renderer.rkt#L892

主要言語: Racket
スター: 70
フォーク: 11
PR マージ指標: 30日以内にマージされた PR はありません

コントリビューションガイド

コントリビューションガイドを開く

はじめの一歩

issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
リポジトリをフォークし、ブランチを切って変更します。
issue 番号を参照したプルリクエストを送ります。

jackfirth/resyntax のほかの issue

Grimoire docs need examples オープン

documentation

難易度 2/5 1〜3時間初心者へのやさしさ 68/100

jackfirth/resyntax#803 · 2026年7月9日に作成
Docs previews don't work quite right オープン

documentation

難易度 4/5 3〜5日初心者へのやさしさ 45/100

jackfirth/resyntax#831 · 2026年7月15日に作成
UTS checker and renderer are completely untested オープン

testing

難易度 4/5 3〜5日初心者へのやさしさ 45/100

jackfirth/resyntax#822 · 2026年7月14日に作成
Meta-tests for resyntax/test failure reporting オープン

testing

難易度 3/5 1〜2日初心者へのやさしさ 75/100

jackfirth/resyntax#821 · 2026年7月14日に作成
Cover the CLI's run and output paths オープン

testing

難易度 4/5 3〜5日初心者へのやさしさ 68/100

jackfirth/resyntax#818 · 2026年7月14日に作成

jackfirth/resyntax の issue をすべて見る

似ている issue

[Bug] reef-hermes tells me to resume with hermes --resume, which does not work from my shell オープン

area: harness bug status: needs-triage

難易度 2/5 1〜3時間初心者へのやさしさ 75/100

Human-Agent-Society/reef#625 · 2026年9月24日に作成
ls -i / --inode not supported ("invalid option -- i"), breaks autoconf's `ls -di .` working-directory check オープン

難易度 2/5 1〜3時間初心者へのやさしさ 75/100

vercel-labs/just-bash#464 · 2026年9月24日に作成
`pnpm run version` will add an empty linebreak to changelogs for some reason オープン

難易度 2/5 1〜3時間初心者へのやさしさ 70/100

OpenFn/adaptors#1795 · 2026年9月24日に作成
audeer.path(): optionally expand environment variables オープン

難易度 2/5 1〜3時間初心者へのやさしさ 75/100

audeering/audeer#212 · コメント 1 件 · 2026年9月24日に作成
[FEATURE] opt-out for the Kitty keyboard protocol (env or config) — terminals that mishandle it corrupt input オープン

難易度 2/5 1〜3時間初心者へのやさしさ 75/100

anomalyco/opencode#51086 · 2026年9月24日に作成

DevTools の issue をもっと見る