Bugfix: more cases in types publishing infra

- Remove a needless runtime type check in one of the visitors.
- Remove `declare` from interface declarations.

Author: chriskrycho

types/publish.mjs

@@ -542,12 +542,19 @@ export function rewriteModule(code, moduleName) {
 
     // Remove `declare` from `declare function` in the top-level module.
     visitTSDeclareFunction(path) {
-      if (isTSDeclareFunction(path.node) && !hasParentModuleDeclarationBlock(path)) {
+      if (!hasParentModuleDeclarationBlock(path)) {
         path.node.declare = false;
       }
       this.traverse(path);
     },
 
+    visitTSInterfaceDeclaration(path) {
+      if (!hasParentModuleDeclarationBlock(path)) {
+        path.node.declare = false;
+      }
+      this.traverse(false);
+    },
+
     // Remove `declare` from `declare enum` in the top-level module.
     visitTSEnumDeclaration(path) {
       if (isTSEnumDeclaration(path.node) && !hasParentModuleDeclarationBlock(path)) {
@@ -584,6 +591,14 @@ export function rewriteModule(code, moduleName) {
       this.traverse(path);
     },
 
+    visitExportAllDeclaration(path) {
+      let specifier = path.node.source;
+      if (isStringLiteral(specifier)) {
+        specifier.value = normalizeSpecifier(moduleName, specifier.value);
+      }
+      this.traverse(path);
+    },
+
     // We need to rewrite annotations like `export const: import('./foo').foo`
     // to use relative paths, as well.
     visitTSImportType(path) {