loader: conditional_importations have only MModule now
[nit.git] / src / loader.nit
index 00d957a..56b7058 100644 (file)
@@ -73,8 +73,6 @@ redef class ModelBuilder
                for a in modules do
                        var nmodule = self.load_module(a)
                        if nmodule == null then continue # Skip error
-                       # Load imported module
-                       build_module_importation(nmodule)
                        var mmodule = nmodule.mmodule
                        if mmodule == null then continue # skip error
                        mmodules.add mmodule
@@ -92,52 +90,18 @@ redef class ModelBuilder
                return mmodules.to_a
        end
 
-       # Load recursively all modules of the group `mgroup`.
-       # See `parse` for details.
-       fun parse_group(mgroup: MGroup): Array[MModule]
-       do
-               var res = new Array[MModule]
-               scan_group(mgroup)
-               for mg in mgroup.in_nesting.smallers do
-                       for mp in mg.module_paths do
-                               var nmodule = self.load_module(mp.filepath)
-                               if nmodule == null then continue # Skip error
-                               # Load imported module
-                               build_module_importation(nmodule)
-                               var mmodule = nmodule.mmodule
-                               if mmodule == null then continue # Skip error
-                               res.add mmodule
-                       end
-               end
-               return res
-       end
-
-       # Load a bunch of modules and groups.
-       #
-       # Each name can be:
+       # Identify a bunch of modules and groups.
        #
-       # * a path to a module, a group or a directory of packages.
-       # * a short name of a module or a group that are looked in the `paths` (-I)
-       #
-       # Then, for each entry, if it is:
-       #
-       # * a module, then is it parser and returned.
-       # * a group then recursively all its modules are parsed.
-       # * a directory of packages then all the modules of all packages are parsed.
-       # * else an error is displayed.
-       #
-       # See `parse` for details.
-       fun parse_full(names: Sequence[String]): Array[MModule]
+       # This does the same as `parse_full` but does only the identification (cf. `identify_module`)
+       fun scan_full(names: Sequence[String]): Array[MModule]
        do
-               var time0 = get_time
-               # Parse and recursively load
-               self.toolcontext.info("*** PARSE ***", 1)
-               var mmodules = new ArraySet[MModule]
+               var mmodules = new Array[MModule]
                for a in names do
-                       # Case of a group
-                       var mgroup = self.get_mgroup(a)
+                       # Case of a group (root or sub-directory)
+                       var mgroup = self.identify_group(a)
                        if mgroup != null then
-                               mmodules.add_all parse_group(mgroup)
+                               scan_group(mgroup)
+                               for mg in mgroup.in_nesting.smallers do mmodules.add_all mg.mmodules
                                continue
                        end
 
@@ -149,18 +113,14 @@ redef class ModelBuilder
                                # Try each entry as a group or a module
                                for f in fs do
                                        var af = a/f
-                                       mgroup = get_mgroup(af)
+                                       mgroup = identify_group(af)
                                        if mgroup != null then
-                                               mmodules.add_all parse_group(mgroup)
+                                               scan_group(mgroup)
+                                               for mg in mgroup.in_nesting.smallers do mmodules.add_all mg.mmodules
                                                continue
                                        end
-                                       var mp = identify_file(af)
-                                       if mp != null then
-                                               var nmodule = self.load_module(af)
-                                               if nmodule == null then continue # Skip error
-                                               build_module_importation(nmodule)
-                                               var mmodule = nmodule.mmodule
-                                               if mmodule == null then continue # Skip error
+                                       var mmodule = identify_module(af)
+                                       if mmodule != null then
                                                mmodules.add mmodule
                                        else
                                                self.toolcontext.info("ignore file {af}", 2)
@@ -169,12 +129,41 @@ redef class ModelBuilder
                                continue
                        end
 
-                       var nmodule = self.load_module(a)
-                       if nmodule == null then continue # Skip error
-                       # Load imported module
-                       build_module_importation(nmodule)
-                       var mmodule = nmodule.mmodule
-                       if mmodule == null then continue # Skip error
+                       var mmodule = identify_module(a)
+                       if mmodule == null then
+                               continue
+                       end
+
+                       mmodules.add mmodule
+               end
+               return mmodules
+       end
+
+       # Load a bunch of modules and groups.
+       #
+       # Each name can be:
+       #
+       # * a path to a module, a group or a directory of packages.
+       # * a short name of a module or a group that are looked in the `paths` (-I)
+       #
+       # Then, for each entry, if it is:
+       #
+       # * a module, then is it parsed and returned.
+       # * a group then recursively all its modules are parsed.
+       # * a directory of packages then all the modules of all packages are parsed.
+       # * else an error is displayed.
+       #
+       # See `parse` for details.
+       fun parse_full(names: Sequence[String]): Array[MModule]
+       do
+               var time0 = get_time
+               # Parse and recursively load
+               self.toolcontext.info("*** PARSE ***", 1)
+               var mmodules = new ArraySet[MModule]
+               var scans = scan_full(names)
+               for mmodule in scans do
+                       var ast = mmodule.load(self)
+                       if ast == null then continue # Skip error
                        mmodules.add mmodule
                end
                var time1 = get_time
@@ -199,8 +188,8 @@ redef class ModelBuilder
        # Path can be added (or removed) by the client
        var paths = new Array[String]
 
-       # Like (and used by) `get_mmodule_by_name` but just return the ModulePath
-       fun search_mmodule_by_name(anode: nullable ANode, mgroup: nullable MGroup, name: String): nullable ModulePath
+       # Like (and used by) `get_mmodule_by_name` but does not force the parsing of the MModule (cf. `identify_module`)
+       fun search_mmodule_by_name(anode: nullable ANode, mgroup: nullable MGroup, name: String): nullable MModule
        do
                # First, look in groups
                var c = mgroup
@@ -208,7 +197,7 @@ redef class ModelBuilder
                        var r = c.mpackage.root
                        assert r != null
                        scan_group(r)
-                       var res = r.mmodule_paths_by_name(name)
+                       var res = r.mmodules_by_name(name)
                        if res.not_empty then return res.first
                end
 
@@ -247,42 +236,32 @@ redef class ModelBuilder
        # If no module exists or there is a name conflict, then an error on `anode` is displayed and null is returned.
        fun get_mmodule_by_name(anode: nullable ANode, mgroup: nullable MGroup, name: String): nullable MModule
        do
-               var path = search_mmodule_by_name(anode, mgroup, name)
-               if path == null then return null # Forward error
-               return load_module_path(path)
-       end
-
-       # Load and process importation of a given ModulePath.
-       #
-       # Basically chains `load_module` and `build_module_importation`.
-       fun load_module_path(path: ModulePath): nullable MModule
-       do
-               var res = self.load_module(path.filepath)
-               if res == null then return null # Forward error
-               # Load imported module
-               build_module_importation(res)
-               return res.mmodule
+               var mmodule = search_mmodule_by_name(anode, mgroup, name)
+               if mmodule == null then return null # Forward error
+               var ast = mmodule.load(self)
+               if ast == null then return null # Forward error
+               return mmodule
        end
 
        # Search a module `name` from path `lookpaths`.
-       # If found, the path of the file is returned
-       private fun search_module_in_paths(location: nullable Location, name: String, lookpaths: Collection[String]): nullable ModulePath
+       # If found, the module is returned.
+       private fun search_module_in_paths(location: nullable Location, name: String, lookpaths: Collection[String]): nullable MModule
        do
-               var res = new ArraySet[ModulePath]
+               var res = new ArraySet[MModule]
                for dirname in lookpaths do
                        # Try a single module file
-                       var mp = identify_file((dirname/"{name}.nit").simplify_path)
+                       var mp = identify_module((dirname/"{name}.nit").simplify_path)
                        if mp != null then res.add mp
                        # Try the default module of a group
-                       var g = get_mgroup((dirname/name).simplify_path)
+                       var g = identify_group((dirname/name).simplify_path)
                        if g != null then
                                scan_group(g)
-                               res.add_all g.mmodule_paths_by_name(name)
+                               res.add_all g.mmodules_by_name(name)
                        end
                end
                if res.is_empty then return null
                if res.length > 1 then
-                       toolcontext.error(location, "Error: conflicting module files for `{name}`: `{res.join(",")}`")
+                       toolcontext.error(location, "Error: conflicting module files for `{name}`: `{[for x in res do x.filepath or else x.full_name].join("`, `")}`")
                end
                return res.first
        end
@@ -293,7 +272,7 @@ redef class ModelBuilder
                var res = new ArraySet[MGroup]
                for dirname in lookpaths do
                        # try a single group directory
-                       var mg = get_mgroup(dirname/name)
+                       var mg = identify_group(dirname/name)
                        if mg != null then
                                res.add mg
                        end
@@ -301,26 +280,36 @@ redef class ModelBuilder
                return res
        end
 
-       # Cache for `identify_file` by realpath
-       private var identified_files_by_path = new HashMap[String, nullable ModulePath]
+       # Cache for `identify_module` by relative and real paths
+       private var identified_modules_by_path = new HashMap[String, nullable MModule]
 
        # All the currently identified modules.
-       # See `identify_file`.
-       var identified_files = new Array[ModulePath]
+       # See `identify_module`.
+       #
+       # An identified module exists in the model but might be not yet parsed (no AST), or not yet analysed (no importation).
+       var identified_modules = new Array[MModule]
+
+       # All the currently parsed modules.
+       #
+       # A parsed module exists in the model but might be not yet analysed (no importation).
+       var parsed_modules = new Array[MModule]
 
        # Identify a source file and load the associated package and groups if required.
        #
        # This method does what the user expects when giving an argument to a Nit tool.
        #
        # * If `path` is an existing Nit source file (with the `.nit` extension),
-       #   then the associated ModulePath is returned
+       #   then the associated MModule is returned
        # * If `path` is a directory (with a `/`),
-       #   then the ModulePath of its default module is returned (if any)
+       #   then the MModule of its default module is returned (if any)
        # * If `path` is a simple identifier (eg. `digraph`),
        #   then the main module of the package `digraph` is searched in `paths` and returned.
        #
        # Silently return `null` if `path` does not exists or cannot be identified.
-       fun identify_file(path: String): nullable ModulePath
+       #
+       # On success, it returns a module that is possibly not yet parsed (no AST), or not yet analysed (no importation).
+       # If the module was already identified, or loaded, it is returned.
+       fun identify_module(path: String): nullable MModule
        do
                # special case for not a nit file
                if not path.has_suffix(".nit") then
@@ -333,7 +322,7 @@ redef class ModelBuilder
                        # Found nothing? maybe it is a group...
                        var candidate = null
                        if path.file_exists then
-                               var mgroup = get_mgroup(path)
+                               var mgroup = identify_group(path)
                                if mgroup != null then
                                        var owner_path = mgroup.filepath.join_path(mgroup.name + ".nit")
                                        if owner_path.file_exists then candidate = owner_path
@@ -352,13 +341,15 @@ redef class ModelBuilder
                end
 
                # Fast track, the path is already known
-               var pn = path.basename(".nit")
+               if identified_modules_by_path.has_key(path) then return identified_modules_by_path[path]
                var rp = module_absolute_path(path)
-               if identified_files_by_path.has_key(rp) then return identified_files_by_path[rp]
+               if identified_modules_by_path.has_key(rp) then return identified_modules_by_path[rp]
+
+               var pn = path.basename(".nit")
 
                # Search for a group
                var mgrouppath = path.join_path("..").simplify_path
-               var mgroup = get_mgroup(mgrouppath)
+               var mgroup = identify_group(mgrouppath)
 
                if mgroup == null then
                        # singleton package
@@ -376,11 +367,14 @@ redef class ModelBuilder
                        end
                end
 
-               var res = new ModulePath(pn, path, mgroup)
-               mgroup.module_paths.add(res)
+               var src = new SourceFile.from_string(path, "")
+               var loc = new Location(src, 0, 0, 0, 0)
+               var res = new MModule(model, mgroup, pn, loc)
+               res.filepath = path
 
-               identified_files_by_path[rp] = res
-               identified_files.add(res)
+               identified_modules_by_path[rp] = res
+               identified_modules_by_path[path] = res
+               identified_modules.add(res)
                return res
        end
 
@@ -391,12 +385,17 @@ redef class ModelBuilder
        # If the directory is not a group null is returned.
        #
        # Note: `paths` is also used to look for mgroups
-       fun get_mgroup(dirpath: String): nullable MGroup
+       fun identify_group(dirpath: String): nullable MGroup
        do
-               if not dirpath.file_exists then do
+               var stat = dirpath.file_stat
+
+               if stat == null then do
+                       # search dirless directories in known -I paths
+                       if dirpath.chars.has('/') then return null
                        for p in paths do
                                var try = p / dirpath
-                               if try.file_exists then
+                               stat = try.file_stat
+                               if stat != null then
                                        dirpath = try
                                        break label
                                end
@@ -404,20 +403,19 @@ redef class ModelBuilder
                        return null
                end label
 
+               # Filter out non-directories
+               if not stat.is_dir then
+                       return null
+               end
+
+               # Fast track, the path is already known
                var rdp = module_absolute_path(dirpath)
                if mgroups.has_key(rdp) then
                        return mgroups[rdp]
                end
 
-               # Filter out non-directories
-               var stat = dirpath.file_stat
-               if stat == null or not stat.is_dir then
-                       mgroups[rdp] = null
-                       return null
-               end
-
                # By default, the name of the package or group is the base_name of the directory
-               var pn = rdp.basename(".nit")
+               var pn = rdp.basename
 
                # Check `package.ini` that indicate a package
                var ini = null
@@ -448,7 +446,7 @@ redef class ModelBuilder
                        var stopper = parentpath / "packages.ini"
                        if not stopper.file_exists then
                                # Recursively get the parent group
-                               parent = get_mgroup(parentpath)
+                               parent = identify_group(parentpath)
                                if parent == null then
                                        # Parent is not a group, thus we are not a group either
                                        mgroups[rdp] = null
@@ -523,12 +521,13 @@ redef class ModelBuilder
                if p == null then return
                for f in p.files do
                        var fp = p/f
-                       var g = get_mgroup(fp)
+                       var g = identify_group(fp)
                        # Recursively scan for groups of the same package
-                       if g != null and g.mpackage == mgroup.mpackage then
+                       if g == null then
+                               identify_module(fp)
+                       else if g.mpackage == mgroup.mpackage then
                                scan_group(g)
                        end
-                       identify_file(fp)
                end
        end
 
@@ -579,7 +578,7 @@ redef class ModelBuilder
                var keep = new Array[String]
                var res = new Array[String]
                for a in args do
-                       var l = identify_file(a)
+                       var l = identify_module(a)
                        if l == null then
                                keep.add a
                        else
@@ -595,13 +594,12 @@ redef class ModelBuilder
        # Display an error if there is a problem (IO / lexer / parser) and return null.
        # Note: usually, you do not need this method, use `get_mmodule_by_name` instead.
        #
-       # The MModule is created however, the importation is not performed,
-       # therefore you should call `build_module_importation`.
+       # The MModule is located, created, parsed and the importation is performed.
        fun load_module(filename: String): nullable AModule
        do
                # Look for the module
-               var file = identify_file(filename)
-               if file == null then
+               var mmodule = identify_module(filename)
+               if mmodule == null then
                        if filename.file_exists then
                                toolcontext.error(null, "Error: `{filename}` is not a Nit source file.")
                        else
@@ -610,25 +608,8 @@ redef class ModelBuilder
                        return null
                end
 
-               # Already known and loaded? then return it
-               var mmodule = file.mmodule
-               if mmodule != null then
-                       return mmodule2nmodule[mmodule]
-               end
-
-               # Load it manually
-               var nmodule = load_module_ast(file.filepath)
-               if nmodule == null then return null # forward error
-
-               # build the mmodule and load imported modules
-               mmodule = build_a_mmodule(file.mgroup, file.name, nmodule)
-
-               if mmodule == null then return null # forward error
-
-               # Update the file information
-               file.mmodule = mmodule
-
-               return nmodule
+               # Load it
+               return mmodule.load(self)
        end
 
        # Injection of a new module without source.
@@ -657,22 +638,25 @@ redef class ModelBuilder
        end
 
        # Visit the AST and create the `MModule` object
-       private fun build_a_mmodule(mgroup: nullable MGroup, mod_name: String, nmodule: AModule): nullable MModule
+       private fun build_a_mmodule(mgroup: nullable MGroup, nmodule: AModule)
        do
+               var mmodule = nmodule.mmodule
+               assert mmodule != null
+
                # Check the module name
                var decl = nmodule.n_moduledecl
                if decl != null then
                        var decl_name = decl.n_name.n_id.text
-                       if decl_name != mod_name then
-                               error(decl.n_name, "Error: module name mismatch; declared {decl_name} file named {mod_name}.")
+                       if decl_name != mmodule.name then
+                               error(decl.n_name, "Error: module name mismatch; declared {decl_name} file named {mmodule.name}.")
                        end
                end
 
                # Check for conflicting module names in the package
                if mgroup != null then
-                       var others = model.get_mmodules_by_name(mod_name)
+                       var others = model.get_mmodules_by_name(mmodule.name)
                        if others != null then for other in others do
-                               if other.mgroup!= null and other.mgroup.mpackage == mgroup.mpackage then
+                               if other != mmodule and mmodule2nmodule.has_key(mmodule) and other.mgroup!= null and other.mgroup.mpackage == mgroup.mpackage then
                                        var node: ANode
                                        if decl == null then node = nmodule else node = decl.n_name
                                        error(node, "Error: a module named `{other.full_name}` already exists at {other.location}.")
@@ -681,9 +665,6 @@ redef class ModelBuilder
                        end
                end
 
-               # Create the module
-               var mmodule = new MModule(model, mgroup, mod_name, nmodule.location)
-               nmodule.mmodule = mmodule
                nmodules.add(nmodule)
                self.mmodule2nmodule[mmodule] = nmodule
 
@@ -706,14 +687,12 @@ redef class ModelBuilder
                        # Is the module a test suite?
                        mmodule.is_test_suite = not decl.get_annotations("test_suite").is_empty
                end
-
-               return mmodule
        end
 
        # Resolve the module identification for a given `AModuleName`.
        #
        # This method handles qualified names as used in `AModuleName`.
-       fun seach_module_by_amodule_name(n_name: AModuleName, mgroup: nullable MGroup): nullable ModulePath
+       fun seach_module_by_amodule_name(n_name: AModuleName, mgroup: nullable MGroup): nullable MModule
        do
                var mod_name = n_name.n_id.text
 
@@ -733,12 +712,12 @@ redef class ModelBuilder
                        assert r != null
                        scan_group(r)
                        # Get all modules with the final name
-                       var res = r.mmodule_paths_by_name(mod_name)
+                       var res = r.mmodules_by_name(mod_name)
                        # Filter out the name that does not match the qualifiers
                        res = [for x in res do if match_amodulename(n_name, x) then x]
                        if res.not_empty then
                                if res.length > 1 then
-                                       error(n_name, "Error: conflicting module files for `{mod_name}`: `{res.join(",")}`")
+                                       error(n_name, "Error: conflicting module files for `{mod_name}`: `{[for x in res do x.filepath or else x.full_name].join("`, `")}`")
                                end
                                return res.first
                        end
@@ -753,16 +732,16 @@ redef class ModelBuilder
                        return null
                end
 
-               var res = new ArraySet[ModulePath]
+               var res = new ArraySet[MModule]
                for r in roots do
                        # Then, for each root, collect modules that matches the qualifiers
                        scan_group(r)
-                       var root_res = r.mmodule_paths_by_name(mod_name)
+                       var root_res = r.mmodules_by_name(mod_name)
                        for x in root_res do if match_amodulename(n_name, x) then res.add x
                end
                if res.not_empty then
                        if res.length > 1 then
-                               error(n_name, "Error: conflicting module files for `{mod_name}`: `{res.join(",")}`")
+                               error(n_name, "Error: conflicting module files for `{mod_name}`: `{[for x in res do x.filepath or else x.full_name].join("`, `")}`")
                        end
                        return res.first
                end
@@ -777,7 +756,7 @@ redef class ModelBuilder
        # but not `baz/foo.nit` nor `foo/bar.nit`
        #
        # Is used by `seach_module_by_amodule_name` to validate qualified names.
-       private fun match_amodulename(n_name: AModuleName, m: ModulePath): Bool
+       private fun match_amodulename(n_name: AModuleName, m: MModule): Bool
        do
                var g: nullable MGroup = m.mgroup
                for grp in n_name.n_path.reverse_iterator do
@@ -809,13 +788,15 @@ redef class ModelBuilder
                        end
 
                        # Load the imported module
-                       var suppath = seach_module_by_amodule_name(aimport.n_name, mmodule.mgroup)
-                       if suppath == null then
+                       var sup = seach_module_by_amodule_name(aimport.n_name, mmodule.mgroup)
+                       if sup == null then
+                               mmodule.is_broken = true
                                nmodule.mmodule = null # invalidate the module
                                continue # Skip error
                        end
-                       var sup = load_module_path(suppath)
-                       if sup == null then
+                       var ast = sup.load(self)
+                       if ast == null then
+                               mmodule.is_broken = true
                                nmodule.mmodule = null # invalidate the module
                                continue # Skip error
                        end
@@ -824,16 +805,20 @@ redef class ModelBuilder
                        imported_modules.add(sup)
                        var mvisibility = aimport.n_visibility.mvisibility
                        if mvisibility == protected_visibility then
+                               mmodule.is_broken = true
                                error(aimport.n_visibility, "Error: only properties can be protected.")
+                               mmodule.is_broken = true
                                nmodule.mmodule = null # invalidate the module
                                return
                        end
                        if sup == mmodule then
                                error(aimport.n_name, "Error: dependency loop in module {mmodule}.")
+                               mmodule.is_broken = true
                                nmodule.mmodule = null # invalidate the module
                        end
                        if sup.in_importation < mmodule then
                                error(aimport.n_name, "Error: dependency loop between modules {mmodule} and {sup}.")
+                               mmodule.is_broken = true
                                nmodule.mmodule = null # invalidate the module
                                return
                        end
@@ -843,6 +828,7 @@ redef class ModelBuilder
                        var mod_name = "core"
                        var sup = self.get_mmodule_by_name(nmodule, null, mod_name)
                        if sup == null then
+                               mmodule.is_broken = true
                                nmodule.mmodule = null # invalidate the module
                        else # Skip error
                                imported_modules.add(sup)
@@ -867,7 +853,7 @@ redef class ModelBuilder
                                end
 
                                # The rule
-                               var rule = new Array[Object]
+                               var rule = new Array[MModule]
 
                                # First element is the goal, thus
                                rule.add suppath
@@ -925,14 +911,11 @@ redef class ModelBuilder
        # It means that the first module is the module to automatically import.
        # The remaining modules are the conditions of the rule.
        #
-       # Each module is either represented by a MModule (if the module is already loaded)
-       # or by a ModulePath (if the module is not yet loaded).
-       #
        # Rules are declared by `build_module_importation` and are applied by `apply_conditional_importations`
        # (and `build_module_importation` that calls it).
        #
        # TODO (when the loader will be rewritten): use a better representation and move up rules in the model.
-       private var conditional_importations = new Array[SequenceRead[Object]]
+       private var conditional_importations = new Array[SequenceRead[MModule]]
 
        # Extends the current importations according to imported rules about conditional importation
        fun apply_conditional_importations(mmodule: MModule)
@@ -946,28 +929,16 @@ redef class ModelBuilder
                        for ci in conditional_importations do
                                # Check conditions
                                for i in [1..ci.length[ do
-                                       var rule_element = ci[i]
-                                       # An element of a rule is either a MModule or a ModulePath
-                                       # We need the mmodule to resonate on the importation
-                                       var m
-                                       if rule_element isa MModule then
-                                               m = rule_element
-                                       else if rule_element isa ModulePath then
-                                               m = rule_element.mmodule
-                                               # Is loaded?
-                                               if m == null then continue label
-                                       else
-                                               abort
-                                       end
+                                       var m = ci[i]
                                        # Is imported?
                                        if not mmodule.in_importation.greaters.has(m) then continue label
                                end
                                # Still here? It means that all conditions modules are loaded and imported
 
                                # Identify the module to automatically import
-                               var suppath = ci.first.as(ModulePath)
-                               var sup = load_module_path(suppath)
-                               if sup == null then continue
+                               var sup = ci.first
+                               var ast = sup.load(self)
+                               if ast == null then continue
 
                                # Do nothing if already imported
                                if mmodule.in_importation.greaters.has(sup) then continue label
@@ -1003,22 +974,50 @@ redef class ModelBuilder
        end
 end
 
-# File-system location of a module (file) that is identified but not always loaded.
-class ModulePath
-       # The name of the module
-       # (it's the basename of the filepath)
-       var name: String
+redef class MModule
+       # The path of the module source
+       var filepath: nullable String = null
 
-       # The human path of the module
-       var filepath: String
+       # Force the parsing of the module using `modelbuilder`.
+       #
+       # If the module was already parsed, the existing ASI is returned.
+       # Else the source file is loaded, and parsed and some
+       #
+       # The importation is not done by this
+       #
+       # REQUIRE: `filepath != null`
+       # ENSURE: `modelbuilder.parsed_modules.has(self)`
+       fun parse(modelbuilder: ModelBuilder): nullable AModule
+       do
+               # Already known and loaded? then return it
+               var nmodule = modelbuilder.mmodule2nmodule.get_or_null(self)
+               if nmodule != null then return nmodule
 
-       # The group (and the package) of the possible module
-       var mgroup: MGroup
+               var filepath = self.filepath
+               assert filepath != null
+               # Load it manually
+               nmodule = modelbuilder.load_module_ast(filepath)
+               if nmodule == null then return null # forward error
 
-       # The loaded module (if any)
-       var mmodule: nullable MModule = null
+               # build the mmodule
+               nmodule.mmodule = self
+               modelbuilder.build_a_mmodule(mgroup, nmodule)
+
+               modelbuilder.parsed_modules.add self
+               return nmodule
+       end
 
-       redef fun to_s do return filepath
+       # Parse and process importation of a given MModule.
+       #
+       # Basically chains `parse` and `build_module_importation`.
+       fun load(modelbuilder: ModelBuilder): nullable AModule
+       do
+               var nmodule = parse(modelbuilder)
+               if nmodule == null then return null
+
+               modelbuilder.build_module_importation(nmodule)
+               return nmodule
+       end
 end
 
 redef class MPackage
@@ -1031,9 +1030,6 @@ redef class MPackage
 end
 
 redef class MGroup
-       # Modules paths associated with the group
-       var module_paths = new Array[ModulePath]
-
        # Is the group interesting for a final user?
        #
        # Groups are mandatory in the model but for simple packages they are not
@@ -1046,8 +1042,7 @@ redef class MGroup
        # * it has a documentation
        fun is_interesting: Bool
        do
-               return module_paths.length > 1 or
-                       mmodules.length > 1 or
+               return mmodules.length > 1 or
                        not in_nesting.direct_smallers.is_empty or
                        mdoc != null or
                        (mmodules.length == 1 and default_mmodule == null)
@@ -1062,11 +1057,11 @@ redef class MGroup
        #
        # If `self` is not scanned (see `ModelBuilder::scan_group`) the
        # results might be partial.
-       fun mmodule_paths_by_name(name: String): Array[ModulePath]
+       fun mmodules_by_name(name: String): Array[MModule]
        do
-               var res = new Array[ModulePath]
+               var res = new Array[MModule]
                for g in in_nesting.smallers do
-                       for mp in g.module_paths do
+                       for mp in g.mmodules do
                                if mp.name == name then
                                        res.add mp
                                end
@@ -1088,7 +1083,8 @@ end
 
 redef class AModule
        # The associated MModule once build by a `ModelBuilder`
-       var mmodule: nullable MModule
+       var mmodule: nullable MModule = null
+
        # Flag that indicate if the importation is already completed
        var is_importation_done: Bool = false
 end