gleam-lang/gleam

LSP code action: turn (wrong) C-style type definition into a single constructor type

Open

#3,479 opened on Aug 2, 2024

View on GitHub
 (3 comments) (1 reaction) (0 assignees)Rust (960 forks)batch import
help wanted

Repository metrics

Stars
 (21,417 stars)
PR merge metrics
 (Avg merge 10d 19h) (69 merged PRs in 30d)

Description

Now that the compiler can recognize that when the user is creating this type

pub type User {
  name: String
}

they actually need to define it like this

pub type User {
  User(
    name: String,
  )
}

it would make sense to offer a code action on hover to do that automatically.

Contributor guide

Research direction
Investigate the compiler's AST and type checking logic to detect C style type definitions, then add a code action in the LSP server.
Tech stack
rust
Domain
backenddevtools
Issue type
Feature
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
3
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
Half day
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
Active
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
Clear
Prerequisites
RustGit
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
40