Skip to content

x/tools/gopls: Add refactor "move to a new file" #65707

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
golopot opened this issue Feb 14, 2024 · 5 comments
Closed

x/tools/gopls: Add refactor "move to a new file" #65707

golopot opened this issue Feb 14, 2024 · 5 comments
Assignees
@golopot
Labels
FeatureRequest Issues asking for a new feature that does not need a proposal. gopls Issues related to the Go language server, gopls. Refactoring Issues related to refactoring tools Tools This label describes issues relating to any tools in the x/tools repository.
Milestone
gopls/v0.17.0

Comments

@golopot
Copy link

golopot commented Feb 14, 2024 •
edited
Loading

gopls version

Build info ---------- golang.org/x/tools/gopls v0.14.2 golang.org/x/tools/[email protected] h1:sIw6vjZiuQ9S7s0auUUkHlWgsCkKZFWDHmrge8LYsnc= github.com/BurntSushi/[email protected] h1:9F2/+DoOYIOksmaJFPw1tGFy1eDnIJXg+UHjuD8lTak= github.com/google/[email protected] h1:O2Tfq5qg4qc4AmwVlvv0oLiVAGB7enBSJ2x2DqQFi38= github.com/sergi/[email protected] h1:we8PVUC3FE2uYfodKH/nBHMSetSfHDR6scGdBi+erh0= golang.org/x/exp/[email protected] h1:2O2DON6y3XMJiQRAS1UWU+54aec2uopH3x7MAiqGW6Y= golang.org/x/[email protected] h1:dGoOF9QVLYng8IHTm7BAyWqCqSheQ5pYWGhzW00YJr0= golang.org/x/[email protected] h1:zxkM55ReGkDlKSM+Fu41A+zmbZuaPVbGMzvvdUPznYQ= golang.org/x/[email protected] h1:Vz7Qs629MkJkGyHxUlRHizWJRG2j8fbQKjELVSNhy7Q= golang.org/x/[email protected] h1:brbkEFfGwNGAEkykUOcryE/JiHUMMJouzE0fWWmz/QU= golang.org/x/[email protected] h1:ablQoSUd0tRdKxZewP80B+BaqeKJuVhuRxj/dkrun3k= golang.org/x/[email protected] h1:Oku7E+OCrXHyst1dG1z10etCTxewCHXNFLRlyMPbh3w= golang.org/x/[email protected] h1:KUas02EjQK5LTuIx1OylBQdKKZ9jeugs+HiqO5HormU= honnef.co/go/[email protected] h1:YGD4H+SuIOOqsyoLOpZDWcieM28W47/zRO7f+9V3nvo= mvdan.cc/[email protected] h1:JVf4NN1mIpHogBj7ABpgOyZc65/UUOkKQFkoURsz4MM= mvdan.cc/xurls/[email protected] h1:tzxjVAj+wSBmDcF6zBB7/myTy3gX9xvi8Tyr28AuQgc= go: go1.21.6

go env

go env
GO111MODULE=''
GOARCH='arm64'
GOBIN=''
GOCACHE='/Users/chiawenchen/Library/Caches/go-build'
GOENV='/Users/chiawenchen/Library/Application Support/go/env'
GOEXE=''
GOEXPERIMENT=''
GOFLAGS=''
GOHOSTARCH='arm64'
GOHOSTOS='darwin'
GOINSECURE=''
GOMODCACHE='/Users/chiawenchen/go/pkg/mod'
GONOPROXY=''
GONOSUMDB=''
GOOS='darwin'
GOPATH='/Users/chiawenchen/go'
GOPRIVATE=''
GOPROXY='https://proxy.golang.org,direct'
GOROOT='/opt/homebrew/Cellar/go/1.21.6/libexec'
GOSUMDB='sum.golang.org'
GOTMPDIR=''
GOTOOLCHAIN='auto'
GOTOOLDIR='/opt/homebrew/Cellar/go/1.21.6/libexec/pkg/tool/darwin_arm64'
GOVCS=''
GOVERSION='go1.21.6'
GCCGO='gccgo'
AR='ar'
CC='cc'
CXX='c++'
CGO_ENABLED='1'
GOMOD='/Users/chiawenchen/testgopls/go.mod'
GOWORK=''
CGO_CFLAGS='-O2 -g'
CGO_CPPFLAGS=''
CGO_CXXFLAGS='-O2 -g'
CGO_FFLAGS='-O2 -g'
CGO_LDFLAGS='-O2 -g'
PKG_CONFIG='pkg-config'
GOGCCFLAGS='-fPIC -arch arm64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -ffile-prefix-map=/var/folders/ls/sjfzwn0x09g0_ckr_7m5ybpc0000gn/T/go-build2538955256=/tmp/go-build -gno-record-gcc-switches -fno-common'

What did you do?

.

What did you see happen?

.

What did you expect to see?

When selecting severals lines containing functions, types, etc, offer a refactor for moving these things to a new file in the same package. The exact changes this refactor does is:

  1. Create a new file in the the same package. The name of the new file will be the name of the first entity selected. If a file of the same name already exists, append underscores until it doesn't.
  2. Add line "package packagename" to the new file
  3. Add selected lines to new file
  4. Add needed imports for the new file
  5. Remove selected lines from the old file
  6. Remove redundant imports from the source file

Editor and settings

No response

Logs

No response
@golopot golopot added gopls Issues related to the Go language server, gopls. Tools This label describes issues relating to any tools in the x/tools repository. labels Feb 14, 2024
@gopherbot gopherbot added this to the Unreleased milestone Feb 14, 2024
@findleyr findleyr added Refactoring Issues related to refactoring tools FeatureRequest Issues asking for a new feature that does not need a proposal. labels Feb 14, 2024
@findleyr findleyr modified the milestones: Unreleased, gopls/backlog Feb 14, 2024
@findleyr
Copy link
Member

I agree this is a good and simple refactoring to apply. However we should fix AST comment handling (#20744) first -- something that we have on our radar for this year.

CC @adonovan @suzmue

@golopot
Copy link
Author

golopot commented Feb 15, 2024

However we should fix AST comment handling (#20744) first

Is this issue somewhat depended on AST comment handling changes, or is it just a priority issue? in that case I can do the implementation.

@adonovan
Copy link
Member

The feature is unlikely to work well until comment support is improved, but comment support will improve this year, so I don't think it should stop you from contributing this feature in the meantime, if that's something you're interested in doing.

golopot added a commit to golopot/tools that referenced this issue Feb 22, 2024
@golopot
gopls/internal: add code action "move to a new file"
2cfb2f0 
This code action moves selected code sections to a newly created file
within the same package. The created filename is chosen as the first
{function, type, const, var} name encountered. In addition, import
declarations are added or removed as needed.

Fixes golang/go#65707
@golopot golopot mentioned this issue Feb 22, 2024
Closed
golopot added a commit to golopot/tools that referenced this issue Feb 22, 2024
@golopot
gopls/internal: add code action "move to a new file"
d03849b 
This code action moves selected code sections to a newly created file
within the same package. The created filename is chosen as the first
{function, type, const, var} name encountered. In addition, import
declarations are added or removed as needed.

Fixes golang/go#65707
golopot added a commit to golopot/tools that referenced this issue Feb 22, 2024
@golopot
gopls/internal: add code action "move to a new file"
8c85d43 
This code action moves selected code sections to a newly created file
within the same package. The created filename is chosen as the first
{function, type, const, var} name encountered. In addition, import
declarations are added or removed as needed.

Fixes golang/go#65707
@gopherbot
Copy link
Contributor

Change https://go.dev/cl/565895 mentions this issue: gopls/internal: add code action "move to a new file"

golopot added a commit to golopot/tools that referenced this issue Feb 22, 2024
@golopot
gopls/internal: add code action "move to a new file"
46ec4ff 
This code action moves selected code sections to a newly created file
within the same package. The created filename is chosen as the first
{function, type, const, var} name encountered. In addition, import
declarations are added or removed as needed.

Fixes golang/go#65707
golopot added a commit to golopot/tools that referenced this issue Feb 23, 2024
@golopot
gopls/internal: add code action "extract declarations to new file"
da8e51f 
This code action moves selected code sections to a newly created file
within the same package. The created filename is chosen as the first
{function, type, const, var} name encountered. In addition, import
declarations are added or removed as needed.

Fixes golang/go#65707
golopot added a commit to golopot/tools that referenced this issue Feb 28, 2024
@golopot
gopls/internal: add code action "extract declarations to new file"
fa8abe3 
This code action moves selected code sections to a newly created file
within the same package. The created filename is chosen as the first
{function, type, const, var} name encountered. In addition, import
declarations are added or removed as needed.

Fixes golang/go#65707
golopot added a commit to golopot/tools that referenced this issue May 29, 2024
@golopot
gopls/internal: add code action "extract declarations to new file"
bea896f 
This code action moves selected code sections to a newly created file
within the same package. The created filename is chosen as the first
{function, type, const, var} name encountered. In addition, import
declarations are added or removed as needed.

Fixes golang/go#65707
golopot added a commit to golopot/tools that referenced this issue Jun 18, 2024
@golopot
gopls/internal: add code action "extract declarations to new file"
d2fd0f2 
This code action moves selected code sections to a newly created file
within the same package. The created filename is chosen as the first
{function, type, const, var} name encountered. In addition, import
declarations are added or removed as needed.

Fixes golang/go#65707
@findleyr findleyr modified the milestones: gopls/backlog, gopls/v0.17.0 Jun 18, 2024
golopot added a commit to golopot/tools that referenced this issue Jun 26, 2024
@golopot
gopls/internal: add code action "extract declarations to new file"
eb38a3e 
This code action moves selected code sections to a newly created file
within the same package. The created filename is chosen as the first
{function, type, const, var} name encountered. In addition, import
declarations are added or removed as needed.

Fixes golang/go#65707
@adonovan
Copy link
Member

Thanks @golopot for contributing this feature--and sorry for the review process took so long!

@gabyhelp gabyhelp mentioned this issue Aug 8, 2024
Closed
@gabyhelp gabyhelp mentioned this issue Sep 8, 2024
Closed
@gabyhelp gabyhelp mentioned this issue Sep 20, 2024
Closed
@gabyhelp gabyhelp mentioned this issue Oct 29, 2024
Open
This was referenced Nov 22, 2024
Closed
Closed
@gabyhelp gabyhelp mentioned this issue Jan 17, 2025
Closed
@gabyhelp gabyhelp mentioned this issue Mar 18, 2025
Closed
@gabyhelp gabyhelp mentioned this issue Apr 30, 2025
Open
@gabyhelp gabyhelp mentioned this issue May 9, 2025
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@golopot golopot

Labels
FeatureRequest Issues asking for a new feature that does not need a proposal. gopls Issues related to the Go language server, gopls. Refactoring Issues related to refactoring tools Tools This label describes issues relating to any tools in the x/tools repository.
Projects
None yet
Milestone
gopls/v0.17.0
Development

Successfully merging a pull request may close this issue.

gopls/internal: add code action "extract declarations to new file" golopot/tools
4 participants
@adonovan @gopherbot @golopot @findleyr