Reverts commits from 65826db to 9cea4e7

Revert "local refactorings: looks for the innermost scope"

This reverts commit 9cea4e7.

Revert "return on error and report back the error message"

This reverts commit b8802f3.

Revert "simplify the refactoring logic"

This reverts commit df13448.

Revert "simpler returns"

This reverts commit 75e2592.

Revert "local refactor only on the current scope"

This reverts commit 486b619.

Revert "improve toplevel binding detection logic"

This reverts commit 517d645.

Revert "handle non-writable case"

This reverts commit df93c72.

Revert "improve parent module detection logic"

This reverts commit be7c0d3.

Revert "handle entrypath for Main module gracefully"

This reverts commit 39392af.

Revert "add goto uri 🎉"

This reverts commit 6f84942.

Revert "organize description"

This reverts commit d746383.

Revert "global refactor only from its definition"

This reverts commit e4d7ee4.

Revert "moar docs"

This reverts commit bf977e5.

Revert "create uri utilities"

This reverts commit 713312a.

Revert "even better messaging"

This reverts commit 4682c11.

Revert "jumpable global rename refactor information"

This reverts commit e618a16.

Revert "add more methods for get utilities"

This reverts commit 79bd617.

Revert "better messaging & suppress field renaming"

This reverts commit abbb84c.

Revert "global refactoring"

This reverts commit e2da219.

Revert "local refactoring"

This reverts commit 65826db.

Date:      Wed Oct 23 19:43:30 2019 +0900

Changes to be committed:
	modified:   src/Atom.jl
	modified:   src/completions.jl
	deleted:   src/docs.jl
	modified:   src/refactor.jl
	modified:   src/utils.jl
	modified:   test/goto.jl

Author: aviatesk

src/Atom.jl

@@ -1,6 +1,5 @@
 __precompile__()
 
-@doc read(joinpath(dirname(@__DIR__), "README.md"), String)
 module Atom
 
 using Juno, Lazy, JSON, MacroTools, Media, Base.StackTraces
@@ -52,7 +51,6 @@ include("outline.jl")
 include("completions.jl")
 include("goto.jl")
 include("datatip.jl")
-include("refactor.jl")
 include("misc.jl")
 include("formatter.jl")
 include("frontend.jl")

src/completions.jl

@@ -145,14 +145,19 @@ completionurl(c::REPLCompletions.ModuleCompletion) = begin
   mod, name = c.parent, c.mod
   val = getfield′(mod, name)
   if val isa Module # module info
-    urimoduleinfo(parentmodule(val) == val || val ∈ (Base, Core) ? name : "$mod.$name")
+    parentmodule(val) == val || val ∈ (Main, Base, Core) ?
+      "atom://julia-client/?moduleinfo=true&mod=$(name)" :
+      "atom://julia-client/?moduleinfo=true&mod=$(mod).$(name)"
   else
-    uridocs(mod, name)
+    "atom://julia-client/?docs=true&mod=$(mod)&word=$(name)"
   end
 end
-completionurl(c::REPLCompletions.MethodCompletion) = uridocs(c.method.module, c.method.name)
-completionurl(c::REPLCompletions.PackageCompletion) = urimoduleinfo(c.package)
-completionurl(c::REPLCompletions.KeywordCompletion) = uridocs("Main", c.keyword)
+completionurl(c::REPLCompletions.MethodCompletion) =
+  "atom://julia-client/?docs=true&mod=$(c.method.module)&word=$(c.method.name)"
+completionurl(c::REPLCompletions.PackageCompletion) =
+  "atom://julia-client/?moduleinfo=true&mod=$(c.package)"
+completionurl(c::REPLCompletions.KeywordCompletion) =
+  "atom://julia-client/?docs=true&mod=Main&word=$(c.keyword)"
 
 completionmodule(mod, c) = shortstr(mod)
 completionmodule(mod, c::REPLCompletions.ModuleCompletion) = shortstr(c.parent)

src/docs.jl

@@ -32,7 +32,7 @@ function renderitem(x)
 
   mod = getmodule(x.mod)
   name = Symbol(x.name)
-  r[:typ], r[:icon], r[:nativetype] = if (name !== :ans || mod === Base) && iskeyword(name)
+  r[:typ], r[:icon], r[:nativetype] = if (name !== :ans || mod === Base) && name ∈ keys(Docs.keywords)
     "keyword", "k", x.typ
   else
     val = getfield′(mod, name)