fix: recognize more Unity internal packages

lib/cmd-add.js

@@ -134,7 +134,7 @@ const _add = async function({ pkg, testables, force }) {
       });
       // add depsValid to pkgsInScope.
       depsValid
-        .filter(x => !x.upstream && !x.module)
+        .filter(x => !x.upstream && !x.internal)
         .map(x => x.name)
         .forEach(name => pkgsInScope.push(name));
       // print suggestion for depsInvalid

lib/core.js

@@ -189,7 +189,7 @@ const fetchPackageInfo = async function(name, registry) {
       version,
       upstream,   // whether belongs to upstream registry
       self,       // whether is the source package
-      module,     // whether is an unity module
+      internal,     // whether is an internal package
       reason      // invalid reason of "version404", "package404"
     }, ...
   ]
@@ -214,14 +214,12 @@ const fetchPackageDependencies = async function({ name, version, deep }) {
       // create valid depedenency structure
       const depObj = {
         ...entry,
-        module:
-          /com.unity.modules/i.test(entry.name) ||
-          entry.name == "com.unity.ugui",
+        internal: isInternalPackage(entry.name),
         upstream: false,
         self: entry.name == name,
         reason: null
       };
-      if (!depObj.module) {
+      if (!depObj.internal) {
         // try fetching package info from cache
         let { pkgInfo, upstream } = _.get(cachedPacakgeInfoDict, entry.name, {
           pkgInfo: null,
@@ -287,9 +285,9 @@ const fetchPackageDependencies = async function({ name, version, deep }) {
       depsValid.push(depObj);
       log.verbose(
         "dependency",
-        `${entry.name}@${entry.version} ${depObj.module ? "[module] " : ""}${
-          depObj.upstream ? "[upstream]" : ""
-        }`
+        `${entry.name}@${entry.version} ${
+          depObj.internal ? "[internal] " : ""
+        }${depObj.upstream ? "[upstream]" : ""}`
       );
     }
   }
@@ -453,6 +451,18 @@ const parseEditorVersion = function(version) {
   return result;
 };
 
+// Detect if the given package name is an internal package
+const isInternalPackage = function(name) {
+  const internals = [
+    "com.unity.ugui",
+    "com.unity.2d.sprite",
+    "com.unity.2d.tilemap",
+    "com.unity.package-manager-ui",
+    "com.unity.ugui"
+  ];
+  return /com.unity.modules/i.test(name) || internals.includes(name);
+};
+
 module.exports = {
   cleanCache,
   compareEditorVersion,
@@ -463,6 +473,7 @@ module.exports = {
   getLatestVersion,
   getNpmFetchOptions,
   getUpmConfigDir,
+  isInternalPackage,
   loadManifest,
   loadUpmConfig,
   parseEnv,